Skip to content

Read Copilot's Emoji-Prefixed Review Effort Line in pr_review.py #2487

Closed

Description

@ptr727

What happens

Copilot's revised ccr-overview-v2 body states its review effort on a line that opens with an emoji:

🧠 **Review effort:** Lite

EFFORT_LINE in scripts/pr_review.py allows only whitespace or a list bullet before **Review effort, so this line does not match, and status and wait print effort=unknown effort_source=unknown over a body that states the level outright.

Seen on #2484, round on aaddb224.

Impact

Effort does not gate coverage or completion, so no merge decision is affected. The digest under-reports what the round said, which is the misreading the EFFORT_LINE comment says the second spelling was added to stop.

Done means

  • EFFORT_LINE reads the emoji-prefixed spelling, with a fixture of that line.
  • The effort= field reports lite for a body carrying it.

Related: #2479 and #2482 cover the same revised format's resolved-count and open-findings lines, not this one.

Activity

  1. added a commit that references this issue on Oct 8, 2026
    4a0d9a0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions