brooktrails/pr-reviewer
2
0
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity Actions

fix: post pending comment before settle delay #13

Merged
rcsheets merged 1 commit from fix/immediate-pending-comment into main 2026-03-21 20:07:22 +00:00
Conversation 1 Commits 1 Files changed 1 +44 -34
rcsheets commented 2026-03-21 20:05:43 +00:00
Owner
Copy link

The pending comment is now posted immediately after fetching the diff,
before the settle period. Users see feedback within ~1 second instead
of waiting 5+ seconds. If the tier upgrades after settling, the
comment is updated to reflect the change.

Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com

The pending comment is now posted immediately after fetching the diff, before the settle period. Users see feedback within ~1 second instead of waiting 5+ seconds. If the tier upgrades after settling, the comment is updated to reflect the change. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The pending comment is now posted immediately after fetching the diff,
before the settle period. Users see feedback within ~1 second instead
of waiting 5+ seconds. If the tier upgrades after settling, the
comment is updated to reflect the change.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
pr-reviewer-bot commented 2026-03-21 20:06:46 +00:00
Collaborator
Copy link

Automated review by pr-reviewer v0.13.0 | Safety Check | anthropic | tracking id r-befa19-61e820
This is an AI-generated review and may contain mistakes.

Status: ✅ Completed

✅ Verdict: LGTM — The change improves user experience by posting feedback immediately while preserving the existing logic flow and error handling.

This is a straightforward refactor that moves the pending comment posting to happen immediately after fetching the diff, before the settle delay. The logic for tier upgrades and settle gap recording is preserved and moved after the initial comment. The change adds proper comment updating when a tier upgrade occurs. No security issues, breaking changes, or obvious bugs detected.

*Automated review by [pr-reviewer](https://git.brooktrails.org/brooktrails/pr-reviewer) v0.13.0 | Safety Check | anthropic | tracking id `r-befa19-61e820`* *This is an AI-generated review and may contain mistakes.* **Status:** ✅ Completed --- **✅ Verdict: LGTM** — The change improves user experience by posting feedback immediately while preserving the existing logic flow and error handling. This is a straightforward refactor that moves the pending comment posting to happen immediately after fetching the diff, before the settle delay. The logic for tier upgrades and settle gap recording is preserved and moved after the initial comment. The change adds proper comment updating when a tier upgrade occurs. No security issues, breaking changes, or obvious bugs detected.
rcsheets deleted branch fix/immediate-pending-comment 2026-03-21 20:07:22 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
brooktrails/pr-reviewer!13
No description provided.