Found while fixing #796 (PR #797). corvid-pet's status table has no rows for preflight or lifecycle-gate, and it labels every skipped job "Not selected". So on a PR that needs a scoped review, a failed lifecycle gate still gets a passing review summary. After #797 the merge is blocked in that case, but the review comment is still misleading.
Expected: the summary lists preflight and lifecycle-gate, and tells "not selected" apart from "skipped because an upstream job failed" (the same rule #797 adds to implementation-gate).
🤖 Generated with Claude Code
Found while fixing #796 (PR #797).
corvid-pet's status table has no rows forpreflightorlifecycle-gate, and it labels every skipped job "Not selected". So on a PR that needs a scoped review, a failed lifecycle gate still gets a passing review summary. After #797 the merge is blocked in that case, but the review comment is still misleading.Expected: the summary lists
preflightandlifecycle-gate, and tells "not selected" apart from "skipped because an upstream job failed" (the same rule #797 adds toimplementation-gate).🤖 Generated with Claude Code