Bump quadraui pin to HEAD (42 commits) and fix the 4 removed-API call sites — next bump does not compile

[JDonaghy]

Problem

Cargo.toml pins quadraui at 42e0f8fa55bd2f9f94bb77f612d70355a88c9356 (2026-09-03). The authoritative remote develop is 4ff2a645023637bba7229e0aeee5968a9c6f3652 — 42 commits ahead (gh api repos/JDonaghy/quadraui/compare/42e0f8f...4ff2a64 → ahead_by: 42, behind_by: 0).

The next pin bump will not compile. quadraui#704 removed modal_stack_mut and drag_and_modal_mut from the Backend trait. At the true HEAD, quadraui/src/backend.rs has only:

726:    fn modal_stack_handle(&self) -> Rc<RefCell<ModalStack>>;
731:    fn drag_state_handle(&self) -> Rc<RefCell<DragState>>;

vimcode calls the removed method at four live sites:

• src/tui_main/shell_app.rs:1258 — let (_, modal_stack) = backend.drag_and_modal_mut();
• src/tui_main/shell_app.rs:1541 — let (drag_state, modal_stack) = backend.drag_and_modal_mut();
• src/tui_main/shell_app.rs:8105 — let (_, modal_stack) = backend.drag_and_modal_mut();
• src/tui_main/shell_app.rs:8181 — let (_, modal_stack) = backend.drag_and_modal_mut();

Stale doc references to fix while here: src/tui_main/shell_app.rs:171, :174, :1209, :2470.

Fix

1. Rewrite the four call sites against the handle API. drag_state_handle() / modal_stack_handle() return Rc<RefCell<_>>, so the two-at-once borrow that drag_and_modal_mut existed to provide is no longer needed — take two handles and borrow_mut() each.
2. Bump both rev entries in Cargo.toml (the quadraui dependency at :90 and the [patch.crates-io] vt100 entry at :152 — they must match) to 4ff2a645023637bba7229e0aeee5968a9c6f3652.
3. Run cargo test to relock Cargo.lock and re-run snapshots against the new rev.

Why this is first in the chain

The bump delivers infrastructure that several queued issues depend on:

• quadraui#705 — CommandLineLayout::hit_test / selection_bounds (596d8c8). Required by the GTK half of Status-bar / command-line messages aren't mouse-selectable (GTK can't; TUI has offset bug) #194.
• quadraui#703 — wide_glyph_x_scale for macos/win (d23bea5).
• quadraui#741 — shared text-selection state machine (887436e).
• Palette effective_offset guard lifted into the primitive (0f1a958); vimcode has its own copy at src/render.rs:2410-2450.

Acceptance

• cargo build, cargo test, cargo clippy -- -D warnings, cargo fmt all pass on the new rev.
• vimcode --version reports the new rev.
• Pure-refactor + dependency bump: no user-visible behaviour change, so no new black-box test is required — say so explicitly in the PR per CLAUDE.md's acceptance bar.

[JDonaghy]

Correction: do NOT bump to the SHA named in the body

This issue's body names 4ff2a645023637bba7229e0aeee5968a9c6f3652. That SHA is already stale and will be staler at pickup. quadraui HEAD has moved to ece5d5d (44 ahead of vimcode's pin) since this was filed, and three more issues are landing into it right now.

Bump to quadraui origin/develop HEAD as resolved at pickup time, not to any SHA typed into this issue. Read it with:

gh api repos/JDonaghy/quadraui/git/ref/heads/develop --jq '.object.sha'

The bump must include these three

Filed from the 2026-09-04 platform-neutrality audit and queued as a parallel quadraui chain. All three are prerequisites for vimcode work queued behind this issue:

quadraui	What it delivers	Status
#774	presence-gated z-ordered frame composition + paint/hit-test order invariant	✅ merged
#775	Theme::from_vscode_json + Color maths	✅ merged
#776	MinimapSizing::VsCodeParity + Backend::scrollbar_reserve	🔄 running

A --after quadraui#776 edge has been added to this queue entry, so the drive will not dispatch this issue until #776 lands. Verify all three commits are ancestors of the rev you pin before opening the PR:

git -C ~/src/quadraui merge-base --is-ancestor <fix-sha> <pinned-rev>

Everything else in the body stands — in particular the four drag_and_modal_mut call sites in src/tui_main/shell_app.rs still have to move to the handle API, or the bump does not compile.

[JDonaghy]

Before you start: re-verify the citations, do not trust them

Every file:line in this issue was measured on develop @ ee26268 (2026-09-04) by an audit, not by reading the tree you will check out.

They will have drifted by the time you pick this up. This issue sits in a serialised chain, and several entries ahead of it move code on purpose — #812 deletes ~1,300 lines from src/render.rs, #813 retypes fields in src/app.rs, #785 and #814 delete whole functions. Anything filed after those shifts.

