-
Notifications
You must be signed in to change notification settings - Fork 42
fix(planner): add clarification questions before plan generation #1254
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…tion Allow the planner agent to ask clarification questions before generating implementation plans. This improves plan quality by ensuring ambiguities are resolved with user input rather than assumptions. Changes: - Add askFollowupQuestion to planner's available tools - Update planner workflow with clarification phase and ambiguity gate - Ensure planner agent always runs in manual mode for user interaction - Fix typo: askFollowupQuestions → askFollowupQuestion in prompts 🤖 Generated with [Pochi](https://getpochi.com) | [Task](https://app.getpochi.com/share/p-5029f743fae34ddab41d50fedc543a10) Co-Authored-By: Pochi <noreply@getpochi.com>
Codecov Report❌ Patch coverage is ❌ Your patch status has failed because the patch coverage (62.50%) is below the target coverage (75.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #1254 +/- ##
==========================================
+ Coverage 13.61% 13.65% +0.04%
==========================================
Files 671 673 +2
Lines 59721 59958 +237
Branches 1749 1761 +12
==========================================
+ Hits 8130 8189 +59
- Misses 51320 51497 +177
- Partials 271 272 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
wsxiaoys
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
otherwise LG
|
Please share a few sessions / screenshots to show a few examples? |
…vent UI display in subtasks
…or improved state handling
… directly check last part type
* feat(planner): enable askFollowupQuestion for planner agent clarification Allow the planner agent to ask clarification questions before generating implementation plans. This improves plan quality by ensuring ambiguities are resolved with user input rather than assumptions. Changes: - Add askFollowupQuestion to planner's available tools - Update planner workflow with clarification phase and ambiguity gate - Ensure planner agent always runs in manual mode for user interaction - Fix typo: askFollowupQuestions → askFollowupQuestion in prompts 🤖 Generated with [Pochi](https://getpochi.com) | [Task](https://app.getpochi.com/share/p-5029f743fae34ddab41d50fedc543a10) Co-Authored-By: Pochi <noreply@getpochi.com> * fix(vscode-webui): implement filterTrailingAskFollowupQuestion to prevent UI display in subtasks * fix(tools): add isLastPart prop to ToolProps and related components for improved state handling * fix(task-thread): simplify filterTrailingAskFollowupQuestion logic to directly check last part type --------- Co-authored-by: Pochi <noreply@getpochi.com>
Screen record
https://jam.dev/c/d13db240-0509-44b3-b547-faadbdcd4717
Summary
askFollowupQuestiontool before generating implementation plansaskFollowupQuestions→askFollowupQuestionin promptsTest plan
🤖 Generated with Pochi | Task