rollup-plugin-node-resolve - npm Package Compare versions

Comparing version 2.1.0 to 2.1.1

CHANGELOG.md

 # rollup-plugin-node-resolve changelog
 
+## 2.1.1
+
+* Prevent `jail` from breaking builds on Windows ([#93](https://github.com/rollup/rollup-plugin-node-resolve/issues/93))
+
 ## 2.1.0
@@ -4,0 +8,0 @@

dist/rollup-plugin-node-resolve.cjs.js

@@ -30,3 +30,3 @@ 'use strict';
 	var customResolveOptions = options.customResolveOptions || {};
-	var jail = options.jail || '/';
+	var jail = options.jail;
 
@@ -101,3 +101,3 @@ var onwarn = options.onwarn || CONSOLE_WARN;
 								accept( null );
-							} else if (resolved.indexOf(path.normalize(jail.trim(path.sep))) !== 0) {
+							} else if ( jail && resolved.indexOf( path.normalize( jail.trim( path.sep ) ) ) !== 0 ) {
 								accept( null );
@@ -104,0 +104,0 @@ } else {

dist/rollup-plugin-node-resolve.es.js

@@ -26,3 +26,3 @@ import { dirname, normalize, resolve, sep } from 'path';
 	var customResolveOptions = options.customResolveOptions || {};
-	var jail = options.jail || '/';
+	var jail = options.jail;
 
@@ -97,3 +97,3 @@ var onwarn = options.onwarn || CONSOLE_WARN;
 								accept( null );
-							} else if (resolved.indexOf(normalize(jail.trim(sep))) !== 0) {
+							} else if ( jail && resolved.indexOf( normalize( jail.trim( sep ) ) ) !== 0 ) {
 								accept( null );
@@ -100,0 +100,0 @@ } else {

package.json

 {
   "name": "rollup-plugin-node-resolve",
   "description": "Bundle third-party dependencies in node_modules",
-  "version": "2.1.0",
+  "version": "2.1.1",
   "devDependencies": {
@@ -6,0 +6,0 @@ "buble": "^0.15.2",

README.md

@@ -16,9 +16,13 @@ # rollup-plugin-node-resolve
 ```js
+// rollup.config.js
 import { rollup } from 'rollup';
-import nodeResolve from 'rollup-plugin-node-resolve';
+import resolve from 'rollup-plugin-node-resolve';
 
-rollup({
+export default {
   entry: 'main.js',
+  dest: 'bundle.js',
+  moduleName: 'MyModule',
+  format: 'iife'
   plugins: [
-    nodeResolve({
+    resolve({
       // use "module" field for ES6 module if possible
@@ -53,26 +57,38 @@ module: true, // Default: true
       // local ones with the same names
-      preferBuiltins: false  // Default: true
+      preferBuiltins: false,  // Default: true
 
       // Lock the module search in this path (like a chroot). Module defined
       // outside this path will be mark has external
-      jail: '/my/jail/path' // Default: '/'
+      jail: '/my/jail/path', // Default: '/'
 
+      // Any additional options that should be passed through
+      // to node-resolve
+      customResolveOptions: {
+        moduleDirectory: 'js_modules'
+      }
     })
   ]
-}).then( bundle => bundle.write({ dest: 'bundle.js', format: 'iife' }) );
+};
+```
 
-// alongside rollup-plugin-commonjs, for using non-ES6 third party modules
+## Using with rollup-plugin-commonjs
+
+Since most packages in your node_modules folder are probably legacy CommonJS rather than JavaScript modules, you may need to use [rollup-plugin-commonjs](https://github.com/rollup/rollup-plugin-commonjs):
+
+```js
+// rollup.config.js
+import { rollup } from 'rollup';
+import resolve from 'rollup-plugin-node-resolve';
 import commonjs from 'rollup-plugin-commonjs';
 
-rollup({
+export default {
   entry: 'main.js',
+  dest: 'bundle.js',
+  moduleName: 'MyModule',
+  format: 'iife'
   plugins: [
-    nodeResolve({ jsnext: true, main: true }),
+    resolve({ jsnext: true, main: true }),
     commonjs()
   ]
-}).then(bundle => bundle.write({
-  dest: 'bundle.js',
-  moduleName: 'MyModule',
-  format: 'iife'
-})).catch(err => console.log(err.stack));
+};
 ```
@@ -79,0 +95,0 @@

src/index.js

@@ -19,3 +19,3 @@ import { dirname, resolve, normalize, sep } from 'path';
 	const customResolveOptions = options.customResolveOptions || {};
-	const jail = options.jail || '/';
+	const jail = options.jail;
 
@@ -90,3 +90,3 @@ const onwarn = options.onwarn || CONSOLE_WARN;
 								accept( null );
-							} else if (resolved.indexOf(normalize(jail.trim(sep))) !== 0) {
+							} else if ( jail && resolved.indexOf( normalize( jail.trim( sep ) ) ) !== 0 ) {
 								accept( null );
@@ -93,0 +93,0 @@ } else {

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

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

Improved metrics

Total package byte prevSize

25241

increased by2.23%

Number of lines in readme file

98

increased by19.51%

Number of medium supply chain risk alerts

0

decreased by-100%

No dependency changes