Broken typescript generated, when using dots in method names about generator HOT 6 CLOSED

I encounter the same problem just in a hour, I use zenrpc to build server side rpc server. And I try to convert its smd schema into openrpc and generate client from the converted one. But the problem is zenrpc have the concept called namespace, which just add namespace in from of the method name. And cursed the openrpc generator generated invalid typescript like this.

/**
   * 
   */
  // tslint:disable-next-line:max-line-length
  public user.Login: UserLogin = (...params) => {
    return this.request("user.Login", params);
  }

from generator.

I figured out a workaround - I just wrote a post-generation script, that replaces dots with underscores in the final file. Here is the function, it's quite naive, but works:

import { readJson, readFile, writeFile } from "fs-extra";
import { OpenrpcDocument as OpenRPC, MethodObject } from "@open-rpc/meta-schema";

const fixMethodsWithDots = async (openrpcFilePath: string, generatedFilePath: string, finalOutputPath: string) => {
  const openrpcDocument: OpenRPC = await readJson(openrpcFilePath);
  let generatedFileContents: string = await readFile(generatedFilePath, "utf-8");
  const methodNames: string[] = openrpcDocument.methods.map(method => (<MethodObject>method).name);

  const methodNamesReplacements: [string, string][] = methodNames.map(methodName => (["public "+methodName, "public "+methodName.replace(".", "_")]));

  for (const [withDot, withUnderscore] of methodNamesReplacements) {
    generatedFileContents = generatedFileContents.replace(withDot, withUnderscore);
  }

  await writeFile(finalOutputPath, generatedFileContents);
}

I'll keep it for now, so it just works, but I'm really looking forward to this bugfix.

from generator.

I modify the master to generate code I want. https://github.com/Laysi/open-rpc-generator
It would generated nested class to move the method into its own namespace class.

  ...
  public user = new class{
    constructor(public superThis: Catcus) {
    }
    
    /**
     * 
     */
    // tslint:disable-next-line:max-line-length
    public Login: UserLogin = (...params) => {
      return this.superThis.request("user.Login", params);
    }
  }
  ...

And I have add some tweak to made it working under angular environment and ci. Compare to openapi generator it still lack some custom field for further usage in template.

from generator.

Thanks for the issue and nice catch.

Im happy to implement or take a PR for the initially suggested fix. The nesting fix is very interesting, and I don't want to completely dismiss it, but its quite a bit bigger of a change. Great work on creating a fork to implement it though - really nice!

comment if you want to make a PR, otherwise ill put one together soon.

from generator.

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

from generator.

Thanks for the issue and nice catch.

Im happy to implement or take a PR for the initially suggested fix. The nesting fix is very interesting, and I don't want to completely dismiss it, but its quite a bit bigger of a change. Great work on creating a fork to implement it though - really nice!

comment if you want to make a PR, otherwise ill put one together soon.

I'm glad to make a PR. :)

from generator.