Skip to content

feat(middleware): implement LIVE=1 smoke test for Codex CLI runtime (#37)#40

Merged
alexey-pelykh merged 1 commit intomainfrom
feat/codex-live-smoke-test
Feb 26, 2026
Merged

feat(middleware): implement LIVE=1 smoke test for Codex CLI runtime (#37)#40
alexey-pelykh merged 1 commit intomainfrom
feat/codex-live-smoke-test

Conversation

@alexey-pelykh
Copy link
Copy Markdown

@alexey-pelykh alexey-pelykh commented Feb 26, 2026

Summary

  • Add src/middleware/__smoke__/codex.live.test.ts — end-to-end smoke test for the Codex CLI runtime
  • Mirrors the Gemini smoke test pattern: ChannelBridgeCodexCliRuntimecodex CLI → NDJSON streaming → assertion
  • Gated behind LIVE=1 environment variable (skipped in CI)
  • Tests single-turn response (payloads, text, sessionId, aborted=false) and session resumption
  • Adds git init in temp dir setup (codex requires a trusted git directory)
  • Fixes two pre-existing CodexCliRuntime bugs discovered during live testing:
    • serializeMcpServersToToml: emit command as TOML string with separate args array (codex expects string, not array)
    • buildArgs for resume: include prompt and drop unsupported --color flag

Test plan

  • Test skips correctly without LIVE=1 (verified: 2 tests skipped)
  • LIVE=1 smoke test passes (single-turn: 11s, resume: 1.8s)
  • Codex unit tests pass (48/48)
  • Full test suite passes (861/861)
  • Formatting clean (pnpm format)
  • Linting clean (pnpm lint)
  • Type-check clean (pnpm tsgo)

Closes #37

🤖 Generated with Claude Code

@alexey-pelykh alexey-pelykh force-pushed the feat/codex-live-smoke-test branch from c9447e4 to a3312de Compare February 26, 2026 11:36
)

Add codex.live.test.ts with single-turn and session resumption tests,
gated behind LIVE=1. Fix two pre-existing CodexCliRuntime bugs discovered
during live testing:

- serializeMcpServersToToml: emit command as TOML string with separate
  args array (codex expects string, not array)
- buildArgs for resume: include prompt and drop unsupported --color flag

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@alexey-pelykh alexey-pelykh force-pushed the feat/codex-live-smoke-test branch from a3312de to 7b1432e Compare February 26, 2026 11:42
@alexey-pelykh alexey-pelykh enabled auto-merge (squash) February 26, 2026 11:42
@alexey-pelykh alexey-pelykh merged commit f19d4e5 into main Feb 26, 2026
2 checks passed
@alexey-pelykh alexey-pelykh deleted the feat/codex-live-smoke-test branch February 26, 2026 11:53
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.

Implement LIVE=1 smoke test for Codex CLI runtime

1 participant