Skip to content

test: harden fetch fixture synchronization#76

Merged
BartWaardenburg merged 3 commits into
mainfrom
codex/harden-fetch-fixture-test
Jul 14, 2026
Merged

test: harden fetch fixture synchronization#76
BartWaardenburg merged 3 commits into
mainfrom
codex/harden-fetch-fixture-test

Conversation

@BartWaardenburg

Copy link
Copy Markdown
Collaborator

Summary

  • replace the scheduler-sensitive negative timeout with a staged reader and explicit channel synchronization
  • keep the real TCP fixture timeout and complete-header behavior unchanged
  • correct the previous implementation plan's expected file scope

Verification

  • deterministic regression test failed against request-line-only behavior, then passed with complete-header reading restored
  • focused CLI fetch integration tests
  • pnpm check
  • pnpm test

@codspeed-hq

codspeed-hq Bot commented Jul 14, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 188 untouched benchmarks
⏩ 76 skipped benchmarks1


Comparing codex/harden-fetch-fixture-test (eed42f8) with main (9967c5c)

Open in CodSpeed

Footnotes

  1. 76 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@BartWaardenburg
BartWaardenburg merged commit a307917 into main Jul 14, 2026
26 checks passed
@BartWaardenburg
BartWaardenburg deleted the codex/harden-fetch-fixture-test branch July 14, 2026 10:52
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