Merged
Conversation
…ancel_resume_bug fix: Preserve parent-child relationship when cancelling subtasks
…ined-exit-code refactor terminal architecture to address critical issues with the current design
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
PowerShell requires special handling for command output due to two issues: - A sleep delay is required to prevent the ]633;D marker from losing the original output - A counter is needed to work around a bug where identical commands are not executed Changes: - Add cmdCounter to Terminal class for unique command tracking - Add PowerShell detection via platform and default shell profile - Add sleep delay to ensure output is captured before command completion Signed-off-by: Eric Wheeler <roo-code@z.ewheeler.org>
Mock VSCode workspace configuration to handle PowerShell detection in terminal tests. This prevents TypeError when accessing getConfiguration() in TerminalProcess.run() Signed-off-by: Eric Wheeler <roo-code@z.ewheeler.org>
Add VSCode workspace configuration mock to TerminalProcess.test.ts to handle PowerShell detection in terminal tests, matching the fix in TerminalProcessExec.test.ts Signed-off-by: Eric Wheeler <roo-code@z.ewheeler.org>
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
…lock-text-color Fix MarkdownBlock text color for Dark High Contrast theme
Fix terminal process unit tests on MacOS
…ixes ContextProxy fix - constructor should not be async
…edrock Add Deepseek R1 On-Demand to Bedrock
feat: Add prompt caching to OpenAI-compatible custom models
…mpatible-deepseek-reasoning-support feat: openai-compatible deepseek/qwq reasoning support
…-dup-command-issue
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
More chat i18n
…etion Fix old task deletion bug
Fix infinite loop that happens when using direnv
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
Localize formatting helpers
…ontent contain html code (RooCodeInc#1662) * fix: an issue in the HistoryView component where keywords in copied content contain html code * Update webview-ui/src/components/history/HistoryView.tsx * Fix test --------- Co-authored-by: Matt Rubens <mrubens@users.noreply.github.com>
* Choose specific provider when using OpenRouter * Add translations
* Add text-to-speech functionality * Add speed config option to text-to-speech * Fix test case for tts speed slider * Fix test case for tts speed slider (really) * Disabled error message logging in tts.ts * ignore markdown and mermaid diagrams in TTS * add ttsEnabled and ttsSpeed to GlobalStateKey * fix failing webview test for save button * Translations * Fix tests --------- Co-authored-by: Matt Rubens <mrubens@users.noreply.github.com>
* fix: Add special handling to human-relay providers, optimize configuration check logic * fix: Remove the debug log in the checkExistKey function
feat: add multiple workspaces support - Add getWorkspacePath function to centralize workspace directory path retrieval - Use the new workspace directory retrieval logic in Cline, Mentions, ClineProvider, and WorkspaceTracker - Update WorkspaceFile on tab switch and prevent redundant updates by checking prevWorkSpacePath - Fix the bug that loads the contents of the previous tab when quickly switching tabs - Optimize getWorkspacePath return value for better reliability Co-authored-by: xiong <yueminxiong.xym@alibaba-inc.com>
docs: update contributors list [skip ci] Co-authored-by: mrubens <mrubens@users.noreply.github.com>
…ooCodeInc#1660) * fix: clarify PowerShell command completion workaround The command completion detection approach in PowerShell requires an output string to allow duplicate commands to execute in some versions of code. Update the string to explicitly indicate it is a Roo PowerShell workaround, making it clear in terminal output that this is intentional behavior rather than a side effect. Signed-off-by: Eric Wheeler <roo-code@z.ewheeler.org> * cleanup: improve terminal logging and error handling No functional changes - purely improves error handling and logging clarity. Terminal.ts: - Handle undefined process state in setActiveStream without throwing - Add terminal IDs to all log messages for better traceability - Improve error message clarity in shell integration timeout TerminalRegistry.ts: - Reorganize shell execution event handlers for better flow - Log shell execution events before processing for reliable debugging - Add detailed context to terminal not found scenarios - Include command and execution state in error messages Signed-off-by: Eric Wheeler <roo-code@z.ewheeler.org> * feat: make terminal shell integration timeout configurable Users with long shell startup times were encountering "Shell Integration Unavailable" errors due to the hard-coded 4s timeout. The timeout is now configurable through Advanced Settings (1-60s). Thanks @FILTHY for troubleshooting and @kiwina for suggesting making the timeout configurable. Fixes RooCodeInc#1654 Signed-off-by: Eric Wheeler <roo-code@z.ewheeler.org> * critical fix: race condition that prevents command completion Terminal running state is now managed in TerminalRegistry instead of Terminal to prevent race between stream close and shell completion. While this race may not trigger on current VSCode versions, newer releases with additional terminal fixes may expose the issue. This proactively prevents "Shell execution end event received, but process is not running" errors. Signed-off-by: Eric Wheeler <roo-code@z.ewheeler.org> * fix: improve command execution path reporting Enhance clarity of command execution context and error reporting: - Check to see if the directory changed because of the command - Clarify execution path message - Add explicit message when command exits with non-zero code Signed-off-by: Eric Wheeler <roo-code@z.ewheeler.org> * system instructions: clarify terminal directory operations Clear guidance for the AI system on: - Working directory constraints - Path handling requirements - Tool vs terminal directory behavior Signed-off-by: Eric Wheeler <roo-code@z.ewheeler.org> * test: update snapshots for system prompt working directory instructions Signed-off-by: Eric Wheeler <roo-code@z.ewheeler.org> --------- Signed-off-by: Eric Wheeler <roo-code@z.ewheeler.org> Co-authored-by: Eric Wheeler <roo-code@z.ewheeler.org> Co-authored-by: Chris Estreich <cestreich@gmail.com>
* batch clear history * catch Task not found error * fix deleteMultipleTasksWithIds * add i18n for batch clear history * add i18n for history * debug * remove unused logs * fix trans * fix trans * Cleanup --------- Co-authored-by: aheizi <aheiz@outlook.com> Co-authored-by: Matt Rubens <mrubens@users.noreply.github.com>
* feat(i18n): Add i18n support for Human Relay dialog without monitoring * feat(i18n): add humanRelay.json translations for all supported languages - Created English version as base translation - Added translations for: ca, de, es, fr, hi, it, ja, ko, pl, pt-BR, tr, vi, zh-TW - Verified all translations are complete with no missing entries
* Extension-side internationalization * Update script * Cleaner esbuild * Turn off debugging * PR feedback * Update src/i18n/locales/en/common.json Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com> * Update src/i18n/locales/ca/common.json Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com> --------- Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
* Allow webviews to make OpenRouter API calls * Update package-lock.json
Jdo300
pushed a commit
that referenced
this pull request
Mar 18, 2025
Syncing sbc-roo-code main branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
19 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Type of change
How Has This Been Tested?
Checklist:
Additional context
Related Issues
Reviewers