ts-proto 2.10.1 → 2.11.1

package/README.markdown

@@ -414,6 +414,9 @@ Generated code will be placed in the Gradle build directory.
 
   The default behavior is `keys_json`, i.e. both will be camel cased, and `json_name` will be used if set.
 
+- With `--ts_proto_opt=protoJsonFormat=true` (default), the `toJSON` and `fromJSON`  methods follow the [protoJSON format](https://protobuf.dev/programming-guides/json). This means `toJSON` will output standard lowerCamelCase keys (or custom `json_name` if set). `fromJSON` will accept both the lowerCamelCase key (or `json_name`) AND the original proto field name.
+With `--ts_proto_opt=protoJsonFormat=false`, strict proto3 compliance is disabled. The JSON keys will simply follow the `snakeToCamel` option, and `fromJSON` will only accept that specific format.
+
 - With `--ts_proto_opt=outputEncodeMethods=false`, the `Message.encode` and `Message.decode` methods for working with protobuf-encoded/binary data will not be output.
 
   This is useful if you want "only types".

package/build/src/main.js

@@ -1964,6 +1964,9 @@ function generateFromJson(ctx, fullName, fullTypeName, messageDesc) {
         };
         const noDefaultValue = !options.initializeFieldsAsUndefined &&
             (0, types_1.isOptionalProperty)(field, messageDesc.options, options, currentFile.isProto3Syntax);
+        // if the protoJsonFormat option is enabled and the field name is different from the json name, we need to
+        // accept both formats in the fromJSON method.
+        let protoJsonComparison = (0, ts_poet_1.code) ``;
         // and then use the snippet to handle repeated fields if necessary
         if (canonicalFromJson[fullTypeName]?.[fieldName]) {
             chunks.push((0, ts_poet_1.code) `${fieldName}: ${canonicalFromJson[fullTypeName][fieldName]("object")},`);
@@ -1975,23 +1978,45 @@ function generateFromJson(ctx, fullName, fullTypeName, messageDesc) {
                 const i = convertFromObjectKey(ctx, messageDesc, field, "key");
                 if ((0, types_1.shouldGenerateJSMapType)(ctx, messageDesc, field)) {
                     const fallback = noDefaultValue ? (0, utils_1.nullOrUndefined)(options) : "new Map()";
+                    if (options.protoJsonFormat && field.name !== field.jsonName) {
+                        const protoJsonProperty = (0, utils_1.getPropertyAccessor)("object", field.name);
+                        protoJsonComparison = (0, ts_poet_1.code) `
+                : ${ctx.utils.isObject}(${protoJsonProperty})
+                ? (${ctx.utils.globalThis}.Object.entries(${protoJsonProperty}) as [string, any][]).reduce((acc: ${fieldType}, [key, value]: [string, any]) => {
+                    acc.set(${i}, ${readSnippet("value")});
+                    return acc;
+                  }, new Map())
+              `;
+                    }
                     chunks.push((0, ts_poet_1.code) `
             ${fieldKey}: ${ctx.utils.isObject}(${jsonProperty})
               ? (${ctx.utils.globalThis}.Object.entries(${jsonProperty}) as [string, any][]).reduce((acc: ${fieldType}, [key, value]: [string, any]) => {
                   acc.set(${i}, ${readSnippet("value")});
                   return acc;
                 }, new Map())
+              ${protoJsonComparison}
               : ${fallback},
           `);
                 }
                 else {
                     const fallback = noDefaultValue ? (0, utils_1.nullOrUndefined)(options) : "{}";
+                    if (options.protoJsonFormat && field.name !== field.jsonName) {
+                        const protoJsonProperty = (0, utils_1.getPropertyAccessor)("object", field.name);
+                        protoJsonComparison = (0, ts_poet_1.code) `
+                 : ${ctx.utils.isObject}(${protoJsonProperty})
+                 ? (${ctx.utils.globalThis}.Object.entries(${protoJsonProperty}) as [string, any][]).reduce((acc: ${fieldType}, [key, value]: [string, any]) => {
+                     acc[${i}] = ${readSnippet("value")};
+                     return acc;
+                   }, {})
+               `;
+                    }
                     chunks.push((0, ts_poet_1.code) `
             ${fieldKey}: ${ctx.utils.isObject}(${jsonProperty})
               ? (${ctx.utils.globalThis}.Object.entries(${jsonProperty}) as [string, any][]).reduce((acc: ${fieldType}, [key, value]: [string, any]) => {
                   acc[${i}] = ${readSnippet("value")};
                   return acc;
                 }, {})
+              ${protoJsonComparison}
               : ${fallback},
           `);
                 }
@@ -2000,12 +2025,22 @@ function generateFromJson(ctx, fullName, fullTypeName, messageDesc) {
                 const fallback = noDefaultValue ? (0, utils_1.nullOrUndefined)(options) : "[]";
                 const needMap = readSnippet("e").toString() !== (0, ts_poet_1.code) `e`.toString();
                 if (!needMap) {
-                    chunks.push((0, ts_poet_1.code) `${fieldKey}: ${ctx.utils.globalThis}.Array.isArray(${jsonPropertyOptional}) ? [...${jsonProperty}] : [],`);
+                    if (options.protoJsonFormat && field.name !== field.jsonName) {
+                        const protoJsonProperty = (0, utils_1.getPropertyAccessor)("object", field.name);
+                        const protoJsonPropertyOptional = (0, utils_1.getPropertyAccessor)("object", field.name, true);
+                        protoJsonComparison = (0, ts_poet_1.code) ` : ${ctx.utils.globalThis}.Array.isArray(${protoJsonPropertyOptional}) ? [...${protoJsonProperty}]`;
+                    }
+                    chunks.push((0, ts_poet_1.code) `${fieldKey}: ${ctx.utils.globalThis}.Array.isArray(${jsonPropertyOptional}) ? [...${jsonProperty}] ${protoJsonComparison} : [],`);
                 }
                 else {
                     // Explicit `any` type required to make TS with noImplicitAny happy. `object` is also `any` here.
+                    if (options.protoJsonFormat && field.name !== field.jsonName) {
+                        const protoJsonProperty = (0, utils_1.getPropertyAccessor)("object", field.name);
+                        const protoJsonPropertyOptional = (0, utils_1.getPropertyAccessor)("object", field.name, true);
+                        protoJsonComparison = (0, ts_poet_1.code) ` : ${ctx.utils.globalThis}.Array.isArray(${protoJsonPropertyOptional}) ? ${protoJsonProperty}.map((e: any) => ${readSnippet("e")})`;
+                    }
                     chunks.push((0, ts_poet_1.code) `
-            ${fieldKey}: ${ctx.utils.globalThis}.Array.isArray(${jsonPropertyOptional}) ? ${jsonProperty}.map((e: any) => ${readSnippet("e")}): ${fallback},
+            ${fieldKey}: ${ctx.utils.globalThis}.Array.isArray(${jsonPropertyOptional}) ? ${jsonProperty}.map((e: any) => ${readSnippet("e")}) ${protoJsonComparison} : ${fallback},
           `);
                 }
             }
@@ -2022,33 +2057,59 @@ function generateFromJson(ctx, fullName, fullTypeName, messageDesc) {
             const ternaryIf = (0, ts_poet_1.code) `${ctx.utils.isSet}(${jsonProperty})`;
             const ternaryThen = (0, ts_poet_1.code) `{ $case: '${fieldName}', ${valueName}: ${readSnippet(`${jsonProperty}`)}`;
             chunks.push((0, ts_poet_1.code) `${ternaryIf} ? ${ternaryThen}} : `);
+            if (options.protoJsonFormat && field.name !== field.jsonName) {
+                const protoJsonProperty = (0, utils_1.getPropertyAccessor)("object", field.name);
+                const ternaryOriginalIf = (0, ts_poet_1.code) `${ctx.utils.isSet}(${protoJsonProperty})`;
+                const ternaryOriginalThen = (0, ts_poet_1.code) `{ $case: '${fieldName}', ${valueName}: ${readSnippet(`${protoJsonProperty}`)}`;
+                chunks.push((0, ts_poet_1.code) `${ternaryOriginalIf} ? ${ternaryOriginalThen}} : `);
+            }
             if (field === lastCase) {
                 chunks.push((0, ts_poet_1.code) `${(0, utils_1.nullOrUndefined)(options)},`);
             }
         }
         else if ((0, types_1.isAnyValueType)(field)) {
+            if (options.protoJsonFormat && field.name !== field.jsonName) {
+                const protoJsonProperty = (0, utils_1.getPropertyAccessor)("object", field.name);
+                const protoJsonPropertyOptional = (0, utils_1.getPropertyAccessor)("object", field.name, true);
+                protoJsonComparison = (0, ts_poet_1.code) ` : ${ctx.utils.isSet}(${protoJsonPropertyOptional}) ? ${readSnippet(`${protoJsonProperty}`)}`;
+            }
             chunks.push((0, ts_poet_1.code) `${fieldKey}: ${ctx.utils.isSet}(${jsonPropertyOptional})
         ? ${readSnippet(`${jsonProperty}`)}
+        ${protoJsonComparison}
         : ${(0, utils_1.nullOrUndefined)(options)},
       `);
         }
         else if ((0, types_1.isStructType)(field)) {
+            if (options.protoJsonFormat && field.name !== field.jsonName) {
+                const protoJsonProperty = (0, utils_1.getPropertyAccessor)("object", field.name);
+                protoJsonComparison = (0, ts_poet_1.code) ` : ${ctx.utils.isObject}(${protoJsonProperty}) ? ${readSnippet(`${protoJsonProperty}`)}`;
+            }
             chunks.push((0, ts_poet_1.code) `${fieldKey}: ${ctx.utils.isObject}(${jsonProperty})
           ? ${readSnippet(`${jsonProperty}`)}
+          ${protoJsonComparison}
           : ${(0, utils_1.nullOrUndefined)(options)},`);
         }
         else if ((0, types_1.isListValueType)(field)) {
+            if (options.protoJsonFormat && field.name !== field.jsonName) {
+                const protoJsonProperty = (0, utils_1.getPropertyAccessor)("object", field.name);
+                protoJsonComparison = (0, ts_poet_1.code) ` : ${ctx.utils.globalThis}.Array.isArray(${protoJsonProperty}) ? ${readSnippet(`${protoJsonProperty}`)}`;
+            }
             chunks.push((0, ts_poet_1.code) `
         ${fieldKey}: ${ctx.utils.globalThis}.Array.isArray(${jsonProperty})
           ? ${readSnippet(`${jsonProperty}`)}
+          ${protoJsonComparison}
           : ${(0, utils_1.nullOrUndefined)(options)},
       `);
         }
         else {
             const fallback = (0, types_1.isWithinOneOf)(field) || noDefaultValue ? (0, utils_1.nullOrUndefined)(options) : (0, types_1.defaultValue)(ctx, field);
+            if (options.protoJsonFormat && field.name !== field.jsonName) {
+                const protoJsonProperty = (0, utils_1.getPropertyAccessor)("object", field.name);
+                protoJsonComparison = (0, ts_poet_1.code) ` : ${ctx.utils.isSet}(${protoJsonProperty}) ? ${readSnippet(`${protoJsonProperty}`)}`;
+            }
             chunks.push((0, ts_poet_1.code) `
-        ${fieldKey}: ${ctx.utils.isSet}(${jsonProperty})
-          ? ${readSnippet(`${jsonProperty}`)}
+        ${fieldKey}: ${ctx.utils.isSet}(${jsonProperty}) ? ${readSnippet(`${jsonProperty}`)}
+        ${protoJsonComparison}
           : ${fallback},
       `);
         }

package/build/src/options.d.ts

@@ -45,6 +45,7 @@ export declare enum OutputSchemaOption {
 export type Options = {
     context: boolean;
     snakeToCamel: Array<"json" | "keys">;
+    protoJsonFormat: boolean;
     forceLong: LongOption;
     useJsTypeOverride: boolean;
     globalThisPolyfill: boolean;

package/build/src/options.js

@@ -60,6 +60,7 @@ function defaultOptions() {
     return {
         context: false,
         snakeToCamel: ["json", "keys"],
+        protoJsonFormat: true,
         emitDefaultValues: [],
         globalThisPolyfill: false,
         forceLong: LongOption.NUMBER,
@@ -204,6 +205,11 @@ function optionsFromParameter(parameter) {
     else if (typeof options.snakeToCamel === "string") {
         options.snakeToCamel = options.snakeToCamel.split("_");
     }
+    if (options.protoJsonFormat && !options.snakeToCamel.includes("json")) {
+        // protoJSONFormat implies snakeToCamel=json, Message field names must be mapped to lowerCamelCase and become JSON object keys.
+        // If the json_name field option is specified, the specified value will be used as the key instead.
+        options.snakeToCamel.push("json");
+    }
     if (options.emitDefaultValues === "json-methods") {
         options.emitDefaultValues = ["json-methods"];
     }

package/build/src/schema.js

@@ -70,7 +70,7 @@ function generateSchema(ctx, fileDesc, sourceInfo) {
         }
     });
     const dependencies = fileDesc.dependency.map((dep) => {
-        return (0, ts_poet_1.code) `${(0, utils_1.impFile)(options, `protoMetadata@./${dep.replace(".proto", "")}`)}`;
+        return (0, ts_poet_1.code) `${(0, utils_1.impFile)(options, `protoMetadata@./${dep.replace(".proto", "")}${options.fileSuffix}`)}`;
     });
     // Use toObject so that we get enums as numbers (instead of the default toJSON behavior)
     const descriptor = ts_proto_descriptors_1.FileDescriptorProto.fromPartial(fileDesc);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ts-proto",
-  "version": "2.10.1",
+  "version": "2.11.1",
   "description": "",
   "main": "build/src/plugin.js",
   "repository": "github:stephenh/ts-proto",