Public: anyone can read this; the crew can change it.

A repository's app, answers, now and state pages name their Server-Timing phases and answer under 50 ms #4320

shipped production is serving it opened this has it
opened this

The Server-Timing sweep (test/server-timing-sweep.test.ts) still lists /alice/forge/app, /app/answers, /app/now and /app/state in UNNAMED, under the closed #3390. Follow #4116's shape (src/web/pages/site.ts with timePhase / notePhase from src/core/route_timing.ts, and test/signin-timing.test.ts).

Done:

  • [ ] /:owner/:name/app, …/app/answers, …/app/now and …/app/state each answer with a Server-Timing header naming its phases: session, one per read it makes (name them in the pull), and render. The phases add up to within 1 ms of total.
  • [ ] Those routes leave UNNAMED in test/server-timing-sweep.test.ts, so the sweep now holds them to it.
  • [ ] A new test asserts the phase names for each route, signed in, and that each answers under 50 ms warm on the test fixture.
  • [ ] The HTML is byte-for-byte the same as before (the phases are measurement, not change).

Files: the page modules that draw them under src/web/pages/ and src/web/studio/, test/server-timing-sweep.test.ts, a new test. Before pushing, run bun x tsc --noEmit and bun test test/server-timing-sweep.test.ts and the new test.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs