Skip to content

docs(repo): add solo-maintainer review exception to workflow - #9

Merged
raelldottin merged 1 commit into
mainfrom
chore/solo-maintainer-review-exception
Sep 7, 2026
Merged

raelldottin merged 1 commit into
mainfrom
chore/solo-maintainer-review-exception

Conversation

@raelldottin

Copy link
Copy Markdown
Contributor

Summary

The website repo's AGENTS.md workflow still requires review → merge unconditionally. Alepes is currently a solo maintainer, so an external-approval requirement adds ceremony without improving review quality. This records the same narrowly-scoped review exception that governs the product repo (alepes-com/alepes), trimmed to this static-site repository.

Change

  • Added a Solo-maintainer exception to review section under ## Workflow.
  • The author may merge without an external approver only while a solo maintainer holds merge authority, and only when all of: required CI green on the exact head, full validation ladder passes, a substantive self-review is recorded on the PR, no unresolved blocking review threads remain, and a normal merge commit (never squash/rebase) is used.
  • The exception self-terminates the moment a second maintainer with merge authority appears.

Note on scoping

Dropped the product repo's milestone-certification bullet (e.g. credentialed Plaid Sandbox) — nothing of that kind applies to a static marketing site.

No code, behavior, or validation changes.

Record the narrow review-policy exception under which a single maintainer
may merge without an external approver, conditioned on green CI on the exact
head, a passing validation ladder, and a recorded substantive self-review.
Mirrors the product repo's exception, trimmed to this static-site repo
(no Plaid/certification provisions apply here). The exception ceases the
moment a second maintainer with merge authority is available.
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
alepes-website 4dbb966 Sep 07 2026, 11:34 AM

@raelldottin raelldottin left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Self-review (solo-maintainer exception)

Scope: documentation-only — adds the review-policy exception to AGENTS.md. No code, no behavior, no validation changes.

Verified against the exception's own bar:

  • CI green on exact head (validate (bun 1.4) pass; Workers Builds: alepes-website pass).
  • Change is a faithful, narrowly-scoped clone of the product repo's exception, with the product-specific certification bullet (Plaid Sandbox) intentionally removed — nothing of that kind applies to this static marketing site.

Material findings: none. The added text is declarative and cannot affect the build, tests, or the static export.

Reviewed-by self per the recorded exception; merging with a normal merge commit (not squash).

@raelldottin
raelldottin merged commit acb82f5 into main Sep 7, 2026
2 checks passed
@raelldottin
raelldottin deleted the chore/solo-maintainer-review-exception branch September 7, 2026 11:35
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