@sentry/wizard 6.10.0 → 6.11.0

package/dist/test/utils/clack/mcp-config.test.js

@@ -37,6 +37,7 @@ vitest_1.vi.mock('../../../src/utils/clack', () => ({
 vitest_1.vi.mock('@clack/prompts', () => ({
     confirm: vitest_1.vi.fn(),
     select: vitest_1.vi.fn(),
+    multiselect: vitest_1.vi.fn(),
     isCancel: vitest_1.vi.fn(() => false),
     cancel: vitest_1.vi.fn(),
     log: {
@@ -77,9 +78,8 @@ vitest_1.vi.mock('node:child_process');
         });
         (0, vitest_1.it)('should configure for Cursor when selected', async () => {
             const { clack, clackUtils } = await getMocks();
-            vitest_1.vi.mocked(clack.select)
-                .mockResolvedValueOnce('yes')
-                .mockResolvedValueOnce('cursor');
+            vitest_1.vi.mocked(clack.select).mockResolvedValueOnce('yes');
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce(['cursor']);
             vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
             const mockReadFile = vitest_1.vi
                 .fn()
@@ -90,9 +90,8 @@ vitest_1.vi.mock('node:child_process');
             vitest_1.vi.spyOn(fs.promises, 'writeFile').mockImplementation(mockWriteFile);
             vitest_1.vi.spyOn(fs, 'mkdirSync').mockImplementation(mockMkdirSync);
             await (0, mcp_config_1.offerProjectScopedMcpConfig)();
-            (0, vitest_1.expect)(clack.select).toHaveBeenCalledTimes(2);
-            (0, vitest_1.expect)(clack.select).toHaveBeenNthCalledWith(2, vitest_1.expect.objectContaining({
-                message: 'Which editor do you want to configure?',
+            (0, vitest_1.expect)(clack.multiselect).toHaveBeenCalledWith(vitest_1.expect.objectContaining({
+                message: 'Which editor(s) do you want to configure?',
                 options: vitest_1.expect.arrayContaining([
                     vitest_1.expect.objectContaining({ value: 'cursor' }),
                     vitest_1.expect.objectContaining({ value: 'vscode' }),
@@ -101,14 +100,13 @@ vitest_1.vi.mock('node:child_process');
             }));
             (0, vitest_1.expect)(mockWriteFile).toHaveBeenCalledWith(vitest_1.expect.stringContaining('.cursor/mcp.json'), vitest_1.expect.stringContaining('"mcpServers"'), 'utf8');
             (0, vitest_1.expect)(clack.log.success).toHaveBeenCalledWith(vitest_1.expect.stringContaining('.cursor/mcp.json'));
-            (0, vitest_1.expect)(clack.log.success).toHaveBeenCalledWith('Added project-scoped Sentry MCP configuration.');
+            (0, vitest_1.expect)(clack.log.success).toHaveBeenCalledWith('Added project-scoped Sentry MCP configuration for Cursor.');
             (0, vitest_1.expect)(clack.log.info).toHaveBeenCalledWith(vitest_1.expect.stringContaining('reload your editor'));
         });
         (0, vitest_1.it)('should configure for VS Code when selected', async () => {
             const { clack, clackUtils } = await getMocks();
-            vitest_1.vi.mocked(clack.select)
-                .mockResolvedValueOnce('yes')
-                .mockResolvedValueOnce('vscode');
+            vitest_1.vi.mocked(clack.select).mockResolvedValueOnce('yes');
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce(['vscode']);
             vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
             const mockReadFile = vitest_1.vi
                 .fn()
@@ -124,9 +122,8 @@ vitest_1.vi.mock('node:child_process');
         });
         (0, vitest_1.it)('should configure for Claude Code when selected', async () => {
             const { clack, clackUtils } = await getMocks();
-            vitest_1.vi.mocked(clack.select)
-                .mockResolvedValueOnce('yes')
-                .mockResolvedValueOnce('claudeCode');
+            vitest_1.vi.mocked(clack.select).mockResolvedValueOnce('yes');
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce(['claudeCode']);
             vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
             const mockReadFile = vitest_1.vi
                 .fn()
@@ -142,9 +139,8 @@ vitest_1.vi.mock('node:child_process');
         });
         (0, vitest_1.it)('should update existing Cursor config file', async () => {
             const { clack, clackUtils } = await getMocks();
-            vitest_1.vi.mocked(clack.select)
-                .mockResolvedValueOnce('yes')
-                .mockResolvedValueOnce('cursor');
+            vitest_1.vi.mocked(clack.select).mockResolvedValueOnce('yes');
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce(['cursor']);
             vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
             const existingConfig = JSON.stringify({
                 mcpServers: {
@@ -169,9 +165,8 @@ vitest_1.vi.mock('node:child_process');
         });
         (0, vitest_1.it)('should update existing VS Code config file', async () => {
             const { clack, clackUtils } = await getMocks();
-            vitest_1.vi.mocked(clack.select)
-                .mockResolvedValueOnce('yes')
-                .mockResolvedValueOnce('vscode');
+            vitest_1.vi.mocked(clack.select).mockResolvedValueOnce('yes');
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce(['vscode']);
             vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
             const existingConfig = JSON.stringify({
                 servers: {
@@ -196,9 +191,8 @@ vitest_1.vi.mock('node:child_process');
         });
         (0, vitest_1.it)('should update existing Claude Code config file', async () => {
             const { clack, clackUtils } = await getMocks();
-            vitest_1.vi.mocked(clack.select)
-                .mockResolvedValueOnce('yes')
-                .mockResolvedValueOnce('claudeCode');
+            vitest_1.vi.mocked(clack.select).mockResolvedValueOnce('yes');
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce(['claudeCode']);
             vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
             const existingConfig = JSON.stringify({
                 mcpServers: {
@@ -219,11 +213,85 @@ vitest_1.vi.mock('node:child_process');
             (0, vitest_1.expect)(writtenContent.mcpServers).toHaveProperty('Sentry');
             (0, vitest_1.expect)(clack.log.success).toHaveBeenCalledWith('Updated .mcp.json');
         });
+        (0, vitest_1.it)('should configure for OpenCode when selected', async () => {
+            const { clack, clackUtils } = await getMocks();
+            vitest_1.vi.mocked(clack.select).mockResolvedValueOnce('yes');
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce(['openCode']);
+            vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
+            const mockReadFile = vitest_1.vi
+                .fn()
+                .mockRejectedValue(new Error('File not found'));
+            const mockWriteFile = vitest_1.vi.fn().mockResolvedValue(undefined);
+            const mockMkdirSync = vitest_1.vi.fn();
+            vitest_1.vi.spyOn(fs.promises, 'readFile').mockImplementation(mockReadFile);
+            vitest_1.vi.spyOn(fs.promises, 'writeFile').mockImplementation(mockWriteFile);
+            vitest_1.vi.spyOn(fs, 'mkdirSync').mockImplementation(mockMkdirSync);
+            await (0, mcp_config_1.offerProjectScopedMcpConfig)();
+            (0, vitest_1.expect)(mockWriteFile).toHaveBeenCalledWith(vitest_1.expect.stringContaining('opencode.json'), vitest_1.expect.stringContaining('"mcp"'), 'utf8');
+            // Verify the written content has the correct structure for OpenCode
+            const writtenContent = JSON.parse(mockWriteFile.mock.calls[0][1]);
+            (0, vitest_1.expect)(writtenContent.$schema).toBe('https://opencode.ai/config.json');
+            (0, vitest_1.expect)(writtenContent.mcp).toHaveProperty('Sentry');
+            (0, vitest_1.expect)(writtenContent.mcp?.Sentry).toHaveProperty('type', 'remote');
+            (0, vitest_1.expect)(writtenContent.mcp?.Sentry).toHaveProperty('url');
+            (0, vitest_1.expect)(writtenContent.mcp?.Sentry?.oauth).toEqual({});
+            (0, vitest_1.expect)(clack.log.success).toHaveBeenCalledWith(vitest_1.expect.stringContaining('opencode.json'));
+            (0, vitest_1.expect)(clack.log.success).toHaveBeenCalledWith('Added project-scoped Sentry MCP configuration for OpenCode.');
+            (0, vitest_1.expect)(clack.log.info).toHaveBeenCalledWith(vitest_1.expect.stringContaining('restart OpenCode'));
+        });
+        (0, vitest_1.it)('should update existing OpenCode config file', async () => {
+            const { clack, clackUtils } = await getMocks();
+            vitest_1.vi.mocked(clack.select).mockResolvedValueOnce('yes');
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce(['openCode']);
+            vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
+            const existingConfig = JSON.stringify({
+                mcp: {
+                    OtherServer: {
+                        type: 'remote',
+                        url: 'https://other.example.com',
+                    },
+                },
+            });
+            const mockReadFile = vitest_1.vi.fn().mockResolvedValue(existingConfig);
+            const mockWriteFile = vitest_1.vi.fn().mockResolvedValue(undefined);
+            const mockMkdirSync = vitest_1.vi.fn();
+            vitest_1.vi.spyOn(fs.promises, 'readFile').mockImplementation(mockReadFile);
+            vitest_1.vi.spyOn(fs.promises, 'writeFile').mockImplementation(mockWriteFile);
+            vitest_1.vi.spyOn(fs, 'mkdirSync').mockImplementation(mockMkdirSync);
+            await (0, mcp_config_1.offerProjectScopedMcpConfig)();
+            const writtenContent = JSON.parse(mockWriteFile.mock.calls[0][1]);
+            (0, vitest_1.expect)(writtenContent.mcp).toHaveProperty('OtherServer');
+            (0, vitest_1.expect)(writtenContent.mcp).toHaveProperty('Sentry');
+            (0, vitest_1.expect)(writtenContent.mcp?.Sentry).toHaveProperty('type', 'remote');
+            (0, vitest_1.expect)(clack.log.success).toHaveBeenCalledWith('Updated opencode.json');
+        });
+        (0, vitest_1.it)('should handle file write errors gracefully for OpenCode', async () => {
+            const { clack, clackUtils } = await getMocks();
+            vitest_1.vi.mocked(clack.select).mockResolvedValueOnce('yes');
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce(['openCode']);
+            vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
+            const mockReadFile = vitest_1.vi
+                .fn()
+                .mockRejectedValue(new Error('File not found'));
+            const mockWriteFile = vitest_1.vi
+                .fn()
+                .mockRejectedValue(new Error('Permission denied'));
+            const mockMkdirSync = vitest_1.vi.fn();
+            vitest_1.vi.spyOn(fs.promises, 'readFile').mockImplementation(mockReadFile);
+            vitest_1.vi.spyOn(fs.promises, 'writeFile').mockImplementation(mockWriteFile);
+            vitest_1.vi.spyOn(fs, 'mkdirSync').mockImplementation(mockMkdirSync);
+            await (0, vitest_1.expect)((0, mcp_config_1.offerProjectScopedMcpConfig)()).resolves.toBeUndefined();
+            (0, vitest_1.expect)(clack.log.warn).toHaveBeenCalledWith(vitest_1.expect.stringContaining('Failed to write MCP config for openCode'));
+            (0, vitest_1.expect)(clackUtils.showCopyPasteInstructions).toHaveBeenCalledWith(vitest_1.expect.objectContaining({
+                filename: 'opencode.json',
+                codeSnippet: vitest_1.expect.stringContaining('"mcp"'),
+                hint: 'create the file if it does not exist',
+            }));
+        });
         (0, vitest_1.it)('should handle file write errors gracefully for Cursor', async () => {
             const { clack, clackUtils } = await getMocks();
-            vitest_1.vi.mocked(clack.select)
-                .mockResolvedValueOnce('yes')
-                .mockResolvedValueOnce('cursor');
+            vitest_1.vi.mocked(clack.select).mockResolvedValueOnce('yes');
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce(['cursor']);
             vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
             const mockReadFile = vitest_1.vi
                 .fn()
@@ -236,7 +304,7 @@ vitest_1.vi.mock('node:child_process');
             vitest_1.vi.spyOn(fs.promises, 'writeFile').mockImplementation(mockWriteFile);
             vitest_1.vi.spyOn(fs, 'mkdirSync').mockImplementation(mockMkdirSync);
             await (0, vitest_1.expect)((0, mcp_config_1.offerProjectScopedMcpConfig)()).resolves.toBeUndefined();
-            (0, vitest_1.expect)(clack.log.warn).toHaveBeenCalledWith(vitest_1.expect.stringContaining('Failed to write MCP config automatically'));
+            (0, vitest_1.expect)(clack.log.warn).toHaveBeenCalledWith(vitest_1.expect.stringContaining('Failed to write MCP config'));
             (0, vitest_1.expect)(clackUtils.showCopyPasteInstructions).toHaveBeenCalledWith(vitest_1.expect.objectContaining({
                 filename: path.join('.cursor', 'mcp.json'),
                 codeSnippet: vitest_1.expect.stringContaining('mcpServers'),
@@ -245,9 +313,8 @@ vitest_1.vi.mock('node:child_process');
         });
         (0, vitest_1.it)('should handle file write errors gracefully for VS Code', async () => {
             const { clack, clackUtils } = await getMocks();
-            vitest_1.vi.mocked(clack.select)
-                .mockResolvedValueOnce('yes')
-                .mockResolvedValueOnce('vscode');
+            vitest_1.vi.mocked(clack.select).mockResolvedValueOnce('yes');
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce(['vscode']);
             vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
             const mockReadFile = vitest_1.vi
                 .fn()
@@ -268,9 +335,8 @@ vitest_1.vi.mock('node:child_process');
         });
         (0, vitest_1.it)('should handle file write errors gracefully for Claude Code', async () => {
             const { clack, clackUtils } = await getMocks();
-            vitest_1.vi.mocked(clack.select)
-                .mockResolvedValueOnce('yes')
-                .mockResolvedValueOnce('claudeCode');
+            vitest_1.vi.mocked(clack.select).mockResolvedValueOnce('yes');
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce(['claudeCode']);
             vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
             const mockReadFile = vitest_1.vi
                 .fn()
@@ -291,9 +357,8 @@ vitest_1.vi.mock('node:child_process');
         });
         (0, vitest_1.it)('should handle update errors and show copy-paste instructions', async () => {
             const { clack, clackUtils } = await getMocks();
-            vitest_1.vi.mocked(clack.select)
-                .mockResolvedValueOnce('yes')
-                .mockResolvedValueOnce('cursor');
+            vitest_1.vi.mocked(clack.select).mockResolvedValueOnce('yes');
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce(['cursor']);
             vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
             // Mock existing file and simulate write error during update
             const existingConfig = JSON.stringify({
@@ -312,14 +377,13 @@ vitest_1.vi.mock('node:child_process');
             vitest_1.vi.spyOn(fs.promises, 'writeFile').mockImplementation(mockWriteFile);
             vitest_1.vi.spyOn(fs, 'mkdirSync').mockImplementation(mockMkdirSync);
             await (0, vitest_1.expect)((0, mcp_config_1.offerProjectScopedMcpConfig)()).resolves.toBeUndefined();
-            (0, vitest_1.expect)(clack.log.warn).toHaveBeenCalledWith(vitest_1.expect.stringContaining('Failed to write MCP config automatically'));
+            (0, vitest_1.expect)(clack.log.warn).toHaveBeenCalledWith(vitest_1.expect.stringContaining('Failed to write MCP config'));
             (0, vitest_1.expect)(clackUtils.showCopyPasteInstructions).toHaveBeenCalled();
         });
         (0, vitest_1.it)('should handle mkdirSync errors', async () => {
             const { clack, clackUtils } = await getMocks();
-            vitest_1.vi.mocked(clack.select)
-                .mockResolvedValueOnce('yes')
-                .mockResolvedValueOnce('cursor');
+            vitest_1.vi.mocked(clack.select).mockResolvedValueOnce('yes');
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce(['cursor']);
             vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
             const mockReadFile = vitest_1.vi
                 .fn()
@@ -332,14 +396,13 @@ vitest_1.vi.mock('node:child_process');
             vitest_1.vi.spyOn(fs.promises, 'writeFile').mockImplementation(mockWriteFile);
             vitest_1.vi.spyOn(fs, 'mkdirSync').mockImplementation(mockMkdirSync);
             await (0, vitest_1.expect)((0, mcp_config_1.offerProjectScopedMcpConfig)()).resolves.toBeUndefined();
-            (0, vitest_1.expect)(clack.log.warn).toHaveBeenCalledWith(vitest_1.expect.stringContaining('Failed to write MCP config automatically'));
+            (0, vitest_1.expect)(clack.log.warn).toHaveBeenCalledWith(vitest_1.expect.stringContaining('Failed to write MCP config'));
             (0, vitest_1.expect)(clackUtils.showCopyPasteInstructions).toHaveBeenCalled();
         });
         (0, vitest_1.it)('should create config with empty servers/mcpServers when existing config lacks them', async () => {
             const { clack, clackUtils } = await getMocks();
-            vitest_1.vi.mocked(clack.select)
-                .mockResolvedValueOnce('yes')
-                .mockResolvedValueOnce('vscode');
+            vitest_1.vi.mocked(clack.select).mockResolvedValueOnce('yes');
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce(['vscode']);
             vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
             const existingConfig = JSON.stringify({
                 otherProperty: 'value',
@@ -361,8 +424,8 @@ vitest_1.vi.mock('node:child_process');
             const { clack, clackUtils } = await getMocks();
             vitest_1.vi.mocked(clack.select)
                 .mockResolvedValueOnce('yes')
-                .mockResolvedValueOnce('jetbrains')
                 .mockResolvedValueOnce(true); // For the clipboard copy prompt
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce(['jetbrains']);
             vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
             // Mock clipboard copy
             const mockSpawn = vitest_1.vi.fn().mockReturnValue({
@@ -393,8 +456,8 @@ vitest_1.vi.mock('node:child_process');
             const { clack, clackUtils } = await getMocks();
             vitest_1.vi.mocked(clack.select)
                 .mockResolvedValueOnce('yes')
-                .mockResolvedValueOnce('other')
                 .mockResolvedValueOnce(true); // For the clipboard copy prompt
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce(['other']);
             vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
             // Mock clipboard copy failure to test fallback
             const mockSpawn = vitest_1.vi.fn().mockReturnValue({
@@ -426,8 +489,8 @@ vitest_1.vi.mock('node:child_process');
             const { clack, clackUtils } = await getMocks();
             vitest_1.vi.mocked(clack.select)
                 .mockResolvedValueOnce('yes')
-                .mockResolvedValueOnce('jetbrains')
                 .mockResolvedValueOnce(true); // For clipboard copy prompt
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce(['jetbrains']);
             vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
             // Mock clipboard copy to throw error
             const mockSpawn = vitest_1.vi.fn().mockImplementation(() => {
@@ -450,8 +513,8 @@ vitest_1.vi.mock('node:child_process');
             const { clack, clackUtils } = await getMocks();
             vitest_1.vi.mocked(clack.select)
                 .mockResolvedValueOnce('explain') // User selects "What is MCP?"
-                .mockResolvedValueOnce(true) // User selects "Yes" after explanation
-                .mockResolvedValueOnce('cursor'); // User selects Cursor
+                .mockResolvedValueOnce(true); // User selects "Yes" after explanation
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce(['cursor']); // User selects Cursor
             vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
             const mockReadFile = vitest_1.vi
                 .fn()
@@ -477,8 +540,8 @@ vitest_1.vi.mock('node:child_process');
             const { clack, clackUtils } = await getMocks();
             vitest_1.vi.mocked(clack.select)
                 .mockResolvedValueOnce('yes')
-                .mockResolvedValueOnce('jetbrains')
                 .mockResolvedValueOnce(false); // User declines to copy to clipboard
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce(['jetbrains']);
             vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
             const mockSpawn = vitest_1.vi.fn();
             vitest_1.vi.spyOn(childProcess, 'spawn').mockImplementation(mockSpawn);
@@ -511,8 +574,70 @@ vitest_1.vi.mock('node:child_process');
                 message: 'Would you like to configure MCP for your IDE now?',
             }));
             // Should NOT proceed with editor selection
-            (0, vitest_1.expect)(clack.select).not.toHaveBeenCalledWith(vitest_1.expect.objectContaining({
-                message: 'Which editor do you want to configure?',
+            (0, vitest_1.expect)(clack.multiselect).not.toHaveBeenCalled();
+        });
+        (0, vitest_1.it)('should configure multiple editors when selected', async () => {
+            const { clack, clackUtils } = await getMocks();
+            vitest_1.vi.mocked(clack.select).mockResolvedValueOnce('yes');
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce([
+                'cursor',
+                'vscode',
+                'claudeCode',
+            ]);
+            vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
+            const mockReadFile = vitest_1.vi
+                .fn()
+                .mockRejectedValue(new Error('File not found'));
+            const mockWriteFile = vitest_1.vi.fn().mockResolvedValue(undefined);
+            const mockMkdirSync = vitest_1.vi.fn();
+            vitest_1.vi.spyOn(fs.promises, 'readFile').mockImplementation(mockReadFile);
+            vitest_1.vi.spyOn(fs.promises, 'writeFile').mockImplementation(mockWriteFile);
+            vitest_1.vi.spyOn(fs, 'mkdirSync').mockImplementation(mockMkdirSync);
+            await (0, mcp_config_1.offerProjectScopedMcpConfig)();
+            // Should write config for all three editors
+            (0, vitest_1.expect)(mockWriteFile).toHaveBeenCalledTimes(3);
+            (0, vitest_1.expect)(mockWriteFile).toHaveBeenCalledWith(vitest_1.expect.stringContaining('.cursor/mcp.json'), vitest_1.expect.stringContaining('"mcpServers"'), 'utf8');
+            (0, vitest_1.expect)(mockWriteFile).toHaveBeenCalledWith(vitest_1.expect.stringContaining('.vscode/mcp.json'), vitest_1.expect.stringContaining('"servers"'), 'utf8');
+            (0, vitest_1.expect)(mockWriteFile).toHaveBeenCalledWith(vitest_1.expect.stringContaining('.mcp.json'), vitest_1.expect.stringContaining('"mcpServers"'), 'utf8');
+            // Should show success messages for each (twice per editor: filename + editor-specific message)
+            (0, vitest_1.expect)(clack.log.success).toHaveBeenCalledWith('Added project-scoped Sentry MCP configuration for Cursor.');
+            (0, vitest_1.expect)(clack.log.success).toHaveBeenCalledWith('Added project-scoped Sentry MCP configuration for VS Code.');
+            (0, vitest_1.expect)(clack.log.success).toHaveBeenCalledWith('Added project-scoped Sentry MCP configuration for Claude Code.');
+        });
+        (0, vitest_1.it)('should return early when no editors are selected', async () => {
+            const { clack, clackUtils } = await getMocks();
+            vitest_1.vi.mocked(clack.select).mockResolvedValueOnce('yes');
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce([]);
+            vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
+            const mockWriteFile = vitest_1.vi.fn();
+            vitest_1.vi.spyOn(fs.promises, 'writeFile').mockImplementation(mockWriteFile);
+            await (0, mcp_config_1.offerProjectScopedMcpConfig)();
+            (0, vitest_1.expect)(clack.log.info).toHaveBeenCalledWith('No editors selected. You can add MCP configuration later.');
+            (0, vitest_1.expect)(mockWriteFile).not.toHaveBeenCalled();
+        });
+        (0, vitest_1.it)('should handle mixed success and failure for multiple editors', async () => {
+            const { clack, clackUtils } = await getMocks();
+            vitest_1.vi.mocked(clack.select).mockResolvedValueOnce('yes');
+            vitest_1.vi.mocked(clack.multiselect).mockResolvedValueOnce(['cursor', 'vscode']);
+            vitest_1.vi.mocked(clackUtils.abortIfCancelled).mockImplementation((value) => Promise.resolve(value));
+            const mockReadFile = vitest_1.vi
+                .fn()
+                .mockRejectedValue(new Error('File not found'));
+            const mockWriteFile = vitest_1.vi
+                .fn()
+                .mockResolvedValueOnce(undefined) // Cursor succeeds
+                .mockRejectedValueOnce(new Error('Permission denied')); // VS Code fails
+            const mockMkdirSync = vitest_1.vi.fn();
+            vitest_1.vi.spyOn(fs.promises, 'readFile').mockImplementation(mockReadFile);
+            vitest_1.vi.spyOn(fs.promises, 'writeFile').mockImplementation(mockWriteFile);
+            vitest_1.vi.spyOn(fs, 'mkdirSync').mockImplementation(mockMkdirSync);
+            await (0, mcp_config_1.offerProjectScopedMcpConfig)();
+            // Cursor should succeed
+            (0, vitest_1.expect)(clack.log.success).toHaveBeenCalledWith(vitest_1.expect.stringContaining('.cursor/mcp.json'));
+            // VS Code should fail and show fallback
+            (0, vitest_1.expect)(clack.log.warn).toHaveBeenCalledWith(vitest_1.expect.stringContaining('Failed to write MCP config for vscode'));
+            (0, vitest_1.expect)(clackUtils.showCopyPasteInstructions).toHaveBeenCalledWith(vitest_1.expect.objectContaining({
+                filename: path.join('.vscode', 'mcp.json'),
             }));
         });
     });