devbonzai 2.2.3 → 2.2.201

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "devbonzai",
-  "version": "2.2.3",
+  "version": "2.2.201",
   "description": "Quickly set up a local file server in any repository for browser-based file access",
   "main": "cli.js",
   "bin": {

package/templates/handlers/scan_code_quality.js

@@ -11,9 +11,11 @@ module.exports = async function scanCodeQuality(req, res) {
       projectPath, 
       maxFileLines = 500, 
       maxFolderFiles = 20,
-      checkUnusedImports = true,
-      checkDeadCode = true,
-      checkCircularDeps = false
+      willEnforceLines = true,
+      willEnforceFolders = true,
+      willFlagUnusedImports = true,
+      willFlagDeadCode = true,
+      willFlagCircularDeps = false
     } = req.body;
     
     if (!projectPath) {
@@ -23,56 +25,76 @@ module.exports = async function scanCodeQuality(req, res) {
     const ignorePatterns = getIgnorePatterns();
     const issues = [];
     
-    // 1. Check file sizes
+    // Common build/output directories to always ignore
+    const defaultIgnoreGlobs = [
+      '**/node_modules/**',
+      '**/dist/**',
+      '**/build/**',
+      '**/static/**',
+      '**/out/**',
+      '**/.next/**',
+      '**/*.min.js',
+      '**/*.bundle.js',
+      '**/*.chunk.js'
+    ];
+    
+    // Get all files (needed for multiple checks)
     const allFiles = glob.sync('**/*.{js,ts,jsx,tsx,py,java,go}', { 
       cwd: projectPath,
-      nodir: true
+      nodir: true,
+      ignore: defaultIgnoreGlobs
     });
     
     const files = allFiles.filter(file => !shouldIgnore(file, ignorePatterns));
     
-    files.forEach(file => {
-      const fullPath = path.join(projectPath, file);
-      const content = fs.readFileSync(fullPath, 'utf8');
-      const lines = content.split('\n').length;
-      
-      if (lines > maxFileLines) {
-        issues.push({
-          type: 'large_file',
-          file: file,
-          lines: lines,
-          limit: maxFileLines
-        });
-      }
-    });
+    // 1. Check file sizes
+    if (willEnforceLines) {
+      files.forEach(file => {
+        const fullPath = path.join(projectPath, file);
+        const content = fs.readFileSync(fullPath, 'utf8');
+        const lines = content.split('\n').length;
+        
+        if (lines > maxFileLines) {
+          issues.push({
+            type: 'large_file',
+            file: file,
+            lines: lines,
+            limit: maxFileLines
+          });
+        }
+      });
+    }
     
     // 2. Check folder sizes
-    const allFolders = glob.sync('**/', { 
-      cwd: projectPath
-    });
-    
-    const folders = allFolders.filter(folder => !shouldIgnore(folder, ignorePatterns));
-    
-    folders.forEach(folder => {
-      const fullPath = path.join(projectPath, folder);
-      const contents = fs.readdirSync(fullPath);
-      const fileCount = contents.filter(item => {
-        const itemPath = path.join(fullPath, item);
-        return fs.statSync(itemPath).isFile();
-      }).length;
+    if (willEnforceFolders) {
+      const allFolders = glob.sync('**/', { 
+        cwd: projectPath,
+        ignore: defaultIgnoreGlobs
+      });
       
-      if (fileCount > maxFolderFiles) {
-        issues.push({
-          type: 'crowded_folder',
-          folder: folder,
-          count: fileCount,
-          limit: maxFolderFiles
-        });
-      }
-    });
+      const folders = allFolders.filter(folder => !shouldIgnore(folder, ignorePatterns));
+      
+      folders.forEach(folder => {
+        const fullPath = path.join(projectPath, folder);
+        const contents = fs.readdirSync(fullPath);
+        const fileCount = contents.filter(item => {
+          const itemPath = path.join(fullPath, item);
+          return fs.statSync(itemPath).isFile();
+        }).length;
+        
+        if (fileCount > maxFolderFiles) {
+          issues.push({
+            type: 'crowded_folder',
+            folder: folder,
+            count: fileCount,
+            limit: maxFolderFiles
+          });
+        }
+      });
+    }
     
     // 3. Check unused imports and dead code with ESLint
-    if (checkUnusedImports || checkDeadCode) {
+    if (willFlagUnusedImports || willFlagDeadCode) {
       try {
         const eslint = new ESLint({
           cwd: projectPath,
@@ -89,8 +111,7 @@ module.exports = async function scanCodeQuality(req, res) {
               ecmaFeatures: { jsx: true }
             },
             rules: {
-              'no-unused-vars': checkDeadCode ? 'error' : 'off',
-              '@typescript-eslint/no-unused-vars': checkDeadCode ? 'error' : 'off'
+              'no-unused-vars': willFlagDeadCode ? 'error' : 'off'
             }
           }
         });
@@ -102,7 +123,7 @@ module.exports = async function scanCodeQuality(req, res) {
           const relativePath = path.relative(projectPath, result.filePath);
           
           result.messages.forEach(msg => {
-            if (msg.ruleId === 'no-unused-vars' || msg.ruleId === '@typescript-eslint/no-unused-vars') {
+            if (msg.ruleId === 'no-unused-vars') {
               issues.push({
                 type: 'unused_variable',
                 file: relativePath,
@@ -120,7 +141,7 @@ module.exports = async function scanCodeQuality(req, res) {
     }
     
     // 4. Check circular dependencies
-    if (checkCircularDeps) {
+    if (willFlagCircularDeps) {
       try {
         const result = await madge(projectPath, {
           fileExtensions: ['js', 'jsx', 'ts', 'tsx'],