Shell refuses a name that is no executable, and bare restores the default

Shell took any word and the host quietly ran another shell at the next
spawn. It now refuses a path or name that is not executable ("Shell: x:
no executable by that name", failing a ctl write, logged err), a bare
Shell goes back to the default, and a default $SHELL that is not
executable falls back to /bin/sh. shellset's golden takes the refusal on
the message row (re-recorded by name).

Co-Authored-By: Claude Opus 5.5 <[email protected]>

1 files changed, 4 insertions, 3 deletions

diff --git a/src/builtins.zig b/src/builtins.zig
index f7104a8c..f7dd543a 100644
--- a/src/builtins.zig
+++ b/src/builtins.zig
@@ -170,9 +170,10 @@ pub const registry = struct {
             if (@intFromEnum(b) == i) return @hasDecl(T, "requires_arg") and T.requires_arg;
         inline for (comptime settingList(), manualBuiltinCount()..) |setting, i|
             if (@intFromEnum(b) == i) return switch (setting.action) {
-                // a switch flips bare, a choice steps, and DumpDir bare is the default
-                .toggle, .transition, .scene, .dump_dir, .choice, .lift, .shader_animation, .motion => false,
-                .shell, .theme, .font, .tagline_size, .window_opacity, .window_blur, .message_ms, .shader, .inactive_dim, .grip_width => true,
+                // a switch flips bare, a choice steps, and DumpDir and Shell bare
+                // are the default
+                .toggle, .transition, .scene, .dump_dir, .choice, .lift, .shader_animation, .motion, .shell => false,
+                .theme, .font, .tagline_size, .window_opacity, .window_blur, .message_ms, .shader, .inactive_dim, .grip_width => true,
             };
         unreachable;
     }