fix: clone input schemas before transforming them

[pvdbosch]

Description

Transformation from OpenAPI to JSON draft4 schema modifies the input data, even though cloneSchema: true is used.
This leads to the bug described in #275 when the same schema is transformed multiple times during parsing.

This PR adds a deep clone step before the transformation, and a test case to validate the fix.

PR #279 is an alternative fix, which upgrades @openapi-contrib/openapi-schema-to-json-schema to resolve the issue, but there may be more risk there because it's a new major version of the dependency.

Related issue(s)

Resolves #275

[github-actions]

Welcome to AsyncAPI. Thanks a lot for creating your first pull request. Please check out our contributors guide useful for opening a pull request.
Keep in mind there are also other channels you can use to interact with AsyncAPI community. For more details check out this issue.

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity 😴

It will be closed in 120 days if no further activity occurs. To unstale this pull request, add a comment with detailed explanation.

There can be many reasons why some specific pull request has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model.

Let us figure out together how to push this pull request forward. Connect with us through one of many communication channels we established here.

Thank you for your patience ❤️

[pvdbosch]

I fixed a test that was failing after latest merge with master, but the test failing in the GitHub Actions build seems another one. Still a mystery to why it's failing because I can't reproduce it locally. I'm trying to investigate further.

[pvdbosch]

Tests are working now on my forked github build.

There were missing await statements in another test somehow interfering, and the node.js version also needed to be updated to a newer one that supports structuredClone. I set minimum version to Node.js 18, aligned with https://github.com/asyncapi/parser-js/blob/master/package.json

Could you review these latest changes?

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[pvdbosch]

@fmvilas , could you review this PR again? All tests should work now in the GitHub Actions build.

This PR is probably less risky than the alternative one in #279, because that one requires a major update of the openapi-schema-to-json-schema dependency with many changes. Other AsyncAPI components still seem to be using the older version.

This issue is quite a roadblock for Belgian Government organizations, because we have a lot of existing schemas in OpenAPI 3.0 format.