fix: use program name when detecting shell via $SHELL

ethanndickson · ethanndickson · commit 150623e62dfa · 2025-10-24T02:47:20.000Z
diff --git a/completion/all.go b/completion/all.go
@@ -59,7 +59,7 @@ func ShellOptions(choice *string) *serpent.Enum {
 func DetectUserShell(programName string) (Shell, error) {
 	// Attempt to get the SHELL environment variable first
 	if shell := os.Getenv("SHELL"); shell != "" {
-		return ShellByName(filepath.Base(shell), "")
+		return ShellByName(filepath.Base(shell), programName)
 	}
 
 	// Fallback: Look up the current user and parse /etc/passwd

Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ func ShellOptions(choice string) serpent.Enum {`
`59`	`59`	`func DetectUserShell(programName string) (Shell, error) {`
`60`	`60`	`// Attempt to get the SHELL environment variable first`
`61`	`61`	`if shell := os.Getenv("SHELL"); shell != "" {`
`62`		`- return ShellByName(filepath.Base(shell), "")`
	`62`	`+ return ShellByName(filepath.Base(shell), programName)`
`63`	`63`	`}`
`64`	`64`
`65`	`65`	`// Fallback: Look up the current user and parse /etc/passwd`