Fix: @path property should be included in unreachable models (#3218)

fix #3217

Author: timotheeguerin

.chronus/changes/fix-unrechable-models-path-2024-3-23-19-55-13.md

@@ -0,0 +1,8 @@
+---
+# Change versionKind to one of: internal, fix, dependencies, feature, deprecation, breaking
+changeKind: fix
+packages:
+  - "@typespec/openapi3"
+---
+
+Fix: `@path` property should be included in unreachable models

.chronus/changes/fix-unrechable-models-path-2024-3-23-19-55-14.md

@@ -0,0 +1,8 @@
+---
+# Change versionKind to one of: internal, fix, dependencies, feature, deprecation, breaking
+changeKind: fix
+packages:
+  - "@typespec/http"
+---
+
+Fix: `@path` property shouldn't be applicableMetadata if the visibility contain `Read`

packages/http/src/metadata.ts

@@ -356,7 +356,7 @@ function isApplicableMetadataCore(
     return false;
   }
 
-  if (visibility === Visibility.Read) {
+  if (visibility & Visibility.Read) {
     return isHeader(program, property) || isStatusCode(program, property);
   }
 
@@ -538,6 +538,7 @@ export function createMetadataInfo(program: Program, options?: MetadataInfoOptio
     if (isOptional(property, canonicalVisibility) !== isOptional(property, visibility)) {
       return true;
     }
+
     return (
       isPayloadProperty(property, visibility, undefined, /* keep shared */ true) !==
       isPayloadProperty(property, canonicalVisibility, undefined, /*keep shared*/ true)

packages/openapi3/test/metadata.test.ts

@@ -1170,4 +1170,49 @@ describe("openapi3: metadata", () => {
       "WidgetCreate",
     ]);
   });
+
+  it("unreachable models include @path properties", async () => {
+    const res = await openApiFor(`
+      model Unreachable {
+        @path name: string;
+      }
+    `);
+
+    deepStrictEqual(res.components.schemas.Unreachable, {
+      type: "object",
+      properties: {
+        name: {
+          type: "string",
+        },
+      },
+      required: ["name"],
+    });
+  });
+
+  it("inheritance tree unreachable with @path doesn't get conflicts", async () => {
+    const res = await openApiFor(`
+      model Base {
+      }
+
+      model Child extends Base {
+        @path name: string;
+      }
+    `);
+
+    deepStrictEqual(Object.keys(res.components.schemas), ["Base", "Child"]);
+    deepStrictEqual(res.components.schemas.Child, {
+      type: "object",
+      allOf: [
+        {
+          $ref: "#/components/schemas/Base",
+        },
+      ],
+      properties: {
+        name: {
+          type: "string",
+        },
+      },
+      required: ["name"],
+    });
+  });
 });