sneak/secret
1
0
Fork 0
Code Issues 29 Pull Requests 6 Actions 4 Packages Projects Releases Wiki Activity

Stop secret mv deleting a secret moved onto itself (closes #73) #76

Open
clawbot wants to merge 1 commits from issue-73-mv-onto-itself into next
pull from: issue-73-mv-onto-itself
merge into: :next
Conversation 1 Commits 1 Files Changed 3
+131 -21
1 Commits
Author SHA1 Message Date
sneak 974b1b6dc5 Stop secret mv deleting a secret moved onto itself (closes #73)
check / check (push) Successful in 1m32s
Details 
`secret mv --force x x` deleted the secret: a move within one vault
removes an existing destination before renaming the source onto it. The
same happened for `work:x work:`, `work:x work` and `work:x ""`, where an
empty destination defaults to the source name.

moveSecretWithinVault now rejects a move whose two names are the same
before touching anything. A move within a named vault works in that
vault directly instead of selecting it, so the current vault never
changes; the vault must be one of the existing vaults.

The test runs each rejected move on a copy of two in-memory vaults and
requires the exact error and an unchanged state directory.

Model: opus-5-5
 2026-10-03 23:47:57 +00:00