Skip to content

fix(server): omit empty AskUserQuestion question in thread snapshot - #50

Closed
macodev00 wants to merge 42 commits into
mainfrom
cursor/askuserquestion-omit-empty-question-75a9
Closed

macodev00 wants to merge 42 commits into
mainfrom
cursor/askuserquestion-omit-empty-question-75a9

Conversation

@macodev00

@macodev00 macodev00 commented Sep 26, 2026 •

Copy link
Copy Markdown
Owner

What Changed

projectQuestionToolInput omits question when an AskUserQuestion entry has no question text.

Why

Entries with header and options but no question (including a streamed questions: [{}]) were projected as { question: undefined }. Schema.Unknown rejected that payload, so GET /api/orchestration/threads/:id and the WebSocket snapshot returned an empty HTTP 400 and the thread never loaded.

Clients already read record(question)?.question, so a missing key is safe. No database rewrite is required.

The branch is one commit ahead of pingdotgg/t3code main (a21b42ce). This fork's main is behind that, so the compare here also lists commits that are already on upstream.

Fixes pingdotgg#13762

Checklist

  • This PR is small and focused
  • I explained what changed and why
Open in Web Open in Cursor 

aaditagrawal and others added 30 commits September 25, 2026 16:04
Co-authored-by: shivam <91240327+shivamhwp@users.noreply.github.com>
Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
)

Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: shivam <91240327+shivamhwp@users.noreply.github.com>
Co-authored-by: shivam <91240327+shivamhwp@users.noreply.github.com>
…rotocol variables (pingdotgg#13492)

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…and WSL backends (pingdotgg#13641)

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
Co-authored-by: Julius Marminge <51714798+juliusmarminge@users.noreply.github.com>
…gg#13705)

Co-authored-by: Julius Marminge <julius0216@outlook.com>
Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…13701)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…13685)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…gdotgg#13684)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…gdotgg#13698)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…equests (pingdotgg#13704)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…gg#13694)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…ingdotgg#13688)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
pingdotgg#13693)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…oading it twice (pingdotgg#13683)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
… in memory (pingdotgg#13686)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…he whole thread list (pingdotgg#13691)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
t3dotgg and others added 12 commits September 25, 2026 20:38
…ngdotgg#13697)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…dotgg#13689)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
pingdotgg#13669)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…irectory (pingdotgg#13695)

Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
AskUserQuestion entries that have header and options but no question text were projected as { question: undefined }. Schema.Unknown rejects that value, so the thread snapshot returned an empty HTTP 400.

Leave the question key out when there is no text. Clients already read it with record(question)?.question.

Fixes pingdotgg#13762
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XXL labels Sep 26, 2026
@macodev00 macodev00 closed this Sep 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XXL vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(server): thread snapshot returns empty HTTP 400 when an AskUserQuestion entry has no question text