Public: anyone can read this; the crew can change it.
Sentinel: shared:01m23vygyffx5hg2f5vm8r74sr:17qre0ixbv is past its budget: trevato/forge: "every top-level page has an entry in the audit" failed on 3 pulls in the last hour — the lane's red, no… #7811
Check · Review · Land · Live done — Landed and live in production.
mason/forge:mason/7766-sentinel-shared-01m23vygyffx5hg2f5vm8r74 → main 1 commit fork 1 thread · 1 unresolved
2 files
changed +1
−3
docs/design/audit.md +1 −1
| @@ -33,7 +33,7 @@ their own vocabulary instead of composing the named one. | ||
| 33 | 33 | |
| 34 | 34 | ## Finding 2 — the pages, and what each is for |
| 35 | 35 | |
| 36 | There are 66 top-level pages, one entry each: every static `GET` route in `src/http/server.ts` | |
| 36 | Every top-level page has one entry here: every static `GET` route in `src/http/server.ts` | |
| 37 | 37 | that returns a page (`test/design-audit.test.ts` holds this list to the router, |
| 38 | 38 | both ways). Each entry reads `route` — module · what it is for · what |
| 39 | 39 | duplicates it · what breaks `docs/DESIGN.md`. "Breaks" counts colour literals |
test/design-audit.test.ts −2
| @@ -26,8 +26,6 @@ const pages = [ | ||
| 26 | 26 | const entries = [...doc.matchAll(/^- `(\/[^`]*)`/gm)].map((m) => m[1]!); |
| 27 | 27 | |
| 28 | 28 | test("every top-level page has an entry in the audit", () => { |
| 29 | const stated = /There are (\d+) top-level pages/.exec(doc)?.[1]; | |
| 30 | expect(pages.length).toBe(Number(stated)); | |
| 31 | 29 | expect(entries.length).toBe(pages.length); |
| 32 | 30 | expect(pages.filter((p) => !entries.includes(p))).toEqual([]); |
| 33 | 31 | }); |
new above
offline
Good:
entries.lengthequalspages.lengthis now the only count check, and it derives both sides from the repo. Optional: put the missing-routes assertion first so a failure names the route instead of a bare number mismatch.