• Joined on 2026-02-08
clawbot commented on pull request sneak/dnswatcher#97 2026-08-08 01:45:07 +02:00
Remove DNS mocking from tests

Manager: post-rebase re-review by a fresh independent reviewer PASSED with zero findings (comment above). Setting merge-ready and assigning to sneak for merge (protected main).

Rebase…

clawbot commented on pull request sneak/dnswatcher#97 2026-08-08 01:44:44 +02:00
Remove DNS mocking from tests

Re-review (round 2, post-rebase): PR #97 (head a535ae8)

Verdict: PASS

Scope of this round: verify the rebase onto main 9347a28 (which merged #96) introduced no regressions relative to…

clawbot commented on pull request sneak/dnswatcher#97 2026-08-08 01:41:40 +02:00
Remove DNS mocking from tests

Rebased remove-dns-mocking onto current main (9347a28, which includes #96). New head: a535ae8. The branch remains a single clean commit and Gitea now reports the PR mergeable.

**Conflicts…

clawbot pushed to remove-dns-mocking at sneak/dnswatcher 2026-08-08 01:41:14 +02:00
a535ae864b Remove DNS mocking from tests; use live DNS everywhere
9347a2838b build: update golangci-lint to v2.12.2 with org-standard v2 config (#96)
f79cd98107 docs: document the no-DNS-mocking policy in README (closes #94) (#95)
Compare 3 commits »
clawbot commented on pull request sneak/dnswatcher#97 2026-08-08 01:39:54 +02:00
Remove DNS mocking from tests

Manager: #96 merged first, so this PR is the anticipated merges-second rebase case (needs-rebase, assignee clawbot). Rebase onto current main (9347a28) dispatched: #97's live-DNS rewrite…

clawbot opened issue sneak/pixa#57 2026-08-07 23:14:45 +02:00
Migrate .golangci.yml from deprecated gomodguard to gomodguard_v2
clawbot commented on issue sneak/pixa#51 2026-08-07 23:13:22 +02:00
P0: implement cache size management and eviction

Implementation is up as PR #55 (sneak/pixa#55), labeled needs-review and assigned to clawbot for the review loop. It follows the plan comment above; deviations and…

clawbot commented on pull request sneak/pixa#55 2026-08-07 23:13:15 +02:00
feat: cache size management and LRU eviction (closes #51)

Built and verified as described in the PR body. Summary of what was done and how it was checked:

Commits (branch feature/cache-size-eviction from main at 61f42e6, head c1ec038):

-…

clawbot opened issue sneak/pixa#56 2026-08-07 23:13:03 +02:00
Cache.Stats reads never-populated output_content/request_cache tables, so TotalItems and TotalSizeBytes are always 0
clawbot created pull request sneak/pixa#55 2026-08-07 23:12:47 +02:00
feat: cache size management and LRU eviction (closes #51)
clawbot pushed to feature/cache-size-eviction at sneak/pixa 2026-08-07 23:12:12 +02:00
c1ec038c99 docs: document cache_max_bytes, update TODO.md (closes #51)
bdd86a4c1e feat: DB-tracked cache size accounting with background LRU eviction
8cb09b6aaf feat: add cache_max_bytes config key with statfs-derived default
3963ec31c1 test: add failing tests for cache_max_bytes config and cache eviction
Compare 4 commits »
clawbot created branch feature/cache-size-eviction in sneak/pixa 2026-08-07 23:12:12 +02:00
clawbot commented on pull request sneak/webhooker#86 2026-08-07 23:06:18 +02:00
Update golangci-lint to v2.12.2 with canonical config

Manager note: independent review above (comment 44174) PASSED the rebased head 36a1bac with no blocking findings — conflict resolution verified lossless on both sides (#86 dedup and #84 archiving…

clawbot commented on pull request sneak/webhooker#86 2026-08-07 23:05:27 +02:00
Update golangci-lint to v2.12.2 with canonical config

Independent review of PR #86 (head 36a1bac)

VERDICT: PASS

Fresh adversarial review of head 36a1bacf11a2ba4d185fb5d5b448143c1978bc2d, a single commit rebased onto current main (ee7c626,…

clawbot commented on pull request sneak/pixa#54 2026-08-07 23:02:37 +02:00
Update golangci-lint to v2.12.2 with canonical config

Resolved the merge conflicts with main (PR #53, startup config validation) and brought the branch back to green. Two new commits:

  • 1a15b88 merge of main into golangci-v2.12.2
  • ce06170
clawbot pushed to golangci-v2.12.2 at sneak/pixa 2026-08-07 23:02:15 +02:00
ce06170604 chore: conform post-merge config validation code to v2.12.2 lint config
1a15b88971 Merge branch 'main' into golangci-v2.12.2
61f42e6602 feat: validate configuration on startup, fail fast on bad config (closes #52) (#53)
Compare 3 commits »
clawbot commented on pull request sneak/dnswatcher#97 2026-08-07 23:01:27 +02:00
Remove DNS mocking from tests

Manager: independent adversarial review PASSED (comment above). Setting merge-ready and assigning to sneak for merge (protected main).

Review highlights: no DNS mock/fake/stub remains…

clawbot commented on pull request sneak/dnswatcher#97 2026-08-07 23:00:59 +02:00
Remove DNS mocking from tests

Review: PR #97 (head 3959aed)

Verdict: PASS

Mandate verification

  1. No DNS mocking remains anywhere in the tree at this head. Full-tree grep for mock/fake/stub near DNS/resolver/n…
clawbot commented on pull request sneak/dnswatcher#96 2026-08-07 23:00:37 +02:00
build: update golangci-lint to v2.12.2 with org-standard v2 config

Manager: round-4 review PASSED with zero findings (comment above). Setting merge-ready and assigning to sneak for merge (protected main).

Final state at head 9054db8 (one clean commit on…

clawbot commented on pull request sneak/dnswatcher#96 2026-08-07 23:00:13 +02:00
build: update golangci-lint to v2.12.2 with org-standard v2 config

Independent re-review of PR #96 (round 4, head 9054db8)

VERDICT: PASS

Fresh adversarial review of head 9054db8d869c85e511949de21b1868c00d1afc55, one commit on current main (f79cd9810743e…