Skip to content

[codex] Harden Mirror Codex app preflight#362

Merged
YSCJRH merged 1 commit into
mainfrom
codex/mirror-codex-app-preflight-hardening
May 2, 2026
Merged

[codex] Harden Mirror Codex app preflight#362
YSCJRH merged 1 commit into
mainfrom
codex/mirror-codex-app-preflight-hardening

Conversation

@YSCJRH
Copy link
Copy Markdown
Owner

@YSCJRH YSCJRH commented May 2, 2026

Summary

  • Make the app-server preflight report mcpServerStatus/list timeouts as open MCP status evidence by default instead of failing the entire install/skill preflight.
  • Add strict mode via --require-mcp-status so status enumeration can still be enforced when needed.
  • Include request method names and per-request timeout detail in app-server timeout errors.
  • Make the Mirror Codex MCP server prefer the workspace cwd when it is launched from an installed plugin cache, so canonical demo artifacts resolve from the repo instead of the cache path.
  • Update plugin docs to preserve the evidence boundary: app-server install/skill checks are not interactive UI tool-card evidence.

Validation

  • python -m pytest plugins\mirror-codex\tests -q
  • python plugins\mirror-codex\scripts\smoke_mcp_stdio.py
  • ./make.ps1 plugin-app-preflight
  • python plugins\mirror-codex\scripts\app_protocol_preflight.py --require-mcp-status --mcp-status-timeout 3 fails as expected on mcpServerStatus/list
  • ./make.ps1 plugin-release-check
  • python -m pytest backend\tests\test_api.py -q
  • git diff --check

No frontend files changed, so npm run build --prefix frontend was not run.

@github-actions github-actions Bot added the lane:auto-safe Eligible for autonomous safe-lane execution and merge. label May 2, 2026
@YSCJRH YSCJRH marked this pull request as ready for review May 2, 2026 00:17
@YSCJRH YSCJRH merged commit 7505647 into main May 2, 2026
5 checks passed
@YSCJRH YSCJRH deleted the codex/mirror-codex-app-preflight-hardening branch May 2, 2026 00:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lane:auto-safe Eligible for autonomous safe-lane execution and merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant