Newcol at the column cap says so, and a desktop session holds 16 columns

Newcol with every column slot taken, or a column too narrow to split,
did nothing and said nothing. It now fails (a ctl write with it) with no
space for a column: 16 max (ENOSPC through 9ns), or that the column is
too narrow. MAX_COLS is 16 on a desktop, 6 on the board; each column
costs about 8 KB of core state (the Pardes struct grows from 1.47 MB to
1.56 MB), most of it its tag's text. fs.md and the skill give the cap.

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

1 files changed, 22 insertions, 0 deletions

diff --git a/src/ninep/ctl.zig b/src/ninep/ctl.zig
index 307921b9..3282c743 100644
--- a/src/ninep/ctl.zig
+++ b/src/ninep/ctl.zig
@@ -1721,3 +1721,25 @@ test "each word's /commands description is its own doc comment's first sentence,
         try testing.expect(std.mem.indexOf(u8, source[block_start..at], head) != null);
     }
 }
+
+test "Newcol at the column cap fails the write and says the cap" {
+    const p = try withFile(testing.allocator, "x\n");
+    defer p.deinit();
+    p.update(.{ .resize = .{ .cols = 4000, .rows = 30 } });
+    const root_ctl = @intFromEnum(tree.TopFile.ctl);
+    while (p.ncol < pardes.MAX_COLS) {
+        // From the widest column's tag, so no column gets too narrow first.
+        p.sync();
+        var widest: usize = 0;
+        for (0..p.ncol) |c| if (p.col_w[c] > p.col_w[widest]) {
+            widest = c;
+        };
+        pardes.tagline.enterHeader(p, widest);
+        try testing.expectEqual(Status.ok, wr(p, root_ctl, "Newcol\n").reply.status);
+    }
+    const refused = wr(p, root_ctl, "Newcol\n");
+    try testing.expectEqual(Status.err, refused.reply.status);
+    var want: [64]u8 = undefined;
+    try testing.expect(std.mem.indexOf(u8, refused.reply.ename, try std.fmt.bufPrint(&want, "no space for a column: {d} max", .{pardes.MAX_COLS})) != null);
+    try testing.expect(th.logHas(p, "no space for a column"));
+}