git.iko.soy/crush
on main help sign in
fix: more reliably detect windows drive (#2273)
commit54d924b0a8d8c09717d893e3230dc8913d56b01f authorAndrey Nering <andreynering@users.noreply.github.com> dateFri 20 Feb 2026, 15:03 (8 months ago) parentc43192c
3 files changed+44−7
compare with main
unifiedsplit
internal/agent/tools/bash.go+2−7 internal/fsext/drive_other.go+16 internal/fsext/drive_windows.go+26
internal/agent/tools/bash.go+2−7
@@ -7,7 +7,6 @@ import (
77 _ "embed"
88 "fmt"
99 "html/template"
10− "os"
1110 "path/filepath"
1211 "runtime"
1312 "strings"
@@ -15,6 +14,7 @@ import (
1514
1615 "charm.land/fantasy"
1716 "github.com/charmbracelet/crush/internal/config"
17+ "github.com/charmbracelet/crush/internal/fsext"
1818 "github.com/charmbracelet/crush/internal/permission"
1919 "github.com/charmbracelet/crush/internal/shell"
2020)
@@ -431,12 +431,7 @@ func countLines(s string) int {
431431
432432func normalizeWorkingDir(path string) string {
433433 if runtime.GOOS == "windows" {
434− cwd, err := os.Getwd()
435− if err != nil {
436− cwd = "C:"
437− }
438− path = strings.ReplaceAll(path, filepath.VolumeName(cwd), "")
434+ path = strings.ReplaceAll(path, fsext.WindowsWorkingDirDrive(), "")
439435 }
440−
441436 return filepath.ToSlash(path)
442437}
internal/fsext/drive_other.gonew file+16
@@ -0,0 +1,16 @@
1+//go:build !windows
2+
3+package fsext
4+
5+// WindowsWorkingDirDrive returns the drive letter of the current working
6+// directory, e.g. "C:".
7+// Falls back to the system drive if the current working directory cannot be
8+// determined.
9+func WindowsWorkingDirDrive() string {
10+ panic("cannot call fsext.WindowsWorkingDirDrive() on non-Windows OS")
11+}
12+
13+// WindowsSystemDrive returns the drive letter of the system drive, e.g. "C:".
14+func WindowsSystemDrive() string {
15+ panic("cannot call fsext.WindowsSystemDrive() on non-Windows OS")
16+}
internal/fsext/drive_windows.gonew file+26
@@ -0,0 +1,26 @@
1+//go:build windows
2+
3+package fsext
4+
5+import (
6+ "cmp"
7+ "os"
8+ "path/filepath"
9+)
10+
11+// WindowsWorkingDirDrive returns the drive letter of the current working
12+// directory, e.g. "C:".
13+// Falls back to the system drive if the current working directory cannot be
14+// determined.
15+func WindowsWorkingDirDrive() string {
16+ if cwd, err := os.Getwd(); err == nil {
17+ return filepath.VolumeName(cwd)
18+ }
19+ return WindowsSystemDrive()
20+}
21+
22+// WindowsSystemDrive returns the drive letter of the system drive, e.g. "C:".
23+func WindowsSystemDrive() string {
24+ systemRoot := cmp.Or(os.Getenv("SYSTEMROOT"), os.Getenv("WINDIR"))
25+ return filepath.VolumeName(systemRoot)
26+}