Skip to content

[codex] Relax startup catch-up waits for Windows CI#123

Merged
ScriptedAlchemy merged 1 commit into
masterfrom
codex/release-ci-windows-catchup
Jun 24, 2026
Merged

[codex] Relax startup catch-up waits for Windows CI#123
ScriptedAlchemy merged 1 commit into
masterfrom
codex/release-ci-windows-catchup

Conversation

@ScriptedAlchemy

Copy link
Copy Markdown
Owner

Summary

  • Increase MCP startup catch-up test waits from 2s/5s/10s to 30s.
  • Keep the same assertions; this only aligns test budgets with the server's 20s transcript-ingest timeout guard.

Why

Release PR #122 is blocked by Windows CI shard 6/8. The failing test waits for wait_for_startup_catch_up, which includes the detached transcript ingest task. That task can legally take up to 20s before its timeout guard completes, so shorter waits are flaky on loaded Windows runners.

Verification

  • cargo fmt --check
  • cargo test mcp_server_owns_watcher_and_refreshes_token_map_on_change --test mcp_handler_test
  • cargo test wait_for_startup_catch_up_waits_for_transcript_ingest_flag --test mcp_handler_test
  • cargo test two_mcps_on_same_project_coordinate_via_sync_lock --test multi_mcp_coordination_test

@changeset-bot

changeset-bot Bot commented Jun 24, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: d9e234b

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@ScriptedAlchemy ScriptedAlchemy merged commit ca8ed54 into master Jun 24, 2026
13 of 15 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