fix(msw): make sure to early out when detecting loop in additionalproperties (#1362)

Co-authored-by: Alfred Jonsson <alfred.jonsson@decerno.se>

Author: AllieJonsson

packages/mock/src/faker/getters/object.ts

@@ -149,6 +149,14 @@ export const getMockObject = ({
     if (isBoolean(item.additionalProperties)) {
       return { value: `{}`, imports: [], name: item.name };
     }
+    if (
+      isReference(item.additionalProperties) &&
+      existingReferencedProperties.includes(
+        item.additionalProperties.$ref.split('/').pop()!,
+      )
+    ) {
+      return { value: `{}`, imports: [], name: item.name };
+    }
 
     const resolvedValue = resolveMockValue({
       schema: {

packages/mock/src/msw/mocks.ts

@@ -100,7 +100,7 @@ const getMockScalarJsTypes = (
             `min: ${mockOptionsWithoutFunc.arrayMin}, ` +
             `max: ${mockOptionsWithoutFunc.arrayMax}}` +
             `)}, () => faker.number.int())`
-        : 'faker.number.int().toString()';
+        : 'faker.number.int()';
     case 'string':
       return isArray
         ? `Array.from({length: faker.number.int({` +