Move ignore list to functionOptions

Sophia Marie Terry · Sophia Marie Terry · commit 1d6e1e48d413 · 2025-08-19T12:17:08.000+01:00
diff --git a/tools/spectral/ipa/rulesets/IPA-104.yaml b/tools/spectral/ipa/rulesets/IPA-104.yaml
@@ -115,3 +115,5 @@ rules:
       function: 'IPA104ValidOperationID'
       functionOptions:
         methodName: 'get'
+        ignoreList:
+          - 'Fts'
diff --git a/tools/spectral/ipa/rulesets/IPA-105.yaml b/tools/spectral/ipa/rulesets/IPA-105.yaml
@@ -95,3 +95,5 @@ rules:
       function: 'IPA105ValidOperationID'
       functionOptions:
         methodName: 'list'
+        ignoreList:
+          - 'Fts'
diff --git a/tools/spectral/ipa/rulesets/IPA-106.yaml b/tools/spectral/ipa/rulesets/IPA-106.yaml
@@ -129,3 +129,5 @@ rules:
       function: 'IPA106ValidOperationID'
       functionOptions:
         methodName: 'create'
+        ignoreList:
+          - 'Fts'
diff --git a/tools/spectral/ipa/rulesets/IPA-107.yaml b/tools/spectral/ipa/rulesets/IPA-107.yaml
@@ -130,3 +130,5 @@ rules:
       function: 'IPA107ValidOperationID'
       functionOptions:
         methodName: 'update'
+        ignoreList:
+          - 'Fts'
diff --git a/tools/spectral/ipa/rulesets/IPA-108.yaml b/tools/spectral/ipa/rulesets/IPA-108.yaml
@@ -71,6 +71,8 @@ rules:
       function: 'IPA108ValidOperationID'
       functionOptions:
         methodName: 'delete'
+        ignoreList:
+          - 'Fts'
 
 functions:
   - IPA108DeleteMethodResponseShouldNotHaveSchema
diff --git a/tools/spectral/ipa/rulesets/IPA-109.yaml b/tools/spectral/ipa/rulesets/IPA-109.yaml
@@ -75,3 +75,6 @@ rules:
     given: '$.paths[*][*]'
     then:
       function: 'IPA109ValidOperationID'
+      functionOptions:
+        ignoreList:
+          - 'Fts'
diff --git a/tools/spectral/ipa/rulesets/functions/IPA104ValidOperationID.js b/tools/spectral/ipa/rulesets/functions/IPA104ValidOperationID.js
@@ -6,7 +6,7 @@ import { validateOperationIdAndReturnErrors } from './utils/validations/validate
 
 const RULE_NAME = 'xgen-IPA-104-valid-operation-id';
 
