[codex] Explore Codex CLI app-server feasibility#33
Merged
Conversation
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR adds a bounded Codex CLI/app-server feasibility spike alongside the existing Claude-focused Switchboard workflow, documenting what is usable today versus experimental.
Changes:
- Adds Codex CLI and app-server probe scripts for command surface, protocol compatibility, preflight/auth, lifecycle, and in-session model override evidence.
- Adds automated tests with fake Codex binaries/app-server processes for the new probes.
- Updates README, product scope, PRD index, decision log, package scripts, and agent onboarding docs to publish the spike framing.
Reviewed changes
Copilot reviewed 16 out of 16 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
test/codex-cli-feasibility-probe.test.js |
Tests CLI command-surface and live resume probe behavior. |
test/codex-app-server-switch-probe.test.js |
Tests app-server one-thread model override probe. |
test/codex-app-server-protocol-check.test.js |
Tests protocol shape validation and failure reporting. |
test/codex-app-server-preflight.test.js |
Tests Codex version, app-server availability, and auth preflight paths. |
test/codex-app-server-lifecycle-probe.test.js |
Tests lifecycle, crash, malformed output, and spawn-failure handling. |
scripts/codex-cli-feasibility-probe.js |
Adds Codex CLI route/resume feasibility probe. |
scripts/codex-app-server-switch-probe.js |
Adds app-server model override feasibility probe. |
scripts/codex-app-server-protocol-check.js |
Adds generated-binding protocol compatibility check. |
scripts/codex-app-server-preflight.js |
Adds install/auth readiness diagnostics for app-server usage. |
scripts/codex-app-server-lifecycle-probe.js |
Adds process lifecycle safety probe. |
README.md |
Reframes product usage paths and documents spike commands. |
package.json |
Adds npm scripts for the new spike probes. |
docs/product/CODEX-CLI-SPIKE-SCOPE.md |
Adds detailed spike scope, gates, evidence, and decision criteria. |
docs/PRD.md |
Links the Codex spike scope from the PRD index. |
docs/decision-log.md |
Records the Codex feasibility spike decision and evidence. |
AGENTS.md |
Adds the Codex spike scope to recommended reading order. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
Signed-off-by: Hanna Rosengren <4538260+hannasoderstromdev@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR publishes the Codex CLI/app-server feasibility spike as an experimental product direction, not a polished replacement workflow.
It adds repeatable probes and documentation for three usage paths:
What changed
Gate status
Validation
npm run lintnpm testnpm run switchboard:spike:codex-app-server:preflightnpm run switchboard:spike:codex-app-server:lifecyclenpm run switchboard:spike:codex-app-serverLive Codex app-server commands require normal local Codex auth/state and were run outside the sandbox where needed.
Remaining caveats