Add git repo telemetry properties #4594
Closed
Conversation
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
hannesrudolph
added
the
Issue/PR - Triage
hannesrudolph
added
PR - Draft / In Progress
and removed
Issue/PR - Triage
* feat: Add DeepSeek R1 support to Chutes provider (#4523) - Modified BaseOpenAiCompatibleProvider to expose client as protected - Enhanced ChutesHandler to detect DeepSeek R1 models and parse reasoning chunks - Applied R1 format conversion for message formatting - Set appropriate temperature (0.6) for DeepSeek models - Migrated tests from Jest to Vitest format - Added comprehensive tests for DeepSeek R1 functionality This ensures reasoning chunks are properly separated from regular content when using DeepSeek R1 models via Chutes provider. * feat: Enhance DeepSeek R1 support with <think> tag handling in Chutes provider * fix: Correct temperature retrieval in ChutesHandler to use model's info * fix: Update condition for DeepSeek-R1 model identification in createMessage method --------- Co-authored-by: Daniel Riccio <[email protected]>
… (#4450) * feat: add prompt history navigation with arrow keys (#4139) - Navigate through prompt history using arrow up/down keys - Only triggers when cursor is at first line (up) or last line (down) - Preserves current input when starting navigation - Resets navigation state when typing or sending messages - Follows VSCode's standard UX patterns for history navigation * fix: correct prompt history order and add workspace filtering (#4139) - Remove reverse() to maintain chronological order in history array - Add workspace filtering to only show prompts from current workspace - Ensure arrow up navigates to older prompts (as expected) - Filter history items by workspace field matching current cwd * test: Fix Windows unit test failures for prompt history navigation - Add missing taskHistory and cwd properties to all useExtensionState mocks - Add comprehensive test coverage for prompt history navigation feature - Ensure all 25 tests pass including new prompt history functionality Fixes failing Windows CI test in PR #4450 * refactor: Improve cursor positioning with useLayoutEffect - Replace setTimeout(..., 0) with useLayoutEffect for more reliable cursor positioning - Implement state-based cursor positioning pattern suggested by @mochiya98 - Add CursorPositionState interface for better type safety - Maintain all existing functionality while improving timing reliability This addresses the technical suggestion in PR #4450 comment about using useLayoutEffect instead of setTimeout for DOM manipulation timing. * feat: optimize prompt history with performance improvements and memory management - Add useMemo for prompt history filtering to prevent unnecessary re-computations - Implement MAX_PROMPT_HISTORY_SIZE = 100 limit for memory management - Extract logic into usePromptHistory custom hook for better code organization - Simplify ChatTextArea component by delegating history logic to custom hook Addresses review feedback on PR #4450 for issue #4139 * refactor: clean up unused code and fix linting issues in prompt history - Remove unused CursorPositionState interface from ChatTextArea - Remove unused destructured variables from usePromptHistory hook - Fix missing dependency in useEffect dependency array - Rename unused parameter with underscore prefix Related to #4139 * feat: implement hybrid prompt history with position reset - In chat: Use conversation messages (user_feedback), newest first - Out of chat: Use task history, oldest first - Reset navigation position when switching between history sources - Switch from taskHistory to clineMessages for active conversations - Maintain backward compatibility with task history fallback - Add comprehensive tests for hybrid behavior and position reset This provides intuitive UX where: - Users navigate recent conversation messages during tasks (newest first) - Users access initial task prompts when starting fresh (oldest first) - Navigation always starts fresh when switching contexts * fix: correct task history slicing order for prompt navigation Task history was using .slice(-100) which gets the newest 100 tasks, but we want to show oldest tasks first when navigating. Changed to .slice(0, 100) to get the oldest 100 tasks instead. This ensures that when starting fresh (no conversation), up arrow shows the oldest task prompts first, which is the intended behavior. * refactor: remove comment on task history size limitation and clarify order preservation * refactor: replace local ClineMessage and TaskHistoryItem interfaces with imported types * fix: prevent prompt history fallback to task list during active conversation When an active task has only an initial prompt with no follow-up user messages, the prompt history should return empty instead of falling back to task history. This fixes the "Starting Fresh" behavior appearing inappropriately. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]> --------- Co-authored-by: Daniel Riccio <[email protected]> Co-authored-by: Claude <[email protected]>
docs: add comprehensive JSDoc documentation for ClineAsk and ClineSay types
* allow escaping of context mentions * refactor: update comment --------- Co-authored-by: cannuri <[email protected]> Co-authored-by: Daniel <[email protected]>
* feat(tests): add apply_diff tool tests * feat(tests): add tests for write_to_file tool functionality * feat(tests): add comprehensive tests for read_file tool functionality * feat(tests): add tests for execute_command tool functionality * feat(integration-tester): add integration testing role with comprehensive guidelines * feat(tests): enhance test runner with grep and specific file filtering * feat(tests): add comprehensive tests for search_files tool functionality * feat(tests): add comprehensive tests for list_files tool functionality * feat(tests): add tests for insert_content tool functionality * feat(tests): add comprehensive tests for search_and_replace tool functionality * feat(tests): add comprehensive tests for use_mcp_tool functionality * feat(tests): increase timeout values for various tool tests to improve reliability * fix(tests): add non-null assertion for workspaceDir assignment in multiple test files * feat(tests): enhance read_file tool tests with increased timeouts and improved prompts * feat(tests): enhance read_file tool tests to extract and verify tool results * feat(tests): enhance execute_command tool tests with additional context in prompts * refactor(tests): remove script execution test and related setup for execute_command tool * fix(tests): increase timeout for task start and completion in apply_diff and read_file tests * fix(tests): clarify error handling message in command execution test * refactor(tests): remove error handling test and related setup for execute_command tool * fix: update openRouterModelId to use anthropic/claude-3.5-sonnet * fix: update openRouterModelId to use openai/gpt-4.1 * fix(tests): increase timeouts for apply_diff, execute_command, and search_and_replace tests * fix(tests): disable terminal shell integration for execute_command tool tests * chore: rewrite integration tester mode * Update .roo/rules-integration-tester/1_workflow.xml Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com> --------- Co-authored-by: Daniel Riccio <[email protected]> Co-authored-by: Daniel <[email protected]> Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
* update xai models and pricing * cache accounting for xAI * change log
…s & batch UI (#3342) * feat: add BatchDiffApproval component for multi-file diff application - Introduced a new component `BatchDiffApproval` to handle the approval of batch changes across multiple files. - Integrated the `BatchDiffApproval` component into `ChatRow` to display batch diff requests. - Updated experimental settings to include a toggle for multi-file apply diff functionality. - Enhanced localization files to support new strings related to batch changes in multiple languages. - Updated tests to cover the new multi-file apply diff feature. * revert this * fix: update applyDiff parameter type to accept string or DiffItem * refactor: keep original file name for apply diff tool * revert this * Update src/core/webview/__tests__/ClineProvider.test.ts * revert this * fix: keep the original path if the experiment is disabled * test: add dynamic strategy selection tests for MultiSearchReplaceDiffStrategy and MultiFileSearchReplaceDiffStrategy * fix: mock applyDiffTool module and ensure legacy tool resolves successfully in tests * remove this * ellipsis suggestion Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com> * refactor: mirror concurrent file reads --------- Co-authored-by: Daniel Riccio <[email protected]> Co-authored-by: Daniel <[email protected]> Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
…nking (#4201) (#4481) * Add reasoning budget support to Bedrock models and update related components - Introduced `supportsReasoningBudget` property in Bedrock models. - Enhanced `AwsBedrockHandler` to handle reasoning budget in payloads. - Updated `ThinkingBudget` component to dynamically set max tokens based on reasoning support. - Modified `ApiOptions` and `Bedrock` components to conditionally render `ThinkingBudget`. - Added tests for extended thinking functionality in `bedrock-reasoning.test.ts`. * Add BedrockThinkingConfig interface and update payload structure * fix: address PR review feedback (#4481) - Simplify ThinkingBudget ternary logic since component only renders when reasoning budget supported - Break down complex thinking enabled condition with clear documentation - Replace 'as any' usage with proper TypeScript interfaces for AWS SDK events - Add comprehensive documentation for multiple stream structures explaining AWS SDK compatibility
* add mermaid buttons * feat: Add Modal, TabButton, and ZoomControls components * feat: Add error handling messages for image operations and file opening * mermaid: Add drag functionality and support contious zooming * add active color for tabbutton * refactor zoom controls * refactor: Remove unused svgToPng prop and simplify handleCopy function * Move zoom to constants and increase max zoom * feat: add save image functionality and refactor image handling * feat: add translations --------- Co-authored-by: Daniel Riccio <[email protected]> Co-authored-by: Matt Rubens <[email protected]>
* Manually specify openai-compat format and parse it * fixup! Manually specify openai-compat format and parse it * Expect base64 in embedding test arguments * fixup! Manually specify openai-compat format and parse it Remove debug logs * fixup! Manually specify openai-compat format and parse it Improve comment * fixup! Manually specify openai-compat format and parse it * Add tests to exercise base64 decode of embeddings * Add tests to verify openai base64 and brokenness behavior * feat: improve typing * refactor: switch from jest to vitest for mocking in tests --------- Co-authored-by: Dixie Flatline <dflatline> Co-authored-by: Daniel Riccio <[email protected]>
…kspaceFolder`) (#4442) * feat: add `injectVariables()`, support magic variables for MCPs * fix: fallback for `workspaceFolder` should just be an empty string Previously this is intended so that the CLI receives a correct empty path argument, but on a second thought, if the user have added the quotes themselves it might cause error. Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com> * chore: remove unused import * chore: better log format * chore: better describe the accepted config type and more extensive test --------- Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
* Move concurrent reads into context settings and default to 5 * Spacing tweaks * Change max back to 100
feat: enhance file handling in `apply_diff` case to support legacy and multi-file formats
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Matt Rubens <[email protected]>
) * Added Enable Roo Code quick fixes setting to disable quick fixes. * Fix: Address PR review comments and failing tests for #4878 * Fix localization consistency and add test coverage - Add missing period to French translation for consistency - Add test case for disabled enableCodeActions setting - Fix existing test mock setup for proper configuration handling --------- Co-authored-by: hannesrudolph <[email protected]> Co-authored-by: RooCode <[email protected]>
) * Fix: Allow write_to_file to handle newline-only and empty content * fix: update writeToFileTool to return early without error on missing or empty parameters * fix: preserve newlines in content parameters and update error handling in writeToFileTool tests * fix: update parseAssistantMessage and parseAssistantMessageV2 to preserve newlines in content parameters while stripping leading and trailing newlines --------- Co-authored-by: Daniel Riccio <[email protected]>
* memory cleaanup * forgot to run linter
* fix: address multiple memory leaks in CodeBlock component (CodeBlock_247, CodeBlock_459, CodeBlock_694) * fix: Address review feedback for CodeBlock memory leak fixes - Consolidate isMountedRef management into syntax highlighting useEffect - Unify timeout cleanup patterns for consistent maintainability - Maintain all existing memory leak protections - Improve code organization and readability Addresses review comments from daniel-lxs in PR #4244 --------- Co-authored-by: Daniel Riccio <[email protected]>
Co-authored-by: kiwina <[email protected]>
…exts (#4745) (#4748) Co-authored-by: Daniel Riccio <[email protected]>
…o directories (#5016) Co-authored-by: Daniel Riccio <[email protected]>
Co-authored-by: Bruno Bergher <[email protected]> Co-authored-by: Matt Rubens <[email protected]> Co-authored-by: Daniel Riccio <[email protected]>
liwilliam2021
added
PR - Needs Preliminary Review
and removed
PR - Draft / In Progress
labels
hannesrudolph
added
PR - Draft / In Progress
and removed
PR - Needs Preliminary Review
labels
Contributor
ContextThis PR implements git utility functions to collect repository information for telemetry purposes, providing valuable context about which git repositories are being used with the extension for cloud clients. ImplementationGit Utility Functions and Telemetry System RefactoringImplements getGitRepositoryInfo function to extract repository URL, name, and default branch TestingAdded comprehensive tests in packages/cloud/src/tests/TelemetryClient.test.ts to verify git property handling |
Collaborator
Author
|
Closing this in favor of #5119 |
github-project-automation
bot
moved this from PR [Draft / In Progress]
to Done
in Roo Code Roadmap
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.