Raise script/test's per-package timeout to 90s (closes #194)

`go test -timeout` is per package, so the budget has to clear the slowest
single package: internal/handlers. Measured in a cache-defeated builder
stage on the 48-core shared build host, 2026-08-18:

  16.9s                  host load 5-20, GOMAXPROCS 48
  45.9s / 47.3s / 49.0s  three runs at deliberate host load 31-73
  30.6s / 39.7s          host load 5-20, GOMAXPROCS 6 / 4
  67.3s / 97.5s          host load 5-20, GOMAXPROCS 2 / 1
  67.3s                  GOMAXPROCS 4 at deliberate host load 52-68

The old 30s budget was breached by every loaded run and by every
GOMAXPROCS at or below 6. At GOMAXPROCS 4 it failed outright on plain
33e4fa4 with no other change ("panic: test timed out after 30s"); with
90s the same run passes at 41.816s.

90s is the org-wide backstop in REPO_POLICIES.md. It holds against these
measurements: the worst case under native parallelism is 49.0s, and the
compound GOMAXPROCS-4-under-load case at 67.3s sits at 75% of it. The
rows above 90s come from GOMAXPROCS 1 and 2, a synthetic core floor
rather than a condition CI runs under.

No test is changed, skipped, shortened or desampled.

#194

TODO.md

