ftreeview 0.1.1 → 0.1.3

package/src/hooks/useIgnore.js

@@ -1,182 +1,139 @@
-/**
- * useIgnore - Gitignore Filtering Hook
- *
- * Provides gitignore-style filtering functionality using the 'ignore' package.
- * Supports loading .gitignore files from the root and nested directories.
- */
-
-import { existsSync, readdirSync, statSync, readFileSync } from 'node:fs';
-import { resolve, join, relative, sep } from 'node:path';
-import ignore from 'ignore';
-
-/**
- * Default patterns that are always applied (unless --no-ignore is set)
- */
-const DEFAULT_PATTERNS = [
-  '.git',
-  '.git/',
-  '.git/**',
-  'node_modules',
-  'node_modules/',
-];
-
-/**
- * Finds all .gitignore files in a directory tree
- *
- * @param {string} rootPath - Root directory path
- * @returns {Map<string, string>} Map of directory paths to their .gitignore content
- */
-function findGitignoreFiles(rootPath) {
-  const gitignoreMap = new Map();
-
-  /**
-   * Recursively scan for .gitignore files
-   * @param {string} dirPath - Current directory to scan
-   */
-  function scanDirectory(dirPath) {
-    try {
-      const entries = readdirSync(dirPath, { withFileTypes: true });
-
-      // Check if this directory has a .gitignore file
-      const gitignorePath = join(dirPath, '.gitignore');
-      if (existsSync(gitignorePath)) {
-        // We'll read these lazily when needed to avoid excessive I/O
-        gitignoreMap.set(dirPath, gitignorePath);
-      }
-
-      // Recurse into subdirectories
-      for (const entry of entries) {
-        if (entry.isDirectory() && entry.name !== '.git') {
-          const subPath = join(dirPath, entry.name);
-          scanDirectory(subPath);
-        }
-      }
-    } catch (error) {
-      // Skip directories we can't read (permission errors, etc.)
-    }
-  }
-
-  scanDirectory(rootPath);
-  return gitignoreMap;
-}
-
-/**
- * Creates an ignore filter function based on .gitignore files
- *
- * @param {string} rootPath - Root directory path
- * @param {object} options - Configuration options
- * @param {boolean} [options.showIgnored=false] - If true, don't filter anything (--no-ignore)
- * @param {string[]} [options.extraIgnorePatterns] - Additional patterns to ignore
- * @returns {object} Object with shouldIgnore method and addPattern method
- */
-export function createIgnoreFilter(rootPath, options = {}) {
-  const { showIgnored = false, extraIgnorePatterns = [] } = options;
-
-  // If showIgnored is true, return a pass-through filter
-  if (showIgnored) {
-    return {
-      shouldIgnore: (relativePath) => false,
-      addPattern: () => {},
-      filter: (paths) => paths,
-    };
-  }
-
-  // Initialize the ignore instance
-  const ig = ignore();
-
-  // Add default patterns (unless showIgnored is true)
-  for (const pattern of DEFAULT_PATTERNS) {
-    ig.add(pattern);
-  }
-
-  // Add extra patterns from options
-  if (extraIgnorePatterns.length > 0) {
-    ig.add(extraIgnorePatterns);
-  }
-
-  // Find all .gitignore files
-  const gitignoreMap = findGitignoreFiles(rootPath);
-
-  // Track which .gitignore files we've already loaded
-  const loadedIgnores = new Set();
-
-  /**
-   * Ensures .gitignore patterns for a given path are loaded
-   * This implements lazy loading - we only load patterns when we encounter a directory
-   *
-   * @param {string} dirAbsPath - Absolute path to directory
-   */
-  function loadPatternsForDirectory(dirAbsPath) {
-    if (loadedIgnores.has(dirAbsPath)) {
-      return; // Already loaded
-    }
-
-    const gitignorePath = gitignoreMap.get(dirAbsPath);
-    if (gitignorePath && existsSync(gitignorePath)) {
-      try {
-        // Read and parse the .gitignore file
-        const content = readFileSync(gitignorePath, 'utf-8');
-        ig.add(content);
-        loadedIgnores.add(dirAbsPath);
-      } catch (error) {
-        // Silently ignore read errors
-      }
-    }
-  }
-
-  /**
-   * Checks if a relative path should be ignored
-   *
-   * @param {string} relativePath - Path relative to root (forward slashes)
-   * @returns {boolean} True if the path should be ignored
-   */
-  function shouldIgnore(relativePath) {
-    // Empty string (root directory) is never ignored
-    if (relativePath === '') {
-      return false;
-    }
-
-    // Normalize path to use forward slashes
-    const normalizedPath = relativePath.split(sep).join('/');
-
-    return ig.ignores(normalizedPath);
-  }
-
-  /**
-   * Filters an array of paths, keeping only non-ignored ones
-   *
-   * @param {string[]} paths - Array of relative paths
-   * @returns {string[]} Filtered array of paths
-   */
-  function filter(paths) {
-    return ig.filter(paths);
-  }
-
-  /**
-   * Adds a pattern to the ignore filter
-   *
-   * @param {string|string[]} pattern - Pattern or array of patterns to add
-   */
-  function addPattern(pattern) {
-    ig.add(pattern);
-  }
-
-  return {
-    shouldIgnore,
-    filter,
-    addPattern,
-    loadPatternsForDirectory,
-  };
-}
-
-/**
- * React hook for creating an ignore filter
- * This is a convenience wrapper around createIgnoreFilter
- *
- * @param {string} rootPath - Root directory path
- * @param {object} options - Configuration options
- * @returns {object} Filter object with shouldIgnore method
- */
-export function useIgnore(rootPath, options = {}) {
-  return createIgnoreFilter(rootPath, options);
-}
+/**
+ * useIgnore - Gitignore Filtering Hook
+ *
+ * Provides gitignore-style filtering functionality using the 'ignore' package.
+ * Supports loading .gitignore files from the root and nested directories.
+ */
+
+import { existsSync, readFileSync } from 'node:fs';
+import { join, sep } from 'node:path';
+import ignore from 'ignore';
+
+/**
+ * Default patterns that are always applied (unless --no-ignore is set)
+ */
+const DEFAULT_PATTERNS = [
+  '.git',
+  '.git/',
+  '.git/**',
+  'node_modules',
+  'node_modules/',
+];
+
+/**
+ * Creates an ignore filter function based on .gitignore files
+ *
+ * @param {string} rootPath - Root directory path
+ * @param {object} options - Configuration options
+ * @param {boolean} [options.showIgnored=false] - If true, don't filter anything (--no-ignore)
+ * @param {string[]} [options.extraIgnorePatterns] - Additional patterns to ignore
+ * @returns {object} Object with shouldIgnore method and addPattern method
+ */
+export function createIgnoreFilter(rootPath, options = {}) {
+  const { showIgnored = false, extraIgnorePatterns = [] } = options;
+
+  // If showIgnored is true, return a pass-through filter
+  if (showIgnored) {
+    return {
+      shouldIgnore: (relativePath) => false,
+      addPattern: () => {},
+      filter: (paths) => paths,
+    };
+  }
+
+  // Initialize the ignore instance
+  const ig = ignore();
+
+  // Add default patterns (unless showIgnored is true)
+  for (const pattern of DEFAULT_PATTERNS) {
+    ig.add(pattern);
+  }
+
+  // Add extra patterns from options
+  if (extraIgnorePatterns.length > 0) {
+    ig.add(extraIgnorePatterns);
+  }
+
+  // Track which .gitignore files we've already loaded
+  const loadedIgnores = new Set();
+
+  /**
+   * Ensures .gitignore patterns for a given path are loaded
+   * This implements lazy loading - we only load patterns when we encounter a directory
+   *
+   * @param {string} dirAbsPath - Absolute path to directory
+   */
+  function loadPatternsForDirectory(dirAbsPath) {
+    if (loadedIgnores.has(dirAbsPath)) {
+      return; // Already loaded
+    }
+
+    const gitignorePath = join(dirAbsPath, '.gitignore');
+    if (existsSync(gitignorePath)) {
+      try {
+        // Read and parse the .gitignore file
+        const content = readFileSync(gitignorePath, 'utf-8');
+        ig.add(content);
+        loadedIgnores.add(dirAbsPath);
+      } catch (error) {
+        // Silently ignore read errors
+      }
+    }
+  }
+
+  /**
+   * Checks if a relative path should be ignored
+   *
+   * @param {string} relativePath - Path relative to root (forward slashes)
+   * @returns {boolean} True if the path should be ignored
+   */
+  function shouldIgnore(relativePath) {
+    // Empty string (root directory) is never ignored
+    if (relativePath === '') {
+      return false;
+    }
+
+    // Normalize path to use forward slashes
+    const normalizedPath = relativePath.split(sep).join('/');
+
+    return ig.ignores(normalizedPath);
+  }
+
+  /**
+   * Filters an array of paths, keeping only non-ignored ones
+   *
+   * @param {string[]} paths - Array of relative paths
+   * @returns {string[]} Filtered array of paths
+   */
+  function filter(paths) {
+    return ig.filter(paths);
+  }
+
+  /**
+   * Adds a pattern to the ignore filter
+   *
+   * @param {string|string[]} pattern - Pattern or array of patterns to add
+   */
+  function addPattern(pattern) {
+    ig.add(pattern);
+  }
+
+  return {
+    shouldIgnore,
+    filter,
+    addPattern,
+    loadPatternsForDirectory,
+  };
+}
+
+/**
+ * React hook for creating an ignore filter
+ * This is a convenience wrapper around createIgnoreFilter
+ *
+ * @param {string} rootPath - Root directory path
+ * @param {object} options - Configuration options
+ * @returns {object} Filter object with shouldIgnore method
+ */
+export function useIgnore(rootPath, options = {}) {
+  return createIgnoreFilter(rootPath, options);
+}

