Skip to content

[codex] Prebuild HTTP no-auto parity archives#4373

Merged
proggeramlug merged 1 commit into
PerryTS:mainfrom
andrewtdiz:codex/node-compat-http-noauto-link
Jun 4, 2026
Merged

[codex] Prebuild HTTP no-auto parity archives#4373
proggeramlug merged 1 commit into
PerryTS:mainfrom
andrewtdiz:codex/node-compat-http-noauto-link

Conversation

@andrewtdiz
Copy link
Copy Markdown
Contributor

@andrewtdiz andrewtdiz commented Jun 4, 2026

Fixes #3954

Summary

  • prebuild perry-ext-http, perry-ext-net, and perry-ext-ws in run_parity_tests.sh when PERRY_NO_AUTO_OPTIMIZE is set for node-suite --module http
  • leave the compiler resolver and HTTP runtime unchanged; this only makes the prebuilt archives available for the existing no-auto link path
  • convert the current no-auto HTTP module sweep from link-time compile failures into ordinary runtime parity results

Why This Cut

The current #3954 failure mode is coherent at the harness/link boundary: HTTP fixtures can emit HTTP, net, and WS well-known FFI owners, but the no-auto parity harness only prebuilt the core perry, perry-runtime, and perry-stdlib archives. The compiler already records the needed owners; the missing piece was archive availability before the per-test compile loop.

Tests

  • bash -n run_parity_tests.sh
  • cargo fmt --all -- --check
  • git diff --check
  • ./scripts/check_file_size.sh
  • Before fix: npm exec --yes --package=node@26 -- bash -lc 'node --version; PERRY_NO_AUTO_OPTIMIZE=1 ./run_parity_tests.sh --suite node-suite --module http' reported 3 pass / 14 compile fail / 0 skipped
  • After fix: same command reported 5 pass / 12 parity fail / 0 compile fail / 0 skipped

Known Limitations / Non-goals

  • This does not change HTTP runtime behavior. The remaining 12 HTTP failures are output mismatches and crashes in the existing HTTP behavior fixtures, not link failures.
  • This keeps the trigger narrow to the no-auto node-suite --module http harness path.

@proggeramlug proggeramlug force-pushed the codex/node-compat-http-noauto-link branch from 5eb35e3 to 9ce14ba Compare June 4, 2026 11:59
@proggeramlug proggeramlug marked this pull request as ready for review June 4, 2026 11:59
@proggeramlug proggeramlug merged commit b33cd77 into PerryTS:main Jun 4, 2026
12 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.

node:http: split #2013 argument-validation parity tail

2 participants