ipjs - npm Package Compare versions

Comparing version 5.1.1 to 5.1.2

package.json

 {
   "name": "ipjs",
-  "version": "5.1.1",
+  "version": "5.1.2",
   "description": "Universal JavaScript build and packaging system",
@@ -5,0 +5,0 @@ "main": "src/build.js",

src/package/index.js

@@ -168,7 +168,20 @@ import { promises as fs, existsSync } from 'fs'
 
-  async stubFiles (dist, overrides) {
-    if (typeof overrides === 'string') {
-      overrides = { '.': overrides }
+  calcuateStubs (json) {
+    if (typeof json.browser === 'string') {
+      return { '.': json.browser }
     }
 
+    const stubs = {}
+
+    for (const key in json.browser) {
+      if (!json.exports[key]) {
+        continue
+      }
+      stubs[key] = json.exports[key].require
+    }
+
+    return stubs
+  }
+
+  async stubFiles (dist, overrides) {
     await Promise.all(
@@ -254,2 +267,3 @@ Object.keys(overrides).map(async (file) => {
       if (_import !== _browser) {
+        // https://github.com/mikeal/ipjs/pull/12#issue-943461643
         json.browser[_join('esm', _import)] = _join('esm', _browser)
@@ -261,6 +275,10 @@ json.browser[_join('cjs', _import)] = _join('cjs', _browser)
     if (json.exports.import) {
+      // https://github.com/mikeal/ipjs/pull/18#issue-974673903
       json.exports = json.exports.import
       json.browser = json.browser.import
     }
-    await this.stubFiles(dist, json.browser)
+
+    const stubs = this.calcuateStubs(json)
+    await this.stubFiles(dist, stubs)
+
     let files = Promise.all(pending)
@@ -270,2 +288,3 @@ pending.push(writeFile(new URL(dist + '/package.json'), JSON.stringify(json, null, 2)))
       type: 'module',
+      // https://github.com/mikeal/ipjs/pull/12#issuecomment-879816902
       browser: esmBrowser
@@ -272,0 +291,0 @@ }

test/fixtures/pkg-kitchensink/input/package.json

@@ -21,4 +21,8 @@ {
       "import": "./src/secondary.js"
+    },
+    "./tertiary": {
+      "browser": "./src/tertiary.browser.js",
+      "import": "./src/tertiary.js"
     }
   }
 }

test/fixtures/pkg-kitchensink/output-main/esm/package.json

 {
   "type": "module",
   "browser": {
-    "./src/index.js": "./src/browser.js"
+    "./src/index.js": "./src/browser.js",
+    "./src/tertiary.js": "./src/tertiary.browser.js"
   }
 }

test/fixtures/pkg-kitchensink/output-main/index.js

@@ -1,1 +0,1 @@
-module.exports = require('./cjs/src/browser.js')
+module.exports = require('./cjs/src/index.js')

test/fixtures/pkg-kitchensink/output-main/package.json

@@ -22,2 +22,7 @@ {
       "import": "./esm/src/secondary.js"
+    },
+    "./tertiary": {
+      "browser": "./esm/src/tertiary.browser.js",
+      "require": "./cjs/src/tertiary.js",
+      "import": "./esm/src/tertiary.js"
     }
@@ -29,4 +34,7 @@ },
     "./cjs/src/index.js": "./cjs/src/browser.js",
-    "./secondary": "./cjs/src/secondary.js"
+    "./secondary": "./cjs/src/secondary.js",
+    "./tertiary": "./cjs/src/tertiary.browser.js",
+    "./esm/src/tertiary.js": "./esm/src/tertiary.browser.js",
+    "./cjs/src/tertiary.js": "./cjs/src/tertiary.browser.js"
   }
 }

test/fixtures/pkg-kitchensink/output-notests/esm/package.json

 {
   "type": "module",
   "browser": {
-    "./src/index.js": "./src/browser.js"
+    "./src/index.js": "./src/browser.js",
+    "./src/tertiary.js": "./src/tertiary.browser.js"
   }
 }

test/fixtures/pkg-kitchensink/output-notests/index.js

@@ -1,1 +0,1 @@
-module.exports = require('./cjs/src/browser.js')
+module.exports = require('./cjs/src/index.js')

test/fixtures/pkg-kitchensink/output-notests/package.json

@@ -21,2 +21,7 @@ {
       "import": "./esm/src/secondary.js"
+    },
+    "./tertiary": {
+      "browser": "./esm/src/tertiary.browser.js",
+      "require": "./cjs/src/tertiary.js",
+      "import": "./esm/src/tertiary.js"
     }
@@ -28,4 +33,7 @@ },
     "./cjs/src/index.js": "./cjs/src/browser.js",
-    "./secondary": "./cjs/src/secondary.js"
+    "./secondary": "./cjs/src/secondary.js",
+    "./tertiary": "./cjs/src/tertiary.browser.js",
+    "./esm/src/tertiary.js": "./esm/src/tertiary.browser.js",
+    "./cjs/src/tertiary.js": "./cjs/src/tertiary.browser.js"
   }
 }

test/fixtures/pkg-kitchensink/output-tests/esm/package.json

 {
   "type": "module",
   "browser": {
-    "./src/index.js": "./src/browser.js"
+    "./src/index.js": "./src/browser.js",
+    "./src/tertiary.js": "./src/tertiary.browser.js"
   }
 }

test/fixtures/pkg-kitchensink/output-tests/index.js

@@ -1,1 +0,1 @@
-module.exports = require('./cjs/src/browser.js')
+module.exports = require('./cjs/src/index.js')

test/fixtures/pkg-kitchensink/output-tests/package.json

@@ -21,2 +21,7 @@ {
       "import": "./esm/src/secondary.js"
+    },
+    "./tertiary": {
+      "browser": "./esm/src/tertiary.browser.js",
+      "require": "./cjs/src/tertiary.js",
+      "import": "./esm/src/tertiary.js"
     }
@@ -28,4 +33,7 @@ },
     "./cjs/src/index.js": "./cjs/src/browser.js",
-    "./secondary": "./cjs/src/secondary.js"
+    "./secondary": "./cjs/src/secondary.js",
+    "./tertiary": "./cjs/src/tertiary.browser.js",
+    "./esm/src/tertiary.js": "./esm/src/tertiary.browser.js",
+    "./cjs/src/tertiary.js": "./cjs/src/tertiary.browser.js"
   }
 }

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

64806

increased by4.19%

Number of package files

142

increased by13.6%

Lines of code

1621

increased by2.14%

No dependency changes