fix: align useChatWebSocket test with 2-param onCatchUp contract

[simple-agent-manager]

Summary

One-line fix for CI failure after PR #898. The useChatWebSocket.behavioral.test.ts test still expected onCatchUp to be called with 3 arguments (including hasMore), but #898 simplified the contract to 2 params. This removes the stale false assertion.

Test plan

• useChatWebSocket.behavioral.test.ts — all 31 tests pass

Staging Verification

N/A: test-only change, no runtime behavior affected. Staging was verified for parent PR #898.

Agent Preflight (Required)

• Preflight completed before code changes

Classification

• external-api-change
• cross-component-change
• business-logic-change
• public-surface-change
• docs-sync-change
• security-sensitive-change
• ui-change
• infra-change

External References

N/A: test-only assertion alignment with already-merged code behavior.

Codebase Impact Analysis

• apps/web/tests/unit/hooks/useChatWebSocket.behavioral.test.ts removes the obsolete third-argument expectation from the catch-up callback assertion.

Documentation & Specs

N/A: no user-facing behavior or docs changed; this only aligns a stale test with the existing two-parameter callback contract.

Constitution & Risk Check

Checked Principle XI: no constants, URLs, limits, or configuration values changed. Risk is low because this is a one-line test-only fix.

Specialist Review Evidence

Reviewer	Status	Outcome
N/A	PASS	One-line test-only assertion update; no runtime, UI, security, Cloudflare, or Go code changed.

Co-Authored-By: Claude Opus 4.6 [email protected]

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud