Skip to content

added cursor support#240

Merged
3 commits merged intosubsy:mainfrom
allora-network:feature-cursor-support
Feb 3, 2026
Merged

added cursor support#240
3 commits merged intosubsy:mainfrom
allora-network:feature-cursor-support

Conversation

@srt0422
Copy link

@srt0422 srt0422 commented Jan 29, 2026

Summary by CodeRabbit

  • New Features
    • Cursor Agent CLI added as a new agent option with configurable model, execution mode and force support; integrated into the skill installer.
    • Supports streamed agent output and richer output parsing for more reliable real-time feedback.
  • Tests
    • Comprehensive test coverage added for initialization, setup/validation, output parsing and error handling.
  • Chores
    • Package version bumped.

✏️ Tip: You can customize this high-level summary in your review settings.

@vercel
Copy link

vercel bot commented Jan 29, 2026

Someone is attempting to deploy a commit to the plgeek Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 29, 2026

Walkthrough

Adds a new Cursor Agent plugin with CLI integration, JSONL parsing utilities, registration in builtin agents, skill-installer mapping, and comprehensive tests validating setup, argument formation, execution and output parsing.

Changes

Cohort / File(s) Summary
Cursor Agent Plugin Implementation
src/plugins/agents/builtin/cursor.ts
New CursorAgentPlugin class plus helper functions: extractErrorMessage(), parseCursorJsonLine(), parseCursorOutputToEvents(). Implements detection, version probe, CLI arg building, stdin streaming, JSONL buffering/parsing, setup/validation and execution hooks.
Cursor Agent Tests
src/plugins/agents/builtin/cursor.test.ts
New comprehensive test suite covering metadata, initialization variants, setup questions and validation, sandbox requirements, argument construction, protected-method behaviour, error extraction, JSONL line parsing and multi-line aggregation including edge cases.
Builtin Agents Registry
src/plugins/agents/builtin/index.ts
Registers and re-exports the Cursor agent factory and plugin type in the builtin agents registry.
Skill Installer Mapping
src/setup/skill-installer.ts
Adds cursor: 'cursor' to AGENT_ID_MAP to enable adding the Cursor agent via skill installer.
Package Version
package.json
Version bump (0.6.0 → 0.6.1).

Sequence Diagram(s)

sequenceDiagram
    participant User
    participant Agent as CursorAgentPlugin
    participant CLI as Cursor CLI
    participant Parser as JSONL Parser
    participant Events as Event System

    User->>Agent: execute(prompt)
    Agent->>Agent: buildArgs()
    Agent->>CLI: spawn with args + stdin
    CLI-->>Agent: stdout (JSONL lines / partial lines)

    loop per received chunk/line
        Agent->>Parser: parseCursorOutputToEvents(chunk)
        Parser-->>Agent: AgentDisplayEvent[]
        Agent->>Events: onStdout / onJsonlMessage (forward events)
    end

    CLI-->>Agent: stdout end
    Agent->>Agent: flush buffers
    Agent->>Events: onEnd()
    Events-->>User: finalised event stream
Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'added cursor support' accurately summarises the main change—introducing Cursor Agent CLI plugin support across the codebase.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

🧪 Unit Test Generation v2 is now available!

We have significantly improved our unit test generation capabilities.

To enable: Add this to your .coderabbit.yaml configuration:

reviews:
  finishing_touches:
    unit_tests:
      enabled: true

Try it out by using the @coderabbitai generate unit tests command on your code files or under ✨ Finishing Touches on the walkthrough!

Have feedback? Share your thoughts on our Discord thread!


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

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

Scott Terry added 2 commits January 29, 2026 15:46
Remove special handling for nested tool_call object format. The cursor
agent now uses the standard event.name and event.input fields directly.
@vercel
Copy link

vercel bot commented Jan 30, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
ralph-tui Ignored Ignored Preview Jan 30, 2026 0:08am

Request Review

@codecov
Copy link

codecov bot commented Jan 30, 2026

Codecov Report

❌ Patch coverage is 51.42180% with 205 lines in your changes missing coverage. Please review.
✅ Project coverage is 45.55%. Comparing base (b0c08d6) to head (ab45426).
⚠️ Report is 140 commits behind head on main.

Files with missing lines Patch % Lines
src/plugins/agents/builtin/cursor.ts 50.95% 205 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #240      +/-   ##
==========================================
+ Coverage   45.45%   45.55%   +0.10%     
==========================================
  Files          84       85       +1     
  Lines       24396    24818     +422     
==========================================
+ Hits        11088    11305     +217     
- Misses      13308    13513     +205     
Files with missing lines Coverage Δ
src/plugins/agents/builtin/index.ts 92.00% <100.00%> (+1.09%) ⬆️
src/setup/skill-installer.ts 66.48% <100.00%> (+0.18%) ⬆️
src/plugins/agents/builtin/cursor.ts 50.95% <50.95%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@subsy
Copy link
Owner

subsy commented Jan 30, 2026

@srt0422 thanks for this. Can you please:

  • add accompanying website documentation
  • remove the package.json version bump

Thanks!

@subsy subsy closed this pull request by merging all changes into subsy:main in 952550d Feb 3, 2026
@coderabbitai coderabbitai bot mentioned this pull request Feb 18, 2026
13 tasks
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

Comments