Skip to content

docs(adc): spec and ADR-029 corrections from ADC Local Test Run 3 - #235

Merged
ivanball merged 1 commit into
mainfrom
docs/test-run-3-spec-corrections
Oct 4, 2026
Merged

ivanball merged 1 commit into
mainfrom
docs/test-run-3-spec-corrections

Conversation

@ivanball

@ivanball ivanball commented Oct 4, 2026

Copy link
Copy Markdown
Owner

Spec and ADR corrections from ADC Local Test Run 3, each confirmed against source by an independent reviewer.

  • adc-specifications, BR-213 and the error table: the registration throttle answers 401 Auth.RegistrationRateLimitExceeded (ADR-029's uniform login-protection failure), not 429. The error table gets its own 401 row; BR-213 also states that both UI paths forward the browser IP (MMCA.Common #492).
  • adc-specifications, UC-27 and the scoring endpoint row: a trigger for a run already queued or in progress answers 202 and the handler de-duplicates (it was documented as 409); a missing event answers 404 (ADC #245).
  • ADR-029, Revision 2026-10-03: records the cache-outage decision. Login protection fails open with a Warning log instead of answering 500 (MMCA.Common #492). Rationale: CacheSettings' "an outage never becomes an error" promise, ephemeral counters, and the per-instance auth-ip window that keeps bounding guessing during an outage.

Merge after MMCA.Common #492 and ADC #245, so the docs describe shipped behavior. Merging publishes the site.

🤖 Generated with Claude Code

https://claude.ai/code/session_012sDGvYdTD88ZjjD7if2SMW

- adc-specifications: BR-213 registration throttle answers 401
  Auth.RegistrationRateLimitExceeded (ADR-029), not 429; the error table
  gets its own 401 row. The client IP is the browser's on both UI paths.
- adc-specifications: UC-27 / scoring trigger answers 202 for a run already
  queued (the handler de-duplicates), not 409; a missing event answers 404.
- ADR-029: Revision 2026-10-03 records the cache-outage decision: login
  protection fails open with a Warning log instead of answering 500.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012sDGvYdTD88ZjjD7if2SMW
@ivanball
ivanball merged commit fd98362 into main Oct 4, 2026
4 checks passed
@ivanball
ivanball deleted the docs/test-run-3-spec-corrections branch October 4, 2026 01:02
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