feat: add keychain CLI commands#820
Conversation
81e4c26 to
fd3bab3
Compare
|
I finished a closeout autoreview pass and made a few follow-up fixes before marking this ready for review. What changed in this branchThis PR adds a native
Fixes from autoreviewThe review found several real edge cases, all fixed now:
I also reran lint, the focused keychain CLI tests, Swift tests, and the final autoreview. The final autoreview result was clean: no accepted/actionable findings. CLI namespace/scoping rationaleI think the command split is in the right place semantically:
That keeps The new subcommands are backend/admin operations, so they fit under
So I would avoid backend-specific duplicates of graph-level commands like |
Fixes #819
Adds a native
varlock keychainCLI namespace for managing macOS Keychain-backed secrets:keychain setstores secrets through Varlock's own daemon/helper and can write matchingkeychain(...)refs back to env files.keychain importmigrates sensitive plaintext values from an env file into Keychain using the file's schema to decide what should be imported.keychain fix-accessgrants Varlock's helper access to existing explicitkeychain(...)refs, for cases where items were created by another tool.keychain listprovides a metadata-only view of matching Keychain items.Also updates docs and release workflows for the native Keychain/Rust signing changes, and adds tests for the new CLI parsing/import behavior.