archetech / archon / 24753489161

89%

push

github

fix(keymaster-py): restore drop-in parity with ts flavor (#458)

PR #455 (review comment fix) switched docker-compose.keymaster-py.yml to
a named volume (keymaster_py_data) while leaving the host UID/GID
`user:` override in place. The image creates /app/keymaster/data as
root, so the named volume inherits root ownership; the container then
runs as ${ARCHON_UID}:${ARCHON_GID} and fails on first wallet write
with:

    PermissionError: [Errno 13] Permission denied: 'data/wallet.tmp'

The intended invariant is that py and ts are interchangeable via
ARCHON_KEYMASTER_FLAVOR. Restore that:

- docker-compose.keymaster-py.yml: bind mount ./data:/app/keymaster/data
  exactly like docker-compose.keymaster-ts.yml; drop the top-level
  volumes: block.
- Dockerfile.keymaster-py: drop redundant 'RUN mkdir -p
  /app/keymaster/data' (the bind mount supplies the directory; this
  also keeps the Dockerfile structurally aligned with
  Dockerfile.keymaster-ts).
- AGENTS.md: replace the obsolete 'named volume avoids UID mismatch'
  note with an explicit drop-in-replacement rule that forbids
  re-introducing the workaround.

1994 of 2200 branches covered (90.64%)

Branch coverage included in aggregate %.

4243 of 4359 relevant lines covered (97.34%)

1163.7 hits per line