fix: resolve Go duplicate references in tree-sitter queries (#5367)

- Replace broad statement captures with function-scoped queries
- Eliminates overlapping captures that caused duplicate references
- Improves search quality and indexing performance for Go projects
- Add test to validate no duplicate line ranges are captured
- Maintains backward compatibility with existing functionality

Fixes #5367

Author: MuriloFP

src/services/tree-sitter/__tests__/inspectGo.spec.ts

@@ -20,4 +20,49 @@ describe("Go Tree-sitter Parser", () => {
 		const result = await testParseSourceCodeDefinitions("file.go", sampleGoContent, testOptions)
 		expect(result).toBeDefined()
 	})
+
+	// Test 3: Verify no duplicate captures for Go constructs
+	it("should not create duplicate captures for Go constructs", async () => {
+		const testOptions = {
+			language: "go",
+			wasmFile: "tree-sitter-go.wasm",
+			queryString: goQuery,
+			extKey: "go",
+		}
+
+		const result = await testParseSourceCodeDefinitions("file.go", sampleGoContent, testOptions)
+
+		// Check that we have results
+		expect(result).toBeDefined()
+		expect(typeof result).toBe("string")
+		expect(result!.length).toBeGreaterThan(0)
+
+		// Parse the result to extract line ranges and function names
+		const lines = result!.split("\n").filter((line) => line.trim() && !line.startsWith("#"))
+
+		// Extract line ranges from the format "startLine--endLine | content"
+		const lineRanges = lines
+			.map((line) => {
+				const match = line.match(/^(\d+)--(\d+)/)
+				return match ? `${match[1]}-${match[2]}` : null
+			})
+			.filter(Boolean)
+
+		// Check for duplicate line ranges (which was the original problem)
+		const uniqueLineRanges = [...new Set(lineRanges)]
+		expect(lineRanges.length).toBe(uniqueLineRanges.length)
+
+		// Verify we capture function-based constructs instead of broad statements
+		// The result should contain function definitions that use Go constructs
+		expect(result).toContain("TestGoroutineDefinition")
+		expect(result).toContain("TestDeferDefinition")
+		expect(result).toContain("TestSelectDefinition")
+		expect(result).toContain("TestChannelDefinition")
+
+		// Should not contain raw go/defer/select statements without function context
+		// (our fix should prevent capturing these as standalone definitions)
+		expect(result).not.toMatch(/\|\s*go\s+func\(\)/) // raw go statement
+		expect(result).not.toMatch(/\|\s*defer\s+func\(\)/) // raw defer statement
+		expect(result).not.toMatch(/\|\s*select\s*{/) // raw select statement
+	})
 })

src/services/tree-sitter/queries/go.ts

@@ -44,15 +44,28 @@ export default `
 (method_declaration
   name: (field_identifier) @name.definition.method)
 
-; Channel operations
-(channel_type) @name.definition.channel
+; Functions containing goroutines (instead of capturing go statements)
+(function_declaration
+  name: (identifier) @name.definition.function
+  body: (block
+    (go_statement))) @definition.goroutine_function
 
-; Goroutine declarations
-(go_statement) @name.definition.goroutine
+; Functions containing defer statements
+(function_declaration
+  name: (identifier) @name.definition.function
+  body: (block
+    (defer_statement))) @definition.defer_function
 
-; Defer statements
-(defer_statement) @name.definition.defer
+; Functions containing select statements
+(function_declaration
+  name: (identifier) @name.definition.function
+  body: (block
+    (select_statement))) @definition.select_function
 
-; Select statements
-(select_statement) @name.definition.select
+; Functions with channel parameters or operations
+(function_declaration
+  name: (identifier) @name.definition.function
+  parameters: (parameter_list
+    (parameter_declaration
+      type: (channel_type)))) @definition.channel_function
 `