Skip to content

Recognize Copilot Approval Recommended Review Heading #894

Description

@ptr727

Problem

scripts/pr_review.py treats the current Copilot code review heading ### Approval recommended as an unrecognized reviewer-output shape and blocks an otherwise complete review.

Observed on PR #893 at commit 5d02f7909d76d8bc40c695b791d034e2951de231.

The review begins:

Approval recommended

Documentation-only updates appear internally consistent with the updated finding log totals and the runner-selection plans stated routing controls.

The review also carries <!-- fleet-review: reviewed=2 changed=2 findings=0 --> and reports 2/2 files reviewed with zero comments.

Expected

Recognize this approval heading as a no-findings review while preserving the existing full-coverage and review-shape checks. Add a regression fixture for the exact heading.

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