-export default (input, { methodName }, { path, documentInventory }) => {
+export default (input, { methodName, ignoreList }, { path, documentInventory }) => {
   const resourcePath = path[1];
   const oas = documentInventory.resolved;
   const resourcePaths = getResourcePathItems(resourcePath, oas.paths);
@@ -23,7 +23,7 @@ export default (input, { methodName }, { path, documentInventory }) => {
     if (hasMethodVerbOverride(input, methodName)) {
       methodName = input[VERB_OVERRIDE_EXTENSION].verb;
     }
-    const errors = validateOperationIdAndReturnErrors(methodName, resourcePath, input, path);
+    const errors = validateOperationIdAndReturnErrors(methodName, resourcePath, input, path, ignoreList);
 
     return evaluateAndCollectAdoptionStatus(errors, RULE_NAME, input, path);
   } catch (e) {
diff --git a/tools/spectral/ipa/rulesets/functions/IPA105ValidOperationID.js b/tools/spectral/ipa/rulesets/functions/IPA105ValidOperationID.js
@@ -6,7 +6,7 @@ import { validateOperationIdAndReturnErrors } from './utils/validations/validate
 
 const RULE_NAME = 'xgen-IPA-105-valid-operation-id';
 
-export default (input, { methodName }, { path, documentInventory }) => {
+export default (input, { methodName, ignoreList }, { path, documentInventory }) => {
   const resourcePath = path[1];
   const oas = documentInventory.resolved;
   const resourcePaths = getResourcePathItems(resourcePath, oas.paths);
@@ -24,7 +24,7 @@ export default (input, { methodName }, { path, documentInventory }) => {
   }
 
   try {
-    const errors = validateOperationIdAndReturnErrors(methodName, resourcePath, input, path);
+    const errors = validateOperationIdAndReturnErrors(methodName, resourcePath, input, path, ignoreList);
     return evaluateAndCollectAdoptionStatus(errors, RULE_NAME, input, path);
   } catch (e) {
     return handleInternalError(RULE_NAME, path, e);
diff --git a/tools/spectral/ipa/rulesets/functions/IPA106ValidOperationID.js b/tools/spectral/ipa/rulesets/functions/IPA106ValidOperationID.js
@@ -5,7 +5,7 @@ import { validateOperationIdAndReturnErrors } from './utils/validations/validate
 
 const RULE_NAME = 'xgen-IPA-106-valid-operation-id';
 
-export default (input, { methodName }, { path }) => {
+export default (input, { methodName, ignoreList }, { path }) => {
   const resourcePath = path[1];
 
   if (hasCustomMethodOverride(input) || isCustomMethodIdentifier(resourcePath)) {
@@ -17,7 +17,7 @@ export default (input, { methodName }, { path }) => {
   }
 
   try {
-    const errors = validateOperationIdAndReturnErrors(methodName, resourcePath, input, path);
+    const errors = validateOperationIdAndReturnErrors(methodName, resourcePath, input, path, ignoreList);
     return evaluateAndCollectAdoptionStatus(errors, RULE_NAME, input, path);
   } catch (e) {
     return handleInternalError(RULE_NAME, path, e);
diff --git a/tools/spectral/ipa/rulesets/functions/IPA107ValidOperationID.js b/tools/spectral/ipa/rulesets/functions/IPA107ValidOperationID.js
@@ -5,7 +5,7 @@ import { validateOperationIdAndReturnErrors } from './utils/validations/validate
 
 const RULE_NAME = 'xgen-IPA-107-valid-operation-id';
 
-export default (input, { methodName }, { path }) => {
+export default (input, { methodName, ignoreList }, { path }) => {
   const resourcePath = path[1];
 
   if (isCustomMethodIdentifier(resourcePath) || hasCustomMethodOverride(input)) {
@@ -17,7 +17,7 @@ export default (input, { methodName }, { path }) => {
   }
 
   try {
-    const errors = validateOperationIdAndReturnErrors(methodName, resourcePath, input, path);
+    const errors = validateOperationIdAndReturnErrors(methodName, resourcePath, input, path, ignoreList);
     return evaluateAndCollectAdoptionStatus(errors, RULE_NAME, input, path);
   } catch (e) {
     return handleInternalError(RULE_NAME, path, e);
diff --git a/tools/spectral/ipa/rulesets/functions/IPA108ValidOperationID.js b/tools/spectral/ipa/rulesets/functions/IPA108ValidOperationID.js
@@ -5,7 +5,7 @@ import { validateOperationIdAndReturnErrors } from './utils/validations/validate
 
 const RULE_NAME = 'xgen-IPA-108-valid-operation-id';
 
-export default (input, { methodName }, { path }) => {
+export default (input, { methodName, ignoreList }, { path }) => {
   const resourcePath = path[1];
 
   if (isCustomMethodIdentifier(resourcePath) || hasCustomMethodOverride(input)) {
@@ -17,7 +17,7 @@ export default (input, { methodName }, { path }) => {
   }
 
   try {
-    const errors = validateOperationIdAndReturnErrors(methodName, resourcePath, input, path);
+    const errors = validateOperationIdAndReturnErrors(methodName, resourcePath, input, path, ignoreList);
     return evaluateAndCollectAdoptionStatus(errors, RULE_NAME, input, path);
   } catch (e) {
     return handleInternalError(RULE_NAME, path, e);
diff --git a/tools/spectral/ipa/rulesets/functions/IPA109ValidOperationID.js b/tools/spectral/ipa/rulesets/functions/IPA109ValidOperationID.js
@@ -5,7 +5,7 @@ import { validateOperationIdAndReturnErrors } from './utils/validations/validate
 
 const RULE_NAME = 'xgen-IPA-109-valid-operation-id';
 
-export default (input, _, { path }) => {
+export default (input, { ignoreList }, { path }) => {
   const resourcePath = path[1];
 
   if (!isCustomMethodIdentifier(resourcePath) && !hasCustomMethodOverride(input)) {
@@ -28,7 +28,7 @@ export default (input, _, { path }) => {
       return;
     }
 
-    const errors = validateOperationIdAndReturnErrors(methodName, endpointUrl, input, path);
+    const errors = validateOperationIdAndReturnErrors(methodName, endpointUrl, input, path, ignoreList);
     return evaluateAndCollectAdoptionStatus(errors, RULE_NAME, input, path);
   } catch (e) {
     return handleInternalError(RULE_NAME, path, e);
diff --git a/tools/spectral/ipa/rulesets/functions/utils/operationIdGeneration.js b/tools/spectral/ipa/rulesets/functions/utils/operationIdGeneration.js
@@ -4,16 +4,13 @@ import { isPathParam, removePrefix, isSingleResourceIdentifier } from './resourc
 const CAMEL_CASE = /[A-Z]?[a-z]+/g;
 const CAMEL_CASE_WITH_ABBREVIATIONS = /[A-Z]+(?![a-z])|[A-Z]*[a-z]+/g;
 
-// List of capitalized nouns that should not be singularized in any case.
-const IGNORE_LIST = ['Fts'];
-
 /**
  * Returns IPA Compliant Operation ID.
  *
  * @param method the standard method name (create, update, get etc.), custom method name, or empty string (only for legacy custom methods)
  * @param path the path for the endpoint
  */
-export function generateOperationID(method, path) {
+export function generateOperationID(method, path, ignoreList = []) {
   if (!path) {
     return method;
   }
@@ -43,7 +40,7 @@ export function generateOperationID(method, path) {
 
   let opID = verb;
   for (let i = 0; i < nouns.length - 1; i++) {
-    opID += singularize(nouns[i]);
+    opID += singularize(nouns[i], ignoreList);
   }
 
   // singularize final noun, dependent on resource identifier - leave custom nouns alone
@@ -52,7 +49,7 @@ export function generateOperationID(method, path) {
       !camelCaseCustomMethod) ||
     verb === 'create'
   ) {
-    nouns[nouns.length - 1] = singularize(nouns[nouns.length - 1]);
+    nouns[nouns.length - 1] = singularize(nouns[nouns.length - 1], ignoreList);
   }
 
   opID += nouns.pop();
@@ -96,8 +93,8 @@ function capitalize(val) {
   return String(val).charAt(0).toUpperCase() + String(val).slice(1);
 }
 
-function singularize(noun) {
-  if (!IGNORE_LIST.includes(noun)) {
+function singularize(noun, ignoreList = []) {
+  if (!ignoreList.includes(noun)) {
     return inflection.singularize(noun);
   }
   return noun;
diff --git a/tools/spectral/ipa/rulesets/functions/utils/validations/validateOperationIdAndReturnErrors.js b/tools/spectral/ipa/rulesets/functions/utils/validations/validateOperationIdAndReturnErrors.js
@@ -18,9 +18,9 @@ const TOO_LONG_OP_ID_ERROR_MESSAGE =
  * @param path the path to the operation object being evaluated, used for error reporting with Spectral.
  * @returns {[{path: string[], message: string}]} an array of error objects, each containing a path and a message, or an empty array if no errors are found.
  */
-export function validateOperationIdAndReturnErrors(methodName, resourcePath, operationObject, path) {
+export function validateOperationIdAndReturnErrors(methodName, resourcePath, operationObject, path, ignoreList) {
   const operationId = operationObject.operationId;
-  const expectedOperationId = generateOperationID(methodName, resourcePath);
+  const expectedOperationId = generateOperationID(methodName, resourcePath, ignoreList);
 
   const operationIdPath = path.concat(['operationId']);
 
