Skip to content

fix(test): give stalled Antigravity body failover a 500ms budget - #402

Open
tt-a1i wants to merge 1 commit into
mainfrom
hive/delivery-antigravity-stalled-body
Open

fix(test): give stalled Antigravity body failover a 500ms budget#402
tt-a1i wants to merge 1 commit into
mainfrom
hive/delivery-antigravity-stalled-body

Conversation

@tt-a1i

@tt-a1i tt-a1i commented Sep 5, 2026

Copy link
Copy Markdown
Collaborator

Summary

Only the stalled non-2xx Antigravity body failover test.

  • timeoutMs 50 → 500
  • Outer bound 2 * 500 + 500 (1500ms)
  • First 503 body still stalls (write partial, no end)
  • Still expects requests === 2 and done.reason === "stop"

Production streamAntigravity is unchanged. This does not claim local reproduction of CI Node 26 / Windows requests 1 !== 2.

Test plan

  • Delivery head: bun run check; focused case ×8 8/0; antigravity.test.ts ×3 37/37
  • bun run test: this case passed (~508ms); the local suite later hung on workflows/execute.e2e.test.ts and was killed. Vitest was not reached via run-tests.mjs.
  • Separate Vitest: tests/extensions/file-search/index.spec.ts 30/30
  • CI Node 22 / 24 on Ubuntu
  • Windows CI job (this is the missing proof for the original Windows flake)

Node 26 is not in the current CI matrix and is not claimed here.

The 50ms timeout was too tight on loaded CI: first-body stall and the
follow-up request shared a wall-clock window that production resets
per endpoint. Widen only this test; keep the hung 503 body.
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