[AXON-1978] Support /mcp command in Rovo Dev (not BBY) (#1666)

Author: marcomura

CHANGELOG.md

@@ -1,25 +1,11 @@
 ### [Report an Issue](https://github.com/atlassian/atlascode/issues)
 
-## What's new in 4.0.24
-
-### Improvements
-
-- **RovoDev**: Enhanced "Fix by Rovo Dev" and "Explain by Rovo Dev" code actions with rich context extraction - now includes actual code content, surrounding code, import statements, and structured diagnostics for significantly better AI responses
-
-### Bug Fixes
-
-- **RovoDev**: Fixed chat message not appearing when clicking "Fix with Rovo Dev" before the chat view is fully initialized - now waits for the webview to be ready before executing the chat command
-- Fixed "Cannot read properties of undefined (reading 'initiateApiTokenAuth')" error
-
-### Features
-
-- **RovoDev**: Added copy code button within the Rovo Dev chat for code blocks in the chat.
-
 ## What's new in 4.0.23
 
 ### Features
 
 - **Rovo Dev**: Support new `plan` mode with `deferred_request` handling
+- **RovoDev**: Added copy code button within the Rovo Dev chat for code blocks in the chat.
 
 - **Rovo Dev**: Added Rovo Dev icon to the editor title bar for quick access — clicking it focuses the existing Rovo Dev sidebar panel
 
@@ -32,14 +18,17 @@
 - **RovoDev**: Fixed prompt input performance degradation after extended idle periods by properly disposing Monaco editor resources and event listeners on component cleanup
 - **RovoDev**: Refactored JSON parsing logic with `safeJsonParse` helper function to reduce code duplication and improve maintainability
 - **RovoDev**: Centered text within tool call statements in RovoDev chat.
+- **RovoDev**: Enhanced "Fix by Rovo Dev" and "Explain by Rovo Dev" code actions with rich context extraction - now includes actual code content, surrounding code, import statements, and structured diagnostics for significantly better AI responses
 
 ### Bug Fixes
 
 - Bitbucket DC: Fixed PRs list pagination
 
 - **RovoDev**: Fixed JSON parsing errors in ToolReturnMessage handling - added type checking before JSON.parse() to prevent "Input data should be a String" and invalid JSON errors
-- Added comprehensive test coverage for parseToolReturnMessage with both string and pre-parsed object inputs
-- Rovo Dev: Unsupported slash commands now show a helpful warning instead of an error dialog
+- **Rovo Dev**: Unsupported slash commands now show a helpful warning instead of an error dialog
+- **Rovo Dev**: Added support for /mcp command
+- **RovoDev**: Fixed chat message not appearing when clicking "Fix with Rovo Dev" before the chat view is fully initialized - now waits for the webview to be ready before executing the chat command
+- Fixed "Cannot read properties of undefined (reading 'initiateApiTokenAuth')" error
 
 ## What's new in 4.0.22
 

src/rovo-dev/rovoDevWebviewProvider.ts

@@ -562,6 +562,10 @@ export class RovoDevWebviewProvider extends Disposable implements WebviewViewPro
                         await env.openExternal(Uri.parse(e.href));
                         break;
 
+                    case RovoDevViewResponseType.OpenMcpConfiguration:
+                        await commands.executeCommand(RovodevCommands.OpenRovoDevMcpJson);
+                        break;
+
                     case RovoDevViewResponseType.OpenRovoDevLogFile:
                         await commands.executeCommand(RovodevCommands.OpenRovoDevLogFile);
                         break;

src/rovo-dev/ui/prompt-box/prompt-input/PromptInput.test.tsx

@@ -78,6 +78,7 @@ describe('PromptInputBox', () => {
         sendButtonDisabled: false,
         onAddContext: jest.fn(),
         onCopy: jest.fn(),
+        handleMcpConfigurationCommand: jest.fn(),
         handleMemoryCommand: jest.fn(),
         handleTriggerFeedbackCommand: jest.fn(),
         handleSessionCommand: jest.fn(),

src/rovo-dev/ui/prompt-box/prompt-input/PromptInput.tsx

@@ -23,11 +23,11 @@ import {
     createMonacoPromptEditor,
     createPromptCompletionProvider,
     createSlashCommandProvider,
+    getSlashCommands,
     removeMonacoStyles,
     setupAutoResize,
     setupMonacoCommands,
     setupPromptKeyBindings,
-    SLASH_COMMANDS,
 } from './utils';
 
 type NonDisabledState = Exclude<State, DisabledState>;
@@ -55,6 +55,7 @@ interface PromptInputBoxProps {
     onCancel: () => void;
     onAddContext: () => void;
     onCopy: () => void;
+    handleMcpConfigurationCommand: () => void;
     handleMemoryCommand: () => void;
     handleTriggerFeedbackCommand: () => void;
     handleSessionCommand?: () => void;
@@ -85,11 +86,7 @@ let monacoInitialized = false;
 
 function initMonaco(isBBY: boolean) {
     if (!monacoInitialized) {
-        let commands = SLASH_COMMANDS;
-        if (isBBY) {
-            commands = commands.filter((command) => command.label !== '/yolo' && command.label !== '/sessions');
-        }
-
+        const commands = getSlashCommands(isBBY).filter((command) => !command.disabled);
         monaco.languages.registerCompletionItemProvider('plaintext', createSlashCommandProvider(commands));
 
         monacoInitialized = true;
@@ -131,6 +128,7 @@ export const PromptInputBox: React.FC<PromptInputBoxProps> = ({
     onCancel,
     onAddContext,
     onCopy,
+    handleMcpConfigurationCommand,
     handleMemoryCommand,
     handleTriggerFeedbackCommand,
     handleSessionCommand,
@@ -229,6 +227,7 @@ export const PromptInputBox: React.FC<PromptInputBoxProps> = ({
             editor,
             onSend,
             onCopy,
+            handleMcpConfigurationCommand,
             handleMemoryCommand,
             handleTriggerFeedbackCommand,
             handleSessionCommand,
@@ -243,6 +242,7 @@ export const PromptInputBox: React.FC<PromptInputBoxProps> = ({
         editor,
         onSend,
         onCopy,
+        handleMcpConfigurationCommand,
         handleMemoryCommand,
         handleTriggerFeedbackCommand,
         onYoloModeToggled,

src/rovo-dev/ui/prompt-box/prompt-input/utils.tsx

@@ -76,100 +76,120 @@ export const createMonacoPromptEditor = (container: HTMLElement) => {
 interface SlashCommand {
     label: string;
     insertText: string;
+    disabled?: boolean;
     description?: string;
     command?: monaco.languages.Command;
 }
 
-export const SLASH_COMMANDS: SlashCommand[] = [
-    {
-        label: '/clear',
-        insertText: '/clear',
-        description: 'Clear the chat',
-        command: { title: 'Clear', id: 'rovo-dev.clearChat', tooltip: 'Clear the chat' },
-    },
-    {
-        label: '/copy',
-        insertText: '/copy',
-        description: 'Copy the last response to clipboard',
-        command: { title: 'Copy', id: 'rovo-dev.copyResponse', tooltip: 'Copy the last response to clipboard' },
-    },
-    {
-        label: '/feedback',
-        insertText: '/feedback',
-        description: 'Provide feedback on Rovo Dev',
-        command: { title: 'Feedback', id: 'rovo-dev.triggerFeedback', tooltip: 'Provide feedback on Rovo Dev' },
-    },
-    {
-        label: '/memory',
-        insertText: '/memory',
-        description: 'Show agent memory',
-        command: { title: 'Agent Memory', id: 'rovo-dev.agentMemory', tooltip: 'Show agent memory' },
-    },
-    {
-        label: '/models',
-        insertText: '/models',
-        description: 'Show available agent models',
-        command: {
-            title: 'Models',
-            id: 'rovo-dev.triggerAgentModels',
-            tooltip: 'Show available agent models',
+export function getSlashCommands(isBBY: boolean): SlashCommand[] {
+    return [
+        {
+            label: '/clear',
+            insertText: '/clear',
+            description: 'Clear the chat',
+            command: { title: 'Clear', id: 'rovo-dev.clearChat', tooltip: 'Clear the chat' },
         },
-    },
-    {
-        label: '/prompts',
-        insertText: '!',
-        description: 'Show saved prompts',
-        command: {
-            title: 'Prompts',
-            id: 'rovo-dev.triggerPrompts',
-            tooltip: 'Show saved prompts',
+        {
+            label: '/copy',
+            insertText: '/copy',
+            description: 'Copy the last response to clipboard',
+            command: { title: 'Copy', id: 'rovo-dev.copyResponse', tooltip: 'Copy the last response to clipboard' },
         },
-    },
-    {
-        label: '/prune',
-        insertText: '/prune',
-        description: 'Prune the chat',
-        command: { title: 'Prune', id: 'rovo-dev.pruneChat', tooltip: 'Prune the chat' },
-    },
-    {
-        label: '/status',
-        insertText: '/status',
-        description: 'Show Rovo Dev status including version, account details and model',
-        command: {
-            title: 'Status',
-            id: 'rovo-dev.triggerStatus',
-            tooltip: 'Show Rovo Dev status including version, account details and model',
+        {
+            label: '/feedback',
+            insertText: '/feedback',
+            description: 'Provide feedback on Rovo Dev',
+            command: { title: 'Feedback', id: 'rovo-dev.triggerFeedback', tooltip: 'Provide feedback on Rovo Dev' },
         },
-    },
-    {
-        label: '/usage',
-        insertText: '/usage',
-        description: 'Show Rovo Dev credit usage',
-        command: {
-            title: 'Usage',
-            id: 'rovo-dev.triggerUsage',
-            tooltip: 'Show Rovo Dev credit usage',
+        {
+            label: '/mcp',
+            insertText: '',
+            description: 'Open the MCP configuration file',
+            command: {
+                title: 'MCP configuration',
+                id: 'rovo-dev.mcpConfiguration',
+                tooltip: 'Open the MCP configuration file',
+            },
+            disabled: isBBY,
         },
-    },
-    {
-        label: '/yolo',
-        insertText: '/yolo',
-        description: 'Toggle tool confirmations',
-        command: { title: 'Yolo Mode', id: 'rovo-dev.toggleYoloMode', tooltip: 'Toggle tool confirmations' },
-    },
-    {
-        label: '/ask',
-        insertText: '/ask ',
-        description: 'Ask a question in read-only mode (no file changes or terminal access)',
-        command: { title: 'Ask Mode', id: 'rovo-dev.invokeAskMode', tooltip: 'Invoke Ask mode' },
-    },
-    {
-        label: '/sessions',
-        insertText: '/sessions',
-        description: 'View and manage your agent sessions',
-        command: { title: 'Sessions', id: 'rovo-dev.triggerSessions', tooltip: 'View and manage your agent sessions' },
-    },
-];
+        {
+            label: '/memory',
+            insertText: '/memory',
+            description: 'Show agent memory',
+            command: { title: 'Agent Memory', id: 'rovo-dev.agentMemory', tooltip: 'Show agent memory' },
+        },
+        {
+            label: '/models',
+            insertText: '/models',
+            description: 'Show available agent models',
+            command: {
+                title: 'Models',
+                id: 'rovo-dev.triggerAgentModels',
+                tooltip: 'Show available agent models',
+            },
+        },
+        {
+            label: '/prompts',
+            insertText: '!',
+            description: 'Show saved prompts',
+            command: {
+                title: 'Prompts',
+                id: 'rovo-dev.triggerPrompts',
+                tooltip: 'Show saved prompts',
+            },
+        },
+        {
+            label: '/prune',
+            insertText: '/prune',
+            description: 'Prune the chat',
+            command: { title: 'Prune', id: 'rovo-dev.pruneChat', tooltip: 'Prune the chat' },
+        },
+        {
+            label: '/status',
+            insertText: '/status',
+            description: 'Show Rovo Dev status including version, account details and model',
+            command: {
+                title: 'Status',
+                id: 'rovo-dev.triggerStatus',
+                tooltip: 'Show Rovo Dev status including version, account details and model',
+            },
+        },
+        {
+            label: '/usage',
+            insertText: '/usage',
+            description: 'Show Rovo Dev credit usage',
+            command: {
+                title: 'Usage',
+                id: 'rovo-dev.triggerUsage',
+                tooltip: 'Show Rovo Dev credit usage',
+            },
+        },
+        {
+            label: '/yolo',
+            insertText: '/yolo',
+            description: 'Toggle tool confirmations',
+            command: { title: 'Yolo Mode', id: 'rovo-dev.toggleYoloMode', tooltip: 'Toggle tool confirmations' },
+            disabled: isBBY,
+        },
+        {
+            label: '/ask',
+            insertText: '/ask ',
+            description: 'Ask a question in read-only mode (no file changes or terminal access)',
+            command: { title: 'Ask Mode', id: 'rovo-dev.invokeAskMode', tooltip: 'Invoke Ask mode' },
+        },
+        {
+            label: '/sessions',
+            insertText: '/sessions',
+            description: 'View and manage your agent sessions',
+            command: {
+                title: 'Sessions',
+                id: 'rovo-dev.triggerSessions',
+                tooltip: 'View and manage your agent sessions',
+            },
+            disabled: isBBY,
+        },
+    ];
+}
 
 export const createSlashCommandProvider = (commands: SlashCommand[]): monaco.languages.CompletionItemProvider => {
     return {
@@ -279,6 +299,7 @@ export function setupMonacoCommands(
     editor: monaco.editor.IStandaloneCodeEditor,
     onSend: (text: string) => boolean,
     onCopy: () => void,
+    handleMcpConfigurationCommand: () => void,
     handleMemoryCommand: () => void,
     handleTriggerFeedbackCommand: () => void,
     handleSessionCommand?: () => void,
@@ -316,6 +337,13 @@ export function setupMonacoCommands(
         }),
     );
 
+    disposables.push(
+        monaco.editor.registerCommand('rovo-dev.mcpConfiguration', () => {
+            handleMcpConfigurationCommand();
+            editor.setValue('');
+        }),
+    );
+
     disposables.push(
         monaco.editor.registerCommand('rovo-dev.triggerPrompts', () => {
             // Trigger suggestions for saved prompts

src/rovo-dev/ui/rovoDevView.tsx

@@ -809,6 +809,12 @@ const RovoDevView: React.FC = () => {
         navigator.clipboard?.writeText(lastMessage.content);
     }, [currentState, history]);
 
+    const executeOpenMcpConfigurationFile = useCallback(() => {
+        postMessage({
+            type: RovoDevViewResponseType.OpenMcpConfiguration,
+        });
+    }, [postMessage]);
+
     const executeGetAgentMemory = useCallback(() => {
         postMessage({
             type: RovoDevViewResponseType.GetAgentMemory,
@@ -1237,6 +1243,7 @@ const RovoDevView: React.FC = () => {
                                             onCancel={cancelResponse}
                                             onAddContext={onAddContext}
                                             onCopy={handleCopyResponse}
+                                            handleMcpConfigurationCommand={executeOpenMcpConfigurationFile}
                                             handleMemoryCommand={executeGetAgentMemory}
                                             handleTriggerFeedbackCommand={handleShowFeedbackForm}
                                             promptText={promptText}

src/rovo-dev/ui/rovoDevViewMessages.tsx

@@ -28,6 +28,7 @@ export const enum RovoDevViewResponseType {
     ReportCreatePrButtonClicked = 'reportCreatePrButtonClicked',
     CheckGitChanges = 'checkGitChanges',
     WebviewReady = 'webviewReady',
+    OpenMcpConfiguration = 'openMcpConfiguration',
     GetAgentMemory = 'getAgentMemory',
     TriggerFeedback = 'triggerFeedback',
     SendFeedback = 'sendFeedback',
@@ -86,6 +87,7 @@ export type RovoDevViewResponse =
     | ReducerAction<RovoDevViewResponseType.ReportCreatePrButtonClicked>
     | ReducerAction<RovoDevViewResponseType.CheckGitChanges>
     | ReducerAction<RovoDevViewResponseType.WebviewReady>
+    | ReducerAction<RovoDevViewResponseType.OpenMcpConfiguration>
     | ReducerAction<RovoDevViewResponseType.GetAgentMemory>
     | ReducerAction<RovoDevViewResponseType.TriggerFeedback>
     | ReducerAction<