fix: wire up Cursor integration in installer (#1605)
* fix: wire up Cursor integration in installer — was incorrectly marked "coming soon" CursorHooksInstaller.ts was fully built but never connected to the installer. Set supported: true in IDE detection and call installCursorHooks in the setup flow, matching the pattern used by other integrations. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * fix: wire up Cursor MCP configuration during install PR review flagged that the hint says "hooks + MCP integration" but configureCursorMcp() was never called during install. Now invoked after hooks install with graceful fallback if MCP setup fails. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -116,7 +116,8 @@ export function detectInstalledIDEs(): IDEInfo[] {
|
|||||||
id: 'cursor',
|
id: 'cursor',
|
||||||
label: 'Cursor',
|
label: 'Cursor',
|
||||||
detected: existsSync(join(home, '.cursor')),
|
detected: existsSync(join(home, '.cursor')),
|
||||||
supported: false,
|
supported: true,
|
||||||
|
hint: 'hooks + MCP integration',
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
id: 'copilot-cli',
|
id: 'copilot-cli',
|
||||||
|
|||||||
@@ -135,9 +135,22 @@ async function setupIDEs(selectedIDEs: string[]): Promise<string[]> {
|
|||||||
break;
|
break;
|
||||||
}
|
}
|
||||||
|
|
||||||
case 'cursor':
|
case 'cursor': {
|
||||||
log.warn('Cursor: integration not yet implemented. Skipping.');
|
const { installCursorHooks, configureCursorMcp } = await import('../../services/integrations/CursorHooksInstaller.js');
|
||||||
|
const cursorResult = await installCursorHooks('user');
|
||||||
|
if (cursorResult === 0) {
|
||||||
|
const mcpResult = configureCursorMcp('user');
|
||||||
|
if (mcpResult === 0) {
|
||||||
|
log.success('Cursor: hooks + MCP installed.');
|
||||||
|
} else {
|
||||||
|
log.success('Cursor: hooks installed (MCP setup failed — run `npx claude-mem cursor mcp` to retry).');
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
log.error('Cursor: hook installation failed.');
|
||||||
|
failedIDEs.push(ideId);
|
||||||
|
}
|
||||||
break;
|
break;
|
||||||
|
}
|
||||||
|
|
||||||
case 'gemini-cli': {
|
case 'gemini-cli': {
|
||||||
const { installGeminiCliHooks } = await import('../../services/integrations/GeminiCliHooksInstaller.js');
|
const { installGeminiCliHooks } = await import('../../services/integrations/GeminiCliHooksInstaller.js');
|
||||||
|
|||||||
Reference in New Issue
Block a user