ctxman 3.0.0 → 3.1.1

package/lib/utils/error-handler.js

@@ -6,140 +6,140 @@
 import fs from 'fs';
 
 class ErrorHandler {
-    constructor(options = {}) {
-        this.verbose = options.verbose || false;
-        this.logFile = options.logFile || null;
+  constructor(options = {}) {
+    this.verbose = options.verbose || false;
+    this.logFile = options.logFile || null;
+  }
+
+  /**
+   * Handle format errors
+   */
+  handleFormatError(error, format) {
+    const message = `Format error (${format}): ${error.message}`;
+
+    if (this.verbose) {
+      console.error('❌ ' + message);
+      console.error('   Stack:', error.stack);
+    } else {
+      console.error('❌ ' + message);
+      console.error('   Use --verbose for detailed error information');
     }
 
-    /**
-     * Handle format errors
-     */
-    handleFormatError(error, format) {
-        const message = `Format error (${format}): ${error.message}`;
+    this.logError(message, error);
+  }
 
-        if (this.verbose) {
-            console.error('❌ ' + message);
-            console.error('   Stack:', error.stack);
-        } else {
-            console.error('❌ ' + message);
-            console.error('   Use --verbose for detailed error information');
-        }
-
-        this.logError(message, error);
-    }
-
-    /**
-     * Handle file errors
-     */
-    handleFileError(error, filePath) {
-        const message = `File error (${filePath}): ${error.message}`;
+  /**
+   * Handle file errors
+   */
+  handleFileError(error, filePath) {
+    const message = `File error (${filePath}): ${error.message}`;
 
-        console.error('❌ ' + message);
+    console.error('❌ ' + message);
 
-        // Provide helpful suggestions
-        if (error.code === 'ENOENT') {
-            console.error('   Suggestion: Check if the file path is correct');
-        } else if (error.code === 'EACCES') {
-            console.error('   Suggestion: Check file permissions');
-        }
-
-        this.logError(message, error);
+    // Provide helpful suggestions
+    if (error.code === 'ENOENT') {
+      console.error('   Suggestion: Check if the file path is correct');
+    } else if (error.code === 'EACCES') {
+      console.error('   Suggestion: Check file permissions');
     }
 
-    /**
-     * Handle parse errors
-     */
-    handleParseError(error, format, content) {
-        const message = `Parse error (${format}): ${error.message}`;
+    this.logError(message, error);
+  }
 
-        console.error('❌ ' + message);
+  /**
+   * Handle parse errors
+   */
+  handleParseError(error, format, content) {
+    const message = `Parse error (${format}): ${error.message}`;
 
-        if (this.verbose && content) {
-            console.error('   Content preview:', content.substring(0, 200) + '...');
-        }
+    console.error('❌ ' + message);
 
-        this.logError(message, error);
+    if (this.verbose && content) {
+      console.error('   Content preview:', content.substring(0, 200) + '...');
     }
 
-    /**
-     * Handle validation errors
-     */
-    handleValidationError(errors, context) {
-        console.error(`❌ Validation failed (${context}):`);
+    this.logError(message, error);
+  }
 
-        for (const error of errors) {
-            console.error(`   • ${error}`);
-        }
+  /**
+   * Handle validation errors
+   */
+  handleValidationError(errors, context) {
+    console.error(`❌ Validation failed (${context}):`);
 
-        this.logError(`Validation errors in ${context}`, { errors });
+    for (const error of errors) {
+      console.error(`   • ${error}`);
     }
 
-    /**
-     * Log error to file (if configured)
-     */
-    logError(message, error) {
-        if (!this.logFile) return;
+    this.logError(`Validation errors in ${context}`, { errors });
+  }
 
-        try {
-            const timestamp = new Date().toISOString();
-            const logEntry = `[${timestamp}] ${message}\n${error.stack || ''}\n\n`;
+  /**
+   * Log error to file (if configured)
+   */
+  logError(message, error) {
+    if (!this.logFile) return;
 
-            fs.appendFileSync(this.logFile, logEntry, 'utf8');
-        } catch (logError) {
-            // Silently fail if logging fails
-        }
-    }
+    try {
+      const timestamp = new Date().toISOString();
+      const logEntry = `[${timestamp}] ${message}\n${error.stack || ''}\n\n`;
 
-    /**
-     * Wrap async function with error handling
-     */
-    wrapAsync(fn, context) {
-        return async (...args) => {
-            try {
-                return await fn(...args);
-            } catch (error) {
-                console.error(`❌ Error in ${context}:`, error.message);
-                if (this.verbose) {
-                    console.error('   Stack:', error.stack);
-                }
-                this.logError(`Error in ${context}`, error);
-                throw error;
-            }
-        };
+      fs.appendFileSync(this.logFile, logEntry, 'utf8');
+    } catch (_logError) {
+      // Silently fail if logging fails
     }
-
-    /**
-     * Validate format support
-     */
-    validateFormat(format, supportedFormats) {
-        if (!supportedFormats.includes(format)) {
-            const message = `Unsupported format: ${format}`;
-            const suggestion = `Supported formats: ${supportedFormats.join(', ')}`;
-
-            console.error(`❌ ${message}`);
-            console.error(`   ${suggestion}`);
-
-            throw new Error(message);
+  }
+
+  /**
+   * Wrap async function with error handling
+   */
+  wrapAsync(fn, context) {
+    return async (...args) => {
+      try {
+        return await fn(...args);
+      } catch (error) {
+        console.error(`❌ Error in ${context}:`, error.message);
+        if (this.verbose) {
+          console.error('   Stack:', error.stack);
         }
+        this.logError(`Error in ${context}`, error);
+        throw error;
+      }
+    };
+  }
+
+  /**
+   * Validate format support
+   */
+  validateFormat(format, supportedFormats) {
+    if (!supportedFormats.includes(format)) {
+      const message = `Unsupported format: ${format}`;
+      const suggestion = `Supported formats: ${supportedFormats.join(', ')}`;
+
+      console.error(`❌ ${message}`);
+      console.error(`   ${suggestion}`);
+
+      throw new Error(message);
     }
-
-    /**
-     * Create user-friendly error message
-     */
-    createUserMessage(error, context = '') {
-        const prefix = context ? `${context}: ` : '';
-
-        // Map common errors to user-friendly messages
-        const errorMap = {
-            'ENOENT': 'File not found',
-            'EACCES': 'Permission denied',
-            'EISDIR': 'Expected file, got directory',
-            'ENOTDIR': 'Expected directory, got file'
-        };
-
-        const userMessage = errorMap[error.code] || error.message;
-        return `${prefix}${userMessage}`;
-    }
+  }
+
+  /**
+   * Create user-friendly error message
+   */
+  createUserMessage(error, context = '') {
+    const prefix = context ? `${context}: ` : '';
+
+    // Map common errors to user-friendly messages
+    const errorMap = {
+      ENOENT: 'File not found',
+      EACCES: 'Permission denied',
+      EISDIR: 'Expected file, got directory',
+      ENOTDIR: 'Expected directory, got file',
+    };
+
+    const userMessage = errorMap[error.code] || error.message;
+    return `${prefix}${userMessage}`;
+  }
 }
 
 export default ErrorHandler;

package/lib/utils/file-utils.js

@@ -5,85 +5,169 @@
 
 import path from 'path';
 
-class FileUtils {
-    /**
-     * Check if file is a text file
-     * @param {string} filePath - File path
-     * @returns {boolean}
-     */
-    static isText(filePath) {
-        const ext = path.extname(filePath).toLowerCase();
-        const basename = path.basename(filePath).toLowerCase();
+// Dependency, VCS and build-output directories that are never scanned, whatever the
+// ignore files say
+const SKIPPED_DIRECTORIES = new Set([
+  'node_modules',
+  '.git',
+  '.svn',
+  '.hg',
+  'coverage',
+  'dist',
+  'build',
+]);
 
-        const textExtensions = new Set([
-            '.js', '.ts', '.jsx', '.tsx', '.json', '.md', '.txt', '.yml', '.yaml',
-            '.html', '.css', '.scss', '.sass', '.less', '.xml', '.svg',
-            '.sh', '.bash', '.zsh', '.py', '.rb', '.php', '.java', '.c', '.cpp', '.cc', '.h', '.hpp',
-            '.go', '.rs', '.swift', '.kt', '.kts', '.cs', '.scala', '.sql', '.toml', '.ini', '.conf'
-        ]);
+class FileUtils {
+  /**
+   * Check if file is a text file
+   * @param {string} filePath - File path
+   * @returns {boolean}
+   */
+  static isText(filePath) {
+    const ext = path.extname(filePath).toLowerCase();
+    const basename = path.basename(filePath).toLowerCase();
 
-        const textFiles = ['dockerfile', 'makefile', 'license', 'readme', 'changelog'];
+    const textExtensions = new Set([
+      '.js',
+      '.ts',
+      '.jsx',
+      '.tsx',
+      '.json',
+      '.md',
+      '.txt',
+      '.yml',
+      '.yaml',
+      '.html',
+      '.css',
+      '.scss',
+      '.sass',
+      '.less',
+      '.xml',
+      '.svg',
+      '.sh',
+      '.bash',
+      '.zsh',
+      '.py',
+      '.rb',
+      '.php',
+      '.java',
+      '.c',
+      '.cpp',
+      '.cc',
+      '.h',
+      '.hpp',
+      '.go',
+      '.rs',
+      '.swift',
+      '.kt',
+      '.kts',
+      '.cs',
+      '.scala',
+      '.sql',
+      '.toml',
+      '.ini',
+      '.conf',
+    ]);
 
-        return textExtensions.has(ext) ||
-               textFiles.some(name => basename.includes(name));
-    }
+    const textFiles = ['dockerfile', 'makefile', 'license', 'readme', 'changelog'];
 
-    /**
-     * Check if file is a code file (supports method extraction)
-     * @param {string} filePath - File path
-     * @returns {boolean}
-     */
-    static isCode(filePath) {
-        const ext = path.extname(filePath).toLowerCase();
-        return [
-            '.js', '.ts', '.jsx', '.tsx',     // JavaScript/TypeScript
-            '.rs',                             // Rust
-            '.cs',                             // C#
-            '.go',                             // Go
-            '.java',                           // Java
-            '.py',                             // Python
-            '.php',                            // PHP
-            '.rb',                             // Ruby
-            '.kt', '.kts',                     // Kotlin
-            '.swift',                          // Swift
-            '.c', '.cpp', '.cc', '.h', '.hpp', // C/C++
-            '.scala'                           // Scala
-        ].includes(ext);
-    }
+    return textExtensions.has(ext) || textFiles.some((name) => basename.includes(name));
+  }
 
-    /**
-     * Get file type category
-     * @param {string} filePath - File path
-     * @returns {string} Type: 'code', 'config', 'doc', 'style', 'other'
-     */
-    static getType(filePath) {
-        const ext = path.extname(filePath).toLowerCase();
+  /**
+   * Check if file is a code file (supports method extraction)
+   * @param {string} filePath - File path
+   * @returns {boolean}
+   */
+  static isCode(filePath) {
+    const ext = path.extname(filePath).toLowerCase();
+    return [
+      '.js',
+      '.ts',
+      '.jsx',
+      '.tsx', // JavaScript/TypeScript
+      '.rs', // Rust
+      '.cs', // C#
+      '.go', // Go
+      '.java', // Java
+      '.py', // Python
+      '.php', // PHP
+      '.rb', // Ruby
+      '.kt',
+      '.kts', // Kotlin
+      '.swift', // Swift
+      '.c',
+      '.cpp',
+      '.cc',
+      '.h',
+      '.hpp', // C/C++
+      '.scala', // Scala
+    ].includes(ext);
+  }
 
-        const categories = {
-            code: ['.js', '.ts', '.jsx', '.tsx', '.py', '.rb', '.php', '.java', '.c', '.cpp', '.cc', '.h', '.hpp', '.go', '.rs', '.cs', '.kt', '.kts', '.swift', '.scala'],
-            config: ['.json', '.yml', '.yaml', '.toml', '.ini', '.conf', '.xml'],
-            doc: ['.md', '.txt', '.pdf', '.doc', '.docx'],
-            style: ['.css', '.scss', '.sass', '.less'],
-        };
+  /**
+   * Get file type category
+   * @param {string} filePath - File path
+   * @returns {string} Type: 'code', 'config', 'doc', 'style', 'other'
+   */
+  static getType(filePath) {
+    const ext = path.extname(filePath).toLowerCase();
 
-        for (const [type, extensions] of Object.entries(categories)) {
-            if (extensions.includes(ext)) {
-                return type;
-            }
-        }
+    const categories = {
+      code: [
+        '.js',
+        '.ts',
+        '.jsx',
+        '.tsx',
+        '.py',
+        '.rb',
+        '.php',
+        '.java',
+        '.c',
+        '.cpp',
+        '.cc',
+        '.h',
+        '.hpp',
+        '.go',
+        '.rs',
+        '.cs',
+        '.kt',
+        '.kts',
+        '.swift',
+        '.scala',
+      ],
+      config: ['.json', '.yml', '.yaml', '.toml', '.ini', '.conf', '.xml'],
+      doc: ['.md', '.txt', '.pdf', '.doc', '.docx'],
+      style: ['.css', '.scss', '.sass', '.less'],
+    };
 
-        return 'other';
+    for (const [type, extensions] of Object.entries(categories)) {
+      if (extensions.includes(ext)) {
+        return type;
+      }
     }
 
-    /**
-     * Get file extension without dot
-     * @param {string} filePath - File path
-     * @returns {string} Extension or 'no-extension'
-     */
-    static getExtension(filePath) {
-        const ext = path.extname(filePath).toLowerCase();
-        return ext ? ext.slice(1) : 'no-extension';
-    }
+    return 'other';
+  }
+
+  /**
+   * Check if a directory is always skipped when scanning a project
+   * @param {string} name - Directory name (not a path)
+   * @returns {boolean}
+   */
+  static isSkippedDirectory(name) {
+    return SKIPPED_DIRECTORIES.has(name);
+  }
+
+  /**
+   * Get file extension without dot
+   * @param {string} filePath - File path
+   * @returns {string} Extension or 'no-extension'
+   */
+  static getExtension(filePath) {
+    const ext = path.extname(filePath).toLowerCase();
+    return ext ? ext.slice(1) : 'no-extension';
+  }
 }
 
 export default FileUtils;