clawbot commented on pull request sneak/secret#174 2026-10-15 02:21:15 +02:00
Remove secret vault import

Rework:

  • Conflict with next in internal/vault/management.go: rebased onto the current next. The CreateVault doc comment takes the wording next now has and still ends the sentence about…
clawbot pushed to fix/139-remove-vault-import at sneak/secret 2026-10-15 02:21:09 +02:00
8beedb7407 Remove secret vault import (closes #139)
13c7721bad Refuse an existing vault before asking for the mnemonic (closes #151)
b8842b6184 secret get fails when it cannot write the value (closes #155)
Compare 3 commits »
clawbot commented on pull request sneak/secret#178 2026-10-15 02:20:07 +02:00
Print an empty unlocker or vault list as [] in JSON

internal/cli/unlockers_list_test.go, lines 3 to 12: the opening comment says the tests it lists cover secret unlocker list when an unlocker's metadata cannot be read or used, and the PR adds…

clawbot commented on pull request sneak/secret#170 2026-10-15 02:19:16 +02:00
A secret too large for the locked-memory limit fails with an error, not a panic
  1. internal/cli/secrets_size_test.go lines 408 to 412 and 425 to 432: three cases check that a value over 100 MiB is refused (secret add and secret import of a 100 MiB + 1 byte sparse file,…
clawbot commented on pull request sneak/secret#172 2026-10-15 02:16:20 +02:00
Stop vault creation when a vault's metadata cannot be read
  1. The commit message (first paragraph, second sentence) and the PR body (first paragraph, last sentence) say that secret vault create and secret init stop before writing anything. They do…
clawbot commented on pull request sneak/secret#176 2026-10-15 02:12:06 +02:00
Count every file of each version in secret info's Total Size
  • Unreadable version directory: the error is now checked and warned about with secret.Warn, naming the directory and the error; the timestamp step still runs, and the comment above says so. A…
clawbot pushed to fix/147-info-total-size at sneak/secret 2026-10-15 02:12:01 +02:00
1bb8134756 Count every file of each version in secret info's Total Size (closes #147)
13c7721bad Refuse an existing vault before asking for the mnemonic (closes #151)
b8842b6184 secret get fails when it cannot write the value (closes #155)
Compare 3 commits »
clawbot pushed to fix/156-missing-current-vault at sneak/secret 2026-10-15 02:11:26 +02:00
744e7d5dd5 Commands fail when currentvault names no vault (closes #156)
clawbot created branch fix/156-missing-current-vault in sneak/secret 2026-10-15 02:11:26 +02:00
clawbot commented on pull request sneak/secret#175 2026-10-15 02:04:54 +02:00
Ask for the new passphrase in unlocker add when the variable is set (closes #154)

Rework:

  • The README and the comment in addPassphraseUnlocker no longer give the false reason; they say the command asks for the new passphrase even when SB_UNLOCK_PASSPHRASE is set and uses…
clawbot pushed to fix/154-new-passphrase-prompt at sneak/secret 2026-10-15 02:04:46 +02:00
22bda7b100 Ask for the new passphrase in unlocker add when the variable is set (closes #154)
13c7721bad Refuse an existing vault before asking for the mnemonic (closes #151)
b8842b6184 secret get fails when it cannot write the value (closes #155)
Compare 3 commits »
clawbot deleted branch issue-389-marker-check-put-only from sneak/vaultik 2026-10-15 02:02:37 +02:00
clawbot pushed to next at sneak/vaultik 2026-10-15 02:02:37 +02:00
850e2de25a Go on with a backup when the destination marker cannot be checked (closes #389)
clawbot closed issue sneak/vaultik#389 2026-10-15 02:02:36 +02:00
The documented PUT-only source host fails every backup after its first: the destination marker check needs HEAD permission on S3
clawbot merged pull request sneak/vaultik#392 2026-10-15 02:02:36 +02:00
Go on with a backup when the destination marker cannot be checked (closes #389)
clawbot pushed to fix/143-atomic-output at sneak/secret 2026-10-15 02:01:16 +02:00
9a74d2e18c Hold the encrypt and decrypt result in memory until it is written (closes #143)
clawbot commented on pull request sneak/vaultik#394 2026-10-15 02:00:04 +02:00
Do not prune after a backup that found a configured path empty (closes #388)

Rework:

  1. README.md: the empty-path sentence now follows the unreadable-files one ("the prune does not run either"), and the removed-file sentence moved to the end of the paragraph.
  2. The…
clawbot pushed to issue-388-empty-path-prune at sneak/vaultik 2026-10-15 01:59:58 +02:00
fd5e077fa0 Do not prune after a backup that found a configured path empty (closes #388)
clawbot commented on pull request sneak/vaultik#392 2026-10-15 01:50:15 +02:00
Go on with a backup when the destination marker cannot be checked (closes #389)

PASS: the change does what sneak/vaultik#389 asks, and its description now stays within the length limit and claims nothing beyond the change.

Model: opus-5-5

clawbot pushed to fix/143-atomic-output at sneak/secret 2026-10-15 01:49:52 +02:00
34f3ff17c0 Hold the encrypt and decrypt result in memory until it is written (closes #143)