@redocly/openapi-core - npm Package Compare versions

Comparing version 1.25.0 to 1.25.1

CHANGELOG.md

 # @redocly/openapi-core
 
+## 1.25.1
+
+### Patch Changes
+
+- Added additional checks to `criteria-unique` Arazzo rule.
+
 ## 1.25.0
@@ -4,0 +10,0 @@

lib/config/config.js

@@ -42,3 +42,3 @@ "use strict";
             [oas_types_1.SpecVersion.Arazzo]: {
-                ...rawConfig.arazzoRules,
+                ...(rawConfig.arazzoRules || {}),
                 ...(rawConfig.rules?.assertions ? { assertions: rawConfig.rules.assertions } : {}),
@@ -45,0 +45,0 @@ },

lib/rules/arazzo/criteria-unique.js

@@ -8,3 +8,6 @@ "use strict";
             enter(action, { report, location }) {
-                const criterias = action.criteria;
+                const criterias = action?.criteria;
+                if (!Array.isArray(criterias)) {
+                    return;
+                }
                 const seen = new Set();
@@ -27,3 +30,6 @@ for (const criteria of criterias) {
             enter(action, { report, location }) {
-                const criterias = action.criteria;
+                const criterias = action?.criteria;
+                if (!Array.isArray(criterias)) {
+                    return;
+                }
                 const seen = new Set();
@@ -46,6 +52,6 @@ for (const criteria of criterias) {
             enter(step, { report, location }) {
-                if (!step.successCriteria) {
+                const successCriterias = step?.successCriteria;
+                if (!Array.isArray(successCriterias)) {
                     return;
                 }
-                const successCriterias = step.successCriteria;
                 const seen = new Set();
@@ -52,0 +58,0 @@ for (const criteria of successCriterias) {

package.json

 {
   "name": "@redocly/openapi-core",
-  "version": "1.25.0",
+  "version": "1.25.1",
   "description": "",
@@ -5,0 +5,0 @@ "main": "lib/index.js",

src/config/config.ts

@@ -77,3 +77,3 @@ import * as fs from 'fs';
       [SpecVersion.Arazzo]: {
-        ...rawConfig.arazzoRules,
+        ...(rawConfig.arazzoRules || {}),
         ...(rawConfig.rules?.assertions ? { assertions: rawConfig.rules.assertions } : {}),
@@ -80,0 +80,0 @@ },

src/rules/arazzo/criteria-unique.ts

@@ -8,3 +8,6 @@ import type { ArazzoRule } from '../../visitors';
       enter(action, { report, location }: UserContext) {
-        const criterias = action.criteria;
+        const criterias = action?.criteria;
+        if (!Array.isArray(criterias)) {
+          return;
+        }
         const seen = new Set<string>();
@@ -26,3 +29,6 @@ for (const criteria of criterias) {
       enter(action, { report, location }: UserContext) {
-        const criterias = action.criteria;
+        const criterias = action?.criteria;
+        if (!Array.isArray(criterias)) {
+          return;
+        }
         const seen = new Set<string>();
@@ -44,7 +50,8 @@ for (const criteria of criterias) {
       enter(step, { report, location }: UserContext) {
-        if (!step.successCriteria) {
+        const successCriterias = step?.successCriteria;
+
+        if (!Array.isArray(successCriterias)) {
           return;
         }
 
-        const successCriterias = step.successCriteria;
         const seen = new Set<string>();
@@ -51,0 +58,0 @@

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

3033473

increased by0.02%

Lines of code

54452

increased by0.02%

No dependency changes