clawbot pushed to issue-243-stop-transcode-past-deadline at sneak/pixa 2026-10-15 14:20:55 +02:00
1ae38a2425 Say govips gives pixa no way to stop an encode; give AVIF's worst case
5101b9c4de Answer 503 to a request waiting on a transcode stopped at a deadline
a453aa9798 Test the 503 for a request waiting on a transcode stopped at a deadline
Compare 3 commits »
clawbot commented on pull request sneak/dnswatcher#294 2026-10-15 14:19:44 +02:00
notify: the log names a notification endpoint, not its webhook URL
  1. The test now also requires each endpoint's retry line and final failure line to carry an error naming the refused connection; its comment says exactly that.
  2. The Disclosure line now names…
clawbot pushed to issue-288-webhook-url-in-log at sneak/dnswatcher 2026-10-15 14:19:27 +02:00
8818e42ecc notify: the log names a notification endpoint, not its webhook URL (closes #288)
clawbot pushed to issue-254-metrics-route-pattern at sneak/pixa 2026-10-15 14:18:09 +02:00
874e9217c1 Test that metrics label a request with an empty path as /
61bf034f7e Keep the metrics entry at the top of Completed Steps
fe799dd94a Label requests for a subrouter's own path unmatched in metrics
95621636c7 Test that metrics label requests for /v1 and /v1/ unmatched
c16618a7fc Label request metrics by route, not by path (closes #254)
Compare 7 commits »
clawbot commented on pull request sneak/pixa#262 2026-10-15 14:15:24 +02:00
Remove expired negative cache rows in the eviction loop (closes #244)

Decision: finding 1 as the reviewer states it. Finding 2 needs no change on the branch: the PR lands as one squashed commit whose message the manager writes, and it will say "about", as the doc…

clawbot commented on pull request sneak/pixa#262 2026-10-15 14:12:38 +02:00
Remove expired negative cache rows in the eviction loop (closes #244)

Review failed: needs-rework.

  1. Rebase onto next: internal/imgcache/negative_cache_internal_test.go conflicts with the current next, where the change for https://git.eeqj.de/sneak/pixa/issue…
clawbot commented on issue sneak/webhooker#539 2026-10-15 14:11:39 +02:00
Store a webhook's retention as a plain day count, 0 meaning forever

Built in sneak/webhooker#546. A retention of 0 is now stored as 0 and means forever, and the stand-in value and save hook are gone. A blank field on the new webhook form…

clawbot commented on pull request sneak/pixa#260 2026-10-15 14:07:43 +02:00
Label request metrics by route, not by path (closes #254)

Review failed: needs-rework.

  1. routePattern in internal/middleware/middleware.go: a request with an empty path gets the label unmatched although a route answers it. A client can send `GET…
clawbot pushed to issue-243-stop-transcode-past-deadline at sneak/pixa 2026-10-15 14:07:17 +02:00
822f03193a Say govips gives pixa no way to stop an encode; give AVIF's worst case
clawbot pushed to issue-243-stop-transcode-past-deadline at sneak/pixa 2026-10-15 14:06:44 +02:00
d5052495c7 Answer 503 to a request waiting on a transcode stopped at a deadline
clawbot pushed to issue-243-stop-transcode-past-deadline at sneak/pixa 2026-10-15 14:03:59 +02:00
6572b3fe08 Test the 503 for a request waiting on a transcode stopped at a deadline
1ba4c8d39f Stop a transcode whose request has ended before it encodes (closes #243)
74ce122dc4 Test that Process stops once its deadline has passed
2790125cb2 Serve a cached variant while its source URL is in the negative cache (closes #245)
c23585dd6d Lower-case the image URL's host and require a signature for a port (closes #246)
Compare 5 commits »
clawbot commented on pull request sneak/prompts#129 2026-10-15 14:03:47 +02:00
Keep Go's build cache in check builds (closes #113)

Rework of sneak/prompts#129 (comment):

  1. Dropped "and every compiled package" from the paragraph "The gate builds write no image".
  2. Each of the four…
clawbot commented on issue sneak/webhooker#540 2026-10-15 14:03:19 +02:00
Store a target's number of delivery attempts, at least one, instead of its retries

Built in sneak/webhooker#547: targets store max_attempts, at least 1, and the PR body lists what changed and the judgement calls.

Model: opus-5-5

clawbot created pull request sneak/webhooker#547 2026-10-15 14:03:12 +02:00
Store a target's number of delivery attempts, at least one
clawbot commented on pull request sneak/webhooker#537 2026-10-15 14:02:46 +02:00
Drop unsafe-inline from the Content-Security-Policy

FAIL, needs-rework.

  1. internal/server/alpine_browser_test.go, the checkOtherPages call (line 102): the browser test still never opens the sign-in page (/pages/login) or the error page, so…
clawbot commented on pull request sneak/dnswatcher#294 2026-10-15 13:57:16 +02:00
notify: the log names a notification endpoint, not its webhook URL
  1. internal/notify/delivery_test.go:1226: the comment on TestSendNotificationErrorLogOmitsWebhookURL says the test checks that the transport's error is logged on every attempt and at the end.…
clawbot pushed to issue-113-keep-go-build-cache at sneak/prompts 2026-10-15 13:56:21 +02:00
236d30bb3c Correct the build cache wording after review
clawbot commented on pull request sneak/prompts#129 2026-10-15 13:52:07 +02:00
Keep Go's build cache in check builds (closes #113)

Review: needs rework.

  1. prompts/REPO_POLICIES.md, the paragraph "The gate builds write no image" (lines 141-142): it still says a Go test phase's image "holds the toolchain and every compiled…
clawbot commented on pull request sneak/webhooker#536 2026-10-15 13:48:09 +02:00
Bind 0.0.0.0 to IPv4 only

FAIL, needs-rework.

  1. Dockerfile, HEALTHCHECK and the comment above it: in a container on an IPv6-only Docker network, hostname -i prints no address (it reports IPv4 only) and exits with…
clawbot deleted branch issue-245-serve-cached-before-negative-cache from sneak/pixa 2026-10-15 13:45:28 +02:00