docs(review): record the actual cause — the Greptile trial ran out (GTM-45) - #18
Merged
Conversation
…TM-45) The withdrawal record said "emission on this repository is not dependable." That was accurate about the symptom and wrong about the cause, and the difference decides what anyone does next: a flaky app is waited out, an exhausted trial is topped up. Established 2026-08-13. Greptile posts, in place of a review: `qmarcelle` has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan. and emits no check run. That is why the check fired on #10 and #13 and then stopped at #14 — the credits ran out in between, not the app's emission becoming unreliable here. The prediction that follows was confirmed the same day. `workspacejson/standard` kept working only until its own next pull request: #37 reviewed the exact head 4f9e8f6f and emitted zero check runs, against exactly one each on #34, #35 and #36. It hit the identical deadlock and its requirement was withdrawn the same way. So the "next door still works" observation in the original evidence was true but temporary, and reading it as repository-specific would have sent the investigation somewhere there was nothing to find. Restoring the signal is a billing action.
Reviewer's GuideThis PR updates the review documentation to correct the recorded root cause of missing Greptile check runs from a supposed repository-specific emission issue to the actual Greptile trial credit limit exhaustion, and aligns merge policy text with the corrected diagnosis and implications for remediation. Flow diagram for Greptile review behavior at trial credit limitflowchart TD
A[GitHub PR opened] --> B[Greptile trial credits exhausted?]
B -->|no| C[Greptile posts review]
C --> D[Greptile emits check run]
D --> E[Status check requirement can remain]
B -->|yes| F[Greptile posts credit limit notice]
F --> G[Greptile emits no check run]
G --> H[Status check requirement must be withdrawn]
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
qmarcelle has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.
|
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- The explanation of the Greptile credit-limit behavior is now duplicated across
calibration-2026-08.mdandmerge-policy.md; consider tightening the narrative in one place and linking to it from the other to keep future updates simpler. - In both files you describe timelines and PR numbers in prose; consider adding a short, structured summary (e.g., a small table or bullet list of PR → behavior) to make the causal chain easier to scan when debugging similar issues later.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The explanation of the Greptile credit-limit behavior is now duplicated across `calibration-2026-08.md` and `merge-policy.md`; consider tightening the narrative in one place and linking to it from the other to keep future updates simpler.
- In both files you describe timelines and PR numbers in prose; consider adding a short, structured summary (e.g., a small table or bullet list of PR → behavior) to make the causal chain easier to scan when debugging similar issues later.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.



Follow-up to #17, correcting a root cause I recorded wrong in that PR.
#17 stated "emission on this repository is not dependable." That was accurate about the symptom and wrong about the cause, and the difference decides what anyone does next: a flaky app is waited out, an exhausted trial is topped up.
What is actually happening
Greptile posts this in place of a review, and emits no check run:
That explains the timeline exactly: the check fired on
integrations#10 and #13 and stopped at #14 because the credits ran out in between — not because emission became unreliable on this repository.The prediction it implies was confirmed the same day
If the cause is account-wide,
workspacejson/standardwas not working — it simply had not opened a pull request since the credits ran out. It has now:standardhit the identical deadlock and its requirement was withdrawn the same way, recorded in that repository'sdocs/repository-settings.md.Why this correction is worth a PR
The original evidence leaned on "it still works next door" as proof the fault was local to
integrations. That observation was true and temporary, and acting on it would have sent the investigation looking for a repository-specific defect that does not exist.Restoring the signal is a billing action, not a debugging one. The re-admission rule is unchanged and still requires both a substantive review on the current head and a mechanically enforceable current-head signal — Greptile satisfied the second only while its trial lasted.
Summary by Sourcery
Clarify Greptile review behavior by correcting the documented root cause from a repository-specific emission issue to an exhausted trial credit limit and its implications for merge policy.
Documentation: