mbeditor 0.4.5 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 75e7d28ba5664e0aeae526a954b2032c040d6ac2f20ed327bef9507a29bc3e21
-  data.tar.gz: 34b2ba82856f0fef274049050a55e95a465b9d9bdb7e86e3a2299c30ce0e00c4
+  metadata.gz: 79b3c0567fff4f540eeca0a01d4f5133192e55b200b920808295c443b67d5f6a
+  data.tar.gz: 54f7fa3e433a9dbd34b17556fa0abfb244b220b4800af1997bea0be700955b22
 SHA512:
-  metadata.gz: ba333da50a99c900136a52832821b7a2d939f7836b5aa05d2bdaedb259167c3c8a32f7ec3e13e854501c51a7686dd354973b3ff9a299c87af2b0e41cfaa8a649
-  data.tar.gz: 9fd083956de7ac0c3443715ba15fe63af069822043ab9e4113b8ab9af4096e5ac582e29d8377cd63f1c681b9fbcca5097dbc7f1d20592f937df6bb208fe91988
+  metadata.gz: 1f2e2cec02b0019326c1e16e3659da31c9d6387fd84c8d38e0ca72be1ea007e22b4e5fc26512caa0a620f30ca589718e149bab210a1eeef17f349ba452289c1d
+  data.tar.gz: bd145f73f0df001904fa139a8365f9dae815c9ad391bc8bc77aaefd0cb64a6317f8c74f0d43bdabcbab6c5878a6cba7b0155219dceb0b57d038814bd96096864

data/CHANGELOG.md

@@ -5,6 +5,23 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.5.0] - 2026-04-30
+
+### Added
+- **Zen / focus mode** — `Cmd+Shift+Z` hides the sidebar and git panel for a distraction-free editing experience.
+- **Bulk find-and-replace** — search and replace across all workspace files in one operation.
+- **File content prefetch on hover** — opening a file from the tree is now instant; content is fetched while hovering the row.
+- **Client-side search cache** — search results are cached client-side for 30 s and invalidated automatically on save.
+- **Monaco model cache with LRU eviction** — in-memory Monaco models are capped at 15 and evicted in LRU order to keep memory use bounded.
+
+### Changed
+- **Faster initial render** — Monaco startup is now decoupled from the React mount lifecycle, cutting time-to-first-edit.
+- **Robust dirty-state tracking** — dirty state is now driven by Monaco's `alternativeVersionId`; `cleanVersionId` is reset correctly on save-on-close so re-opened tabs start clean.
+
+### Fixed
+- ActionCable reconnect logic hardened; regression tests added to confirm websocket lifecycle log filtering still works after reconnect.
+- Bulk replace: fixed a security issue and a correctness bug in the replacement pipeline; added covering tests.
+
 ## [0.4.5] - 2026-04-23
 
 ### Fixed

data/app/assets/javascripts/mbeditor/application_iife_tail.js

@@ -1,3 +1,4 @@
 window.SearchService = SearchService;
 window.GitService = GitService;
+window.FileService = FileService;
 })(window.MbeditorRuntime.React, window.MbeditorRuntime.ReactDOM);

data/app/assets/javascripts/mbeditor/components/EditorPanel.js

