Merge pull request #705 from murgatroid99/grpc-native_loadPackageDefinition_fix

grpc native: Fix handling of non-service objects in package definitions

packages/grpc-native-core/index.d.ts

@@ -259,11 +259,20 @@ declare module "grpc" {
     readonly [I in keyof ImplementationType]: MethodDefinition<any, any>;
   }
 
+  /**
+   * An object that defines a protobuf type
+   */
+  export interface ProtobufTypeDefinition {
+    format: string;
+    type: object;
+    fileDescriptorProtos: Buffer[];
+  }
+
   /**
    * An object that defines a package containing multiple services
    */
   export type PackageDefinition = {
-    readonly [fullyQualifiedName: string]: ServiceDefinition<any>;
+    readonly [fullyQualifiedName: string]: ServiceDefinition<any> | ProtobufTypeDefinition;
   }
 
   /**

packages/grpc-native-core/index.js

@@ -169,8 +169,12 @@ exports.loadPackageDefinition = function loadPackageDefintion(packageDef) {
       }
       current = current[packageName];
     }
+    if (service.hasOwnProperty('format')) {
+      current[serviceName] = service;
+    } else {
       current[serviceName] = client.makeClientConstructor(service, serviceName, {});
     }
+  }
   return result;
 };
 

packages/grpc-native-core/src/common.js

@@ -317,9 +317,18 @@ exports.zipObject = function(props, values) {
  * @typedef {Object.<string, grpc~MethodDefinition>} grpc~ServiceDefinition
  */
 
+ /**
+  * An object that defines a protobuf type
+  * @typedef {object} grpc~ProtobufTypeDefinition
+  * @param {string} format The format of the type definition object
+  * @param {*} type The type definition object
+  * @param {Buffer[]} fileDescriptorProtos Binary protobuf file
+  *     descriptors for all files loaded to construct this type
+  */
+
 /**
  * An object that defines a package hierarchy with multiple services
- * @typedef {Object.<string, grpc~ServiceDefinition>} grpc~PackageDefinition
+ * @typedef {Object.<string, grpc~ServiceDefinition|grpc~ProtobufTypeDefinition>} grpc~PackageDefinition
  */
 
 /**