fix(mock): treat leading [] segment transparently in override key

Initial implementation in the previous commit normalized `.[]` substrings
only, which broke previously-working override keys with a leading `[]`
(no preceding dot) — e.g. `properties: { '[].id': ... }` at the
operation level, as used in `samples/vue-query/vue-query-basic`.

Switch to segment-based stripping: split on `.`, drop `[]` segments,
rejoin. Now `[].id`, `foo.[].id`, `foo.[].bar.[].id`, etc. all normalize
equivalently to the corresponding `.[]`-free form.

Regression test added in `resolveMockOverride` unit suite covering the
leading-bracket case explicitly.

Sample snapshot updates surface the #2465 fix correctly applying
operation-level `id` overrides for array-of-object responses in
`angular-app`, `angular-query`, and `basic` (previously the override was
silently dropped, so the snapshots stored the default
`faker.number.int(...)`). The `basic` sample's `orval.config.ts` also
wraps its `id` override in a thunk so the generated value is stable
across regenerations (matches the pattern already used by `angular-app`,
`angular-query`, and `vue-query-basic`).

Author: wadakatu

packages/mock/src/faker/resolvers/value.test.ts

@@ -51,6 +51,22 @@ describe('resolveMockOverride (#2465 — bare-key matching across array boundari
     expect(result?.value).toBe('() => faker.person.firstName()');
   });
 
