[jayminwest]

Ports the plan pl-10db track-1 defect fixes that were closed as "done on ui/polish-overhaul" but never reached main, because PR #1286 was closed unmerged. warren-e9cd was already ported in #1357. Each defect follows the repro-first convention in AGENTS.md ("UI bugs: reproduction first"). The repro is committed red, and the minimal fix is committed green without the polish branch's wider restyle.

The golden changes need human baseline review. Once #1356 (warren-4780) lands, they need the ui-baseline-approved label. This PR carries no-automerge so it waits for that review.

Per defect

Seed	Repro	Red	Green	Already fixed on main?
warren-40f1	scripts/ui-visual/repros/warren-40f1.pw.ts	n/a, green on main (e1cbc71)	n/a	Yes. warren-99e1 (#1350) moved the guard to src/ui/public/theme-init.js
warren-dac1	scripts/ui-visual/repros/warren-dac1.pw.ts	3b6d0b9	fc27339	No
warren-3de0	scripts/ui-visual/repros/warren-3de0.pw.ts	3f03c45	50e5ec7	No
warren-c023	scripts/ui-visual/repros/warren-c023.pw.ts	00a50f4	7308f23	No
warren-6bca	scripts/ui-visual/repros/warren-6bca.pw.ts	f1357ec	55d9b27	No

Goldens: 634ce8c

warren-40f1: CSP blocked the inline FOUC theme script (already fixed)

The repro checks three things. The SPA document carries script-src 'self'. No securitypolicyviolation event fires. <html data-theme> already matches the stored theme when <body> is parsed, before React mounts. It is green on main because #1350 already shipped the fix. I confirmed it is a real regression guard by restoring the pre-#1350 inline-script index.html locally. It then fails:

Error: no CSP violations
+   "script-src-elem inline",

warren-dac1: hollow stretched cards

A card sized to its content keeps the same height when pinned to align-self: flex-start. The repro measures that difference.

Error: Services stretched below its content     Expected: <= 1  Received: 75.5
Error: Child walk stretched below its content   Expected: <= 1  Received: 280

Fix: md:items-start on the Operations services/lifecycle row and lg:items-start on the plan-run-detail body row. I ran a throwaway rendered audit of every desktop page. It found no other hollow card in run-detail, project-detail or dispatch. Telemetry > Delivery "Stage timings" stretches by 50px under an explicit flex-1. It is outside this seed and was left alone.

warren-3de0: phone layout

Runs at 393px and 360px.

project-detail (31-char git URL, served via page.route) @360:  Error: both actions on one row  Received: 43
project-detail (fixture URL) @393:  Error: the actions are right-aligned  Received: 174.8
dispatch:  Error: Model starts below Agent  Expected: >= 601  Received: 557
runs:      Error: left edges agree  Expected: <= 0.5  Received: 14

Fix: below sm, the project header gives the git URL its own truncating row, and Refresh clone and Delete share one right-aligned group. Dispatch stacks Agent, Model and Provider below sm. The mobile runs card and its footer drop their extra 14px inset. Desktop is unchanged.

warren-c023: implementation-note captions and bottom-nav numbering

dispatch.desktop.light: Error: implementation notes on screen
+ "PROJECT CONTEXT APPENDED", "ENFORCED FROM LIVE USAGE EVENTS", "WEAKEST:", "TIMEOUT API", "FREE TEXT"
operations.phone.light bottom nav: - "04", "05"  + "06", "··"

Where	Before	After
Operations, active workloads (md+)	NEWEST-RUNS WINDOW · FALLBACK POLL	FROM THE NEWEST RUNS
Operations, active workloads (<md)	LIVE · REFRESHED 1S AGO	UPDATED 1S AGO
Operations, events panel (no seq range)	NEWEST FIRST · BOUNDED POLL	NEWEST FIRST
Dispatch, prompt	N CHARACTERS · PROJECT CONTEXT APPENDED AT DISPATCH	N CHARACTERS
Dispatch, cost cap	ENFORCED FROM LIVE USAGE EVENTS · WEAKEST: PROJECT DEFAULT	THE RUN STOPS AT THIS SPEND
Dispatch and plan dispatch, provider/model (no default)	OVERRIDE · FREE TEXT	OPTIONAL
Dispatch and plan dispatch, Timeout	disabled select + NO PER-RUN TIMEOUT API YET	removed (no API on main)
Plan dispatch, cost cap per child	ENFORCED FROM LIVE USAGE EVENTS	EACH CHILD RUN STOPS AT THIS SPEND
Plan dispatch, prompt template	{seed_id} IS SUBSTITUTED PER CHILD · PROJECT CONTEXT APPENDED AT DISPATCH	{seed_id} IS SUBSTITUTED PER CHILD
Plan dispatch, children summary	SERVER WALKS THE PLAN'S CHILD ORDER	CHILDREN RUN IN PLAN ORDER
Project detail, Ready plans	UNBLOCKED ONLY	removed

The phone bottom nav now numbers its own tabs 01-05 (Telemetry 04, More 05). The desktop sidebar keeps its indices. The raw-form-element allowlist drops by one in each dispatch form, because the two Timeout selects are gone.

warren-6bca: telemetry deep links on phone

telemetry-behavior.phone.light: Error: the deep link survives
Expected: "#/telemetry/behavior"   Received: "#/telemetry"

The same failure shows on all four tabs. Desktop passes.

Fix: the viewport-width redirect and the stacked phone arm are gone. The tab strip now shows at every width and scrolls sideways on a phone. use-is-desktop.ts had no other caller and is deleted. src/ui-tests/telemetry-deep-links.test.ts is ported from the polish branch as a source guard.

Golden changes (need human review)

These were regenerated in CI with update_goldens=true on 55d9b27 (run 37812981630) and committed as-is:

• operations desktop light/dark: Services card shorter (dac1), new caption (c023). Phone: UPDATED caption and bottom nav.
• plan-run-detail desktop light/dark: Child walk ends at its rows (dac1).
• dispatch and dispatch-plan, all three: captions and no Timeout field (c023). Phone: Agent and Model stack (3de0).
• project-detail desktop: Ready plans caption removed (c023). Phone: header actions on one right-aligned row (3de0).
• runs phone: the card sits on the page gutter (3de0).
• telemetry-loop / -behavior / -judge / -economics phone: show the linked tab alone instead of four stacked tabs (6bca), so the pages are much shorter.
• every other phone page (agents, events, instance, plan-runs, projects, run-detail, setup): bottom nav ordinals only (about 107 px).
• run-detail desktop, events desktop dark, telemetry desktop: 25 px or fewer differ. This is render noise with no intended change.

Validation

• Each repro was run with bun run check:ui-visual --build repros/<id>.pw.ts, red before its fix and green after.
• A full local bun run check:ui-visual passed: 179 tests (smoke, a11y, every repro). Goldens are skipped locally.
• bun run check:all passed through the pre-commit hook on every commit.
• bun run check:ui-goldens passed.