[hivecommons-hive]

Test Improvement

Adds five cases to __tests__/plugins/approveReview.test.ts, one per non-equivalent mutant of src/plugins/approveReview.ts that survived the full related suite (46 files / 920 tests) despite 100 % line coverage:

line	mutant	new case
188	identity.login !== undefined && identity.login === owners.author → ||	approve.github_review: approved › a user token (GET /user → Prow-Bot) on a pull request by some-author submits the review: calls = ['label', 'notifier', 'review'], no warning
105	uncoveredFiles.length > reasonFiles → >=	withdrawalReason › exactly five files → no approver covers a, b, c, d, e
111	continue → break	withdrawalReason › carol lgtm-cancel then bob cancel with lgtm_acts_as_approve: false → withdrawn by bob (/approve cancel)
113	event.user.toLowerCase() → event.user	withdrawalReason › bob cancel then Bob approve → no withdrawal; Bob approve then BOB cancel → withdrawn by bob
118	.sort() removed	withdrawalReason › dave, bob, carol acting in that order → listed bob (changes requested), carol (/approve cancel), dave (/approve cancel)

Verified by re-applying each mutant with the new file: every one now fails exactly one test (npx vitest run __tests__/plugins/approveReview.test.ts → 1 failed / 42 passed); on clean main the file passes 43/43 and eslint is clean.

Test-only: no src/ or dist/ change. Files/functions claimed: __tests__/plugins/approveReview.test.ts (approve.github_review: approved, withdrawalReason describes) — disjoint from held PRs #510–#525 (#522 touches approve.test.ts/approveEvents.test.ts, not this file).

Related Issue

Closes #526

---

Filed by quality agent (hold-gated mode). Human review required.

— hive: agent=quality backend=copilot model=claude-fable-5.1 copilot=1.0.88

[hivecommons-hive]

Important

Held for human review by the hive's ACMM level gate.

This PR was opened by the "quality" agent while Hive policy required a human checkpoint for that agent. Agent PRs are held at ACMM L3–L5; an outreach PR that targets a repository this hive does not manage is held at every level because it publishes project-facing communication into someone else's project.

Hive will keep the hold label until a human removes it. Operators can make a deliberate one-off release during an ACMM level change with release_level_holds=true, but level changes never release this hold automatically.

[github-actions]

Please add a kind label with /kind feature, /kind failing-test or /kind cleanup.

[mrbobbytables]

Copilot review performed under the maintainer's explicit authorization to update, review, release reviewed holds, and merge eligible PRs in this cleanup.

The non-author PAT case observes the review write, and the withdrawal cases distinguish the five-file boundary, case-insensitive identity, lgtm skipping and sorted names.

Verified successful build-test, e2e-coverage and DCO results on head 28def9c. Any folded issue findings remain assigned to the canonical destinations in the updated scope tables.