Fix x-ms-identifiers rule to support inner properties

.chronus/changes/fix-x-ms-identifiers-rule-2024-9-25-10-44-57.md

@@ -0,0 +1,7 @@
+---
+changeKind: fix
+packages:
+  - "@azure-tools/typespec-azure-resource-manager"
+---
+
+Fix the `x-ms-identifier` rule. The `x-ms-identifier` supports indexing into inner properties, but the linter does not support that and reports a warning.

packages/typespec-azure-resource-manager/src/rules/missing-x-ms-identifiers.ts

@@ -58,19 +58,29 @@ export const missingXmsIdentifiersRule = createRule({
       }
 
       if (Array.isArray(xmsIdentifiers)) {
-        for (const prop of xmsIdentifiers) {
-          if (typeof prop === "string") {
-            if (getProperty(elementType, prop) === undefined) {
-              context.reportDiagnostic({
-                messageId: "missingProperty",
-                format: { propertyName: prop, targetModelName: elementType.name },
-                target: property,
-              });
+        for (const propIdentifier of xmsIdentifiers) {
+          if (typeof propIdentifier === "string") {
+            const props = propIdentifier.split("/");
+            let element = elementType;
+            for (const prop of props) {
+              const propertyValue = getProperty(element, prop);
+              if (propertyValue === undefined) {
+                context.reportDiagnostic({
+                  messageId: "missingProperty",
+                  format: { propertyName: prop, targetModelName: elementType.name },
+                  target: property,
+                });
+              }
+
+              const propertyType = propertyValue?.type as ArrayModelType;
+              if (propertyType !== undefined && propertyType.kind === "Model") {
+                element = propertyType;
+              }
             }
           } else {
             context.reportDiagnostic({
               messageId: "notArray",
-              format: { valueType: typeof prop },
+              format: { valueType: typeof propIdentifier },
               target: property,
             });
           }

packages/typespec-azure-resource-manager/test/rules/missing-x-ms-identifiers.test.ts

@@ -130,4 +130,29 @@ describe("typespec-azure-core: no-enum rule", () => {
       )
       .toBeValid();
   });
+
+  it("allow array of x-ms-identifiers", async () => {
+    await tester
+      .expect(
+        `
+        model Pet {
+          @OpenAPI.extension("x-ms-identifiers", ["food/brand/name"])
+          pet: Dog[];
+        }
+
+        model Dog {
+          food: Food;
+        }
+
+        model Food {
+          brand: Brand;
+        }
+
+        model Brand {
+          name: string;
+        }
+        `,
+      )
+      .toBeValid();
+  });
 });