Skip to content

fix: bound prepublication scanner retries - #3353

Merged
Patrick-Erichsen merged 1 commit into
mainfrom
pe/investigate-prepublication-queue-backlog
Aug 2, 2026
Merged

fix: bound prepublication scanner retries#3353
Patrick-Erichsen merged 1 commit into
mainfrom
pe/investigate-prepublication-queue-backlog

Conversation

@Patrick-Erichsen

Copy link
Copy Markdown
Collaborator

Summary

  • bound consecutive pre-publication scanner execution failures to three attempts instead of retrying poison records forever
  • preserve fail-closed behavior and legacy in-flight compatibility by counting an existing failed scanner state as the first failure
  • clear the failure counter on clean, blocked, and finalization-retry transitions

Production evidence

Validation

  • VITE_CONVEX_URL=https://example.invalid bun run test -- convex/publishAttempts.test.ts convex/publishAttempts.runtime.test.ts convex/prepublicationObservability.test.ts scripts/security/run-prepublication-worker.test.ts --coverage=false — 58 passed
  • bun run ci:static
  • bun run ci:unit — 5,846 passed, 2 skipped; coverage gate passed
  • bun run ci:types-build
  • bun run ci:packages
  • bun run ci:static and bun run ci:types-build rerun after rebasing onto a15f97470f
  • .agents/skills/autoreview/scripts/autoreview --mode local --stream-engine-output — clean, no accepted/actionable findings

Safety

  • no production data changes
  • no deployment or moderation actions
  • terminal failed attempts remain republishable under the existing idempotency contract

@Patrick-Erichsen
Patrick-Erichsen requested a review from a team as a code owner July 31, 2026 15:38
@vercel

vercel Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
clawhub Ready Ready Preview Jul 31, 2026 3:41pm

Request Review

@Patrick-Erichsen
Patrick-Erichsen merged commit 50c4ffc into main Aug 2, 2026
30 of 31 checks passed
@Patrick-Erichsen
Patrick-Erichsen deleted the pe/investigate-prepublication-queue-backlog branch August 2, 2026 00:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant