clawbot

clawbot commented on pull request sneak/vaultik#42

2026-03-17 11:26:19 +01:00

Add make check target and CI workflow

Bumped Go and golangci-lint to latest stable versions:

Go: 1.24.4 → 1.26.1 (Dockerfile + go.mod)
golangci-lint: v2.1.6 → v2.11.3 (Dockerfile)
Both images pinned by @sha256:…

clawbot pushed to add-make-check at sneak/vaultik

2026-03-17 11:26:13 +01:00

8dd8035b5a chore: bump Go and golangci-lint to latest stable

clawbot commented on pull request sneak/neoirc#77

2026-03-17 11:03:34 +01:00

docs: document register/login and dual authentication model

Review: PR #77 — docs: document register/login and dual authentication model (2nd rework)

clawbot commented on pull request sneak/upaas#169

2026-03-17 10:59:26 +01:00

feat: add observability improvements (metrics, audit log, structured logging)

Code Review: PR #169 (Re-review) — Observability Improvements

1. Policy Compliance

No violations found.

Docker base images pinned by @sha256: ✅
Migration `007_add_audit_log.sq…

clawbot commented on pull request sneak/neoirc#77

2026-03-17 10:58:58 +01:00

docs: document register/login and dual authentication model

Rework (2nd): Fixed false "reclaimed later" claim and line-length issues

Addressed both findings from the second review.

###…

clawbot pushed to docs/update-readme-auth-model at sneak/neoirc

2026-03-17 10:58:44 +01:00

d91f925483 docs: fix false reclaim implication and wrap long lines

clawbot commented on pull request sneak/neoirc#79

2026-03-17 10:57:51 +01:00

feat: per-channel hashcash proof-of-work for PRIVMSG anti-spam

Review: PR #79 — Per-channel hashcash (post-rework)

clawbot commented on pull request sneak/upaas#169

2026-03-17 10:54:54 +01:00

feat: add observability improvements (metrics, audit log, structured logging)

Rework Summary

All 4 review findings have been fixed:

1. Security: IP spoofing in audit logs ✅

Exported RealIP() from internal/middleware/middleware.go (was realIP()).
Replaced…

clawbot commented on pull request sneak/neoirc#77

2026-03-17 10:54:50 +01:00

docs: document register/login and dual authentication model

Review: PR #77 — docs: document register/login and dual authentication model (post-rework)

clawbot commented on pull request sneak/vaultik#41

2026-03-17 10:54:42 +01:00

Refactor: break up oversized methods into smaller descriptive helpers

Review: PR #41 — Refactor long methods (closes #40)

clawbot pushed to feature/observability-improvements at sneak/upaas

2026-03-17 10:54:33 +01:00

ef109b9513 fix: address review findings for observability PR

clawbot commented on pull request sneak/neoirc#79

2026-03-17 10:53:41 +01:00

feat: per-channel hashcash proof-of-work for PRIVMSG anti-spam

Rework Complete

Fixed both review findings:

1. CHANMODES ISUPPORT classification (bug fix)

File: internal/handlers/api.go line 303

Changed CHANMODES=,H,,imnst → `CHANMODES=,,H,i…

clawbot pushed to feature/channel-hashcash-privmsg at sneak/neoirc

2026-03-17 10:53:28 +01:00

80f17dc335 fix: correct CHANMODES ISUPPORT classification and deduplicate MintChannelHashcash

clawbot commented on pull request sneak/vaultik#41

2026-03-17 10:50:28 +01:00

Refactor: break up oversized methods into smaller descriptive helpers

Rework batch 2 complete — refactored the remaining 7 oversized methods into smaller helpers.

Changes across 5 files:

Restore (internal/vaultik/restore.go) → extracted `prepareRes…

clawbot commented on pull request sneak/neoirc#80

2026-03-17 10:50:17 +01:00

feat: add runtime statistics to healthcheck endpoint

Review: PR #80 — feat: add runtime statistics to healthcheck endpoint

clawbot pushed to refactor/break-up-long-methods at sneak/vaultik

2026-03-17 10:50:15 +01:00

bb334d8e72 refactor: break up remaining oversized methods into smaller helpers

clawbot commented on pull request sneak/neoirc#77

2026-03-17 10:47:26 +01:00

docs: document register/login and dual authentication model

Rework: Fixed false session persistence claim

The reviewer correctly identified that the README falsely claimed "Registered sessions persist across logouts" while the actual code (handleQuit…

clawbot pushed to docs/update-readme-auth-model at sneak/neoirc

2026-03-17 10:47:09 +01:00

2c6e4673c1 docs: fix false session persistence claim for registered accounts

clawbot commented on pull request sneak/neoirc#79

2026-03-17 10:46:51 +01:00

feat: per-channel hashcash proof-of-work for PRIVMSG anti-spam

Review: PR #79 — Per-channel hashcash requirement for PRIVMSG

Build Result

docker build . PASSES — all tests pass (12 unit + 10 integration for hashcash, all existing tests green),…

clawbot commented on pull request sneak/upaas#167

2026-03-17 10:45:22 +01:00

feat: add private Docker registry authentication for base images

Review: PR #77 — docs: document register/login and dual authentication model (2nd rework)

Code Review: PR #169 (Re-review) — Observability Improvements

1. Policy Compliance

Rework (2nd): Fixed false "reclaimed later" claim and line-length issues

Review: PR #79 — Per-channel hashcash (post-rework)

Rework Summary

1. Security: IP spoofing in audit logs ✅

Review: PR #77 — docs: document register/login and dual authentication model (post-rework)

Review: PR #41 — Refactor long methods (closes #40)

Rework Complete

1. CHANMODES ISUPPORT classification (bug fix)

Review: PR #80 — feat: add runtime statistics to healthcheck endpoint

Rework: Fixed false session persistence claim

Review: PR #79 — Per-channel hashcash requirement for PRIVMSG

Build Result

Code Review: PR #167 — Private Docker Registry Authentication (Re-review)

Policy Compliance Check