mbeditor 0.5.0 → 0.5.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 79b3c0567fff4f540eeca0a01d4f5133192e55b200b920808295c443b67d5f6a
-  data.tar.gz: 54f7fa3e433a9dbd34b17556fa0abfb244b220b4800af1997bea0be700955b22
+  metadata.gz: cd0837fbeccc3634d7804e6f9957bbbaca06bb4c02942eb4decc5e8b174499ab
+  data.tar.gz: 9f97239369b2968c17cd3909abdced4917329a349564d02ebde6beb89cb8c36f
 SHA512:
-  metadata.gz: 1f2e2cec02b0019326c1e16e3659da31c9d6387fd84c8d38e0ca72be1ea007e22b4e5fc26512caa0a620f30ca589718e149bab210a1eeef17f349ba452289c1d
-  data.tar.gz: bd145f73f0df001904fa139a8365f9dae815c9ad391bc8bc77aaefd0cb64a6317f8c74f0d43bdabcbab6c5878a6cba7b0155219dceb0b57d038814bd96096864
+  metadata.gz: 6310dc010236970bf25c0945a465122e1f8a081c399696d8870dc852d85454153863b01b1f1e9a8871abc3fbe07b1489724a065835c2909daa15426899701833
+  data.tar.gz: f8ed953601769d6785a4fecd4c3b16a6ce820e6e7f5c88a7cb12750d8f32b5122dc6bc25fd67be5e86bd7ef02ae1beba9ece2dbc5fc5bc0324df47785a03c9ba

data/CHANGELOG.md

@@ -5,6 +5,12 @@ 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.1] - 2026-04-30
+
+### Added
+- **Large file pagination** — files over 5 MB now open in read-only paginated mode (500 lines per page) instead of showing an error. A bar below the toolbar shows the current line range, total line count, and file size, with Prev/Next navigation. The backend streams only the requested line slice via `File.foreach` so arbitrarily large files never load fully into memory.
+- **JSON auto pretty-print** — `.json` files are automatically formatted with 2-space indentation when opened. Invalid JSON falls back to raw display with Monaco's built-in error markers. The formatted content is set as the editor baseline so the file does not appear dirty after opening.
+
 ## [0.5.0] - 2026-04-30
 
 ### Added

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

