@davidwells/extract-deps 0.0.6 → 0.0.7

package/package.json

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

package/src/dep-graph.js

@@ -14,6 +14,14 @@ function fsExistsSync(path) {
   }
 }
 
+function isFileSync(filePath) {
+  try {
+    return fs.statSync(filePath).isFile()
+  } catch (error) {
+    return false
+  }
+}
+
 /**
  * Resolve a relative import to an actual file path
  * @param {string} baseDir - Directory containing the importing file
@@ -23,10 +31,10 @@ function fsExistsSync(path) {
 function resolveImportPath(baseDir, importPath) {
   const possibleExtensions = ['.js', '.mjs', '.ts', '.json', '']
 
-  // Try direct file resolution
+  // Try direct file resolution (must be a file, not directory)
   for (const ext of possibleExtensions) {
     const testPath = path.resolve(baseDir, importPath + ext)
-    if (fsExistsSync(testPath)) {
+    if (isFileSync(testPath)) {
       return testPath
     }
   }

package/tests/dep-graph.test.js

@@ -355,4 +355,31 @@ module.exports = { missing1, missing2 }
   }
 })
 
+// Directory to index.js resolution tests
+const DIR_INDEX_FIXTURE_PATH = path.resolve(__dirname, './fixtures/dir-index-example/entry.js')
+
+test('should resolve directory import to index.js', () => {
+  const graph = depGraph(DIR_INDEX_FIXTURE_PATH)
+
+  assert.ok(graph instanceof Map, 'Should return a Map')
+  assert.ok(graph.size === 2, 'Graph should have 2 files (entry.js and shared-pkg/index.js)')
+
+  const entries = Array.from(graph.keys())
+  const hasEntry = entries.some(key => key.includes('entry.js'))
+  const hasIndex = entries.some(key => key.includes('shared-pkg/index.js'))
+
+  assert.ok(hasEntry, 'Graph should include entry.js')
+  assert.ok(hasIndex, 'Graph should resolve directory to index.js')
+})
+
+test('resolveDepPaths should work with directory imports', () => {
+  const deps = resolveDepPaths(DIR_INDEX_FIXTURE_PATH)
+
+  assert.ok(Array.isArray(deps), 'Should return an array')
+  assert.is(deps.length, 2, 'Should have 2 files')
+
+  assert.ok(deps.some(p => p.includes('shared-pkg/index.js')), 'Should include resolved index.js')
+  assert.ok(deps.some(p => p.includes('entry.js')), 'Should include entry.js')
+})
+
 test.run()

package/tests/fixtures/dir-index-example/entry.js

@@ -0,0 +1,6 @@
+// Entry file that imports directory (should resolve to index.js)
+const shared = require('./shared-pkg')
+
+module.exports = {
+  run: () => shared.helper()
+}

package/tests/fixtures/dir-index-example/shared-pkg/index.js

@@ -0,0 +1,4 @@
+// Shared package entry point
+module.exports = {
+  helper: () => 'shared'
+}