@@ -24,84 +24,36 @@ event retention (#63), the database archiving target (#43), the admin
 password change flow (#65), policy compliance (#6), pinned lint tooling
 (#55), and fail-loud configuration parsing (#80).
 
-`next` holds the **complete 1.0.0 milestone**: every issue in it is
+`next` holds the 1.0.0 milestone less its final four issues (#176, #178,
-closed, and it is verified green both by CI and by cache-defeated
+#186, #187 — all in review or held on merge order), and is verified
-container runs (`docker build --no-cache-filter=lint
+green by cache-defeated container runs
---no-cache-filter=builder`).
+(`docker build --no-cache-filter=lint --no-cache-filter=builder`). The
-
+CI status is not independently claimed here: a superseded run is
-One caveat on reading a green check, narrower than it used to be. A
+recorded as `skipped` and still rolls up green, so a commit status on
-docs-only commit deliberately replays from the layer cache (#119), so a
+`next` does not by itself evidence an executed check (#152). Before
-green status on such a commit evidences a replay rather than an executed
+#119, a warm layer cache also let the gate report success without
-run; a code commit invalidates the `COPY` layer and genuinely executes.
+executing anything, and replayed the previous build's console log so
-Superseded runs are no longer the hazard they were: before #152 they
+the lie looked like a real run. Note: `TODO.md` was deliberately
-were recorded as `skipped` and rolled up green, and before #119 a warm
-layer cache let the gate report success without executing anything,
-replaying the previous build's console log so the lie looked like a real
-run. Both are fixed. Note: `TODO.md` was deliberately
 deleted from this repo in f9a9569 (2026-03-01, #6); its content was
 folded into the README TODO section, which this draft reconstructs as
 of 2026-07-06.
 
 # Next Step
 
-Merge the milestone PR (#111) to `main` and tag 1.0.0 from it. The
+Land the last four 1.0.0 issues, then merge the milestone PR to `main`
-milestone is empty and `next` is green; nothing else blocks the tag.
+and tag 1.0.0 from it. Merge order is forced by a real conflict on
+`README.md` and `internal/middleware/middleware.go`: #186, then #176,
+then #178, then #187.
 
-Three items belong to the owner, none of them blocking. #150 was decided
+Two items belong to the owner, neither blocking the tag. #150 was
-by the manager rather than left to stall the queue and is flagged on the
+decided by the manager rather than left to stall the queue and is
-issue for reversal if that call was wrong. #112 (whether `Completed
+flagged on the issue for reversal if that call was wrong. #112 (whether
-Steps` should exist at all, given it once conflicted on every unit) is
+`Completed Steps` should exist at all, given it once conflicted on every
-unanswered; the provisional ruling in force is that issue branches do
+unit) is unanswered; the provisional ruling in force is that issue
-not touch this file. #198 records that `make test` is past the org 20s
+branches do not touch this file.
-target — 46s of test execution inside a 62.8s CI layer — and turns on
-which quantity the 60s hard cap governs; it is scoped as the improvement
-bug the 20-60s band requires, and should be milestoned instead if the
-cap is read as covering the whole invocation.
-
-After the tag, the largest open cluster is the unmilestoned follow-up
-backlog these units generated: #183, #184, #185, #190, #191, #193 and
-#198.
 
 # Completed Steps
 
-- 2026-08-18 Raise `script/test`'s per-package timeout from 30s to 90s,
-  matching the org-wide backstop. `go test` applies `-timeout` per
-  package, and `internal/handlers` had grown past the old budget: a
-  cache-defeated build failed outright at `GOMAXPROCS=4`, and every run
-  under deliberate host load breached 30s. The measurement table lives
-  in the script (#194)
-- 2026-08-18 Re-sync `REPO_POLICIES.md` from `prompts`. The local copy
-  was stale and still mandated a 20s test target with a 30s timeout,
-  which the org replaced with a 60s cap and a 90s backstop. A synced
-  copy is not a source; reading it as one nearly produced a PR against
-  `prompts` proposing a change already merged there (#196)
-- 2026-08-18 Report handler panics through the logger and answer 500.
-  chi v1.5.5's `Recoverer` scans for a `panic(0x` frame the runtime no
-  longer emits, then indexes `pkg[-1:]`, so it panicked inside its own
-  stack printer before writing a byte: the recovery never ran, the
-  client got a dropped connection instead of a 500, and the original
-  panic was lost. A local middleware replaces it, bounded by
-  `MaxPanicLogLineBytes` (#187)
-- 2026-08-18 Route GORM's logger through `slog` and bound it. Every
-  `gorm.Open` left `logger.Default` in place at `Warn` with
-  `IgnoreRecordNotFoundError` false, so **every record-not-found
-  printed the fully interpolated SQL to stdout** — including the
-  client-chosen path on `/webhook/{uuid}` and the submitted username on
-  the login form, at no level the operator set and outside
-  `internal/logger` entirely. Three call sites, not the two the issue
-  named (#178)
-- 2026-08-18 Bound every `slog` line against client-chosen text. Eight
-  sites reachable unauthenticated, found by reading every `slog` call in
-  the tree rather than only the one reported; the budget moved to a
-  shared `internal/logfield` so no second truncation exists. `DEBUG`
-  being off by default is not a bound and is not treated as one (#176)
-- 2026-08-18 Stop a slow host turning a login-guard test into a
-  segfault. A non-fatal `assert` on an acquire result was dereferenced
-  on the next line, so one timing miss killed the whole
-  `internal/middleware` binary and reddened CI for unrelated PRs. The
-  fix also removed a real production race — `acquire` could shed a
-  request with a slot standing free, because Go picks uniformly among
-  ready `select` cases (#186)
 - 2026-08-18 Send the chi route pattern to Sentry rather than the
   concrete path. The receiver's path carries the entrypoint capability
   token, so every Sentry event from `/webhook/{uuid}` shipped a live

script/test

@@ -4,7 +4,7 @@
 # -timeout is applied by `go test` per package, not to the run as a whole, so
 # it only has to clear the slowest single package. That is internal/handlers,
 # measured in a cache-defeated builder stage on the 48-core shared build host
-# (2026-08-18); load- and host-dependent, not invariants:
+# (2026-08-18, this tree):
 #
 #   16.9s                  host load 5-20, GOMAXPROCS 48
 #   45.9s / 47.3s / 49.0s  three runs at deliberate host load 31-73
@@ -19,9 +19,9 @@
 # 90s matches the org-wide backstop in REPO_POLICIES.md and is sized here
 # against the figures above: the worst case under native parallelism is 49.0s,
 # and the compound GOMAXPROCS-4-under-load case at 67.3s sits at 75% of it.
-# The one figure above 90s is GOMAXPROCS 1, a synthetic core floor rather than
+# The rows above 90s come from GOMAXPROCS 1 and 2, a synthetic core floor
-# a condition CI runs under. If a CPU-limited runner ever puts a real run near
+# rather than a condition CI runs under. If a CPU-limited runner ever puts a
-# 67s, that is the datum to revisit the org figure with.
+# real run near 67s, that is the datum to revisit the org figure with.
 set -eu
 
 ROOT="$(cd "$(dirname "$0")/.." && pwd -P)"