test(provider): expect the 300 s Altimate Base header timeout set by #1260

AN
2mo ago
+5-2

Issue for this PR

Closes #1264

Type of change

  • Bug fix (test)

What does this PR do?

main's TypeScript job has failed since #1260 on one assertion in test/provider/provider.test.ts: the Altimate Base provider contract test still expects the openai loader's 10 s header timeout, while #1260 deliberately raised it to FREE_TIER_HEADER_TIMEOUT_DEFAULT = 300_000 (env-overridable via ALTIMATE_BASE_HEADER_TIMEOUT_MS) because the gateway can queue for capacity, cold-start, or reason before flushing headers. The assertion now expects 300_000 and its comment explains why the value is not OpenAI's default. One-line behavioural test change; no source change.

How did you verify your code works?

  • Read the failing job on main (run 34200104372): the only TypeScript failure is this assertion (Expected: 10000 Received: 300000); the other failures are the dbt-tools E2E ones fixed by #1258.
  • Confirmed src/provider/provider.ts on main sets headerTimeout: freeTierHeaderTimeout() with default 300_000 and no env override in the test.
  • Not run locally against main's tree: this session's worktree is on another branch. The PR's own TypeScript job is the check.

Committed through the GitHub API from a fetched copy of main's test file.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

🤖 Generated with Claude Code

[!NOTE]
Low Risk
Test-only change that documents and asserts existing provider timeout behavior; no runtime impact.

Overview
Fixes a TypeScript CI failure on main by aligning the Altimate Base provider contract test with behavior introduced in #1260. The assertion on base.options.headerTimeout now expects 300_000 ms (5 minutes) instead of the OpenAI loader’s 10 s default.

The updated comment documents why: Altimate Base can wait on queueing, cold start, or reasoning before response headers, so the default matches the SSE chunk watchdog and remains overridable via ALTIMATE_BASE_HEADER_TIMEOUT_MS. No production code changes—only the test expectation and comment.

Reviewed by Cursor Bugbot for commit bf18b8bba279879a37637f07c5fa52cac0416d1c. Bugbot is set up for automated code reviews on this repo. Configure here.

Summary by cubic

Fixes the failing TypeScript job on main by updating the Altimate Base provider contract test to expect the 300-second header timeout introduced in #1260, replacing the previous 10-second default assertion.

Summary by CodeRabbit

  • Bug Fixes
    • Increased the default header timeout for the Altimate Base hosted model contract to five minutes, improving support for slower server-sent event responses.
    • Added support for overriding this timeout with the ALTIMATE_BASE_HEADER_TIMEOUT_MS environment variable.