The milestones page's Server-Timing names its phases: session, each read, render #3302

The milestones page now answers with a Server-Timing header naming session, repo, plan, seats and render, in addition to the existing app. bun x tsc --noEmit would not run here (bun couldn't write to its temp dir), so the forge's check is the first type check.

• Phases: src/web/pages/milestones.ts wraps milestonesPage with notePhase/timePhase the way boards-view.ts does.

- session is the session-resolution time the server already measured. - repo is loadRepo. - plan is readPlan together with the owner-facts check it needs. - seats is the move controls (moverFor: the admin check plus seating). - render covers building the body and the page.

• Unchanged: reads run in the same order, and the body, status codes and caching are the same. The not-found path still returns early.
• Test: test/milestones-timing.test.ts signs in, requests /alice/forge/milestones, and asserts each phase name appears with a dur= value. It fails without the change and passes with it.
• Other tests: the existing milestone tests still pass.
• Left out: the single-milestone page (/milestones/:number) and the JSON endpoints, since the issue names only the list page.

To try it: curl -sI <app>/<owner>/<repo>/milestones and read the server-timing header.

🤖 Generated with Claude Code

Closes #3281

The milestones page now answers with a Server-Timing header naming session, repo, plan, seats and render, in addition to the existing app. bun x tsc --noEmit would not run here (bun couldn't write to its temp dir), so the forge's check is the first type check.

• Phases: src/web/pages/milestones.ts wraps milestonesPage with notePhase/timePhase the way boards-view.ts does.

- session is the session-resolution time the server already measured. - repo is loadRepo. - plan is readPlan together with the owner-facts check it needs. - seats is the move controls (moverFor: the admin check plus seating). - render covers building the body and the page.

• Unchanged: reads run in the same order, and the body, status codes and caching are the same. The not-found path still returns early.
• Test: test/milestones-timing.test.ts signs in, requests /alice/forge/milestones, and asserts each phase name appears with a dur= value. It fails without the change and passes with it.
• Other tests: the existing milestone tests still pass.
• Left out: the single-milestone page (/milestones/:number) and the JSON endpoints, since the issue names only the list page.

To try it: curl -sI <app>/<owner>/<repo>/milestones and read the server-timing header.

🤖 Generated with Claude Code