diffstalker 0.2.2 → 0.2.4

package/dist/core/ExplorerStateManager.js

@@ -2,6 +2,8 @@ import * as fs from 'node:fs';
 import * as path from 'node:path';
 import { EventEmitter } from 'node:events';
 import { getIgnoredFiles } from '../git/ignoreUtils.js';
+import { listAllFiles } from '../git/status.js';
+import * as logger from '../utils/logger.js';
 const MAX_FILE_SIZE = 1024 * 1024; // 1MB
 const WARN_FILE_SIZE = 100 * 1024; // 100KB
 /**
@@ -25,6 +27,7 @@ export class ExplorerStateManager extends EventEmitter {
     options;
     expandedPaths = new Set();
     gitStatusMap = { files: new Map(), directories: new Set() };
+    _cachedFilePaths = null;
     _state = {
         currentPath: '',
         tree: null,
@@ -61,9 +64,12 @@ export class ExplorerStateManager extends EventEmitter {
     }
     /**
      * Update git status map and refresh display.
+     * Also invalidates the file path cache so the next file finder open gets fresh data.
      */
     setGitStatus(statusMap) {
         this.gitStatusMap = statusMap;
+        // Invalidate file path cache — reload in background
+        this.loadFilePaths();
         // Refresh display to show updated status
         if (this._state.tree) {
             this.applyGitStatusToTree(this._state.tree);
@@ -153,7 +159,8 @@ export class ExplorerStateManager extends EventEmitter {
             }
             return node;
         }
-        catch {
+        catch (err) {
+            logger.warn(`Failed to build tree node for ${relativePath}: ${err instanceof Error ? err.message : err}`);
             return null;
         }
     }
@@ -211,7 +218,8 @@ export class ExplorerStateManager extends EventEmitter {
             this.collapseNode(node, children);
             node.childrenLoaded = true;
         }
-        catch {
+        catch (err) {
+            logger.warn(`Failed to read directory ${node.name}: ${err instanceof Error ? err.message : err}`);
             node.childrenLoaded = true;
             node.children = [];
         }
@@ -538,45 +546,24 @@ export class ExplorerStateManager extends EventEmitter {
         return search(this._state.tree);
     }
     /**
-     * Get all file paths in the repo (for file finder).
-     * Scans the filesystem directly to get all files, not just expanded ones.
+     * Load all file paths using git ls-files (fast, single git command).
+     * Stores result in cache for instant access by FileFinder.
      */
-    async getAllFilePaths() {
-        const paths = [];
-        const scanDir = async (dirPath) => {
-            try {
-                const fullPath = path.join(this.repoPath, dirPath);
-                const entries = await fs.promises.readdir(fullPath, { withFileTypes: true });
-                // Build list of paths for gitignore check
-                const pathsToCheck = entries.map((e) => (dirPath ? path.join(dirPath, e.name) : e.name));
-                // Get ignored files
-                const ignoredFiles = this.options.hideGitignored
-                    ? await getIgnoredFiles(this.repoPath, pathsToCheck)
-                    : new Set();
-                for (const entry of entries) {
-                    // Filter dot-prefixed hidden files
-                    if (this.options.hideHidden && entry.name.startsWith('.')) {
-                        continue;
-                    }
-                    const entryPath = dirPath ? path.join(dirPath, entry.name) : entry.name;
-                    // Filter gitignored files
-                    if (this.options.hideGitignored && ignoredFiles.has(entryPath)) {
-                        continue;
-                    }
-                    if (entry.isDirectory()) {
-                        await scanDir(entryPath);
-                    }
-                    else {
-                        paths.push(entryPath);
-                    }
-                }
-            }
-            catch {
-                // Ignore errors for individual directories
-            }
-        };
-        await scanDir('');
-        return paths;
+    async loadFilePaths() {
+        try {
+            this._cachedFilePaths = await listAllFiles(this.repoPath);
+        }
+        catch (err) {
+            logger.warn(`Failed to load file paths: ${err instanceof Error ? err.message : err}`);
+            this._cachedFilePaths = [];
+        }
+    }
+    /**
+     * Get cached file paths (for file finder).
+     * Returns empty array if not yet loaded.
+     */
+    getCachedFilePaths() {
+        return this._cachedFilePaths ?? [];
     }
     /**
      * Navigate to a specific file path in the tree.