From fda2f546f66d0bb1b30da0ef6aae77fe82803b1c Mon Sep 17 00:00:00 2001
From: Gabriel Schneider <gbrls@0x4200.cafe>
Date: Wed, 30 Sep 2026 11:22:41 -0300
Subject: DumpConfig prints each setting as the ctl word that sets it, reads
 unsupported for one the frontend lacks, and includes LocationsConfig

The report meant to be feedable back as configuration, but several lines
were not. Shell and Font each took three to six lines of "requested",
"effective" and "pending". The transition read "Panel transition
PanelSlide", with no Panel word. WindowOpacity and TaglineSize carried
"(pending)" and "(build-time only)". A GUI-only toggle such as Ligatures was
left out on the tty, and LocationsConfig was missing altogether.

Now every line up to a blank one is a word and its value:
- Shell <path>;
- Font <name>:<size>;
- one on/off line for each Panel word;
- `unsupported` for a setting this frontend does not have;
- LocationsConfig as it would be typed.

After the blank line comes what is in effect but set by no word, phrased so
that none of it starts with a ctl word. The test checks that each line
before the blank one starts with a setting word. The GUI scripts that read
the font lines (font_size.py, appearance_gallery.py) now read the new ones.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 test/output.zig | 26 ++++++++++++--------------
 1 file changed, 12 insertions(+), 14 deletions(-)

(limited to 'test/output.zig')

diff --git a/test/output.zig b/test/output.zig
index c67c021e..51d620c0 100644
--- a/test/output.zig
+++ b/test/output.zig
@@ -544,11 +544,12 @@ test "DumpConfig prints the startup path and refreshes its one output" {
     for ([_][]const u8{
         "Startup config: " ++ path ++ "\n",
         "Theme orchard\n",
-        "Shell effective (last spawn) (none)\n",
-        "Shell pending on\n",
+        "Last shell spawned: (none)\n",
+        "Shell change pending: on\n",
+        "LocationsConfig ",
     }) |line| try std.testing.expect(std.mem.indexOf(u8, out.content, line) != null);
     var default_line: [256]u8 = undefined;
-    try std.testing.expect(std.mem.indexOf(u8, out.content, try std.fmt.bufPrint(&default_line, "Shell requested (new panes) {s} (default)\n", .{config.defaultShell()})) != null);
+    try std.testing.expect(std.mem.indexOf(u8, out.content, try std.fmt.bufPrint(&default_line, "Shell {s}\n", .{config.defaultShell()})) != null);
     try std.testing.expectEqualStrings(config.dump_config_buffer, std.fs.path.basename(out.path));
     try std.testing.expectEqual(panes.Output.Origin{ .cmd = .DumpConfig }, out.output.?.from);
 
@@ -734,14 +735,14 @@ test "every enabled setting builtin mutates the State Config reports" {
         "Tagbottom on\n",
         "Debug on\n",
         "Theme acme\n",
-        "Shell requested (new panes) fish\n",
+        "Shell fish\n",
     }) |line| try std.testing.expect(std.mem.indexOf(u8, report, line) != null);
-    const transition = if (builtins.capabilities.panel_transitions)
-        try std.fmt.bufPrint(&buf, "Panel transition {s}\n", .{
-            config.Runtime.findAction(.{ .transition = p.settings.panel_transition }).?.word,
-        })
+    const transition = if (!builtins.capabilities.panel_transitions)
+        "PanelSlide unsupported\n"
+    else if (p.settings.panel_transition == .off)
+        "PanelSlide off\n"
     else
-        "Panel transition unsupported\n";
+        try std.fmt.bufPrint(&buf, "{s} on\n", .{config.Runtime.findAction(.{ .transition = p.settings.panel_transition }).?.word});
     try std.testing.expect(std.mem.indexOf(u8, report, transition) != null);
     // A bare word turns a bundled pass on at its default level.
     const scene_status = if (builtins.capabilities.scene_shaders) "2" else "unsupported";
@@ -752,14 +753,11 @@ test "every enabled setting builtin mutates the State Config reports" {
     const tagline = if (!builtins.capabilities.tagline_font_size)
         "TaglineSize unsupported\n"
     else
-        try std.fmt.bufPrint(&buf, "TaglineSize {d}{s}\n", .{
-            p.settings.font.tagline_percent,
-            if (builtins.capabilities.font_picker) "" else " (build-time only)",
-        });
+        try std.fmt.bufPrint(&buf, "TaglineSize {d}\n", .{p.settings.font.tagline_percent});
     try std.testing.expect(std.mem.indexOf(u8, report, tagline) != null);
     if (chosen_font) |name| {
         try std.testing.expect(std.mem.indexOf(u8, report, name) != null);
-        try std.testing.expect(std.mem.indexOf(u8, report, "Font pending on\n") != null);
+        try std.testing.expect(std.mem.indexOf(u8, report, "Font change pending: on\n") != null);
     }
 }
 
-- 
cgit v1.3