Skip to content

fix(prompt): drop the unsupported default option from the text prompt - #299

Open
TPSJASPER wants to merge 1 commit into
MiniMax-AI:mainfrom
TPSJASPER:fix/text-prompt-default-option
Open

TPSJASPER wants to merge 1 commit into
MiniMax-AI:mainfrom
TPSJASPER:fix/text-prompt-default-option

Conversation

@TPSJASPER

@TPSJASPER TPSJASPER commented Oct 21, 2026 •

Copy link
Copy Markdown

promptText() passed the same string as both default and placeholder to @clack/prompts. Clack's text prompt has no default option — its option is defaultValue — so that key was ignored and the hint could only ever be rendered as the placeholder.

This builds the options in a small helper, keeps the default strictly as a placeholder, and adds unit tests for both cases so a hint can never be seeded into the prompt as input text.

Refs #278: the placeholder text in that report (Ask Mcode to do anything) is drawn by the MiniMax Code TUI composer in MiniMax-AI/minimax-code, not by this prompt wrapper. This change removes the only place in this repository where a hint could be passed to a prompt as a value.

Tests

  • bun test test/utils/prompt.test.ts — 3 pass
  • bun run typecheck
  • bun run lint

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant