Skip to content

fix(vscode): preserve multi-question answers#1918

Draft
hogeheer499-commits wants to merge 1 commit into
MoonshotAI:mainfrom
hogeheer499-commits:agent/fix-vscode-multi-question-1880
Draft

fix(vscode): preserve multi-question answers#1918
hogeheer499-commits wants to merge 1 commit into
MoonshotAI:mainfrom
hogeheer499-commits:agent/fix-vscode-multi-question-1880

Conversation

@hogeheer499-commits

@hogeheer499-commits hogeheer499-commits commented Jul 19, 2026

Copy link
Copy Markdown

Related Issue

Resolves #1880

The VS Code dialog submitted the answer to questions[0] immediately, so any remaining questions in the same AskUserQuestion request were dropped.

The dialog now walks through the questions in order and submits once, after the last answer. Selection and custom-input state are reset between questions, and multi-question prompts show the current position.

I added a jsdom regression test covering two questions and verified that nothing is submitted after the first answer. The full VS Code suite passes (289 tests), along with both typechecks and the production webview build. Oxlint reports no errors; its four shorthand-arrow warnings were already present.

There is no changeset because the VS Code extension is private rather than part of the published @moonshot-ai/kimi-code package. This restores existing behavior, so no documentation change is needed.

Checklist

  • I have read the CONTRIBUTING document.
  • I have linked a related issue, or explained the problem above.
  • I have added tests that prove my feature works.
  • Ran gen-changesets skill, or this PR needs no changeset.
  • Ran gen-docs skill, or this PR needs no doc update.

@changeset-bot

changeset-bot Bot commented Jul 19, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: e81d586

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@hogeheer499-commits
hogeheer499-commits force-pushed the agent/fix-vscode-multi-question-1880 branch from 90a7ce2 to e81d586 Compare July 19, 2026 11:38
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.

VS Code: AskUserQuestion with multiple questions renders only the first, remaining questions silently dropped

1 participant