Skip to content

feat(nel-assistant): add Cursor IDE support#747

Open
sephmard wants to merge 2 commits intoNVIDIA-NeMo:mainfrom
sephmard:nel-assistant-cursor
Open

feat(nel-assistant): add Cursor IDE support#747
sephmard wants to merge 2 commits intoNVIDIA-NeMo:mainfrom
sephmard:nel-assistant-cursor

Conversation

@sephmard
Copy link
Contributor

@piojanu @gchlebus adding Cursor skill.md support

@sephmard sephmard requested review from a team as code owners February 22, 2026 21:19
@copy-pr-bot
Copy link

copy-pr-bot bot commented Feb 22, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

Copy link
Contributor

Choose a reason for hiding this comment

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

This file is a copy/paste of the .claude/skills/nel-assistant. Can we avoid this? Ideally, we have only one copy of the skill that works across the board.

- `adapter_config.custom_system_prompt` (like `/think`, `/no_think`) and no `adapter_config.params_to_add` (leave `params_to_add` unrelated to reasoning untouched)
- `adapter_config.params_to_add` for payload modifier (like `"chat_template_kwargs": {"enable_thinking": true/false}`) and no `adapter_config.custom_system_prompt` and `adapter_config.use_system_prompt: false` (leave `custom_system_prompt` and `use_system_prompt` unrelated to reasoning untouched).
- reasoning effort/budget (if it's configurable, AskUserQuestion what reasoning effort they want)
- reasoning effort/budget (if it's configurable, ask the user what reasoning effort they want)
Copy link
Contributor

Choose a reason for hiding this comment

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

Is ask the user a special cursor trigger phrase? AskUserQuestion is the tool name the Claude uses to create forms to get user input, it'd be good to check if ask the user doesn't change behavior of Claude.

@sephmard sephmard force-pushed the nel-assistant-cursor branch 2 times, most recently from 59f1a68 to 53bbb97 Compare February 22, 2026 22:22
- Add global note to use AskUserQuestion when available (Claude Code),
  otherwise fall back to asking in chat (Cursor, Codex, etc.)
- Apply AskUserQuestion-with-fallback pattern to every question in the workflow
- For questions exceeding 4-option limit (Deployment, Benchmarks), show top 3
  options + "Let's chat about it" to handle overflow interactively
- Add .cursor/skills/nel-assistant/SKILL.md for @SKILL.md discoverability in Cursor

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@sephmard sephmard force-pushed the nel-assistant-cursor branch from 53bbb97 to f86c8c4 Compare February 22, 2026 22:46
@sephmard sephmard requested a review from gchlebus February 23, 2026 12:46
@piojanu
Copy link
Contributor

piojanu commented Feb 27, 2026

Do we really need this Cursor-native skill path? You can install the skill for Cursor just fine with nel skills install —cursor, see README. Also, if you just clone the repo and open the NEL package directory in Cursor, it will read .claude/skills, see here.

For sure we can’t just copy&paste it like this.

@piojanu
Copy link
Contributor

piojanu commented Feb 27, 2026

I couldn't reproduce the problem with AskQuestion tool in Cursor. The agent properly invokes the tool with both Claude 4.5 Haiku (non-thinking) and Claude 4.6 Sonnet (medium thinking) models. See the screenshots.

image image

NEL version: 0.1.89
Skill installed with nemo-evaluator-launcher skills install --cursor —project in a clean workspace.

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.

3 participants