@@ -86,6 +86,27 @@ var EditorPanel = function EditorPanel(_ref) {
 
   var methodsBtnRef = useRef(null);
 
+  // Local pagination state — initialized from tab props; updated on page navigation
+  var _useState17 = useState(tab.startLine || 0);
+  var _useState18 = _slicedToArray(_useState17, 2);
+  var pageStartLine = _useState18[0];
+  var setPageStartLine = _useState18[1];
+
+  var _useState19 = useState(tab.lineCount || 0);
+  var _useState20 = _slicedToArray(_useState19, 2);
+  var pageLineCount = _useState20[0];
+  var setPageLineCount = _useState20[1];
+
+  var _useState21 = useState(tab.totalLines || 0);
+  var _useState22 = _slicedToArray(_useState21, 2);
+  var pageTotalLines = _useState22[0];
+  var setPageTotalLines = _useState22[1];
+
+  var _useState23 = useState(tab.totalBytes || 0);
+  var _useState24 = _slicedToArray(_useState23, 2);
+  var pageTotalBytes = _useState24[0];
+  var setPageTotalBytes = _useState24[1];
+
   var onFormatRef = useRef(onFormat);
   onFormatRef.current = onFormat;
 
@@ -95,6 +116,12 @@ var EditorPanel = function EditorPanel(_ref) {
   var vimStatusRef = useRef(null);
   var vimModeObjRef = useRef(null);
 
+  function humanSize(bytes) {
+    if (bytes < 1024) return bytes + ' B';
+    if (bytes < 1024 * 1024) return (bytes / 1024).toFixed(1) + ' KB';
+    return (bytes / (1024 * 1024)).toFixed(1) + ' MB';
+  }
+
   var clearTestZones = function clearTestZones(editor) {
     if (!editor) return;
     if (testZoneIdsRef.current.length === 0) return;
@@ -488,7 +515,18 @@ var EditorPanel = function EditorPanel(_ref) {
     } 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);
+
+      // Pretty-print JSON content before initial load
+      var contentForModel = tab.content;
+      if (language === 'json' && contentForModel) {
+        try {
+          contentForModel = JSON.stringify(JSON.parse(contentForModel), null, 2);
+        } catch (_) {
+          // invalid JSON — use raw content; Monaco will show error markers
+        }
+      }
+
+      modelObj = window.monaco.editor.createModel(contentForModel, language);
       window.__mbeditorModels[tab.path] = { model: modelObj, aviBase: null, aviMax: null, lastAccessed: Date.now(), cleanVersionId: null };
       _modelEntry = window.__mbeditorModels[tab.path];
     }
@@ -578,6 +616,10 @@ var EditorPanel = function EditorPanel(_ref) {
 
     monacoRef.current = editor;
     window.__mbeditorActiveEditor = editor;
+    // Apply read-only for paginated (truncated) files
+    if (tab.truncated) {
+      editor.updateOptions({ readOnly: true });
+    }
     setEditorReady(true);
 
     // Stash the workspace-relative path on the model so code-action providers
@@ -768,7 +810,19 @@ var EditorPanel = function EditorPanel(_ref) {
       // 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);
+
+      // Pretty-print JSON content before initial load
+      var contentToSet = tab.content;
+      var modelLang = model.getLanguageId();
+      if (modelLang === 'json' && contentToSet) {
+        try {
+          contentToSet = JSON.stringify(JSON.parse(contentToSet), null, 2);
+        } catch (_) {
+          // invalid JSON — use raw content; Monaco will show error markers
+        }
+      }
+
+      editor.setValue(contentToSet);
       // 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();
@@ -917,6 +971,21 @@ var EditorPanel = function EditorPanel(_ref) {
     }
   }, [markers, tab.id]);
 
+  // Sync pagination state when tab changes (different file or fresh load)
+  useEffect(function () {
+    setPageStartLine(tab.startLine || 0);
+    setPageLineCount(tab.lineCount || 0);
+    setPageTotalLines(tab.totalLines || 0);
+    setPageTotalBytes(tab.totalBytes || 0);
+  }, [tab.id]);
+
+  // Apply read-only mode based on tab.truncated whenever the editor or truncated flag changes
+  useEffect(function () {
+    if (monacoRef.current) {
+      monacoRef.current.updateOptions({ readOnly: !!tab.truncated });
+    }
+  }, [tab.truncated, editorReady]);
+
   // Reset blame + test decorations when file path changes
   useEffect(function () {
     setBlameData(null);
@@ -1456,6 +1525,105 @@ var EditorPanel = function EditorPanel(_ref) {
         !editorPrefs.toolbarIconOnly && !testLoading && React.createElement('span', { className: 'ide-toolbar-label' }, 'Test')
       )
     ),
+    tab.truncated && React.createElement(
+      'div',
+      {
+        className: 'ide-pagination-bar',
+        style: {
+          display: 'flex',
+          alignItems: 'center',
+          gap: '8px',
+          padding: '4px 10px',
+          background: 'var(--ide-toolbar-bg, #252526)',
+          borderBottom: '1px solid var(--ide-border, #3e3e3e)',
+          fontSize: '12px',
+          color: 'var(--ide-toolbar-fg, #ccc)',
+          flexShrink: 0,
+          userSelect: 'none'
+        }
+      },
+      React.createElement(
+        'button',
+        {
+          className: 'ide-icon-btn',
+          style: { padding: '2px 8px', fontSize: '12px' },
+          disabled: pageStartLine === 0,
+          onClick: function() {
+            var newStart = Math.max(0, pageStartLine - 500);
+            FileService.getFileChunk(tab.path, newStart, 500).then(function(data) {
+              var sl = data.start_line || 0;
+              var lc = data.line_count || 0;
+              var tl = data.total_lines || pageTotalLines;
+              var tb = data.total_bytes || pageTotalBytes;
+              setPageStartLine(sl);
+              setPageLineCount(lc);
+              setPageTotalLines(tl);
+              setPageTotalBytes(tb);
+              if (monacoRef.current) {
+                monacoRef.current.setValue(data.content || '');
+                var _paginatedEntry = window.__mbeditorModels && window.__mbeditorModels[tab.path];
+                if (_paginatedEntry) {
+                  var _newBase = monacoRef.current.getModel().getAlternativeVersionId();
+                  aviBaseRef.current = _newBase;
+                  aviMaxRef.current  = _newBase;
+                  _paginatedEntry.cleanVersionId = _newBase;
+                  _paginatedEntry.aviBase = _newBase;
+                  _paginatedEntry.aviMax  = _newBase;
+                }
+                EditorStore.setState({ canUndo: false, canRedo: false });
+                monacoRef.current.updateOptions({ readOnly: true });
+              }
+            }).catch(function(err) {
+              EditorStore.setStatus('Failed to load page: ' + (err && err.message || 'Unknown error'), 'error');
+            });
+          }
+        },
+        '← Prev'
+      ),
+      React.createElement(
+        'span',
+        { style: { flex: 1, textAlign: 'center' } },
+        'Lines ' + (pageStartLine + 1) + '–' + (pageStartLine + pageLineCount) + ' of ' + pageTotalLines + ' (' + humanSize(pageTotalBytes) + ')'
+      ),
+      React.createElement(
+        'button',
+        {
+          className: 'ide-icon-btn',
+          style: { padding: '2px 8px', fontSize: '12px' },
+          disabled: pageStartLine + pageLineCount >= pageTotalLines,
+          onClick: function() {
+            var newStart = pageStartLine + pageLineCount;
+            FileService.getFileChunk(tab.path, newStart, 500).then(function(data) {
+              var sl = data.start_line || 0;
+              var lc = data.line_count || 0;
+              var tl = data.total_lines || pageTotalLines;
+              var tb = data.total_bytes || pageTotalBytes;
+              setPageStartLine(sl);
+              setPageLineCount(lc);
+              setPageTotalLines(tl);
+              setPageTotalBytes(tb);
+              if (monacoRef.current) {
+                monacoRef.current.setValue(data.content || '');
+                var _paginatedEntry = window.__mbeditorModels && window.__mbeditorModels[tab.path];
+                if (_paginatedEntry) {
+                  var _newBase = monacoRef.current.getModel().getAlternativeVersionId();
+                  aviBaseRef.current = _newBase;
+                  aviMaxRef.current  = _newBase;
+                  _paginatedEntry.cleanVersionId = _newBase;
+                  _paginatedEntry.aviBase = _newBase;
+                  _paginatedEntry.aviMax  = _newBase;
+                }
+                EditorStore.setState({ canUndo: false, canRedo: false });
+                monacoRef.current.updateOptions({ readOnly: true });
+              }
+            }).catch(function(err) {
+              EditorStore.setStatus('Failed to load page: ' + (err && err.message || 'Unknown error'), 'error');
+            });
+          }
+        },
+        'Next →'
+      )
+    ),
     React.createElement('div', { ref: editorRef, className: 'monaco-container', style: { flex: 1, minHeight: 0 } }),
     methodsOpen && methodsDropdownPos && React.createElement(
       'div',

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

@@ -12,6 +12,13 @@ var useRef = _React.useRef;
 var useEffect = _React.useEffect;
 var useMemo = _React.useMemo;
 
+function formatSize(bytes) {
+  if (typeof bytes !== 'number' || bytes < 0) return '';
+  if (bytes < 1024)        return bytes + ' B';
+  if (bytes < 1024 * 1024) return (bytes / 1024).toFixed(1) + ' KB';
+  return (bytes / (1024 * 1024)).toFixed(1) + ' MB';
+}
+
 var FileTree = function FileTree(_ref) {
   var items = _ref.items;
   var onSelect = _ref.onSelect;
@@ -477,7 +484,7 @@ var FileTree = function FileTree(_ref) {
           ),
           React.createElement(
             'div',
-            { className: 'tree-item-name', title: node.path },
+            { className: 'tree-item-name', title: node.type === 'file' && node.size != null ? node.path + ' — ' + formatSize(node.size) : node.path },
             node.name
           ),
           statusMeta && React.createElement(

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

@@ -51,6 +51,35 @@
 
   var globalsRegistered = false;
 
+  // Enumerate window for user-defined globals and return a TypeScript declaration string.
+  // Sprockets exposes every top-level var/function as a window property before Monaco
+  // initialises, so scanning at registration time captures all components and helpers.
+  //
+  // Filter: keep only plain writable data properties (configurable, writable, no getter).
+  // Browser built-ins are either non-configurable or accessor properties (hasGet), so
+  // this reliably separates them from user-assigned globals without a native-code test
+  // (which only works for functions, not objects like `document` or `location`).
+  function buildWindowGlobalsShim() {
+    var alreadyDeclared = { React: 1, ReactDOM: 1, PropTypes: 1, MaterialUI: 1, $: 1, jQuery: 1 };
+    var lines = [];
+    try {
+      var keys = Object.keys(window);
+      for (var i = 0; i < keys.length; i++) {
+        var key = keys[i];
+        if (alreadyDeclared[key]) continue;
+        if (!/^[a-zA-Z_$][a-zA-Z0-9_$]*$/.test(key)) continue;
+        var desc;
+        try { desc = Object.getOwnPropertyDescriptor(window, key); } catch (e) { continue; }
+        if (!desc || !desc.configurable || !desc.writable || desc.get) continue;
+        var value;
+        try { value = window[key]; } catch (e) { continue; }
+        if (value === null || value === undefined) continue;
+        lines.push('declare var ' + key + ': any;');
+      }
+    } catch (e) {}
+    return lines.join('\n');
+  }
+
   function leadingWhitespace(line) {
     var match = line.match(/^\s*/);
     return match ? match[0] : '';
@@ -295,6 +324,30 @@
         event.stopPropagation();
       });
 
+      // Navigate to a Ruby symbol: modules/classes go to their definition file,
+      // lowercase symbols go to their def line.
+      function navigateToWord(word) {
+        if (/^[A-Z]/.test(word) && typeof FileService !== 'undefined' && FileService.getModuleMembers) {
+          FileService.getModuleMembers(word).then(function(data) {
+            if (!data || !data.file) return;
+            var filename = data.file.split('/').pop();
+            if (typeof TabManager !== 'undefined' && TabManager.openTab) {
+              TabManager.openTab(data.file, filename, 1);
+            }
+          }).catch(function() {});
+          return;
+        }
+        if (typeof FileService === 'undefined' || !FileService.getDefinition) return;
+        FileService.getDefinition(word, 'ruby').then(function(data) {
+          var results = data && Array.isArray(data.results) ? data.results : [];
+          if (results.length === 0) return;
+          var r = results[0];
+          if (typeof TabManager !== 'undefined' && TabManager.openTab) {
+            TabManager.openTab(r.file, r.file.split('/').pop(), r.line);
+          }
+        }).catch(function() {});
+      }
+
       // Ctrl/Cmd+click — navigate to definition
       gotoMouseDisposable = editor.onMouseDown(function(event) {
         var ctrlOrCmd = event.event.ctrlKey || event.event.metaKey;
@@ -309,19 +362,9 @@
         if (!wordInfo || !wordInfo.word || wordInfo.word.length < 2) return;
         if (RUBY_KEYWORDS[wordInfo.word]) return;
         if (RUBY_CORE_METHODS[wordInfo.word]) return;
-        if (typeof FileService === 'undefined' || !FileService.getDefinition) return;
 
         event.event.preventDefault();
-
-        FileService.getDefinition(wordInfo.word, 'ruby').then(function(data) {
-          var results = data && Array.isArray(data.results) ? data.results : [];
-          if (results.length === 0) return;
-          var r = results[0];
-          var filename = r.file.split('/').pop();
-          if (typeof TabManager !== 'undefined' && TabManager.openTab) {
-            TabManager.openTab(r.file, filename, r.line);
-          }
-        }).catch(function() {});
+        navigateToWord(wordInfo.word);
       });
 
       // F12 — go to definition from keyboard
@@ -338,17 +381,7 @@
           if (!wordInfo || !wordInfo.word || wordInfo.word.length < 2) return;
           if (RUBY_KEYWORDS[wordInfo.word]) return;
           if (RUBY_CORE_METHODS[wordInfo.word]) return;
-          if (typeof FileService === 'undefined' || !FileService.getDefinition) return;
-
-          FileService.getDefinition(wordInfo.word, 'ruby').then(function(data) {
-            var results = data && Array.isArray(data.results) ? data.results : [];
-            if (results.length === 0) return;
-            var r = results[0];
-            var filename = r.file.split('/').pop();
-            if (typeof TabManager !== 'undefined' && TabManager.openTab) {
-              TabManager.openTab(r.file, filename, r.line);
-            }
-          }).catch(function() {});
+          navigateToWord(wordInfo.word);
         }
       });
     }
