Skip to content

Chore(PE-1694): Greptile auto-approve for low risk PRs - #49

Merged
plain-thinh-tran merged 4 commits into
mainfrom
PE-1694/greptile-auto-approve-for-docs-and-tests
Oct 27, 2026
Merged

plain-thinh-tran merged 4 commits into
mainfrom
PE-1694/greptile-auto-approve-for-docs-and-tests

Conversation

@plain-thinh-tran

@plain-thinh-tran plain-thinh-tran commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Fixes PE-1694

Slack Context

Adds a .greptile/config.json that lets Greptile auto-approve any PR it scores 5/5 and rates Low risk.

Why

This PR lets low risk PRs merge without waiting for a human reviewer, the same way dbt, omni and datadog already do. Today every PR here waits for a person, even a README fix.

PR opened ─▶ Greptile review ─▶ 5/5 and Low risk ─▶ approved
                              └▶ anything else ───▶ human review, as today
  • No path limit: Greptile's own risk rating decides. It rates CI, infra, auth and public APIs as Critical, so the Low cap keeps those with a human.
  • Never auto-approved: anything under .greptile/, so the review policy and rules always need a human.
  • The npm release still needs a human: the changesets version PR is opened by github-actions[bot], which Greptile does not review.
  • Greptile reads this policy from the base branch, so a PR cannot loosen its own policy.

What Comes Next

We will dogfood this idea, since we are realizing that reviews are very precious and consume a lot of time.

We will expand the config slowly, and look back at the PRs it touched to learn which really needed a human review and which didn't, narrowing it again where it was wrong.

🤖 Generated with Claude Code


Note

Low Risk
Only adds Greptile review/approval policy JSON; no application or runtime code changes.

Overview
Adds Greptile repo config so reviews run on open and push, and auto-approve is on when Greptile rates the PR Low risk (with a low risk ceiling).

Changes under **/.greptile/** are excluded from auto-approve so policy edits still need a human. Everything else is eligible based on Greptile’s risk score, not path-based allowlists.

Reviewed by Cursor Bugbot for commit bc9d8cb. Bugbot is set up for automated code reviews on this repo. Configure here.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@plain-thinh-tran
plain-thinh-tran force-pushed the PE-1694/greptile-auto-approve-for-docs-and-tests branch from f090f8c to fae0215 Compare October 27, 2026 14:43

@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, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit fae0215. Configure here.

Comment thread .greptile/config.json Resolved
@plain-thinh-tran plain-thinh-tran self-assigned this Oct 27, 2026
@plain-thinh-tran
plain-thinh-tran marked this pull request as ready for review October 27, 2026 14:51
@plain-thinh-tran
plain-thinh-tran requested a review from a team as a code owner October 27, 2026 14:51
@greptile-apps

greptile-apps Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The PR appears safe to merge; no actionable defect was established.

Reviews (3) · Last reviewed commit: "Chore(PE-1694): widen Greptile auto-appr..." · Reviewed by Greptile

@revett revett left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@plain-thinh-tran I'm assuming that this is limited to only folks within our org? As this is a public repo.

The comment from Cursor makes sense 👍

@plain-thinh-tran

Copy link
Copy Markdown
Contributor Author

@revett yeah just saw it! will adjust it on all of the PRs

plain-thinh-tran and others added 3 commits October 27, 2026 15:55
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@plain-thinh-tran plain-thinh-tran changed the title Chore(PE-1694): Greptile auto-approve for docs and tests Chore(PE-1694): Greptile auto-approve for low risk PRs Oct 27, 2026
@plain-thinh-tran
plain-thinh-tran merged commit 388ddd2 into main Oct 27, 2026
11 checks passed
@plain-thinh-tran
plain-thinh-tran deleted the PE-1694/greptile-auto-approve-for-docs-and-tests branch October 27, 2026 16:46
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.

2 participants