Improve Codex/Claude setup flow and installer reporting#82
Merged
Conversation
KjellKod
temporarily deployed
to
codex-ci-review
GitHub Actions
Inactive
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
KjellKod
had a problem deploying
to
codex-ci-review
GitHub Actions
Error
KjellKod
temporarily deployed
to
codex-ci-review
GitHub Actions
Inactive
KjellKod
temporarily deployed
to
codex-ci-review
GitHub Actions
Inactive
KjellKod
changed the title
Extend dual-model planning with explicit fail-closed requirement when operator policy mandates multi-model execution. Add runtime selection truth gate to canonicalization roadmap (item 4.5). Quest/Co-Authored by Claude Opus 4.6, Codex in Collaboration with KjellKod
…list The MCP server self-identifies as codex-cli, so tool names are mcp__codex-cli__codex, not mcp__codex__codex. Without the correct permission entry, Claude Code prompts for approval on every call. Installer now auto-adds the permission after MCP registration. Setup docs updated with manual instructions and explanation. Quest/Co-Authored by Claude Opus 4.6, Codex in Collaboration with KjellKod
Permission prefixes like ["gh","api"] only match direct invocations. Wrapping gh commands in bash -lc defeats prefix matching and causes repeated permission prompts during quest orchestration. Quest/Co-Authored by Claude Opus 4.6, Codex in Collaboration with KjellKod
This reverts commit 6847346.
Permission prefixes like ["gh","pr"] only match when gh is the top-level command. Wrapping in bash -lc defeats prefix matching and causes repeated permission prompts during orchestration. Quest/Co-Authored by Claude Opus 4.6, Codex in Collaboration with KjellKod
Cache successful host Claude bridge probes so later quest starts in sandboxed Codex sessions can reuse them without repeating browser-login auth. Cache fallback is guarded: only auth-related failures (logged out, "Not logged in", timeout) trigger cache use — real bridge breakage stays visible. Also refines co-author trailer format to use standard Co-Authored-By lines, one per model that actually participated. Quest/Co-Authored by Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> in collaboration with KjellKod <kjell.hedstrom@gmail.com>
Quest/Co-Authored by Co-Authored-By: Codex <noreply@openai.com> in collaboration with KjellKod <kjell.hedstrom@gmail.com>
KjellKod
force-pushed
the
fix/codex-mcp-permission
branch
from
0dd06f2 to
41f61d1
Compare
KjellKod
temporarily deployed
to
codex-ci-review
GitHub Actions
Inactive
QUEST_PREFLIGHT_CACHE_TTL_SECONDS is a user override. If set to a non-numeric value (e.g. CI/env typo), the numeric comparison in write_success_cache would error under set -e, causing a hard preflight failure. Fall back to the default 43200 on invalid input. Quest/Co-Authored by Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> in collaboration with KjellKod <kjell.hedstrom@gmail.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
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
.quest_updatedfiles requiring manual mergeChanges
mcp__codex-cli__*to~/.claude/settings.jsonafter Codex MCP registration and on reruns where MCP is already configuredcodex-cliand add Claude bridge guidance forclaude auth login,claude auth status, and rerunning preflight outside a restricted sandbox when auth visibility looks stalescripts/quest_preflight.shrecords visible Claude auth state, exposes probe diagnostics, and emits better remediation text for Codex-led Claude bridge failures.skills/quest/SKILL.mdpauses on second-model preflight failure and gives the user explicit choices: fix it now, continue single-model, or cancelscripts/quest_installer.shnow prints the exact.quest_updatedfiles created during an update instead of a generic reminderTest plan
./scripts/quest_preflight.sh --orchestrator codexafter Claude browser login and verifyavailable: trueclaude auth login,claude auth status, and restricted-sandbox troubleshooting when Claude bridge setup still looks unavailable.quest_updatedfiles🤖 Generated with Claude Code