Issue Station
Sign In
Tools hereToolbox

codex - 💡(How to fix) Fix babysit-pr watcher ignores Copilot PR reviews from copilot-pull-request-reviewer [1 comments, 2 participants]

Official PRs (…)
ON THIS PAGE

Recommended Tools

×6

Utilities matched from this issue’s tags and category — try them while you read without losing context.

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

Helpful · Quick feedback

Loading…
GitHub stats
openai/codex#19148•Fetched 2026-04-24 05:59:51
View on GitHub
Comments
1
Participants
2
Timeline
8
Reactions
0
Author
clairernovotnyclairernovotny
Participants
clairernovotnyclairernovotnyetraut-openaietraut-openai
Timeline (top)
labeled ×4unlabeled ×2closed ×1commented ×1

Root Cause

The babysit-pr watcher can miss actionable PR feedback from GitHub Copilot because it filters review authors too aggressively.

RAW_BUFFERClick to expand / collapse

What issue are you seeing?

The babysit-pr watcher can miss actionable PR feedback from GitHub Copilot because it filters review authors too aggressively.

In the current main version of .codex/skills/babysit-pr/scripts/gh_pr_watch.py, review items are only surfaced when either:

  • the author login ends with [bot] and contains an allowed keyword from REVIEW_BOT_LOGIN_KEYWORDS (currently just "codex"), or
  • the author is treated as a trusted human with authorAssociation in OWNER, MEMBER, or COLLABORATOR

That means a review authored by copilot-pull-request-reviewer with authorAssociation: NONE is dropped entirely, even if it contains actionable feedback.

What steps can reproduce the bug?

  1. Start monitoring an open PR with the babysit-pr skill / gh_pr_watch.py --watch or --once.
  2. Have GitHub Copilot leave a top-level PR review as copilot-pull-request-reviewer.
  3. Observe that the review is present in GitHub, but the watcher reports no new_review_items for it.
  4. Inspect the current filter logic in gh_pr_watch.py around is_bot_login, is_actionable_review_bot_login, is_trusted_human_review_author, and fetch_new_review_items.

What is the expected behavior?

The watcher should surface actionable Copilot review feedback instead of silently ignoring it.

At minimum, copilot-pull-request-reviewer reviews should be treated as eligible review items in the same spirit as the existing Codex review-bot allowance.

Additional information

I verified this against the current openai/codex main branch on April 23, 2026. The local copy of the skill matched upstream for gh_pr_watch.py, so this does not appear to be a stale local fork.

Relevant current logic:

  • REVIEW_BOT_LOGIN_KEYWORDS = {"codex"}
  • is_bot_login(login) requires login.endswith("[bot]")
  • is_trusted_human_review_author(...) only accepts OWNER, MEMBER, or COLLABORATOR

One concrete observed case was a review from copilot-pull-request-reviewer with state: COMMENTED and authorAssociation: NONE; it was visible via gh pr view --json reviews but filtered out by the watcher.

A plausible fix would be to expand the actionable-review-author detection so known automated reviewers like Copilot are not excluded just because they do not use a [bot] suffix or a trusted human association.

extent analysis

TL;DR

The babysit-pr watcher can be fixed by expanding the is_actionable_review_bot_login function to include copilot-pull-request-reviewer as an eligible review author.

Guidance

  • Review the current filter logic in gh_pr_watch.py around is_bot_login, is_actionable_review_bot_login, and is_trusted_human_review_author to understand how reviews are being filtered.
  • Consider adding copilot-pull-request-reviewer to the REVIEW_BOT_LOGIN_KEYWORDS set or creating a separate list of known automated reviewers like Copilot.
  • Update the is_actionable_review_bot_login function to check for copilot-pull-request-reviewer explicitly, in addition to the existing [bot] suffix check.
  • Verify the fix by running the babysit-pr watcher with the updated logic and checking if it correctly surfaces actionable Copilot review feedback.

Example

# Example update to is_actionable_review_bot_login function
def is_actionable_review_bot_login(login):
    # Existing logic
    if login.endswith("[bot]") and any(keyword in login for keyword in REVIEW_BOT_LOGIN_KEYWORDS):
        return True
    # New logic to include copilot-pull-request-reviewer
    elif login == "copilot-pull-request-reviewer":
        return True
    return False

Notes

This fix assumes that copilot-pull-request-reviewer is the only automated reviewer that needs to be included. If there are other reviewers that should be included, they will need to be added to the logic as well.

Recommendation

Apply workaround: Update the is_actionable_review_bot_login function to include copilot-pull-request-reviewer as an eligible review author, as this will allow the babysit-pr watcher to surface actionable Copilot review feedback without requiring a full rewrite of

Vote matrix · Quick signals

Works
Did the solution work? Tap to confirm.
Easy Fix
Was it a quick fix?
Time Saver
Did it save you time?
Blocking
Was it severely blocking?
Common Issue
Are others likely hitting this too?
Flaky / Intermittent
Is it intermittent?
Verified / Reproducible
Can you reproduce it reliably?
Loading…
#device allocation#model download#tokenizer error#prompt formatting#chain error

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Back to top recommendations

TRENDING