The merge-workflow Bats cases are retired where Go covers them #881

Problem

tests/skills/merge-pr/scripts/merge-workflow.bats (757 lines, 60 cases) is the attended end-to-end suite for preflight, squash-message, review and squash-merge, run through fake gh and scratch repositories. After the wrappers exec Go, children 04 and 06 to 12 hold equivalent Go tests, so most of these cases duplicate them. The wrapper-to-binary hand-off and the N-2 fallback are the cases that still need a real shell. The ledger holds a bounded inventory: one row per case, 60 in all, grouped by script.

Part of #324, within #588. Child 19 in #324's Children list. Retires cases of tests/skills/merge-pr/scripts/merge-workflow.bats.

Acceptance criteria

1. AC1: Given each of the 60 cases in the files named above, when the ledger is read, then the case is deleted with the name of the Go test that covers it, or kept with a one-line reason it needs a real shell. The ledger also lists any case a prerequisite issue added to these files before this change, with the same disposition.
2. AC2: Given the Go tests named in the ledger, when mise run test-go runs, then each passes and asserts the same outcome as the Bats case it replaces (the coverage comes from children 04 and 06 to 12).
3. AC3: Given the cases kept, when mise run test-hooks runs on the files, then they pass against the wrappers.
4. AC4: Given a deleted case, when a regression is introduced in the behavior it covered, then a named Go test fails; the pull request shows one such mutation per ten deleted cases.
5. AC5: Given a search of both test families (tests/skills and tests/hooks) and of the documentation for the helper names and files this change leaves unused, when it runs, then no reference to a removed Bats case or helper remains, and the shared helpers that other suites still load stay in place for child 24.
6. AC6: Non-functional: the Bats suite wall time does not increase.

Scope and non-goals

Scope: the Bats files named in the problem, helpers loaded by them alone, and the ledger in the pull request description.

Non-goal: porting new behavior or editing production code. Non-goal: removing shared helpers other suites still load, which child 24 does. Non-goal: the agent-merge-pr suites and the two delegation files, handled by children 15 to 18. Kept cases include one per wrapper proving the exec hand-off and one proving the N-2 fallback.

Approach

Meet AC1 to AC6 by building the ledger first, then deleting cases whose Go counterpart exists, then mutation-checking a sample and running the reference audit.

Dependencies

Blocked by: #874, #875, #876, #877, #878, #879, #880, #333.

Prerequisites: #333.

Size

S. One pull request, about 760 lines changed, almost all deletions, under 800 code lines; if the ledger shows the kept cases push it over, split by script before opening it.

Verify plan

• AC1: the ledger lists all 60 cases, plus prerequisite additions, with a disposition each.
• AC2, AC3: mise run test-go and mise run test-hooks tests/skills/merge-pr at the merged head.
• AC4: the sampled mutation runs, with the failing test names.
• AC5: the search output, run over both test families and the documentation.
• AC6: Bats wall times before and after.
• Definition of done: D1 each AC is verified at the merged head, with the commands and results cited in the pull request's Verification. D2 an independent review approves the exact merged head, and every finding has a recorded disposition. D3 required checks pass with no gate loosened. D4 the leak scans the repository specifies are clean, and every post made for this work passes the per-post leak review before it is posted. D5 every commit is signed. D6 the documentation matches the change or the Verification says why there is no impact. D7 every commit has an Assisted-by trailer and the pull request has the provenance:agent-assisted label. D8 the pull request stays within the size limits. D9 follow-ups are filed as issues and listed under Related. D10 the merge closes the issue through a closing reference, the board shows Done, and the final Status: comment names the merge commit.

Risks

A deleted case with no true Go equivalent loses coverage of an attended refusal path on an irreversible write. The ledger and the mutation sample are the controls. Undo: revert the deletion.

Operator authority

None.