@davidwells/extract-deps 0.0.7 → 0.0.8

package/examples/verify-basic.js

@@ -15,8 +15,6 @@ try {
   console.error(error.message)
 }
 
-console.log('foo')
-
 console.log('Verifying ESM fixture...')
 try {
   const graph = depGraph(esmFixture)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@davidwells/extract-deps",
-  "version": "0.0.7",
+  "version": "0.0.8",
   "description": "Get imports and requires from a codebase without AST",
   "main": "src/index.js",
   "exports": {

package/src/dep-graph.js

@@ -132,16 +132,18 @@ function verifyImportedValuesExist(dep, targetDeps, targetPath, sourcePath, erro
  * @param {boolean} throwOnFirstError - Whether to throw immediately on first error
  * @param {boolean} verify - Whether to verify imports/exports
  * @param {boolean} verifyNodeModules - Whether to verify node_modules can be resolved
+ * @param {boolean} includeNodeModules - Whether to include node_modules in the graph
  * @param {Map<string, string[]>|null} graph - Optional map to collect dependency graph
  * @param {Set} visited - Set of already-visited file paths to avoid cycles
  */
 function buildGraphRecursively(
-  filePath, 
-  errors, 
-  throwOnFirstError, 
-  verify, 
-  verifyNodeModules, 
-  graph = null, 
+  filePath,
+  errors,
+  throwOnFirstError,
+  verify,
+  verifyNodeModules,
+  includeNodeModules,
+  graph = null,
   visited = new Set(),
 ) {
   // Avoid infinite loops
@@ -171,7 +173,7 @@ function buildGraphRecursively(
 
   // Collect all non-relative dependencies (node_modules)
   const nodeModuleDeps = []
-  if (verifyNodeModules) {
+  if (verifyNodeModules || includeNodeModules) {
     if (deps.requires && deps.requires.length > 0) {
       nodeModuleDeps.push(...deps.requires.filter(req => !req.isRelative))
     }
@@ -217,7 +219,14 @@ function buildGraphRecursively(
     }
 
     // Recursively process the imported file
-    buildGraphRecursively(resolvedPath, errors, throwOnFirstError, verify, verifyNodeModules, graph, visited)
+    buildGraphRecursively(resolvedPath, errors, throwOnFirstError, verify, verifyNodeModules, includeNodeModules, graph, visited)
+  }
+
+  // Add node_modules to graph if enabled
+  if (includeNodeModules && graph !== null) {
+    for (const dep of nodeModuleDeps) {
+      graph.get(normalizedPath).push(dep.from)
+    }
   }
 
   // Verify node_modules dependencies if enabled
@@ -255,20 +264,22 @@ function buildGraphRecursively(
  * @param {boolean} [options.buildGraph] - If true, build and return dependency graph. Default: true
  * @param {boolean} [options.verifyGraph] - If true, verify imports/exports. Default: true
  * @param {boolean} [options.verifyNodeModules] - If true, verify node_modules packages can be resolved. Default: false
+ * @param {boolean} [options.includeNodeModules] - If true, include node_modules package names in the graph. Default: false
  * @param {boolean} [options.throwOnFirstError] - If true, throw immediately on first error. Default: false
- * @returns {Map<string, string[]>|undefined} - Dependency graph where keys are file paths and values are arrays of dependency paths. Returns undefined if buildGraph is false.
+ * @returns {Map<string, string[]>|undefined} - Dependency graph where keys are file paths and values are arrays of dependency paths (or package names if includeNodeModules). Returns undefined if buildGraph is false.
  */
 function depGraph(filePath, options = {}) {
-  const { 
-    buildGraph = true, 
-    verifyGraph = true, 
-    verifyNodeModules = false, 
+  const {
+    buildGraph = true,
+    verifyGraph = true,
+    verifyNodeModules = false,
+    includeNodeModules = false,
     throwOnFirstError = false,
   } = options
   const errors = []
   const graph = buildGraph ? new Map() : null
 
-  buildGraphRecursively(filePath, errors, throwOnFirstError, verifyGraph, verifyNodeModules, graph)
+  buildGraphRecursively(filePath, errors, throwOnFirstError, verifyGraph, verifyNodeModules, includeNodeModules, graph)
 
   if (errors.length > 0) {
     const errorMessage = errors.map((err, idx) => `Dep graph error:\n ${err.message}`).join('\n')
@@ -331,7 +342,7 @@ function resolveDepPaths(filePath, options = {}) {
 if (require.main === module) {
   const entry = path.resolve(__dirname, '../tests/fixtures/cjs-deep-example/index.js')
   const graph = depGraph(entry, {
-    verifyNodeModules: false,
+    includeNodeModules: true,
     throwOnFirstError: true
   })
   console.log('graph', graph)

package/src/index.js

@@ -50,6 +50,19 @@ if (require.main === module) {
   }
   `
 
+  const code2 = `
+// Get users Lambda handler
+import _ from 'lodash'
+
+export async function handler(event, context) {
+  const data = { message: 'get function called' }
+  return {
+    statusCode: 200,
+    body: JSON.stringify(_.omit(data, [])),
+  }
+}
+  `
+
   const failCase = `
 const { test } = require('uvu')
 const assert = require('uvu/assert')
@@ -65,7 +78,7 @@ const config = await import('./config.js')
 const lerna = require('lerna')
 const { execSync, spawnSync } = require('child_process')
 `
-  const result = extractDeps(code)
+  const result = extractDeps(code2)
 
   const { imports, requires } = result