Skip to content

Move release security reviews into Factory - #29

Merged
matthewp merged 1 commit into
mainfrom
feat/release-security-review
Sep 1, 2026
Merged

matthewp merged 1 commit into
mainfrom
feat/release-security-review

Conversation

@matthewp

Copy link
Copy Markdown
Contributor

Changes

  • Routes same-repository Astro release and smoke-test PRs through a dedicated durable coordinator and Workflow, including current-head validation, supersession, watchdog recovery, and managed check rerequests.
  • Runs the reviewer against a credential-free, read-only CodeMode workspace with package baselines and published advisories, stores detailed reports privately, and publishes only fail-closed checks and sanitized comments.
  • Adds the required Worker bindings, migrations, sandbox capacity, and an active code-quality workflow.

Testing

  • Adds focused coverage for routing, contracts, checks, sanitized output, private report storage, sandbox staging limits, and package baselines.
  • Adds coordinator regressions for stale deliveries, explicit rerequests, stop retries, pending preservation, supersession ordering, and watchdog timeouts.

Docs

  • Updates the README with release-security behavior, GitHub App permissions, R2 setup, smoke validation, and cutover instructions.

Closes #13

@matthewp
matthewp merged commit 8f8f9ff into main Sep 1, 2026
1 check passed
@matthewp
matthewp deleted the feat/release-security-review branch September 1, 2026 12:33
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.

Move the security bot into the factory

1 participant