100% test coverage of parseToolCallDetails 🚀 (#7818)

* WIP

* parseToolCallDetails 100% coverage and remove unreachable code

Author: alexr00

common/sessionParsing.ts

@@ -192,15 +192,7 @@ export function parseToolCallDetails(
 				const filePath = args.path;
 				let fileLabel = filePath ? toFileLabel(filePath) : undefined;
 
-				if (fileLabel === undefined) {
-					fileLabel = filePath;
-
-					return {
-						toolName: fileLabel ? (`Read ${fileLabel}` + (parsedRange ? `, lines ${parsedRange.start} to ${parsedRange.end}` : '')) : 'Read repository',
-						invocationMessage: fileLabel ? (`Read ${fileLabel}` + (parsedRange ? `, lines ${parsedRange.start} to ${parsedRange.end}` : '')) : 'Read repository',
-						pastTenseMessage: fileLabel ? (`Read ${fileLabel}` + (parsedRange ? `, lines ${parsedRange.start} to ${parsedRange.end}` : '')) : 'Read repository',
-					};
-				} else if (fileLabel === '') {
+				if (fileLabel === undefined || fileLabel === '') {
 					return {
 						toolName: 'Read repository',
 						invocationMessage: 'Read repository',
@@ -250,7 +242,7 @@ export function parseToolCallDetails(
 				filePath: filePath,
 				fileLabel: fileLabel,
 			} : undefined
-		}
+		};
 	} else if (name === 'create') {
 		const filePath = args.path;
 		const fileLabel = filePath && toFileLabel(filePath);
@@ -264,7 +256,7 @@ export function parseToolCallDetails(
 				filePath: filePath,
 				fileLabel: fileLabel,
 			} : undefined
-		}
+		};
 	} else if (name === 'view') {
 		const filePath = args.path;
 		const fileLabel = filePath && toFileLabel(filePath);
@@ -319,12 +311,12 @@ export function parseToolCallDetails(
 		return {
 			toolName: 'read_bash',
 			invocationMessage: 'Read logs from Bash session'
-		}
+		};
 	} else if (name === 'stop_bash') {
 		return {
 			toolName: 'stop_bash',
 			invocationMessage: 'Stop Bash session'
-		}
+		};
 	} else {
 		// Unknown tool type
 		return {

package.json

@@ -4179,6 +4179,7 @@
 		"clean": "rm -r dist/",
 		"compile": "webpack --mode development --env esbuild",
 		"compile:test": "tsc -p tsconfig.test.json",
+		"watch:test": "tsc -w -p tsconfig.test.json",
 		"compile:node": "webpack --mode development --config-name extension:node --config-name webviews",
 		"compile:web": "webpack --mode development --config-name extension:webworker --config-name webviews",
 		"lint": "eslint --fix --cache --config .eslintrc.js --ignore-pattern src/env/browser/**/* \"{src,webviews}/**/*.{ts,tsx}\"",