package/src/hooks/useNavigation.js

@@ -19,9 +19,10 @@ const VIEWPORT_MARGIN = 3; // Keep cursor 3 lines from viewport edges
  * @param {FileNode[]} flatList - Flattened list of visible nodes from useTree
  * @param {Function} rebuildTree - Callback to rebuild tree with modified nodes
  * @param {Function|null} refreshFlatList - Callback to refresh visible list without full rebuild
+ * @param {Function|null} ensureChildrenLoaded - Callback to lazy-load directory children when expanding
  * @returns {{cursor: number, viewportStart: number, exit: Function, setCursorByPath: Function, setCursor: Function}} Navigation state
  */
-export function useNavigation(flatList = [], rebuildTree, refreshFlatList = null) {
+export function useNavigation(flatList = [], rebuildTree, refreshFlatList = null, ensureChildrenLoaded = null) {
   const { exit } = useApp();
 
   // Calculate viewport height (terminal height minus header and status bar)
@@ -108,10 +109,13 @@ export function useNavigation(flatList = [], rebuildTree, refreshFlatList = null
 
     // Only directories can be expanded
     if (node.isDir && !node.expanded) {
+      if (typeof ensureChildrenLoaded === 'function') {
+        ensureChildrenLoaded(node);
+      }
       node.expanded = true;
       refreshVisibleTree();
     }
-  }, [cursor, flatList, refreshVisibleTree]);
+  }, [cursor, flatList, refreshVisibleTree, ensureChildrenLoaded]);
 
   /**
    * Smart collapse: collapse if expanded, jump to parent if collapsed
@@ -144,9 +148,13 @@ export function useNavigation(flatList = [], rebuildTree, refreshFlatList = null
     const node = flatList[cursor];
     if (!node || !node.isDir) return;
 
-    node.expanded = !node.expanded;
+    const nextExpanded = !node.expanded;
+    if (nextExpanded && typeof ensureChildrenLoaded === 'function') {
+      ensureChildrenLoaded(node);
+    }
+    node.expanded = nextExpanded;
     refreshVisibleTree();
-  }, [cursor, flatList, refreshVisibleTree]);
+  }, [cursor, flatList, refreshVisibleTree, ensureChildrenLoaded]);
 
   /**
    * Jump to the first item in the list