[k11kirky]

Problem

The SSE keepalive test was intermittently hanging or failing because the setInterval mock was replacing all interval timers globally, including those used internally by undici, the HTTP server, and MSW. Without their periodic callbacks, those internals would stall and cause the test to hang.

Changes

• The setInterval spy now passes non-keepalive timers through to the real setInterval, so only the SSE keepalive interval is intercepted and captured.
• Increased the vi.waitFor timeout to 10 seconds with a 50ms polling interval to give the keepalive callback more time to register reliably.
• Increased the read loop attempts from 5 to 10 to allow more time for SSE data to arrive.
• Increased the overall test timeout from 20 to 30 seconds to accommodate the more lenient waiting behavior.

[k11kirky]

• feat(code): unify GitHub connect to auto-pick team vs user flow #2059
• feat(code): fix flakey tesk #2052 👈 (View in Graphite)
• feat(code): unify git integration to use git store and always use new git user integration #2050
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[greptile-apps]

_{Reviews (1): Last reviewed commit: "feat(code): fix flakey tesk" | Re-trigger Greptile}

[k11kirky]

Merge activity

• May 6, 4:56 PM UTC: A user started a stack merge that includes this pull request via Graphite.
• May 6, 4:58 PM UTC: Graphite rebased this pull request as part of a merge.
• May 6, 5:07 PM UTC: @k11kirky merged this pull request with Graphite.