@@ -455,6 +488,66 @@
       });
     }
 
+    // Declare globals that the sprockets asset pipeline injects at runtime so
+    // checkJs doesn't flag them as undefined.  `interface Window` augmentation
+    // covers `window.myAppGlobal` access patterns.  For app-specific component
+    // names not listed here, add `/* global MyComponent */` at the top of the
+    // file — TypeScript's checkJs mode respects that directive.
+    if (monaco.languages.typescript && monaco.languages.typescript.javascriptDefaults) {
+      monaco.languages.typescript.javascriptDefaults.addExtraLib(
+        [
+          'declare var React: any;',
+          'declare var ReactDOM: any;',
+          'declare var PropTypes: any;',
+          'declare var MaterialUI: any;',
+          'declare var $: any;',
+          'declare var jQuery: any;',
+          'interface Window { [key: string]: any; }'
+        ].join('\n'),
+        'inmemory://mbeditor/sprockets-globals.d.ts'
+      );
+
+      var dynamicShim = buildWindowGlobalsShim();
+      if (dynamicShim) {
+        monaco.languages.typescript.javascriptDefaults.addExtraLib(
+          dynamicShim,
+          'inmemory://mbeditor/window-globals.d.ts'
+        );
+      }
+
+      // Downgrade "declared but never read" (TS6133) from Error to Warning.
+      // TypeScript has no built-in way to emit this as a warning, so we intercept
+      // the marker set after the worker fires and re-apply with lower severity.
+      // JS files use owner 'javascript', TS files use 'typescript'.
+      var WARN_CODES = { '6133': true };
+      var TS_OWNERS = ['javascript', 'typescript'];
+      var _severityPatchActive = false;
+      monaco.editor.onDidChangeMarkers(function(uris) {
+        if (_severityPatchActive) return;
+        _severityPatchActive = true;
+        try {
+          uris.forEach(function(uri) {
+            var model = monaco.editor.getModel(uri);
+            if (!model) return;
+            TS_OWNERS.forEach(function(owner) {
+              var markers = monaco.editor.getModelMarkers({ resource: uri, owner: owner });
+              var needsPatch = markers.some(function(m) {
+                return m.severity === monaco.MarkerSeverity.Error && WARN_CODES[String(m.code)];
+              });
+              if (!needsPatch) return;
+              monaco.editor.setModelMarkers(model, owner, markers.map(function(m) {
+                return (m.severity === monaco.MarkerSeverity.Error && WARN_CODES[String(m.code)])
+                  ? Object.assign({}, m, { severity: monaco.MarkerSeverity.Warning })
+                  : m;
+              }));
+            });
+          });
+        } finally {
+          _severityPatchActive = false;
+        }
+      });
+    }
+
     // TypeScript: enable JSX for .tsx files and catch unused locals.
     if (monaco.languages.typescript && monaco.languages.typescript.typescriptDefaults) {
       monaco.languages.typescript.typescriptDefaults.setCompilerOptions({

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

@@ -54,6 +54,14 @@ var FileService = (function () {
     return axios.get(window.mbeditorBasePath() + '/file', { params: params }).then(function(res) { return res.data; });
   }
 
+  function getFileChunk(path, startLine, lineCount) {
+    if (lineCount === undefined) {
+      lineCount = 500;
+    }
+    var params = { path: path, start_line: startLine, line_count: lineCount };
+    return axios.get(window.mbeditorBasePath() + '/file', { params: params }).then(function(res) { return res.data; });
+  }
+
   function saveFile(path, code) {
     return axios.post(window.mbeditorBasePath() + '/file', { path: path, code: code }).then(function(res) { return res.data; });
   }
@@ -200,6 +208,7 @@ var FileService = (function () {
     getWorkspace: getWorkspace,
     getTree: getTree,
     getFile: getFile,
+    getFileChunk: getFileChunk,
     saveFile: saveFile,
     createFile: createFile,
     createDir: createDir,

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

@@ -207,6 +207,29 @@ var TabManager = (function () {
       }
     }).catch(function(err) {
       if (path.startsWith('diff://')) return; // diff tabs handle their own loading
+      if (err.response && err.response.status === 413) {
+        FileService.getFileChunk(path, 0, 500).then(function(data) {
+          var loadedContent = typeof data.content === 'string' ? data.content : "";
+          _updateTab(paneId, path, {
+            content: loadedContent,
+            cleanContent: loadedContent,
+            externalContentVersion: 1,
+            isImage: false,
+            fileNotFound: false,
+            dirty: false,
+            loading: false,
+            truncated: true,
+            startLine: data.start_line || 0,
+            lineCount: data.line_count || 0,
+            totalLines: data.total_lines || 0,
+            totalBytes: data.total_bytes || 0
+          });
+        }).catch(function(chunkErr) {
+          EditorStore.setStatus("Failed to load large file: " + ((chunkErr.response && chunkErr.response.data && chunkErr.response.data.error) || chunkErr.message), "error");
+          closeTab(paneId, path);
+        });
+        return;
+      }
       EditorStore.setStatus("Failed to load file: " + ((err.response && err.response.data && err.response.data.error) || err.message), "error");
       closeTab(paneId, path);
     });

data/app/controllers/mbeditor/editors_controller.rb

@@ -170,6 +170,29 @@ module Mbeditor
         return render json: { error: "Not found" }, status: :not_found
       end
 
+      start_line = params[:start_line] ? params[:start_line].to_i : nil
+      line_count  = params.key?(:line_count) ? params[:line_count].to_i : 500
+      line_count  = [line_count, 5000].min
+
+      if start_line
+        total_bytes = File.size(path)
+        chunk = []
+        total_lines = 0
+        File.foreach(path, encoding: "UTF-8", invalid: :replace, undef: :replace) do |line|
+          chunk << line if total_lines >= start_line && chunk.length < line_count
+          total_lines += 1
+        end
+        return render json: {
+          path:        relative_path(path),
+          content:     chunk.join,
+          truncated:   total_lines > start_line + chunk.length,
+          start_line:  start_line,
+          line_count:  chunk.length,
+          total_lines: total_lines,
+          total_bytes: total_bytes
+        }
+      end
+
       size = File.size(path)
       return render_file_too_large(size) if size > MAX_OPEN_FILE_SIZE_BYTES
 
@@ -314,7 +337,8 @@ module Mbeditor
                     workspace_root,
                     symbol,
                     excluded_dirnames: excluded_dirnames,
-                    excluded_paths: excluded_paths
+                    excluded_paths:    excluded_paths,
+                    included_dirs:     ruby_def_include_dirs
                   )
                   ri = RiDefinitionService.call(symbol)
                   workspace + ri
@@ -337,7 +361,8 @@ module Mbeditor
       file = RubyDefinitionService.module_defined_in(
         workspace_root, name,
         excluded_dirnames: excluded_dirnames,
-        excluded_paths:    excluded_paths
+        excluded_paths:    excluded_paths,
+        included_dirs:     ruby_def_include_dirs
       )
       return render json: { name: name, methods: [] } unless file
 
@@ -360,14 +385,16 @@ module Mbeditor
       # Fast no-op on subsequent calls (mtime checks only).
       RubyDefinitionService.scan(workspace_root,
                                  excluded_dirnames: excluded_dirnames,
-                                 excluded_paths:    excluded_paths)
+                                 excluded_paths:    excluded_paths,
+                                 included_dirs:     ruby_def_include_dirs)
 
       module_names = RubyDefinitionService.includes_in_file(path)
       includes = module_names.filter_map do |mod_name|
         mod_file = RubyDefinitionService.module_defined_in(
           workspace_root, mod_name,
           excluded_dirnames: excluded_dirnames,
-          excluded_paths:    excluded_paths
+          excluded_paths:    excluded_paths,
+          included_dirs:     ruby_def_include_dirs
         )
         next unless mod_file
 
@@ -1045,7 +1072,8 @@ module Mbeditor
         if File.directory?(full)
           { name: name, type: "folder", path: rel, children: build_tree(full, depth: depth + 1) }
         else
-          { name: name, type: "file", path: rel }
+          size = File.size(full) rescue nil
+          { name: name, type: "file", path: rel, size: size }
         end
       end
     rescue Errno::EACCES
@@ -1060,6 +1088,10 @@ module Mbeditor
       excluded_paths.filter { |path| !path.include?("/") }
     end
 
+    def ruby_def_include_dirs
+      Array(Mbeditor.configuration.ruby_def_include_dirs).map(&:to_s).reject(&:blank?)
+    end
+
     def excluded_path?(relative_path, name)
       excluded_paths.any? do |pattern|
         if pattern.include?("/")

data/app/services/mbeditor/ruby_definition_service.rb

@@ -51,10 +51,11 @@ module Mbeditor
       attr_reader :file_cache, :mutex
       attr_accessor :cache_path
 
-      def call(workspace_root, symbol, excluded_dirnames: [], excluded_paths: [])
+      def call(workspace_root, symbol, excluded_dirnames: [], excluded_paths: [], included_dirs: [])
         new(workspace_root, symbol,
             excluded_dirnames: excluded_dirnames,
-            excluded_paths: excluded_paths).call
+            excluded_paths:    excluded_paths,
+            included_dirs:     included_dirs).call
       end
 
       # Load the JSON cache from disk exactly once per process (double-checked
@@ -122,20 +123,26 @@ module Mbeditor
       # Searches the cache (and triggers a workspace scan if needed) to find
       # which file in +workspace_root+ defines the given module or class name.
       # Returns the absolute file path string or nil.
-      def module_defined_in(workspace_root, module_name, excluded_dirnames: [], excluded_paths: [])
+      def module_defined_in(workspace_root, module_name, excluded_dirnames: [], excluded_paths: [], included_dirs: [])
         load_disk_cache_once
+        root_prefix = workspace_root.to_s.chomp("/")
+        within_dirs = ->(path) {
+          return true if included_dirs.empty?
+          included_dirs.any? { |d| path.start_with?(File.join(root_prefix, d) + "/") }
+        }
         result = @mutex.synchronize do
-          @file_cache.find { |_path, entry| entry[:module_names]&.include?(module_name) }
+          @file_cache.find { |path, entry| within_dirs.call(path) && entry[:module_names]&.include?(module_name) }
         end
         return result[0] if result
 
         # Cache miss: scan workspace to populate cache entries with module_names.
         new(workspace_root, nil,
             excluded_dirnames: excluded_dirnames,
-            excluded_paths:    excluded_paths).scan_workspace
+            excluded_paths:    excluded_paths,
+            included_dirs:     included_dirs).scan_workspace
 
         result = @mutex.synchronize do
-          @file_cache.find { |_path, entry| entry[:module_names]&.include?(module_name) }
+          @file_cache.find { |path, entry| within_dirs.call(path) && entry[:module_names]&.include?(module_name) }
         end
         result ? result[0] : nil
       end
@@ -153,18 +160,20 @@ module Mbeditor
 
       # Convenience wrapper: scan the whole workspace to warm the cache.
       # Fast on subsequent calls (only re-parses files whose mtime changed).
-      def scan(workspace_root, excluded_dirnames: [], excluded_paths: [])
+      def scan(workspace_root, excluded_dirnames: [], excluded_paths: [], included_dirs: [])
         new(workspace_root, nil,
             excluded_dirnames: excluded_dirnames,
-            excluded_paths:    excluded_paths).scan_workspace
+            excluded_paths:    excluded_paths,
+            included_dirs:     included_dirs).scan_workspace
       end
     end
 
