test(bundle): drive ok-to-test on pull_request synchronize and in the sweep through dist/index.js

[hivecommons-hive]

Test Improvement

Adds the two ok-to-test wiring cases dist/index.js was not driven through, inline in __tests__/bundle/bundle.test.ts:

• pull_request owners-label and blunderbuss describe: synchronize of a pull request labelled ok-to-test → after owners-label and approve's probe, the runs waiting on its head are read and the action_required one is approved; tide skips synchronize. Reuses the shared routeOwners/ownersReads/ownersProbe.
• schedule sweep job describe: a fork pr carrying ok-to-test → its pending run is approved before tide reads the pr and stops at the missing lgtm. Reuses that describe's forkPr/routeList/runSweep/listPage.

The separate sweepOkToTest.test.ts (13 cases, copied forkPr/routeList/routeOwnersLess/config-read list) was dropped on review: the other cases repeat sweep.test.ts:144–200, 454–490 and handlePullReq.test.ts:200, 228.

Verification on main @ a270568: npx eslint, npx tsc --noEmit, npx vitest run __tests__/bundle/bundle.test.ts — 90 pass (88 + 2).

Related Issue

Closes #259

---

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]

Only two cases are new to the bundle tests: ok-to-test on pull_request synchronize and in the sweep. The other ~11 repeat sweep.test.ts (144–200, 454–490) and handlePullReq.test.ts (200, 228).

1. Trim to those two or three wiring cases.
2. forkPr is a verbatim copy of bundle.test.ts:1824, and routeList, routeOwnersLess and the config-read list are copies too. Use shared helpers.

[hivecommons-hive]

Addressed both: trimmed to the two wiring cases (synchronize + sweep), now inline in bundle.test.ts so they reuse the existing forkPr/routeList and the shared routeOwners/ownersReads; the separate file and its copies are gone. +34 lines net. Rebased on main @ a270568.

---

🐝 Hive Agent: quality | Instance: hosted-available-lke648397-260827-5q9t | SHA: 47730be

— 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.

forkPr and routeList aren't in it yet (only one merged suite used them); move them into helpers.ts in this PR and import them from bundle.test.ts too, rather than copying. routeOwnersLess should become a routeOwners() call.

[jeefy]

Asks addressed, thanks.

[jeefy]

/kind cleanup
/lgtm
/approve
/hold cancel