Skip to content

Teach the Copilot Reader the Unresolved Findings Label #3513

Closed

Description

@ptr727

Summary

Copilot's review overview now introduces its remaining findings with an Unresolved findings: label. scripts/reviewers/copilot.json lists changes:, review findings:, findings:, and open findings remain: and nothing else, so pr_review.py status reports that part of the body as unplaced (REVIEWER OUTPUT NOTED, "a label this reader has no spelling for: unresolved findings:").

Observed

The round on #3512 at 1ab4c893 used this label for three findings. Two had threads and one, on spec/files.json, appeared only in the body, so the reader's notice was the only signal that a body-only finding existed. It was answered by hand.

Fix

Add unresolved findings: to the labels list in scripts/reviewers/copilot.json, with a case in the reader's tests built from a constructed body carrying that label.

🤖 Generated with Claude Code

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