browserify - npm Package Compare versions

Comparing version 0.2.2 to 0.2.3

index.js

@@ -40,3 +40,3 @@ var fs = require('fs');
     
-    return fs.readFileSync(__dirname + '/wrappers/prelude.js', 'utf8')
+    var src = fs.readFileSync(__dirname + '/wrappers/prelude.js', 'utf8')
         + fs.readFileSync(__dirname + '/wrappers/node_compat.js', 'utf8')
@@ -48,2 +48,3 @@ + (shim ? source.modules('es5-shim')['es5-shim'] : '')
     ;
+    return opts.filter ? opts.filter(src) : src;
 };
@@ -60,5 +61,7 @@
         var f = __dirname + '/builtins/' + file;
+        var src = fs.readFileSync(f, 'utf8').replace(/^#![^\n]*\n/, '');
+        
         return wrapperBody
             .replace('$body', function () {
-                return fs.readFileSync(f, 'utf8');
+                return src;
             })
@@ -113,3 +116,5 @@ .replace(/\$filename/g, function () {
             source : wrapperBody
-                .replace('$body', function () { return body })
+                .replace('$body', function () {
+                    return body.replace(/^#![^\n]*\n/, '');
+                })
                 .replace(/\$filename/g, function () {
@@ -124,2 +129,3 @@ return JSON.stringify(pkgname);
         var body = fs.readFileSync(require.resolve(libname), 'utf8');
+        
         var pkgname = ((opts.name ? opts.name + '/' : '') + libname)
@@ -134,3 +140,3 @@ .replace(/\/\.\//g, '/')
             .replace('$body', function () {
-                return body
+                return body.replace(/^#![^\n]*\n/, '');
             })
@@ -175,5 +181,6 @@ .replace(/\$filename/g, function () {
                 .map(function (name) {
+                    var src = mods[name].toString().replace(/^#![^\n]*\n/, '');
                     return wrapperBody
                         .replace('$body', function () {
-                            return mods[name].toString()
+                            return src;
                         })
@@ -180,0 +187,0 @@ .replace(/\$filename/g, function () {

package.json

 {
     "name" : "browserify",
-    "version" : "0.2.2",
+    "version" : "0.2.3",
     "description" : "Browser-side require() for js directories and npm modules",
@@ -5,0 +5,0 @@ "main" : "./index.js",

test/simple.js

@@ -12,5 +12,13 @@ var assert = require('assert');
     
+    var tf = setTimeout(function () {
+        assert.fail('never filtered');
+    }, 5000);
+    
     server.use(require('browserify')({
         base : __dirname + '/simple',
         mount : '/bundle.js',
+        filter : function (src) {
+            clearTimeout(tf);
+            return src + ';\n filterHook();\n'
+        },
     }));
@@ -23,2 +31,6 @@ server.listen(port, makeRequest);
     
+    var th = setTimeout(function () {
+        assert.fail('effects of filter not used');
+    }, 5000);
+    
     function makeRequest () {
@@ -32,3 +44,7 @@ clearTimeout(to);
             
-            var context = {};
+            var context = {
+                filterHook : function () {
+                    clearTimeout(th);
+                }
+            };
             var src = '';
@@ -35,0 +51,0 @@ res.on('data', function (buf) {

test/simple/bar.js

@@ -0,3 +1,5 @@
+#!/usr/bin/env node
+
 exports.coeff = function (x) {
     return Math.log(x) / Math.log(2) + 1;
 };

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

36543

increased by4.13%

Number of package files

44

increased by4.76%

Lines of code

832

increased by5.32%

No dependency changes