fix: Relax the aggressive of the id vs identifier detection

rulesets/src/naming.ruleset.yml

@@ -112,7 +112,7 @@ rules:
     then:
       function: pattern
       functionOptions:
-        notMatch: "(^identifier([A-Z]|$)|.*Identifier([A-Z]|$))"
+        notMatch: "^identifier$"
 
   sps-mandate-abbreviations-organization:
     description: Use abbreviations instead of long form names, i.e. organization SHOULD BE org.

rulesets/test/naming/sps-mandate-abbreviations-identifier.test.js

@@ -59,7 +59,7 @@ describe("sps-mandate-abbreviations-identifier", () => {
     await spectral.validateFailure(spec, ruleName, "Warning", 1);
   });
 
-  test("identifier should warn when it is a word present in the field name", async () => {
+  test("identifier should not warn when it is a word present in the field name prefix", async () => {
     const spec = `
       openapi: 3.0.1
       paths: {}
@@ -70,9 +70,24 @@ describe("sps-mandate-abbreviations-identifier", () => {
             properties:
               identifierNumber:
                 type: number
-              NumberIdentifier
         `;
 
-    await spectral.validateFailure(spec, ruleName, "Warning", 1);
+        await spectral.validateSuccess(spec, ruleName);
+  });
+
+  test("identifier should not warn when it is a word present in the field name suffix", async () => {
+    const spec = `
+      openapi: 3.0.1
+      paths: {}
+      components:
+        schemas:
+          User:
+            type: object
+            properties:
+              externalIdentifier:
+                type: number
+        `;
+
+        await spectral.validateSuccess(spec, ruleName);
   });
 });

sps-api-standards.spectral.yml

@@ -283,7 +283,7 @@ rules:
     then:
       function: pattern
       functionOptions:
-        notMatch: "(^identifier([A-Z]|$)|.*Identifier([A-Z]|$))"
+        notMatch: "^identifier$"
   sps-mandate-abbreviations-organization:
     description: Use abbreviations instead of long form names, i.e. organization SHOULD BE org.
     severity: warn