[codex] add plan-only model routing isolation#1
Draft
luobosibing2 wants to merge 4420 commits into
Draft
Conversation
Resolve LazyCodex/Codex integration conflicts and keep Codex install compatibility checks building bundled MCP runtimes before installer tests.
Introduce LazyCodex
…e-opencode-responses fix: dedupe OpenCode idle responses
Plan: plans/codex-windows-git-bash-profile.md
Plan: plans/codex-windows-git-bash-profile.md
…size The Final Quality Gate forced a $code-review reviewer spawn on every final goal. For small, local, low-risk changes that is wasteful. Make step 4 size-aware: spawn $code-review only for large or risky work (multi-file, cross-cutting, new architecture, security/data surfaces, or low confidence), otherwise self-review and record codeReview via the UNCONDITIONAL APPROVAL evidence path with a one-line justification. Keeps the quality-gate-json contract intact (codeReview stays required).
$code-review was not a defined command or tool, just an informal alias. The Codex Tool Mapping already names the real reviewer as the codex-ultrawork-reviewer agent (worker fallback when unavailable). Point the final quality gate (full-workflow.md step 4) and the goal instruction text at that actual spawn_agent call instead of the non-existent $code-review.
Replace the bogus $code-review token in the final review step with the same size-judged codex-ultrawork-reviewer spawn guidance used elsewhere, matching the reviewer-call fix already on dev.
|
Thank you for your contribution! Before we can merge this PR, we need you to sign our Contributor License Agreement (CLA). To sign the CLA, please comment on this PR with: This is a one-time requirement. Once signed, all your future contributions will be automatically accepted. I have read the CLA Document and I hereby sign the CLA 1 out of 2 committers have signed the CLA. |
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
10 participants
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
plan_only_model_routingconfig and inject a fake-keyplan-only/glm-5.1provider through the plugin config hook.Prometheus - Plan Buildermessages to the Plan-only model, restore normal models for non-Plan misuse, and keep Plan-only out of session model state.Test Plan
npx --yes bun@latest run build:schemanpx --yes bun@latest test src/config/schema/oh-my-opencode-config.test.ts src/plugin-handlers/config-handler.test.ts src/plugin/chat-message.test.ts src/plugin/chat-headers.test.ts src/plugin/event.test.ts src/plugin/event-compaction-agent.test.ts --bailnpx --yes bun@latest run typechecknpx --yes bun@latest run build