For: speed you can see, page by page. The new pull page (newPullPage in src/web/pages/pulls.ts, GET /:owner/:name/pulls/new) reports one opaque number in Server-Timing.
Done:
[ ] The handler calls notePhase from src/core/route_timing.ts, the way src/web/pages/releases.ts and src/web/studio/explore.ts do. The phases are session, compare (the branch list and the base…head comparison), and render.
[ ] The page renders exactly as today.
[ ] A new test/new-pull-timing.test.ts GETs the page and finds every phase name in Server-Timing. It fails without the change.
Files: src/web/pages/pulls.ts, test/new-pull-timing.test.ts.
Risk: none to behaviour. Measure only; do not add caching here.
For: speed you can see, page by page. The new pull page (
newPullPageinsrc/web/pages/pulls.ts,GET /:owner/:name/pulls/new) reports one opaque number inServer-Timing.Done:
notePhasefromsrc/core/route_timing.ts, the waysrc/web/pages/releases.tsandsrc/web/studio/explore.tsdo. The phases aresession,compare(the branch list and the base…head comparison), andrender.test/new-pull-timing.test.tsGETs the page and finds every phase name inServer-Timing. It fails without the change.Files:
src/web/pages/pulls.ts,test/new-pull-timing.test.ts. Risk: none to behaviour. Measure only; do not add caching here.