Skip to content

[codex] Guard pre-ready MCP notifications#95

Merged
Atomics-hub merged 1 commit into
masterfrom
codex/pre-ready-notification-guard
May 27, 2026
Merged

[codex] Guard pre-ready MCP notifications#95
Atomics-hub merged 1 commit into
masterfrom
codex/pre-ready-notification-guard

Conversation

@Atomics-hub
Copy link
Copy Markdown
Owner

Summary

  • add release-audit smoke coverage proving pre-ready subprocess MCP notifications are dropped before downstream forwarding
  • verify lifecycle still completes after the dropped notification
  • assert raw pre-ready notification payloads are not returned or logged
  • document the pre-ready notification no-passthrough guard in README, architecture, and roadmap

Why

The subprocess MCP proxy already gates request methods before lifecycle readiness. This adds explicit proof that client notifications cannot bypass that lifecycle boundary into the downstream MCP server before initialize and notifications/initialized complete.

Verification

Locally verified during the GitHub Actions billing outage:

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

GitHub Actions is expected to fail before code checkout while the account billing suspension is active, so this PR is intentionally draft/local-verified until Actions recovers.

@Atomics-hub Atomics-hub marked this pull request as ready for review May 27, 2026 01:16
@Atomics-hub Atomics-hub force-pushed the codex/pre-ready-notification-guard branch from 34d0a5f to 5073fb0 Compare May 27, 2026 01:18
@Atomics-hub Atomics-hub merged commit 30af054 into master May 27, 2026
1 check passed
@Atomics-hub Atomics-hub deleted the codex/pre-ready-notification-guard branch May 27, 2026 01:19
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