From 6f42f51cd652e189408eace598f99ec25b9542f8 Mon Sep 17 00:00:00 2001
From: Gabriel Schneider <gbrls@0x4200.cafe>
Date: Tue, 29 Sep 2026 22:38:16 -0300
Subject: The hosts' Save, Dump and DumpThemes writes and their shell reaper
 are host_io's, once, where the tty, the gui and the detached session each had
 a copy

writeFile, writeDump and dumpThemes were byte-for-byte the same in tty.zig, gui.zig and detached/server.zig but for how each reaches its core, and the gui's and the session's reapers the same but for a pid <= 0 against == 0 (a pid is never negative there). Each vtable entry is now a line calling host_io's. ttyTaken, fgName, killJob and ptyResize stay each shell's: they are already one call to host_io each, and what differs is how a shell reaches its pty (an optional with a file, an optional with an fd, a bounds-checked array). macos.zig keeps its copies (no macOS build here to check it). No behaviour changes.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 src/tty/tty.zig | 26 +++-----------------------
 1 file changed, 3 insertions(+), 23 deletions(-)

(limited to 'src/tty')

diff --git a/src/tty/tty.zig b/src/tty/tty.zig
index 3441bc29..ef7f2761 100644
--- a/src/tty/tty.zig
+++ b/src/tty/tty.zig
@@ -1791,24 +1791,11 @@ const Shell = struct {
 
     fn writeFile(ctx: ?*anyopaque, pane: u8, path: []const u8, bytes: []const u8) void {
         const s = of(ctx);
-        filesystem.write(s.core, path, bytes) catch |err|
-            return s.core.saveFailed(pane, path, err);
-        if (s.core.panes[pane]) |pn| if (pn.file) |f| if (std.mem.eql(u8, f.path, path)) {
-            if (s.watches[pane]) |*w| if (w.serial == pn.serial) switch (w.generation) {
-                .text => w.generation = .{ .text = std.hash.Wyhash.hash(0, bytes) },
-                .pdf => {},
-            };
-        };
-        var mbuf: [256]u8 = undefined;
-        s.core.setMessage(pane, message.stamp(&mbuf, "saved", path));
+        host_io.writeFile(s.core, &s.watches, pane, path, bytes);
     }
 
     fn writeDump(ctx: ?*anyopaque, bytes: []const u8) void {
-        const s = of(ctx);
-        var pbuf: [1024:0]u8 = undefined;
-        const path = pardes.dump.outPath(&pbuf, s.core.settings.dump_dir.get()) orelse return s.core.dumpFailed(s.core.settings.dump_dir.get(), error.NoDumpDirectory);
-        filesystem.write(s.core, path, bytes) catch |err| return s.core.dumpFailed(path, err);
-        s.core.setLastDump(path);
+        host_io.writeDump(of(ctx).core, bytes);
     }
 
     fn watchFile(ctx: ?*anyopaque, pane: u8, _: []const u8, on: bool, mode: pardes.WatchMode) void {
@@ -1825,14 +1812,7 @@ const Shell = struct {
 
     fn dumpThemes(ctx: ?*anyopaque, pane: u8) void {
         const s = of(ctx);
-        const config_dir = s.core.opts.config_dir orelse return;
-        const out_dir = pardes.config.User.dumpThemes(s.io, s.gpa, config_dir, pardes.themes) catch |err| {
-            s.core.reportError(pane, "dump themes", err);
-            return;
-        };
-        defer s.gpa.free(out_dir);
-        var mbuf: [256]u8 = undefined;
-        s.core.setMessage(pane, message.stamp(&mbuf, "dumped themes", out_dir));
+        host_io.dumpThemes(s.core, s.io, s.gpa, pane);
     }
 
     fn setClipboard(ctx: ?*anyopaque, text: []const u8) void {
-- 
cgit v1.3