-    def initialize(workspace_root, symbol, excluded_dirnames: [], excluded_paths: [])
-      @workspace_root   = workspace_root.to_s.chomp("/")
-      @symbol           = symbol
+    def initialize(workspace_root, symbol, excluded_dirnames: [], excluded_paths: [], included_dirs: [])
+      @workspace_root    = workspace_root.to_s.chomp("/")
+      @symbol            = symbol
       @excluded_dirnames = Array(excluded_dirnames)
       @excluded_paths    = Array(excluded_paths)
+      @included_dirs     = Array(included_dirs)
     end
 
     # Walks the entire workspace and populates the per-file cache (including the
@@ -176,27 +185,29 @@ module Mbeditor
       files_scanned = 0
       evict_deleted_cache_entries
 
-      Find.find(@workspace_root) do |path|
-        if File.directory?(path)
-          dirname = File.basename(path)
-          rel_dir = relative_path(path)
-          Find.prune if path != @workspace_root && excluded_dir?(dirname, rel_dir)
-          next
-        end
-        next unless path.end_with?(".rb")
+      search_roots.each do |root|
+        Find.find(root) do |path|
+          if File.directory?(path)
+            dirname = File.basename(path)
+            rel_dir = relative_path(path)
+            Find.prune if path != root && excluded_dir?(dirname, rel_dir)
+            next
+          end
+          next unless path.end_with?(".rb")
 
