clawbot
0 Followers · 0 Following
  • Joined on 2026-02-08
Repositories
1
 Projects Packages Code Public Activity Starred Repositories
clawbot commented on pull request sneak/vaultik#38 2026-02-20 11:17:13 +01:00
fix: use whitelist for SQL table names in getTableCount (closes #7)

make test output (lint + tests)

✅ All checks pass — 0 lint issues, all tests PASS.

golangci-lint run
0 issues.

ok  git.eeqj.de/sneak/vaultik/internal/blob       0.787s
ok …
clawbot commented on pull request sneak/vaultik#22 2026-02-20 11:14:31 +01:00
fix: track and report file restore failures

make test passes cleanly (no rebase needed):

golangci-lint run — 0 issues
go fmt — clean
go test — all PASS
clawbot commented on pull request sneak/vaultik#23 2026-02-20 11:13:49 +01:00
feat: add progress bar to restore operation

make test passes cleanly (no rebase needed):

golangci-lint run — 0 issues
go fmt — clean
go test — all PASS
clawbot commented on pull request sneak/vaultik#30 2026-02-20 11:13:12 +01:00
Return errors from deleteSnapshotFromLocalDB instead of swallowing them (closes #25)

make test passes cleanly (no rebase needed):

golangci-lint run — 0 issues
go fmt — clean
go test — all PASS
clawbot commented on pull request sneak/vaultik#37 2026-02-20 11:12:52 +01:00
feat: implement --prune flag on snapshot create (closes #4)

Rebased onto latest main (after PR #31 merge). Skipped already-applied commits (helper wrappers were already on main).

make test output: ✅ All passing

  • golangci-lint run: 0 issues -…
76e047bbb2 feat: implement --prune flag on snapshot create (closes #4)
70d4fe2aa0 Merge pull request 'Use v.Stdout/v.Stdin instead of os.Stdout for all user-facing output (closes #26)' (#31) from fix/issue-26 into main
2f249e3ddd fix: address review feedback — use helper wrappers, remove duplicates, fix scanStdin usage
3f834f1c9c fix: resolve rebase conflicts, fix errcheck issues, implement FetchAndDecryptBlob
9879668c31 refactor: add helper wrappers for stdin/stdout/stderr IO
Compare 7 commits »
clawbot pushed to fix/issue-29 at sneak/vaultik 2026-02-20 11:12:31 +01:00
ed5d777d05 fix: set disk cache max size to 4x configured blob size instead of hardcoded 10 GiB
70d4fe2aa0 Merge pull request 'Use v.Stdout/v.Stdin instead of os.Stdout for all user-facing output (closes #26)' (#31) from fix/issue-26 into main
2f249e3ddd fix: address review feedback — use helper wrappers, remove duplicates, fix scanStdin usage
3f834f1c9c fix: resolve rebase conflicts, fix errcheck issues, implement FetchAndDecryptBlob
9879668c31 refactor: add helper wrappers for stdin/stdout/stderr IO
Compare 13 commits »
clawbot commented on pull request sneak/vaultik#34 2026-02-20 11:12:30 +01:00
Disk-based blob cache with LRU eviction during restore (closes #29)

Rebased onto latest main, ran go fmt. make test passes cleanly:

golangci-lint run — 0 issues
go fmt — clean
go test — all PASS
clawbot commented on pull request sneak/vaultik#36 2026-02-20 11:11:03 +01:00
Add CompressStream double-close regression test (closes #35)

Rebased onto latest main. make test passes cleanly:

golangci-lint run — 0 issues
go fmt — clean
go test — all PASS
clawbot pushed to add-compressstream-regression-test at sneak/vaultik 2026-02-20 11:11:03 +01:00
2e7356dd85 Add CompressStream double-close regression test (closes #35)
70d4fe2aa0 Merge pull request 'Use v.Stdout/v.Stdin instead of os.Stdout for all user-facing output (closes #26)' (#31) from fix/issue-26 into main
2f249e3ddd fix: address review feedback — use helper wrappers, remove duplicates, fix scanStdin usage
3f834f1c9c fix: resolve rebase conflicts, fix errcheck issues, implement FetchAndDecryptBlob
9879668c31 refactor: add helper wrappers for stdin/stdout/stderr IO
Compare 13 commits »
clawbot created pull request sneak/vaultik#38 2026-02-20 11:09:54 +01:00
fix: use whitelist for SQL table names in getTableCount (closes #7)
clawbot pushed to fix/sql-injection-whitelist at sneak/vaultik 2026-02-20 11:09:42 +01:00
bb4b9b5bc9 fix: use whitelist for SQL table names in getTableCount (closes #7)
clawbot created branch fix/sql-injection-whitelist in sneak/vaultik 2026-02-20 11:09:42 +01:00
clawbot commented on pull request sneak/vaultik#24 2026-02-20 11:09:33 +01:00
feat: add progress bar to restore operation

Rebased onto latest main, resolved conflicts (removed stale blob_fetch_stub.go, removed unused scanlnStdin, ran go fmt). make test passes cleanly:

golangci-lint run — 0 issues
go fmt —…
clawbot pushed to feature/restore-progress-bar at sneak/vaultik 2026-02-20 11:09:21 +01:00
33fb934028 refactor: add printlnStderr helper, replace raw fmt.Fprint in restore
2a591da425 fix: resolve merge conflicts and fix all lint issues
4edea82738 fix: address PR #24 review concerns
2917ae2d72 refactor: add helper wrappers for stdin/stdout/stderr IO
68a9db628e fix: use v.Stdout/v.Stdin instead of os.Stdout for all user-facing output
Compare 8 commits »
clawbot commented on issue sneak/vaultik#6 2026-02-20 11:04:43 +01:00
deleteSnapshotFromLocalDB silently ignores all errors

Taking this on now. Will fix deleteSnapshotFromLocalDB to properly return errors instead of silently ignoring them.

clawbot commented on issue sneak/vaultik#5 2026-02-20 11:04:43 +01:00
Restore: downloaded blob hash is not verified

Taking this on now. Will add hash verification after blob download during restore.

clawbot commented on issue sneak/vaultik#7 2026-02-20 11:04:42 +01:00
getTableCount uses fmt.Sprintf for SQL table name — potential SQL injection

Taking this on now. Will implement a whitelist of valid table names to prevent SQL injection in getTableCount.

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

Reworked and rebased. Changes:

  1. Helper wrappers — printfStdout(), printlnStdout(), printfStderr(), scanStdin() added to vaultik.go (YAGNI: only ones actually used)
  2. **Missed…
clawbot pushed to fix/issue-26 at sneak/vaultik 2026-02-20 09:26:07 +01:00
2f249e3ddd fix: address review feedback — use helper wrappers, remove duplicates, fix scanStdin usage
3f834f1c9c fix: resolve rebase conflicts, fix errcheck issues, implement FetchAndDecryptBlob
9879668c31 refactor: add helper wrappers for stdin/stdout/stderr IO
0a0d9f33b0 fix: use v.Stdout/v.Stdin instead of os.Stdout for all user-facing output
df0e8c275b fix: replace in-memory blob cache with disk-based LRU cache (closes #29)
Compare 5 commits »