Skip to content

Pin relay init to core 15.0.9, and find Cursor's relay skill - #2063

Merged
kjgbot merged 2 commits into
mainfrom
chore/probe-pin-15.0.9
Nov 3, 2026
Merged

kjgbot merged 2 commits into
mainfrom
chore/probe-pin-15.0.9

Conversation

@kjgbot

@kjgbot kjgbot commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor
  • relay init now installs core 15.0.9 (probe-v15.0.9, released from the Bring the core up to the desktop's: port relay-desktop's probe/ changes since the split #2060 merge 05a130289), so CLI 15.0.3 ships the desktop parity changes from Bring the core up to the desktop's: port relay-desktop's probe/ changes since the split #2060.
  • Cursor's relay skill is looked for in ~/.agents/skills. The pinned skills@1.7.4 treats Cursor as a universal agent (skillsDir: ".agents/skills"), so its --global install goes there and never to ~/.cursor/skills. A live relay init --tools cursor against core 15.0.9 printed "Agent skill could not install … for Cursor (it did not land)" and recorded no copy, so --uninstall said it removed the skill but left ~/.agents/skills/messaging-agents-on-the-relay behind. The fake skills CLI in the tests had no Cursor entry, which is why the tests missed it. The new test fails without the fix, printing the same warning.

The live test of relay init against the real 15.0.9 core, in an isolated HOME, passed in both cases:

  • Cursor never opened: init exits 0, says cursor_direct_delivery isn't offered yet, and falls back to the Relay MCP server.
  • Cursor has run: init writes the hooks to ~/.cursor/hooks.json and turns on cursor_direct_delivery and auto_activate. --uninstall turns both off.

The core reported 15.0.9 at 05a130289f24, and the downloaded binary's sha256 matches the release.

Known follow-up, not in this PR: --uninstall leaves an emptied ~/.cursor/hooks.json ({"version":1}) and hooks.json.relay-desktop-backup, even when init created the file.

No changelog entry: Cursor setup is still unreleased, and the core changes are already described.

🤖 Generated with Claude Code


Agent Relay sessions

  • claude session a994d8be · opened via gh pr create · last active 2026-11-03

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review.

Walkthrough

The CLI probe version pin changes from 15.0.8 to 15.0.9. Cursor skill installation now uses the shared ~/.agents/skills directory, and the init test covers installation and uninstall behavior.

Changes: Probe version pin

Layer / File(s) Summary
Update probe version pin
packages/cli/src/cli/lib/probe-core.ts
PROBE_VERSION changes from 15.0.8 to 15.0.9.

Cursor skill directory

Layer / File(s) Summary
Route Cursor skills to shared directory
packages/cli/src/cli/lib/relay-skill.ts, packages/cli/src/cli/commands/init.test.ts
Cursor now resolves to ~/.agents/skills instead of ~/.cursor/skills. The init test checks the recorded skill path and pinned reference, then verifies that --uninstall removes the copy.

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Possibly related PRs

Merge Risk: Minimal · up to 8b803

The update changes the core version pin and aligns Cursor skill lookup with the shared installation directory. No actionable merge-blocking risk is established by the supplied evidence.

Pre-merge checks: Passed 4 · Failed 1

Failed Failed checks: 1 warning

Check name Status Explanation Resolution
Description check Warning The description provides detailed change context and test results, but it does not follow the required template. It omits the Summary, Test Plan, RelayFlow Proof, and Screenshots headings, and it does… Restructure the description using the repository template. Add the required headings, include the test plan checkboxes, and provide valid RelayFlow Proof values for Change type and RelayFlow case. Use non-functional and n/a only if the chan…
Passed Passed checks: 4 passed
Check name Status Explanation
Title check Passed The title clearly identifies both primary changes: pinning relay init to core 15.0.9 and updating Cursor relay skill discovery.
Docstring Coverage Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 2 functions across 3 files.

Full details: Description check

Explanation

The description provides detailed change context and test results, but it does not follow the required template. It omits the Summary, Test Plan, RelayFlow Proof, and Screenshots headings, and it does not provide the required Change type or RelayFlow case values.

Resolution

Restructure the description using the repository template. Add the required headings, include the test plan checkboxes, and provide valid RelayFlow Proof values for Change type and RelayFlow case. Use non-functional and n/a only if the change is considered to have no runtime behavior impact; otherwise add the required RelayFlow case.

Finishing Touches
Generate docstrings
  • Commit to this branch
  • Create a new PR
Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Warning

Some tools did not complete. Review the errors below.

ast-grep (0.51.1)

ast-grep could not parse rule config: /ast-grep-rules/ast-grep-essentials/rules/python/security/avoid_app_run_with_bad_host-python.yml

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.

Share

A rabbit checks the skill path with care,
To the shared skills folder it hops through the air.
The install is recorded, the pin held tight,
Then uninstall clears the copy from sight.
A newer Probe number joins the trail,
And the rabbit bounds on, ears set to sail.

CodeRabbit Verify

  • Run CodeRabbit Verify to test the changes in this pull request.
Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 573029f0-be6e-44d4-8e30-b02ea21cb989
Commits

Reviewing files that changed from the base of the PR and between 1eea49c and 8b803bf.

Files selected for processing: 2
  • packages/cli/src/cli/commands/init.test.ts
  • packages/cli/src/cli/lib/relay-skill.ts

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 1 remain after this review.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

View guided diff | Re-trigger cubic

@greptile-apps

greptile-apps Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The PR appears safe to merge; no new actionable issues were found.

Summary

relay init now pins core 15.0.9 and finds Cursor’s messaging skill in ~/.agents/skills.

  • relay init now installs core version 15.0.9.
  • Cursor’s Relay skill lands where init can track and remove it.

Reviews (2) · Last reviewed commit: "Look for Cursor's relay skill in ~/.agen..." · Reviewed by Greptile

…LI puts it

skills@1.7.4 counts Cursor as a universal agent, so a global install lands
in ~/.agents/skills, not ~/.cursor/skills. relay init looked in the latter,
warned that the skill did not land, kept no record of the copy, and so
--uninstall left it behind.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@kjgbot kjgbot changed the title Pin relay init to core 15.0.9 Pin relay init to core 15.0.9, and find Cursor's relay skill Nov 3, 2026

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 2 files (changes from recent commits).

Reply with feedback, questions, or to request a fix.

View guided diff | Re-trigger cubic

Comment thread packages/cli/src/cli/lib/relay-skill.ts Resolved
@kjgbot
kjgbot merged commit c6b348a into main Nov 3, 2026
45 checks passed
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