-        rel = relative_path(path)
-        next if excluded_rel_path?(rel, File.basename(path))
+          rel = relative_path(path)
+          next if excluded_rel_path?(rel, File.basename(path))
 
-        files_scanned += 1
-        if files_scanned > MAX_FILES_SCANNED
-          Rails.logger.warn("[mbeditor] RubyDefinitionService: workspace exceeds #{MAX_FILES_SCANNED} .rb files; stopping scan early")
-          break
-        end
-        begin
-          cache_entry_for(path)
-        rescue StandardError
-          nil
+          files_scanned += 1
+          if files_scanned > MAX_FILES_SCANNED
+            Rails.logger.warn("[mbeditor] RubyDefinitionService: workspace exceeds #{MAX_FILES_SCANNED} .rb files; stopping scan early")
+            break
+          end
+          begin
+            cache_entry_for(path)
+          rescue StandardError
+            nil
+          end
         end
       end
 
@@ -212,46 +223,46 @@ module Mbeditor
 
       evict_deleted_cache_entries
 
-      Find.find(@workspace_root) do |path|
-        # Prune excluded directories
-        if File.directory?(path)
-          dirname = File.basename(path)
-          rel_dir = relative_path(path)
-          if path != @workspace_root && excluded_dir?(dirname, rel_dir)
-            Find.prune
+      search_roots.each do |root|
+        Find.find(root) do |path|
+          # Prune excluded directories
+          if File.directory?(path)
+            dirname = File.basename(path)
+            rel_dir = relative_path(path)
+            Find.prune if path != root && excluded_dir?(dirname, rel_dir)
+            next
           end
