test: repair main verification fixtures - #349
Conversation
Hold the test host listener open so ephemeral proxy allocation cannot reuse its origin and trip the production isolation guard.
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
commit: |
Keep coverage on documented env substitutions without using the unsupported MCP cwd field tightened by the Claude lifecycle contract.
Summary
cwdto supportedenvRoot cause and provenance
Node 22.19 sandbox origin
port: 0listener the test's hard-coded43123, correctly triggering the production different-origin guard/tmp/ab-rstest-*worker roots are unrelated099744222from feat: add agent-bundle compiler and developer workbench #2;3a1c235c6later reserved the host origin for the first test but left this second occurrence latentNode 24 path token
cwd, butpath-token-resolver.test.tsstill built a fixture withcwd: pluginDataenvsupportTest plan
MCP App sandbox proxy must use a different origin from its hostmcp-app-sandbox.test.ts— 11/11rstest-worker-isolation.test.ts— 2/2 (local-ci: current-node doctor sockets exceed AF_UNIX path cap #335 regression coverage)claude.substitution.token.unsupportedpath-token-resolver.test.ts— 9/9pnpm buildpnpm typecheckpnpm lint— 0 errors, 0 warnings