test(bundle): drive handleIssueComment's prow-commands input arms and per-command error isolation through dist/index.js

[hivecommons-hive]

Test Improvement

Adds __tests__/bundle/dispatcherArms.test.ts with the three handleIssueComment arms the bundle suite did not reach:

• prow-commands missing → exit 1, error contains please provide a list of space delimited commands / jobs to run, no api call
• prow-commands of only whitespace → same (second it.each row)
• /remove foo + /hold in one body with a non-collaborator → /remove fails (commenter is not authorized to remove a label), /hold still writes { labels: ['do-not-merge/hold'] }, exit 1

Errors are asserted as fragments (errors.some(e => e.includes(...))), and comment/repo/token come from the shared __tests__/bundle/helpers.ts. The unmatched-command, unsupported-name, alias, /remove-hold and /remove authorization cases from the first revision were dropped on review (covered by handleIssueComment.test.ts, hold.test.ts:103/172, remove.test.ts:28–100 and bundle.test.ts:803/866).

Verification on main @ a270568: npx eslint, npx tsc --noEmit, npx vitest run __tests__/bundle/dispatcherArms.test.ts — 3 pass.

Related Issue

Closes #251

---

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]

This repeats handleIssueComment.test.ts, hold.test.ts:103/172 and remove.test.ts:28–100. The bundle tests already cover hold-alias removal (/unhold, bundle.test.ts:866) and /remove (:803).

1. Keep only the missing/whitespace prow-commands input cases and the "one failing command doesn't stop another" case.
2. Check error fragments (.some(e => e.includes(...))) instead of pinning the full TypeError: error handling issue comment: Error: … string in four places.
3. Use a shared config-read list and comment() instead of copying them.

[hivecommons-hive]

Addressed all three: kept only the missing/whitespace prow-commands rows and the one-failing-command case; errors are fragment checks; comment/repo/token come from the shared helpers.ts (the copied config-read list is gone — none of the remaining cases reach the sweep). 212 → 46 lines. Rebased on main @ a270568.

---

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

— 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