A column's and the workspace's tag read with no trailing newline, as a pane's does

A pane tag read back its text alone, but a column tag and /tag appended a
newline and gave a stat size one byte longer. So `$(cat tag)` agreed across
the three kinds while `cat tag | wc -c` and a byte compare did not. All
three now read the text alone, and fs.md says so.

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

1 files changed, 4 insertions, 4 deletions

diff --git a/src/ninep/cols.zig b/src/ninep/cols.zig
index 4db78584..1919122d 100644
--- a/src/ninep/cols.zig
+++ b/src/ninep/cols.zig
@@ -69,8 +69,8 @@ pub fn read(p: *Pardes, req: Req, serial: ?u32) Reply {
     applyTruncation(p, serial);
     const text = headerText(p, serial) orelse return Reply.fail(req.tag, E.NOENT);
     const out = p.fs.stage(p.gpa);
+    // As a pane tag reads: the text alone, no newline after it.
     out.appendSlice(p.gpa, text) catch return Reply.fail(req.tag, E.NOMEM);
-    out.append(p.gpa, '\n') catch return Reply.fail(req.tag, E.NOMEM);
     return tree.stagedReply(p, req);
 }
 
@@ -390,15 +390,15 @@ test "a column's tag and the workspace's read, replace and append as a pane tag
     try testing.expectEqual(E.NOENT, th.look_up(p, col_dir, "777").errno());
     const tag = Node.ofCol(serial, .tag);
     try testing.expectEqual(tree.Status.ok, th.look_up(p, Node.ofCol(serial, .dir), "tag").reply.status);
-    try testing.expectEqualStrings("New Tty Find Grep Joincol Delcol\n", th.rd(p, tag, 0, 4096).bytes);
+    try testing.expectEqualStrings("New Tty Find Grep Joincol Delcol", th.rd(p, tag, 0, 4096).bytes);
     // `echo Make > tag`: cleared, then the newline ending the write dropped.
     _ = th.call(p, .{ .tag = 1, .op = .setattr, .node = tag, .truncate = true });
     _ = th.wr(p, tag, "Make\n");
-    try testing.expectEqualStrings("Make\n", th.rd(p, tag, 0, 4096).bytes);
+    try testing.expectEqualStrings("Make", th.rd(p, tag, 0, 4096).bytes);
     _ = th.call(p, .{ .tag = 2, .op = .release, .node = tag });
     // `>>` appends; a newline in a one-line tag is a space.
     _ = th.wr(p, tag, " Lint\nNow");
-    try testing.expectEqualStrings("Make Lint Now\n", th.rd(p, tag, 0, 4096).bytes);
+    try testing.expectEqualStrings("Make Lint Now", th.rd(p, tag, 0, 4096).bytes);
     const top = @intFromEnum(tree.TopFile.tag);
     _ = th.call(p, .{ .tag = 3, .op = .setattr, .node = top, .truncate = true });
     _ = th.wr(p, top, "Newcol Exit\n");