small: the page-load budget lives in `src/perf/page_budget.ts` as plain data #11366

The page-load budget table now lives in src/perf/page_budget.ts as plain data. Adding a page is one line there, and a production probe can import the same table.

• src/perf/page_budget.ts (new): the PageBudget interface and PAGE_BUDGETS (page, pattern, medianMs, statements). The table and every value are unchanged.
• test/helpers/page_budget.ts: keeps only the judging code (median, PageMeasure, countFinding, timeFinding). It re-exports PAGE_BUDGETS and PageBudget from src/perf/, so the test's imports didn't change.
• test/page-budget.test.ts: only its header comment changed, to point at the new location.

I didn't wire up a production probe to read the table; this change just makes that possible. The keep_plan tool wasn't available in this run, so I kept no plan on the issue.

To try it: bun test test/page-budget.test.ts passes (5 tests). bun scripts/gate.ts and bun x tsc --noEmit both pass.

🤖 Generated with Claude Code

Closes #11360

The page-load budget table now lives in src/perf/page_budget.ts as plain data. Adding a page is one line there, and a production probe can import the same table.

• src/perf/page_budget.ts (new): the PageBudget interface and PAGE_BUDGETS (page, pattern, medianMs, statements). The table and every value are unchanged.
• test/helpers/page_budget.ts: keeps only the judging code (median, PageMeasure, countFinding, timeFinding). It re-exports PAGE_BUDGETS and PageBudget from src/perf/, so the test's imports didn't change.
• test/page-budget.test.ts: only its header comment changed, to point at the new location.

I didn't wire up a production probe to read the table; this change just makes that possible. The keep_plan tool wasn't available in this run, so I kept no plan on the issue.

To try it: bun test test/page-budget.test.ts passes (5 tests). bun scripts/gate.ts and bun x tsc --noEmit both pass.

🤖 Generated with Claude Code

The page-load budget table now lives in src/perf/page_budget.ts as plain data, so adding a page is one line there. Something outside the tests now reads it, which clears the "imported by nothing under src/" check. No budget value changes.

• src/perf/page_budget.ts: holds PageBudget and PAGE_BUDGETS, moved unchanged from test/helpers/page_budget.ts. It also has pagePath(budget, owner, name), which fills :owner and :name in a page's path (URL-encoded) and returns the path.
• test/helpers/page_budget.ts: now holds only the judging code (median, PageMeasure, countFinding, timeFinding) and re-exports the table. test/page-budget.test.ts builds its URLs with pagePath instead of string replaces.
• scripts/page-budget.ts (new): checks a running forge against the same table. For each page it sends one warm-up request, then --runs timed ones, and prints the median beside the budget. It exits 1 when a page misses its budget or doesn't answer 200. Counting SQL statements stays with the in-process test.
• test/page-budget-path.test.ts (new): covers how pagePath fills and encodes a path.

The other way to clear the check was adding the module to scripts/unreached-allow.json. I didn't, because that list is meant only to shrink. The script doesn't sign in, so it times pages as a signed-out visitor sees them.

Try it with bun test test/page-budget.test.ts test/page-budget-path.test.ts test/unreached.test.ts (10 pass), or bun scripts/page-budget.ts <base-url> --repo owner/name [--runs 10]. bun scripts/gate.ts and bun x tsc --noEmit pass. A run against production every page over budget, but that run doesn't measure page speed. It timed from the sandbox, network included, and signed out, so /map answered 303. trevato/forge doesn't exist on the lab app, so the app page answered 404.

🤖 Generated with Claude Code

The page-load budget table now lives in src/perf/page_budget.ts as plain data, and the test helper imports it. Budget values are unchanged.

• src/perf/page_budget.ts: holds PageBudget and PAGE_BUDGETS, moved unchanged from test/helpers/page_budget.ts. It also has pagePath(budget, owner, name), which fills :owner and :name in a page's path (URL-encoded). test/page-budget-path.test.ts covers it.
• test/helpers/page_budget.ts: keeps only the judging code (median, PageMeasure, countFinding, timeFinding) and re-exports the table. test/page-budget.test.ts builds its URLs with pagePath.
• scripts/unreached-allow.json: lists src/perf/page_budget.ts. Without this entry the move fails test/unreached.test.ts, because only tests import the module. A person should decide whether this entry is acceptable when merging. I asked on #11360. The entry has to come off the list once something outside the tests imports the table, because the test fails while a reached module stays listed.

I removed scripts/page-budget.ts as review asked. It was outside the issue's scope, and it measured signed out while the budgets are counted signed in, so it misreported a healthy forge. With the script gone, the "median of 20 warm requests" comment matches the test again. A probe that reads the table against production still needs a session cookie, as noted on the issue.

To check it: bun test test/page-budget.test.ts test/page-budget-path.test.ts test/unreached.test.ts gives 10 passes. bun scripts/gate.ts and bun x tsc --noEmit also pass.

🤖 Generated with Claude Code