fix(sidebar): empty environment children from cache poisoning + watcher gap

[jeonghun-jj-lee]

Problem

Expanding a resolved (non-workspace) environment root in the sidebar shows nothing beneath it — the chevron rotates but zero filesystem children render, despite the directory having contents.

Root cause

Two bugs combine:

1. Cache poisoning (sidebar_webview.ts)

readDirectoryEntries catches all errors and returns []. The webview caches this as childrenCache[path] = []. Because [] is truthy in JavaScript, all !childrenCache[path] guards evaluate false, permanently suppressing re-requests for that path.

2. Watcher gap (sidebar_view.ts)

setupWatcher uses createFileSystemWatcher('**/*') (workspace-scoped) and the handler checks getWorkspaceFolder(uri) — both drop non-workspace URIs. Resolved environments (source: "resolved", auto-surfaced from the registry) are not workspace folders, so filesystem changes inside them never invalidate the poisoned cache.

Fix

Fix 1: Never cache empty arrays — delete the key on empty responses so undefined (falsy) lets future renders re-request children.

Fix 2: Create per-path FileSystemWatcher using RelativePattern for resolved environment paths. Watchers are refreshed on roots changes, deduped by path set, debounced (300ms), and disposed cleanly.

Bonus: Added console.warn to readDirectoryEntries catch block for future diagnosis.

Verification

• pnpm -r build — clean
• pnpm --filter amicode test — 2980 passed, 0 failures (including all 278 sidebar tests)

Closes #903

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 98ddeef0-9e9a-49b1-9dc1-672cd558aa5f

---

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.}