feat: load protos from JSON, grpc-fallback support

* [CHANGE ME] Re-generated  to pick up changes in the API or client library generator.

* fixes

* fix webpack.config.js

* no browser support yet

src/v3beta1/translation_service_client.js

@@ -17,7 +17,6 @@
 const gapicConfig = require('./translation_service_client_config.json');
 const gax = require('google-gax');
 const path = require('path');
-const protobuf = require('protobufjs');
 
 const VERSION = require('../../../package.json').version;
 
@@ -59,6 +58,16 @@ class TranslationServiceClient {
     opts = opts || {};
     this._descriptors = {};
 
+    if (global.isBrowser) {
+      // If we're in browser, we use gRPC fallback.
+      opts.fallback = true;
+    }
+
+    // If we are in browser, we are already using fallback because of the
+    // "browser" field in package.json.
+    // But if we were explicitly requested to use fallback, let's do it now.
+    const gaxModule = !global.isBrowser && opts.fallback ? gax.fallback : gax;
+
     const servicePath =
       opts.servicePath || opts.apiEndpoint || this.constructor.servicePath;
 
@@ -75,36 +84,51 @@ class TranslationServiceClient {
     // Create a `gaxGrpc` object, with any grpc-specific options
     // sent to the client.
     opts.scopes = this.constructor.scopes;
-    const gaxGrpc = new gax.GrpcClient(opts);
+    const gaxGrpc = new gaxModule.GrpcClient(opts);
 
     // Save the auth object to the client, for use by other methods.
     this.auth = gaxGrpc.auth;
 
     // Determine the client header string.
-    const clientHeader = [
-      `gl-node/${process.versions.node}`,
-      `grpc/${gaxGrpc.grpcVersion}`,
-      `gax/${gax.version}`,
-      `gapic/${VERSION}`,
-    ];
+    const clientHeader = [];
+
+    if (typeof process !== 'undefined' && 'versions' in process) {
+      clientHeader.push(`gl-node/${process.versions.node}`);
+    }
+    clientHeader.push(`gax/${gaxModule.version}`);
+    if (opts.fallback) {
+      clientHeader.push(`gl-web/${gaxModule.version}`);
+    } else {
+      clientHeader.push(`grpc/${gaxGrpc.grpcVersion}`);
+    }
+    clientHeader.push(`gapic/${VERSION}`);
     if (opts.libName && opts.libVersion) {
       clientHeader.push(`${opts.libName}/${opts.libVersion}`);
     }
 
     // Load the applicable protos.
+    // For Node.js, pass the path to JSON proto file.
+    // For browsers, pass the JSON content.
+
+    const nodejsProtoPath = path.join(
+      __dirname,
+      '..',
+      '..',
+      'protos',
+      'protos.json'
+    );
     const protos = gaxGrpc.loadProto(
-      path.join(__dirname, '..', '..', 'protos'),
-      ['google/cloud/translate/v3beta1/translation_service.proto']
+      opts.fallback ? require('../../protos/protos.json') : nodejsProtoPath
     );
 
     // This API contains "path templates"; forward-slash-separated
     // identifiers to uniquely identify resources within the API.
     // Create useful helper objects for these.
     this._pathTemplates = {
-      glossaryPathTemplate: new gax.PathTemplate(
+      glossaryPathTemplate: new gaxModule.PathTemplate(
         'projects/{project}/locations/{location}/glossaries/{glossary}'
       ),
-      locationPathTemplate: new gax.PathTemplate(
+      locationPathTemplate: new gaxModule.PathTemplate(
         'projects/{project}/locations/{location}'
       ),
     };
@@ -113,28 +137,21 @@ class TranslationServiceClient {
     // (e.g. 50 results at a time, with tokens to get subsequent
     // pages). Denote the keys used for pagination and results.
     this._descriptors.page = {
-      listGlossaries: new gax.PageDescriptor(
+      listGlossaries: new gaxModule.PageDescriptor(
         'pageToken',
         'nextPageToken',
         'glossaries'
       ),
     };
-    let protoFilesRoot = new gax.GoogleProtoFilesRoot();
-    protoFilesRoot = protobuf.loadSync(
-      path.join(
-        __dirname,
-        '..',
-        '..',
-        'protos',
-        'google/cloud/translate/v3beta1/translation_service.proto'
-      ),
-      protoFilesRoot
-    );
+
+    const protoFilesRoot = opts.fallback
+      ? gaxModule.protobuf.Root.fromJSON(require('../../protos/protos.json'))
+      : gaxModule.protobuf.loadSync(nodejsProtoPath);
 
     // This API contains "long-running operations", which return a
     // an Operation object that allows for tracking of the operation,
     // rather than holding a request open.
-    this.operationsClient = new gax.lro({
+    this.operationsClient = new gaxModule.lro({
       auth: gaxGrpc.auth,
       grpc: gaxGrpc.grpc,
     }).operationsClient(opts);
@@ -159,17 +176,17 @@ class TranslationServiceClient {
     );
 
     this._descriptors.longrunning = {
-      batchTranslateText: new gax.LongrunningDescriptor(
+      batchTranslateText: new gaxModule.LongrunningDescriptor(
         this.operationsClient,
         batchTranslateTextResponse.decode.bind(batchTranslateTextResponse),
         batchTranslateTextMetadata.decode.bind(batchTranslateTextMetadata)
       ),
-      createGlossary: new gax.LongrunningDescriptor(
+      createGlossary: new gaxModule.LongrunningDescriptor(
         this.operationsClient,
         createGlossaryResponse.decode.bind(createGlossaryResponse),
         createGlossaryMetadata.decode.bind(createGlossaryMetadata)
       ),
-      deleteGlossary: new gax.LongrunningDescriptor(
+      deleteGlossary: new gaxModule.LongrunningDescriptor(
         this.operationsClient,
         deleteGlossaryResponse.decode.bind(deleteGlossaryResponse),
         deleteGlossaryMetadata.decode.bind(deleteGlossaryMetadata)
@@ -192,7 +209,11 @@ class TranslationServiceClient {
     // Put together the "service stub" for
     // google.cloud.translation.v3beta1.TranslationService.
     const translationServiceStub = gaxGrpc.createStub(
-      protos.google.cloud.translation.v3beta1.TranslationService,
+      opts.fallback
+        ? protos.lookupService(
+            'google.cloud.translation.v3beta1.TranslationService'
+          )
+        : protos.google.cloud.translation.v3beta1.TranslationService,
       opts
     );
 
@@ -209,18 +230,16 @@ class TranslationServiceClient {
       'deleteGlossary',
     ];
     for (const methodName of translationServiceStubMethods) {
-      this._innerApiCalls[methodName] = gax.createApiCall(
-        translationServiceStub.then(
-          stub =>
-            function() {
-              const args = Array.prototype.slice.call(arguments, 0);
-              return stub[methodName].apply(stub, args);
-            },
-          err =>
-            function() {
-              throw err;
-            }
-        ),
+      const innerCallPromise = translationServiceStub.then(
+        stub => (...args) => {
+          return stub[methodName].apply(stub, args);
+        },
+        err => () => {
+          throw err;
+        }
+      );
+      this._innerApiCalls[methodName] = gaxModule.createApiCall(
+        innerCallPromise,
         defaults[methodName],
         this._descriptors.page[methodName] ||
           this._descriptors.longrunning[methodName]

src/v3beta1/translation_service_proto_list.json

@@ -0,0 +1,3 @@
+[
+  "../../protos/google/cloud/translate/v3beta1/translation_service.proto"
+]

synth.metadata

@@ -1,19 +1,19 @@
 {
-  "updateTime": "2019-08-28T11:23:43.481205Z",
+  "updateTime": "2019-08-31T11:21:47.616048Z",
   "sources": [
     {
       "generator": {
         "name": "artman",
-        "version": "0.35.1",
-        "dockerImage": "googleapis/artman@sha256:b11c7ea0d0831c54016fb50f4b796d24d1971439b30fbc32a369ba1ac887c384"
+        "version": "0.36.1",
+        "dockerImage": "googleapis/artman@sha256:7c20f006c7a62d9d782e2665647d52290c37a952ef3cd134624d5dd62b3f71bd"
       }
     },
     {
       "git": {
         "name": "googleapis",
         "remote": "https://github.com/googleapis/googleapis.git",
-        "sha": "dbd38035c35083507e2f0b839985cf17e212cb1c",
-        "internalRef": "265796259"
+        "sha": "82809578652607c8ee29d9e199c21f28f81a03e0",
+        "internalRef": "266247326"
       }
     },
     {

synth.py

@@ -24,7 +24,8 @@
 versions = ['v3beta1']
 for version in versions:
     library = gapic.node_library('translate', version)
-    s.copy(library, excludes=['src/index.js', 'README.md', 'package.json'])
+    s.copy(library, excludes=['src/index.js', 'src/browser.js', 'README.md', 'package.json'])
+# note: no browser.js support until we fully support TypeScript
 
 # Update path discovery due to build/ dir and TypeScript conversion.
 s.replace("src/v3beta1/translation_service_client.js", "../../package.json", "../../../package.json")

test/gapic-v3beta1.js

@@ -46,6 +46,13 @@ describe('TranslationServiceClient', () => {
     assert(client);
   });
 
+  it('should create a client with gRPC fallback', () => {
+    const client = new translateModule.v3beta1.TranslationServiceClient({
+      fallback: true,
+    });
+    assert(client);
+  });
+
   describe('translateText', () => {
     it('invokes translateText without error', done => {
       const client = new translateModule.v3beta1.TranslationServiceClient({

webpack.config.js

@@ -0,0 +1,46 @@
+// Copyright 2019 Google LLC
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     https://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+module.exports = {
+  entry: './src/browser.js',
+  output: {
+    library: 'translate',
+    filename: './translate.js',
+  },
+  node: {
+    child_process: 'empty',
+    fs: 'empty',
+    crypto: 'empty',
+  },
+  resolve: {
+    extensions: ['.js', '.json'],
+  },
+  module: {
+    rules: [
+      {
+        test: /node_modules[\\/]retry-request[\\/]/,
+        use: 'null-loader',
+      },
+      {
+        test: /node_modules[\\/]https-proxy-agent[\\/]/,
+        use: 'null-loader',
+      },
+      {
+        test: /node_modules[\\/]gtoken[\\/]/,
+        use: 'null-loader',
+      },
+    ],
+  },
+  mode: 'production',
+};