Skip to content

fix: add cursor in setup - #934

Closed
xidongc wants to merge 2 commits into
garrytan:mainfrom
xidongc:fix-cursor-setup
Closed

xidongc wants to merge 2 commits into
garrytan:mainfrom
xidongc:fix-cursor-setup

Conversation

@xidongc

@xidongc xidongc commented Apr 9, 2026 •

Copy link
Copy Markdown

when run
./setup --host cursor

the return error is
return: Unknown --host value: cursor (expected claude, codex, kiro, factory, openclaw, or auto)

Add cursor in setup to fix this

@Guria

Guria commented Apr 12, 2026

Copy link
Copy Markdown

This also needs change help text

@xjiaxiang

Copy link
Copy Markdown

and the path of cursor skills should be ~/.cursor/skills-cursor/

@xidongc

xidongc commented Apr 15, 2026 •

Copy link
Copy Markdown
Author

and the path of cursor skills should be ~/.cursor/skills-cursor/

~/.cursor/skills-cursor/ : Holds Cursor’s built-in skills managed by Cursor itself.
~/.cursor/skills/ :Holds user third-party skills.

@nehaaprasad

Copy link
Copy Markdown

AI Code Trust — ship readiness

Score: 100 · Verdict: SAFE · Model: deterministic-v1

No blocking issues detected by automated checks.

@time-attack

Copy link
Copy Markdown
Contributor

@16francej — recommend closing this PR as superseded by #2142, the broader target for complete Cursor support. This patch only changes setup allowlisting and help text; it does not generate, install, own, or uninstall Cursor skills, and unresolved security problems remain. Preserve contributor attribution and the original Cursor setup intent, but not this implementation.

@capy-ai

capy-ai Bot commented Oct 3, 2026

Copy link
Copy Markdown
Contributor

Thanks for this, @xidongc. The fix is now on main: setup accepts --host cursor (already on main as of v1.91.16.0). Closing as superseded; if you still see the problem on the latest gstack, please open an issue with ./setup --status output.

@capy-ai capy-ai Bot closed this Oct 3, 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.

5 participants