[WRONG BRANCH] fix(cli): expose Cursor native integration status - #425
[WRONG BRANCH] fix(cli): expose Cursor native integration status#425luvs01 wants to merge 1 commit into
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. 🗂️ Base branches to auto review (2)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Team Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
⏳ DRAFT
What to do
Its title has been prefixed with |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3c1a315c75
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| ocx inspect star [--json] | ||
| ocx inspect windows-tray [--json] | ||
| ocx integration native [list] [--json] | ||
| ocx integration native cursor [--json] |
There was a problem hiding this comment.
Document Cursor in the actual integration help
When users run ocx integration native --help or ocx help integration, parseCliHead intercepts the request before this handler and renders the entry from src/cli/registry.ts; that entry still says native only shows or flips the Claude/Claude Desktop/Codex/Grok toggles and provides no cursor syntax. The docs site likewise contains no CLI invocation, so the newly exposed status action remains undiscoverable through normal help. Update the registry metadata and integration documentation alongside this private usage string.
AGENTS.md reference: AGENTS.md:L343-L344
Useful? React with 👍 / 👎.
Motivation
ocx integration nativewould read/api/native-integrations/cursor, but the CLI did not actually request that route, leaving the documented Cursor status unreachable.Description
cursoras a read-only action to theintegration nativeCLI flow by updatingsrc/cli/inspect.tsto includeocx integration native cursor [--json]inUSAGEand to handleaction === "cursor"by callingread("/api/native-integrations/cursor", ...).tests/cli-storage-inspect.test.tsthat asserts the CLI issues a singleGET /api/native-integrations/cursorrequest and prints the returned JSON (checks for"privateInference")./api/native-integrations.Testing
bun test tests/cli-storage-inspect.test.tswith the repository Bun (v1.4.0) and all tests in that file passed (22 tests, 0 failures).bun run typecheckandbun run skill:surface:check, both of which succeeded.bun run test:changedcould not run in this checkout because the comparisondevref was not resolvable; the change is covered by the focused test added above.bun test(v1.2.14) fails in this repo environment due to a runtime mismatch (node:zlibexport), but the repository-pinned Bun v1.4.0 was used for the successful test run.Codex Task