Fix doctor repair for disabled Codex runtime plugin#82502
Merged
Conversation
openclaw-barnacle
Bot
added
commands
joshavant
force-pushed
the
fix/doctor-codex-plugin-runtime-82368
branch
from
May 16, 2026 07:58
d268ed3 to
03c5f07
Compare
Contributor
|
ClawSweeper status: review started. I am starting a fresh review of this pull request: Fix doctor repair for disabled Codex runtime plugin This is item 1/1 in the current shard. Shard 0/1. This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking. Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted. |
galiniliev
pushed a commit
to galiniliev/openclaw
that referenced
this pull request
May 20, 2026
* fix doctor codex plugin runtime repair * add changelog for codex doctor repair * avoid implicit codex repair without agent routes * expect codex repair in doctor config flow
github-actions Bot
pushed a commit
to Desicool/openclaw
that referenced
this pull request
May 24, 2026
* fix doctor codex plugin runtime repair * add changelog for codex doctor repair * avoid implicit codex repair without agent routes * expect codex repair in doctor config flow
galiniliev
pushed a commit
to galiniliev/openclaw
that referenced
this pull request
May 25, 2026
* fix doctor codex plugin runtime repair * add changelog for codex doctor repair * avoid implicit codex repair without agent routes * expect codex repair in doctor config flow
qiaokuan1992
pushed a commit
to qiaokuan1992/openclaw
that referenced
this pull request
Jun 2, 2026
* fix doctor codex plugin runtime repair * add changelog for codex doctor repair * avoid implicit codex repair without agent routes * expect codex repair in doctor config flow
jameslcowan
pushed a commit
to jameslcowan/openclaw
that referenced
this pull request
Jun 2, 2026
* fix doctor codex plugin runtime repair * add changelog for codex doctor repair * avoid implicit codex repair without agent routes * expect codex repair in doctor config flow
SYU8384
pushed a commit
to SYU8384/openclaw
that referenced
this pull request
Jun 3, 2026
* fix doctor codex plugin runtime repair * add changelog for codex doctor repair * avoid implicit codex repair without agent routes * expect codex repair in doctor config flow
sablehead
pushed a commit
to sablehead/openclaw
that referenced
this pull request
Jun 10, 2026
* fix doctor codex plugin runtime repair * add changelog for codex doctor repair * avoid implicit codex repair without agent routes * expect codex repair in doctor config flow
2 tasks
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
1 participant
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
Fixes #82368 by teaching
openclaw doctor --fixto repair configs where OpenAI/Codex-runtime agent routes require the Codex plugin butplugins.entries.codexorplugins.allowmakes that harness unavailable.The repair enables
plugins.entries.codexand addscodexto an existing non-empty allowlist only when configured routes actually resolve to Codex runtime. It preserves PI-routed OpenAI configs and non-Codex alias/provider cases.Verification
git diff --check./node_modules/.bin/tsgo -p tsconfig.core.json --pretty false --noEmitnode scripts/run-vitest.mjs src/commands/doctor/shared/codex-route-warnings.test.ts(90 tests)codex review --uncommittedrun_36941cbccbb1Real Behavior Proof
Behavior addressed: upgraded configs with
plugins.entries.codex.enabled: falseand default OpenAI/Codex-runtime models failed at runtime withRequested agent harness "codex" is not registered.Real environment tested: Crabbox AWS Linux, lease
cbx_554d34f93f0f, runrun_36941cbccbb1.Exact steps or command run after this patch: created a temp config with local gateway auth,
plugins.entries.codex.enabled: false, andagents.defaults.model.primary: gpt-5.5; rannode scripts/run-node.mjs doctor --fix --non-interactive; startednode scripts/run-node.mjs gateway; pipedhellointonode scripts/run-node.mjs agent --gateway-url http://127.0.0.1:19868 --gateway-token issue-82368-token.Evidence after fix:
doctor --fixwroteplugins.entries.codex.enabled: true; the gateway/agent logs did not containRequested agent harness "codex" is not registered.Observed result after fix: agent command reached the expected no-provider-credentials/runtime boundary with
missingHarness: falseinstead of the reported missing-harness failure.What was not tested: a successful live OpenAI model completion; no OpenAI API key was needed to prove this missing-harness failure was fixed.