Skip to content

Commit c3ecda6

Browse files
fix(chat-client): fix the warning icon (#1126)
## Problem We were using emoji warning icon, which might cause act differently for different users ## Solution Use the icon provided by flare
1 parent 5e091d7 commit c3ecda6

File tree

3 files changed

+12
-8
lines changed

3 files changed

+12
-8
lines changed

chat-client/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
"dependencies": {
2424
"@aws/chat-client-ui-types": "^0.1.28",
2525
"@aws/language-server-runtimes-types": "^0.1.25",
26-
"@aws/mynah-ui": "^4.31.0-beta.7"
26+
"@aws/mynah-ui": "^4.31.0-beta.8"
2727
},
2828
"devDependencies": {
2929
"@types/jsdom": "^21.1.6",

package-lock.json

Lines changed: 4 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

server/aws-lsp-codewhisperer/src/language-server/agenticChat/agenticChatController.ts

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -836,7 +836,7 @@ export class AgenticChatController implements ChatHandlers {
836836
toolType?: string
837837
): ChatResult {
838838
let buttons: Button[] = []
839-
let header: { body: string; buttons: Button[] }
839+
let header: { body: string; buttons: Button[]; icon?: string; iconForegroundStatus?: string }
840840
let body: string
841841

842842
switch (toolType || toolUse.name) {
@@ -873,7 +873,9 @@ export class AgenticChatController implements ChatHandlers {
873873
},
874874
]
875875
header = {
876-
body: '#### ⚠️ Allow file modification outside of your workspace',
876+
icon: 'warning',
877+
iconForegroundStatus: 'warning',
878+
body: '#### Allow file modification outside of your workspace',
877879
buttons,
878880
}
879881
const writeFilePath = (toolUse.input as unknown as FsWriteParams).path
@@ -892,7 +894,9 @@ export class AgenticChatController implements ChatHandlers {
892894
},
893895
]
894896
header = {
895-
body: '#### ⚠️ Allow read-only tools outside your workspace',
897+
icon: 'warning',
898+
iconForegroundStatus: 'warning',
899+
body: '#### Allow read-only tools outside your workspace',
896900
buttons,
897901
}
898902
// ⚠️ Warning: This accesses files outside the workspace

0 commit comments

Comments
 (0)