-          next
-        end
 
-        next unless path.end_with?(".rb")
+          next unless path.end_with?(".rb")
 
-        rel = relative_path(path)
-        next if excluded_rel_path?(rel, File.basename(path))
-
-        files_scanned += 1
-        if files_scanned > MAX_FILES_SCANNED
-          Rails.logger.warn("[mbeditor] RubyDefinitionService: workspace exceeds #{MAX_FILES_SCANNED} .rb files; stopping scan early")
-          break
-        end
+          rel = relative_path(path)
+          next if excluded_rel_path?(rel, File.basename(path))
 
-        begin
-          cached = cache_entry_for(path)
-          next unless cached
-
-          hit_lines = @symbol ? cached[:all_defs].fetch(@symbol, nil) : nil
-          next unless hit_lines && hit_lines.any?
+          files_scanned += 1
+          if files_scanned > MAX_FILES_SCANNED
+            Rails.logger.warn("[mbeditor] RubyDefinitionService: workspace exceeds #{MAX_FILES_SCANNED} .rb files; stopping scan early")
+            break
+          end
 
-          hit_lines.each do |def_line|
-            results << {
-              file:      rel,
-              line:      def_line,
-              signature: (cached[:lines][def_line - 1] || "").strip,
-              comments:  extract_comments(cached[:lines], def_line)
-            }
-            return results if results.length >= MAX_RESULTS
+          begin
+            cached = cache_entry_for(path)
+            next unless cached
+
+            hit_lines = @symbol ? cached[:all_defs].fetch(@symbol, nil) : nil
+            next unless hit_lines && hit_lines.any?
+
+            hit_lines.each do |def_line|
+              results << {
+                file:      rel,
+                line:      def_line,
+                signature: (cached[:lines][def_line - 1] || "").strip,
+                comments:  extract_comments(cached[:lines], def_line)
+              }
+              return results if results.length >= MAX_RESULTS
+            end
+          rescue StandardError
+            # Malformed file or unreadable; skip silently
           end
