Skip to content

fix(setup): accept --host cursor and --host slate (#2361) - #2432

Closed
gregario wants to merge 1 commit into
garrytan:mainfrom
gregario:fix/issue-2361
Closed

fix(setup): accept --host cursor and --host slate (#2361)#2432
gregario wants to merge 1 commit into
garrytan:mainfrom
gregario:fix/issue-2361

Conversation

@gregario

@gregario gregario commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes #2361./setup --host cursor and ./setup --host slate exit with "Unknown --host value" even though both hosts have full configs in hosts/cursor.ts and hosts/slate.ts and are registered in ALL_HOST_CONFIGS.

The case statement at setup line 104 listed only claude|codex|kiro|factory|opencode|auto as pass-through hosts. Cursor and Slate are local IDE hosts with the same installation pattern (directory-based skill discovery) but were never added to the allowlist. The fix adds them alongside the other local hosts.

Verification

  • bun test test/setup-host-validation.test.ts green (3/3)
  • Regression tests fail on main:
    • "--host cursor is accepted" → fails because stderr contains "Unknown --host value"
    • "--host slate is accepted" → same
    Expected to not contain: "Unknown --host value"
    Received: "Unknown --host value: cursor (...)"
    
  • Both pass after fix applied
  • Discrimination verified: reverting only setup causes both failures

Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com

Fixes garrytan#2361.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@trunk-io

trunk-io Bot commented Aug 1, 2026

Copy link
Copy Markdown

Merging to main in this repository is managed by Trunk.

  • To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

@garrytan

Copy link
Copy Markdown
Owner

Closing: this was absorbed or superseded on main by v1.67.0.0 or earlier (see the v1.67 tracker-wave receipts in PR #2604). Thank you!

@garrytan garrytan closed this Aug 17, 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.

setup --host rejects cursor and slate, but both are registered in ALL_HOST_CONFIGS

2 participants