Skip to content

fix(ibkr): arm handshake listeners before greeting - #731

Merged
luokerenx4 merged 1 commit into
devfrom
codex/fix-ibkr-handshake-close-race
Jul 28, 2026
Merged

fix(ibkr): arm handshake listeners before greeting#731
luokerenx4 merged 1 commit into
devfrom
codex/fix-ibkr-handshake-close-race

Conversation

@luokerenx4

Copy link
Copy Markdown
Contributor

Summary

  • install IBKR server-version handshake listeners before sending the API greeting
  • prevent fast local peer closes, especially on Windows, from being missed until the 10-second fallback timer
  • assert the listener ordering in the existing server-side handshake-close regression test

Verification

  • pnpm exec vitest run services/uta/src/domain/trading/brokers/ibkr/request-bridge.spec.ts services/uta/src/uta-startup-resilience.spec.ts (19 passed)
  • pnpm -F @traderalice/ibkr typecheck
  • npx tsc --noEmit
  • pnpm test (3368 passed, 9 skipped)

The first full-suite run hit an unrelated timeout in SettingsPage.data-home.spec.tsx; that spec passed immediately in isolation and the subsequent full-suite run passed.

@vercel

vercel Bot commented Jul 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
openalice-demo Building Building Preview, Comment Jul 28, 2026 7:51am

Request Review

@luokerenx4
luokerenx4 merged commit a52d747 into dev Jul 28, 2026
12 of 13 checks passed
@luokerenx4
luokerenx4 deleted the codex/fix-ibkr-handshake-close-race branch July 28, 2026 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant