archetech / archon
93%
main: 94%

Repo Added 12 Jan 2026 07:43PM UTC

Token VKRdMqMwqPTOZprp4p5Cb3v3uLt6LWUtk regen

Build 1946 Last

Committed 01 Aug 2026 06:18PM UTC coverage: 93.029% (+0.04%) from 92.987%

Committed by

Commit Message

refactor(drawbridge): extract the v1 router so it can be tested

drawbridge-api.ts built its Express app inside main(), which the module
calls at import time — so importing it booted the server, connected to
Gatekeeper and Redis, and bound a port. That left its 28 v1 routes
untestable and uncounted, the same shape #705/#706 fixed for gatekeeper
and keymaster.

Extract createV1Router() into v1-router.ts, with createRequireAdminKey()
in v1-admin.ts and the options interface in v1-router-types.ts, mirroring
the gatekeeper layout. drawbridge-api.ts now builds its dependencies and
mounts the router.

Behaviour-preserving: the route block was moved programmatically rather
than retyped, and diffed against the original. The only change is
serviceVersion -> getServiceVersion(), which is required — serviceVersion
is a module-scope `let` reassigned after an async package.json read, so
capturing it by value at factory time would pin /version to "unknown"
forever. Verified: the route set is identical (28 before, 28 after) and
`tsc -p tsconfig.json` is clean for the service.

ARCHON_ADMIN_HEADER is used both by the admin middleware and by the
herald proxy for upstream auth, so it is exported from v1-admin.ts rather
than duplicated.

Tests: 18 for the extracted router (capabilities derivation, DIDComm
endpoint including the null case, admin gating across 401/403/pass,
resolve-option passthrough, numeric-vs-hash blockId parsing, a
table-driven 502 check across every proxy route, mid-stream read failure,
and the lightning proxy's 501/200/502 paths), plus 11 for
lightning-mediator-client, which the router test transitively pulled into
the coverage denominator at 5.9%.

Three one-line L402 delegation lines are deliberately left uncovered:
driving those routes with a stub options object leaves one handler never
responding, which hangs Jest (CI runs without --forceExit). Their
handlers are already at 100% via l402-auth.test.ts. A comment in the test
records this.

drawbri... (continued)

Coverage Stats

3328 of 3878 branches covered (85.82%)

Branch coverage included in aggregate %.

174 of 177 new or added lines in 2 files covered. (98.31%)

7762 of 8043 relevant lines covered (96.51%)

707.8 hits per line

8043 RELEVANT LINES 7762 COVERED LINES

707.8 HITS PER LINE

Builds	Branch	Commit	Type	Ran	Committer	Via	Coverage
30712225661	refactor/drawbridge-router	refactor(drawbridge): extract the v1 router so it can be tested drawbridge-api.ts built its Express app inside main(), which the module calls at import time — so importing it booted the server, connected to Gatekeeper and Redis, and bound a port....	Pull #819	01 Aug 2026 06:22PM UTC	macterra	github	93.03

archetech / archon
93%
main: 94%

archetech / archon 93% main: 94%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

CHANGE BRANCH x

Relevant lines Covered

Source Files on refactor/drawbridge-router

Recent builds

archetech / archon
93%
main: 94%

README BADGES
x

CHANGE BRANCH
x