test(node-adapters): run fetch(nodeHandler) suite on Deno and Bun in CI

[pi0x]

closes #132

What

Runs test/node-adapters.test.ts — the fetchNodeHandler / node-handler suite (fetch(nodeHandler)) — against Bun and Deno in CI, alongside the existing Node coverage.

Results by runtime

Suite	Node	Bun	Deno
node-adapters.test.ts	✅ 74 pass	✅ 74 pass	✅ 66 pass / 8 skipped

Changes

• Bun: added bun run test:node-adapters:bun to the tests_bun job — full pass, no skips.

• Deno: skip 7 known failures on Deno (guarded by an isDeno const, each with a comment):

  • the 3 fetchNodeHandler > {node,express,fastify} > through srvx/node variants (over a real socket),
  • streaming the body directly… and cancelling req.body… (body-crash regressions),
  • send error is logged… / send error is not logged… (fetch-spec regressions).

  These trip Deno's node-compat HTTP layer (keep-alive body-read poisoning + a couple of streaming/error paths); they pass on Node and Bun. The in-process direct fetch variants still run on Deno.

• Added test:node-adapters:{deno,bun} scripts mirroring test:node-compat:*.

Why a separate Deno job

The two Deno suites need opposite versions:

	Deno 2.7.12 (pinned)	Deno latest v2
node.test.ts (node-compat)	✅ 61 pass	❌ 27 fail
node-adapters.test.ts	❌ 29 fail	✅ (7 skipped)

Bumping the shared pin to latest v2 would regress the currently-green node.test.ts job (exactly what the pin comment protects against). So node-adapters runs in its own tests_deno_node_adapters job on unpinned latest Deno v2, leaving tests_deno untouched. Wired into both publish needs arrays.

🤖 Generated with Claude Code

Summary by CodeRabbit

• Tests
  • Added automated Node-adapter test coverage for Deno and Bun environments.
  • Improved cross-runtime test handling by excluding scenarios with known Deno-specific behavior.
• CI
  • Deno and Bun adapter tests now run as part of continuous integration.
  • Publishing workflows wait for the new Deno checks to complete successfully.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 5309342a-0b7b-432f-9b10-f722215aa291

📥 Commits

Reviewing files that changed from the base of the PR and between 99ca81a and 7ebd063.

📒 Files selected for processing (3)

• .github/workflows/ci.yml
• package.json
• test/node-adapters.test.ts

---

📝 Walkthrough

Walkthrough

Changes

Node adapter runtime testing

Layer / File(s)	Summary
Runtime-specific adapter test compatibility test/node-adapters.test.ts	The suite detects Deno and skips selected socket, body-state, and send-error tests there.
Deno and Bun CI execution package.json, .github/workflows/ci.yml	Adds Deno and Bun test scripts, runs them in CI, and makes publishing depend on the Deno job.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

• h3js/srvx#243: Related Node adapter regression and Deno-specific test changes.
• h3js/srvx#253: Related updates to Node adapter bridging and runtime-specific assertions.

Suggested reviewers: pi0

Poem

A rabbit hops where adapters run,
Through Deno clouds and Bun-lit sun.
Tests skip paths that cannot stay,
While CI guards the publish way.
“Hop, hop!” says Bunny, “all checks are done!”

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Linked Issues check	⚠️ Warning	[132] The PR improves CI coverage but does not implement the Deno/Bun runtime fixes requested for fetchNodeHandler/toFetchHandler.	Implement the runtime compatibility changes for fetchNodeHandler/toFetchHandler in Deno and Bun, and remove the workaround skips instead of only adjusting CI.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title matches the main change: adding Deno and Bun coverage for the node-adapters suite in CI.
Out of Scope Changes check	✅ Passed	The changes stay within test/CI setup and Deno-specific gating needed to support the node-adapters compatibility work.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch ci/node-adapters-deno-bun

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/srvx@272

commit: 7ebd063