Skip to content

fix(sse): make streaming fetch-start cap configurable - #15542

Merged
diegosouzapw merged 1 commit into
diegosouzapw:release/v3.8.52from
lorenzozanee:fix/configurable-fetch-start-timeout-cap
Oct 6, 2026
Merged

diegosouzapw merged 1 commit into
diegosouzapw:release/v3.8.52from
lorenzozanee:fix/configurable-fetch-start-timeout-cap

Conversation

@lorenzozanee

Copy link
Copy Markdown
Contributor

Summary

The streaming response-header wait remained capped at 110 seconds even when the overall fetch timeout was configured higher. Add OMNIROUTE_FETCH_START_TIMEOUT_CAP_MS to configure that streaming cap; the default remains 110 seconds, 0 disables it, and an explicit provider cap takes precedence.

Related Issues

Fixes #15504

Validation

  • Change type: other (streaming timeout policy)
  • Focused tests and category gates from the golden path
  • npm run lint
  • Reconciled with the current active release base; focused checks rerun afterward
  • Production-code changes include a new or updated automated test in this PR
  • SonarQube is temporarily opt-in while the private project has no quota; it is not a PR gate.

Tests Added Or Updated

  • tests/unit/fetch-start-timeout-policy-11526.test.ts
  • npm exec --yes --package=tsx -- tsx --test tests/unit/fetch-start-timeout-policy-11526.test.ts (4 passed)
  • npm exec --yes --package=tsx -- tsx --test tests/unit/runtime-timeouts.test.ts (12 passed)
  • Env-doc sync, changelog integrity, Prettier, and git diff --check passed locally.

Coverage Notes

The new test covers the timeout policy function. It does not instantiate BaseExecutor or exercise a real abort timer.

Reviewer Notes

The changelog fragment is named with issue number 15504; repository convention asks for a PR-number prefix, which was unavailable when the fragment was authored. npm run lint was not run. CI and review status are unknown.

@diegosouzapw
diegosouzapw merged commit f2b3dcd into diegosouzapw:release/v3.8.52 Oct 6, 2026
43 of 51 checks passed
diegosouzapw added a commit that referenced this pull request Oct 6, 2026
Follow-up da onda 6. Nenhum código de produto.

Tetos medidos no tip depois dos squashes:

- `open-sse/executors/opencode.ts` 1518 → 1519 (#15453)
- `tests/unit/combo/quota-weighted-strategy.test.ts` novo, congelado em 1217 (#15480)

O adopt de i18n grava o hash atual de `README.md` (#15541, #15467) e `docs/reference/ENVIRONMENT.md` (#15542). Não traduz.

File-size OK (171 congelados, 47 testes congelados). i18n drift PASS (sources=152, targets=10032).

⚠️ base-red inherited: #15306
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.

fix(): Fetch timeout even with absurdly high timeout set

2 participants