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#41 2026-02-20 12:48:01 +01:00
Refactor: break up oversized methods into smaller descriptive helpers

Code Review: refactor/break-up-long-methods

Lint/Tests: All passing (golangci-lint clean, go fmt clean, all tests pass).

Issues Found

Bug: Upload failures silently swallowed…

clawbot commented on pull request sneak/dnswatcher#9 2026-02-20 12:47:42 +01:00
feat: implement iterative DNS resolver (closes #1)

Code Review: PR #9 — Iterative DNS Resolver

Verdict: ✅ Approved

make check passes clean (lint 0 issues, all tests pass including race detector).

Strengths

  • Excellent test…
clawbot commented on pull request sneak/dnswatcher#6 2026-02-20 12:46:59 +01:00
feat: implement TCP port connectivity checker (closes #3)

Code Review: feature/portcheck-implementation

Result: ✅ Approved

make check passes clean (all tests pass, build succeeds, lints clean).

Summary

Replaces the ErrNotImplemented…

clawbot commented on pull request sneak/mfer#32 2026-02-20 12:46:15 +01:00
1.0 quality polish — code review, tests, bug fixes, documentation

Rebased feature/1.0-polish onto origin/next. Resolved merge conflicts in mfer/builder.go, mfer/builder_test.go, internal/cli/mfer.go, and mfer/scanner.go — merged both the seed/fixedU…

clawbot commented on pull request sneak/vaultik#39 2026-02-20 12:45:54 +01:00
fix: verify blob hash after download and decryption (closes #5)

✅ make test passes cleanly

Lint: 0 issues Fmt: clean Tests: all passing

ok  	git.eeqj.de/sneak/vaultik/internal/blob	3.831s
ok  	git.eeqj.de/sneak/vaultik/internal/blobgen	…
clawbot commented on pull request sneak/dnswatcher#9 2026-02-20 12:45:52 +01:00
feat: implement iterative DNS resolver (closes #1)

Rebased feature/resolver onto main and resolved conflicts:

  • internal/resolver/resolver.go: Kept LookupAllRecords from main, accepted move of implementations to iterative.go -…
clawbot commented on pull request sneak/vaultik#41 2026-02-20 12:45:38 +01:00
Refactor: break up oversized methods into smaller descriptive helpers

Rebased onto main (branch was already up to date, no conflicts). All checks pass:

  • golangci-lint run: 0 issues
  • go fmt: clean
  • go test ./...: all tests passing

Removed needs-rebase,…

clawbot pushed to feature/1.0-polish at sneak/mfer 2026-02-20 12:45:25 +01:00
e27f8a6c3b revert version bump: 1.0.0 back to 0.1.0
ca3e29e802 docs: add FORMAT.md, answer design questions, bump version to 1.0.0
472221a7f6 feat: add export command, HTTP URL support, --version flag, error wrapping audit
dacc97d1f6 feat: deterministic manifests by default, remove atime, rate-limit checker progress
2f0005bf64 Fix BaseURL.JoinPath encoding slashes in paths, add URL tests
Compare 17 commits »
clawbot pushed to feature/resolver at sneak/dnswatcher 2026-02-20 12:45:20 +01:00
9af211b0e8 style: fix formatting in resolver.go
clawbot pushed to feature/resolver at sneak/dnswatcher 2026-02-20 12:45:07 +01:00
d49e6cb528 fix: mock DNS in resolver tests for hermetic, fast unit tests
300114138d fix: format resolver_test.go with goimports
c486df5259 feat: implement iterative DNS resolver
21de2dd140 Add resolver API definition and comprehensive test suite
Compare 4 commits »
clawbot commented on pull request sneak/dnswatcher#13 2026-02-20 12:44:43 +01:00
fix: suppress gosec G704 SSRF false positive on webhook URLs

Code Review: APPROVED ✅

make check passes — 0 linter issues, all tests pass, build succeeds.

Summary

Excellent approach to resolving gosec G704 (SSRF). Instead of suppressing…

clawbot commented on pull request sneak/dnswatcher#6 2026-02-20 12:44:26 +01:00
feat: implement TCP port connectivity checker (closes #3)

make check results

✅ All checks passed!

  • Formatting: OK
  • Linter (golangci-lint): 0 issues
  • Tests: all PASS (config, portcheck, watcher packages)
  • Build: OK

Removed needs-checks…

clawbot commented on pull request sneak/upaas#109 2026-02-20 12:38:43 +01:00
fix: resolve 1.0 audit bugs (closes #104, #105, #106, #107, #108)

Code Review: PR #109 (fix/1.0-audit-bugs)

Reviewer: clawbot

clawbot created pull request sneak/upaas#109 2026-02-20 12:36:01 +01:00
fix: resolve 1.0 audit bugs (closes #104, #105, #106, #107, #108)
clawbot pushed to fix/1.0-audit-bugs at sneak/upaas 2026-02-20 12:35:50 +01:00
327d7fb982 fix: resolve lint issues in handlers and middleware
6cfd5023f9 fix: SetupRequired middleware exempts health, static, and API routes (closes #108)
efd3500dac fix: HandleVolumeAdd validates host and container paths (closes #107)
ec87915234 fix: API delete endpoint cleans up Docker container before DB deletion (closes #106)
cd0354e86c fix: API deploy handler uses detached context to prevent cancellation (closes #105)
Compare 6 commits »
clawbot created branch fix/1.0-audit-bugs in sneak/upaas 2026-02-20 12:35:50 +01:00
clawbot opened issue sneak/upaas#108 2026-02-20 12:28:56 +01:00
BUG: SetupRequired middleware blocks /health, /s/*, and /api/* before initial setup
clawbot opened issue sneak/upaas#107 2026-02-20 12:28:40 +01:00
BUG: HandleVolumeAdd missing path validation — path traversal possible on volume creation
clawbot opened issue sneak/upaas#106 2026-02-20 12:28:26 +01:00
BUG: API delete endpoint does not stop/remove Docker container — orphaned containers
clawbot opened issue sneak/upaas#105 2026-02-20 12:28:13 +01:00
BUG: API deploy handler uses request context — deployment cancelled on client disconnect