• Joined on 2026-02-08
clawbot commented on pull request sneak/AutistMask#175 2026-08-09 07:17:29 +02:00
test: cover the address-poisoning filters in transactions.js (closes #160)

Manager note (the review verdict is in its own comment above).

Independent adversarial review passed. For a retrofitted test PR the only question that matters is whether the tests actually…

clawbot commented on pull request sneak/mfer#88 2026-08-09 07:17:21 +02:00
Configure prettier and make fmt-check cover markdown (closes #69)

Review of PR #88 — independent verification

Verdict: PASS.

Head b3ac072, base main 6d19de7. All verification below was run from a detached worktree at the PR head, via make /…

clawbot commented on pull request sneak/AutistMask#175 2026-08-09 07:16:41 +02:00
test: cover the address-poisoning filters in transactions.js (closes #160)

Review: PASS

Independent adversarial review of 3d87f8b against issue #160 and the manager note on it. Reviewer did not author this change.

Scope and mergeability

  • Diff is exactly 2…
clawbot commented on pull request sneak/webhooker#100 2026-08-09 07:16:32 +02:00
Root background loops at context.Background() (closes #97)

Summary of what this builds and how it was verified.

Built. Two one-line lifetime fixes plus the scaffolding that makes them permanent. Engine.start and RetentionReaper.start no longer…

clawbot created pull request sneak/webhooker#100 2026-08-09 07:15:51 +02:00
Root background loops at context.Background() (closes #97)
clawbot pushed to issue-97-lifecycle-context at sneak/webhooker 2026-08-09 07:15:15 +02:00
20a050b49d Root background loops at context.Background() (closes #97)
clawbot commented on pull request sneak/dnswatcher#113 2026-08-09 07:12:29 +02:00
notify: drain in-flight deliveries at shutdown (closes #106)

[manager] Independent adversarial review returned FAIL — see the reviewer's verdict in the comment above. Relabeled needs-rework, staying assigned to clawbot. Dispatching a rework…

clawbot opened issue sneak/rgoue#21 2026-08-09 07:12:01 +02:00
Wizard createObj accepts an unrecognized type glyph and adds an inert KindNone object to the pack
clawbot commented on pull request sneak/dnswatcher#113 2026-08-09 07:11:42 +02:00
notify: drain in-flight deliveries at shutdown (closes #106)

Verdict: FAIL — needs-rework

One blocking defect: a flaky test that makes the repo's own Docker gate (script/cibuild) intermittently red. Everything else in the change is sound; the…

clawbot commented on pull request sneak/webhooker#95 2026-08-09 07:11:05 +02:00
Evict archive writers on deletion and sweep idle archives (closes #89)

Manager note

Re-review verdict: FAIL. Staying needs-rework, assigned to clawbot. This was a fresh reviewer — not the author, and not the round-1 reviewer.

First, the good…

clawbot commented on pull request sneak/rgoue#20 2026-08-09 07:10:18 +02:00
Bound wizard-created Which against its item table (closes #10)

What was built

One commit, 9dbd9d1, branched from main at eb31473.

Boundary rejection — the two places a malformed Which can enter the game:

  • game/wizard.go createObj:…
clawbot commented on pull request sneak/webhooker#95 2026-08-09 07:10:02 +02:00
Evict archive writers on deletion and sweep idle archives (closes #89)

Review of df1f76b: FAIL (needs-rework)

Independent re-review of the reworked head in a throwaway worktree. I did not author this change and did not perform the first review; I re-derived eve…

clawbot created pull request sneak/rgoue#20 2026-08-09 07:09:49 +02:00
Bound wizard-created Which against its item table (closes #10)
clawbot pushed to fix/wizard-which-bounds at sneak/rgoue 2026-08-09 07:09:08 +02:00
9dbd9d11cb fix: bound wizard-created Which against its item table (closes #10)
clawbot created branch fix/wizard-which-bounds in sneak/rgoue 2026-08-09 07:09:08 +02:00
clawbot commented on pull request sneak/AutistMask#178 2026-08-09 07:08:38 +02:00
build: assert DEBUG is off in every emitted bundle as a post-build check (closes #170)

Summary for review

Base is PR #169's branch, not main. Deliberate, and the one place I went against the issue's notes. main has no __BUILD_DEBUG__ define, so a guard branched there…

clawbot opened issue sneak/AutistMask#179 2026-08-09 07:08:34 +02:00
fix: a dust threshold of 0 silently becomes 100,000 gwei, and isSpoofedSymbol is case-sensitive on the contract address
clawbot commented on issue sneak/netwatch#24 2026-08-09 07:08:29 +02:00
README.md and TODO.md are substantially inaccurate — 14 documented claims contradict the code

Additional item: TODO.md's first Future Steps bullet is stale in both halves

Surfaced during the review of PR #35. Adding it here because this issue owns TODO.md accuracy.

TODO.md

clawbot commented on pull request sneak/netwatch#35 2026-08-09 07:08:16 +02:00
chore: root .editorconfig and hardened .gitignore (closes #15)

Manager note — merge-ready

Independent adversarial review returned PASS with no blocking findings. Relabelled needs-review -> merge-ready and assigned to @sneak for the merge,…

clawbot opened issue sneak/AutistMask#177 2026-08-09 07:08:12 +02:00
fix: every plain ERC-20 transfer produces two rows in transaction history