From c6a5aaf5c4a020eaefe24943ecb6cf9d4eb26836 Mon Sep 17 00:00:00 2001
From: Gabriel Schneider <gbrls@0x4200.cafe>
Date: Mon, 28 Sep 2026 15:57:22 -0300
Subject: Post chain: Shadertoy passes compiled by glslc; Crt, Ripple and
 Glitch rewritten as Shadertoy files

Stage 9 of docs/render-pipeline.md. src/gui/Post.zig runs the chain over
the finished frame: each pass reads the one before, ping-pong textures
between, the last writes the window; an empty chain is the direct path.
The prefix (shaders/post/prefix.glsl) is ghostty's uniform block, name
for name and offset for offset (tested against a copy of its Uniforms, the
selection colours by their GLSL names), at SDL GPU's bindings, y down.

A user's file (Shader <path>) is compiled by glslc with compileGlsl's own
flags on a thread of its own, taken in at the next loop step; errors name
the file's own lines; a failed compile keeps the last good pipeline;
glslc missing is said once. The bundled passes compile with the build,
through the same prefix.

Crt, Ripple and Glitch are Shadertoy files, rewritten: no barrel (input is
identity, tested), everything in linear light and dithered, keyed to iTime.
scene_effects, crt.zig and crt.frag.glsl are gone; the core keeps no clock
for the chain. ShaderAnimation off|on|always drives redraw level A: the
chain alone over the retained frame (34 us CPU a redraw, measured).

Shared files touched: pardes.zig (nextWake loses the scene line,
disableSceneEffects empties the chain, two tests), builtins.zig (one switch
arm), ninep/ctl.zig (two switch arms), macos.zig (flags from the chain),
gui.zig. Not touched: Messages.zig, mouse.zig, detached/*, host_io, tty.
---
 src/pardes.zig | 28 ++++++++++------------------
 1 file changed, 10 insertions(+), 18 deletions(-)

(limited to 'src/pardes.zig')

diff --git a/src/pardes.zig b/src/pardes.zig
index 616747fa..5fa3307b 100644
--- a/src/pardes.zig
+++ b/src/pardes.zig
@@ -584,18 +584,10 @@ test "startup config runs builtin lines in order and isolates bad lines" {
     };
 }
 
-test "scene effects keep the idle frontend animation clock active" {
+test "a post chain keeps no core clock running: the shell animates it" {
     const p = try Pardes.init(std.testing.allocator, .{ .tty_only = true });
     defer p.deinit();
-    try std.testing.expect(!p.animationActive());
-
-    p.settings.scene_effects.crt = true;
-    try std.testing.expect(p.animationActive());
-    p.settings.scene_effects = .{ .ripple = true };
-    try std.testing.expect(p.animationActive());
-    p.settings.scene_effects = .{ .glitch = true };
-    try std.testing.expect(p.animationActive());
-    p.settings.scene_effects = .{};
+    try std.testing.expect(p.settings.post.set(.crt, "", true));
     try std.testing.expect(!p.animationActive());
 }
 
@@ -5443,10 +5435,10 @@ pub const Pardes = struct {
         look.cancelLookHover(p);
     }
 
-    /// A native scene pass proved unavailable. Config reads these exact bits,
-    /// and the idle-clock predicate stops immediately once they are clear.
+    /// A native post pass proved unavailable: the chain is emptied, which
+    /// Config then reports.
     pub fn disableSceneEffects(p: *Pardes) void {
-        p.settings.scene_effects = .{};
+        p.settings.post.clear();
     }
 
     // ---- doc panes ----
@@ -5744,8 +5736,6 @@ pub const Pardes = struct {
     pub fn nextWake(p: *const Pardes) ?u64 {
         const next = p.stepped_ns + animation.frame_ns;
         if (p.chrome_animation.isActive() or p.presentation.animating()) return next;
-        const scene = p.settings.scene_effects;
-        if (scene.crt or scene.ripple or scene.glitch) return next;
         var frames: ?u64 = null;
         if (p.look_hover_wait) |waiting| {
             const delay = config.look_preview_delay_frames orelse return next;
@@ -6953,8 +6943,8 @@ test "core animation keeps wall time at 60, 120 and 144 Hz and across a stall" {
     const p = try Pardes.init(std.testing.allocator, .{ .tty_only = true, .cols = 80, .rows = 24 });
     defer p.deinit();
     const frame = animation.frame_ns;
-    // Something that animates for as long as it is on.
-    p.settings.scene_effects.crt = true;
+    // Something that animates for as long as the test wants: the theme
+    // fade, held at its start before every step.
     for ([_]u64{ 60, 120, 144 }) |hz| {
         const start: u64 = 5 * std.time.ns_per_s;
         p.clock_started = false;
@@ -6963,6 +6953,7 @@ test "core animation keeps wall time at 60, 120 and 144 Hz and across a stall" {
         var at: u64 = 1;
         while (at <= hz) : (at += 1) {
             const before = p.stepped_ns;
+            p.chrome_animation.step = 0;
             p.advance(start + at * std.time.ns_per_s / hz);
             const taken = (p.stepped_ns - before + frame / 2) / frame;
             // Never two steps in one display frame: that is a visible hitch.
@@ -6979,9 +6970,10 @@ test "core animation keeps wall time at 60, 120 and 144 Hz and across a stall" {
     }
     // A stall of minutes is not minutes of frames.
     const before = p.stepped_ns;
+    p.chrome_animation.step = 0;
     p.advance(before + 600 * std.time.ns_per_s);
     try std.testing.expectEqual(before + 600 * std.time.ns_per_s, p.stepped_ns);
-    p.settings.scene_effects.crt = false;
+    p.chrome_animation.step = animation.transition_steps;
     p.advance(p.stepped_ns + frame);
     try std.testing.expectEqual(@as(?u64, null), p.nextWake());
 }
-- 
cgit v1.3