test(bundle): drive the status event's success path through dist/index.js as a row of the head-sha lookup case

[hivecommons-hive]

Test Improvement

Turns the existing check_suite completed without pull_requests: finds the pr by head sha case in __tests__/bundle/bundle.test.ts (tide describe) into an it.each(['check_suite', 'status']): the second row feeds a legacy commit-status payload ({ sha, state: 'success', context, repository }) through dist/index.js and asserts the same lookup-by-head-sha, merge and request sequence. That is the one status path the bundle suite did not reach; the no-open-pr, merge_on_events: false, missing-head-sha and ::debug:: text cases from the first revision were dropped on review (unit-covered at tide.test.ts:1000/1021/1042), along with the separate file and its copied helpers.

Net diff: +6/−2 in bundle.test.ts, no new file.

Verification on main @ a270568: npx eslint, npx tsc --noEmit, npx vitest run __tests__/bundle/bundle.test.ts -t "only a head sha" — both rows pass.

Related Issue

Closes #261

---

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

[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. Non-outreach agents are held at ACMM L3–L5; the outreach agent is always held because it publishes project-facing communication.

Hive will automatically remove the hold label once current policy no longer requires a level hold for "quality". If this is an outreach PR, a human must review it and remove the label.

[jeefy]

Every case here is already unit-tested (tide.test.ts:1000/1021/1042). The status-success merge follows the same code path as the existing check_suite test at bundle.test.ts:1497; only the payload field differs.

1. Keep only the status-success case, ideally as an it.each(['check_suite','status']) row on bundle.test.ts:1497.
2. Drop the exact ::debug::tide: merge_on_events is false… text check.
3. Don't copy the 10-path config-read list or expectRequests; share them (see the note about a shared __tests__/bundle/helpers.ts).

[hivecommons-hive]

Addressed: statusEvent.test.ts is gone; the status-success case is now the second row of an it.each(['check_suite', 'status']) on the existing head-sha lookup test in bundle.test.ts (+6/−2). No ::debug:: text check, no copied helpers. Rebased on main @ a270568.

---

🐝 Hive Agent: quality | Instance: hosted-available-lke648397-260827-5q9t | SHA: 6cfd4bd

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

[jeefy]

#273 is merged: __tests__/bundle/helpers.ts now exports repo, token, comment(), configReads(), ownersReads, ownersProbe, queueRead, membershipReads(), and via helpersFor(() => gh): calls(), expectRequests(), expectCommandThenConfig() and routeOwners(). Please rebase on main and import these instead of declaring local copies.

[jeefy]

Asks addressed, thanks.

[jeefy]

/kind cleanup
/lgtm
/approve
/hold cancel