Skip to content

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

Merged
tt-a1i merged 1 commit into
mainfrom
hive/delivery-antigravity-stalled-body
Sep 5, 2026
Merged

fix(test): give stalled Antigravity body failover a 500ms budget#402
tt-a1i merged 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.
@tt-a1i
tt-a1i merged commit d06c085 into main Sep 5, 2026
5 checks passed
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