fix(web): preserve composer draft during planning prompts#809
fix(web): preserve composer draft during planning prompts#809Githubguy132010 wants to merge 1 commit intopingdotgg:mainfrom
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip CodeRabbit can use your project's `biome` configuration to improve the quality of JS/TS/CSS/JSON code reviews.Add a configuration file to your project to customize how CodeRabbit runs |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 3a8dc31399
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
dc725a6 to
12520ca
Compare
12520ca to
0767069
Compare
Fixes #808
Summary
Validation
bun lint(passes, with one pre-existing warning inapps/server/src/provider/Layers/ProviderService.ts:295)bun typecheck(fails due to pre-existing unrelated issues inapps/web/src/components/Sidebar.tsxfor missing@dnd-kit/*modules and one implicitany)Note
Preserve composer draft while answering planning prompts in ChatView
ComposerPendingUserInputCardcomponent where users answer planning questions in-place, with number-key shortcuts (1–9), auto-advance, and a per-cardComposerPromptEditor.ComposerPendingUserInputPanelto host the card and wire option selection and custom-answer callbacks independently from the main composer.activePendingProgress.customAnswer, and planning-specific placeholder text is removed from the composer.Macroscope summarized 0767069.