1 files changed, 4 insertions, 21 deletions

diff --git a/src/ninep/pane.zig b/src/ninep/pane.zig
index 6a0fbba5..7526075b 100644
--- a/src/ninep/pane.zig
+++ b/src/ninep/pane.zig
@@ -639,7 +639,7 @@ pub fn tagFault(data: []const u8) ?[]const u8 {
 
 /// A tag write's limit, the same for a pane's, a column's and the
 /// workspace's: whole or not at all.
-pub const e_tag_over = std.fmt.comptimePrint("tag: no space: over {d} bytes", .{limits.max_tag_tail});
+pub const e_tag_over = std.fmt.comptimePrint("no space: over {d} bytes", .{limits.max_tag_tail});
 
 fn writeTag(p: *Pardes, req: Req, pane: *Pane) Reply {
     if (req.data.len == 0) return .{ .tag = req.tag, .written = 0 };
@@ -860,18 +860,14 @@ fn writeName(p: *Pardes, req: Req, id: usize, pane: *Pane) Reply {
         "rename not allowed: a terminal is named by its shell's directory; cd there, or Tty in another"
     else
         "rename not allowed: an image or PDF is named by the file it shows");
-    const full = std.fs.path.resolvePosix(p.scratch.allocator(), &.{ pardes.Pardes.paneDir(pane), name }) catch
+    var home_buf: [4096]u8 = undefined;
+    const full = std.fs.path.resolvePosix(p.scratch.allocator(), &.{ pardes.Pardes.paneDir(pane), fs.expandHome(name, &home_buf) }) catch
         return Reply.fail(req.tag, E.NOMEM);
     if (!std.fs.path.isAbsolute(full) or full.len >= 4096) return tree.failText(req.tag, E.INVAL, "invalid file name: longer than a path may be");
     nameBuffer(p, id, full, false) catch |err| return Reply.fail(req.tag, switch (err) {
         error.OutOfMemory => E.NOMEM,
         else => E.INVAL,
     });
-    // A name into a directory not there yet asks for it, as Config's pane
-    // does: its Save makes the directories (`pardes new/dir/f` forwarded).
-    if (comptime pardes.hosted) if (fs.localPath(std.fs.path.dirname(full) orelse "/")) |local| {
-        if (!exec.isDirectory(local)) fileOf(pane).?.make_dirs = true;
-    };
     return .{ .tag = req.tag, .written = @intCast(req.data.len) };
 }
 
@@ -1319,7 +1315,7 @@ test "a tag write past the limit is refused whole, naming the limit" {
     try testing.expectEqual(E.NOSPC, r.errno());
     // `no space` is what 9ns maps to ENOSPC, so a shell through a mount
     // sees the errno the tree gives.
-    try testing.expect(std.mem.startsWith(u8, r.reply.ename, "tag: no space: over "));
+    try testing.expect(std.mem.startsWith(u8, r.reply.ename, "no space: over "));
     try testing.expectEqualStrings(before, rd(p, tag, 0, 1 << 16).bytes);
 }
 
@@ -1392,19 +1388,6 @@ test "a name with a component over 255 bytes is refused, and a long path's faile
     try testing.expect(std.mem.endsWith(u8, late, "/f.txt: no such directory"));
 }
 
-test "a name into a directory not there has Save make it; one into a directory there does not" {
-    if (comptime !pardes.hosted) return error.SkipZigTest;
-    const p = try withFile(testing.allocator, "x\n");
-    defer p.deinit();
-    const serial = serialOf(p);
-    const f = fileOf(p.panes[p.paneBySerial(serial).?].?).?;
-    try testing.expectEqual(Status.ok, wr(p, Node.of(serial, .name), "/tmp/pardes-no-such-dir-mk/sub/f.txt\n").reply.status);
-    try testing.expect(f.make_dirs);
-    f.make_dirs = false;
-    try testing.expectEqual(Status.ok, wr(p, Node.of(serial, .name), "/tmp/f.txt\n").reply.status);
-    try testing.expect(!f.make_dirs);
-}
-
 test "a name cut across writes is one name, applied once at its newline or its close" {
     const p = try withFile(testing.allocator, "x\n");
     defer p.deinit();