code_search: include line numbers. fix tests

jahooma · jahooma · commit 7e8031bcb14c · 2025-10-26T18:23:30.000-07:00
diff --git a/npm-app/src/tool-handlers.ts b/npm-app/src/tool-handlers.ts
@@ -281,7 +281,8 @@ export const handleCodeSearch: ToolHandler<'code_search'> = async (
       }
       searchCwd = requestedPath
     }
-    const args = [...flags, pattern, '.']
+    // Always include -n flag to ensure line numbers are in output for parsing
+    const args = ['-n', ...flags, pattern, '.']
 
     console.log()
     console.log(
diff --git a/sdk/src/tools/code-search.ts b/sdk/src/tools/code-search.ts
@@ -46,7 +46,8 @@ export function codeSearch({
       searchCwd = requestedPath
     }
 
-    const args = [...flagsArray, pattern, '.']
+    // Always include -n flag to ensure line numbers are in output for parsing
+    const args = ['-n', ...flagsArray, pattern, '.']
 
     const rgPath = getBundledRgPath(import.meta.url)
     const childProcess = spawn(rgPath, args, {

Original file line number	Diff line number	Diff line change
`@@ -281,7 +281,8 @@ export const handleCodeSearch: ToolHandler<'code_search'> = async (`
`281`	`281`	`}`
`282`	`282`	`searchCwd = requestedPath`
`283`	`283`	`}`
`284`		`- const args = [...flags, pattern, '.']`
	`284`	`+ // Always include -n flag to ensure line numbers are in output for parsing`
	`285`	`+ const args = ['-n', ...flags, pattern, '.']`
`285`	`286`
`286`	`287`	`console.log()`
`287`	`288`	`console.log(`
Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,8 @@ export function codeSearch({`
`46`	`46`	`searchCwd = requestedPath`
`47`	`47`	`}`
`48`	`48`
`49`		`- const args = [...flagsArray, pattern, '.']`
	`49`	`+ // Always include -n flag to ensure line numbers are in output for parsing`
	`50`	`+ const args = ['-n', ...flagsArray, pattern, '.']`
`50`	`51`
`51`	`52`	`const rgPath = getBundledRgPath(import.meta.url)`
`52`	`53`	`const childProcess = spawn(rgPath, args, {`