+  it('matches a key with a leading `[].` segment (operation-level array-items syntax)', () => {
+    // The vue-query-basic sample uses `properties: { '[].id': ... }` at the
+    // operation level to target the `id` of items returned by an
+    // array-of-objects endpoint. Path arriving here is `#.[].id`.
+    const item: Item = { name: 'id', path: '#.[].id' };
+
+    const result = resolveMockOverride(
+      { '[].id': '() => faker.number.int({ min: 1, max: 99999 })' },
+      item,
+    );
+
+    expect(result?.value).toBe(
+      '() => faker.number.int({ min: 1, max: 99999 })',
+    );
+  });
+
   it('still does not match bare keys against non-array nested paths (semantic preserved)', () => {
     // The non-array nested case (`#.user.firstName` produced by an object
     // property `user` containing baseUser) was never matched by a bare key

packages/mock/src/faker/resolvers/value.ts

@@ -19,16 +19,26 @@ function isRegex(key: string) {
   return key.startsWith('/') && key.endsWith('/');
 }
 
+// Drop `[]` array-items segments from a dotted JSON-pointer-ish path. Treating
+// the marker as transparent for property override matching lets a bare
+// property-name override apply wherever the property literally appears, even
+// inside arrays (#2465). Segment-based so both leading (`[].id`) and embedded
+// (`foo.[].id`) markers normalize equivalently.
+function stripArrayMarkerSegments(s: string): string {
+  return s
+    .split('.')
+    .filter((seg) => seg !== '[]')
+    .join('.');
+}
+
 export function resolveMockOverride(
   properties: Record<string, unknown> | undefined = {},
   item: OpenApiSchemaObject & { name: string; path?: string },
 ) {
   const path = item.path ?? `#.${item.name}`;
-  // Strip `.[]` array-items markers so a bare property-name override applies
-  // wherever the property literally appears, including inside arrays (#2465).
-  // Regex keys still match against the original (un-stripped) path so users
+  // Regex keys still match against the original (un-normalized) path so users
   // can opt into array-scoped targeting explicitly if ever needed.
-  const pathWithoutArrayMarkers = path.replaceAll('.[]', '');
+  const normalizedPath = stripArrayMarkerSegments(path);
   const property = Object.entries(properties).find(([key]) => {
     if (isRegex(key)) {
       const regex = new RegExp(key.slice(1, -1));
@@ -37,7 +47,7 @@ export function resolveMockOverride(
       }
     }
 
-    if (`#.${key.replaceAll('.[]', '')}` === pathWithoutArrayMarkers) {
+    if (`#.${stripArrayMarkerSegments(key)}` === normalizedPath) {
       return true;
     }
 

samples/angular-app/__snapshots__/api/endpoints-zod/pets/pets.msw.ts

@@ -60,7 +60,7 @@ export const getListPetsResponseMock = (): Pets =>
       { length: faker.number.int({ min: 1, max: 100 }) },
       (_, i) => i + 1,
     ).map(() => ({
-      id: faker.number.int({ min: 1 }),
+      id: (() => faker.number.int({ min: 1, max: 99999 }))(),
       name: (() => faker.person.lastName())(),
       tag: (() => faker.person.lastName())(),
       email: faker.helpers.arrayElement([faker.internet.email(), undefined]),
@@ -101,7 +101,7 @@ export const getListPetsResponseMock = (): Pets =>
       { length: faker.number.int({ min: 1, max: 100 }) },
       (_, i) => i + 1,
     ).map(() => ({
-      id: faker.number.int({ min: 1 }),
+      id: (() => faker.number.int({ min: 1, max: 99999 }))(),
       name: (() => faker.person.lastName())(),
       tag: (() => faker.person.lastName())(),
       email: faker.helpers.arrayElement([faker.internet.email(), undefined]),

samples/angular-app/__snapshots__/api/http-both/pets/pets.msw.ts

@@ -60,7 +60,7 @@ export const getListPetsResponseMock = (): Pets =>
       { length: faker.number.int({ min: 1, max: 100 }) },
       (_, i) => i + 1,
     ).map(() => ({
-      id: faker.number.int({ min: 1 }),
+      id: (() => faker.number.int({ min: 1, max: 99999 }))(),
       name: (() => faker.person.lastName())(),
       tag: (() => faker.person.lastName())(),
       email: faker.helpers.arrayElement([faker.internet.email(), undefined]),
@@ -101,7 +101,7 @@ export const getListPetsResponseMock = (): Pets =>
       { length: faker.number.int({ min: 1, max: 100 }) },
       (_, i) => i + 1,
     ).map(() => ({
-      id: faker.number.int({ min: 1 }),
+      id: (() => faker.number.int({ min: 1, max: 99999 }))(),
       name: (() => faker.person.lastName())(),
       tag: (() => faker.person.lastName())(),
       email: faker.helpers.arrayElement([faker.internet.email(), undefined]),

samples/angular-app/__snapshots__/api/http-client-custom-params/pets/pets.msw.ts

@@ -60,7 +60,7 @@ export const getListPetsResponseMock = (): Pets =>
       { length: faker.number.int({ min: 1, max: 100 }) },
       (_, i) => i + 1,
     ).map(() => ({
-      id: faker.number.int({ min: 1 }),
+      id: (() => faker.number.int({ min: 1, max: 99999 }))(),
       name: (() => faker.person.lastName())(),
       tag: (() => faker.person.lastName())(),
       email: faker.helpers.arrayElement([faker.internet.email(), undefined]),
@@ -101,7 +101,7 @@ export const getListPetsResponseMock = (): Pets =>
       { length: faker.number.int({ min: 1, max: 100 }) },
       (_, i) => i + 1,
     ).map(() => ({
-      id: faker.number.int({ min: 1 }),
+      id: (() => faker.number.int({ min: 1, max: 99999 }))(),
       name: (() => faker.person.lastName())(),
       tag: (() => faker.person.lastName())(),
       email: faker.helpers.arrayElement([faker.internet.email(), undefined]),

samples/angular-app/__snapshots__/api/http-client/pets/pets.msw.ts

@@ -60,7 +60,7 @@ export const getListPetsResponseMock = (): Pets =>
       { length: faker.number.int({ min: 1, max: 100 }) },
       (_, i) => i + 1,
     ).map(() => ({
-      id: faker.number.int({ min: 1 }),
+      id: (() => faker.number.int({ min: 1, max: 99999 }))(),
       name: (() => faker.person.lastName())(),
       tag: (() => faker.person.lastName())(),
       email: faker.helpers.arrayElement([faker.internet.email(), undefined]),
@@ -101,7 +101,7 @@ export const getListPetsResponseMock = (): Pets =>
       { length: faker.number.int({ min: 1, max: 100 }) },
       (_, i) => i + 1,
     ).map(() => ({
-      id: faker.number.int({ min: 1 }),
+      id: (() => faker.number.int({ min: 1, max: 99999 }))(),
       name: (() => faker.person.lastName())(),
       tag: (() => faker.person.lastName())(),
       email: faker.helpers.arrayElement([faker.internet.email(), undefined]),

samples/angular-app/__snapshots__/api/http-resource-zod/pets/pets.msw.ts

@@ -60,7 +60,7 @@ export const getListPetsResponseMock = (): Pets =>
       { length: faker.number.int({ min: 1, max: 100 }) },
       (_, i) => i + 1,
     ).map(() => ({
-      id: faker.number.int({ min: 1 }),
+      id: (() => faker.number.int({ min: 1, max: 99999 }))(),
       name: (() => faker.person.lastName())(),
       tag: (() => faker.person.lastName())(),
       email: faker.helpers.arrayElement([faker.internet.email(), undefined]),
@@ -101,7 +101,7 @@ export const getListPetsResponseMock = (): Pets =>
       { length: faker.number.int({ min: 1, max: 100 }) },
       (_, i) => i + 1,
     ).map(() => ({
-      id: faker.number.int({ min: 1 }),
+      id: (() => faker.number.int({ min: 1, max: 99999 }))(),
       name: (() => faker.person.lastName())(),
       tag: (() => faker.person.lastName())(),
       email: faker.helpers.arrayElement([faker.internet.email(), undefined]),

samples/angular-app/__snapshots__/api/http-resource/pets/pets.msw.ts

@@ -60,7 +60,7 @@ export const getListPetsResponseMock = (): Pets =>
       { length: faker.number.int({ min: 1, max: 100 }) },
       (_, i) => i + 1,
     ).map(() => ({
-      id: faker.number.int({ min: 1 }),
+      id: (() => faker.number.int({ min: 1, max: 99999 }))(),
       name: (() => faker.person.lastName())(),
       tag: (() => faker.person.lastName())(),
       email: faker.helpers.arrayElement([faker.internet.email(), undefined]),
@@ -101,7 +101,7 @@ export const getListPetsResponseMock = (): Pets =>
       { length: faker.number.int({ min: 1, max: 100 }) },
       (_, i) => i + 1,
     ).map(() => ({
-      id: faker.number.int({ min: 1 }),
+      id: (() => faker.number.int({ min: 1, max: 99999 }))(),
       name: (() => faker.person.lastName())(),
       tag: (() => faker.person.lastName())(),
       email: faker.helpers.arrayElement([faker.internet.email(), undefined]),

samples/angular-app/src/api/endpoints-zod/pets/pets.msw.ts

@@ -60,7 +60,7 @@ export const getListPetsResponseMock = (): Pets =>
       { length: faker.number.int({ min: 1, max: 100 }) },
       (_, i) => i + 1,
     ).map(() => ({
-      id: faker.number.int({ min: 1 }),
+      id: (() => faker.number.int({ min: 1, max: 99999 }))(),
       name: (() => faker.person.lastName())(),
       tag: (() => faker.person.lastName())(),
       email: faker.helpers.arrayElement([faker.internet.email(), undefined]),
@@ -101,7 +101,7 @@ export const getListPetsResponseMock = (): Pets =>
       { length: faker.number.int({ min: 1, max: 100 }) },
       (_, i) => i + 1,
     ).map(() => ({
-      id: faker.number.int({ min: 1 }),
+      id: (() => faker.number.int({ min: 1, max: 99999 }))(),
       name: (() => faker.person.lastName())(),
       tag: (() => faker.person.lastName())(),
       email: faker.helpers.arrayElement([faker.internet.email(), undefined]),

samples/angular-app/src/api/http-both/pets/pets.msw.ts

@@ -60,7 +60,7 @@ export const getListPetsResponseMock = (): Pets =>
       { length: faker.number.int({ min: 1, max: 100 }) },
       (_, i) => i + 1,
     ).map(() => ({
-      id: faker.number.int({ min: 1 }),
+      id: (() => faker.number.int({ min: 1, max: 99999 }))(),
       name: (() => faker.person.lastName())(),
       tag: (() => faker.person.lastName())(),
       email: faker.helpers.arrayElement([faker.internet.email(), undefined]),
@@ -101,7 +101,7 @@ export const getListPetsResponseMock = (): Pets =>
       { length: faker.number.int({ min: 1, max: 100 }) },
       (_, i) => i + 1,
     ).map(() => ({
-      id: faker.number.int({ min: 1 }),
+      id: (() => faker.number.int({ min: 1, max: 99999 }))(),
       name: (() => faker.person.lastName())(),
       tag: (() => faker.person.lastName())(),
       email: faker.helpers.arrayElement([faker.internet.email(), undefined]),