• Joined on 2026-02-08
clawbot commented on pull request sneak/mfer#21 2026-02-16 05:56:52 +01:00
Fix FindExtraFiles reporting manifest and dotfiles as extra (closes #16)

Code Review: Fix FindExtraFiles reporting manifest and dotfiles as extra

clawbot commented on pull request sneak/mfer#19 2026-02-16 05:56:52 +01:00
Fix IsHiddenPath treating current directory as hidden (closes #14)

Code Review: Fix IsHiddenPath treating current directory as hidden

clawbot commented on pull request sneak/vaultik#34 2026-02-16 05:56:21 +01:00
Disk-based blob cache with LRU eviction during restore (closes #29)

Review: Bound blob cache during restore with LRU eviction

clawbot commented on pull request sneak/vaultik#33 2026-02-16 05:56:20 +01:00
Prevent double-close of blobgen.Writer in CompressStream (closes #28)

Review: Prevent double-close of blobgen.Writer in CompressStream

clawbot commented on pull request sneak/vaultik#32 2026-02-16 05:56:19 +01:00
Validate table name against allowlist in getTableCount (closes #27)

Review: Validate table name against allowlist in getTableCount

clawbot commented on pull request sneak/vaultik#31 2026-02-16 05:56:18 +01:00
Use v.Stdout/v.Stdin instead of os.Stdout for all user-facing output (closes #26)

Review: Use v.Stdout/v.Stdin instead of os.Stdout for all user-facing output

clawbot commented on pull request sneak/vaultik#30 2026-02-16 05:56:17 +01:00
Return errors from deleteSnapshotFromLocalDB instead of swallowing them (closes #25)

Review: Return errors from deleteSnapshotFromLocalDB instead of swallowing them

clawbot commented on pull request sneak/vaultik#24 2026-02-16 05:56:16 +01:00
feat: add progress bar to restore operation

Review: feat: add progress bar to restore operation

clawbot commented on pull request sneak/secret#8 2026-02-16 05:56:09 +01:00
Fix getLongTermPrivateKey derivation index hardcoded to 0 (closes #3)

test review

clawbot commented on pull request sneak/mfer#19 2026-02-16 05:55:32 +01:00
Fix IsHiddenPath treating current directory as hidden (closes #14)

test review

clawbot commented on pull request sneak/secret#15 2026-02-15 23:17:46 +01:00
Validate secret name in GetSecretVersion to prevent path traversal (closes #13)

Fixed the three issues raised in review:

  1. isValidSecretName() now blocks .. path components — splits on / and rejects any segment equal to .., catching names like foo/../bar and…
clawbot commented on pull request sneak/secret#15 2026-02-15 23:16:29 +01:00
Validate secret name in GetSecretVersion to prevent path traversal (closes #13)

Security Self-Review: Path Traversal Fix

clawbot commented on pull request sneak/mfer#28 2026-02-15 23:16:11 +01:00
Add deterministic file ordering in Builder.Build() (closes #23)

Code Review: Deterministic File Ordering

clawbot commented on pull request sneak/secret#16 2026-02-15 23:16:11 +01:00
Allow uppercase letters in secret names (closes #2)

Self-Review: MERGE-READY

clawbot commented on pull request sneak/secret#18 2026-02-15 23:16:09 +01:00
Return error from GetDefaultStateDir when home directory unavailable (closes #14)

Self-Review: MERGE-READY

clawbot commented on pull request sneak/secret#17 2026-02-15 23:16:04 +01:00
Skip unlocker directories with missing metadata instead of failing (closes #1)

Code Review: MERGE-READY

clawbot commented on issue sneak/secret#14 2026-02-15 23:05:50 +01:00
Bug: GetDefaultStateDir ignores UserHomeDir error, may use empty path

Fix submitted in PR #18.

clawbot created pull request sneak/secret#18 2026-02-15 23:05:31 +01:00
Return error from GetDefaultStateDir when home directory unavailable (closes #14)
clawbot created pull request sneak/secret#17 2026-02-15 23:05:22 +01:00
Skip unlocker directories with missing metadata instead of failing (closes #1)
clawbot commented on pull request sneak/mfer#28 2026-02-15 23:05:17 +01:00
Add deterministic file ordering in Builder.Build() (closes #23)

make test run 1

All tests pass (exit code 0):

error: tool 'git' not found
go test -v --timeout 10s ./...
ok  	sneak.berlin/go/mfer/cmd/mfer	(cached)
ok  	sneak.berlin/go/mfer/internal/bo…