Skip to content

feat: support BYO backup recipient - #27

Merged
k9ert merged 1 commit into
mainfrom
kn/it-backup-recipient-byo
Jun 3, 2026
Merged

feat: support BYO backup recipient#27
k9ert merged 1 commit into
mainfrom
kn/it-backup-recipient-byo

Conversation

@k9ert

@k9ert k9ert commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add --it-recipient for greenfield backup/escrow recipient selection while keeping the bundled Blink key as default
  • respect existing secrets/.gpg-id, log recipients, and warn on missing/malformed/single-recipient stores
  • make doctor key-agnostic with >=2 recipient enforcement and recipient key presence checks
  • document repo-scoped pass trust behavior and bump version to 1.9.0

Tests

  • PYENV_VERSION=3.11.13 PYTHONPATH=src GNUPGHOME=/tmp/argit-unit-gnupg python -m pytest -p no:cacheprovider tests/unit
  • PYENV_VERSION=3.11.13 PYTHONPATH=src python -m pytest -p no:cacheprovider tests/integration/test_setup_integration.py tests/integration/test_roundtrip.py::test_byo_recipient_roundtrip_restores_plaintext

Note: ruff is unavailable in the local Python 3.11 environment (No module named ruff).

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@blink-claw-bot blink-claw-bot left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

HIGH

  • None.

MEDIUM

  • None.

LOW

  • None.

Reviewed the BYO backup recipient flow, the key-agnostic doctor changes, pass trust-model handling, the new argit info command, and the bundled argit-backup skill/docs. The implementation preserves existing .gpg-id authority, keeps greenfield setup explicit, and has coverage for the main recipient/roundtrip paths.

Verified locally:

  • PYTHONPATH=src GNUPGHOME=/tmp/argit-unit-gnupg python3 -m pytest -p no:cacheprovider tests/unit → 380 passed
  • PYTHONPATH=src:/home/openclaw/.local/lib/python3.10/site-packages python3 -m pytest -p no:cacheprovider tests/integration/test_setup_integration.py tests/integration/test_roundtrip.py::test_byo_recipient_roundtrip_restores_plaintext → 8 passed
  • PYTHONPATH=src python3 -m argit.cli info --json | python3 -m json.tool → OK

🤖 Review by GPT-5 Codex via Blink-Claw-Bot

@k9ert
k9ert merged commit fc6db45 into main Jun 3, 2026
2 of 3 checks passed
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.

3 participants