Fix duplicate pathExists method in sessionDiscovery.ts

rajbos · Copilot · rajbos · commit ce87569ca229 · 2026-03-30T10:40:44.000+02:00
Merge introduced two copies: one from the PR branch (line 41) and one
from main's async-IO refactor commit. Remove the second duplicate.

Co-authored-by: Copilot &lt;223556219+Copilot@users.noreply.github.com&gt;
diff --git a/vscode-extension/src/sessionDiscovery.ts b/vscode-extension/src/sessionDiscovery.ts
@@ -520,14 +520,6 @@ export class SessionDiscovery {
 		return sessionFiles;
 	}
 
-	/**
-	 * Async helper: resolves true if the path is accessible, false otherwise.
-	 * Prefer this over fs.existsSync() to avoid blocking the extension-host event loop.
-	 */
-	private async pathExists(p: string): Promise<boolean> {
-		try { await fs.promises.access(p); return true; } catch { return false; }
-	}
-
 	/**
 	 * Recursively scan a directory for session files (.json and .jsonl)
 	 * 

Original file line number	Diff line number	Diff line change
`@@ -520,14 +520,6 @@ export class SessionDiscovery {`
`520`	`520`	`return sessionFiles;`
`521`	`521`	`}`
`522`	`522`
`523`		`- /**`
`524`		`- * Async helper: resolves true if the path is accessible, false otherwise.`
`525`		`- * Prefer this over fs.existsSync() to avoid blocking the extension-host event loop.`
`526`		`- */`
`527`		`- private async pathExists(p: string): Promise<boolean> {`
`528`		`- try { await fs.promises.access(p); return true; } catch { return false; }`
`529`		`- }`
`530`		`-`
`531`	`523`	`/**`
`532`	`524`	`* Recursively scan a directory for session files (.json and .jsonl)`
`533`	`525`	`*`