fix: normalize paths in monaco model cleanup

[matiaspalmac]

Changes

The model cleanup effect in EditorArea.tsx was comparing f.path (Windows backslashes) against Monaco URIs (forward slashes, URL-encoded). None of the three comparisons in .some() ever matched on Windows, so every newly created model was immediately disposed — leaving the editor rendering against a disposed model (empty content).

• Normalize both sides to forward slashes + lowercase before comparing. Uses Set lookup instead of the previous triple .some() fallback.
• Use model.uri.scheme === "inmemory" (cleaner than startsWith("inmemory://")).
• Drop currentFile from the effect deps — only openFiles matters for lifecycle.
• Remove the unused openPaths variable and the console.log on each dispose.

Disposal still happens as originally intended — files that leave openFiles have their models freed. The behavior is now correct on Windows and unchanged on Linux/macOS (where normalization is a no-op on paths that are already forward-slashed).

Issues

• fix [Bug]: Editor shows empty content for opened files on Windows #26

[Copilot]

Pull request overview

Fixes incorrect Monaco model disposal on Windows by normalizing file paths before comparing open files to Monaco model URIs, preventing active models from being disposed and the editor from rendering blank.

Changes:

• Normalize openFiles paths and Monaco model paths before comparison, using a Set for efficient membership checks.
• Use model.uri.scheme === "inmemory" to skip internal Monaco models.
• Reduce effect re-runs by removing currentFile from the cleanup effect dependency list and removing noisy disposal logging.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.