Give the pipe its own fields in Pipe

Not a pure move: state moves. Pardes's `pipe_seq` and `pipe_wait` become
Pipe.zig's own fields `seq` and `wait`, and Pardes embeds one as
`pipe: Pipe = .{}`. pipeRequest reads only the request in flight, so it now
takes `pipe: *const Pipe` and the four shells call
`core.pipe.pipeRequest(id)` (their Pipe imports go away again); the other
pipe functions still need the panes and keep `p: *Pardes`, writing
`p.pipe.seq/wait`. Field reads in dump.zig, the shells' tests and two test
files follow (`core.pipe_wait` becomes `core.pipe.wait`). No behaviour
change.

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

1 files changed, 3 insertions, 4 deletions

diff --git a/src/gui/gui.zig b/src/gui/gui.zig
index ef94d5b3..a11b22c7 100644
--- a/src/gui/gui.zig
+++ b/src/gui/gui.zig
@@ -16,7 +16,6 @@ const crt = @import("crt.zig");
 const pet = @import("pet.zig");
 const fonts = @import("../fonts.zig"); // the Font builtin's half of the seam
 const selection_pipe = @import("../selection_pipe.zig");
-const Pipe = @import("../Pipe.zig");
 const c_heap = @import("c_heap");
 
 const ninep_io = @import("../9p_io.zig");
@@ -1704,7 +1703,7 @@ fn spawnPipe(
         core.update(.{ .pipe_resp = .{ .id = id, .success = false, .outputs = &.{} } });
         return;
     }
-    const view = Pipe.pipeRequest(core, id) orelse return;
+    const view = core.pipe.pipeRequest(id) orelse return;
     const job = selection_pipe.Job.copy(gpa, view) catch |err| {
         core.update(.{ .pipe_resp = .{ .id = id, .success = false, .outputs = &.{} } });
         return core.reportError(core.active, "pipe", err);
@@ -1768,9 +1767,9 @@ test "GUI worker setup failures finish matching LSP and pipe requests" {
         core.update(.{ .key = .{ .cp = '|' } });
         core.update(.{ .key = .{ .cp = 't', .text = "tr a-z A-Z" } });
         core.update(.{ .key = .{ .cp = pardes.Key.enter } });
-        const pipe_id = core.pipe_wait.?.id;
+        const pipe_id = core.pipe.wait.?.id;
         spawnPipe(core, failing_io, failing.allocator(), &queue, &tasks, pipe_id);
-        try std.testing.expect(core.pipe_wait == null);
+        try std.testing.expect(core.pipe.wait == null);
         try std.testing.expectEqualStrings("abc", pane.file.?.content);
         try std.testing.expectEqual(@as(usize, 0), workers.active.load(.monotonic));
         try std.testing.expectEqual(@as(usize, 0), tasks.len);