Skip to content

feat(kryphos): add figment provider for vault-backed config values#37

Merged
forkwright merged 2 commits intomainfrom
feat/kryphos-config
Mar 17, 2026
Merged

feat(kryphos): add figment provider for vault-backed config values#37
forkwright merged 2 commits intomainfrom
feat/kryphos-config

Conversation

@forkwright
Copy link
Owner

Summary

  • Add VaultProvider<P> figment provider that wraps an inner provider and resolves vault:entry_name prefixed config values from the encrypted vault
  • Non-vault values pass through unchanged; missing entries and uninitialized vault produce descriptive errors
  • Five tests covering resolution, passthrough, missing entry errors, and vault-not-initialized errors

Test plan

  • cargo fmt --all -- --check
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo test --workspace (318 tests pass, including 5 new config tests)

Observations

  • Debt: figment::Error is 208 bytes, triggering result_large_err on internal helpers. Suppressed with #[expect] since the type is external. If figment upstream boxes the error, the suppression will auto-warn for removal.

🤖 Generated with Claude Code

Cody Kickertz and others added 2 commits March 17, 2026 14:32
VaultProvider wraps an inner figment provider and transparently
resolves `vault:entry_name` prefixed strings by decrypting the
named entry from the vault. Non-vault values pass through unchanged.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@forkwright forkwright merged commit 6f12afd into main Mar 17, 2026
13 checks passed
@forkwright forkwright deleted the feat/kryphos-config branch March 18, 2026 02:12
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.

1 participant