A repository's commits list's Server-Timing names its phases: session, each read, render #4036

The commits list now answers with a Server-Timing header naming session, repo, log, refs and render. The earlier check failure was in the environment, not the change. I merged the current main into this branch and re-pushed without touching the code.

• Change: commitsPage in src/web/pages/commits.ts uses notePhase, timePhase and timePhaseAsync from src/core/route_timing.ts, the same way boards-view.ts does. Each read is wrapped in place, so order and errors are unchanged, and nothing new is computed for the header. render covers page assembly, on both the empty and the populated path.
• Test: test/commits-timing.test.ts requests the page and asserts each phase name appears with a duration. It would fail without the change. docs/CACHING.md notes the phases.
• Unchanged: the body, status codes and caching.
• Checked: bun test on the new test and on test/web-code.test.ts passes. I did not run the type check here: bun install could not write to the temp directory, so tsc was not available. I left the whole check to the forge.
• Not done: the not-found path (loadRepo returns nothing) emits only session and repo, since it reads and renders nothing else.

To try it, run curl -sI <app>/<owner>/<repo>/commits and read the Server-Timing line.

🤖 Generated with Claude Code

Closes #3277

The commits list now answers with a Server-Timing header naming session, repo, log, refs and render. The earlier check failure was in the environment, not the change. I merged the current main into this branch and re-pushed without touching the code.

• Change: commitsPage in src/web/pages/commits.ts uses notePhase, timePhase and timePhaseAsync from src/core/route_timing.ts, the same way boards-view.ts does. Each read is wrapped in place, so order and errors are unchanged, and nothing new is computed for the header. render covers page assembly, on both the empty and the populated path.
• Test: test/commits-timing.test.ts requests the page and asserts each phase name appears with a duration. It would fail without the change. docs/CACHING.md notes the phases.
• Unchanged: the body, status codes and caching.
• Checked: bun test on the new test and on test/web-code.test.ts passes. I did not run the type check here: bun install could not write to the temp directory, so tsc was not available. I left the whole check to the forge.
• Not done: the not-found path (loadRepo returns nothing) emits only session and repo, since it reads and renders nothing else.

To try it, run curl -sI <app>/<owner>/<repo>/commits and read the Server-Timing line.

🤖 Generated with Claude Code