Skip to content

feat: Render question text and options with markdown#339

Merged
tiann merged 2 commits intotiann:mainfrom
dhbloo:feature/question-markdown-rendering
Mar 22, 2026
Merged

feat: Render question text and options with markdown#339
tiann merged 2 commits intotiann:mainfrom
dhbloo:feature/question-markdown-rendering

Conversation

@dhbloo
Copy link
Contributor

@dhbloo dhbloo commented Mar 22, 2026

Summary

Render markdown in CLI-started question UIs so question prompts and option content display consistently with the main session output.

This updates both question tool variants:

  • AskUserQuestion / ask_user_question
  • request_user_input

And it covers both states:

  • pending interaction footers
  • read-only/history views

Changes

  • render question body text with the existing shared markdown renderer
  • render option labels with the existing shared markdown renderer
  • render option descriptions with the existing shared markdown renderer
  • keep option values, answer matching, and selection logic unchanged

Before

before

After

after

Scope

Intentionally narrow patch:

  • web-only
  • no hub / CLI / shared schema changes
  • no transport changes
  • no shared markdown renderer API changes

Testing

  • bun run typecheck:web

Notes

  • This reuses the app's existing markdown stack, so behavior matches current frontend markdown support.
  • Unrelated local bun.lock changes were not included in this branch.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Findings

  • None.

Summary

  • Review mode: initial. No issues found in modified lines. Residual risk: UI-only markdown rendering change not covered by automated web tests.

Testing

  • Not run (automation).

HAPI Bot

@tiann tiann merged commit bcc2558 into tiann:main Mar 22, 2026
2 checks passed
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.

2 participants