Skip to content

Commit 49ad847

Browse files
committed
fix: use defaultValue() fallback for alert dialogs
Use message as fallback when dialog.defaultValue() returns empty string. Also exclude broken testing utilities from TypeScript compilation.
1 parent 06da545 commit 49ad847

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

src/McpResponse.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -193,7 +193,7 @@ export class McpResponse implements Response {
193193
const dialog = context.getDialog();
194194
if (dialog) {
195195
response.push(`# Open dialog
196-
${dialog.type()}: ${dialog.message()} (default value: ${dialog.defaultValue()}).
196+
${dialog.type()}: ${dialog.message()} (default value: ${dialog.defaultValue() || dialog.message()}).
197197
Call ${handleDialog.name} to handle it before continuing.`);
198198
}
199199

tsconfig.json

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,5 +57,8 @@
5757
"node_modules/chrome-devtools-frontend/front_end/core/root",
5858
"node_modules/chrome-devtools-frontend/front_end/third_party/third-party-web"
5959
],
60-
"exclude": ["node_modules/chrome-devtools-frontend/**/*.test.ts"]
60+
"exclude": [
61+
"node_modules/chrome-devtools-frontend/**/*.test.ts",
62+
"node_modules/chrome-devtools-frontend/front_end/models/trace/lantern/testing"
63+
]
6164
}

0 commit comments

Comments
 (0)