So:

• Locate by symbol, not by line. grep -n "fn build_screen_layout_with_breadcrumb_row" src/render.rs, not "go to line 12886". GOALS.md already carries this rule for the same reason — it cites no line numbers on purpose.
• Re-measure any count before you use it. Line totals here came from python3 scripts/prod_lines.py and scripts/native_lines.py at a named revision. Re-run them rather than quoting the number in this issue. GOALS.md: "regenerate them with the script rather than trusting the table."
• Size the branch from what you find, not from the figure here. The counts are evidence that a problem is large, not estimates of a diff.

If a finding does not reproduce, say so — that is a useful result

This issue was generated from an automated audit. Some of its claims are wrong. Today alone, two confident findings in the same audit chain did not survive checking: a "1,671 byte-identical lines" figure measured out at ~85, and a "zero commits of pin lag" reading was taken against a stale local ref and was actually 42.

If the problem described here is already fixed, was never real, or is real but different from the description: comment with what you actually found and stop. Do not implement around a bad premise to make the issue closeable. A correction is worth more than a merged PR built on a wrong claim.

[JDonaghy]

Coordinator Assignment

Machine: dell64
Repo: vimcode
Files: (unspecified)
Do not touch: (nothing else in flight)

Briefing

Issue #811: Bump quadraui pin to HEAD (42 commits) and fix the 4 removed-API call sites — next bump does not compile

Problem

Cargo.toml pins quadraui at 42e0f8fa55bd2f9f94bb77f612d70355a88c9356 (2026-09-03). The authoritative remote develop is 4ff2a645023637bba7229e0aeee5968a9c6f3652 — 42 commits ahead (gh api repos/JDonaghy/quadraui/compare/42e0f8f...4ff2a64 → ahead_by: 42, behind_by: 0).

The next pin bump will not compile. quadraui#704 removed modal_stack_mut and drag_and_modal_mut from the Backend trait. At the true HEAD, quadraui/src/backend.rs has only:

726:    fn modal_stack_handle(&self) -> Rc<RefCell<ModalStack>>;
731:    fn drag_state_handle(&self) -> Rc<RefCell<DragState>>;

vimcode calls the removed method at four live sites:

• src/tui_main/shell_app.rs:1258 — let (_, modal_stack) = backend.drag_and_modal_mut();
• src/tui_main/shell_app.rs:1541 — let (drag_state, modal_stack) = backend.drag_and_modal_mut();
• src/tui_main/shell_app.rs:8105 — let (_, modal_stack) = backend.drag_and_modal_mut();
• src/tui_main/shell_app.rs:8181 — let (_, modal_stack) = backend.drag_and_modal_mut();

Stale doc references to fix while here: src/tui_main/shell_app.rs:171, :174, :1209, :2470.

Fix

1. Rewrite the four call sites against the handle API. drag_state_handle() / modal_stack_handle() return Rc<RefCell<_>>, so the two-at-once borrow that drag_and_modal_mut existed to provide is no longer needed — take two handles and borrow_mut() each.
2. Bump both rev entries in Cargo.toml (the quadraui dependency at :90 and the [patch.crates-io] vt100 entry at :152 — they must match) to 4ff2a645023637bba7229e0aeee5968a9c6f3652.
3. Run cargo test to relock Cargo.lock and re-run snapshots against the new rev.

Why this is first in the chain

The bump delivers infrastructure that several queued issues depend on:

• quadraui#705 — CommandLineLayout::hit_test / selection_bounds (596d8c8). Required by the GTK half of Status-bar / command-line messages aren't mouse-selectable (GTK can't; TUI has offset bug) #194.
• quadraui#703 — wide_glyph_x_scale for macos/win (d23bea5).
• quadraui#741 — shared text-selection state machine (887436e).
• Palette effective_offset guard lifted into the primitive (0f1a958); vimcode has its own copy at src/render.rs:2410-2450.

Acceptance

• cargo build, cargo test, cargo clippy -- -D warnings, cargo fmt all pass on the new rev.
• vimcode --version reports the new rev.
• Pure-refactor + dependency bump: no user-visible behaviour change, so no new black-box test is required — say so explicitly in the PR per CLAUDE.md's acceptance bar.

[JDonaghy]

Coordinator: Assignment Complete

Machine: dell64
Status: done
Exit code: 0
Duration: 7m 13s
Log: /home/john/.coord/logs/925f41f63b4b.log

[JDonaghy]

Coordinator: Assignment Complete

Machine: dell64
Status: done
Exit code: 0
Duration: 3m 46s
Log: /home/john/.coord/logs/5cd909a91bdd.log