Types aren't correctly exported for CJS

[Oupsla]

Description

Typescript with "moduleResolution": "node16" will generate an error when trying to import the module from a cjs file.

Could not find a declaration file for module 'openapi-typescript'. '/node_modules/openapi-typescript/dist/index.cjs' implicitly has an 'any' type.
  Try `npm i --save-dev @types/openapi-typescript` if it exists or add a new declaration (.d.ts) file containing `declare module 'openapi-typescript';`ts(7016)

Name	Version
openapi-typescript	7.4.4
Node.js	22.9.0

Reproduction

Check here : https://arethetypeswrong.github.io/?p=openapi-typescript%407.4.4

Checklist

• My OpenAPI schema passes the Redocly validator (npx @redocly/cli@latest lint)
• I’m willing to open a PR (see CONTRIBUTING.md)

[Oupsla]

I think the package.json of each module should be something like :

"exports": {
    "./package.json": "./package.json",
    ".": {
      "import": {
        "types": "./dist/index.d.ts",
        "default": "./dist/index.js"
      },
      "require": {
        "types": "./dist/index.d.ts",
        "default": "./dist/index.cjs"
      }
    }
  },

[wattry]

I think the package.json of each module should be something like :

"exports": {
    "./package.json": "./package.json",
    ".": {
      "import": {
        "types": "./dist/index.d.ts",
        "default": "./dist/index.js"
      },
      "require": {
        "types": "./dist/index.d.ts",
        "default": "./dist/index.cjs"
      }
    }
  },

This depends on what the maintainers are attempting to achieve though the TS config is incorrect. The post above is mostly correct though there's no need to define the same types for require vs export as that will be handled by TypeScript.

This "./*": "./*" is allowing the package user to access any file within the project in a import/require and is likely intentional

  "type": "module",
  "types": "./dist/index.d.ts",
  "module": "./dist/index.js",
  "main": "./dist/index.cjs",
  "exports": {
    ".": {
      "types": "./dist/index.d.ts",
      "import": "./dist/index.js",
      "require": "./dist/index.cjs"
    },
    "./*": "./*"
  }

I tested this locally and was able to import TS, ESM and CJS.

Note

[cwhitemaddox]

Just wanting to mention that I have the same issue with a "moduleResolution": "node16" tsconfig.json where I'm unable to import this package to use the Node JS API. Right now, since I'm using this package in a development-time script, my workaround is to use the openapi-typescript CLI (and calling it as a subprocess with execa). Ideally though I'd use the openapi-typescript Node API directly so I could more elegantly handle the TS output before writing it to a file, etc.