Skip to content

docs(spec): backfill DEC-070/071/072 - #56

Merged
omar-nahhas merged 1 commit into
mainfrom
docs/backfill-dec-070-072
Aug 5, 2026
Merged

omar-nahhas merged 1 commit into
mainfrom
docs/backfill-dec-070-072

Conversation

@omar-nahhas

Copy link
Copy Markdown
Contributor

Closes the C3 gap. All three decisions existed only as code comments, so SPEC.md jumped 069 → 073 and anyone reading the architecture would never find them. All three are load-bearing:

070 red-proof runs under its own 30s test timeout — otherwise 12 scenarios × ~113s ≈ 22m40s. Includes the two guards (coverage parity, cap adequacy) that stop the speed-up silently weakening the gate it optimises
071 the devkit harness image is published, and public on purpose — before it existed every consumer rebuilt inline at ~85s/run, and public is what lets an outside contributor run the local loop with no GHCR auth
072 post-login landmark wait is explicitly 30s: the WebUI hydrates ~5s after the redirect, straddling Playwright's default and flaking ~50% (measured — landmark at t=7.755s vs a ~7.4s deadline)

SPEC.md now documents an unbroken DEC-060 → DEC-079.

The numbering note is rewritten to record the lesson rather than the debt.

All three existed only as code comments, so they were invisible to anyone
reading the SPEC — and all three are load-bearing:

  070  the red-proof gate runs under its own 30s test timeout (12 scenarios x
       ~113s = 22m40s otherwise), with coverage-parity and cap-adequacy guards
       so the speed-up cannot silently weaken the gate it optimises
  071  the devkit harness image is published, and PUBLIC on purpose: before it
       existed every consumer rebuilt it inline at ~85s per run, and a public
       image is what lets an outside contributor run the local loop with no
       GHCR auth at all
  072  the post-login landmark wait is explicitly 30s because the WebUI hydrates
       ~5s AFTER the redirect, straddling Playwright default and flaking ~50%
       (measured: landmark at t=7.755s vs a ~7.4s deadline)

The numbering note now records the lesson rather than the debt: a decision that
lives only in a code comment is one nobody reviewing the architecture will find.
@omar-nahhas
omar-nahhas merged commit bfef261 into main Aug 5, 2026
1 check passed
@omar-nahhas
omar-nahhas deleted the docs/backfill-dec-070-072 branch August 5, 2026 00:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant