exportInlineEnums should contain reference to the functions they're used in

[ReptoxX]

Description

The issue is that multiple endpoints use similar and same named enums. Which results in enums being exported multiple times with the same name.

Sometimes it would have a different number suffixed.

A solution could be to prefix the enum with the operationId or similar.

Reproducible example or configuration

https://stackblitz.com/edit/hey-api-client-fetch-example

OpenAPI specification (optional)

No response

System information (optional)

No response

[bolt-new-by-stackblitz]

Start a new pull request in StackBlitz Codeflow.

• hey-api-client-fetch-example (Open reproduction)

[mrlubos]

@ReptoxX Can you list some examples?

[tervay]

@mrlubos I actually was about to file this exact bug, here is an example spec:

{
    "openapi": "3.1.0",
    "info": {
        "title": "Simple Event API",
        "version": "1.0.0"
    },
    "paths": {},
    "components": {
        "schemas": {
            "Event": {
                "type": "object",
                "properties": {
                    "event_type": {
                        "type": "integer",
                        "enum": [
                            0,
                            1
                        ],
                        "x-enumNames": [
                            "REGIONAL",
                            "DISTRICT"
                        ]
                    }
                },
                "required": [
                    "event_type"
                ]
            },
            "EventSimple": {
                "type": "object",
                "properties": {
                    "event_type": {
                        "type": "integer",
                        "enum": [
                            0,
                            1
                        ],
                        "x-enumNames": [
                            "REGIONAL",
                            "DISTRICT"
                        ]
                    }
                },
                "required": [
                    "event_type"
                ]
            }
        }
    }
}

config

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

export default defineConfig({
  input: "spec.json",
  output: "src/",
  plugins: [
    {
      name: "@hey-api/typescript",
      enums: "typescript",
      exportInlineEnums: true,
    },
  ],
});

generated

// This file is auto-generated by @hey-api/openapi-ts

export enum EventType {
    /**
     * REGIONAL
     */
    REGIONAL = 0,
    /**
     * DISTRICT
     */
    DISTRICT = 1
}

export type Event = {
    event_type: 0 | 1;
};

export enum EventType {
    /**
     * REGIONAL
     */
    REGIONAL = 0,
    /**
     * DISTRICT
     */
    DISTRICT = 1
}

export type EventSimple = {
    event_type: 0 | 1;
};

export type ClientOptions = {
    baseUrl: `${string}://spec.json` | (string & {});
};

IMO it would be better to name them EventEventType and EventSimpleEventType

[ReptoxX]

Thanks @tervay i just finished stripping my example to the bare minimum to reproduce. But that's exactly the same scenario.

I just figured out, that the option could help

{
  name: "@hey-api/typescript",
  enums: "typescript+namespace", // Using typescript+namespace here
  exportInlineEnums: true,
},