-        rescue StandardError
-          # Malformed file or unreadable; skip silently
         end
       end
 
@@ -386,6 +397,15 @@ module Mbeditor
       full_path.to_s.delete_prefix(@workspace_root).delete_prefix("/")
     end
 
+    def search_roots
+      return [@workspace_root] if @included_dirs.empty?
+
+      dirs = @included_dirs
+               .map { |d| File.join(@workspace_root, d) }
+               .select { |d| File.directory?(d) }
+      dirs.empty? ? [@workspace_root] : dirs
+    end
+
     def excluded_dir?(dirname, rel_dir)
       @excluded_dirnames.include?(dirname) ||
         @excluded_paths.any? do |pattern|

data/lib/mbeditor/configuration.rb

@@ -6,7 +6,8 @@ module Mbeditor
                   :redmine_enabled, :redmine_url, :redmine_api_key, :redmine_ticket_source,
                   :test_framework, :test_command, :test_timeout,
                   :authenticate_with,
-                  :lint_timeout, :base_branch_candidates, :git_timeout
+                  :lint_timeout, :base_branch_candidates, :git_timeout,
+                  :ruby_def_include_dirs
 
     def initialize
       @allowed_environments = [:development]
@@ -22,7 +23,8 @@ module Mbeditor
       @test_timeout     = 60  # seconds
       @lint_timeout     = 15  # seconds for RuboCop/haml-lint subprocesses
       @base_branch_candidates = %w[origin/develop origin/main origin/master develop main master]
-      @git_timeout      = nil # seconds; nil disables (no timeout on git subprocesses)
+      @git_timeout            = nil # seconds; nil disables (no timeout on git subprocesses)
+      @ruby_def_include_dirs  = %w[app/models app/controllers app/helpers app/concerns]
     end
   end
 end

data/lib/mbeditor/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Mbeditor
-  VERSION = "0.5.0"
+  VERSION = "0.5.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mbeditor
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.5.2
 platform: ruby
 authors:
 - Oliver Noonan