@davidwells/extract-deps 0.0.5 → 0.0.7

package/tests/fixtures/cjs-broken-example/package.json

@@ -0,0 +1,7 @@
+{
+  "name": "cjs-example",
+  "version": "1.0.0",
+  "description": "CJS fixture for testing",
+  "main": "./index.js",
+  "type": "commonjs"
+}

package/tests/fixtures/cjs-broken-example/sub-folder/sub-sub-folder/three.js

@@ -0,0 +1,6 @@
+
+const three = 'three'
+
+module.exports = {
+  three
+}

package/tests/fixtures/cjs-broken-example/sub-folder/two.js

@@ -0,0 +1,8 @@
+const { three } = require('./sub-sub-folder/three')
+
+const two = 'two'
+
+module.exports = {
+  two,
+  three
+}

package/tests/fixtures/cjs-deep-example/package.json

@@ -0,0 +1,15 @@
+{
+  "name": "cjs-deep-example",
+  "version": "1.0.0",
+  "description": "Complex CJS fixture for testing with multiple entry points",
+  "main": "./index.js",
+  "module": "./d-one.js",
+  "exports": {
+    ".": {
+      "require": "./index.js",
+      "import": "./d-one.js"
+    },
+    "./one": "./d-one.js",
+    "./sub": "./d-sub-folder/d-two.js"
+  }
+}

package/tests/fixtures/cjs-example/package.json

@@ -0,0 +1,7 @@
+{
+  "name": "cjs-example",
+  "version": "1.0.0",
+  "description": "CJS fixture for testing",
+  "main": "./index.js",
+  "type": "commonjs"
+}

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'
+}

package/tests/fixtures/esm-broken-fixture/index.js

@@ -0,0 +1,8 @@
+import lodash from 'lodash'
+import { one, two, three } from './one.js'
+
+export {
+  one,
+  two,
+  three,
+}

package/tests/fixtures/esm-broken-fixture/one.js

@@ -0,0 +1,9 @@
+import { two, three, four } from './sub-folder/two.js'
+
+const one = 'one'
+
+export {
+  one,
+  two,
+  three
+}

package/tests/fixtures/esm-broken-fixture/package.json

@@ -0,0 +1,15 @@
+{
+  "name": "esm-fixture",
+  "version": "1.0.0",
+  "description": "ESM fixture for testing",
+  "main": "./index.js",
+  "module": "./index.js",
+  "type": "module",
+  "exports": {
+    ".": "./index.js",
+    "./valid": "./valid.js",
+    "./deep/deep": "./sub-folder/sub-sub-folder-two/deep.js",
+    "./test-deep-missing-import-ref": "./one.js",
+    "./non-existent-file": "./non-existent-file.js"
+  }
+}

package/tests/fixtures/esm-broken-fixture/sub-folder/sub-sub-folder/three.js

@@ -0,0 +1,6 @@
+
+const three = 'three'
+
+export {
+  three
+}

package/tests/fixtures/esm-broken-fixture/sub-folder/sub-sub-folder-two/deep.js

@@ -0,0 +1,6 @@
+import { wowDeep } from './even-deeper/wow-deep.js'
+const deep = 'deep'
+
+export {
+  deep
+}

package/tests/fixtures/esm-broken-fixture/sub-folder/sub-sub-folder-two/even-deeper/wow-deep.js

@@ -0,0 +1,5 @@
+const wowDeep = 'wowDeep'
+
+export {
+  wowDeep
+}

package/tests/fixtures/esm-broken-fixture/sub-folder/two.js

@@ -0,0 +1,8 @@
+import { three } from './sub-sub-folder/three.js'
+
+const two = 'two'
+
+export {
+  two,
+  three
+}

package/tests/fixtures/esm-broken-fixture/valid.js

@@ -0,0 +1,7 @@
+import { three } from './sub-folder/sub-sub-folder/three.js'
+const valid = 'valid'
+
+export {
+  valid,
+  three
+}

