removing specific fish instructions to keep things lean

Author: omerxx

internal/devbox/devbox.go

@@ -376,17 +376,22 @@ func (d *Devbox) EnvExports(ctx context.Context, opts devopt.EnvExportsOpts) (st
 	}
 
 	if opts.RunHooks {
-		var hooksStr string
-		if opts.ShellFormat == devopt.ShellFormatNushell {
-			// Nushell uses 'source' command
-			hooksStr = "source \"" + shellgen.ScriptPath(d.ProjectDir(), shellgen.HooksFilename) + "\""
-		} else {
-			// Bash/Zsh/Fish use '.' command
-			hooksStr = ". \"" + shellgen.ScriptPath(d.ProjectDir(), shellgen.HooksFilename) + "\""
-		}
+		hooksStr := ". \"" + shellgen.ScriptPath(d.ProjectDir(), shellgen.HooksFilename) + "\""
 		envStr = fmt.Sprintf("%s\n%s;\n", envStr, hooksStr)
 	}
 
+	// if opts.RunHooks {
+	// 	var hooksStr string
+	// 	if opts.ShellFormat == devopt.ShellFormatNushell {
+	// 		// Nushell uses 'source' command
+	// 		hooksStr = "source \"" + shellgen.ScriptPath(d.ProjectDir(), shellgen.HooksFilename) + "\""
+	// 	} else {
+	// 		// Bash/Zsh/Fish use '.' command
+	// 		hooksStr = ". \"" + shellgen.ScriptPath(d.ProjectDir(), shellgen.HooksFilename) + "\""
+	// 	}
+	// 	envStr = fmt.Sprintf("%s\n%s;\n", envStr, hooksStr)
+	// }
+
 	if !opts.NoRefreshAlias {
 		envStr += "\n" + d.refreshAliasForShell(string(opts.ShellFormat))
 	}

internal/devbox/refresh.go

@@ -64,9 +64,6 @@ func (d *Devbox) refreshCmdForShell(format string) string {
 		// Nushell doesn't have eval; use overlay or source with temporary file
 		return fmt.Sprintf(`devbox %s | save -f ~/.cache/devbox-env.nu; source ~/.cache/devbox-env.nu`, devboxCmd)
 	}
-	if format == "fish" || isFishShell() {
-		return fmt.Sprintf(`eval (devbox %s  | string collect)`, devboxCmd)
-	}
 	return fmt.Sprintf(`eval "$(devbox %s)" && hash -r`, devboxCmd)
 }