Skip to content

Add Copilot's Open Review Findings Label to the Review Reader #3713

Closed

Description

@ptr727

scripts/pr_review.py status reports REVIEWER OUTPUT NOTED for a Copilot overview whose findings list is headed Open review findings:. The reader has no spelling for that label, so the items under it are placed nowhere and the round shows shapes=NOTED.

Seen on #3709, whose overview listed two findings under that heading. scripts/reviewers/copilot.json carries outstanding findings: and unresolved review findings: under labels and finding_lists (added by #3702 for #3672), but not open review findings:.

Fix: add open review findings: beside them in both lists, as #3702 did for its spellings.

Activity

  1. ptr727 commented on Nov 1, 2026

    @ptr727
    OwnerAuthor

    Released (2026-11-01): worktree and branch removed, so this claim holds no files. Its fix merged to develop in PR #3775. The issue closes when that is promoted.

    Backlog burndown claim (round 1, group G1). Branch: feature/burn-3758. Issues: #3758, #3571, #2180, #3732, #3713, #3726.

    Predicted file set:

    • scripts/pr_review.py, scripts/pr_review_rounds.py, scripts/pr_review_copilot.py, scripts/reviewers/copilot.json
    • tests/test_pr_review.py, tests/test_pr_review_copilot.py
    • .agents/skills/pr-review-conduct/SKILL.md and its generated trees (.github/skills/pr-review-conduct/, .claude-plugin/fleet-skills/skills/pr-review-conduct/, .claude-plugin/fleet-skills/.source-digests/pr-review-conduct)
    • drive-pr generated trees (included section), GOVERNANCE.md "PR Review Etiquette", .github/copilot-instructions.md
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