Skip to content

[codex] Guard subprocess MCP client initialize protocol#99

Merged
Atomics-hub merged 1 commit into
masterfrom
codex/subprocess-client-init-guard
May 27, 2026
Merged

[codex] Guard subprocess MCP client initialize protocol#99
Atomics-hub merged 1 commit into
masterfrom
codex/subprocess-client-init-guard

Conversation

@Atomics-hub
Copy link
Copy Markdown
Owner

Summary

  • Extend the subprocess MCP initialize release-audit smoke to cover invalid client initialize.protocolVersion.
  • Prove bad client initialize params fail closed before downstream forwarding, keep the session unready, and do not return or log the raw client protocol marker.
  • Keep the existing downstream unsupported-protocol redaction coverage in the same release-audit row.

Why

The local subprocess stdio proxy is the v0.1 operator path. Client-side initialization variants should fail before they cross the boundary, while downstream initialization failures should remain sanitized.

Verification

  • cargo fmt --check
  • cargo test release_audit_subprocess_mcp_proxy_initialize_guard_smoke_rejects_bad_protocols -- --nocapture
  • cargo test
  • cargo clippy --all-targets --all-features -- -D warnings
  • cargo run --locked -- release-audit

@Atomics-hub Atomics-hub merged commit 62f9031 into master May 27, 2026
1 check passed
@Atomics-hub Atomics-hub deleted the codex/subprocess-client-init-guard branch May 27, 2026 07:01
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