archetech / archon / 28178752621

86%

main: 92%

Build Type

Pull #648

github

Committed by macterra

Commit Message

fix: make the optional-service off-switch reachable via .env (compose passthrough)

#648 added the config-level off-switch (empty URL disables a service) but the
drawbridge compose block hardcoded the URLs, so .env never reached the container —
the switch was inert. Pass them through with ${VAR-default} (single dash: explicit
empty disables, unset keeps the default — mirroring the code's `??`).

DIDComm now defaults EMPTY (off) in compose, matching its absence from the default
COMPOSE_PROFILES — a stock node honestly reports didcomm:false instead of claiming
true while no relay runs. Enabling DIDComm requires BOTH `didcomm` in COMPOSE_PROFILES
(runs the relay) AND ARCHON_DIDCOMM_URL=http://didcomm:4236 (Drawbridge offers it).
Lightning + names keep non-empty defaults since they're in the default profiles.

sample.env documents the two-knob enable + the empty-to-disable switch; drawbridge
spec notes the compose default-off. Verified via `docker compose config`: unset/empty
→ "", explicit → value.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Pull Request Pull Request #648: feat: node capability manifest + client gating for optional services (Option C)

Coverage Stats

2655 of 3292 branches covered (80.65%)

Branch coverage included in aggregate %.

18 of 36 new or added lines in 2 files covered. (50.0%)

75 existing lines in 1 file now uncovered.

5382 of 6085 relevant lines covered (88.45%)

881.06 hits per line