identifierCase: "preserve" altering response type names

[henry-encord]

I'm migrating from 0.48.2 to 0.66.7. We need to use an unstrict version of PascalCase for types (e.g. MyHTTPType) and thus set identifierCase to "preserve" which preserves the names of these types. However, doing so causes request types to be renamed and have additional fields (additional fields not a problem), e.g. :

export type GetResourceData = {
  headers?: {
    "custom-header"?: string | null;
  };
  path: {
    param1: string;
  };
};

becomes:

export type GetResource_Data = {
  body?: never;
  headers?: {
    "custom-header"?: string | null;
  };
  path: {
    worparam1flow_template_uuid: string;
  };
  query?: never;
  url: "/v2/path/{param1}";
};

This is my config file:

import { defineConfig } from "@hey-api/openapi-ts";

export default defineConfig({
  input: "openapi.json",
  output: {
    format: "prettier",
    path: "lib",
  },
  plugins: [
    {
      name: "@hey-api/client-fetch",
      baseUrl: false,
      exportFromIndex: true,
    },
    {
      name: "@hey-api/sdk",
      asClass: true,
      throwOnError: false,
    },
    {
      name: "@hey-api/typescript",
      enums: "javascript",
      identifierCase: "preserve",
      readableNameBuilder: "Readable{{name}}",
    },
  ],
  schemas: false,
});

If I omit the identifierCase config option then the name is preserved on the response types. I am trying to avoid having to refactor a couple hundred imports with underscores. I have gone through the documentation and can't find anything relating to how to alter the generation of these types. Is there a way to preserve the names of these response types as well?

[mrlubos]

Hey @henry-encord so this is about GetResourceData becoming GetResource_Data? To confirm, what's the value from spec? Is it GetResource? If yes, I think this is a bug/feature request and I'd be open to fixing it

[henry-encord]

Hey @mrlubos, thanks for the reply yes it is GetResource. For a more detailed breakdown:

in our openapi.json file we have:

"schemas" : {
    "AnalyticsDetails": {
        "properties": {
            "items": {
                "items": {
                    "$ref": "#/components/schemas/AnalyticsDetails"
                },
                "title": "Items",
                "type": "array"
            },
            "nextPageToken": {
                "anyOf": [
                    {
                        "type": "string"
                    },
                    {
                        "type": "null"
                    }
                ],
                "title": "Nextpagetoken"
            }
        },
        "required": [
            "nextPageToken",
            "items"
        ],
        "title": "AnalyticsDetails",
        "type": "object"
    },
}

Which results in the types:

export type AnalyticsDetails_Data = {/*type parameters*/}
export type AnalyticsDetails_Errors = {/*type parameters*/}
export type AnalyticsDetails_Responses = {/*type parameters*/}

being generated instead of:

export type AnalyticsDetailsData = {/*type parameters*/}
export type AnalyticsDetailsErrors = {/*type parameters*/}
export type AnalyticsDetailsResponses = {/*type parameters*/}

[mrlubos]

Created #2001, this should be fixed. Thanks for reporting!

[mrlubos]

This will be fixed in the next release