Files Changed Overview (FCO) #5626

playcations · 2025-07-12T14:41:02Z

Related GitHub Issue

Closes: # 4454

Description

This adds a collapsable window in chat to show files that have been modified by agents.

Features include accepting changes, diffing and rejecting file changes outright.
This prevents you from needing to keep git open or going back to a previous checkpoint after every big modifications to the codebase. I definitely had times where some good changes have happened but then the agent decides theya ren't done! You can easily reject some of the changes instead of all of them. If you don't like this feature, you can also turn it off in the settings menu.

Checkpoints system did need modifications to get this to work properly. You can see what was done there and let me know what you think.
ShadowCheckpoints are now stored in a .roo folder in root. This prevents the need to search for all git files and turn off the system if some are found not in root. This was a real bummer when I went to use roo on larger codebases that have .gits throughout.

Test Procedure

run the following tests:
src/core/webview/tests/ClineProvider.spec.ts
src/services/checkpoints/tests/ShadowCheckpointService.spec.ts
src/services/file-changes/tests/FileChangeManager.test.ts
webview-ui/src/components/file-changes/tests/FilesChangedOverview.spec.tsx
webview-ui/src/components/settings/tests/UISettings.spec.tsx
webview-ui/src/context/tests/ExtensionStateContext.spec.tsx
check all languages exist.

open chat and ask it to modify some files, make new files, different types of tool calls, etc. accept and reject the changes
open settings menu and under interfaces, turn off the feature, go through the steps again
turn on the feature on last time and go through the steps again.

make sure checkpoints still work.
when restoring a checkpoint, fco clears and disappears.

Pre-Submission Checklist

Issue Linked: This PR is linked to an approved GitHub Issue (see "Related GitHub Issue" above).
Scope: My changes are focused on the linked issue (one major feature/fix per PR).
Self-Review: I have performed a thorough self-review of my code.
Testing: New and/or updated tests have been added to cover my changes (if applicable).
Documentation Impact: I have considered if my changes require documentation updates (see "Documentation Updates" section below).
Contribution Guidelines: I have read and agree to the Contributor Guidelines.

Screenshots / Videos

Before:

After Checkpoint is hit:

Settings option:

Documentation Updates

You can add a line like, "After every checkpoint, Files changed will appear in chat. You have the option to accept or reject them on a file by file basis. to turn this feature off, find it in the settings menu" but I don't think it makes much to consider it significant.

Additional Notes

I consolidated the feature, made a new branch, pushed just feature changes, and squashed it into a single commit to only contain changes made that involve this feature.

@hannesrudolph I was not sure what you meant by breaking it down by PR as it is one PR. If you have more information, I can get more descriptive. please let me know. If this is due to ShadowCheckpoints changes and there is an issue for it, I can add it.

The nested git repository detection test was failing because FCO integration, changed the shadow git implementation to use .roo directory approach, eliminating the need for nested git repository detection.
add missing UI components and webviewReady message
Import and render FilesChangedOverview component in ChatView
Add webviewReady message to useMount hook for FCO initialization
Add FCO message handling delegation while preserving existing routing (INTEGRATION)
Add FileChangeManager integration for file change tracking (CORE FEATURE)
Add globalFileChangeManager property and disposal cleanup (LIFECYCLE)
Add FCO state properties to provider state (STATE ENHANCEMENT)
Add FileChangeManager accessor methods for FCO components (API)
Add type annotations for task creation callbacks (TYPE SAFETY)
Add filesChangedEnabled message case with state update (NEW MESSAGE HANDLING)
Add filesChangedEnabled property to state object construction (STATE INTEGRATION)
Enhance error handling for API configuration operations (ERROR HANDLING)
Preserve all existing message handling functionality
Add FCO state properties to ExtensionStateContext interface (ENHANCEMENT)
Add filesChanged message handling for real-time updates (NEW CAPABILITY)
Add FCO state update functions for component interaction (STATE MANAGEMENT)
Add currentFileChangeset state for file change tracking (CORE FEATURE)
Initialize filesChangedEnabled in default state (INITIALIZATION)
Add 9 FCO webview message types to existing union (ADDITIVE)
Add 4 FCO extension message types (ADDITIVE)
Add Terminal interface for enhanced functionality (NEW)
Add optional properties for FCO file operations (ENHANCEMENT)
Add filesChangedEnabled state property (STATE ENHANCEMENT)
Preserve all existing message types and backward compatibility
Add FCO component mocks (DecorationController, DiffViewProvider) (TEST INFRASTRUCTURE)
Enhance VSCode mocks with createTextEditorDecorationType, Range, Position (API MOCKS)
Add filesChangedEnabled to test state expectations (STATE TESTING)
Preserve all existing test functionality and coverage
Export FCO types from packages/types (MINIMAL ADDITION)
CheckpointDiff added change type classification (TYPE ENHANCEMENT)
Update checkpoint event interface fto no longer check for nested gits and instead use .roo for all checkpoints (EVENT ENHANCEMENT)
Preserve all main branch type system improvements
Add FileChange and FileChangeset types (NEW)
Add FileChangeManager for state management (NEW)
Add FCOMessageHandler for webview communication (NEW)
Add FilesChangedOverview React components (NEW)
Add comprehensive FCO test coverage (NEW)
Add full internationalization support (18 languages) (NEW)
UI section and FilesChangedEnabled flag added to Settings
testing added to UI section
Add optional uris parameter to WebviewMessage type
Update frontend to send visible file URIs with reject all message
Modify backend to filter files based on provided URIs
Maintain backward compatibility (defaults to all files if no URIs)
Add comprehensive test coverage including scoped rejection scenarios
Fix ShadowCheckpointService test - comment out obsolete nested git check

Get in Touch

Playcations on Discord

Important

Introduces a feature to manage file changes in chat, with UI components, state management, and internationalization support.

Feature:
- Adds FilesChangedOverview component to display modified files in chat.
- Allows accepting, rejecting, and diffing file changes.
- Integrates with settings to enable/disable feature.
State Management:
- Adds filesChangedEnabled to ExtensionStateContext.
- Manages file changes state with FileChangeManager.
UI Components:
- Adds UISettings for managing interface settings.
- Integrates FilesChangedOverview into ChatView.
Internationalization:
- Adds translations for file change actions in 18 languages.
Testing:
- Adds tests for UISettings and FilesChangedOverview.

^{This description was created by}^{for 8ea8926. You can customize this summary. It will automatically update as commits are pushed.}

ellipsis-dev · 2025-07-12T14:44:08Z

webview-ui/src/i18n/locales/id/file-changes.json

