devbonzai 2.1.9 → 2.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "devbonzai",
-  "version": "2.1.9",
+  "version": "2.2.1",
   "description": "Quickly set up a local file server in any repository for browser-based file access",
   "main": "cli.js",
   "bin": {
@@ -26,6 +26,8 @@
     "cors": "^2.8.5",
     "body-parser": "^1.20.2",
     "raw-body": "^2.5.2",
-    "glob": "^10.0.0"
+    "glob": "^10.0.0",
+    "eslint": "^8.57.0",
+    "madge": "^6.1.0"
   }
 }

package/templates/handlers/scan_code_quality.js

@@ -1,24 +1,36 @@
 const fs = require('fs');
 const path = require('path');
 const glob = require('glob');
+const { ESLint } = require('eslint');
+const madge = require('madge');
+const { getIgnorePatterns, shouldIgnore } = require('../utils/ignore');
 
 module.exports = async function scanCodeQuality(req, res) {
   try {
-    const { projectPath, maxFileLines = 500, maxFolderFiles = 20 } = req.body;
+    const { 
+      projectPath, 
+      maxFileLines = 500, 
+      maxFolderFiles = 20,
+      checkUnusedImports = true,
+      checkDeadCode = true,
+      checkCircularDeps = false
+    } = req.body;
     
     if (!projectPath) {
       return res.status(400).json({ error: 'projectPath required' });
     }
     
+    const ignorePatterns = getIgnorePatterns();
     const issues = [];
     
     // 1. Check file sizes
-    const files = glob.sync('**/*.{js,ts,jsx,tsx,py,java,go}', { 
+    const allFiles = glob.sync('**/*.{js,ts,jsx,tsx,py,java,go}', { 
       cwd: projectPath,
-      nodir: true,
-      ignore: ['node_modules/**', 'dist/**', 'build/**', '.git/**']
+      nodir: true
     });
     
+    const files = allFiles.filter(file => !shouldIgnore(file, ignorePatterns));
+    
     files.forEach(file => {
       const fullPath = path.join(projectPath, file);
       const content = fs.readFileSync(fullPath, 'utf8');
@@ -35,11 +47,12 @@ module.exports = async function scanCodeQuality(req, res) {
     });
     
     // 2. Check folder sizes
-    const folders = glob.sync('**/', { 
-      cwd: projectPath,
-      ignore: ['node_modules/**', 'dist/**', 'build/**', '.git/**']
+    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);
@@ -58,6 +71,79 @@ module.exports = async function scanCodeQuality(req, res) {
       }
     });
     
+    // 3. Check unused imports and dead code with ESLint
+    if (checkUnusedImports || checkDeadCode) {
+      try {
+        const eslint = new ESLint({
+          cwd: projectPath,
+          useEslintrc: false,
+          overrideConfig: {
+            env: {
+              browser: true,
+              node: true,
+              es6: true
+            },
+            parserOptions: {
+              ecmaVersion: 2021,
+              sourceType: 'module',
+              ecmaFeatures: { jsx: true }
+            },
+            rules: {
+              'no-unused-vars': checkDeadCode ? 'error' : 'off',
+              '@typescript-eslint/no-unused-vars': checkDeadCode ? 'error' : 'off'
+            }
+          }
+        });
+        
+        const jsFiles = files.filter(f => /\.(js|jsx|ts|tsx)$/.test(f));
+        const results = await eslint.lintFiles(jsFiles.map(f => path.join(projectPath, f)));
+        
+        results.forEach(result => {
+          const relativePath = path.relative(projectPath, result.filePath);
+          
+          result.messages.forEach(msg => {
+            if (msg.ruleId === 'no-unused-vars' || msg.ruleId === '@typescript-eslint/no-unused-vars') {
+              issues.push({
+                type: 'unused_variable',
+                file: relativePath,
+                line: msg.line,
+                variable: msg.message.match(/'(.+?)'/)?.[1] || 'unknown',
+                message: msg.message
+              });
+            }
+          });
+        });
+      } catch (eslintError) {
+        console.error('ESLint scan error:', eslintError);
+        // Don't fail entire scan if ESLint fails
+      }
+    }
+    
+    // 4. Check circular dependencies
+    if (checkCircularDeps) {
+      try {
+        const result = await madge(projectPath, {
+          fileExtensions: ['js', 'jsx', 'ts', 'tsx'],
+          excludeRegExp: ignorePatterns.map(p => new RegExp(p))
+        });
+        
+        const circular = result.circular();
+        
+        if (circular.length > 0) {
+          circular.forEach(cycle => {
+            issues.push({
+              type: 'circular_dependency',
+              files: cycle,
+              message: `Circular dependency: ${cycle.join(' → ')}`
+            });
+          });
+        }
+      } catch (madgeError) {
+        console.error('Madge scan error:', madgeError);
+        // Don't fail entire scan if madge fails
+      }
+    }
+    
     res.json({ issues });
     
   } catch (error) {
@@ -65,4 +151,3 @@ module.exports = async function scanCodeQuality(req, res) {
     res.status(500).json({ error: error.message });
   }
 };
-