From f4f3d63bc1946b191899b6f4ce725de81826e286 Mon Sep 17 00:00:00 2001 From: Levi Neuwirth Date: Tue, 1 Sep 2026 12:48:19 +0200 Subject: [PATCH] docs(gui-1b): repair B4's stale references and a duplicated sentence Deleting `middle_click_paste_source` left a broken intra-doc link on `apply_middle_press` and a stale name in the end-to-end row's comment. Both now name `paste_source_for`, which is what the code calls. And `an_unused_button_produces_no_effect_of_any_kind`'s opening sentence was duplicated on one line --- the tail of the same insertion that split it in the first place. Repaired. WHY NEITHER WAS CAUGHT, which is the part worth keeping: NOTHING RUNS `cargo doc`. It is absent from `scripts/gate` and from every ci.yml job, so broken intra-doc links are ungated across this repository. `git diff --check` cannot see them because they are syntactically valid, and clippy does not read doc links. Running it by hand here confirms my link now resolves --- and surfaces one PRE-EXISTING unresolved link, `MathNode` at pmacs-gpu/src/math_layout.rs:314, in a file this lane has never touched. I have not added a doc step to the gate: that is shared infrastructure and its own lane, alongside the clippy default-features gap this lane already recorded. The finding is carried to 1b's ledger block rather than left in a commit message. --- pmacs-gpu/src/main.rs | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/pmacs-gpu/src/main.rs b/pmacs-gpu/src/main.rs index 626dd29..1a79750 100644 --- a/pmacs-gpu/src/main.rs +++ b/pmacs-gpu/src/main.rs @@ -3439,9 +3439,10 @@ impl App { /// Perform [`PointerRoute::MiddlePress`] — GUI Stage 1b B4. /// - /// Reads the selection [`middle_click_paste_source`] names and ships - /// it as a `Paste`, the same wire operation Ctrl-V uses. The daemon - /// inserts it; this frontend never edits the document itself. + /// Reads the selection [`paste_source_for`] names for this platform + /// and ships it as a `Paste`, the same wire operation Ctrl-V uses. + /// The daemon inserts it; this frontend never edits the document + /// itself. fn apply_middle_press(&mut self) { #[cfg(test)] let is_linux = !self.test_force_non_linux; @@ -4994,7 +4995,7 @@ mod input_routing_tests { /// all. /// /// The two seam rows cannot see this: mutating - /// `middle_click_paste_source` or replacing the dispatch arm with a + /// `paste_source_for` or replacing the dispatch arm with a /// no-op leaves both of them green, because each asserts a function /// in isolation rather than the effect the gesture produces. /// @@ -5085,7 +5086,7 @@ mod input_routing_tests { ); } - /// P2 — a button the frontend has no semantics for reaches no body: /// P2 — a button the frontend has no semantics for reaches no body: + /// P2 — a button the frontend has no semantics for reaches no body: /// nothing local, nothing outbound. The counterpart to the routing /// row that calls it claimed-and-dropped. #[test]