Merge pull request #53 from rezzza/jail

Add a jail feature

README.md

@@ -34,7 +34,7 @@ rollup({
 
       // if there's something your bundle requires that you DON'T
       // want to include, add it to 'skip'. Local and relative imports
-      // can be skipped by giving the full filepath. E.g., 
+      // can be skipped by giving the full filepath. E.g.,
       // `path.resolve('src/relative-dependency.js')`
       skip: [ 'some-big-dependency' ],  // Default: []
 
@@ -50,7 +50,11 @@ rollup({
       // whether to prefer built-in modules (e.g. `fs`, `path`) or
       // local ones with the same names
       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: '/'
+
     })
   ]
 }).then( bundle => bundle.write({ dest: 'bundle.js', format: 'iife' }) );
@@ -64,10 +68,10 @@ rollup({
     nodeResolve({ jsnext: true, main: true }),
     commonjs()
   ]
-}).then(bundle => bundle.write({ 
-  dest: 'bundle.js', 
+}).then(bundle => bundle.write({
+  dest: 'bundle.js',
   moduleName: 'MyModule',
-  format: 'iife' 
+  format: 'iife'
 })).catch(err => console.log(err.stack));
 ```
 

src/index.js

@@ -1,4 +1,4 @@
-import { dirname, resolve, normalize } from 'path';
+import { dirname, resolve, normalize, sep } from 'path';
 import builtins from 'builtin-modules';
 import _nodeResolve from 'resolve';
 import browserResolve from 'browser-resolve';
@@ -14,6 +14,7 @@ export default function nodeResolve ( options = {} ) {
 	const useMain = options.main !== false;
 	const isPreferBuiltinsSet = options.preferBuiltins === true || options.preferBuiltins === false;
 	const preferBuiltins = isPreferBuiltinsSet ? options.preferBuiltins : true;
+	const jail = options.jail || '/';
 
 	const onwarn = options.onwarn || CONSOLE_WARN;
 	const resolveId = options.browser ? browserResolve : _nodeResolve;
@@ -79,6 +80,8 @@ export default function nodeResolve ( options = {} ) {
 									);
 								}
 								accept( null );
+							} else if (resolved.indexOf(normalize(jail.trim(sep))) !== 0) {
+								accept( null );
 							} else {
 								accept( resolved );
 							}

test/samples/jail/main.js

@@ -0,0 +1,3 @@
+import uppercase from 'string/uppercase.js';
+
+export default uppercase( 'foo' );

test/test.js

@@ -415,4 +415,26 @@ describe( 'rollup-plugin-node-resolve', function () {
 			assert.equal( err.message, 'Could not resolve \'foo\' from ' + path.resolve( __dirname, entry ) );
 		});
 	});
+
+	it( 'mark as external to module outside the jail', () => {
+		return rollup.rollup({
+			entry: 'samples/jail/main.js',
+			plugins: [ nodeResolve({
+				jail: `${__dirname}/samples/`
+			}) ]
+		}).then( (bundle) => {
+			assert.deepEqual(bundle.imports, [ 'string/uppercase.js' ]);
+		});
+	});
+
+	it( 'bundle module defined inside the jail', () => {
+		return rollup.rollup({
+			entry: 'samples/jail/main.js',
+			plugins: [ nodeResolve({
+				jail: `${__dirname}/`
+			}) ]
+		}).then( (bundle) => {
+			assert.deepEqual(bundle.imports, []);
+		});
+	});
 });