-
Notifications
You must be signed in to change notification settings - Fork 2.8k
feat: add git status to environment details #9310
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
- Add getGitStatus() function that returns raw git status output - Truncate git status to 20 lines to prevent context bloat - Only show section headers (Visible Files, Open Tabs) when content exists - Add comprehensive tests for git status functionality - All tests passing (43 git tests, 15 environment tests)
Re-reviewed changes since last review. No new issues found. The latest commit adds configurable git status with proper defaults (disabled by default, maxGitStatusFiles=0). The implementation correctly handles the feature flag, passes the maxFiles parameter through the call chain, and includes comprehensive test coverage for all scenarios. Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues. |
- Add maxGitStatusFiles setting (0=disabled by default) - Implement getGitStatus() with configurable file limit - Add slider UI control in Context Management settings - Positioned after workspace files slider, before concurrent reads - Include localization for all 18 languages - Branch info always shown when enabled (maxFiles > 0) - Comprehensive test coverage (44 git tests, 20 env tests)
* fix: change Add to Context keybinding to avoid Redo conflict (RooCodeInc#8653) Co-authored-by: Roo Code <[email protected]> * feat: add Google Ads conversion tracking to reviewer page (RooCodeInc#8831) * feat: add Google Ads conversion tracking to reviewer page * fix: add asChild prop to first button to prevent invalid HTML nesting --------- Co-authored-by: Roo Code <[email protected]> Co-authored-by: daniel-lxs <[email protected]> * Fix provider model loading race conditions (RooCodeInc#8836) * Release v3.29.1 (RooCodeInc#8854) chore: add changeset for v3.29.1 * Changeset version bump (RooCodeInc#8855) * changeset version bump * Update CHANGELOG for version 3.29.1 release Updated version number and added release notes for 3.29.1. --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Matt Rubens <[email protected]> * Merge remote-tracking branch 'upstream/main' into roo-to-main * Fix caching logic in Roo provider (
Adds a new maxGitStatusFiles setting to optionally include git repository status in the environment details section of the system prompt.
Key Features:
## main...origin/main)Implementation:
getGitStatus(cwd, maxFiles)utility insrc/utils/git.tsgetEnvironmentDetails()with conditional renderingBehavior:
0= Disabled (no git status included) - default1-50= Enabled, showing branch header + up to N file entries