[openapi-metadata] Support for CommonJS

[acs]

Description

Hi team!

We are trying to use openapi-metadata in a large TypeScript project which is based on CommonJS (we plan to migrate to ESM at some point but not yet). It seems that openapi-metadata does not support CommonJS (the TS compiler suggest to change 'moduleResolution' setting 'node16', 'nodenext', or 'bundler'.).

Do you plan to support CommonJS in openapi-metadata?

Than you for your effort and time in this great project

-- Alvaro

Proposal

Support CommonJS in openapi-metadata

Extra

• I’m willing to open a PR (see CONTRIBUTING.md)

[kerwanp]

Hi @acs,

You are right openapi-metadata does not support CJS, there is in fact a legacy tsup.config.ts file but is not used anymore. We are directly using tsc.

I've opened a pull-request #2115. It would be great if you could build the library yourself and tell me if everything works as intended!

[acs]

Thanks guys! I have seen that this is already included in 0.2.0. We have added a task for testing it in Feb. I will report the results here. We are in a MVP testing tsoc and openapi-metadata for a large TS project.

[acs]

@kerwanp trying to use openapi-metadata importing it with CommonJS we get the error:

Uncaught Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: 
	Package subpath './dist/decorators' is not defined by "exports" in 
		/Users/acs/devel/backend/node_modules/openapi-metadata/package.json

[kerwanp]

@acs could you show me your require ?
It seems you are trying to import decorators using ./dist/decorators where it should just be ./decorators.

// Not good
const { ApiOperation } = require('openapi-metadata/dist/decorators')

// Good
const { ApiOperation } = require('openapi-metadata/decorators')