fix(backend): use official cursor CLI install command in InstallScript() (#1099)

This commit is contained in:
Carlos Florêncio
2026-05-27 23:33:27 +01:00
committed by GitHub
parent 0c2df491cc
commit a340f528e4
2 changed files with 6 additions and 1 deletions
@@ -100,7 +100,7 @@ func (a *CursorACP) Runtime() *RuntimeConfig {
func (a *CursorACP) RemoteAuth() *RemoteAuth { return nil }
func (a *CursorACP) InstallScript() string {
return "Install Cursor CLI from https://cursor.com/cli"
return "curl https://cursor.com/install -fsS | bash"
}
func (a *CursorACP) PermissionSettings() map[string]PermissionSetting {
@@ -28,6 +28,7 @@ type acpAgentSpec struct {
inferenceArgv []string // InferenceConfig.Command
passthroughArgv []string // PassthroughCmd (zero-args allowed)
installViaNpm bool // InstallScript starts with "npm install -g"
installScript string // expected InstallScript() value (empty = unchecked)
}
var newACPAgentSpecs = []struct {
@@ -75,6 +76,7 @@ var newACPAgentSpecs = []struct {
inferenceArgv: []string{"cursor-agent", "acp"},
passthroughArgv: []string{"cursor-agent"},
installViaNpm: false,
installScript: "curl https://cursor.com/install -fsS | bash",
}},
{func() Agent { return NewKimiACP() }, acpAgentSpec{
id: "kimi-acp", displayName: "Kimi", detectBinaries: []string{"kimi"},
@@ -180,6 +182,9 @@ func TestNewACPAgents_InstallScript(t *testing.T) {
if !tc.spec.installViaNpm && hasNpm {
t.Errorf("InstallScript() should NOT use npm for native-binary agent: %q", got)
}
if tc.spec.installScript != "" && got != tc.spec.installScript {
t.Errorf("InstallScript() = %q, want %q", got, tc.spec.installScript)
}
// The primary detection binary must be referenced somewhere
// actionable — either argv (native binaries) or InstallScript
// (npm install -g <pkg> ships the bin), so users with "Available