From dcc1482912f138c7d08318b200129f2d8515e55e Mon Sep 17 00:00:00 2001 From: Alex Hancock Date: Mon, 3 Mar 2025 13:15:49 -0500 Subject: [PATCH] fix: remove MCP extension install debug shortcut - fixes #1433 --- ui/desktop/src/main.ts | 107 +++-------------------------------------- 1 file changed, 7 insertions(+), 100 deletions(-) diff --git a/ui/desktop/src/main.ts b/ui/desktop/src/main.ts index b7864d184..669d69d3c 100644 --- a/ui/desktop/src/main.ts +++ b/ui/desktop/src/main.ts @@ -194,102 +194,9 @@ const createChat = async (app, query?: string, dir?: string, version?: string) = globalShortcut.unregister('Alt+Command+I'); }; - // Install MCP Extension shortcut - const registerMCPExtensionsShortcut = () => { - globalShortcut.register('Shift+Command+Y', () => { - const defaultUrl = - 'goose://extension?cmd=npx&arg=-y&arg=%40modelcontextprotocol%2Fserver-github&id=github&name=GitHub&description=Repository%20management%2C%20file%20operations%2C%20and%20GitHub%20API%20integration&env=GITHUB_TOKEN%3DGitHub%20personal%20access%20token'; - - const result = dialog.showMessageBoxSync({ - type: 'question', - buttons: ['Install', 'Edit URL', 'Cancel'], - defaultId: 0, - cancelId: 2, - title: 'Install MCP Extension', - message: 'Install MCP Extension', - detail: `Current extension URL:\n\n${defaultUrl}`, - }); - - if (result === 0) { - // User clicked Install - const mockEvent = { - preventDefault: () => { - console.log('Default handling prevented.'); - }, - }; - app.emit('open-url', mockEvent, defaultUrl); - } else if (result === 1) { - // User clicked Edit URL - // Create a simple input dialog - const win = new BrowserWindow({ - width: 800, - height: 120, - frame: false, - transparent: false, - resizable: false, - minimizable: false, - maximizable: false, - parent: BrowserWindow.getFocusedWindow(), - modal: true, - show: false, - webPreferences: { - nodeIntegration: true, - contextIsolation: false, - }, - }); - - win.loadURL(`data:text/html, - - - -
- - -
- - - - `); - - win.once('ready-to-show', () => { - win.show(); - }); - - // Handle the URL submission - ipcMain.once('install-extension-url', (event, url) => { - win.close(); - const mockEvent = { - preventDefault: () => { - console.log('Default handling prevented.'); - }, - }; - if (url && url.trim()) { - app.emit('open-url', mockEvent, url); - } - }); - } - }); - }; - - const unRegisterMCPExtensionsShortcut = () => { - globalShortcut.unregister('Shift+Command+Y'); - }; - // Register shortcuts when window is focused mainWindow.on('focus', () => { registerDevToolsShortcut(mainWindow); - registerMCPExtensionsShortcut(); // Register reload shortcut globalShortcut.register('CommandOrControl+R', () => { mainWindow.reload(); @@ -299,7 +206,6 @@ const createChat = async (app, query?: string, dir?: string, version?: string) = // Unregister shortcuts when window loses focus mainWindow.on('blur', () => { unregisterDevToolsShortcut(); - unRegisterMCPExtensionsShortcut(); globalShortcut.unregister('CommandOrControl+R'); }); @@ -477,7 +383,8 @@ app.whenReady().then(async () => { const appMenu = menu.items.find((item) => item.label === 'Goose'); // add Settings to app menu after About appMenu.submenu.insert(1, new MenuItem({ type: 'separator' })); - appMenu.submenu.insert(1, + appMenu.submenu.insert( + 1, new MenuItem({ label: 'Settings', accelerator: 'CmdOrCtrl+,', @@ -645,16 +552,16 @@ app.whenReady().then(async () => { exec(`cat ${filePath}`, (error, stdout, stderr) => { if (error) { // File not found - resolve({ file: "", filePath, error: null, found: false }); + resolve({ file: '', filePath, error: null, found: false }); } if (stderr) { console.error('Error output:', stderr); - resolve({ file: "", filePath, error, found: false }); + resolve({ file: '', filePath, error, found: false }); } resolve({ file: stdout, filePath, error: null, found: true }); }); - }) - }) + }); + }); ipcMain.handle('write-file', (event, filePath, content) => { return new Promise((resolve) => { @@ -681,4 +588,4 @@ app.on('window-all-closed', () => { if (process.platform !== 'darwin') { app.quit(); } -}); \ No newline at end of file +});