CreatePassphraseUnlocker already writes longterm.age. Init was doing it a second time, overwriting with an equivalent but separately encrypted blob. Removed the duplicate code.
`CreatePassphraseUnlocker` already writes `longterm.age`. Init was doing it a second time, overwriting with an equivalent but separately encrypted blob. Removed the duplicate code.
sneak
was assigned by clawbot2026-02-08 21:06:03 +01:00
CreatePassphraseUnlocker already encrypts and writes the long-term
private key to longterm.age. The Init command was doing this a second
time, overwriting the file with a functionally equivalent but
separately encrypted blob. This was wasteful and a maintenance hazard.
i don’t see test output on this PR. remember to include test output in a markdown code block comment (verbose for failing, normal for passing) on each PR
i don’t see test output on this PR. remember to include test output in a markdown code block comment (verbose for failing, normal for passing) on each PR
This PR removes redundant code (duplicate longterm.age encryption) but does not add new test files. Existing tests all pass:
golangci-lint run --timeout 5m
0 issues.
go vet ./...
go test ./... || go test -v ./...
? git.eeqj.de/sneak/secret/cmd/secret [no test files]
ok git.eeqj.de/sneak/secret/internal/cli 4.964s
ok git.eeqj.de/sneak/secret/internal/secret 3.267s
ok git.eeqj.de/sneak/secret/internal/vault 0.895s
ok git.eeqj.de/sneak/secret/pkg/agehd 0.640s
ok git.eeqj.de/sneak/secret/pkg/bip85 0.616s
## `make test` output (branch `fix/issue-6`)
This PR removes redundant code (duplicate `longterm.age` encryption) but does not add new test files. Existing tests all pass:
```
golangci-lint run --timeout 5m
0 issues.
go vet ./...
go test ./... || go test -v ./...
? git.eeqj.de/sneak/secret/cmd/secret [no test files]
ok git.eeqj.de/sneak/secret/internal/cli 4.964s
ok git.eeqj.de/sneak/secret/internal/secret 3.267s
ok git.eeqj.de/sneak/secret/internal/vault 0.895s
ok git.eeqj.de/sneak/secret/pkg/agehd 0.640s
ok git.eeqj.de/sneak/secret/pkg/bip85 0.616s
```
Lint: ✅ clean (0 issues)
Vet: ✅ clean
Tests: ✅ all pass
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
CreatePassphraseUnlockeralready writeslongterm.age. Init was doing it a second time, overwriting with an equivalent but separately encrypted blob. Removed the duplicate code.i don’t see test output on this PR. remember to include test output in a markdown code block comment (verbose for failing, normal for passing) on each PR
make testoutput (branchfix/issue-6)This PR removes redundant code (duplicate
longterm.ageencryption) but does not add new test files. Existing tests all pass:Lint: ✅ clean (0 issues)
Vet: ✅ clean
Tests: ✅ all pass