Tate/fix json client

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## v0.0.59
+
+- Fixes generated JSON client when using nested messages. The generated JSON serialization names were invalid for nested messages. See [#176](https://github.com/tatethurston/TwirpScript/issues/176) for more context.
+
 ## v0.0.58
 
 - Better insight into internal server Errors. TwirpScript catches any errors thrown internally. Errors that aren't an `instanceof` `TwirpError` present to users as a generic Twirp `internal_error` for security so that internal details don't leak. This can hurt the DX for identifying issues during development, and also can hide important debugging information when relying on error reporting using `hooks`.

public.package.json

@@ -1,6 +1,6 @@
 {
   "name": "twirpscript",
-  "version": "0.0.58",
+  "version": "0.0.59",
   "description": "Protobuf RPC framework for JavaScript and TypeScript",
   "license": "MIT",
   "author": "Tate <[email protected]>",

src/autogenerate/index.ts

@@ -822,10 +822,10 @@ export async function ${method.name}JSON(${input}${printIfTypescript(
       )}, config${printIfTypescript(
         `?: ClientConfiguration`
       )})${printIfTypescript(`: Promise<${method.output}>`)} {
-  const response = await JSONrequest('${path}', ${
-        method.input
-      }JSON.encode(${input}), config);
-  return ${method.output}JSON.decode(response);
+  const response = await JSONrequest('${path}', ${addJSONSuffixToFullyQualifiedName(
+        method.input!
+      )}.encode(${input}), config);
+  return ${addJSONSuffixToFullyQualifiedName(method.output!)}.decode(response);
 }
 
 `;
@@ -868,7 +868,15 @@ function writeServers(
     name: '${[packageName, service.name].filter(Boolean).join(".")}',
     methods: {\n`;
     service.methods.forEach((method) => {
-      result += `${method.name}: { name: '${method.name}', handler: service.${method.name}, input: { protobuf: ${method.input}, json: ${method.input}JSON }, output: { protobuf: ${method.output}, json: ${method.output}JSON } },`;
+      result += `${method.name}: { name: '${method.name}', handler: service.${
+        method.name
+      }, input: { protobuf: ${
+        method.input
+      }, json: ${addJSONSuffixToFullyQualifiedName(
+        method.input!
+      )} }, output: { protobuf: ${
+        method.output
+      }, json: ${addJSONSuffixToFullyQualifiedName(method.output!)} } },`;
     });
     result += "}\n";
     result += `} ${printIfTypescript("as const")}\n`;