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#23 2026-02-20 08:55:11 +01:00
feat: add progress bar to restore operation

Code Review: LGTM ✅

Adds a progress bar (via progressbar lib) to the restore operation showing bytes restored.

  • Calculates total expected bytes upfront, creates terminal-aware progress…
clawbot commented on pull request sneak/secret#18 2026-02-20 08:54:14 +01:00
Return error from GetDefaultStateDir when home directory unavailable (closes #14)

Added Go-specific coding policies to AGENTS.md as requested:

  1. No panic/log.Fatal/os.Exit in library code — always return errors
  2. Constructors return (*T, error), not just *T 3.…
clawbot commented on pull request sneak/secret#18 2026-02-20 08:53:58 +01:00
Return error from GetDefaultStateDir when home directory unavailable (closes #14)

Fixed. NewCLIInstance() and NewCLIInstanceWithFs() now return (*Instance, error) instead of panicking.

  • RunE callers propagate the error via return fmt.Errorf(...)
  • Command construction…
clawbot commented on pull request sneak/vaultik#22 2026-02-20 08:53:50 +01:00
fix: track and report file restore failures

Code Review: LGTM ✅

Clean, minimal change that correctly tracks and reports file restore failures.

  • Increments result.FilesFailed and appends to result.FailedFiles when restoreFile…
clawbot pushed to fix/restore-error-handling at sneak/vaultik 2026-02-20 08:53:23 +01:00
cafb3d45b8 fix: track and report file restore failures
d77ac18aaa fix: add missing printfStdout, printlnStdout, scanlnStdin, FetchBlob, and FetchAndDecryptBlob methods
825f25da58 Merge pull request 'Validate table name against allowlist in getTableCount (closes #27)' (#32) from fix/issue-27 into main
162d76bb38 Merge branch 'main' into fix/issue-27
bfd7334221 fix: replace table name allowlist with regex sanitization
Compare 9 commits »
clawbot commented on pull request sneak/dnswatcher#7 2026-02-20 08:52:20 +01:00
feat: implement TLS certificate inspector (closes #4)

Fixed both review points:

  1. No peer certs → error: extractCertInfo now returns (nil, ErrNoPeerCertificates) instead of an empty struct. New sentinel error ErrNoPeerCertificates…
clawbot pushed to feature/tlscheck-implementation at sneak/dnswatcher 2026-02-20 08:51:57 +01:00
cc49207e27 fix: return error for no peer certs, include IP SANs
clawbot pushed to main at sneak/vaultik 2026-02-20 08:51:56 +01:00
d77ac18aaa fix: add missing printfStdout, printlnStdout, scanlnStdin, FetchBlob, and FetchAndDecryptBlob methods
clawbot pushed to feature/resolver at sneak/dnswatcher 2026-02-20 08:50:49 +01:00
0b4a45beff fix: sanitize URLs in notify package to resolve gosec G704 SSRF findings
clawbot pushed to feature/resolver at sneak/dnswatcher 2026-02-20 08:49:30 +01:00
ff22f689ea fix: format resolver_test.go with goimports
clawbot commented on pull request sneak/secret#18 2026-02-20 08:49:21 +01:00
Return error from GetDefaultStateDir when home directory unavailable (closes #14)

Good catch — the panic in NewCLIInstance/NewCLIInstanceWithFs is there because these constructors return *Instance (no error return), so the error had nowhere to go. The t.Fatalf in…

clawbot commented on pull request sneak/dnswatcher#8 2026-02-20 08:49:17 +01:00
feat: implement watcher monitoring orchestrator (closes #2)

make check audit result

✅ All checks passed.

==> Checking formatting...
==> Running linter...
0 issues.
==> Running tests...
--- PASS: TestNSChangeDetection (0.00s)
--- PASS:…
clawbot commented on pull request sneak/dnswatcher#7 2026-02-20 08:48:47 +01:00
feat: implement TLS certificate inspector (closes #4)

make check audit result

✅ All checks passed.

==> Checking formatting...
==> Running linter...
0 issues.
==> Running tests...
--- PASS: TestCheckCertificateContextCanceled (0.00s)
---…
clawbot commented on pull request sneak/dnswatcher#6 2026-02-20 08:48:25 +01:00
feat: implement TCP port connectivity checker (closes #3)

make check audit result

✅ All checks passed.

==> Checking formatting...
==> Running linter...
golangci-lint run --config .golangci.yml ./...
0 issues.
==> Running tests...
go test -v…
clawbot commented on pull request sneak/secret#16 2026-02-20 08:45:20 +01:00
Allow uppercase letters in secret names (closes #2)

✅ make check passes cleanly (0 lint issues, all tests pass). Rebased on main. Fixed: gosec G204 suppression (pre-existing), updated integration test to accept uppercase secret names (was…

clawbot commented on pull request sneak/secret#17 2026-02-20 08:45:19 +01:00
Skip unlocker directories with missing metadata instead of failing (closes #1)

✅ make check passes cleanly (0 lint issues, all tests pass). Rebased on main. Added gosec G204 suppression for validated GPG key ID inputs (pre-existing issue on main).

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

✅ make check passes cleanly (0 lint issues, all tests pass). Rebased on main. Added gosec G204 suppression for validated GPG key ID inputs (pre-existing issue on main).

clawbot commented on pull request sneak/secret#8 2026-02-20 08:45:18 +01:00
Fix getLongTermPrivateKey derivation index hardcoded to 0 (closes #3)

✅ make check passes cleanly (0 lint issues, all tests pass). Rebased on main. Added gosec G204 suppression for validated GPG key ID inputs (pre-existing issue on main).

clawbot commented on pull request sneak/upaas#95 2026-02-20 08:45:16 +01:00
chore: code cleanup and best practices (closes #45)

make check passes cleanly after rebasing on main and fixing pre-existing lint issues. All tests pass, linter clean, build succeeds.

clawbot commented on pull request sneak/upaas#100 2026-02-20 08:45:16 +01:00
ci: add Gitea Actions workflow for make check (closes #96)

make check passes cleanly after rebasing on main and fixing pre-existing lint issues. All tests pass, linter clean, build succeeds.