I3226 — Every UI and Cockpit response says where its time went

○PŘIPRAVENO vlna 0 · p2 · profil implementation · bezpečné souběžně

Part of the-ui-runs-and-explains — The UI runs any capability and explains what needs the owner, whose version it shows and where its time went.

Cíl

Write a Server-Timing header on every /ui, /cockpit and /api response from a request-scoped recorder installed in Router::handle and fed by ui/source.rs's capability reads (with cache hit or miss) and by the executor, named in the perf::Phase vocabulary plus the route, render and bytes stages this issue adds, and let the bench record it per route so a budget miss names its stage.

Proč

Owner, 2026-10-11: majordomus ui must be blazing fast; the 2026-10-11 and 2026-10-13 plans both put per-stage measurement before optimisation. No response carries a timing today. perf.rs holds process-global AtomicU64 totals over eight phases and has no per-request record, and the server is multi-threaded, so the header needs its own recorder. Sources: chatgpt-majordomus:c223#0, chatgpt-majordomus:c223#2, chatgpt-majordomus:c255#7.

Aktuální stav

crate::perf::COUNTERS (perf.rs) is one process-global set of AtomicU64 totals over eight phases (repository_discovery, index_build, registry_build, open_api_build, mcp_projection_build, graph_build, handler_execution, cache_lookup), with no route, render or bytes phase and no per-request record; the HTTP server runs several worker threads; capability/builtin/perf.rs reports those totals; no Server-Timing in apps/ or share/; I2489 holds budgets; I2828 (branch) caches the Cockpit overview.

Cílový stav

The header is recorded per request in the same Phase vocabulary perf.counters reports, written by the router, partial for streamed and upgraded responses, absent from published snapshots, and explained to a reader in docs/MAJORDOMUS_UI.md.

Rozsah

• apps/majordomus-cli/src/http/router.rs
• apps/majordomus-cli/src/http/server.rs
• apps/majordomus-cli/src/capability/builtin/perf.rs
• docs/MAJORDOMUS_UI.md
• apps/majordomus-cli/src/perf.rs
• apps/majordomus-cli/src/ui/source.rs
• apps/majordomus-cli/src/capability/executor.rs
• .ai/repo/project/intents

Mimo rozsah

• Browser paint timing
• Optimising any route

Závislosti

Žádné. Tento úkol je kořenem grafu.

Co na tom čeká

• I3227✕BLOCKED A UI link a reader is about to follow is fetched ahead when the route is cheap and the connection allows it

Kritéria přijetí

• A served /ui response carries Server-Timing entries whose sum is within the measured total
• A repeated capability read on one request is visible as two entries
• docs/MAJORDOMUS_UI.md explains to a reader what each Server-Timing entry names and how to read it in a browser's network panel
• A request-scoped recorder is installed in Router::handle and fed by ui/source.rs's reads and by the executor; global perf::COUNTERS are not subtracted around a request; two concurrent requests in the test do not see each other's entries
• Entry names use the perf::Phase vocabulary, plus the route, render and bytes stages this issue adds to it
• A WebSocket upgrade (/events) or a streamed response carries only the stages completed before its headers, marked partial and without a total; documented in docs/MAJORDOMUS_UI.md
• The first commit adds apps/majordomus-cli/tests/http_server_timing.rs and rewrites criterion a-response-says-where-its-time-went to `evidence: test` with that ref, worded 'in the same Phase vocabulary perf.counters reports'; `cargo test --manifest-path apps/majordomus-cli/Cargo.toml --test http_server_timing` passes

Validace

• cargo test --manifest-path apps/majordomus-cli/Cargo.toml --lib http::
• majordomus bench --check
• cargo test --manifest-path apps/majordomus-cli/Cargo.toml --test http_server_timing

Požadované důkazy

• apps/majordomus-cli/tests/http_server_timing.rs

Důkazy

Nic nezaznamenáno. Za každým tokenem výše musí stát příkaz nebo artefakt, než lze tento úkol dokončit; vyprávění se odmítá.

Časová osa

zahájeno

—

ověřeno

—

dokončeno

—

Tato tři pole, důkazy výše a stav závislostí jsou vše, z čeho se stav skládá. Neexistuje žádné pole stavu, které by s nimi mohlo být v rozporu.

Canonical record: .ai/repo/project/issues/I3226.yaml. Read it back with majordomus plan show I3226.