feat: let user pass package name to the generator

templates/typescript_gapic/package.json.njk

@@ -16,7 +16,7 @@ limitations under the License.
 
 -#}
 {
-  "name": "{{ api.naming.productName.toKebabCase() }}",
+  "name": "{{ api.publishName}}",
   "version": "0.1.0",
   "description": "{{ api.naming.productName }} client for Node.js",
   "repository": "googleapis/nodejs-{{ api.naming.productName.toKebabCase() }}",

templates/typescript_gapic/system-test/fixtures/sample/src/index.js.njk

@@ -19,7 +19,7 @@ limitations under the License.
 {{license.license()}}
 
 /* eslint-disable node/no-missing-require, no-unused-vars */
-const {{ api.naming.productName.toKebabCase()}} = require('{{ api.naming.productName.toKebabCase() }}');
+const {{ api.naming.productName.toKebabCase()}} = require('{{ api.publishName }}');
 
 function main() {
 {%- for service in api.services %}

templates/typescript_gapic/system-test/fixtures/sample/src/index.ts.njk

@@ -23,7 +23,7 @@ import {
     {{- serviceJoiner() -}}
     {{- service.name.toPascalCase() + 'Client' -}}
   {%- endfor -%}
-} from '{{ api.naming.productName.toKebabCase() }}';
+} from '{{ api.publishName }}';
 
 function main() {
 {%- for service in api.services %}

typescript/src/generator.ts

@@ -39,6 +39,8 @@ export class Generator {
   request: plugin.google.protobuf.compiler.CodeGeneratorRequest;
   response: plugin.google.protobuf.compiler.CodeGeneratorResponse;
   grpcServiceConfig: plugin.grpc.service_config.ServiceConfig;
+  //This field is for users passing proper module name for system test.
+  publishName?: string;
 
   constructor() {
     this.request = plugin.google.protobuf.compiler.CodeGeneratorRequest.create();
@@ -66,7 +68,7 @@ export class Generator {
   }
 
   private async readGrpcServiceConfig(parameter: string) {
-    const match = parameter.match(/^["']?grpc-service-config=([^"]+)["']?$/);
+    const match = parameter.match(/^["']?grpc-service-config=([^"]+)["']?/);
     if (!match) {
       throw new Error(`Parameter ${parameter} was not recognized.`);
     }
@@ -83,13 +85,19 @@ export class Generator {
     );
   }
 
+  private async readPublishPackageName(parameter: string) {
+    const match = parameter.match(/["']?package-name=([^"]+)["']?$/);
+    if (match && match.length > 1) this.publishName = match[1];
+  }
+
   async initializeFromStdin() {
     const inputBuffer = await getStdin.buffer();
     this.request = plugin.google.protobuf.compiler.CodeGeneratorRequest.decode(
       inputBuffer
     );
     if (this.request.parameter) {
       await this.readGrpcServiceConfig(this.request.parameter);
+      await this.readPublishPackageName(this.request.parameter);
     }
   }
 
@@ -125,7 +133,8 @@ export class Generator {
     const api = new API(
       this.request.protoFile,
       packageName,
-      this.grpcServiceConfig
+      this.grpcServiceConfig,
+      this.publishName
     );
     return api;
   }

typescript/src/schema/api.ts

@@ -32,19 +32,28 @@ export class API {
   hostName?: string;
   port?: string;
   mainServiceName?: string;
+  //This field is for users passing roper module name for system test.
+  publishName: string;
   // oauth_scopes: plugin.google.protobuf.IServiceOptions.prototype[".google.api.oauthScopes"];
   // TODO: subpackages
 
   constructor(
     fileDescriptors: plugin.google.protobuf.IFileDescriptorProto[],
     packageName: string,
-    grpcServiceConfig: plugin.grpc.service_config.ServiceConfig
+    grpcServiceConfig: plugin.grpc.service_config.ServiceConfig,
+    publishName?: string
   ) {
     this.naming = new Naming(
       fileDescriptors.filter(
         fd => fd.package && fd.package.startsWith(packageName)
       )
     );
+    // users specify the actual package name, if not, set it to product name.
+    if (publishName) {
+      this.publishName = publishName;
+    } else {
+      this.publishName = this.naming.productName.toKebabCase();
+    }
     // construct resource map
     const resourceMap = getResourceMap(fileDescriptors);
     // parse resource map to Proto constructor

typescript/src/start_script.ts

@@ -34,6 +34,8 @@ const argv = yargs
   .describe('output_dir', 'Path to a directory for the generated code')
   .alias('grpc-service-config', 'grpc_service_config')
   .describe('grpc-service-config', 'Path to gRPC service config JSON')
+  .alias('package-name', 'package-name')
+  .describe('package-name', 'Publish package name')
   .alias('common-proto-path', 'common_protos_path')
   .describe(
     'common_proto_path',
@@ -43,6 +45,7 @@ const argv = yargs
   google/example/api/v1/api.proto`).argv;
 const outputDir = argv.outputDir as string;
 const grpcServiceConfig = argv.grpcServiceConfig as string | undefined;
+const packageName = argv.packageName as string | undefined;
 
 const protoDirs: string[] = [];
 if (argv.I) {
@@ -71,6 +74,9 @@ if (grpcServiceConfig) {
     `--typescript_gapic_opt="grpc-service-config=${grpcServiceConfig}"`
   );
 }
+if (packageName) {
+  protocCommand.push(`--typescript_gapic_opt="package-name=${packageName}"`);
+}
 protocCommand.push(...protoDirsArg);
 protocCommand.push(...protoFiles);
 try {

typescript/test/testdata/texttospeech/package.json.baseline

@@ -1,5 +1,5 @@
 {
-  "name": "texttospeech",
+  "name": "@google-cloud/text-to-speech",
   "version": "0.1.0",
   "description": "Texttospeech client for Node.js",
   "repository": "googleapis/nodejs-texttospeech",

typescript/test/testdata/texttospeech/system-test/fixtures/sample/src/index.js.baseline

@@ -18,7 +18,7 @@
 
 
 /* eslint-disable node/no-missing-require, no-unused-vars */
-const texttospeech = require('texttospeech');
+const texttospeech = require('@google-cloud/text-to-speech');
 
 function main() {
   const textToSpeechClient = new texttospeech.TextToSpeechClient();

typescript/test/testdata/texttospeech/system-test/fixtures/sample/src/index.ts.baseline

@@ -16,7 +16,7 @@
 // ** https://github.com/googleapis/gapic-generator-typescript **
 // ** All changes to this file may be overwritten. **
 
-import {TextToSpeechClient} from 'texttospeech';
+import {TextToSpeechClient} from '@google-cloud/text-to-speech';
 
 function main() {
   const textToSpeechClient = new TextToSpeechClient();

typescript/test/unit/grpc-client-config.ts → typescript/test/unit/extra-option.ts

@@ -58,12 +58,13 @@ const BASELINE_DIR = path.join(
   'texttospeech'
 );
 
+const PACKAGE_NAME = '@google-cloud/text-to-speech';
 const SRCDIR = path.join(cwd, 'build', 'src');
 const CLI = path.join(SRCDIR, 'cli.js');
 
-describe('gRPC Client Config', () => {
+describe('Package Name & grpc Config', () => {
   describe('Generate Text-to-Speech library', () => {
-    it('Generated proto list should have same output with baseline.', function() {
+    it('Generated library name & grpc Config should be same with baseline.', function() {
       this.timeout(10000);
       if (fs.existsSync(OUTPUT_DIR)) {
         rimraf.sync(OUTPUT_DIR);
@@ -82,6 +83,7 @@ describe('gRPC Client Config', () => {
           `-I ${GOOGLE_GAX_PROTOS_DIR} ` +
           `-I ${PROTOS_DIR} ` +
           `--grpc-service-config=${GRPC_SERVICE_CONFIG} ` +
+          `--package-name=${PACKAGE_NAME} ` +
           TTS_PROTO_FILE
       );
       assert(equalToBaseline(OUTPUT_DIR, BASELINE_DIR));