fix: path display concatenation and add startLine for file navigation

hannesrudolph · hannesrudolph · commit 5c7a77365e2b · 2026-01-26T17:11:59.000-07:00
- Fix formatPathTooltip to add space before additionalContent (fixes display bug where 'index.ts' + 'up to 50 lines' showed as 'index.tsup to 50 lines')
- Add startLine field to ClineSayTool type for read_file operations
- Update ReadFileTool to include startLine in the message
- Update ChatRow to pass startLine to openFile for navigation to the correct line
diff --git a/packages/types/src/vscode-extension-host.ts b/packages/types/src/vscode-extension-host.ts
@@ -801,6 +801,7 @@ export interface ClineSayTool {
 	isProtected?: boolean
 	additionalFileCount?: number // Number of additional files in the same read_file request
 	lineNumber?: number
+	startLine?: number // Starting line for read_file operations (for navigation on click)
 	query?: string
 	batchFiles?: Array<{
 		path: string
diff --git a/src/core/tools/ReadFileTool.ts b/src/core/tools/ReadFileTool.ts
@@ -461,12 +461,15 @@ export class ReadFileTool extends BaseTool<"read_file"> {
 			const isOutsideWorkspace = isPathOutsideWorkspace(fullPath)
 			const lineSnippet = this.getLineSnippet(fileResult.entry!)
 
+			const startLine = this.getStartLine(fileResult.entry!)
+
 			const completeMessage = JSON.stringify({
 				tool: "readFile",
 				path: getReadablePath(task.cwd, relPath),
 				isOutsideWorkspace,
 				content: fullPath,
 				reason: lineSnippet,
+				startLine,
 			} satisfies ClineSayTool)
 
 			const { response, text, images } = await task.ask("tool", completeMessage, false)
@@ -487,6 +490,17 @@ export class ReadFileTool extends BaseTool<"read_file"> {
 		}
 	}
 
+	/**
+	 * Get the starting line number for navigation purposes.
+	 */
+	private getStartLine(entry: InternalFileEntry): number | undefined {
+		if (entry.mode === "indentation" && entry.anchor_line !== undefined) {
+			return entry.anchor_line
+		}
+		const offset = entry.offset ?? 1
+		return offset > 1 ? offset : undefined
+	}
+
 	/**
 	 * Generate a human-readable line snippet for approval messages.
 	 */
diff --git a/webview-ui/src/components/chat/ChatRow.tsx b/webview-ui/src/components/chat/ChatRow.tsx
@@ -649,7 +649,13 @@ export const ChatRowContent = ({
 							<ToolUseBlock>
 								<ToolUseBlockHeader
 									className="group"
-									onClick={() => vscode.postMessage({ type: "openFile", text: tool.content })}>
+									onClick={() =>
+										vscode.postMessage({
+											type: "openFile",
+											text: tool.content,
+											values: tool.startLine ? { line: tool.startLine } : undefined,
+										})
+									}>
 									{tool.path?.startsWith(".") && <span>.</span>}
 									<PathTooltip content={formatPathTooltip(tool.path, tool.reason)}>
 										<span className="whitespace-nowrap overflow-hidden text-ellipsis text-left mr-2 rtl">
diff --git a/webview-ui/src/utils/formatPathTooltip.ts b/webview-ui/src/utils/formatPathTooltip.ts
@@ -21,7 +21,7 @@ export function formatPathTooltip(path?: string, additionalContent?: string): st
 	const formattedPath = removeLeadingNonAlphanumeric(path) + "\u200E"
 
 	if (additionalContent) {
-		return formattedPath + additionalContent
+		return formattedPath + " " + additionalContent
 	}
 
 	return formattedPath

Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ export function formatPathTooltip(path?: string, additionalContent?: string): st`
`21`	`21`	`const formattedPath = removeLeadingNonAlphanumeric(path) + "\u200E"`
`22`	`22`
`23`	`23`	`if (additionalContent) {`
`24`		`- return formattedPath + additionalContent`
	`24`	`+ return formattedPath + " " + additionalContent`
`25`	`25`	`}`
`26`	`26`
`27`	`27`	`return formattedPath`