refactor(file-browser): Replace plain object directory cache with Map

[AuDevTist1C]

Replaces the plain object ({}) used for directory caching (cachedDir) in src/pages/fileBrowser/fileBrowser.js with an ES6 Map instance.

Motivation

• Key Safety: Plain objects inherit properties from Object.prototype, which can lead to collisions or unexpected behavior if file paths match built-in property names. Map provides clean key isolation.

References

• MDN Map vs. Object Comparison: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Map#objects_vs._maps

---

Summary of Code Changes

• src/pages/fileBrowser/fileBrowser.js:
  • Initialized cachedDir variable using new Map().
  • Replaced url in cachedDir with cachedDir.has(url).
  • Replaced cachedDir[url] access with cachedDir.get(url).
  • Replaced property assignment cachedDir[dir.url] = dir with cachedDir.set(dir.url, dir).
  • Replaced property deletion delete cachedDir[url] with native cachedDir.delete(url).

_{(PR name and description are AI generated (Gemini 3.6 flash))}

[greptile-apps]

Greptile Summary

Replaces the file browser’s plain-object directory cache with an ES6 Map.

• Converts cache initialization, lookup, insertion, deletion, and scroll-state access to Map APIs.
• Removes redundant invalidation from the reload action because reload() already deletes the current cache entry.

Confidence Score: 5/5

The PR appears safe to merge with no actionable correctness or security issues identified.

All directory-cache operations consistently use Map, the cache remains closure-local, and every reload continues to invalidate the current directory before navigating and rereading it.

Important Files Changed

Filename	Overview
src/pages/fileBrowser/fileBrowser.js	The directory-cache conversion is complete and preserves cache lookup, invalidation, reload, and scroll-restoration behavior.

_{Reviews (1): Last reviewed commit: "refactor(file-browser): Convert director..." | Re-trigger Greptile}