a1800a8e88removed binary erroneously committed by LLM :/sneak2025-06-11 15:28:14 -0700
03e0ee2f95refactor: remove confusing dual ID method pattern from Unlocker interface - Removed redundant ID() method from Unlocker interface - Removed ID field from UnlockerMetadata struct - Modified GetID() to generate IDs dynamically based on unlocker type and data - Updated vault package to create unlocker instances when searching by ID - Fixed all tests and CLI code to remove ID field references - IDs are now consistently generated from unlocker data, preventing redundancysneak2025-06-11 15:21:20 -0700
9adf0c0803refactor: fix redundant metadata fields across the codebase - Removed VaultMetadata.Name (redundant with directory structure) - Removed SecretMetadata.Name (redundant with Secret.Name field) - Removed AgePublicKey and AgeRecipient from PGPUnlockerMetadata - Removed AgePublicKey from KeychainUnlockerMetadata - Changed PGP and Keychain unlockers to store recipient in pub.txt instead of pub.age - Fixed all tests to reflect these changes - Follows DRY principle and prevents data inconsistencysneak2025-06-09 17:44:10 -0700
e9d03987f9refactor: remove redundant SecretName and Version fields from VersionMetadata - Removed SecretName and Version fields that were redundant with directory structure and parent SecretVersion struct - Updated tests to remove references to deleted fields - Follows DRY principle and prevents potential data inconsistencysneak2025-06-09 17:26:57 -0700
b0e3cdd3d0fix: Restore fmt target to Makefilesneak2025-06-09 17:22:44 -0700
2e3fc475cffix: Use vault metadata derivation index for environment mnemonic - Fixed bug where GetValue() used hardcoded index 0 instead of vault metadata - Added test31 to verify environment mnemonic respects vault derivation index - Rewrote test19DisasterRecovery to actually test manual recovery process - Removed all test skip statements as requestedsneak2025-06-09 17:21:02 -0700
f59ee4d2d6'unlock keys' renamed to 'unlockers'sneak2025-05-30 07:29:02 -0700
0bf8e71b52fix: resolve ineffectual assignment lint error in pgpunlock_test.gosneak2025-05-29 16:36:10 -0700
34d6870e6afeat: add derivation index to vault metadata for unique keys - Add VaultMetadata fields: DerivationIndex, LongTermKeyHash, MnemonicHash - Implement GetNextDerivationIndex() to track and increment indices for same mnemonics - Update init and import commands to use proper derivation indices - Add ComputeDoubleSHA256() for hash calculations - Save vault metadata on creation with all derivation information - Add comprehensive tests for metadata functionality. This ensures multiple vaults using the same mnemonic will derive different long-term keys by using incremented derivation indices. The mnemonic is double SHA256 hashed and stored to track which vaults share mnemonics. Fixes TODO item #5sneak2025-05-29 16:23:29 -0700
ddb395901bRefactor vault functionality to dedicated package, fix import cycles with interface pattern, fix testssneak2025-05-29 12:48:36 -0700
c33385be6cClean up integration test script: remove redundant tests and fix misleading output - Remove redundant manual input tests that were actually using environment variables - Update all test output to honestly reflect automated testing with env vars - Consolidate similar test cases to reduce duplication - Fix cross-vault operations test by properly recreating work vault after reset_state - Import mnemonic into work vault so it can store secrets - Update test descriptions to be accurate about automation vs manual input - All tests now pass successfully with proper environment variable usagesneak2025-05-29 11:04:31 -0700