Skip to content

Add APPROVAL_POLICY for CLI sync auto-approval - #339

Merged
no-itsbackpack merged 2 commits into
mainfrom
egomez/approval-policy
Aug 12, 2026
Merged

Add APPROVAL_POLICY for CLI sync auto-approval#339
no-itsbackpack merged 2 commits into
mainfrom
egomez/approval-policy

Conversation

@no-itsbackpack

@no-itsbackpack no-itsbackpack commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add root APPROVAL_POLICY.md so Cursor PR Routing & Approval can auto-approve mechanical planetscale-cli[bot] sync PRs.
  • Low-severity Bugbot findings are explicitly not a hard stop; Medium/High still require Surfaces.

Test plan

  • Confirm Approval Agent is scoped to planetscale/planetscale-go and reads APPROVAL_POLICY.md
  • On the next sync PR (or re-trigger on Sync API client from cli@40cd697 #338), verify Approval Agent approves despite Low Bugbot findings
  • Confirm non-sync PRs are still routed to Surfaces and not auto-approved

Co-authored-by: Cursor <cursoragent@cursor.com>
@no-itsbackpack
no-itsbackpack requested a review from a team as a code owner August 12, 2026 20:35

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 68ecc6a. Configure here.

Comment thread APPROVAL_POLICY.md
Co-authored-by: Cursor <cursoragent@cursor.com>
@no-itsbackpack

Copy link
Copy Markdown
Collaborator Author

Addressed Bugbot: sync auto-approve now requires bot author and matching branch/title (not any).

@no-itsbackpack
no-itsbackpack merged commit 7b3d4f8 into main Aug 12, 2026
6 checks passed
@no-itsbackpack
no-itsbackpack deleted the egomez/approval-policy branch August 12, 2026 20:50
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.

3 participants