archetech / archon
92%
main: 95%

Repo Added 12 Jan 2026 07:43PM UTC

Token VKRdMqMwqPTOZprp4p5Cb3v3uLt6LWUtk regen

Build 2834 Last

Committed 10 Sep 2026 05:09PM UTC coverage: 92.413%. Remained the same

Committed by

Commit Message

fix(keymaster-service): Handle review — the 400 handler was being overwritten

Three issues from review:

- KeymasterServiceError is an alias of KeymasterError (runtime.py), so the
  separate exception handler for it registered the same class again and
  overwrote the new 400 handler with the old 500 one -- KeymasterError still
  returned 500. It is raised only at startup, never per-request, so its handler
  was removed; the base KeymasterError handler now stands and unexpected errors
  still reach the generic 500 handler. Confirmed against real FastAPI dispatch.

- The status tests called handlers directly and so could not see that
  overwrite. The stubbed FastAPI now records its registrations, and a new test
  asserts KeymasterError resolves to the 400 handler -- it fails if the
  colliding handler returns.

- README §16.3 overstated the 404 category: an explicit did: that fails to
  resolve raises the base KeymasterError (400), not UnknownIDError; only an
  unresolved name is 404. Narrowed the row and recorded the nuance, left to
  #1108 since fixing it changes the shared library's resolve_did.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GYGMfX3foBcVqP7AQNXjWj

Coverage Stats

4084 of 4701 branches covered (86.88%)

Branch coverage included in aggregate %.

8766 of 9204 relevant lines covered (95.24%)

742.89 hits per line

9204 RELEVANT LINES 8766 COVERED LINES

742.89 HITS PER LINE

Builds	Branch	Commit	Type	Ran	Committer	Via	Coverage
34506478875	fix/1103-python-service-error-status	fix(keymaster-service): Handle review — the 400 handler was being overwritten Three issues from review: - KeymasterServiceError is an alias of KeymasterError (runtime.py), so the separate exception handler for it registered the same class agai...	Pull #1107	10 Sep 2026 05:13PM UTC	macterra	github	92.41
34503200079	fix/1103-python-service-error-status	docs(keymaster-service): Record the error-status classification decision The error-status convention this PR adopts lived only in the code and the commit message. Write it into the service contract: README §16.3 states that status is classified b...	Pull #1107	10 Sep 2026 04:41PM UTC	macterra	github	92.41
34502835181	fix/1103-python-service-error-status	fix(keymaster-service): Return 400/404 for client errors, not 500 The Python keymaster service caught nothing per-route and let every KeymasterError reach the generic handler, so a caller's bad input or a not-found came back 500 -- a server error...	Pull #1107	10 Sep 2026 04:36PM UTC	macterra	github	92.41

archetech / archon
92%
main: 95%

archetech / archon 92% main: 95%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

CHANGE BRANCH x

Relevant lines Covered

Source Files on fix/1103-python-service-error-status

Recent builds

archetech / archon
92%
main: 95%

README BADGES
x

CHANGE BRANCH
x