Skip to content

chore(stamphog): Raise auto-review line ceiling to 1000#3246

Merged
charlesvien merged 1 commit into
mainfrom
chore/stamphog-loc-threshold
Jul 7, 2026
Merged

chore(stamphog): Raise auto-review line ceiling to 1000#3246
charlesvien merged 1 commit into
mainfrom
chore/stamphog-loc-threshold

Conversation

@charlesvien

Copy link
Copy Markdown
Member

Problem

Stamphog hard-refuses any PR over 500 changed lines, and team-code PRs keep landing just over that. Per Julian, this repo owns its copy of the agent so the limit is ours to tune. Upstream's .stamphog/policy.yml config doesn't apply here yet: our copy predates it and hardcodes the gate.

Changes

Raise MAX_LINES from 500 to 1000 in tools/pr-approval-agent/gates.py and update the README. MAX_FILES stays at 20. The refusal message in review_pr.py interpolates the constant, so no other references exist.

How did you test this?

No tests run. Grepped the tool and workflow for every reference to the ceiling; the constant and one README line are the only two.

Automatic notifications

  • Publish to changelog?
  • Alert Sales and Marketing teams?

@trunk-io

trunk-io Bot commented Jul 7, 2026

Copy link
Copy Markdown

Merging to main in this repository is managed by Trunk.

  • To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

@charlesvien charlesvien added the Stamphog This will request an autostamp by stamphog on small changes label Jul 7, 2026
@greptile-apps

greptile-apps Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "raise stamphog line ceiling to 1000" | Re-trigger Greptile

@github-actions github-actions 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.

Trivial config change raising the auto-review line ceiling in tooling only — no production code affected.

@charlesvien charlesvien enabled auto-merge (squash) July 7, 2026 19:34
@charlesvien charlesvien merged commit e5ce2e3 into main Jul 7, 2026
23 checks passed
@charlesvien charlesvien deleted the chore/stamphog-loc-threshold branch July 7, 2026 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Stamphog This will request an autostamp by stamphog on small changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant