CLI: Reuse specified root name in pbjs to be able to split definitions over multiple files more easily, see #653

Author: dcodeIO

.vscode/settings.json

@@ -1,3 +1,4 @@
 {
-    "typescript.tsdk": "./node_modules/typescript/lib"
+    "typescript.tsdk": "./node_modules/typescript/lib",
+    "vsicons.presets.angular": false
 }

cli/targets/static.js

@@ -39,7 +39,8 @@ function static_target(root, options, callback) {
             else
                 push("// Exported root namespace");
         }
-        push("var $root = {};");
+        var rootName = config.root || "default";
+        push("var $root = $protobuf.roots[" + JSON.stringify(rootName) + "] || ($protobuf.roots[" + JSON.stringify(rootName) + "] = {});");
         buildNamespace(null, root);
         push("");
         if (config.comments)

cli/util.js

@@ -124,7 +124,6 @@ exports.wrap = function(OUTPUT, options) {
         // otherwise fetch the custom one
         wrap = fs.readFileSync(path.resolve(process.cwd(), name)).toString("utf8");
     }
-    wrap = wrap.replace(/%ROOT%/g, JSON.stringify(options.root || "default"));
     wrap = wrap.replace(/%DEPENDENCY%/g, JSON.stringify(options.dependency || "protobufjs"));
     wrap = wrap.replace(/( *)%OUTPUT%/, function($0, $1) {
         return $1.length ? OUTPUT.replace(/^/mg, $1) : OUTPUT;

cli/wrappers/amd.js

@@ -3,5 +3,5 @@ define(["protobuf"], function($protobuf) {
 
     %OUTPUT%
 
-    return $protobuf.roots[%ROOT%] = $root;
+    return $root;
 });

cli/wrappers/commonjs.js

@@ -4,4 +4,4 @@ var $protobuf = require(%DEPENDENCY%);
 
 %OUTPUT%
 
-module.exports = $protobuf.roots[%ROOT%] = $root;
+module.exports = $root;

cli/wrappers/default.js

@@ -11,5 +11,5 @@
 
     %OUTPUT%
 
-    return $protobuf.roots[%ROOT%] = $root;
+    return $root;
 });

cli/wrappers/es6.js

@@ -2,6 +2,4 @@ import * as $protobuf from "protobufjs";
 
 %OUTPUT%
 
-$protobuf.roots[%ROOT%] = $root;
-
 export { $root as default };

tests/data/comments.js

@@ -12,7 +12,7 @@ var $Reader = $protobuf.Reader,
 var $lazyTypes = [];
 
 // Exported root namespace
-var $root = {};
+var $root = $protobuf.roots["test_comments"] || ($protobuf.roots["test_comments"] = {});
 
 $root.Test1 = (function() {
 
@@ -47,7 +47,7 @@ $root.Test1 = (function() {
     Test1.prototype.field2 = 0;
 
     /**
-     * Field with a comment.
+     * Field with a comment and a <a href="http://example.com/foo/">link</a>
      * @type {boolean}
      */
     Test1.prototype.field3 = false;
@@ -377,4 +377,4 @@ $root.Test3 = (function() {
 // Resolve lazy type references to actual types
 $util.lazyResolve($root, $lazyTypes);
 
-module.exports = $protobuf.roots["test_comments"] = $root;
+module.exports = $root;

tests/data/convert.js

@@ -12,7 +12,7 @@ var $Reader = $protobuf.Reader,
 var $lazyTypes = [];
 
 // Exported root namespace
-var $root = {};
+var $root = $protobuf.roots["test_convert"] || ($protobuf.roots["test_convert"] = {});
 
 $root.Message = (function() {
 
@@ -520,4 +520,4 @@ $root.Message = (function() {
 // Resolve lazy type references to actual types
 $util.lazyResolve($root, $lazyTypes);
 
-module.exports = $protobuf.roots["test_convert"] = $root;
+module.exports = $root;

tests/data/mapbox/vector_tile.js

@@ -12,7 +12,7 @@ var $Reader = $protobuf.Reader,
 var $lazyTypes = [];
 
 // Exported root namespace
-var $root = {};
+var $root = $protobuf.roots["test_vector_tile"] || ($protobuf.roots["test_vector_tile"] = {});
 
 $root.vector_tile = (function() {
 
@@ -1182,4 +1182,4 @@ $root.vector_tile = (function() {
 // Resolve lazy type references to actual types
 $util.lazyResolve($root, $lazyTypes);
 
-module.exports = $protobuf.roots["test_vector_tile"] = $root;
+module.exports = $root;