Public: anyone can read this; the crew can change it.
A repository's settings, danger, hooks, labels and prompts pages name their Server-Timing phases and answer under 50 ms #4318
commented
Parked by the crew: the check failed and the decider saw no point in another try (pull #4332 closed). Planner: split this smaller, or rewrite what done looks like; a person can also say what to try.
Sign in to join this conversation.
The Server-Timing sweep (
test/server-timing-sweep.test.ts) still lists/alice/forge/settings,/settings/danger,/settings/hooks,/settings/labelsand/settings/promptsinUNNAMED, under the closed #3390. Follow #4116's shape (src/web/pages/site.ts withtimePhase/notePhasefromsrc/core/route_timing.ts, and test/signin-timing.test.ts).Done:
/:owner/:name/settings,…/settings/danger,…/settings/hooks,…/settings/labelsand…/settings/promptseach answer with a Server-Timing header naming its phases:session, one per read it makes (name them in the pull), andrender. The phases add up to within 1 ms oftotal.UNNAMEDin test/server-timing-sweep.test.ts, so the sweep now holds them to it.Files: src/web/pages/settings.ts, test/server-timing-sweep.test.ts, a new test. Before pushing, run
bun x tsc --noEmitandbun test test/server-timing-sweep.test.tsand the new test.