fix(ipa): return internal errors as Spectral errors (#912)

yelizhenden-mdb · web-flow · commit 5590ee6f56db · 2025-08-20T16:15:59.000+01:00
diff --git a/tools/spectral/ipa/rulesets/functions/IPA005ExceptionExtensionFormat.js b/tools/spectral/ipa/rulesets/functions/IPA005ExceptionExtensionFormat.js
@@ -29,6 +29,6 @@ function checkViolationsAndReturnErrors(input, path) {
     });
     return errors;
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA102CollectionIdentifierCamelCase.js b/tools/spectral/ipa/rulesets/functions/IPA102CollectionIdentifierCamelCase.js
@@ -106,7 +106,7 @@ function checkViolations(pathKey, path, ignoredValues = []) {
       }
     });
   } catch (e) {
-    handleInternalError(RULE_NAME, [...path, pathKey], e);
+    return handleInternalError(RULE_NAME, [...path, pathKey], e);
   }
 
   return violations;
diff --git a/tools/spectral/ipa/rulesets/functions/IPA102EachPathAlternatesBetweenResourceNameAndPathParam.js b/tools/spectral/ipa/rulesets/functions/IPA102EachPathAlternatesBetweenResourceNameAndPathParam.js
@@ -67,6 +67,6 @@ function checkViolationsAndReturnErrors(suffixWithLeadingSlash, path) {
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA104EachResourceHasGetMethod.js b/tools/spectral/ipa/rulesets/functions/IPA104EachResourceHasGetMethod.js
@@ -1,9 +1,9 @@
 import {
-  hasGetMethod,
-  isSingletonResource,
   getResourcePathItems,
+  hasGetMethod,
   isResourceCollectionIdentifier,
   isSingleResourceIdentifier,
+  isSingletonResource,
 } from './utils/resourceEvaluation.js';
 import { evaluateAndCollectAdoptionStatus, handleInternalError } from './utils/collectionUtils.js';
 
@@ -42,6 +42,6 @@ function checkViolationsAndReturnErrors(oasPaths, input, path) {
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA104GetMethodReturnsSingleResource.js b/tools/spectral/ipa/rulesets/functions/IPA104GetMethodReturnsSingleResource.js
@@ -52,6 +52,6 @@ function checkViolationsAndReturnErrors(contentPerMediaType, path, isSingleton)
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA105ListMethodResponseIsGetMethodResponse.js b/tools/spectral/ipa/rulesets/functions/IPA105ListMethodResponseIsGetMethodResponse.js
@@ -5,7 +5,7 @@ import {
 } from './utils/resourceEvaluation.js';
 import { resolveObject } from './utils/componentUtils.js';
 import { evaluateAndCollectAdoptionStatus, handleInternalError } from './utils/collectionUtils.js';
-import { getSchemaRef, getSchemaNameFromRef, getResponseOfGetMethodByMediaType } from './utils/methodUtils.js';
+import { getResponseOfGetMethodByMediaType, getSchemaNameFromRef, getSchemaRef } from './utils/methodUtils.js';
 import { schemaIsPaginated } from './utils/schemaUtils.js';
 
 const RULE_NAME = 'xgen-IPA-105-list-method-response-is-get-method-response';
@@ -93,6 +93,6 @@ function checkViolationsAndReturnErrors(path, listMethodResultItems, getMethodRe
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA106CreateMethodResponseIsGetMethodResponse.js b/tools/spectral/ipa/rulesets/functions/IPA106CreateMethodResponseIsGetMethodResponse.js
@@ -5,7 +5,7 @@ import {
 } from './utils/resourceEvaluation.js';
 import { resolveObject } from './utils/componentUtils.js';
 import { evaluateAndCollectAdoptionStatus, handleInternalError } from './utils/collectionUtils.js';
-import { getSchemaRef, getResponseOfGetMethodByMediaType } from './utils/methodUtils.js';
+import { getResponseOfGetMethodByMediaType, getSchemaRef } from './utils/methodUtils.js';
 
 const RULE_NAME = 'xgen-IPA-106-create-method-response-is-get-method-response';
 const ERROR_MESSAGE =
@@ -72,6 +72,6 @@ function checkViolationsAndReturnErrors(path, createMethodResponseContent, getMe
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA106CreateMethodShouldNotHaveQueryParameters.js b/tools/spectral/ipa/rulesets/functions/IPA106CreateMethodShouldNotHaveQueryParameters.js
@@ -50,6 +50,6 @@ function checkViolationsAndReturnErrors(postMethodParameters, path, opts) {
     }
     return errors;
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA107UpdateMethodMustNotHaveQueryParams.js b/tools/spectral/ipa/rulesets/functions/IPA107UpdateMethodMustNotHaveQueryParams.js
@@ -51,6 +51,6 @@ function checkViolationsAndReturnErrors(postMethodParameters, path, ruleName, op
     }
     return errors;
   } catch (e) {
-    handleInternalError(ruleName, path, e);
+    return handleInternalError(ruleName, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA107UpdateMethodResponseIsGetMethodResponse.js b/tools/spectral/ipa/rulesets/functions/IPA107UpdateMethodResponseIsGetMethodResponse.js
@@ -6,7 +6,7 @@ import {
 } from './utils/resourceEvaluation.js';
 import { resolveObject } from './utils/componentUtils.js';
 import { evaluateAndCollectAdoptionStatus, handleInternalError } from './utils/collectionUtils.js';
-import { getSchemaRef, getGETMethodResponseSchemaFromPathItem } from './utils/methodUtils.js';
+import { getGETMethodResponseSchemaFromPathItem, getSchemaRef } from './utils/methodUtils.js';
 
 const RULE_NAME = 'xgen-IPA-107-update-method-response-is-get-method-response';
 const ERROR_MESSAGE =
@@ -84,6 +84,6 @@ function checkViolationsAndReturnErrors(path, updateMethodResponseContent, getMe
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA108DeleteMethod204Response.js b/tools/spectral/ipa/rulesets/functions/IPA108DeleteMethod204Response.js
@@ -37,6 +37,6 @@ function checkViolationsAndReturnErrors(input, path) {
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA108DeleteMethodResponseShouldNotHaveSchema.js b/tools/spectral/ipa/rulesets/functions/IPA108DeleteMethodResponseShouldNotHaveSchema.js
@@ -41,7 +41,7 @@ function checkViolationsAndReturnErrors(input, jsonPathArray) {
       }
     }
   } catch (e) {
-    handleInternalError(RULE_NAME, jsonPathArray, e);
+    return handleInternalError(RULE_NAME, jsonPathArray, e);
   }
   return errors;
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA109CustomMethodIdentifierFormat.js b/tools/spectral/ipa/rulesets/functions/IPA109CustomMethodIdentifierFormat.js
@@ -62,6 +62,6 @@ function checkViolationsAndReturnErrors(pathKey, path) {
 
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA109EachCustomMethodMustBeGetOrPost.js b/tools/spectral/ipa/rulesets/functions/IPA109EachCustomMethodMustBeGetOrPost.js
@@ -37,6 +37,6 @@ function checkViolationsAndReturnErrors(input, path) {
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA109EachCustomMethodMustUseCamelCase.js b/tools/spectral/ipa/rulesets/functions/IPA109EachCustomMethodMustUseCamelCase.js
@@ -30,6 +30,6 @@ function checkViolationsAndReturnErrors(pathKey, path) {
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA110CollectionsResponseDefineLinksArray.js b/tools/spectral/ipa/rulesets/functions/IPA110CollectionsResponseDefineLinksArray.js
@@ -44,6 +44,6 @@ function checkViolationsAndReturnErrors(listResponseSchema, oas, path) {
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA110CollectionsResponseDefineResultsArray.js b/tools/spectral/ipa/rulesets/functions/IPA110CollectionsResponseDefineResultsArray.js
@@ -42,6 +42,6 @@ function checkViolationsAndReturnErrors(listResponseSchema, oas, path) {
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA110CollectionsUsePaginatedPrefix.js b/tools/spectral/ipa/rulesets/functions/IPA110CollectionsUsePaginatedPrefix.js
@@ -61,6 +61,6 @@ function checkViolationsAndReturnErrors(listMethodResponse, oas, path) {
 
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA112AvoidProjectFieldNames.js b/tools/spectral/ipa/rulesets/functions/IPA112AvoidProjectFieldNames.js
@@ -47,6 +47,6 @@ function checkViolationsAndReturnErrors(input, options, path) {
 
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA112BooleanFieldNamesAvoidIsPrefix.js b/tools/spectral/ipa/rulesets/functions/IPA112BooleanFieldNamesAvoidIsPrefix.js
@@ -27,6 +27,6 @@ function checkViolationsAndReturnErrors(input, path) {
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA112FieldNamesAreCamelCase.js b/tools/spectral/ipa/rulesets/functions/IPA112FieldNamesAreCamelCase.js
@@ -26,6 +26,6 @@ function checkViolationsAndReturnErrors(input, path) {
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA113SingletonHasNoDeleteMethod.js b/tools/spectral/ipa/rulesets/functions/IPA113SingletonHasNoDeleteMethod.js
@@ -1,8 +1,8 @@
 import {
   getResourcePathItems,
-  isSingletonResource,
-  isResourceCollectionIdentifier,
   hasDeleteMethod,
+  isResourceCollectionIdentifier,
+  isSingletonResource,
 } from './utils/resourceEvaluation.js';
 import { evaluateAndCollectAdoptionStatus, handleInternalError } from './utils/collectionUtils.js';
 
@@ -30,6 +30,6 @@ function checkViolationsAndReturnErrors(input, path) {
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA113SingletonHasNoId.js b/tools/spectral/ipa/rulesets/functions/IPA113SingletonHasNoId.js
@@ -1,8 +1,8 @@
 import {
   getResourcePathItems,
   hasGetMethod,
-  isSingletonResource,
   isResourceCollectionIdentifier,
+  isSingletonResource,
 } from './utils/resourceEvaluation.js';
 import { getAllSuccessfulResponseSchemas } from './utils/methodUtils.js';
 import { evaluateAndCollectAdoptionStatus, handleInternalError } from './utils/collectionUtils.js';
@@ -43,6 +43,6 @@ function checkViolationsAndReturnErrors(input, path) {
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA113SingletonHasUpdateMethod.js b/tools/spectral/ipa/rulesets/functions/IPA113SingletonHasUpdateMethod.js
@@ -1,9 +1,9 @@
 import {
   getResourcePathItems,
-  isSingletonResource,
-  isResourceCollectionIdentifier,
-  hasPutMethod,
   hasPatchMethod,
+  hasPutMethod,
+  isResourceCollectionIdentifier,
+  isSingletonResource,
 } from './utils/resourceEvaluation.js';
 import { evaluateAndCollectAdoptionStatus, handleInternalError } from './utils/collectionUtils.js';
 
@@ -31,6 +31,6 @@ function checkViolationsAndReturnErrors(input, path) {
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA114ApiErrorHasBadRequestDetail.js b/tools/spectral/ipa/rulesets/functions/IPA114ApiErrorHasBadRequestDetail.js
@@ -58,12 +58,6 @@ function checkViolationsAndReturnErrors(apiErrorSchema, documentInventory, path)
 
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
-    return [
-      {
-        path,
-        message: `Internal error during validation: ${e.message}`,
-      },
-    ];
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA114AuthenticatedEndpointsHaveAuthErrors.js b/tools/spectral/ipa/rulesets/functions/IPA114AuthenticatedEndpointsHaveAuthErrors.js
@@ -61,6 +61,6 @@ function checkViolationsAndReturnErrors(responses, path) {
 
     return errors;
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA114ErrorResponsesReferToApiError.js b/tools/spectral/ipa/rulesets/functions/IPA114ErrorResponsesReferToApiError.js
@@ -75,6 +75,6 @@ function checkViolationsAndReturnErrors(apiResponseObject, oas, path, errorCode)
     }
     return errors;
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA114ParameterizedPathsHave404NotFound.js b/tools/spectral/ipa/rulesets/functions/IPA114ParameterizedPathsHave404NotFound.js
@@ -52,6 +52,6 @@ function checkViolationsAndReturnErrors(responses, path) {
 
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA117DescriptionEndsWithPeriod.js b/tools/spectral/ipa/rulesets/functions/IPA117DescriptionEndsWithPeriod.js
@@ -22,6 +22,6 @@ function checkViolationsAndReturnErrors(description, path) {
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA117DescriptionMustNotUseHtml.js b/tools/spectral/ipa/rulesets/functions/IPA117DescriptionMustNotUseHtml.js
@@ -32,6 +32,6 @@ function checkViolationsAndReturnErrors(description, path) {
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA117DescriptionShouldNotUseLinks.js b/tools/spectral/ipa/rulesets/functions/IPA117DescriptionShouldNotUseLinks.js
@@ -23,6 +23,6 @@ function checkViolationsAndReturnErrors(description, path) {
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA117DescriptionShouldNotUseTables.js b/tools/spectral/ipa/rulesets/functions/IPA117DescriptionShouldNotUseTables.js
@@ -23,6 +23,6 @@ function checkViolationsAndReturnErrors(description, path) {
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA117DescriptionStartsWithUpperCase.js b/tools/spectral/ipa/rulesets/functions/IPA117DescriptionStartsWithUpperCase.js
@@ -21,6 +21,6 @@ function checkViolationsAndReturnErrors(description, path) {
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA117HasDescription.js b/tools/spectral/ipa/rulesets/functions/IPA117HasDescription.js
@@ -16,6 +16,6 @@ function checkViolationsAndReturnErrors(input, path) {
     }
     return [];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA117ObjectsMustBeWellDefined.js b/tools/spectral/ipa/rulesets/functions/IPA117ObjectsMustBeWellDefined.js
@@ -62,6 +62,6 @@ function checkViolationsAndReturnErrors(object, path) {
     }
     return [{ path, message: ERROR_MESSAGE }];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA117OperationSummaryFormat.js b/tools/spectral/ipa/rulesets/functions/IPA117OperationSummaryFormat.js
@@ -21,6 +21,6 @@ function checkViolationsAndReturnErrors(summary, ignoreList, grammaticalWords, p
     }
     return [];
   } catch (e) {
-    handleInternalError(ruleName, path, e);
+    return handleInternalError(ruleName, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA117ParameterHasExamplesOrSchema.js b/tools/spectral/ipa/rulesets/functions/IPA117ParameterHasExamplesOrSchema.js
@@ -15,6 +15,6 @@ function checkViolationsAndReturnErrors(object, path) {
     }
     return [{ path, message: ERROR_MESSAGE }];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA117PlaintextResponseMustHaveExample.js b/tools/spectral/ipa/rulesets/functions/IPA117PlaintextResponseMustHaveExample.js
@@ -45,6 +45,6 @@ function checkViolationsAndReturnErrors(response, path) {
     }
     return [{ path, message: ERROR_MESSAGE }];
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA118NoAdditionalPropertiesFalse.js b/tools/spectral/ipa/rulesets/functions/IPA118NoAdditionalPropertiesFalse.js
@@ -33,6 +33,6 @@ function checkViolationsAndReturnErrors(schemaObject, path) {
 
     return errors;
   } catch (e) {
-    handleInternalError(RULE_NAME, path, e);
+    return handleInternalError(RULE_NAME, path, e);
   }
 }
diff --git a/tools/spectral/ipa/rulesets/functions/IPA119NoDefaultForCloudProviders.js b/tools/spectral/ipa/rulesets/functions/IPA119NoDefaultForCloudProviders.js
diff --git a/tools/spectral/ipa/rulesets/functions/IPA123EachEnumValueMustBeUpperSnakeCase.js b/tools/spectral/ipa/rulesets/functions/IPA123EachEnumValueMustBeUpperSnakeCase.js
diff --git a/tools/spectral/ipa/rulesets/functions/IPA124ArrayMaxItems.js b/tools/spectral/ipa/rulesets/functions/IPA124ArrayMaxItems.js
diff --git a/tools/spectral/ipa/rulesets/functions/utils/validations/checkPaginationQueryParameterAndReturnErrors.js b/tools/spectral/ipa/rulesets/functions/utils/validations/checkPaginationQueryParameterAndReturnErrors.js
diff --git a/tools/spectral/ipa/rulesets/functions/utils/validations/checkResponseCodeAndReturnErrors.js b/tools/spectral/ipa/rulesets/functions/utils/validations/checkResponseCodeAndReturnErrors.js
diff --git a/tools/spectral/ipa/rulesets/functions/utils/validations/checkSchemaRefSuffixAndReturnErrors.js b/tools/spectral/ipa/rulesets/functions/utils/validations/checkSchemaRefSuffixAndReturnErrors.js

Original file line number	Diff line number	Diff line change
`@@ -29,6 +29,6 @@ function checkViolationsAndReturnErrors(input, path) {`
`29`	`29`	`});`
`30`	`30`	`return errors;`
`31`	`31`	`} catch (e) {`
`32`		`- handleInternalError(RULE_NAME, path, e);`
	`32`	`+ return handleInternalError(RULE_NAME, path, e);`
`33`	`33`	`}`
`34`	`34`	`}`
Original file line number	Diff line number	Diff line change
`@@ -106,7 +106,7 @@ function checkViolations(pathKey, path, ignoredValues = []) {`
`106`	`106`	`}`
`107`	`107`	`});`
`108`	`108`	`} catch (e) {`
`109`		`- handleInternalError(RULE_NAME, [...path, pathKey], e);`
	`109`	`+ return handleInternalError(RULE_NAME, [...path, pathKey], e);`
`110`	`110`	`}`
`111`	`111`
`112`	`112`	`return violations;`
Original file line number	Diff line number	Diff line change
`@@ -67,6 +67,6 @@ function checkViolationsAndReturnErrors(suffixWithLeadingSlash, path) {`
`67`	`67`	`}`
`68`	`68`	`return [];`
`69`	`69`	`} catch (e) {`
`70`		`- handleInternalError(RULE_NAME, path, e);`
	`70`	`+ return handleInternalError(RULE_NAME, path, e);`
`71`	`71`	`}`
`72`	`72`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,9 +1,9 @@`
`1`	`1`	`import {`
`2`		`- hasGetMethod,`
`3`		`- isSingletonResource,`
`4`	`2`	`getResourcePathItems,`
	`3`	`+ hasGetMethod,`
`5`	`4`	`isResourceCollectionIdentifier,`
`6`	`5`	`isSingleResourceIdentifier,`
	`6`	`+ isSingletonResource,`
`7`	`7`	`} from './utils/resourceEvaluation.js';`
`8`	`8`	`import { evaluateAndCollectAdoptionStatus, handleInternalError } from './utils/collectionUtils.js';`
`9`	`9`
`@@ -42,6 +42,6 @@ function checkViolationsAndReturnErrors(oasPaths, input, path) {`
`42`	`42`	`}`
`43`	`43`	`return [];`
`44`	`44`	`} catch (e) {`
`45`		`- handleInternalError(RULE_NAME, path, e);`
	`45`	`+ return handleInternalError(RULE_NAME, path, e);`
`46`	`46`	`}`
`47`	`47`	`}`
Original file line number	Diff line number	Diff line change
`@@ -52,6 +52,6 @@ function checkViolationsAndReturnErrors(contentPerMediaType, path, isSingleton)`
`52`	`52`	`}`
`53`	`53`	`return [];`
`54`	`54`	`} catch (e) {`
`55`		`- handleInternalError(RULE_NAME, path, e);`
	`55`	`+ return handleInternalError(RULE_NAME, path, e);`
`56`	`56`	`}`
`57`	`57`	`}`
Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@ import {`
`5`	`5`	`} from './utils/resourceEvaluation.js';`
`6`	`6`	`import { resolveObject } from './utils/componentUtils.js';`
`7`	`7`	`import { evaluateAndCollectAdoptionStatus, handleInternalError } from './utils/collectionUtils.js';`
`8`		`-import { getSchemaRef, getSchemaNameFromRef, getResponseOfGetMethodByMediaType } from './utils/methodUtils.js';`
	`8`	`+import { getResponseOfGetMethodByMediaType, getSchemaNameFromRef, getSchemaRef } from './utils/methodUtils.js';`
`9`	`9`	`import { schemaIsPaginated } from './utils/schemaUtils.js';`
`10`	`10`
`11`	`11`	`const RULE_NAME = 'xgen-IPA-105-list-method-response-is-get-method-response';`
`@@ -93,6 +93,6 @@ function checkViolationsAndReturnErrors(path, listMethodResultItems, getMethodRe`
`93`	`93`	`}`
`94`	`94`	`return [];`
`95`	`95`	`} catch (e) {`
`96`		`- handleInternalError(RULE_NAME, path, e);`
	`96`	`+ return handleInternalError(RULE_NAME, path, e);`
`97`	`97`	`}`
`98`	`98`	`}`
Original file line number	Diff line number	Diff line change
`@@ -50,6 +50,6 @@ function checkViolationsAndReturnErrors(postMethodParameters, path, opts) {`
`50`	`50`	`}`
`51`	`51`	`return errors;`
`52`	`52`	`} catch (e) {`
`53`		`- handleInternalError(RULE_NAME, path, e);`
	`53`	`+ return handleInternalError(RULE_NAME, path, e);`
`54`	`54`	`}`
`55`	`55`	`}`
Original file line number	Diff line number	Diff line change
`@@ -51,6 +51,6 @@ function checkViolationsAndReturnErrors(postMethodParameters, path, ruleName, op`
`51`	`51`	`}`
`52`	`52`	`return errors;`
`53`	`53`	`} catch (e) {`
`54`		`- handleInternalError(ruleName, path, e);`
	`54`	`+ return handleInternalError(ruleName, path, e);`
`55`	`55`	`}`
`56`	`56`	`}`
Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,7 @@ import {`
`6`	`6`	`} from './utils/resourceEvaluation.js';`
`7`	`7`	`import { resolveObject } from './utils/componentUtils.js';`
`8`	`8`	`import { evaluateAndCollectAdoptionStatus, handleInternalError } from './utils/collectionUtils.js';`
`9`		`-import { getSchemaRef, getGETMethodResponseSchemaFromPathItem } from './utils/methodUtils.js';`
	`9`	`+import { getGETMethodResponseSchemaFromPathItem, getSchemaRef } from './utils/methodUtils.js';`
`10`	`10`
`11`	`11`	`const RULE_NAME = 'xgen-IPA-107-update-method-response-is-get-method-response';`
`12`	`12`	`const ERROR_MESSAGE =`
`@@ -84,6 +84,6 @@ function checkViolationsAndReturnErrors(path, updateMethodResponseContent, getMe`
`84`	`84`	`}`
`85`	`85`	`return [];`
`86`	`86`	`} catch (e) {`
`87`		`- handleInternalError(RULE_NAME, path, e);`
	`87`	`+ return handleInternalError(RULE_NAME, path, e);`
`88`	`88`	`}`
`89`	`89`	`}`