Skip to content

Conversation

@samhvw8
Copy link
Contributor

@samhvw8 samhvw8 commented Mar 18, 2025

Context

Suggest at ask follow up question

Implementation

Screenshots

image image

How to Test

Get in Touch


Important

Introduces follow-up question suggestions with XML parsing and UI components for handling and displaying suggestions.

  • Behavior:
    • Adds follow-up question suggestions in Cline.ts using parseXml() from xml.ts.
    • Updates ask_followup_question to include follow_up parameter with suggestions.
    • Handles follow-up suggestions in ChatRow.tsx and ChatView.tsx.
  • UI Components:
    • Adds FollowUpSuggest and FollowUpSuggestionsWrapper components for displaying suggestions.
    • Updates button and badge styles in button.tsx and badge.tsx.
  • Utilities:
    • Introduces parseXml() in xml.ts for XML parsing.
  • Misc:
    • Adds fast-xml-parser to package.json dependencies.

This description was created by Ellipsis for 74c4cadc361aeb5075af211970cd411635b43f05. It will automatically update as commits are pushed.

@changeset-bot
Copy link

changeset-bot bot commented Mar 18, 2025

⚠️ No Changeset found

Latest commit: 4264377

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

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Mar 18, 2025
@dosubot dosubot bot added the Enhancement New feature or request label Mar 18, 2025
@samhvw8
Copy link
Contributor Author

samhvw8 commented Mar 18, 2025

need optimize system prompt :(

@samhvw8 samhvw8 force-pushed the feat/follow-up-suggest branch 2 times, most recently from bc0cee7 to e75cf78 Compare March 18, 2025 19:00
@samhvw8 samhvw8 force-pushed the feat/follow-up-suggest branch from e75cf78 to 73868f3 Compare March 19, 2025 03:59
@mrubens mrubens force-pushed the feat/follow-up-suggest branch 7 times, most recently from d0dada1 to 25a8ec0 Compare March 20, 2025 14:33
@mrubens mrubens force-pushed the feat/follow-up-suggest branch from 25a8ec0 to 4264377 Compare March 20, 2025 14:39
@mrubens
Copy link
Collaborator

mrubens commented Mar 20, 2025

I updated the UX to look like this

Screenshot 2025-03-20 at 10 41 02 AM

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Mar 20, 2025
@mrubens mrubens merged commit f1efeca into RooCodeInc:main Mar 20, 2025
10 checks passed
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Mar 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement New feature or request lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants