fix: migration from /agents ux (aws#2248)

Author: laileni-aws

chat-client/src/client/mynahUi.ts

@@ -69,12 +69,7 @@ import {
     toMynahIcon,
 } from './utils'
 import { ChatHistory, ChatHistoryList } from './features/history'
-import {
-    pairProgrammingModeOff,
-    pairProgrammingModeOn,
-    programmerModeCard,
-    createRerouteCard,
-} from './texts/pairProgramming'
+import { pairProgrammingModeOff, pairProgrammingModeOn, programmerModeCard } from './texts/pairProgramming'
 import { ContextRule, RulesList } from './features/rules'
 import { getModelSelectionChatItem, modelUnavailableBanner, modelThrottledBanner } from './texts/modelSelection'
 import {
@@ -285,11 +280,6 @@ export const handleChatPrompt = (
     // For /doc command, don't show any prompt in UI
     const displayPrompt = isReroutedCommand && prompt.command === '/doc' ? '' : userPrompt
     initializeChatResponse(mynahUi, tabId, displayPrompt, agenticMode)
-
-    // If this is a rerouted command AND reroute feature is enabled, show the reroute card after the prompt
-    if (isReroutedCommand && tabFactory?.isRerouteEnabled() && prompt.command) {
-        mynahUi.addChatItem(tabId, createRerouteCard(prompt.command))
-    }
 }
 
 const initializeChatResponse = (mynahUi: MynahUI, tabId: string, userPrompt?: string, agenticMode?: boolean) => {
@@ -404,9 +394,6 @@ export const createMynahUi = (
             const defaultTabBarData = tabFactory.getDefaultTabData()
             const defaultTabConfig: Partial<MynahUIDataModel> = {
                 quickActionCommands: defaultTabBarData.quickActionCommands,
-                ...(tabFactory.isRerouteEnabled()
-                    ? { quickActionCommandsHeader: defaultTabBarData.quickActionCommandsHeader }
-                    : {}),
                 tabBarButtons: defaultTabBarData.tabBarButtons,
                 contextCommands: [
                     ...(contextCommandGroups || []),

chat-client/src/client/tabs/tabFactory.ts

@@ -4,7 +4,6 @@ import {
     MynahIcons,
     MynahUIDataModel,
     QuickActionCommandGroup,
-    QuickActionCommandsHeader,
     TabBarMainAction,
 } from '@aws/mynah-ui'
 import { disclaimerCard } from '../texts/disclaimer'
@@ -143,17 +142,6 @@ Select code & ask me to explain, debug or optimize it, or type \`/\` for quick a
                       quickActionCommands: this.quickActionCommands,
                   }
                 : {}),
-            ...(this.reroute
-                ? {
-                      quickActionCommandsHeader: {
-                          status: 'warning',
-                          icon: MynahIcons.INFO,
-                          title: 'Q Developer agentic capabilities',
-                          description:
-                              "You can now ask Q directly in the chat to generate code, documentation, and unit tests. You don't need to explicitly use /dev, /test, /review or /doc",
-                      } as QuickActionCommandsHeader,
-                  }
-                : {}),
         }
 
         tabData.tabBarButtons = this.getTabBarButtons()
@@ -229,50 +217,4 @@ Select code & ask me to explain, debug or optimize it, or type \`/\` for quick a
 
         return tabBarButtons.length ? tabBarButtons : undefined
     }
-
-    // Enhanced welcome messages block for non-agentic mode
-    private getEnhancedWelcomeBlock() {
-        return {
-            text: '',
-            options: [
-                {
-                    pillText: 'Getting Started',
-                    prompt: 'What can Amazon Q help me with?',
-                    type: 'help',
-                },
-            ],
-        }
-    }
-
-    // Agentic welcome messages block
-    private getAgenticWelcomeBlock() {
-        return {
-            text: '',
-            options: [
-                {
-                    pillText: 'Getting Started',
-                    prompt: 'What can Amazon Q help me with?',
-                    type: 'help',
-                },
-            ],
-        }
-    }
-
-    // Legacy welcome messages block
-    private getWelcomeBlock() {
-        return {
-            text: 'Try Examples:',
-            options: [
-                {
-                    pillText: 'Explain selected code',
-                    prompt: 'Explain selected code',
-                    type: 'init-prompt',
-                },
-                {
-                    pillText: 'How can Amazon Q help me?',
-                    type: 'help',
-                },
-            ],
-        }
-    }
 }

chat-client/src/client/texts/pairProgramming.test.ts

@@ -1,14 +1,10 @@
 import * as assert from 'assert'
-import { ChatItemType, MynahIcons } from '@aws/mynah-ui'
+import { ChatItemType } from '@aws/mynah-ui'
 import {
     programmerModeCard,
     pairProgrammingPromptInput,
     pairProgrammingModeOn,
     pairProgrammingModeOff,
-    testRerouteCard,
-    docRerouteCard,
-    devRerouteCard,
-    createRerouteCard,
 } from './pairProgramming'
 
 describe('pairProgramming', () => {
@@ -56,67 +52,4 @@ describe('pairProgramming', () => {
             assert.equal(pairProgrammingModeOff.body, 'Agentic coding - OFF')
         })
     })
-
-    describe('testRerouteCard', () => {
-        it('has correct properties', () => {
-            assert.equal(testRerouteCard.type, ChatItemType.ANSWER)
-            assert.equal(testRerouteCard.border, true)
-            assert.equal(testRerouteCard.header?.padding, true)
-            assert.equal(testRerouteCard.header?.iconForegroundStatus, 'warning')
-            assert.equal(testRerouteCard.header?.icon, MynahIcons.INFO)
-            assert.ok(testRerouteCard.header?.body?.includes('generate unit tests'))
-            assert.ok(testRerouteCard.body?.includes("You don't need to explicitly use /test"))
-        })
-    })
-
-    describe('docRerouteCard', () => {
-        it('has correct properties', () => {
-            assert.equal(docRerouteCard.type, ChatItemType.ANSWER)
-            assert.equal(docRerouteCard.border, true)
-            assert.equal(docRerouteCard.header?.padding, true)
-            assert.equal(docRerouteCard.header?.iconForegroundStatus, 'warning')
-            assert.equal(docRerouteCard.header?.icon, MynahIcons.INFO)
-            assert.ok(docRerouteCard.header?.body?.includes('generate documentation'))
-            assert.ok(docRerouteCard.body?.includes("You don't need to explicitly use /doc"))
-        })
-    })
-
-    describe('devRerouteCard', () => {
-        it('has correct properties', () => {
-            assert.equal(devRerouteCard.type, ChatItemType.ANSWER)
-            assert.equal(devRerouteCard.border, true)
-            assert.equal(devRerouteCard.header?.padding, true)
-            assert.equal(devRerouteCard.header?.iconForegroundStatus, 'warning')
-            assert.equal(devRerouteCard.header?.icon, MynahIcons.INFO)
-            assert.ok(devRerouteCard.header?.body?.includes('generate code'))
-            assert.ok(devRerouteCard.body?.includes("You don't need to explicitly use /dev"))
-        })
-    })
-
-    describe('createRerouteCard', () => {
-        it('returns testRerouteCard for /test command', () => {
-            const result = createRerouteCard('/test')
-            assert.deepEqual(result, testRerouteCard)
-        })
-
-        it('returns docRerouteCard for /doc command', () => {
-            const result = createRerouteCard('/doc')
-            assert.deepEqual(result, docRerouteCard)
-        })
-
-        it('returns devRerouteCard for /dev command', () => {
-            const result = createRerouteCard('/dev')
-            assert.deepEqual(result, devRerouteCard)
-        })
-
-        it('returns devRerouteCard for unknown command', () => {
-            const result = createRerouteCard('/unknown')
-            assert.deepEqual(result, devRerouteCard)
-        })
-
-        it('returns devRerouteCard for empty string', () => {
-            const result = createRerouteCard('')
-            assert.deepEqual(result, devRerouteCard)
-        })
-    })
 })

chat-client/src/client/texts/pairProgramming.ts

@@ -1,4 +1,4 @@
-import { ChatItem, ChatItemFormItem, ChatItemType, MynahIcons } from '@aws/mynah-ui'
+import { ChatItem, ChatItemFormItem, ChatItemType } from '@aws/mynah-ui'
 
 export const programmerModeCard: ChatItem = {
     type: ChatItemType.ANSWER,
@@ -36,76 +36,3 @@ export const pairProgrammingModeOff: ChatItem = {
     fullWidth: true,
     body: 'Agentic coding - OFF',
 }
-
-export const testRerouteCard: ChatItem = {
-    type: ChatItemType.ANSWER,
-    border: true,
-    header: {
-        padding: true,
-        iconForegroundStatus: 'warning',
-        icon: MynahIcons.INFO,
-        body: 'You can now ask to generate unit tests directly in the chat.',
-    },
-    body: `You don't need to explicitly use /test. We've redirected your request to chat.
-Ask me to do things like:
-• Add unit tests for highlighted functions in my active file
-• Generate tests for null and empty inputs in my project`,
-}
-
-export const docRerouteCard: ChatItem = {
-    type: ChatItemType.ANSWER,
-    border: true,
-    header: {
-        padding: true,
-        iconForegroundStatus: 'warning',
-        icon: MynahIcons.INFO,
-        body: 'You can now ask to generate documentation directly in the chat.',
-    },
-    body: `You don't need to explicitly use /doc. We've redirected your request to chat.`,
-}
-
-export const devRerouteCard: ChatItem = {
-    type: ChatItemType.ANSWER,
-    border: true,
-    header: {
-        padding: true,
-        iconForegroundStatus: 'warning',
-        icon: MynahIcons.INFO,
-        body: 'You can now ask to generate code directly in the chat.',
-    },
-    body: `You don't need to explicitly use /dev. We've redirected your request to chat.
-Ask me to do things like:
-1. Create a project
-2. Add a feature
-3. Modify your files`,
-}
-
-export const reviewRerouteCard: ChatItem = {
-    type: ChatItemType.ANSWER,
-    border: true,
-    header: {
-        padding: true,
-        iconForegroundStatus: 'warning',
-        icon: MynahIcons.INFO,
-        body: 'You can now ask to run code reviews directly in the chat.',
-    },
-    body: `You don't need to explicitly use /review. We've redirected your request to chat.
-Ask me to do things like:
-• Perform a code review of uncommitted changes in my active file
-• Perform a code review of my active file`,
-}
-
-export const createRerouteCard = (command: string): ChatItem => {
-    switch (command) {
-        case '/test':
-            return testRerouteCard
-        case '/doc':
-            return docRerouteCard
-        case '/dev':
-            return devRerouteCard
-        case '/review':
-            return reviewRerouteCard
-        default:
-            return devRerouteCard // Default fallback
-    }
-}