A dump holds every column a session may, 16, not 6

Dump of a session with more than 6 columns failed bad dump columns, since dump.zig's max_cols was 6 while a session holds MAX_COLS. It is MAX_COLS now; the limit's lines in docs/fs.md and docs/divergences.md go.

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

1 files changed, 19 insertions, 1 deletions

diff --git a/src/dump.zig b/src/dump.zig
index fca88a2d..194c1ed3 100644
--- a/src/dump.zig
+++ b/src/dump.zig
@@ -129,7 +129,7 @@ pub fn defaultDirectory(buf: []u8) ?[]const u8 {
 pub const magic = "pardes-dump";
 pub const version: u32 = 1;
 pub const max_panes: usize = MAX_PANES;
-pub const max_cols: usize = 6;
+pub const max_cols: usize = MAX_COLS;
 /// Output arguments are typed in the same bounded one-line tag storage. Keep
 /// the schema limit named independently so a dump reader can validate it
 /// without importing the output-pane implementation. The bound itself is
@@ -987,6 +987,24 @@ test "a dump keeps the settings that differ from a fresh session's, and a restor
     try std.testing.expect(!restored.settings.verbose);
 }
 
+test "a session with every column it may hold dumps and restores them all" {
+    const gpa = std.testing.allocator;
+    const p = try Pardes.init(gpa, .{ .tty_only = true, .cols = 1200, .rows = 24 });
+    defer p.deinit();
+    while (p.ncol < MAX_COLS) {
+        layout.compute(p);
+        var widest: usize = 0;
+        for (0..p.ncol) |c| if (p.col_w[c] > p.col_w[widest]) {
+            widest = c;
+        };
+        _ = layout.insertColumn(p, widest, false) orelse return error.NoColumn;
+    }
+    try dumpState(p);
+    const restored = try initFromDump(gpa, .{ .tty_only = true, .cols = 1200, .rows = 24 }, p.dump_out.?);
+    defer restored.deinit();
+    try std.testing.expectEqual(@as(usize, MAX_COLS), restored.ncol);
+}
+
 test "a restored pane keeps its dot, as acme's dump keeps a window's" {
     const gpa = std.testing.allocator;
     const p = try Pardes.init(gpa, .{ .tty_only = true });