package/tests/fixtures/esm-fixture/package.json

@@ -0,0 +1,11 @@
+{
+  "name": "esm-fixture",
+  "version": "1.0.0",
+  "description": "ESM fixture for testing",
+  "main": "./index.js",
+  "module": "./index.js",
+  "type": "module",
+  "exports": {
+    ".": "./index.js"
+  }
+}

package/tests/get-cjs-exports.test.js

@@ -588,4 +588,175 @@ module.exports = async function handler(event) {
   assert.equal(foundExports[0].exportName, '', 'exportName should empty because it default function')
 })
 
+test('should handle Object.defineProperty on module.exports', () => {
+  const code = `
+function assembleStyles() {
+  return { color: 'red', fontSize: '16px' }
+}
+
+Object.defineProperty(module, 'exports', {
+  enumerable: true,
+  get: assembleStyles
+})
+  `
+
+  const foundExports = getCjsExports(code)
+
+  assert.ok(foundExports.length === 1, 'Should find 1 export')
+  assert.equal(foundExports[0].exportType, 'default', 'Should be default export')
+  assert.equal(foundExports[0].exportName, '', 'Default export should have empty name')
+  assert.equal(foundExports[0].exportDefinitionName, 'assembleStyles', 'Should extract getter function name')
+  assert.ok(foundExports[0].isDefineProperty, 'Should have isDefineProperty flag')
+  assert.ok(foundExports[0].exportValue.includes('Object.defineProperty'), 'Should include full defineProperty call')
+})
+
+test('should handle Object.defineProperty on exports with named function', () => {
+  const code = `
+function getVersion() {
+  return '1.0.0'
+}
+
+Object.defineProperty(exports, 'version', {
+  enumerable: true,
+  get: getVersion
+})
+  `
+
+  const foundExports = getCjsExports(code)
+
+  assert.ok(foundExports.length === 1, 'Should find 1 export')
+  assert.equal(foundExports[0].exportType, 'named', 'Should be named export')
+  assert.equal(foundExports[0].exportName, 'version', 'Should extract property name')
+  assert.equal(foundExports[0].exportDefinitionName, 'getVersion', 'Should extract getter function name')
+  assert.ok(foundExports[0].isDefineProperty, 'Should have isDefineProperty flag')
+})
+
+test('should handle Object.defineProperty with arrow function getter', () => {
+  const code = `
+Object.defineProperty(exports, 'version', {
+  enumerable: true,
+  get: () => '1.0.0'
+})
+  `
+
+  const foundExports = getCjsExports(code)
+
+  assert.ok(foundExports.length === 1, 'Should find 1 export')
+  assert.equal(foundExports[0].exportType, 'named', 'Should be named export')
+  assert.equal(foundExports[0].exportName, 'version', 'Should extract property name')
+  assert.equal(foundExports[0].exportDefinitionName, 'arrow function', 'Should detect arrow function')
+  assert.ok(foundExports[0].isDefineProperty, 'Should have isDefineProperty flag')
+})
+
+test('should handle Object.defineProperty with getter method', () => {
+  const code = `
+Object.defineProperty(exports, 'config', {
+  enumerable: true,
+  get() {
+    return { debug: true }
+  }
+})
+  `
+
+  const foundExports = getCjsExports(code)
+
+  assert.ok(foundExports.length === 1, 'Should find 1 export')
+  assert.equal(foundExports[0].exportType, 'named', 'Should be named export')
+  assert.equal(foundExports[0].exportName, 'config', 'Should extract property name')
+  assert.equal(foundExports[0].exportDefinitionName, 'getter method', 'Should detect getter method')
+  assert.ok(foundExports[0].isDefineProperty, 'Should have isDefineProperty flag')
+})
+
+test('should handle multiple Object.defineProperty exports', () => {
+  const code = `
+Object.defineProperty(exports, 'foo', {
+  get: getFoo
+})
+
+Object.defineProperty(exports, 'bar', {
+  get: () => 'bar value'
+})
+
+Object.defineProperty(exports, 'baz', {
+  get() {
+    return 'baz value'
+  }
+})
+  `
+
+  const foundExports = getCjsExports(code)
+
+  assert.ok(foundExports.length === 3, 'Should find 3 exports')
+  
+  const fooExport = foundExports.find(exp => exp.exportName === 'foo')
+  assert.ok(fooExport, 'Should find foo export')
+  assert.equal(fooExport.exportDefinitionName, 'getFoo', 'Should extract named function')
+  
+  const barExport = foundExports.find(exp => exp.exportName === 'bar')
+  assert.ok(barExport, 'Should find bar export')
+  assert.equal(barExport.exportDefinitionName, 'arrow function', 'Should detect arrow function')
+  
+  const bazExport = foundExports.find(exp => exp.exportName === 'baz')
+  assert.ok(bazExport, 'Should find baz export')
+  assert.equal(bazExport.exportDefinitionName, 'getter method', 'Should detect getter method')
+  
+  foundExports.forEach(exp => {
+    assert.ok(exp.isDefineProperty, 'All should have isDefineProperty flag')
+  })
+})
+
+test('should handle mixed exports with Object.defineProperty', () => {
+  const code = `
+// Standard exports
+module.exports.hello = function() { return 'hello' }
+exports.foo = function() {}
+exports.bar = 'bar'
+
+// Object.defineProperty exports
+function assembleStyles() {
+  return { color: 'red' }
+}
+
+Object.defineProperty(module, 'exports', {
+  enumerable: true,
+  get: assembleStyles
+})
+
+Object.defineProperty(exports, 'version', {
+  enumerable: true,
+  get: () => '1.0.0'
+})
+  `
+
+  const foundExports = getCjsExports(code)
+
+  assert.ok(foundExports.length === 5, 'Should find 5 exports total')
+  
+  // Check standard exports
+  const helloExport = foundExports.find(exp => exp.exportName === 'hello')
+  assert.ok(helloExport, 'Should find hello export')
+  assert.ok(!helloExport.isDefineProperty, 'Standard export should not have isDefineProperty flag')
+  
+  const barExport = foundExports.find(exp => exp.exportName === 'bar')
+  assert.ok(barExport, 'Should find bar export')
+  assert.equal(barExport.exportValue, "'bar'", 'Should only capture bar value, not following code')
+  
+  // Check Object.defineProperty exports
+  const definePropertyExports = foundExports.filter(exp => exp.isDefineProperty)
+  assert.equal(definePropertyExports.length, 2, 'Should find 2 Object.defineProperty exports')
+  
+  const moduleExport = definePropertyExports.find(exp => exp.exportType === 'default')
+  assert.ok(moduleExport, 'Should find module.exports defineProperty')
+  assert.equal(moduleExport.exportDefinitionName, 'assembleStyles', 'Should extract getter name')
+  
+  const versionExport = definePropertyExports.find(exp => exp.exportName === 'version')
+  assert.ok(versionExport, 'Should find version defineProperty')
+  assert.equal(versionExport.exportDefinitionName, 'arrow function', 'Should detect arrow function')
+  
+  // Check sorting by index
+  for (let i = 1; i < foundExports.length; i++) {
+    assert.ok(foundExports[i].index >= foundExports[i-1].index, 'Exports should be sorted by index')
+  }
+})
+
 test.run()

