Skip to content

bug: Dialog default value displays message instead of defaultValue #361

@mithun50

Description

@mithun50

Description of the bug

Bug Description

In McpResponse.ts, the dialog information display shows
dialog.message() twice instead of showing the actual
default value.

Location

File: src/McpResponse.ts
Line: 196

Current Behavior

${dialog.type()}: ${dialog.message()} (default value:
${dialog.message()})

This displays the dialog message as both the message AND the
 default value, which is incorrect.

Expected Behavior

${dialog.type()}: ${dialog.message()} (default value:
${dialog.defaultValue()})

Should use dialog.defaultValue() to display the actual
default value of the dialog.

Example

Current output:
prompt: Enter your name (default value: Enter your name)

Expected output:
prompt: Enter your name (default value: John Doe)

Impact

  • Severity: Minor
  • User Impact: Confusing UX - users see incorrect
    information about dialog default values
  • Functionality: Does not break functionality, but displays
    misleading information

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions