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

/explore's Server-Timing names its phases: session, rank, render #3230

shipped production is serving it opened this has it
opened this

/explore (src/web/studio/explore.ts, explorePage) is the ranked page P2 is built around. Its Server-Timing has only the route total. The tree, blob and /map pages already name their phases with notePhase from src/core/route_timing.ts.

Done looks like:

  • [ ] explorePage notes session (takeSessionTime), rank (the await explore(...) call in actions/explore.ts) and render (building the page). It follows the pattern in src/web/pages/repo.ts near line 1148.
  • [ ] A test in test/explore.test.ts, or a new test/explore-timing.test.ts, GETs /explore and /explore?q=kind:app sort:forks. It asserts that the Server-Timing header names session, rank and render, each with a numeric dur, and that the route total is still there. The test fails without the change.
  • [ ] bun test test/explore.test.ts test/route-timing.test.ts passes.

Files: src/web/studio/explore.ts, a test. Risk: none to output. Phase names must stay the bare tokens Server-Timing allows.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs