fix(desktop): refresh button not showing new files in nested folders

[de1mat]

Fixes #1227

Summary

The refresh button only invalidated the root directory cache, causing new files in nested folders to remain invisible until the panel was reopened.

Changes

Updated handleRefresh to invalidate all expanded directories, matching the pattern already used by handleToggleHiddenFiles.

Test Plan

1. Open a workspace and expand a nested folder
2. Create a file externally in that folder
3. Click refresh - file should now appear

Summary by CodeRabbit

• Bug Fixes
  • Fixed an issue where newly added or moved files within subdirectories weren't properly reflected in the file view. The file display now correctly refreshes all directory contents during updates.

[coderabbitai]

📝 Walkthrough

Walkthrough

The handleRefresh function in FilesView.tsx now explicitly invalidates cache for all directory items, not just the root. This ensures newly created or moved files in expanded nested folders appear immediately after clicking refresh, rather than requiring a side panel close/reopen.

Changes

Cohort / File(s)	Summary
File Tree Refresh Cache Invalidation apps/desktop/src/renderer/screens/main/components/WorkspaceView/RightSidebar/FilesView/FilesView.tsx	Added iteration over all items to invalidate children cache for directories during refresh, complementing root item invalidation to address stale nested folder caches.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• Fix file tree staying stale when switching workspaces #1143: Modifies FilesView.tsx refresh logic by introducing invalidateChildrenIds usage and expanding child invalidation scope to handle nested directory caches.

Poem

🐰 A refresh that hops through folders deep,
No more caches the tree must keep,
Nested files now appear with glee,
When new-born code joins the tree! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely describes the main bug fix: the refresh button not showing new files in nested folders, which matches the primary change in the PR.
Description check	✅ Passed	The PR description includes a clear summary, related issue link, specific changes made, and a detailed test plan covering the fixed functionality.
Linked Issues check	✅ Passed	The PR successfully addresses issue #1227 by updating handleRefresh to invalidate all expanded directories, enabling proper display of newly created files in nested folders after refresh.
Out of Scope Changes check	✅ Passed	All changes in the PR are directly related to the linked issue #1227 regarding the file tree refresh functionality for nested folders, with no out-of-scope modifications.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Kitenite]

Works great, thanks @de1mat !