fix(tsc): exactOptionalPropertyTypes — bus.ts/claim.ts/bus.test.ts

[AceHack]

Summary

• bus.ts: Replace two identical list({ topic: topicFilter, to: toFilter }) calls (where both args are T | undefined) with a shared listOpts object built via conditional spread — omits the property entirely when undefined, satisfying exactOptionalPropertyTypes: true
• claim.ts: Conditional spread for optional branch field in ClaimRecord push — property is omitted when p.branch is undefined
• bus.test.ts: Non-null assertion on msgs[0] guarded by the preceding expect(msgs.length).toBeGreaterThan(0)

These were pre-existing non-required lint (tsc tools) failures firing on every PR since the exactOptionalPropertyTypes: true tsconfig flag rejects T | undefined being passed to an optional T? parameter.

Test plan

• bun --bun tsc --noEmit -p tsconfig.json → 0 errors (verified locally)
• dotnet build -c Release → 0 warnings, 0 errors (no .NET changes)

🤖 Generated with Claude Code

Co-Authored-By: Claude noreply@anthropic.com

[Copilot]

Pull request overview

Fixes TypeScript exactOptionalPropertyTypes errors in the inter-agent bus tooling by ensuring optional fields are omitted (not passed as T | undefined) and by making a single test assertion type-safe under noUncheckedIndexedAccess.

Changes:

• tools/bus/bus.ts: Build a shared listOpts object that conditionally includes topic/to, and reuse it for both list(...) calls in watch.
• tools/bus/claim.ts: Omit branch from ClaimRecord when the payload has no branch.
• tools/bus/bus.test.ts: Add a non-null assertion for msgs[0] after a length guard.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
tools/bus/claim.ts	Omits optional branch when undefined to satisfy exactOptionalPropertyTypes.
tools/bus/bus.ts	Reuses a conditionally-built filter options object when listing messages in watch.
tools/bus/bus.test.ts	Makes array indexing type-safe with a non-null assertion after an explicit length check.