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

Comparing version 3.0.3 to 3.1.0

CHANGELOG.md

 # rollup-plugin-node-resolve changelog
 
+## 3.0.4 (unreleased)
+
+* Update lockfile [#137](https://github.com/rollup/rollup-plugin-node-resolve/issues/137)
+* Update rollup dependency [#138](https://github.com/rollup/rollup-plugin-node-resolve/issues/138)
+* Enable installation from Github [#142](https://github.com/rollup/rollup-plugin-node-resolve/issues/142)
+
 ## 3.0.3
@@ -4,0 +10,0 @@

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

@@ -42,5 +42,11 @@ 'use strict';
 
+	var preserveSymlinks;
+
 	return {
 		name: 'node-resolve',
 
+		options: function options ( options$1 ) {
+			preserveSymlinks = options$1.preserveSymlinks;
+		},
+
 		resolveId: function resolveId$1 ( importee, importer ) {
@@ -64,3 +70,3 @@ if ( /\0/.test( importee ) ) { return null; } // ignore IDs with null character, these belong to other plugins
 
-			var parts = importee.split( /[\/\\]/ );
+			var parts = importee.split( /[/\\]/ );
 			var id = parts.shift();
@@ -80,39 +86,45 @@
 
+				var resolveOptions = {
+					basedir: path.dirname( importer ),
+					packageFilter: function packageFilter ( pkg, pkgPath ) {
+						var pkgRoot = path.dirname( pkgPath );
+						if (options.browser && typeof pkg[ 'browser' ] === 'object') {
+							packageBrowserField = Object.keys(pkg[ 'browser' ]).reduce(function (browser, key) {
+								var resolved = pkg[ 'browser' ][ key ] === false ? false : path.resolve( pkgRoot, pkg[ 'browser' ][ key ] );
+								browser[ key ] = resolved;
+								if ( key[0] === '.' ) {
+									var absoluteKey = path.resolve( pkgRoot, key );
+									browser[ absoluteKey ] = resolved;
+									if ( !path.extname(key) ) {
+										exts.reduce( function ( browser, ext ) {
+											browser[ absoluteKey + ext ] = browser[ key ];
+											return browser;
+										}, browser );
+									}
+								}
+								return browser;
+							}, {});
+						}
+	
+						if (options.browser && typeof pkg[ 'browser' ] === 'string') {
+							pkg[ 'main' ] = pkg[ 'browser' ];
+						} else if ( useModule && pkg[ 'module' ] ) {
+							pkg[ 'main' ] = pkg[ 'module' ];
+						} else if ( useJsnext && pkg[ 'jsnext:main' ] ) {
+							pkg[ 'main' ] = pkg[ 'jsnext:main' ];
+						} else if ( ( useJsnext || useModule ) && !useMain ) {
+							disregardResult = true;
+						}
+						return pkg;
+					},
+					extensions: options.extensions
+				};
+
+				if (preserveSymlinks !== undefined) {
+					resolveOptions.preserveSymlinks = preserveSymlinks;
+				}
+
 				resolveId(
 					importee,
-					Object.assign({
-						basedir: path.dirname( importer ),
-						packageFilter: function packageFilter ( pkg, pkgPath ) {
-							var pkgRoot = path.dirname( pkgPath );
-							if (options.browser && typeof pkg[ 'browser' ] === 'object') {
-								packageBrowserField = Object.keys(pkg[ 'browser' ]).reduce(function (browser, key) {
-									var resolved = pkg[ 'browser' ][ key ] === false ? false : path.resolve( pkgRoot, pkg[ 'browser' ][ key ] );
-									browser[ key ] = resolved;
-									if ( key[0] === '.' ) {
-										var absoluteKey = path.resolve( pkgRoot, key );
-										browser[ absoluteKey ] = resolved;
-										if ( !path.extname(key) ) {
-											exts.reduce( function ( browser, ext ) {
-												browser[ absoluteKey + ext ] = browser[ key ];
-												return browser;
-											}, browser );
-										}
-									}
-									return browser;
-								}, {});
-							}
-
-							if (options.browser && typeof pkg[ 'browser' ] === 'string') {
-								pkg[ 'main' ] = pkg[ 'browser' ];
-							} else if ( useModule && pkg[ 'module' ] ) {
-								pkg[ 'main' ] = pkg[ 'module' ];
-							} else if ( useJsnext && pkg[ 'jsnext:main' ] ) {
-								pkg[ 'main' ] = pkg[ 'jsnext:main' ];
-							} else if ( ( useJsnext || useModule ) && !useMain ) {
-								disregardResult = true;
-							}
-							return pkg;
-						},
-						extensions: options.extensions
-					}, customResolveOptions ),
+					Object.assign( resolveOptions, customResolveOptions ),
 					function ( err, resolved ) {
@@ -127,3 +139,3 @@ if (options.browser && packageBrowserField) {
 						if ( !disregardResult && !err ) {
-							if ( resolved && fs.existsSync( resolved ) ) {
+							if ( !preserveSymlinks && resolved && fs.existsSync( resolved ) ) {
 								resolved = fs.realpathSync( resolved );
@@ -130,0 +142,0 @@ }

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

@@ -1,2 +0,2 @@
-import { dirname, extname, normalize, resolve, sep } from 'path';
+import { dirname, resolve, extname, normalize, sep } from 'path';
 import builtins from 'builtin-modules';
@@ -38,5 +38,11 @@ import resolveId from 'resolve';
 
+	var preserveSymlinks;
+
 	return {
 		name: 'node-resolve',
 
+		options: function options ( options$1 ) {
+			preserveSymlinks = options$1.preserveSymlinks;
+		},
+
 		resolveId: function resolveId$1 ( importee, importer ) {
@@ -60,3 +66,3 @@ if ( /\0/.test( importee ) ) { return null; } // ignore IDs with null character, these belong to other plugins
 
-			var parts = importee.split( /[\/\\]/ );
+			var parts = importee.split( /[/\\]/ );
 			var id = parts.shift();
@@ -76,39 +82,45 @@
 
+				var resolveOptions = {
+					basedir: dirname( importer ),
+					packageFilter: function packageFilter ( pkg, pkgPath ) {
+						var pkgRoot = dirname( pkgPath );
+						if (options.browser && typeof pkg[ 'browser' ] === 'object') {
+							packageBrowserField = Object.keys(pkg[ 'browser' ]).reduce(function (browser, key) {
+								var resolved = pkg[ 'browser' ][ key ] === false ? false : resolve( pkgRoot, pkg[ 'browser' ][ key ] );
+								browser[ key ] = resolved;
+								if ( key[0] === '.' ) {
+									var absoluteKey = resolve( pkgRoot, key );
+									browser[ absoluteKey ] = resolved;
+									if ( !extname(key) ) {
+										exts.reduce( function ( browser, ext ) {
+											browser[ absoluteKey + ext ] = browser[ key ];
+											return browser;
+										}, browser );
+									}
+								}
+								return browser;
+							}, {});
+						}
+	
+						if (options.browser && typeof pkg[ 'browser' ] === 'string') {
+							pkg[ 'main' ] = pkg[ 'browser' ];
+						} else if ( useModule && pkg[ 'module' ] ) {
+							pkg[ 'main' ] = pkg[ 'module' ];
+						} else if ( useJsnext && pkg[ 'jsnext:main' ] ) {
+							pkg[ 'main' ] = pkg[ 'jsnext:main' ];
+						} else if ( ( useJsnext || useModule ) && !useMain ) {
+							disregardResult = true;
+						}
+						return pkg;
+					},
+					extensions: options.extensions
+				};
+
+				if (preserveSymlinks !== undefined) {
+					resolveOptions.preserveSymlinks = preserveSymlinks;
+				}
+
 				resolveId(
 					importee,
-					Object.assign({
-						basedir: dirname( importer ),
-						packageFilter: function packageFilter ( pkg, pkgPath ) {
-							var pkgRoot = dirname( pkgPath );
-							if (options.browser && typeof pkg[ 'browser' ] === 'object') {
-								packageBrowserField = Object.keys(pkg[ 'browser' ]).reduce(function (browser, key) {
-									var resolved = pkg[ 'browser' ][ key ] === false ? false : resolve( pkgRoot, pkg[ 'browser' ][ key ] );
-									browser[ key ] = resolved;
-									if ( key[0] === '.' ) {
-										var absoluteKey = resolve( pkgRoot, key );
-										browser[ absoluteKey ] = resolved;
-										if ( !extname(key) ) {
-											exts.reduce( function ( browser, ext ) {
-												browser[ absoluteKey + ext ] = browser[ key ];
-												return browser;
-											}, browser );
-										}
-									}
-									return browser;
-								}, {});
-							}
-
-							if (options.browser && typeof pkg[ 'browser' ] === 'string') {
-								pkg[ 'main' ] = pkg[ 'browser' ];
-							} else if ( useModule && pkg[ 'module' ] ) {
-								pkg[ 'main' ] = pkg[ 'module' ];
-							} else if ( useJsnext && pkg[ 'jsnext:main' ] ) {
-								pkg[ 'main' ] = pkg[ 'jsnext:main' ];
-							} else if ( ( useJsnext || useModule ) && !useMain ) {
-								disregardResult = true;
-							}
-							return pkg;
-						},
-						extensions: options.extensions
-					}, customResolveOptions ),
+					Object.assign( resolveOptions, customResolveOptions ),
 					function ( err, resolved ) {
@@ -123,3 +135,3 @@ if (options.browser && packageBrowserField) {
 						if ( !disregardResult && !err ) {
-							if ( resolved && fs.existsSync( resolved ) ) {
+							if ( !preserveSymlinks && resolved && fs.existsSync( resolved ) ) {
 								resolved = fs.realpathSync( resolved );
@@ -126,0 +138,0 @@ }

package.json

 {
   "name": "rollup-plugin-node-resolve",
   "description": "Bundle third-party dependencies in node_modules",
-  "version": "3.0.3",
+  "version": "3.1.0",
   "devDependencies": {
-    "buble": "^0.15.2",
-    "es5-ext": "^0.10.11",
-    "eslint": "^3.18.0",
-    "mocha": "^3.0.2",
-    "rollup": "^0.41.6",
-    "rollup-plugin-buble": "^0.15.0",
-    "rollup-plugin-commonjs": "^8.0.2",
+    "buble": "^0.19.3",
+    "es5-ext": "^0.10.39",
+    "eslint": "^4.18.0",
+    "mocha": "^5.0.1",
+    "rollup": "^0.56.3",
+    "rollup-plugin-buble": "^0.19.2",
+    "rollup-plugin-commonjs": "^8.3.0",
     "string-capitalize": "^1.0.1",
-    "vlq": "^0.2.1"
+    "vlq": "^1.0.0"
   },
@@ -24,4 +24,5 @@ "main": "dist/rollup-plugin-node-resolve.cjs.js",
     "posttest": "eslint src test/*.js",
-    "prepublish": "npm test",
-    "lint": "eslint src"
+    "lint": "eslint src",
+    "prepublishOnly": "npm test",
+    "prepare": "npm run build"
   },
@@ -33,3 +34,3 @@ "files": [
   "dependencies": {
-    "builtin-modules": "^1.1.0",
+    "builtin-modules": "^2.0.0",
     "is-module": "^1.0.0",
@@ -36,0 +37,0 @@ "resolve": "^1.1.6"

src/index.js

@@ -31,5 +31,11 @@ import { dirname, resolve, extname, normalize, sep } from 'path';
 
+	let preserveSymlinks;
+
 	return {
 		name: 'node-resolve',
 
+		options ( options ) {
+			preserveSymlinks = options.preserveSymlinks;
+		},
+
 		resolveId ( importee, importer ) {
@@ -53,3 +59,3 @@ if ( /\0/.test( importee ) ) return null; // ignore IDs with null character, these belong to other plugins
 
-			const parts = importee.split( /[\/\\]/ );
+			const parts = importee.split( /[/\\]/ );
 			let id = parts.shift();
@@ -69,39 +75,45 @@
 
+				const resolveOptions = {
+					basedir: dirname( importer ),
+					packageFilter ( pkg, pkgPath ) {
+						const pkgRoot = dirname( pkgPath );
+						if (options.browser && typeof pkg[ 'browser' ] === 'object') {
+							packageBrowserField = Object.keys(pkg[ 'browser' ]).reduce((browser, key) => {
+								const resolved = pkg[ 'browser' ][ key ] === false ? false : resolve( pkgRoot, pkg[ 'browser' ][ key ] );
+								browser[ key ] = resolved;
+								if ( key[0] === '.' ) {
+									const absoluteKey = resolve( pkgRoot, key );
+									browser[ absoluteKey ] = resolved;
+									if ( !extname(key) ) {
+										exts.reduce( ( browser, ext ) => {
+											browser[ absoluteKey + ext ] = browser[ key ];
+											return browser;
+										}, browser );
+									}
+								}
+								return browser;
+							}, {});
+						}
+	
+						if (options.browser && typeof pkg[ 'browser' ] === 'string') {
+							pkg[ 'main' ] = pkg[ 'browser' ];
+						} else if ( useModule && pkg[ 'module' ] ) {
+							pkg[ 'main' ] = pkg[ 'module' ];
+						} else if ( useJsnext && pkg[ 'jsnext:main' ] ) {
+							pkg[ 'main' ] = pkg[ 'jsnext:main' ];
+						} else if ( ( useJsnext || useModule ) && !useMain ) {
+							disregardResult = true;
+						}
+						return pkg;
+					},
+					extensions: options.extensions
+				};
+
+				if (preserveSymlinks !== undefined) {
+					resolveOptions.preserveSymlinks = preserveSymlinks;
+				}
+
 				resolveId(
 					importee,
-					Object.assign({
-						basedir: dirname( importer ),
-						packageFilter ( pkg, pkgPath ) {
-							const pkgRoot = dirname( pkgPath );
-							if (options.browser && typeof pkg[ 'browser' ] === 'object') {
-								packageBrowserField = Object.keys(pkg[ 'browser' ]).reduce((browser, key) => {
-									const resolved = pkg[ 'browser' ][ key ] === false ? false : resolve( pkgRoot, pkg[ 'browser' ][ key ] );
-									browser[ key ] = resolved;
-									if ( key[0] === '.' ) {
-										const absoluteKey = resolve( pkgRoot, key );
-										browser[ absoluteKey ] = resolved;
-										if ( !extname(key) ) {
-											exts.reduce( ( browser, ext ) => {
-												browser[ absoluteKey + ext ] = browser[ key ];
-												return browser;
-											}, browser );
-										}
-									}
-									return browser;
-								}, {});
-							}
-
-							if (options.browser && typeof pkg[ 'browser' ] === 'string') {
-								pkg[ 'main' ] = pkg[ 'browser' ];
-							} else if ( useModule && pkg[ 'module' ] ) {
-								pkg[ 'main' ] = pkg[ 'module' ];
-							} else if ( useJsnext && pkg[ 'jsnext:main' ] ) {
-								pkg[ 'main' ] = pkg[ 'jsnext:main' ];
-							} else if ( ( useJsnext || useModule ) && !useMain ) {
-								disregardResult = true;
-							}
-							return pkg;
-						},
-						extensions: options.extensions
-					}, customResolveOptions ),
+					Object.assign( resolveOptions, customResolveOptions ),
 					( err, resolved ) => {
@@ -116,3 +128,3 @@ if (options.browser && packageBrowserField) {
 						if ( !disregardResult && !err ) {
-							if ( resolved && fs.existsSync( resolved ) ) {
+							if ( !preserveSymlinks && resolved && fs.existsSync( resolved ) ) {
 								resolved = fs.realpathSync( resolved );
@@ -119,0 +131,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

New author

Supply chain risk

A new npm collaborator published a version of the package for the first time. New collaborators are usually benign additions to a project, but do indicate a change to the security surface area of a package.

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

23735

increased by4.94%

Lines of code

435

increased by5.84%

Worsened metrics

Number of medium supply chain risk alerts

1

increased byInfinity%

Dependency changes

• + Addedbuiltin-modules@2.0.0(transitive)

• - Removedbuiltin-modules@1.1.1(transitive)

• Updatedbuiltin-modules@^2.0.0