feat: validate API requests and responses according to the OpenApi spec

[feloy]

What does this PR do?

Validate API requests and responses according to the OpenApi spec

Using the express-openapi-validator needs to patch the ono dependency with JS-DevTools/ono#20

We use the @rollup/replace plugin for this (another solution would be to use yarn patch to patch the node_modules directly, but the project is currently using yarn v1.22, which does not include this command, available only starting from v3)

What issues does this PR fix or reference?

Fixes #1578

How to test this PR?

[benoitf]

curious why we need that ? these one are usually just for the local source code new links

[feloy]

Ono (a dependency of express-openapi-validator) provides both versions, cjs and esm, and vite includes the esm one by default.

But the esm version includes this code, which fails because module.exports.default is not defined:

// CommonJS default export hack
if (typeof module === "object" && typeof module.exports === "object") {
    module.exports = Object.assign(module.exports.default, module.exports);
}

[benoitf]

shouldn't the extension be set as a module ?

type: "module" ?

I did that for the podman extension
https://github.com/containers/podman-desktop/blob/main/extensions/podman/package.json#L7

and we say we generate a cjs file
https://github.com/containers/podman-desktop/blob/main/extensions/podman/package.json#L13

[benoitf]

the rationale is that we have more and more libraries that are only esm
so instead of trying to grab the commonjs files, we should switch to be esm and allow to use esm easily as well

some libraries are now esm only
podman-desktop/podman-desktop@8e6debf

[feloy]

@benoitf I just tested with type: "module" (and removing the aliases), but the error is the same. The hack in the esm version of ono seems not compatible with vite

[benoitf]

I think that we can't use cjs as for long term it'll be a burden

we may patch the module using JS-DevTools/ono#20 or getting rid of the lines ?

or switch to another library

[feloy]

OK, I'll make changes to patch ono's esm version, so we can use it. Thanks @benoitf