Skip to content

test: tidy the proxy conformance lane - #467

Open
jancurn wants to merge 1 commit into
mainfrom
claude/friendly-babbage-t3a0py
Open

jancurn wants to merge 1 commit into
mainfrom
claude/friendly-babbage-t3a0py

Conversation

@jancurn

@jancurn jancurn commented Oct 21, 2026

Copy link
Copy Markdown
Member

Small follow-up to #464 covering the nits from its review. Test-only, no runtime change.

  • List dns-rebinding-protection in the conformance coverage table in test/conformance/README.md
  • Make the proxy lane honour the workflow's verbose input, like the client lane
  • Spawn npx through a shell on Windows, where it is a .cmd shim that Node refuses to spawn directly (EINVAL)

Verified locally: the lane still passes 2/2 with --verbose --output-dir passed through, lint, build and unit tests are green.

Refs #334


Generated by Claude Code

Follow-up to #464. List `dns-rebinding-protection` in the conformance
coverage table, honour the workflow's `verbose` input in the proxy lane
like the client lane does, and spawn `npx` through a shell on Windows,
where it is a .cmd shim that Node refuses to spawn directly (EINVAL).

Refs #334

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BbA1fNDr3b1HbZc7Q6qFvo

This branch has not been deployed

No deployments
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.

3 participants