ai-workspace-services/qmd
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: skip Git Bash path detection on WSL

Browse Source 
On WSL, paths like /c/work/... are valid drvfs mount points, not Git
Bash drive-letter shortcuts. The existing code in isAbsolutePath() and
resolve() detected /c/ as a Windows C: path, converting drvfs paths to
C:/work/... which broke indexing entirely.

Fix: detect WSL via WSL_DISTRO_NAME or WSL_INTEROP environment variables
and skip the Git Bash /c/ -> C: branch on WSL. Native Linux path handling
continues as before.
This commit is contained in:
Øystein Krog 2026-03-11 12:25:28 +01:00
parent ae3604cb88
commit 26d4ebfa56
1 changed files with 14 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
src/store.ts
View File

@ -270,7 +270,8 @@ export function isAbsolutePath(path: string): boolean {
if (path.startsWith('/')) { if (path.startsWith('/')) {
// Check if it's a Git Bash style path like /c/ or /c/Users (C-Z only, not A or B) // Check if it's a Git Bash style path like /c/ or /c/Users (C-Z only, not A or B)
// Requires path[2] === '/' to distinguish from Unix paths like /c or /cache // Requires path[2] === '/' to distinguish from Unix paths like /c or /cache
if (path.length >= 3 && path[2] === '/') { // Skipped on WSL where /c/ is a valid drvfs mount point, not a drive letter
if (!isWSL() && path.length >= 3 && path[2] === '/') {
const driveLetter = path[1]; const driveLetter = path[1];
if (driveLetter && /[c-zC-Z]/.test(driveLetter)) { if (driveLetter && /[c-zC-Z]/.test(driveLetter)) {
return true; return true;
@ -296,6 +297,14 @@ export function normalizePathSeparators(path: string): string {
return path.replace(/\\/g, '/'); return path.replace(/\\/g, '/');
} }
/**
* Detect if running inside WSL (Windows Subsystem for Linux).
* On WSL, paths like /c/work/... are valid drvfs mount points, not Git Bash paths.
*/
function isWSL(): boolean {
return !!(process.env.WSL_DISTRO_NAME || process.env.WSL_INTEROP);
}
/** /**
* Get the relative path from a prefix. * Get the relative path from a prefix.
* Returns null if path is not under prefix. * Returns null if path is not under prefix.
@ -348,8 +357,9 @@ export function resolve(...paths: string[]): string {
if (firstPath.length >= 2 && /[a-zA-Z]/.test(firstPath[0]!) && firstPath[1] === ':') { if (firstPath.length >= 2 && /[a-zA-Z]/.test(firstPath[0]!) && firstPath[1] === ':') {
windowsDrive = firstPath.slice(0, 2); windowsDrive = firstPath.slice(0, 2);
result = firstPath.slice(2); result = firstPath.slice(2);
} else if (firstPath.startsWith('/') && firstPath.length >= 3 && firstPath[2] === '/') { } else if (!isWSL() && firstPath.startsWith('/') && firstPath.length >= 3 && firstPath[2] === '/') {
// Git Bash style: /c/ -> C: (C-Z drives only, not A or B) // Git Bash style: /c/ -> C: (C-Z drives only, not A or B)
// Skipped on WSL where /c/ is a valid drvfs mount point, not a drive letter
const driveLetter = firstPath[1]; const driveLetter = firstPath[1];
if (driveLetter && /[c-zC-Z]/.test(driveLetter)) { if (driveLetter && /[c-zC-Z]/.test(driveLetter)) {
windowsDrive = driveLetter.toUpperCase() + ':'; windowsDrive = driveLetter.toUpperCase() + ':';
@ -380,8 +390,9 @@ export function resolve(...paths: string[]): string {
if (p.length >= 2 && /[a-zA-Z]/.test(p[0]!) && p[1] === ':') { if (p.length >= 2 && /[a-zA-Z]/.test(p[0]!) && p[1] === ':') {
windowsDrive = p.slice(0, 2); windowsDrive = p.slice(0, 2);
result = p.slice(2); result = p.slice(2);
} else if (p.startsWith('/') && p.length >= 3 && p[2] === '/') { } else if (!isWSL() && p.startsWith('/') && p.length >= 3 && p[2] === '/') {
// Git Bash style (C-Z drives only, not A or B) // Git Bash style (C-Z drives only, not A or B)
// Skipped on WSL where /c/ is a valid drvfs mount point, not a drive letter
const driveLetter = p[1]; const driveLetter = p[1];
if (driveLetter && /[c-zC-Z]/.test(driveLetter)) { if (driveLetter && /[c-zC-Z]/.test(driveLetter)) {
windowsDrive = driveLetter.toUpperCase() + ':'; windowsDrive = driveLetter.toUpperCase() + ':';