rollup-plugin-worker-factory - npm Package Compare versions

Comparing version 0.4.1 to 0.5.0

CHANGELOG.md

@@ -0,1 +1,8 @@
+<a name="0.5.0"></a>
+# [0.5.0](https://github.com/brandonocasey/rollup-plugin-worker-factory/compare/v0.4.1...v0.5.0) (2021-02-16)
+
+### Bug Fixes
+
+* should work with minifiers and platforms without workers ([4685d39](https://github.com/brandonocasey/rollup-plugin-worker-factory/commit/4685d39))
+
 <a name="0.4.1"></a>
@@ -2,0 +9,0 @@ ## [0.4.1](https://github.com/brandonocasey/rollup-plugin-worker-factory/compare/v0.4.0...v0.4.1) (2021-01-25)

package.json

 {
   "name": "rollup-plugin-worker-factory",
-  "version": "0.4.1",
+  "version": "0.5.0",
   "description": "Bundle web workers that work in nodejs and the browser, without a separate build target.",
@@ -5,0 +5,0 @@ "main": "src/index.js",

src/get-worker-string.js

@@ -6,2 +6,1 @@ const getWorkerString = function(fn) {
 export default getWorkerString;
-

src/index.js

 const rollup = require('rollup');
+const fs = require('fs');
 const path = require('path');
-const universalFactory = 'rollup-plugin-worker-factory/src/universal-factory.js';
-const nodeFactory = 'rollup-plugin-worker-factory/src/node-factory.js';
-const browserFactory = 'rollup-plugin-worker-factory/src/browser-factory.js';
-const getWorkerString = 'rollup-plugin-worker-factory/src/get-worker-string.js';
-const mockFactory = 'rollup-plugin-worker-factory/src/mock-factory.js';
+const universal = 'rollup-plugin-worker-factory/src/universal.js';
+const node = 'rollup-plugin-worker-factory/src/node.js';
+const browser = 'rollup-plugin-worker-factory/src/browser.js';
+const mock = 'rollup-plugin-worker-factory/src/mock.js';
+const getWorkerString = fs.readFileSync(path.join(__dirname, 'get-worker-string.js'), 'utf8');
 
@@ -15,2 +16,5 @@ module.exports = function(options) {
     resolveId(importee, importer) {
+      if (importee === 'rollup-plugin-worker-factory/src/get-worker-string') {
+        return importee;
+      }
       // if this is not an id we can resolve return
@@ -29,2 +33,5 @@ if (importee.indexOf('worker!') !== 0) {
     load(id) {
+      if (id === 'rollup-plugin-worker-factory/src/get-worker-string') {
+        return getWorkerString;
+      }
       if (id.indexOf('worker!') !== 0) {
@@ -35,10 +42,10 @@ return null;
 
-      let factoryPath = universalFactory;
+      let factoryPath = universal;
 
       if (options && options.type === 'mock') {
-        factoryPath = mockFactory;
+        factoryPath = mock;
       } else if (options && options.type === 'browser') {
-        factoryPath = browserFactory;
+        factoryPath = browser;
       } else if (options && options.type === 'node') {
-        factoryPath = nodeFactory;
+        factoryPath = node;
       }
@@ -52,7 +59,2 @@
         external: [
-          browserFactory,
-          universalFactory,
-          nodeFactory,
-          mockFactory,
-          getWorkerString,
           'worker_threads'
@@ -73,11 +75,11 @@ ]
 
-        // we make self the first argument even though it will almost always
-        // be a global so that mockFactory doesn't have to "eval" code
-        return Promise.resolve(`import workerFactory from "${factoryPath}";\n` +
+        const newCode = '' +
           `/* rollup-plugin-worker-factory start for ${id} */\n` +
-          'const workerFunction = function(self) {\n' +
-          code +
-          '}\n' +
-          `/* rollup-plugin-worker-factory end for ${id} */\n` +
-          'export default workerFactory(workerFunction);');
+          `import {transform, factory} from "${factoryPath}";\n` +
+          'import getWorkerString from "rollup-plugin-worker-factory/src/get-worker-string";\n' +
+          `const workerCode = transform(getWorkerString(function() {\n${code}\n}));\n` +
+          'export default factory(workerCode);\n' +
+          `/* rollup-plugin-worker-factory end for ${id} */\n`;
+
+        return Promise.resolve(newCode);
       });
@@ -84,0 +86,0 @@ }

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

Major refactor

Supply chain risk

Package has recently undergone a major refactor. It may be unstable or indicate significant internal changes. Use caution when updating to versions that include significant changes.

Found 1 instance in 1 package

Filesystem access

Supply chain risk

Accesses the file system, and could potentially read sensitive data.

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

Uses eval

Supply chain risk

Package uses eval() which is a dangerous function. This prevents the code from running in certain environments and increases the risk that the code may contain exploits or malicious behavior.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

23686

increased by3.26%

Lines of code

233

increased by11.48%

Worsened metrics

Number of low supply chain risk alerts

1

increased byInfinity%

No dependency changes