package/tests/get-imports.test.js

@@ -96,7 +96,7 @@ test('get jsDoc Imports', async () => {
     const importsFound = getTypeImports(code)
     /*
     console.log('importsFound')
-    deepLog(importsFound)
+    console.log(importsFound)
     /** */
     assert.equal(normalizeTestResult(importsFound), [
       {
@@ -104,6 +104,7 @@ test('get jsDoc Imports', async () => {
         importPath: './foo',
         importName: 'hdhdhdhdhddh',
         isImport: true,
+        isTypeImport: true,
         isRelative: true
       },
       {
@@ -111,6 +112,7 @@ test('get jsDoc Imports', async () => {
         importPath: 'lodash',
         importName: 'omit',
         isImport: true,
+        isTypeImport: true,
         isRelative: false
       },
       {
@@ -118,6 +120,7 @@ test('get jsDoc Imports', async () => {
         importPath: 'uvu/assert',
         importName: '',
         isImport: true,
+        isTypeImport: true,
         isRelative: false
       }
     ])

package/tests/manual/broken-cjs.js

@@ -0,0 +1,38 @@
+// Verify package.json entry points and their dependencies
+const path = require('path')
+const { getPackageEntryPoints, verifyPackageEntries } = require('../../src/verify-package-entries')
+
+if (require.main === module) {
+  // const pkgPath = path.resolve(__dirname, '../fixtures/esm-broken-fixture/package.json')
+  const pkgPath = path.resolve(__dirname, '../fixtures/cjs-broken-example/package.json')
+  console.log('=== Package Entry Points ===\n')
+  const entryPoints = getPackageEntryPoints(pkgPath)
+  entryPoints.forEach(ep => {
+    console.log(`${ep.field}: ${ep.path}`)
+  })
+
+  console.log('\n=== Verification Results ===\n')
+  const throwIfErrors = false
+  const results = verifyPackageEntries(pkgPath, { throwIfErrors })
+  console.log('results', results)
+
+  results.entryPoints.forEach(ep => {
+    console.log('\n──────────────────────────────────────────────────────────────')
+    console.log('PATH:  ', ep.path)
+    console.log(`Field:  "${ep.field}"`)
+    console.log(`Valid:  ${ep.valid ? '✅' : '❌'}`)
+    if (ep.sourceFiles) {
+      console.log(`srcFiles:  ${ep.sourceFiles.length} files`)
+      ep.sourceFiles.forEach(sf => {
+        console.log(`- ${sf}`)
+      })
+    }
+    if (ep.error) {
+      console.log(`Error:`)
+      console.log(ep.error.message)
+    }
+  })
+  console.log('\n──────────────────────────────────────────────────────────────')
+  console.log(`\nTotal unique dependencies: ${results.sourceFiles.size}`)
+  console.log(`Total errors: ${results.errors.length}`)
+}

package/tests/manual/broken-esm.js

@@ -0,0 +1,37 @@
+// Verify package.json entry points and their dependencies
+const path = require('path')
+const { getPackageEntryPoints, verifyPackageEntries } = require('../../src/verify-package-entries')
+
+if (require.main === module) {
+  const pkgPath = path.resolve(__dirname, '../fixtures/esm-broken-fixture/package.json')
+  console.log('=== Package Entry Points ===\n')
+  const entryPoints = getPackageEntryPoints(pkgPath)
+  entryPoints.forEach(ep => {
+    console.log(`${ep.field}: ${ep.path}`)
+  })
+
+  console.log('\n=== Verification Results ===\n')
+  const throwIfErrors = false
+  const results = verifyPackageEntries(pkgPath, { throwIfErrors })
+  console.log('results', results)
+
+  results.entryPoints.forEach(ep => {
+    console.log('\n──────────────────────────────────────────────────────────────')
+    console.log('PATH:  ', ep.path)
+    console.log(`Field:  "${ep.field}"`)
+    console.log(`Valid:  ${ep.valid ? '✅' : '❌'}`)
+    if (ep.sourceFiles) {
+      console.log(`srcFiles: ${ep.sourceFiles.length} files`)
+    }
+    if (ep.error) {
+      console.log(`Error:`)
+      console.log(ep.error.message)
+    }
+    ep.sourceFiles.forEach(dep => {
+      console.log(`- ${dep}`)
+    })
+  })
+  console.log('\n──────────────────────────────────────────────────────────────')
+  console.log(`\nTotal unique dependencies: ${results.sourceFiles.size}`)
+  console.log(`Total errors: ${results.errors.length}`)
+}