feat: add generated code comments (#1037)

Closes #531
stephenh · May 1, 2024 · cdd4a76 · cdd4a76
1 parent 577def4
commit cdd4a76
Show file tree

Hide file tree

Showing 260 changed files with 1,587 additions and 22 deletions.
diff --git a/integration/angular/simple-message.ts b/integration/angular/simple-message.ts
diff --git a/integration/async-iterable-services-abort-signal/simple.ts b/integration/async-iterable-services-abort-signal/simple.ts
diff --git a/integration/async-iterable-services/simple.ts b/integration/async-iterable-services/simple.ts
diff --git a/integration/avoid-import-conflicts-folder-name/ui.ts b/integration/avoid-import-conflicts-folder-name/ui.ts
diff --git a/integration/avoid-import-conflicts-folder-name/ui/simple.ts b/integration/avoid-import-conflicts-folder-name/ui/simple.ts
diff --git a/integration/avoid-import-conflicts-types-only/simple.ts b/integration/avoid-import-conflicts-types-only/simple.ts
diff --git a/integration/avoid-import-conflicts-types-only/simple2.ts b/integration/avoid-import-conflicts-types-only/simple2.ts
diff --git a/integration/avoid-import-conflicts/simple.ts b/integration/avoid-import-conflicts/simple.ts
diff --git a/integration/avoid-import-conflicts/simple2.ts b/integration/avoid-import-conflicts/simple2.ts
diff --git a/integration/barrel-imports/bar.ts b/integration/barrel-imports/bar.ts
diff --git a/integration/barrel-imports/foo.ts b/integration/barrel-imports/foo.ts
diff --git a/integration/batching-with-context-esModuleInterop/batching.ts b/integration/batching-with-context-esModuleInterop/batching.ts
diff --git a/integration/batching-with-context/batching.ts b/integration/batching-with-context/batching.ts
diff --git a/integration/batching/batching.ts b/integration/batching/batching.ts
diff --git a/integration/before-after-request-streaming/example.ts b/integration/before-after-request-streaming/example.ts
diff --git a/integration/before-after-request-streaming/google/protobuf/wrappers.ts b/integration/before-after-request-streaming/google/protobuf/wrappers.ts
diff --git a/integration/before-after-request/simple.ts b/integration/before-after-request/simple.ts
diff --git a/integration/before-after-request/simple2.ts b/integration/before-after-request/simple2.ts
diff --git a/integration/bytes-as-base64-browser/message.ts b/integration/bytes-as-base64-browser/message.ts
diff --git a/integration/bytes-as-base64-node/message.ts b/integration/bytes-as-base64-node/message.ts
diff --git a/integration/bytes-as-base64/message.ts b/integration/bytes-as-base64/message.ts
diff --git a/integration/bytes-node/google/protobuf/wrappers.ts b/integration/bytes-node/google/protobuf/wrappers.ts
diff --git a/integration/bytes-node/point.ts b/integration/bytes-node/point.ts
diff --git a/integration/codegen.ts b/integration/codegen.ts
@@ -4,7 +4,7 @@ import { parse } from "path";
 import { promisify } from "util";
 import { generateFile, makeUtils } from "../src/main";
 import { createTypeMap } from "../src/types";
-import { generateIndexFiles } from "../src/utils";
+import { generateIndexFiles, getVersions } from "../src/utils";
 import { getTsPoetOpts, optionsFromParameter } from "../src/options";
 import { BaseContext, createFileContext } from "../src/context";
 import { generateTypeRegistry } from "../src/generate-type-registry";
@@ -28,6 +28,8 @@ async function generate(binFile: string, baseDir: string, parameter: string) {
   const request = CodeGeneratorRequest.decode(stdin);
   request.parameter = parameter;
 
+  const { protocVersion, tsProtoVersion } = await getVersions(request);
+
   const options = optionsFromParameter(parameter || "");
   const typeMap = createTypeMap(request, options);
 
@@ -42,7 +44,10 @@ async function generate(binFile: string, baseDir: string, parameter: string) {
     const filePath = `${baseDir}/${path}`;
     const dirPath = parse(filePath).dir;
     await promisify(mkdir)(dirPath, { recursive: true }).catch(() => {});
-    await promisify(writeFile)(filePath, code.toString({ ...getTsPoetOpts(options), path }));
+    await promisify(writeFile)(
+      filePath,
+      code.toString({ ...getTsPoetOpts(options, tsProtoVersion, protocVersion, file.name), path }),
+    );
   }
 
   if (options.outputTypeRegistry) {
@@ -54,15 +59,21 @@ async function generate(binFile: string, baseDir: string, parameter: string) {
 
     const filePath = `${baseDir}/${path}`;
 
-    await promisify(writeFile)(filePath, code.toString({ ...getTsPoetOpts(options), path }));
+    await promisify(writeFile)(
+      filePath,
+      code.toString({ ...getTsPoetOpts(options, tsProtoVersion, protocVersion), path }),
+    );
   }
 
   if (options.outputIndex) {
     for (const [path, code] of generateIndexFiles(request.protoFile, options)) {
       const filePath = `${baseDir}/${path}`;
       const dirPath = parse(filePath).dir;
       await promisify(mkdir)(dirPath, { recursive: true }).catch(() => {});
-      await promisify(writeFile)(filePath, code.toString({ ...getTsPoetOpts(options), path }));
+      await promisify(writeFile)(
+        filePath,
+        code.toString({ ...getTsPoetOpts(options, tsProtoVersion, protocVersion), path }),
+      );
     }
   }
 }

diff --git a/integration/const-enum/const-enum.ts b/integration/const-enum/const-enum.ts
diff --git a/integration/default-with-custom-metadata/default-with-custom-metadata.ts b/integration/default-with-custom-metadata/default-with-custom-metadata.ts
diff --git a/integration/default-with-metadata/default-with-metadata.ts b/integration/default-with-metadata/default-with-metadata.ts
diff --git a/integration/emit-default-values-json/google/protobuf/timestamp.ts b/integration/emit-default-values-json/google/protobuf/timestamp.ts
diff --git a/integration/emit-default-values-json/test.ts b/integration/emit-default-values-json/test.ts
diff --git a/integration/enums-as-literals-with-string-enums/enums-as-literals-with-string-enums.ts b/integration/enums-as-literals-with-string-enums/enums-as-literals-with-string-enums.ts
diff --git a/integration/enums-as-literals/enums-as-literals.ts b/integration/enums-as-literals/enums-as-literals.ts
diff --git a/integration/enums-with-unrecognized-name-value/test.ts b/integration/enums-with-unrecognized-name-value/test.ts
diff --git a/integration/extensions/test.ts b/integration/extensions/test.ts
diff --git a/integration/fieldmask-optional-all/fieldmask-optional.ts b/integration/fieldmask-optional-all/fieldmask-optional.ts
diff --git a/integration/fieldmask-optional-all/google/protobuf/field_mask.ts b/integration/fieldmask-optional-all/google/protobuf/field_mask.ts