@@ -30,6 +30,7 @@ var EditorPanel = function EditorPanel(_ref) {
   var testLoading = _ref.testLoading;
   var testInlineVisible = _ref.testInlineVisible;
   var editorPrefs = _ref.editorPrefs || {};
+  var monacoReady = _ref.monacoReady !== false; // undefined means Monaco already loaded (legacy callers)
 
   var editorRef = useRef(null);
   var monacoRef = useRef(null);
@@ -132,7 +133,7 @@ var EditorPanel = function EditorPanel(_ref) {
 
   useEffect(function () {
     if (tab.isPreview) return;
-    if (!editorRef.current || !window.monaco) return;
+    if (!monacoReady || !editorRef.current || !window.monaco) return;
 
     if (window.MbeditorEditorPlugins && window.MbeditorEditorPlugins.registerGlobalExtensions) {
       window.MbeditorEditorPlugins.registerGlobalExtensions(window.monaco);
@@ -478,13 +479,17 @@ var EditorPanel = function EditorPanel(_ref) {
     if (_modelEntry && _modelEntry.model && !_modelEntry.model.isDisposed()) {
       modelObj = _modelEntry.model;
       _reusingModel = true;
+      // Update access timestamp so LRU eviction knows this model was recently used.
+      _modelEntry.lastAccessed = Date.now();
       // Re-apply language in case it changed (e.g. file renamed)
       if (modelObj.getLanguageId() !== language) {
         window.monaco.editor.setModelLanguage(modelObj, language);
       }
     } else {
+      // Evict the LRU model if the cache is at capacity before creating a new one.
+      TabManager.evictLruModel();
       modelObj = window.monaco.editor.createModel(tab.content, language);
-      window.__mbeditorModels[tab.path] = { model: modelObj, aviBase: null, aviMax: null };
+      window.__mbeditorModels[tab.path] = { model: modelObj, aviBase: null, aviMax: null, lastAccessed: Date.now(), cleanVersionId: null };
       _modelEntry = window.__mbeditorModels[tab.path];
     }
 
@@ -645,6 +650,8 @@ var EditorPanel = function EditorPanel(_ref) {
     } else {
       aviBaseRef.current = avi;
       aviMaxRef.current = avi;
+      // Record the clean baseline for dirty-state tracking on initial model creation.
+      _modelEntry.cleanVersionId = avi;
     }
     EditorStore.setState({ canUndo: avi > aviBaseRef.current, canRedo: avi < aviMaxRef.current });
 
@@ -660,19 +667,16 @@ var EditorPanel = function EditorPanel(_ref) {
 
       var val = editor.getValue();
 
-      // Belt-and-suspenders: when undoing, directly check if content matches the
-      // saved clean state. This guarantees the dirty flag clears on full undo even
-      // if the latestContentRef comparison path misses an event.
-      if (e.isUndoing) {
-        var _st = EditorStore.getState();
-        var _pane = _st.panes.find(function(p) { return p.id === paneId; });
-        var _tab = _pane && _pane.tabs.find(function(t) { return t.id === tab.id; });
-        if (_tab && _tab.dirty) {
-          var _cleanNorm = ((_tab.cleanContent) || '').replace(/\r\n/g, '\n');
-          var _valNorm = val.replace(/\r\n/g, '\n');
-          if (_valNorm === _cleanNorm) {
-            TabManager.markClean(paneId, tab.id, val);
-          }
+      // Dirty-state tracking via alternativeVersionId — O(1), no string comparison.
+      // AVI decrements on undo so it returns to cleanVersionId after a full undo.
+      // Skip entirely when cleanVersionId is null — file is mid-load, not yet settled.
+      var _entry = window.__mbeditorModels && window.__mbeditorModels[tab.path];
+      var _cleanAvi = _entry && _entry.cleanVersionId;
+      if (_cleanAvi !== null && _cleanAvi !== undefined) {
+        if (currentAvi !== _cleanAvi) {
+          TabManager.markDirty(paneId, tab.id, val);
+        } else {
+          TabManager.markClean(paneId, tab.id, val);
         }
       }
 
@@ -734,7 +738,7 @@ var EditorPanel = function EditorPanel(_ref) {
       editor.setModel(null);
       editor.dispose();
     };
-  }, [tab.id, tab.isPreview]); // re-run ONLY on tab switch, not on content change (Monaco handles its own content state)
+  }, [tab.id, tab.isPreview, monacoReady]); // re-run on tab switch or when Monaco becomes ready
 
   // Listen for external content changes (e.g. after Format/Load)
   // Only applies when externalContentVersion advances — prevents stale typing-originated
@@ -760,12 +764,17 @@ var EditorPanel = function EditorPanel(_ref) {
     if (!vNorm) {
       // If the editor is currently completely empty, treat it as an initial load.
       // setValue clears the undo stack which is correct for initial load.
+      // Null cleanVersionId before setValue so the synchronous onDidChangeContent
+      // fires during setValue and skips the dirty check (cleanVersionId is null).
+      var _initEntry = window.__mbeditorModels && window.__mbeditorModels[tab.path];
+      if (_initEntry) _initEntry.cleanVersionId = null;
       editor.setValue(tab.content);
       // Reset the AVI baseline: setValue clears the undo stack so anything before
       // this point is no longer reachable. Also clear the canUndo/canRedo display.
       var newBase = model.getAlternativeVersionId();
       aviBaseRef.current = newBase;
       aviMaxRef.current = newBase;
+      if (_initEntry) _initEntry.cleanVersionId = newBase;
       EditorStore.setState({ canUndo: false, canRedo: false });
     } else {
       // Keep undo stack for formats or replaces by using executeEdits
@@ -1363,6 +1372,16 @@ var EditorPanel = function EditorPanel(_ref) {
     return '\u2026/' + parts.slice(-2).join('/');
   }
 
+  // While Monaco is still loading, show a lightweight skeleton so the UI is
+  // visible immediately without calling monaco.editor.create() too early.
+  if (!monacoReady) {
+    return React.createElement(
+      'div',
+      { className: 'monaco-container monaco-loading-skeleton', style: { display: 'flex', alignItems: 'center', justifyContent: 'center', height: '100%', color: '#888', fontSize: '13px' } },
+      'Loading editor…'
+    );
+  }
+
   // Always render the same wrapper structure so the editorRef div is never
   // unmounted when gitAvailable changes (e.g. loaded async after workspace
   // call returns). The toolbar is conditionally included inside the wrapper.

data/app/assets/javascripts/mbeditor/components/FileTree.js

@@ -51,6 +51,8 @@ var FileTree = function FileTree(_ref) {
   var containerRef = useRef(null);
   var typeaheadBufferRef = useRef('');
   var typeaheadTimerRef = useRef(null);
+  var hoverTimerRef = useRef(null);
+  var hoverPathRef = useRef(null);
   // Ref that always points to the latest onNodeSelect prop, avoiding stale closures in the effect.
   var onNodeSelectRef = useRef(onNodeSelect);
   onNodeSelectRef.current = onNodeSelect;
@@ -131,6 +133,12 @@ var FileTree = function FileTree(_ref) {
     }
   }, [pendingCreate, pendingRename]);
 
+  // Clear any pending hover timer when the component unmounts to prevent
+  // a prefetch from firing against an unmounted component.
+  useEffect(function() {
+    return function() { clearTimeout(hoverTimerRef.current); };
+  }, []);
+
   var toggleFolder = function toggleFolder(path, e) {
     e.stopPropagation();
     var next = !(expandedDirs && expandedDirs[path]);
@@ -441,6 +449,20 @@ var FileTree = function FileTree(_ref) {
                 onFileDoubleClick(node.path, node.name);
               }
             },
+            onMouseEnter: function () {
+              if (isFolder) return;
+              clearTimeout(hoverTimerRef.current);
+              hoverTimerRef.current = setTimeout(function () {
+                hoverPathRef.current = node.path;
+                FileService.prefetch(node.path);
+              }, 200);
+            },
+            onMouseLeave: function () {
+              if (isFolder) return;
+              clearTimeout(hoverTimerRef.current);
+              FileService.cancelPrefetch(hoverPathRef.current);
+              hoverPathRef.current = null;
+            },
             onContextMenu: function (e) {
               e.preventDefault();
               e.stopPropagation();
@@ -498,7 +520,7 @@ var FileTreeMemo = React.memo(FileTree, function(prev, next) {
     prev.activePath === next.activePath &&
     prev.selectedPaths === next.selectedPaths &&
     prev.anchorPath === next.anchorPath &&
-    prev.gitFiles === next.gitFiles &&
+    JSON.stringify(prev.gitFiles) === JSON.stringify(next.gitFiles) &&
     prev.expandedDirs === next.expandedDirs &&
     prev.pendingCreate === next.pendingCreate &&
     prev.pendingRename === next.pendingRename;