fix(mock): use indexed access types for inline enum properties in MSW mocks (orval-labs#3224)

When enumGenerationType is 'enum', MSW mock generator imported inline
enum properties using the raw property name (e.g., countryCode) which
did not exist in the model. The model generates PascalCase enum names
(e.g., UserCountryCode), causing TS2305 compilation errors.

The fix conditionally propagates the response's isRef flag to
getMockScalar only when enumGenerationType is 'enum'. This ensures
the parent type name is added to existingReferencedProperties, so
getEnum() generates indexed access types (e.g., User['countryCode'])
instead of using the non-existent property name as a type.

The condition is scoped to enumGenerationType 'enum' to avoid
affecting circular reference detection in default (const) mode.

Fixes: orval-labs#3223

Author: wadakatu

packages/mock/src/msw/mocks.ts

@@ -204,7 +204,7 @@ export function getResponsesMockDefinition({
   };
 
   for (const response of responses) {
-    const { value: definition, example, examples, imports } = response;
+    const { value: definition, example, examples, imports, isRef } = response;
     let { originalSchema } = response;
 
     if (context.output.override.mock?.useExamples || mockOptions?.useExamples) {
@@ -246,6 +246,9 @@ export function getResponsesMockDefinition({
       item: {
         ...(resolvedSchema as Record<string, unknown>),
         name: definition,
+        ...(context.output.override.enumGenerationType === 'enum' && isRef
+          ? { isRef: true }
+          : {}),
       },
       imports,
       mockOptions: mockOptionsWithoutFunc,

tests/__snapshots__/mock/enums-inline-tags-split-native/model/index.ts

@@ -0,0 +1,14 @@
+/**
+ * Generated by orval v8.7.0 🍺
+ * Do not edit manually.
+ * Enums Inline
+ * Test spec with inline (dereferenced) enum properties
+ * OpenAPI spec version: 1.0.0
+ */
+
+export * from './template';
+export * from './templateCategory';
+export * from './templateStatus';
+export * from './user';
+export * from './userCountryCode';
+export * from './userLanguage';

tests/__snapshots__/mock/enums-inline-tags-split-native/model/template.ts

@@ -0,0 +1,15 @@
+/**
+ * Generated by orval v8.7.0 🍺
+ * Do not edit manually.
+ * Enums Inline
+ * Test spec with inline (dereferenced) enum properties
+ * OpenAPI spec version: 1.0.0
+ */
+import type { TemplateCategory } from './templateCategory';
+import type { TemplateStatus } from './templateStatus';
+
+export interface Template {
+  id: number;
+  status?: TemplateStatus;
+  category?: TemplateCategory;
+}

tests/__snapshots__/mock/enums-inline-tags-split-native/model/templateCategory.ts

@@ -0,0 +1,13 @@
+/**
+ * Generated by orval v8.7.0 🍺
+ * Do not edit manually.
+ * Enums Inline
+ * Test spec with inline (dereferenced) enum properties
+ * OpenAPI spec version: 1.0.0
+ */
+
+export enum TemplateCategory {
+  SOFTWARE = 'SOFTWARE',
+  CREATIVE = 'CREATIVE',
+  BUSINESS = 'BUSINESS',
+}

tests/__snapshots__/mock/enums-inline-tags-split-native/model/templateStatus.ts

@@ -0,0 +1,13 @@
+/**
+ * Generated by orval v8.7.0 🍺
+ * Do not edit manually.
+ * Enums Inline
+ * Test spec with inline (dereferenced) enum properties
+ * OpenAPI spec version: 1.0.0
+ */
+
+export enum TemplateStatus {
+  DRAFT = 'DRAFT',
+  PUBLISHED = 'PUBLISHED',
+  ARCHIVED = 'ARCHIVED',
+}

tests/__snapshots__/mock/enums-inline-tags-split-native/model/user.ts

@@ -0,0 +1,17 @@
+/**
+ * Generated by orval v8.7.0 🍺
+ * Do not edit manually.
+ * Enums Inline
+ * Test spec with inline (dereferenced) enum properties
+ * OpenAPI spec version: 1.0.0
+ */
+import type { UserCountryCode } from './userCountryCode';
+import type { UserLanguage } from './userLanguage';
+
+export interface User {
+  id: number;
+  name: string;
+  /** country code */
+  countryCode?: UserCountryCode;
+  language?: UserLanguage;
+}

tests/__snapshots__/mock/enums-inline-tags-split-native/model/userCountryCode.ts

@@ -0,0 +1,17 @@
+/**
+ * Generated by orval v8.7.0 🍺
+ * Do not edit manually.
+ * Enums Inline
+ * Test spec with inline (dereferenced) enum properties
+ * OpenAPI spec version: 1.0.0
+ */
+
+/**
+ * country code
+ */
+export enum UserCountryCode {
+  US = 'US',
+  JP = 'JP',
+  DE = 'DE',
+  FR = 'FR',
+}

tests/__snapshots__/mock/enums-inline-tags-split-native/model/userLanguage.ts

@@ -0,0 +1,12 @@
+/**
+ * Generated by orval v8.7.0 🍺
+ * Do not edit manually.
+ * Enums Inline
+ * Test spec with inline (dereferenced) enum properties
+ * OpenAPI spec version: 1.0.0
+ */
+
+export enum UserLanguage {
+  EN = 'EN',
+  JP = 'JP',
+}

tests/__snapshots__/mock/enums-inline-tags-split-native/template/template.msw.ts

@@ -0,0 +1,61 @@
+/**
+ * Generated by orval v8.7.0 🍺
+ * Do not edit manually.
+ * Enums Inline
+ * Test spec with inline (dereferenced) enum properties
+ * OpenAPI spec version: 1.0.0
+ */
+import { faker } from '@faker-js/faker';
+
+import { HttpResponse, http } from 'msw';
+import type { RequestHandlerOptions } from 'msw';
+
+import type { Template } from '../model';
+
+export const getGetTemplateResponseMock = (
+  overrideResponse: Partial<Extract<Template, object>> = {},
+): Template => ({
+  id: faker.number.int(),
+  status: faker.helpers.arrayElement([
+    faker.helpers.arrayElement([
+      'DRAFT',
+      'PUBLISHED',
+      'ARCHIVED',
+    ] as Template['status'][]),
+    undefined,
+  ]),
+  category: faker.helpers.arrayElement([
+    faker.helpers.arrayElement([
+      'SOFTWARE',
+      'CREATIVE',
+      'BUSINESS',
+    ] as Template['category'][]),
+    undefined,
+  ]),
+  ...overrideResponse,
+});
+
+export const getGetTemplateMockHandler = (
+  overrideResponse?:
+    | Template
+    | ((
+        info: Parameters<Parameters<typeof http.get>[1]>[0],
+      ) => Promise<Template> | Template),
+  options?: RequestHandlerOptions,
+) => {
+  return http.get(
+    '*/api/template',
+    async (info: Parameters<Parameters<typeof http.get>[1]>[0]) => {
+      return HttpResponse.json(
+        overrideResponse !== undefined
+          ? typeof overrideResponse === 'function'
+            ? await overrideResponse(info)
+            : overrideResponse
+          : getGetTemplateResponseMock(),
+        { status: 200 },
+      );
+    },
+    options,
+  );
+};
+export const getTemplateMock = () => [getGetTemplateMockHandler()];

tests/__snapshots__/mock/enums-inline-tags-split-native/template/template.ts

@@ -0,0 +1,21 @@
+/**
+ * Generated by orval v8.7.0 🍺
+ * Do not edit manually.
+ * Enums Inline
+ * Test spec with inline (dereferenced) enum properties
+ * OpenAPI spec version: 1.0.0
+ */
+import axios from 'axios';
+import type { AxiosRequestConfig, AxiosResponse } from 'axios';
+
+import type { Template } from '../model';
+
+/**
+ * @summary get template
+ */
+export const getTemplate = (
+  options?: AxiosRequestConfig,
+): Promise<AxiosResponse<Template>> => {
+  return axios.get(`/api/template`, options);
+};
+export type GetTemplateResult = AxiosResponse<Template>;