address the comments

Author: yelizhenden-mdb

tools/spectral/ipa/__tests__/IPA126TagNamesShouldUseTitleCase.test.js

@@ -128,8 +128,8 @@ testRule('xgen-IPA-126-tag-names-should-use-title-case', [
       tags: [
         { name: 'Api V1' },
         { name: 'Version 2 Resources' },
-        { name: 'Push-Based Log Export' },
-        { name: 'AWS Clusters DNS' },
+        { name: 'Push-Based Log Export' }, //valid
+        { name: 'AWS Clusters DNS' }, // valid
         { name: 'Encryption at Rest using Customer Key Management' },
       ],
     },
@@ -146,18 +146,6 @@ testRule('xgen-IPA-126-tag-names-should-use-title-case', [
         path: ['tags', '1'],
         severity: DiagnosticSeverity.Warning,
       },
-      {
-        code: 'xgen-IPA-126-tag-names-should-use-title-case',
-        message: 'Tag name should use Title Case, found: "Push-Based Log Export".',
-        path: ['tags', '2'],
-        severity: DiagnosticSeverity.Warning,
-      },
-      {
-        code: 'xgen-IPA-126-tag-names-should-use-title-case',
-        message: 'Tag name should use Title Case, found: "AWS Clusters DNS".',
-        path: ['tags', '3'],
-        severity: DiagnosticSeverity.Warning,
-      },
       {
         code: 'xgen-IPA-126-tag-names-should-use-title-case',
         message: 'Tag name should use Title Case, found: "Encryption at Rest using Customer Key Management".',

tools/spectral/ipa/rulesets/IPA-126.yaml

@@ -17,3 +17,12 @@ rules:
     given: $.tags[?(@.name && @.name.length > 0)]
     then:
       function: 'IPA126TagNamesShouldUseTitleCase'
+      functionOptions:
+        ignoreList:
+          - 'AWS'
+          - 'DNS'
+          - 'API'
+          - 'IP'
+          - 'MongoDB'
+          - 'LDAP'
+          - 'GCP'

tools/spectral/ipa/rulesets/functions/IPA126TagNamesShouldUseTitleCase.js

@@ -1,18 +1,17 @@
 import { hasException } from './utils/exceptions.js';
 import { collectAdoption, collectAndReturnViolation, collectException } from './utils/collectionUtils.js';
-import { casing } from '@stoplight/spectral-functions';
 
 const RULE_NAME = 'xgen-IPA-126-tag-names-should-use-title-case';
 
-export default (input, options, { path }) => {
+export default (input, { ignoreList }, { path }) => {
   const tagName = input.name;
   if (hasException(input, RULE_NAME)) {
     collectException(input, RULE_NAME, path);
     return;
   }
 
   // Check if the tag name uses Title Case
-  if (casing(tagName, { type: 'pascal', disallowDigits: true, separator: { char: ' ' } })) {
+  if (!isTitleCase(tagName, ignoreList)) {
     return collectAndReturnViolation(path, RULE_NAME, [
       {
         path,
@@ -24,3 +23,26 @@ export default (input, options, { path }) => {
   // Tag name uses Title Case
   collectAdoption(path, RULE_NAME);
 };
+
+function isTitleCase(str, ignoreList) {
+  // Split by spaces to check each word/word-group
+  const words = str.split(' ');
+
+  return words.every((wordGroup) => {
+    // For hyphenated words, check each part
+    if (wordGroup.includes('-')) {
+      const hyphenatedParts = wordGroup.split('-');
+      return hyphenatedParts.every((part) => {
+        if (part === '') return true; // Skip empty parts
+        if (ignoreList.includes(part)) return true;
+        // First character should be uppercase, rest lowercase, all alphabetical
+        return /^[A-Z][a-z]*$/.test(part);
+      });
+    }
+
+    // For regular words
+    if (wordGroup === '') return true;
+    if (ignoreList.includes(wordGroup)) return true;
+    return /^[A-Z][a-z]*$/.test(wordGroup);
+  });
+}