fix(automation): scan all Claude PRs for review comments; drop author-based bot filter #131

Merged

shad0w merged 1 commit from claude/busy-boyd-b16092 into master

2026-06-24 10:28:17 +00:00

shad0w commented

2026-06-24 10:28:12 +00:00

Owner

Two issues surfaced on first live run:

The per-run cap truncated the PR list to the lowest-numbered PR, so a
comment on a higher-numbered PR was never scanned. The cap now limits only
how many PRs Claude RUNS on; comment-less PRs are skipped for free, so newer
PRs are never crowded out.
The bot posts as the repo owner's account, so excluding "the bot's own
comments" by author also excluded the owner's legitimate review comments
(and required a read:user token scope, which 403'd). Replaced with a guard
that excludes only comments carrying the HANDLED_TAG marker -- robust even
when the bot and the reviewer are the same account. The /user call is gone.

Co-Authored-By: Claude Opus 4.8 noreply@anthropic.com

Two issues surfaced on first live run: - The per-run cap truncated the PR list to the lowest-numbered PR, so a comment on a higher-numbered PR was never scanned. The cap now limits only how many PRs Claude RUNS on; comment-less PRs are skipped for free, so newer PRs are never crowded out. - The bot posts as the repo owner's account, so excluding "the bot's own comments" by author also excluded the owner's legitimate review comments (and required a read:user token scope, which 403'd). Replaced with a guard that excludes only comments carrying the HANDLED_TAG marker -- robust even when the bot and the reviewer are the same account. The /user call is gone. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

shad0w added 1 commit 2026-06-24 10:28:12 +00:00

fix(automation): scan all Claude PRs for review comments; drop author-based bot filter

PR checks / checks (pull_request) Successful in 49s

Details

PR checks / integration (pull_request) Successful in 32s

Details

e1907e6aec

Two issues surfaced on first live run:

- The per-run cap truncated the PR list to the lowest-numbered PR, so a
  comment on a higher-numbered PR was never scanned. The cap now limits only
  how many PRs Claude RUNS on; comment-less PRs are skipped for free, so newer
  PRs are never crowded out.
- The bot posts as the repo owner's account, so excluding "the bot's own
  comments" by author also excluded the owner's legitimate review comments
  (and required a read:user token scope, which 403'd). Replaced with a guard
  that excludes only comments carrying the HANDLED_TAG marker -- robust even
  when the bot and the reviewer are the same account. The /user call is gone.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>