Skip to content

Commit 1631813

Browse files
authored
fix(resolver): fix side effects in OpenAPI 3.1.0 resolving (#3456)
Refs #3455
1 parent 0446d56 commit 1631813

File tree

3 files changed

+12
-11
lines changed

3 files changed

+12
-11
lines changed

package-lock.json

Lines changed: 5 additions & 5 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -110,11 +110,11 @@
110110
},
111111
"dependencies": {
112112
"@babel/runtime-corejs3": "^7.22.15",
113-
"@swagger-api/apidom-core": ">=0.98.0 <1.0.0",
114-
"@swagger-api/apidom-error": ">=0.98.0 <1.0.0",
115-
"@swagger-api/apidom-json-pointer": ">=0.98.0 <1.0.0",
116-
"@swagger-api/apidom-ns-openapi-3-1": ">=0.98.0 <1.0.0",
117-
"@swagger-api/apidom-reference": ">=0.98.0 <1.0.0",
113+
"@swagger-api/apidom-core": ">=0.99.0 <1.0.0",
114+
"@swagger-api/apidom-error": ">=0.99.0 <1.0.0",
115+
"@swagger-api/apidom-json-pointer": ">=0.99.0 <1.0.0",
116+
"@swagger-api/apidom-ns-openapi-3-1": ">=0.99.0 <1.0.0",
117+
"@swagger-api/apidom-reference": ">=0.99.0 <1.0.0",
118118
"cookie": "~0.6.0",
119119
"deepmerge": "~4.3.0",
120120
"fast-json-patch": "^3.0.0-1",

src/resolver/apidom/reference/dereference/strategies/openapi-3-1-swagger-client/index.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ import ModelPropertyMacroVisitor from './visitors/properties.js';
1010
import AllOfVisitor from './visitors/all-of.js';
1111

1212
const visitAsync = visit[Symbol.for('nodejs.util.promisify.custom')];
13+
const mergeAllVisitorsAsync = mergeAllVisitors[Symbol.for('nodejs.util.promisify.custom')];
1314

1415
const OpenApi3_1SwaggerClientDereferenceStrategy = OpenApi3_1DereferenceStrategy.compose({
1516
props: {
@@ -87,7 +88,7 @@ const OpenApi3_1SwaggerClientDereferenceStrategy = OpenApi3_1DereferenceStrategy
8788
}
8889

8990
// establish root visitor by visitor merging
90-
const rootVisitor = mergeAllVisitors(visitors, { nodeTypeGetter: getNodeType });
91+
const rootVisitor = mergeAllVisitorsAsync(visitors, { nodeTypeGetter: getNodeType });
9192

9293
const dereferencedElement = await visitAsync(refSet.rootRef.value, rootVisitor, {
9394
keyMap,

0 commit comments

Comments
 (0)