+	"header": {
+		"files_changed": "File yang Diubah",
+		"expand": "Perluas daftar file",
+		"collapse": "Ciutkan daftar file"


Typographical consistency: The translation for 'collapse' appears as "Ciutkan daftar file" in the header section but as "Diciutkan" in the accessibility section. Consider using a consistent term (e.g., "Diciutkan daftar file") for clarity.

Suggested change

"collapse": "Ciutkan daftar file"

"collapse": "Diciutkan daftar file"

^{This comment was generated because it violated a code review rule: irule_C0ez7Rji6ANcGkkX.}

adamhill · 2025-07-12T14:54:28Z

Wow. @playcations This is NICE!!!

playcations · 2025-07-12T15:01:35Z

@adamhill I forgot the language files and am failing the cloud tests.. is that okay? I know my edits changed the shadow checkpoint system but running tests locally didn't test this.

playcations · 2025-07-12T19:00:37Z

I see I failed cloudServices and I forgot the language changes for settings... I can work on adding those later today. My mistake. I can close this and make another pr if need be.

playcations · 2025-07-13T04:54:33Z

updated:

updated language files for settings to include FilesChangedOverview.
made Files changed overview match closer to Todo list since that is the theming now. (nice and slim)
small bug fix and added test procedure for the added todo feature and working with it properly.

playcations · 2025-07-13T15:12:24Z

Fco edge case fixes and tests (final push until review)

folders were getting diffed, added a check to prevent this.
.roo now excluded from checkpoint diffs
diffs for FCO now only check line changes vs full diff changes. This is a preventative measure due to search and replace calls rewriting the entire file.
tests for these edge cases added.

mkdir700 · 2025-07-19T03:03:52Z

That's great

src/shared/ExtensionMessage.ts

daniel-lxs

Thank you for taking the time to implement this feature, I feel like it's a nice idea however I do think it would be better to move this to the experimental settings section, and that means removing the new UI section.

I also left a couple of minor points that I think are worth solving.

Let me know what you think!

webview-ui/src/components/file-changes/FilesChangedOverview.tsx

hannesrudolph · 2025-07-21T21:53:06Z

@roomote-agent fix the merge conflicts using pr-fixer mode

I think I saw someone post a screenshot where "errors.claudeCode.notFound" was shown untranslated.

Co-authored-by: mrubens <[email protected]>

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Matt Rubens <[email protected]>

…eInc#7464) Co-authored-by: Roo Code <[email protected]> Co-authored-by: Daniel Riccio <[email protected]>

RooCodeInc#7555) Co-authored-by: Roo Code <[email protected]> Co-authored-by: Matt Rubens <[email protected]> Co-authored-by: roomote[bot] <219738659+roomote[bot]@users.noreply.github.com>

ClineProvider creation was moved before CloudService which broke the old way of doing things.

…nc#7602) * fix: add cache reporting support for OpenAI-Native provider - Add normalizeUsage method to properly extract cache tokens from Responses API - Support both detailed token shapes (input_tokens_details) and legacy fields - Calculate cache read/write tokens with proper fallbacks - Include reasoning tokens when available in output_tokens_details - Ensure accurate cost calculation using uncached input tokens This fixes the issue where caching information was not being reported when using the OpenAI-Native provider with the Responses API. * fix: improve cache token normalization and add comprehensive tests - Add fallback to derive total input tokens from details when totals are missing - Remove unused convertToOpenAiMessages import - Add comment explaining cost calculation alignment with Gemini provider - Add comprehensive test coverage for normalizeUsage method covering: - Detailed token shapes with cached/miss tokens - Legacy field names and SSE-only events - Edge cases including missing totals with details-only - Cost calculation with uncached input tokens * fix: address PR review comments - Remove incorrect fallback to missFromDetails for cache write tokens - Fix cost calculation to pass total input tokens (calculateApiCostOpenAI handles subtraction) - Improve readability by extracting cache detail checks to intermediate variables - Remove redundant ?? undefined - Update tests to reflect correct behavior (miss tokens are not cache writes) - Add clarifying comments about cache miss vs cache write tokens

…ebase - Remove deprecated BridgeOrchestrator imports and usage from Task.ts, extension.ts - Replace removed getUserSettings() method calls with fallbacks - Fix type compatibility issues with clineMessages parameter - Fix FileChangeManager baseline assignment and rejection logic - Auto-assign fromCheckpoint as initial baseline when files enter FCO - Fix rejection to preserve existing baselines - Fix acceptance to properly update baselines to current checkpoint - Add missing mock setups in tests for applyPerFileBaselines calls - Update test expectations to match calculated line differences - All TypeScript type checking now passes (11/11 packages)

playcations · 2025-09-03T05:51:50Z

I used the merge agent in Roo and now I have more. I can get to these tomorrow.

playcations · 2025-09-03T13:59:34Z

@roomote-agent fix the merge conflicts using merge resolver mode

roomote · 2025-09-03T14:01:15Z

Hi @playcations! I'm on it - I'll resolve the merge conflicts for the Files Changed Overview (FCO) PR. Let me check out the branch and analyze the conflicts now.

roomote · 2025-09-03T14:29:49Z

Merge Conflict Resolution Update

I've successfully resolved all 36 merge conflicts in the PR and pushed the changes to the fco-clean branch. Here's what was done:

Conflicts Resolved:

CHANGELOG.md: Merged version 3.26.5 release notes from both branches
README files (19 total): Updated contributor tables across all locale versions
ExtensionChannel.ts: Added new events (TaskPaused, TaskUnpaused, TaskSpawned, TaskUserMessage)
Type definitions: Added missing types for the Files Changed Overview feature:
- Added message types: webviewReady, filesChangedRequest, filesChangedEnabled, etc.
- Added properties: fileChanges, baseline, filesChangedEnabled
- Added methods: getFileChangeManager(), ensureFileChangeManager()
Fixed TypeScript compilation errors:
- Added type annotations for implicit any parameters
- Fixed Set vs Map usage for ongoingCheckpointSaves
- Added missing properties to Task and ClineProvider classes

Current Status:

The branch has been successfully pushed with all conflicts resolved and TypeScript compilation passing. However, GitHub is still showing the PR as conflicting. This may be due to:

GitHub needing time to update the merge status
The PR base branch may need to be updated

Next Steps:

Please try refreshing the PR page or updating the base branch if the conflicts persist. All code changes have been properly merged and the branch compiles successfully.

The resolved branch is available at: fco-clean

playcations · 2025-09-05T00:31:00Z

@roomote-agent fix the merge conflicts you previously mentioned using merge resolver mode

roomote · 2025-09-05T00:32:26Z

Hi @playcations! I see your request to resolve the merge conflicts. I'm switching to merge resolver mode now to handle this. I'll analyze the conflicts and resolve them appropriately. Will update you once complete.

hannesrudolph · 2025-09-08T23:38:49Z

This PR has been rebased onto the latest main branch to resolve merge conflicts.

New PR: #7800

The rebased version preserves all the Files Changed Overview feature implementation while integrating with recent main branch changes. Please review the new PR for the updated code.

Note: The new PR has some TypeScript compilation errors that resulted from the merge conflict resolution and will need manual cleanup before merging.

playcations · 2025-09-09T14:08:53Z

A new PR has been created for this #7816

A new branch was made stemming from the head of main and Files changed overview was refactored to work from there.

playcations requested review from cte, jr and mrubens as code owners July 12, 2025 14:41

github-project-automation bot added this to Roo Code Roadmap and Roo Code Roadmap Jul 12, 2025

github-project-automation bot moved this to Triage in Roo Code Roadmap Jul 12, 2025

github-project-automation bot moved this to New in Roo Code Roadmap Jul 12, 2025

dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. enhancement New feature or request labels Jul 12, 2025

daniel-lxs moved this from Triage to PR [Needs Prelim Review] in Roo Code Roadmap Jul 12, 2025

ellipsis-dev bot reviewed Jul 12, 2025

View reviewed changes

hannesrudolph added the PR - Needs Preliminary Review label Jul 12, 2025

mkdir700 suggested changes Jul 19, 2025

View reviewed changes

src/shared/ExtensionMessage.ts Outdated Show resolved Hide resolved

daniel-lxs requested changes Jul 21, 2025

View reviewed changes

daniel-lxs moved this from PR [Needs Prelim Review] to PR [Changes Requested] in Roo Code Roadmap Jul 21, 2025

hannesrudolph added PR - Changes Requested and removed PR - Needs Preliminary Review labels Jul 21, 2025

RooCodeInc deleted a comment from roomote bot Jul 23, 2025

RooCodeInc deleted a comment from hannesrudolph Jul 23, 2025

RooCodeInc deleted a comment from roomote bot Jul 23, 2025

RooCodeInc deleted a comment from hannesrudolph Jul 23, 2025

chrarnoldus and others added 8 commits September 3, 2025 00:24

Fix claudeCode.notFound translation key (RooCodeInc#7571)

4a128df

I think I saw someone post a screenshot where "errors.claudeCode.notFound" was shown untranslated.

v3.26.4 (RooCodeInc#7579)

953da9f

Update contributors list (RooCodeInc#7462)

2155473

Co-authored-by: mrubens <[email protected]>

Changeset version bump (RooCodeInc#7580)

005ab1b

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Matt Rubens <[email protected]>

feat: add configurable embedding batch size for code indexing (RooCod…

84543c5

…eInc#7464) Co-authored-by: Roo Code <[email protected]> Co-authored-by: Daniel Riccio <[email protected]>

Shows a pill with the base Roo Code Cloud URL when not pointing to pr… (

0eb97d0

RooCodeInc#7555) Co-authored-by: Roo Code <[email protected]> Co-authored-by: Matt Rubens <[email protected]> Co-authored-by: roomote[bot] <219738659+roomote[bot]@users.noreply.github.com>

Cloud: fix provider syncing (RooCodeInc#7603)

a0e13fe

ClineProvider creation was moved before CloudService which broke the old way of doing things.

playcations force-pushed the fco-clean branch from 016c19f to 8af0d55 Compare September 3, 2025 04:40

roomote added 4 commits September 3, 2025 14:12

Merge branch 'origin/main' into fco-clean - resolved conflicts

007b47a

fix: Add missing types for Files Changed Overview feature

db6cd50

fix: Add remaining missing types for Files Changed Overview feature

886e817

fix: Complete type fixes for Files Changed Overview feature

eabbe46

hannesrudolph mentioned this pull request Sep 8, 2025

Files Changed Overview feature (rebased from #5626) #7800

Closed

playcations mentioned this pull request Sep 9, 2025

Files Changed Overview (Rebased from #5626) #7816

Closed

6 tasks

playcations mentioned this pull request Sep 10, 2025

Files Changed Overview ( from #5626 rebased and within scope) #7856

Closed

6 tasks

This was referenced Sep 20, 2025

No Diff Viewer or Per-File Summary for Multi-File AI Edits #5067

Closed

feat: Add Files Changed Overview (FCO) - Real-time file diff tracking and UI (refactored from #5626) #8193

Open

hannesrudolph closed this Sep 23, 2025

github-project-automation bot moved this from New to Done in Roo Code Roadmap Sep 23, 2025

github-project-automation bot moved this from PR [Changes Requested] to Done in Roo Code Roadmap Sep 23, 2025

	"collapse": "Ciutkan daftar file"
	"collapse": "Diciutkan daftar file"

Files Changed Overview (FCO) #5626

Files Changed Overview (FCO) #5626

Uh oh!

Conversation

playcations commented Jul 12, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Related GitHub Issue

Description

Test Procedure

Pre-Submission Checklist

Screenshots / Videos

Before:

After Checkpoint is hit:

Settings option:

Documentation Updates

Additional Notes

Get in Touch

Uh oh!

ellipsis-dev bot Jul 12, 2025

Choose a reason for hiding this comment

Uh oh!

adamhill commented Jul 12, 2025

Uh oh!

playcations commented Jul 12, 2025

Uh oh!

playcations commented Jul 12, 2025

Uh oh!

playcations commented Jul 13, 2025

Uh oh!

playcations commented Jul 13, 2025

Uh oh!

mkdir700 commented Jul 19, 2025

Uh oh!

Uh oh!

daniel-lxs left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

hannesrudolph commented Jul 21, 2025

Uh oh!

playcations commented Sep 3, 2025

Uh oh!

playcations commented Sep 3, 2025

Uh oh!

roomote bot commented Sep 3, 2025

Uh oh!

roomote bot commented Sep 3, 2025

Merge Conflict Resolution Update

Conflicts Resolved:

Current Status:

Next Steps:

Uh oh!

playcations commented Sep 5, 2025

Uh oh!

roomote bot commented Sep 5, 2025

Uh oh!

hannesrudolph commented Sep 8, 2025

Uh oh!

playcations commented Sep 9, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

14 participants

playcations commented Jul 12, 2025 •

edited

Loading