prepublish script for bin files, fixes #511; swallow enum value options (no wrapper), fixes #510; expose zero on LongBits, fixes #508

Author: dcodeIO

dist/protobuf.js

@@ -34,12 +34,14 @@
     "zuul-local": "zuul --ui tape --no-coverage --concurrency 1 --local 8080 --disable-tunnel -- tests/*.js",
     "bench": "node bench",
     "all": "npm run lint && npm run test && npm run build && npm run types && npm run docs && npm run bench",
-    "postinstall": "npm install @types/node @types/long"
+    "prepublish": "node scripts/prepublish.js"
   },
   "optionalDependencies": {
     "long": "^3.2.0"
   },
   "devDependencies": {
+    "@types/long": "^3.0.31",
+    "@types/node": "0.0.1",
     "benchmark": "^2.1.2",
     "browserify": "^13.1.1",
     "bundle-collapser": "^1.2.1",

dist/protobuf.js.map

@@ -0,0 +1,7 @@
+var path = require("path"),
+    fs   = require("fs");
+
+// ensure LF on bin files
+[ path.join(__dirname, "..", "bin", "pbjs") ].forEach(function(file) {
+    fs.writeFileSync(file, fs.readFileSync(file).toString("utf8").replace(/\r?\n/g, "\n"), "utf8");
+});

dist/protobuf.min.js

@@ -360,7 +360,8 @@ function parse(source, root) {
         var name = token;
         skip("=");
         var value = parseId(next(), true);
-        parseInlineOptions(parent.values[name] = new Number(value)); // eslint-disable-line no-new-wrappers
+        parent.values[name] = value;
+        parseInlineOptions({}); // skips enum value options
     }
 
     function parseOption(parent, token) {

dist/protobuf.min.js.gz

@@ -151,15 +151,19 @@ ServicePrototype.remove = function remove(object) {
 
 /**
  * Creates a runtime service using the specified rpc implementation.
- * @param {RPCImpl} rpc RPC implementation
+ * @param {function(Method, Uint8Array, function)} rpc RPC implementation ({@link RPCImpl|see})
  * @param {boolean} [requestDelimited=false] Whether request data is length delimited
  * @param {boolean} [responseDelimited=false] Whether response data is length delimited
  * @returns {Object} Runtime service
  */
 ServicePrototype.create = function create(rpc, requestDelimited, responseDelimited) {
     var rpcService = {};
+    Object.defineProperty(rpcService, "$rpc", {
+        value: rpc
+    });
     this.getMethodsArray().forEach(function(method) {
-        rpcService[method.resolve().name] = function(request, callback) {
+        rpcService[method.name] = function(request, callback) {
+            method.resolve();
             var requestData;
             try {
                 requestData = (requestDelimited && method.resolvedRequestType.encodeDelimited(request) || method.resolvedRequestType.encode(request)).finish();

dist/protobuf.min.js.map

@@ -35,7 +35,7 @@ var LongBitsPrototype = LongBits.prototype;
  * @memberof util.LongBits
  * @type {util.LongBits}
  */
-var zero = new LongBits(0, 0);
+var zero = LongBits.zero = new LongBits(0, 0);
 
 zero.toNumber = function() { return 0; };
 zero.zzEncode = zero.zzDecode = function() { return this; };
@@ -71,9 +71,13 @@ LongBits.fromNumber = function fromNumber(value) {
  * @returns {util.LongBits} Instance
  */
 LongBits.from = function from(value) {
-    return typeof value === 'number'
-        ? LongBits.fromNumber(value)
-        : new LongBits(value.low >>> 0, value.high >>> 0);
+    switch (typeof value) {
+        case 'number':
+            return LongBits.fromNumber(value);
+        case 'string':
+            value = util.Long.fromString(value); // throws without a long lib
+    }
+    return (value.low || value.high) && new LongBits(value.low >>> 0, value.high >>> 0) || zero;
 };
 
 /**