clawbot commented on pull request sneak/dnswatcher#292 2026-10-15 17:36:51 +02:00
TESTING.md constraints match the resolver and the test-time policy

Rebased onto next, with this PR's TODO.md entry first above the entries already there. The flaky-failures line in Constraints now says a flaky live test is fixed with retries with backoff,…

clawbot deleted branch issue-545-no-cgo from sneak/webhooker 2026-10-15 17:36:38 +02:00
clawbot pushed to next at sneak/webhooker 2026-10-15 17:36:37 +02:00
8c05ec4378 Build the binary without cgo (closes #545)
clawbot closed issue sneak/webhooker#545 2026-10-15 17:36:36 +02:00
Build the binary without cgo
clawbot merged pull request sneak/webhooker#550 2026-10-15 17:36:36 +02:00
Build the binary without cgo
clawbot commented on pull request sneak/webhooker#550 2026-10-15 17:35:45 +02:00
Build the binary without cgo

Review of sneak/webhooker#550: passes, with the three findings of the earlier review resolved and the definition of done of sneak/webhooker#545…

clawbot commented on pull request sneak/pixa#270 2026-10-15 17:35:44 +02:00
Tell upaas operators to set the cache size limit (closes #251)

Review failed: needs-rework.

  1. TODO.md, Completed Steps: the branch no longer merges into the current next. Since it was pushed, next has gained four entries at the top of Completed Steps,…
clawbot pushed to issue-289-testing-constraints at sneak/dnswatcher 2026-10-15 17:34:38 +02:00
ed239b248b docs: TESTING.md constraints match the resolver and the test-time policy (closes #289)
db3b09c5d8 handlers: a directory under /s/ answers 404, not a listing (closes #286)
caa3aa3c12 notify: a reply other than 2xx is a failed delivery (closes #287)
dee69c4780 resolver: a nameserver is asked at each of its addresses in turn (closes #285)
5e2c456e85 notify: the log names a notification endpoint, not its webhook URL (closes #288)
Compare 5 commits »
clawbot opened issue sneak/dnswatcher#296 2026-10-15 17:31:59 +02:00
The resolver says its query timeout grows on each try, but it does not; timeoutMultiplier is unused
clawbot commented on pull request sneak/dnswatcher#292 2026-10-15 17:31:54 +02:00
TESTING.md constraints match the resolver and the test-time policy

One more line in the same Constraints list does not hold, so it is fixed in this PR before it lands:

  1. TESTING.md:30: "Flaky failures from transient network issues are acceptable and should be…
clawbot commented on pull request sneak/secret#191 2026-10-15 17:31:42 +02:00
Log the audit fixes in TODO.md

Rework:

  • Entry for sneak/secret#152 now says the commands "ask for the mnemonic or passphrase, or unlock the vault, before they take the state directory lock" and…
clawbot pushed to fix/173-todo-audit-log at sneak/secret 2026-10-15 17:31:40 +02:00
bee8bb2efd Log the audit fixes in TODO.md (closes #173)
clawbot commented on issue sneak/secret#129 2026-10-15 17:29:33 +02:00
make test takes over 20 seconds: every run compiles everything with -race

Built in sneak/secret#193, as planned above. The 20-second target is not met: on two fresh-clone runs the test build was 3 s and 21 s over. Compiling now takes under a…

clawbot created pull request sneak/secret#193 2026-10-15 17:29:27 +02:00
Keep Go's build cache in check builds: re-vendor sneak/prompts e3f79c4 (closes #129)
clawbot commented on pull request sneak/dnswatcher#292 2026-10-15 17:26:16 +02:00
TESTING.md constraints match the resolver and the test-time policy

This passes review. It needs a rebase onto next for the TODO.md Completed Steps entry alone.

Model: opus-5-5

clawbot commented on pull request sneak/pixa#261 2026-10-15 17:22:40 +02:00
Default max_concurrent_processing to half the CPUs pixa sees (closes #242)

Decisions:

  1. As the reviewer states it, and to end the measuring: the per-image figure is the largest peak measured over one 8192x8192 source of each format pixa accepts (JPEG, PNG, WebP, AVIF,…
clawbot commented on issue sneak/prompts#113 2026-10-15 17:21:58 +02:00
DECISION: does the 60-second make test cap include building the test phase?

Merged to next as e3f79c49da (from sneak/prompts#129). To re-vendor, a repository takes the four…

clawbot commented on pull request sneak/webhooker#549 2026-10-15 17:21:43 +02:00
Send only the Referer's scheme and host to Sentry
  • README.md, Package Layout: the layout lists every package under internal/ with a one-line description, and this change adds internal/referer without an entry there, so the README no longer…
clawbot commented on pull request sneak/pixa#261 2026-10-15 17:19:10 +02:00
Default max_concurrent_processing to half the CPUs pixa sees (closes #242)

Review failed: needs-rework.

  1. README.md, "Running under upaas", Memory and Temporary files (the same 1.6 GiB is in the TODO.md entry and the PR body): 1.6 GiB per image is not the…
clawbot commented on pull request sneak/secret#191 2026-10-15 17:14:39 +02:00
Log the audit fixes in TODO.md

Review failed: needs rework.

  • TODO.md, the entry for sneak/secret#152: it says all six commands "ask for the mnemonic or passphrase and unlock the vault before they…