devbonzai 1.6.9 → 1.7.1

package/cli.js

@@ -382,6 +382,11 @@ function extractJavaScriptFunctions(filePath) {
       return { functions: [], classes: [] };
     }
     
+    // Skip .d.ts files, minified files, and node_modules
+    if (filePath.endsWith('.d.ts') || filePath.endsWith('.min.js') || filePath.includes('node_modules')) {
+      return { functions: [], classes: [] };
+    }
+    
     const content = fs.readFileSync(filePath, 'utf8');
     const functions = [];
     const classes = [];
@@ -413,12 +418,20 @@ function extractJavaScriptFunctions(filePath) {
         return content.substring(node.start, node.end);
       };
       
+      // Track visited nodes to avoid duplicates
+      const visitedNodes = new Set();
+      
       // Traverse AST
-      function traverse(node) {
+      function traverse(node, parentType = null) {
         if (!node) return;
         
+        // Skip if already visited (avoid processing same node twice)
+        if (visitedNodes.has(node)) return;
+        visitedNodes.add(node);
+        
         // Function declarations: function myFunc() {}
-        if (node.type === 'FunctionDeclaration' && node.id) {
+        // Skip if inside ExportNamedDeclaration (will be handled below)
+        if (node.type === 'FunctionDeclaration' && node.id && parentType !== 'ExportNamedDeclaration') {
           functions.push({
             name: node.id.name,
             content: getCode(node),
@@ -432,7 +445,6 @@ function extractJavaScriptFunctions(filePath) {
             node.init && 
             (node.init.type === 'ArrowFunctionExpression' || node.init.type === 'FunctionExpression') &&
             node.id && node.id.type === 'Identifier') {
-          const funcNode = node.init;
           const funcContent = getCode(node);
           functions.push({
             name: node.id.name,
@@ -443,7 +455,8 @@ function extractJavaScriptFunctions(filePath) {
         }
         
         // Class declarations: class User { ... }
-        if (node.type === 'ClassDeclaration' && node.id) {
+        // Skip if inside ExportNamedDeclaration (will be handled below)
+        if (node.type === 'ClassDeclaration' && node.id && parentType !== 'ExportNamedDeclaration') {
           const className = node.id.name;
           const methods = [];
           
@@ -486,6 +499,8 @@ function extractJavaScriptFunctions(filePath) {
               endLine: node.declaration.loc ? node.declaration.loc.end.line : 0,
               isExported: true
             });
+            // Mark as visited to avoid duplicate processing
+            visitedNodes.add(node.declaration);
           } else if (node.declaration.type === 'ClassDeclaration' && node.declaration.id) {
             const className = node.declaration.id.name;
             const methods = [];
@@ -517,6 +532,8 @@ function extractJavaScriptFunctions(filePath) {
               endLine: node.declaration.loc ? node.declaration.loc.end.line : 0,
               isExported: true
             });
+            // Mark as visited to avoid duplicate processing
+            visitedNodes.add(node.declaration);
           }
         }
         
@@ -525,16 +542,27 @@ function extractJavaScriptFunctions(filePath) {
           if (key === 'parent' || key === 'leadingComments' || key === 'trailingComments') continue;
           const child = node[key];
           if (Array.isArray(child)) {
-            child.forEach(traverse);
+            child.forEach(c => traverse(c, node.type));
           } else if (child && typeof child === 'object' && child.type) {
-            traverse(child);
+            traverse(child, node.type);
           }
         }
       }
       
       traverse(ast);
     } catch (parseError) {
-      console.warn('Failed to parse JavaScript/TypeScript file:', filePath, parseError.message);
+      // Silently skip parsing errors - these are expected for some files
+      // Only log if it's not in node_modules or a known problematic file type
+      if (!filePath.includes('node_modules') && !filePath.endsWith('.d.ts') && !filePath.endsWith('.min.js')) {
+        // Suppress warnings for common parsing issues
+        const errorMsg = parseError.message || '';
+        if (!errorMsg.includes('outside of function') && 
+            !errorMsg.includes('Missing initializer') &&
+            !errorMsg.includes('Export') &&
+            !errorMsg.includes('Unexpected token')) {
+          // Only log unexpected errors
+        }
+      }
       return { functions: [], classes: [] };
     }
     
@@ -594,11 +622,20 @@ function extractVueFunctions(filePath) {
         return scriptContent.substring(node.start, node.end);
       };
       
+      // Track visited nodes to avoid duplicates
+      const visitedNodes = new Set();
+      
       // Traverse AST (same logic as JavaScript parser)
-      function traverse(node) {
+      function traverse(node, parentType = null) {
         if (!node) return;
         
-        if (node.type === 'FunctionDeclaration' && node.id) {
+        // Skip if already visited (avoid processing same node twice)
+        if (visitedNodes.has(node)) return;
+        visitedNodes.add(node);
+        
+        // Function declarations: function myFunc() {}
+        // Skip if inside ExportNamedDeclaration (will be handled below)
+        if (node.type === 'FunctionDeclaration' && node.id && parentType !== 'ExportNamedDeclaration') {
           functions.push({
             name: node.id.name,
             content: getCode(node),
@@ -607,6 +644,7 @@ function extractVueFunctions(filePath) {
           });
         }
         
+        // Arrow functions: const myFunc = () => {}
         if (node.type === 'VariableDeclarator' && 
             node.init && 
             (node.init.type === 'ArrowFunctionExpression' || node.init.type === 'FunctionExpression') &&
@@ -620,7 +658,9 @@ function extractVueFunctions(filePath) {
           });
         }
         
-        if (node.type === 'ClassDeclaration' && node.id) {
+        // Class declarations: class User { ... }
+        // Skip if inside ExportNamedDeclaration (will be handled below)
+        if (node.type === 'ClassDeclaration' && node.id && parentType !== 'ExportNamedDeclaration') {
           const className = node.id.name;
           const methods = [];
           
@@ -652,6 +692,7 @@ function extractVueFunctions(filePath) {
           });
         }
         
+        // Export declarations: export function, export default
         if (node.type === 'ExportNamedDeclaration' && node.declaration) {
           if (node.declaration.type === 'FunctionDeclaration' && node.declaration.id) {
             functions.push({
@@ -661,6 +702,8 @@ function extractVueFunctions(filePath) {
               endLine: node.declaration.loc ? node.declaration.loc.end.line : 0,
               isExported: true
             });
+            // Mark as visited to avoid duplicate processing
+            visitedNodes.add(node.declaration);
           } else if (node.declaration.type === 'ClassDeclaration' && node.declaration.id) {
             const className = node.declaration.id.name;
             const methods = [];
@@ -692,23 +735,26 @@ function extractVueFunctions(filePath) {
               endLine: node.declaration.loc ? node.declaration.loc.end.line : 0,
               isExported: true
             });
+            // Mark as visited to avoid duplicate processing
+            visitedNodes.add(node.declaration);
           }
         }
         
+        // Recursively traverse children
         for (const key in node) {
           if (key === 'parent' || key === 'leadingComments' || key === 'trailingComments') continue;
           const child = node[key];
           if (Array.isArray(child)) {
-            child.forEach(traverse);
+            child.forEach(c => traverse(c, node.type));
           } else if (child && typeof child === 'object' && child.type) {
-            traverse(child);
+            traverse(child, node.type);
           }
         }
       }
       
       traverse(ast);
     } catch (parseError) {
-      console.warn('Failed to parse Vue script:', filePath, parseError.message);
+      // Silently skip parsing errors for Vue files
       return { functions: [], classes: [] };
     }
     
@@ -739,11 +785,20 @@ function listAllFiles(dir, base = '', ignorePatterns = null) {
     
     const stat = fs.statSync(fullPath);
     if (stat && stat.isDirectory()) {
+      // Skip node_modules directories explicitly
+      if (file === 'node_modules' || relativePath.includes('node_modules/')) {
+        continue;
+      }
       // Add the directory itself to results
       results.push(relativePath + '/');
       // Recursively list files inside the directory
       results = results.concat(listAllFiles(fullPath, relativePath, ignorePatterns));
     } else {
+      // Skip files in node_modules explicitly
+      if (relativePath.includes('node_modules/') || fullPath.includes('node_modules')) {
+        continue;
+      }
+      
       results.push(relativePath);
       
       // Helper function to add functions, classes, and methods as virtual files
@@ -778,7 +833,9 @@ function listAllFiles(dir, base = '', ignorePatterns = null) {
       }
       
       // Handle JavaScript/TypeScript files
-      if (file.endsWith('.js') || file.endsWith('.jsx') || file.endsWith('.ts') || file.endsWith('.tsx')) {
+      // Skip .d.ts files (TypeScript declaration files) and .min.js files (minified)
+      if ((file.endsWith('.js') || file.endsWith('.jsx') || file.endsWith('.ts') || file.endsWith('.tsx')) &&
+          !file.endsWith('.d.ts') && !file.endsWith('.min.js')) {
         const parseResult = extractJavaScriptFunctions(fullPath);
         addVirtualFiles(parseResult, relativePath);
       }

package/package.json

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