webpack - npm Package Compare versions

Comparing version 0.1.4 to 0.1.5

examples/code-splitted-require.context/README.md

@@ -61,3 +61,3 @@ # example.js
 	require.ensure(1, function(require) {
-		callback(require(1)("./"+templateName));
+		callback(require(/* ../require.context/templates */1)("./"+templateName));
 	});
@@ -127,3 +127,3 @@ }
   modulesFirstChunk: 1,
-  fileSizes: { 'output.js': 1855, '1.output.js': 729 } }
+  fileSizes: { 'output.js': 1889, '1.output.js': 729 } }
 ```
@@ -130,0 +130,0 @@

examples/code-splitting/README.md

@@ -55,7 +55,7 @@ # example.js
 
-var a = require(1);
-var b = require(2);
+var a = require(/* a */1);
+var b = require(/* b */4);
 require.ensure(1, function(require) {
-    require(2).xyz();
-    var d = require(3);
+    require(/* b */4).xyz();
+    var d = require(/* d */2);
 });
@@ -71,3 +71,3 @@
 /******/
-/******/2: function(module, exports, require) {
+/******/4: function(module, exports, require) {
 
@@ -116,3 +116,3 @@ // module b
   modulesFirstChunk: 3,
-  fileSizes: { 'output.js': 1948, '1.output.js': 200 } }
+  fileSizes: { 'output.js': 1935, '1.output.js': 200 } }
 ```
@@ -119,0 +119,0 @@

examples/require.context/README.md

@@ -48,3 +48,3 @@ # example.js
 function getTemplate(templateName) {
-	return require(1)("./"+templateName);
+	return require(/* ./templates */1)("./"+templateName);
 }
@@ -59,3 +59,3 @@ console.log(getTemplate("a"));
 /***/module.exports = function(name) {
-/***/	var map = {"./b.js":3,"./a.js":2,"./c.js":4};
+/***/	var map = {"./a.js":2,"./b.js":3,"./c.js":4};
 /***/	return require(map[name]||map[name+".web.js"]||map[name+".js"]);
@@ -103,3 +103,3 @@ /***/};
   modulesFirstChunk: 5,
-  fileSizes: { 'output.js': 1454 } }
+  fileSizes: { 'output.js': 1471 } }
 ```
@@ -106,0 +106,0 @@

lib/buildDeps.js

@@ -162,2 +162,3 @@ /*
 			var contextModule = depTree.modules[dirname] = {
+				name: contextModuleName,
 				id: depTree.nextModuleId++,
@@ -164,0 +165,0 @@ requireMap: {},

lib/writeSource.js

@@ -13,6 +13,9 @@ /*
 		if(requireItem.nameRange && requireItem.id !== undefined) {
+			var prefix = "";
+			if(requireItem.name)
+				prefix += "/* " + requireItem.name + " */";
 			replaces.push({
 				from: requireItem.nameRange[0],
 				to: requireItem.nameRange[1],
-				value: "" + requireItem.id
+				value: prefix + requireItem.id
 			});
@@ -23,2 +26,5 @@ }
 		var postfix = "";
+		var prefix = "";
+		if(contextItem.name)
+			prefix += "/* " + contextItem.name + " */";
 		if(contextItem.require) {
@@ -28,3 +34,3 @@ replaces.push({
 				to: contextItem.calleeRange[1],
-				value: "require(" + ((contextItem.id || "throw new Error('there is not id for this')") + "") + ")"
+				value: "require(" + prefix + ((contextItem.id || "throw new Error('there is not id for this')") + "") + ")"
 			});
@@ -40,3 +46,3 @@ replaces.push({
 				to: contextItem.expressionRange[1],
-				value: "require(" + ((contextItem.id || "throw new Error('there is not id for this')") + "") + ")" + postfix
+				value: "require(" + prefix + ((contextItem.id || "throw new Error('there is not id for this')") + "") + ")" + postfix
 			});
@@ -43,0 +49,0 @@ }

package.json

 {
 	"name": "webpack",
-	"version": "0.1.4",
+	"version": "0.1.5",
 	"author": "Tobias Koppers @sokra",
@@ -5,0 +5,0 @@ "description": "Packs CommonJs Modules for the browser. Allows to split your codebase into multiple bundles, which can be loaded on demand.",

test/browsertest/lib/index.web.js

@@ -16,7 +16,7 @@ window.test(true, "index.js should be replaced with index.web.js");
 window.test(require("./singluar.js").value === 1, "sigular module loaded");
-require("./singluar.js").value = 2;
+require("./sing" + "luar.js").value = 2;
 window.test(require("./singluar").value === 2, "exported object is singluar");
 window.test(require("subfilemodule") === "subfilemodule", "Modules as single file should load");
 window.test(require.context("../templates")("./tmpl") === "test template", "Context should work");
-window.test(require.context("../templates")("./subdir/tmpl.js") === "subdir test template", "Context should work with subdirectories");
+window.test(require . context ( "." + "." + "/" + "templ" + "ates" ) ( "./subdir/tmpl.js" ) === "subdir test template", "Context should work with subdirectories and splitted");
 var template = "tmpl";
@@ -28,2 +28,4 @@ window.test(require("../templates/" + template) === "test template", "Automatical context should work");
 	window.test(contextRequire("./singluar").value === 2, "Context works in chunk");
+	var singl = "singl";
+	window.test(require("." + "/" + singl + "uar").value === 2, "Context works in chunk, when splitted");
 });
@@ -30,0 +32,0 @@

New alerts

Dynamic require

Supply chain risk

Dynamic require can indicate the package is performing dangerous or unsafe dynamic code execution.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

69938

increased by0.69%

Lines of code

1574

increased by0.58%

Worsened metrics

Number of low supply chain risk alerts

31

increased by10.71%

No dependency changes