Skip to content

Document Cursor exec policy and catalog troubleshooting#303

Closed
diegocantarero wants to merge 1 commit into
lidge-jun:devfrom
diegocantarero:agent/document-cursor-codex-routing
Closed

Document Cursor exec policy and catalog troubleshooting#303
diegocantarero wants to merge 1 commit into
lidge-jun:devfrom
diegocantarero:agent/document-cursor-codex-routing

Conversation

@diegocantarero

Copy link
Copy Markdown
Contributor

What changed

  • document the current nativeLocalExec modes for the experimental Cursor adapter
  • mark unsafeAllowNativeLocalExec as the legacy compatibility option
  • explain the fail-closed behavior of codex-sandbox and the loopback trust boundary
  • add catalog troubleshooting for discovery filters, Cursor GetUsableModels, cache refresh, and ocx sync
  • clarify that competing local writers and Codex model-cache refreshes can make model visibility appear unstable without implying an OpenCodex defect

Why

The runtime already implements nativeLocalExec: "off" | "codex-sandbox" | "on", but the public configuration reference documented only the legacy boolean. That made the safest current configuration and precedence rules difficult to discover.

Catalog visibility can also look intermittent when filters, cached discovery, or another local writer changes the managed files. The new troubleshooting section keeps those causes separate and explicitly identifies external writers as an operator-side hazard rather than a confirmed OpenCodex bug.

Safety

  • off remains the documented safest default
  • on is described as a trusted-local opt-in that bypasses Codex approval and sandbox semantics
  • codex-sandbox is documented as currently fail-closed because caller-controlled request text cannot attest a real sandbox
  • no runtime, authentication, credential, or provider behavior changes

Validation

  • bun run prepush — passed
  • 3513 tests passed, 0 failed
  • typecheck passed
  • GUI lint passed
  • privacy scan passed
  • docs-site Astro/Starlight build passed (121 pages)
  • git diff --check passed

Localization

This PR updates the English source documentation. Existing localized pages still describe the functional legacy boolean and can be synchronized separately by fluent contributors.

@coderabbitai

coderabbitai Bot commented Jul 22, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 60ca33d8-ab20-418d-8d1b-2597e2ff806a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@Ingwannu

Copy link
Copy Markdown
Owner

This documentation is accurate and worth keeping, but #307 made the stale-base draft conflict in the same guide. I preserved your original commit authorship and moved it onto current dev as #321, resolving the conflict by keeping both the custom-display-name and troubleshooting sections. The replacement builds all 121 docs pages and was included in the combined 3,667-test integration pass. Closing this draft in favor of #321.

@Ingwannu Ingwannu closed this Jul 23, 2026
eachann1024 pushed a commit to eachann1024/opencodex that referenced this pull request Jul 23, 2026
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.

2 participants