Fix list_code_definition_names to support files (#2046)

* feat: enhance list_code_definition_names to support files

Fix 'cwd option must be a path to a directory' error.
Add support for analyzing individual source files.
Update tool description to clarify file and directory usage.

Signed-off-by: Eric Wheeler <[email protected]>

* test: updated system instruction snapshots

Update system instruction snapshots for list_code_definition_names tool.
The snapshots now reflect enhanced documentation that clarifies
the tool can analyze both individual files and directories.

Signed-off-by: Eric Wheeler <[email protected]>

---------

Signed-off-by: Eric Wheeler <[email protected]>
Co-authored-by: Eric Wheeler <[email protected]>

Author: KJ7LNW

src/core/Cline.ts

@@ -2508,10 +2508,10 @@ export class Cline extends EventEmitter<ClineEvents> {
 						}
 					}
 					case "list_code_definition_names": {
-						const relDirPath: string | undefined = block.params.path
+						const relPath: string | undefined = block.params.path
 						const sharedMessageProps: ClineSayTool = {
 							tool: "listCodeDefinitionNames",
-							path: getReadablePath(this.cwd, removeClosingTag("path", relDirPath)),
+							path: getReadablePath(this.cwd, removeClosingTag("path", relPath)),
 						}
 						try {
 							if (block.partial) {
@@ -2522,19 +2522,35 @@ export class Cline extends EventEmitter<ClineEvents> {
 								await this.ask("tool", partialMessage, block.partial).catch(() => {})
 								break
 							} else {
-								if (!relDirPath) {
+								if (!relPath) {
 									this.consecutiveMistakeCount++
 									pushToolResult(
 										await this.sayAndCreateMissingParamError("list_code_definition_names", "path"),
 									)
 									break
 								}
 								this.consecutiveMistakeCount = 0
-								const absolutePath = path.resolve(this.cwd, relDirPath)
-								const result = await parseSourceCodeForDefinitionsTopLevel(
-									absolutePath,
-									this.rooIgnoreController,
-								)
+								const absolutePath = path.resolve(this.cwd, relPath)
+								let result: string
+								try {
+									const stats = await fs.stat(absolutePath)
+									if (stats.isFile()) {
+										const fileResult = await parseSourceCodeDefinitionsForFile(
+											absolutePath,
+											this.rooIgnoreController,
+										)
+										result = fileResult ?? "No source code definitions found in this file."
+									} else if (stats.isDirectory()) {
+										result = await parseSourceCodeForDefinitionsTopLevel(
+											absolutePath,
+											this.rooIgnoreController,
+										)
+									} else {
+										result = "The specified path is neither a file nor a directory."
+									}
+								} catch {
+									result = `${absolutePath}: does not exist or cannot be accessed.`
+								}
 								const completeMessage = JSON.stringify({
 									...sharedMessageProps,
 									content: result,