Skip to content

Conversation

@mrubens
Copy link
Collaborator

@mrubens mrubens commented Jun 27, 2025

Let the LLM suggest answers that also switch modes to save you a round trip

Screenshot 2025-06-27 at 3 27 18 PM


Important

This PR adds mode-switching capabilities to follow-up question suggestions, allowing users to switch modes directly from the suggestion, with backend and frontend updates to support this feature.

  • Behavior:
    • Adds optional mode attribute to <suggest> tags in ask_followup_question to switch modes when selected.
    • Mode change is automatic and indicated by a visual badge.
  • Backend:
    • Updates askFollowupQuestionTool in askFollowupQuestionTool.ts to parse and handle mode attributes.
    • Adds tests in askFollowupQuestionTool.spec.ts to verify parsing of suggestions with and without mode attributes.
  • Frontend:
    • Updates FollowUpSuggest.tsx to handle mode attribute, sending mode switch messages via vscode.postMessage.
    • Displays mode badge on suggestions with a mode attribute.

This description was created by Ellipsis for 5cb527c. You can customize this summary. It will automatically update as commits are pushed.

@mrubens mrubens requested review from cte and jr as code owners June 27, 2025 19:28
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. enhancement New feature or request labels Jun 27, 2025
@delve-auditor
Copy link

delve-auditor bot commented Jun 27, 2025

No security or compliance issues detected. Reviewed everything up to 5cb527c.

Security Overview
  • 🔎 Scanned files: 17 changed file(s)
Detected Code Changes
Change Type Relevant files
Enhancement ► askFollowupQuestionTool.ts
    Add optional mode suggestions support
► CustomModesManager.ts
    Update YAML string handling
► FollowUpSuggest.tsx
    Add mode switching UI for suggestions
Configuration changes ► package.json
    Update version from 3.22.2 to 3.22.1

Reply to this PR with @delve-auditor followed by a description of what change you want and we'll auto-submit a change to this PR to implement it.

@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Jun 27, 2025
@mrubens mrubens force-pushed the follow_up_suggestion_modes branch from 5b019c9 to 5cb527c Compare June 27, 2025 19:42
Copy link
Collaborator

@jr jr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I find it hard to predict when the modes will actually use this, but it's nice when they do.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 27, 2025
@mrubens mrubens merged commit 719d661 into main Jun 27, 2025
14 checks passed
@mrubens mrubens deleted the follow_up_suggestion_modes branch June 27, 2025 20:26
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jun 27, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Jun 27, 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 Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants