@davidwells/extract-deps 0.1.0 → 0.1.2

package/.claude/settings.local.json

@@ -8,7 +8,8 @@
       "Bash(tar:*)",
       "Bash(npm install)",
       "Bash(npm test)",
-      "Bash(npm test:*)"
+      "Bash(npm test:*)",
+      "Bash(npm run types:*)"
     ],
     "deny": [],
     "ask": []

package/package.json

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

package/src/dep-graph.js

@@ -4,6 +4,7 @@ const fs = require('fs')
 const { pathToFileURL, fileURLToPath } = require('url')
 const { resolve: resolveImport } = require('@dual-bundle/import-meta-resolve')
 const { extractDeps } = require('./index')
+const { isBuiltIn } = require('./utils')
 
 function fsExistsSync(path) {
   try {
@@ -229,13 +230,18 @@ function buildGraphRecursively(
     }
   }
 
-  // Verify node_modules dependencies if enabled
+  // Verify node_modules dependencies if enabled (skip built-ins)
   if (verifyNodeModules && nodeModuleDeps.length > 0) {
     const parentURL = pathToFileURL(filePath).href
 
     for (const dep of nodeModuleDeps) {
       const { from, line } = dep
 
+      // Skip verification for built-in modules
+      if (dep.isBuiltIn) {
+        continue
+      }
+
       try {
         // Try to resolve the module using Node's resolution algorithm
         const resolved = resolveImport(from, parentURL)
@@ -266,7 +272,8 @@ function buildGraphRecursively(
  * @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 (or package names if includeNodeModules). Returns undefined if buildGraph is false.
+ * @param {boolean} [options.returnErrors] - If true, return { graph, errors } instead of throwing. Default: false
+ * @returns {Map<string, string[]>|{graph: Map, errors: Error[]}|undefined} - Dependency graph, or { graph, errors } if returnErrors is true. Returns undefined if buildGraph is false.
  */
 function depGraph(filePath, options = {}) {
   const {
@@ -275,12 +282,17 @@ function depGraph(filePath, options = {}) {
     verifyNodeModules = false,
     includeNodeModules = false,
     throwOnFirstError = false,
+    returnErrors = false,
   } = options
   const errors = []
   const graph = buildGraph ? new Map() : null
 
   buildGraphRecursively(filePath, errors, throwOnFirstError, verifyGraph, verifyNodeModules, includeNodeModules, graph)
 
+  if (returnErrors) {
+    return { graph, errors }
+  }
+
   if (errors.length > 0) {
     const errorMessage = errors.map((err, idx) => `Dep graph error:\n ${err.message}`).join('\n')
     throw new Error(
@@ -298,17 +310,43 @@ function depGraph(filePath, options = {}) {
  * Get all dependency file paths in build order (dependencies first), deduplicated
  * @param {string} filePath - Entry file path
  * @param {object} options - Options to pass to depGraph
- * @returns {{ files: string[], nodeModules: string[] }} - Object with files in topological order and unique node_modules
+ * @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.throwOnFirstError] - If true, throw immediately on first error. Default: false
+ * @param {boolean} [options.returnErrors] - If true, include errors array in result instead of throwing
+ * @returns {{ files: string[], nodeModules: string[], builtIns: string[], errors?: Error[] }} - Object with files in topological order, npm packages, and Node.js built-ins
  */
 function resolveDepPaths(filePath, options = {}) {
-  const graph = depGraph(filePath, { ...options, includeNodeModules: true })
+  const { returnErrors = false, ...restOptions } = options
+
+  /** @type {Map<string, string[]>} */
+  let graph
+  /** @type {Error[]} */
+  let errors = []
 
-  // Collect node_modules (non-path strings in dependency arrays)
+  if (returnErrors) {
+    const result = /** @type {{ graph: Map<string, string[]>, errors: Error[] }} */ (
+      depGraph(filePath, { ...restOptions, includeNodeModules: true, returnErrors: true })
+    )
+    graph = result.graph
+    errors = result.errors
+  } else {
+    graph = /** @type {Map<string, string[]>} */ (
+      depGraph(filePath, { ...restOptions, includeNodeModules: true })
+    )
+  }
+
+  // Collect node_modules and built-ins (non-path strings in dependency arrays)
   const nodeModulesSet = new Set()
+  const builtInsSet = new Set()
   for (const deps of graph.values()) {
     for (const dep of deps) {
       if (!dep.startsWith('/')) {
-        nodeModulesSet.add(dep)
+        if (isBuiltIn(dep)) {
+          builtInsSet.add(dep)
+        } else {
+          nodeModulesSet.add(dep)
+        }
       }
     }
   }
@@ -347,10 +385,17 @@ function resolveDepPaths(filePath, options = {}) {
     }
   }
 
-  return {
+  const result = {
     files: buildOrder,
-    nodeModules: Array.from(nodeModulesSet)
+    nodeModules: Array.from(nodeModulesSet),
+    builtIns: Array.from(builtInsSet)
+  }
+
+  if (returnErrors) {
+    result.errors = errors
   }
+
+  return result
 }
 
 

package/src/get-imports.js

@@ -1,4 +1,4 @@
-const { isRelativeImport, isRelativeImportWithQuotes, getNames } = require('./utils')
+const { isBuiltIn, isRelativeImport, isRelativeImportWithQuotes, getNames } = require('./utils')
 
 // https://regex101.com/r/cfEmlm/1
 const IMPORT_REGEX = /import(?:["'\s]*([\w*{}\n\r\t, ]+)from\s*)?["'\s](.*[@\w_-]+)["'\s]+;?$/gm
@@ -26,7 +26,7 @@ function getStaticImports(targetText = '') {
   while((result = IMPORT_REGEX.exec(targetText)) !== null) {
     const [ line, names, from ] = result
     const namesValues = (names || '').trim()
-    importStatements.push({ 
+    importStatements.push({
       line,
       varType: 'const',
       varName: namesValues,
@@ -34,7 +34,8 @@ function getStaticImports(targetText = '') {
       from,
       index: result.index,
       isImport: true,
-      isRelative: isRelativeImport(from)
+      isRelative: isRelativeImport(from),
+      isBuiltIn: isBuiltIn(from)
     })
   }
   return importStatements
@@ -53,7 +54,8 @@ function getTypeImports(targetText = '') {
       index: result.index,
       isImport: true,
       isTypeImport: true,
-      isRelative: isRelativeImport(importPath)
+      isRelative: isRelativeImport(importPath),
+      isBuiltIn: isBuiltIn(importPath)
     })
   }
   return importStatements
@@ -75,6 +77,7 @@ function getDynamicImports(targetText = '') {
       index: result.index,
       isImport: true,
       isRelative: isRelativeImport(from) || isRelativeImportWithQuotes(from),
+      isBuiltIn: isBuiltIn(from.trim()),
       isDynamic: true
     })
   }
@@ -101,6 +104,7 @@ function getAssertionImports(targetText = '') {
       index: result.index,
       isImport: true,
       isRelative: isRelativeImport(from),
+      isBuiltIn: isBuiltIn(from),
       isAssertion: true
     })
   }

package/src/get-requires.js

@@ -1,4 +1,4 @@
-const { isRelativeImport, getNames } = require('./utils')
+const { isBuiltIn, isRelativeImport, getNames } = require('./utils')
 
 // Updated regex to exclude ES6 dynamic imports
 const REQUIRE_REGEX = /(?!.*await\s+import)(const|var|let)(?:["'\s]*([\w*{}:=\n\r\t,:= ]+)\s*=\s*require\(\s*)?["'\s](.*[@\w_-]+)["'\s]+\);?$|(?!.*await\s+import)require\(["'\s](.*[@\w_-]+)["'\s]\)/gm
@@ -28,6 +28,7 @@ function getDynamicRequires(code) {
       index: match.index,
       isRequire: true,
       isRelative: false,
+      isBuiltIn: isBuiltIn(from.trim()),
       isDynamic: true
     })
   }
@@ -53,6 +54,7 @@ function getStaticRequires(targetText = '') {
       index: result.index,
       isRequire: true,
       isRelative: isRelativeImport(importLocation),
+      isBuiltIn: isBuiltIn(importLocation),
       isDynamic: false
     })
   }

package/src/utils.js

@@ -1,4 +1,20 @@
 const { inspectParameters, getParametersNames } = require('./vendor/inspect-parameters-declaration')
+const builtinModules = require('module').builtinModules
+
+/**
+ * Check if a module specifier is a Node.js built-in
+ * @param {string} moduleName - The module specifier (e.g., 'fs', 'node:fs/promises')
+ * @returns {boolean}
+ */
+function isBuiltIn(moduleName) {
+  // Handle node: prefix (always a built-in)
+  if (moduleName.startsWith('node:')) {
+    return true
+  }
+  // Handle subpaths like 'fs/promises' - check the base name
+  const baseName = moduleName.split('/')[0]
+  return builtinModules.includes(baseName)
+}
 
 function isRelativeImport(importLocation) {
   return !!importLocation.match(/^\./)
@@ -195,6 +211,7 @@ function extractExportName(exportValue) {
 }
 
 module.exports = {
+  isBuiltIn,
   isRelativeImport,
   isRelativeImportWithQuotes,
   getNames,

package/tests/builtins.test.js

@@ -0,0 +1,134 @@
+// Test handling of Node.js built-in modules
+const { test } = require('uvu')
+const assert = require('uvu/assert')
+const path = require('path')
+const { isBuiltIn } = require('../src/utils')
+const { getStaticImports } = require('../src/get-imports')
+const { getStaticRequires } = require('../src/get-requires')
+const { resolveDepPaths } = require('../src/dep-graph')
+
+const BUILTINS_FIXTURE_PATH = path.resolve(__dirname, './fixtures/builtins-example/index.js')
+
+// isBuiltIn utility tests
+test('isBuiltIn should detect node: prefix', () => {
+  assert.is(isBuiltIn('node:fs'), true)
+  assert.is(isBuiltIn('node:path'), true)
+  assert.is(isBuiltIn('node:stream'), true)
+})
+
+test('isBuiltIn should detect node: prefix with subpaths', () => {
+  assert.is(isBuiltIn('node:fs/promises'), true)
+  assert.is(isBuiltIn('node:stream/promises'), true)
+  assert.is(isBuiltIn('node:timers/promises'), true)
+})
+
+test('isBuiltIn should detect legacy bare built-ins', () => {
+  assert.is(isBuiltIn('fs'), true)
+  assert.is(isBuiltIn('path'), true)
+  assert.is(isBuiltIn('stream'), true)
+  assert.is(isBuiltIn('http'), true)
+  assert.is(isBuiltIn('crypto'), true)
+})
+
+test('isBuiltIn should detect legacy bare built-ins with subpaths', () => {
+  assert.is(isBuiltIn('fs/promises'), true)
+  assert.is(isBuiltIn('stream/promises'), true)
+  assert.is(isBuiltIn('timers/promises'), true)
+})
+
+test('isBuiltIn should return false for npm packages', () => {
+  assert.is(isBuiltIn('lodash'), false)
+  assert.is(isBuiltIn('express'), false)
+  assert.is(isBuiltIn('@org/package'), false)
+  assert.is(isBuiltIn('uvu/assert'), false)
+})
+
+test('isBuiltIn should return false for relative imports', () => {
+  assert.is(isBuiltIn('./foo'), false)
+  assert.is(isBuiltIn('../bar'), false)
+  assert.is(isBuiltIn('./node/fake'), false)
+})
+
+// getStaticImports with isBuiltIn flag
+test('getStaticImports should mark built-ins with isBuiltIn flag', () => {
+  const code = `
+import fs from 'fs'
+import { readFile } from 'node:fs/promises'
+import path from 'path'
+import lodash from 'lodash'
+import { foo } from './local'
+  `
+
+  const imports = getStaticImports(code)
+
+  const fsImport = imports.find(i => i.from === 'fs')
+  const nodeFsImport = imports.find(i => i.from === 'node:fs/promises')
+  const pathImport = imports.find(i => i.from === 'path')
+  const lodashImport = imports.find(i => i.from === 'lodash')
+  const localImport = imports.find(i => i.from === './local')
+
+  assert.is(fsImport.isBuiltIn, true, 'fs should be built-in')
+  assert.is(nodeFsImport.isBuiltIn, true, 'node:fs/promises should be built-in')
+  assert.is(pathImport.isBuiltIn, true, 'path should be built-in')
+  assert.is(lodashImport.isBuiltIn, false, 'lodash should not be built-in')
+  assert.is(localImport.isBuiltIn, false, './local should not be built-in')
+})
+
+// getStaticRequires with isBuiltIn flag
+test('getStaticRequires should mark built-ins with isBuiltIn flag', () => {
+  const code = `
+const fs = require('fs')
+const { readFile } = require('node:fs/promises')
+const path = require('path')
+const lodash = require('lodash')
+const local = require('./local')
+  `
+
+  const requires = getStaticRequires(code)
+
+  const fsReq = requires.find(r => r.from === 'fs')
+  const nodeFsReq = requires.find(r => r.from === 'node:fs/promises')
+  const pathReq = requires.find(r => r.from === 'path')
+  const lodashReq = requires.find(r => r.from === 'lodash')
+  const localReq = requires.find(r => r.from === './local')
+
+  assert.is(fsReq.isBuiltIn, true, 'fs should be built-in')
+  assert.is(nodeFsReq.isBuiltIn, true, 'node:fs/promises should be built-in')
+  assert.is(pathReq.isBuiltIn, true, 'path should be built-in')
+  assert.is(lodashReq.isBuiltIn, false, 'lodash should not be built-in')
+  assert.is(localReq.isBuiltIn, false, './local should not be built-in')
+})
+
+// resolveDepPaths tests for builtIns
+test('resolveDepPaths should return separate builtIns array', () => {
+  const result = resolveDepPaths(BUILTINS_FIXTURE_PATH)
+
+  assert.ok(Array.isArray(result.builtIns), 'should have builtIns array')
+  assert.ok(Array.isArray(result.nodeModules), 'should have nodeModules array')
+  assert.ok(Array.isArray(result.files), 'should have files array')
+})
+
+test('resolveDepPaths should separate built-ins from nodeModules', () => {
+  const { builtIns, nodeModules } = resolveDepPaths(BUILTINS_FIXTURE_PATH)
+
+  // Built-ins should include fs, node:fs/promises, path, node:stream, node:crypto
+  assert.ok(builtIns.includes('fs'), 'builtIns should include fs')
+  assert.ok(builtIns.includes('node:fs/promises'), 'builtIns should include node:fs/promises')
+  assert.ok(builtIns.includes('path'), 'builtIns should include path')
+  assert.ok(builtIns.includes('node:stream'), 'builtIns should include node:stream')
+  assert.ok(builtIns.includes('node:crypto'), 'builtIns should include node:crypto')
+
+  // nodeModules should include lodash but not built-ins
+  assert.ok(nodeModules.includes('lodash'), 'nodeModules should include lodash')
+  assert.ok(!nodeModules.includes('fs'), 'nodeModules should NOT include fs')
+  assert.ok(!nodeModules.includes('node:fs/promises'), 'nodeModules should NOT include node:fs/promises')
+})
+
+test('resolveDepPaths builtIns should not duplicate entries', () => {
+  const { builtIns } = resolveDepPaths(BUILTINS_FIXTURE_PATH)
+
+  const uniqueBuiltIns = [...new Set(builtIns)]
+  assert.is(builtIns.length, uniqueBuiltIns.length, 'builtIns should be deduplicated')
+})
+
+test.run()

package/tests/dep-graph.test.js

@@ -438,4 +438,87 @@ test('includeNodeModules works independently of verifyNodeModules', () => {
   assert.ok(indexDeps.includes('lodash'), 'Should include lodash without verification')
 })
 
+// returnErrors tests
+test('returnErrors: true should return { graph, errors } instead of throwing', () => {
+  const fs = require('fs')
+  const testFile = path.resolve(__dirname, 'fixtures/test-return-errors.js')
+
+  fs.writeFileSync(testFile, `
+const missing = require('missing-package-xyz')
+module.exports = { missing }
+`)
+
+  try {
+    const result = depGraph(testFile, { verifyNodeModules: true, returnErrors: true })
+
+    assert.ok(result.graph instanceof Map, 'Should return graph in result')
+    assert.ok(Array.isArray(result.errors), 'Should return errors array')
+    assert.ok(result.errors.length > 0, 'Should have errors for missing package')
+    assert.ok(result.errors[0].message.includes('missing-package-xyz'), 'Error should mention package')
+  } finally {
+    fs.unlinkSync(testFile)
+  }
+})
+
+test('returnErrors: true with no errors returns empty errors array', () => {
+  const result = depGraph(CJS_FIXTURE_PATH, { returnErrors: true })
+
+  assert.ok(result.graph instanceof Map, 'Should return graph')
+  assert.ok(Array.isArray(result.errors), 'Should return errors array')
+  assert.is(result.errors.length, 0, 'Should have no errors')
+})
+
+test('returnErrors: true collects all errors without throwing', () => {
+  const fs = require('fs')
+  const testFile = path.resolve(__dirname, 'fixtures/test-return-multi-errors.js')
+
+  fs.writeFileSync(testFile, `
+const missing1 = require('missing-pkg-1-xyz')
+const missing2 = require('missing-pkg-2-xyz')
+module.exports = { missing1, missing2 }
+`)
+
+  try {
+    const result = depGraph(testFile, { verifyNodeModules: true, returnErrors: true })
+
+    assert.is(result.errors.length, 2, 'Should collect both errors')
+    assert.ok(result.errors.some(e => e.message.includes('missing-pkg-1-xyz')), 'Should have first error')
+    assert.ok(result.errors.some(e => e.message.includes('missing-pkg-2-xyz')), 'Should have second error')
+  } finally {
+    fs.unlinkSync(testFile)
+  }
+})
+
+// resolveDepPaths with returnErrors
+test('resolveDepPaths with returnErrors: true returns { files, nodeModules, errors }', () => {
+  const fs = require('fs')
+  const testFile = path.resolve(__dirname, 'fixtures/test-resolve-return-errors.js')
+
+  fs.writeFileSync(testFile, `
+const missing = require('missing-resolve-pkg-xyz')
+module.exports = { missing }
+`)
+
+  try {
+    const result = resolveDepPaths(testFile, { verifyNodeModules: true, returnErrors: true })
+
+    assert.ok(Array.isArray(result.files), 'Should have files array')
+    assert.ok(Array.isArray(result.nodeModules), 'Should have nodeModules array')
+    assert.ok(Array.isArray(result.errors), 'Should have errors array')
+    assert.ok(result.errors.length > 0, 'Should have errors')
+    assert.ok(result.errors[0].message.includes('missing-resolve-pkg-xyz'), 'Error should mention package')
+  } finally {
+    fs.unlinkSync(testFile)
+  }
+})
+
+test('resolveDepPaths with returnErrors: true and no errors', () => {
+  const result = resolveDepPaths(CJS_FIXTURE_PATH, { returnErrors: true })
+
+  assert.ok(Array.isArray(result.files), 'Should have files')
+  assert.ok(Array.isArray(result.nodeModules), 'Should have nodeModules')
+  assert.ok(Array.isArray(result.errors), 'Should have errors array')
+  assert.is(result.errors.length, 0, 'Should have no errors')
+})
+
 test.run()

package/tests/fixtures/builtins-example/helper.js

@@ -0,0 +1,8 @@
+// Local helper using built-ins
+const { createHash } = require('node:crypto')
+
+function helper() {
+  return createHash('sha256')
+}
+
+module.exports = { helper }

package/tests/fixtures/builtins-example/index.js

@@ -0,0 +1,16 @@
+// Fixture using Node.js built-ins and npm packages
+const fs = require('fs')
+const { readFile } = require('node:fs/promises')
+const path = require('path')
+const { Transform } = require('node:stream')
+const lodash = require('lodash')
+const { helper } = require('./helper')
+
+module.exports = {
+  fs,
+  readFile,
+  path,
+  Transform,
+  lodash,
+  helper
+}

package/tests/get-imports.test.js

@@ -50,7 +50,8 @@ test('getStaticImports', async () => {
       },
       from: 'url',
       isImport: true,
-      isRelative: false
+      isRelative: false,
+      isBuiltIn: true
     },
     {
       line: "import foo from 'fs'",
@@ -59,7 +60,8 @@ test('getStaticImports', async () => {
       definitions: { declaration: 'foo', name: 'foo' },
       from: 'fs',
       isImport: true,
-      isRelative: false
+      isRelative: false,
+      isBuiltIn: true
     },
     {
       line: "import * as Fs from 'fs'",
@@ -68,7 +70,8 @@ test('getStaticImports', async () => {
       definitions: { declaration: '* as Fs', name: '*', alias: 'Fs' },
       from: 'fs',
       isImport: true,
-      isRelative: false
+      isRelative: false,
+      isBuiltIn: true
     },
     {
       line: "import assert from 'uvu/assert'",
@@ -77,7 +80,8 @@ test('getStaticImports', async () => {
       definitions: { declaration: 'assert', name: 'assert' },
       from: 'uvu/assert',
       isImport: true,
-      isRelative: false
+      isRelative: false,
+      isBuiltIn: false
     }
   ])
 })
@@ -105,7 +109,8 @@ test('get jsDoc Imports', async () => {
         importName: 'hdhdhdhdhddh',
         isImport: true,
         isTypeImport: true,
-        isRelative: true
+        isRelative: true,
+        isBuiltIn: false
       },
       {
         importStatement: 'import("lodash").omit',
@@ -113,7 +118,8 @@ test('get jsDoc Imports', async () => {
         importName: 'omit',
         isImport: true,
         isTypeImport: true,
-        isRelative: false
+        isRelative: false,
+        isBuiltIn: false
       },
       {
         importStatement: 'import("uvu/assert")',
@@ -121,7 +127,8 @@ test('get jsDoc Imports', async () => {
         importName: '',
         isImport: true,
         isTypeImport: true,
-        isRelative: false
+        isRelative: false,
+        isBuiltIn: false
       }
     ])
 })
@@ -166,6 +173,7 @@ test('getDynamicImports', async () => {
     from: "'./module.js'",
     isImport: true,
     isRelative: true,
+    isBuiltIn: false,
     isDynamic: true
   },
   {
@@ -176,6 +184,7 @@ test('getDynamicImports', async () => {
     from: 'moduleName',
     isImport: true,
     isRelative: false,
+    isBuiltIn: false,
     isDynamic: true
   },
   {
@@ -186,6 +195,7 @@ test('getDynamicImports', async () => {
     from: '`./src/${folder}/Button.js`',
     isImport: true,
     isRelative: false,
+    isBuiltIn: false,
     isDynamic: true
   },
   {
@@ -196,6 +206,7 @@ test('getDynamicImports', async () => {
     from: 'modulePath',
     isImport: true,
     isRelative: false,
+    isBuiltIn: false,
     isDynamic: true
   },
   {
@@ -218,6 +229,7 @@ test('getDynamicImports', async () => {
     from: "'./Component.js'",
     isImport: true,
     isRelative: true,
+    isBuiltIn: false,
     isDynamic: true
   },
   {
@@ -228,6 +240,7 @@ test('getDynamicImports', async () => {
     from: 'getModulePath()',
     isImport: true,
     isRelative: false,
+    isBuiltIn: false,
     isDynamic: true
   }
 ])
@@ -324,6 +337,7 @@ import styles from './styles.css' with { type: 'css' }
       assertionType: 'assert',
       isImport: true,
       isRelative: true,
+      isBuiltIn: false,
       isAssertion: true
     },
     {
@@ -336,6 +350,7 @@ import styles from './styles.css' with { type: 'css' }
       assertionType: 'assert',
       isImport: true,
       isRelative: true,
+      isBuiltIn: false,
       isAssertion: true
     },
     {
@@ -348,6 +363,7 @@ import styles from './styles.css' with { type: 'css' }
       assertionType: 'with',
       isImport: true,
       isRelative: true,
+      isBuiltIn: false,
       isAssertion: true
     },
     {
@@ -360,6 +376,7 @@ import styles from './styles.css' with { type: 'css' }
       assertionType: 'with',
       isImport: true,
       isRelative: true,
+      isBuiltIn: false,
       isAssertion: true
     }
   ])

package/tests/get-requires.test.js

@@ -33,6 +33,7 @@ test('getRequires', async () => {
       from: 'uvu',
       isRequire: true,
       isRelative: false,
+      isBuiltIn: false,
       isDynamic: false
     },
     {
@@ -43,6 +44,7 @@ test('getRequires', async () => {
       from: 'uvu/assert',
       isRequire: true,
       isRelative: false,
+      isBuiltIn: false,
       isDynamic: false
     },
     {
@@ -58,6 +60,7 @@ test('getRequires', async () => {
       from: './get-imports',
       isRequire: true,
       isRelative: true,
+      isBuiltIn: false,
       isDynamic: false
     }
   ])
@@ -96,6 +99,7 @@ test('getRequires 2', async () => {
       from: 'uvu',
       isRequire: true,
       isRelative: false,
+      isBuiltIn: false,
       isDynamic: false
     },
     {
@@ -106,6 +110,7 @@ test('getRequires 2', async () => {
       from: 'uvu/assert',
       isRequire: true,
       isRelative: false,
+      isBuiltIn: false,
       isDynamic: false
     },
     {
@@ -121,6 +126,7 @@ test('getRequires 2', async () => {
       from: './get-imports',
       isRequire: true,
       isRelative: true,
+      isBuiltIn: false,
       isDynamic: false
     }
   ])
@@ -151,6 +157,7 @@ test('getRequires with dynamic requires', async () => {
       from: 'moduleName',
       isRequire: true,
       isRelative: false,
+      isBuiltIn: false,
       isDynamic: true
     },
     {
@@ -161,6 +168,7 @@ test('getRequires with dynamic requires', async () => {
       from: "path.join(__dirname, 'utils')",
       isRequire: true,
       isRelative: false,
+      isBuiltIn: false,
       isDynamic: true
     },
     {
@@ -171,6 +179,7 @@ test('getRequires with dynamic requires', async () => {
       from: '`./${folder}/file`',
       isRequire: true,
       isRelative: false,
+      isBuiltIn: false,
       isDynamic: true
     },
     {
@@ -193,6 +202,7 @@ test('getRequires with dynamic requires', async () => {
       from: 'moduleTwo',
       isRequire: true,
       isRelative: false,
+      isBuiltIn: false,
       isDynamic: true
     }
   ])

package/types/dep-graph.d.ts

@@ -7,7 +7,8 @@
  * @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 (or package names if includeNodeModules). Returns undefined if buildGraph is false.
+ * @param {boolean} [options.returnErrors] - If true, return { graph, errors } instead of throwing. Default: false
+ * @returns {Map<string, string[]>|{graph: Map, errors: Error[]}|undefined} - Dependency graph, or { graph, errors } if returnErrors is true. Returns undefined if buildGraph is false.
  */
 export function depGraph(filePath: string, options?: {
     buildGraph?: boolean;
@@ -15,15 +16,29 @@ export function depGraph(filePath: string, options?: {
     verifyNodeModules?: boolean;
     includeNodeModules?: boolean;
     throwOnFirstError?: boolean;
-}): Map<string, string[]> | undefined;
+    returnErrors?: boolean;
+}): Map<string, string[]> | {
+    graph: Map<any, any>;
+    errors: Error[];
+} | undefined;
 /**
  * Get all dependency file paths in build order (dependencies first), deduplicated
  * @param {string} filePath - Entry file path
  * @param {object} options - Options to pass to depGraph
- * @returns {{ files: string[], nodeModules: string[] }} - Object with files in topological order and unique node_modules
+ * @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.throwOnFirstError] - If true, throw immediately on first error. Default: false
+ * @param {boolean} [options.returnErrors] - If true, include errors array in result instead of throwing
+ * @returns {{ files: string[], nodeModules: string[], errors?: Error[] }} - Object with files in topological order and unique node_modules
  */
-export function resolveDepPaths(filePath: string, options?: object): {
+export function resolveDepPaths(filePath: string, options?: {
+    verifyGraph?: boolean;
+    verifyNodeModules?: boolean;
+    throwOnFirstError?: boolean;
+    returnErrors?: boolean;
+}): {
     files: string[];
     nodeModules: string[];
+    errors?: Error[];
 };
 //# sourceMappingURL=dep-graph.d.ts.map

package/types/dep-graph.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"dep-graph.d.ts","sourceRoot":"","sources":["../src/dep-graph.js"],"names":[],"mappings":"AAmQA;;;;;;;;;;GAUG;AACH,mCATW,MAAM,YAEd;IAA0B,UAAU,GAA5B,OAAO;IACW,WAAW,GAA7B,OAAO;IACW,iBAAiB,GAAnC,OAAO;IACW,kBAAkB,GAApC,OAAO;IACW,iBAAiB,GAAnC,OAAO;CACf,GAAU,GAAG,CAAC,MAAM,EAAE,MAAM,EAAE,CAAC,GAAC,SAAS,CAyB3C;AAGD;;;;;GAKG;AACH,0CAJW,MAAM,YACN,MAAM,GACJ;IAAE,KAAK,EAAE,MAAM,EAAE,CAAC;IAAC,WAAW,EAAE,MAAM,EAAE,CAAA;CAAE,CAqDtD"}
+{"version":3,"file":"dep-graph.d.ts","sourceRoot":"","sources":["../src/dep-graph.js"],"names":[],"mappings":"AAmQA;;;;;;;;;;;GAWG;AACH,mCAVW,MAAM,YAEd;IAA0B,UAAU,GAA5B,OAAO;IACW,WAAW,GAA7B,OAAO;IACW,iBAAiB,GAAnC,OAAO;IACW,kBAAkB,GAApC,OAAO;IACW,iBAAiB,GAAnC,OAAO;IACW,YAAY,GAA9B,OAAO;CACf,GAAU,GAAG,CAAC,MAAM,EAAE,MAAM,EAAE,CAAC,GAAC;IAAC,KAAK,gBAAM;IAAC,MAAM,EAAE,KAAK,EAAE,CAAA;CAAC,GAAC,SAAS,CA8BzE;AAGD;;;;;;;;;GASG;AACH,0CARW,MAAM,YAEd;IAA0B,WAAW,GAA7B,OAAO;IACW,iBAAiB,GAAnC,OAAO;IACW,iBAAiB,GAAnC,OAAO;IACW,YAAY,GAA9B,OAAO;CACf,GAAU;IAAE,KAAK,EAAE,MAAM,EAAE,CAAC;IAAC,WAAW,EAAE,MAAM,EAAE,CAAC;IAAC,MAAM,CAAC,EAAE,KAAK,EAAE,CAAA;CAAE,CA4ExE"}