@schmock/openapi 1.1.1 → 1.1.2

package/dist/normalizer.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"normalizer.d.ts","sourceRoot":"","sources":["../src/normalizer.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAa/C;;;;;;;;;;GAUG;AACH,wBAAgB,eAAe,CAC7B,MAAM,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC/B,SAAS,EAAE,SAAS,GAAG,UAAU,GAChC,WAAW,CAEb"}
+{"version":3,"file":"normalizer.d.ts","sourceRoot":"","sources":["../src/normalizer.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAG/C;;;;;;;;;;GAUG;AACH,wBAAgB,eAAe,CAC7B,MAAM,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,EAC/B,SAAS,EAAE,SAAS,GAAG,UAAU,GAChC,WAAW,CAEb"}

package/dist/normalizer.js

@@ -1,13 +1,5 @@
 /// <reference path="../../../types/schmock.d.ts" />
-function isRecord(value) {
-    return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-function toJsonSchema(node) {
-    // Object.assign merges unknown-keyed properties into JSONSchema7.
-    // This is safe because the normalizer has already ensured the shape
-    // is valid JSON Schema 7 before calling this function.
-    return Object.assign({}, node);
-}
+import { isRecord, toJsonSchema } from "./utils.js";
 /**
  * Normalize an OpenAPI schema to pure JSON Schema 7 that json-schema-faker understands.
  *

package/dist/parser.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"parser.d.ts","sourceRoot":"","sources":["../src/parser.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAI/C,MAAM,WAAW,UAAU;IACzB,KAAK,EAAE,MAAM,CAAC;IACd,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,EAAE,MAAM,CAAC;IACjB,KAAK,EAAE,UAAU,EAAE,CAAC;CACrB;AAED,MAAM,WAAW,UAAU;IACzB,6CAA6C;IAC7C,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,OAAO,CAAC,UAAU,CAAC;IAC3B,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,UAAU,EAAE,eAAe,EAAE,CAAC;IAC9B,WAAW,CAAC,EAAE,WAAW,CAAC;IAC1B,SAAS,EAAE,GAAG,CAAC,MAAM,EAAE;QAAE,MAAM,CAAC,EAAE,WAAW,CAAC;QAAC,WAAW,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;IACtE,IAAI,EAAE,MAAM,EAAE,CAAC;CAChB;AAED,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,MAAM,CAAC;IACb,EAAE,EAAE,MAAM,GAAG,OAAO,GAAG,QAAQ,CAAC;IAChC,QAAQ,EAAE,OAAO,CAAC;IAClB,MAAM,CAAC,EAAE,WAAW,CAAC;CACtB;AAkDD;;;GAGG;AACH,wBAAsB,SAAS,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM,GAAG,OAAO,CAAC,UAAU,CAAC,CAuH5E"}
+{"version":3,"file":"parser.d.ts","sourceRoot":"","sources":["../src/parser.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAK/C,MAAM,WAAW,UAAU;IACzB,KAAK,EAAE,MAAM,CAAC;IACd,OAAO,EAAE,MAAM,CAAC;IAChB,QAAQ,EAAE,MAAM,CAAC;IACjB,KAAK,EAAE,UAAU,EAAE,CAAC;CACrB;AAED,MAAM,WAAW,UAAU;IACzB,6CAA6C;IAC7C,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,OAAO,CAAC,UAAU,CAAC;IAC3B,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,UAAU,EAAE,eAAe,EAAE,CAAC;IAC9B,WAAW,CAAC,EAAE,WAAW,CAAC;IAC1B,SAAS,EAAE,GAAG,CAAC,MAAM,EAAE;QAAE,MAAM,CAAC,EAAE,WAAW,CAAC;QAAC,WAAW,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;IACtE,IAAI,EAAE,MAAM,EAAE,CAAC;CAChB;AAED,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,MAAM,CAAC;IACb,EAAE,EAAE,MAAM,GAAG,OAAO,GAAG,QAAQ,CAAC;IAChC,QAAQ,EAAE,OAAO,CAAC;IAClB,MAAM,CAAC,EAAE,WAAW,CAAC;CACtB;AA8CD;;;GAGG;AACH,wBAAsB,SAAS,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM,GAAG,OAAO,CAAC,UAAU,CAAC,CAuH5E"}

package/dist/parser.js

@@ -2,6 +2,7 @@
 import SwaggerParser from "@apidevtools/swagger-parser";
 import { toHttpMethod } from "@schmock/core";
 import { normalizeSchema } from "./normalizer.js";
+import { isRecord } from "./utils.js";
 const HTTP_METHOD_KEYS = new Set([
     "get",
     "post",
@@ -11,9 +12,6 @@ const HTTP_METHOD_KEYS = new Set([
     "head",
     "options",
 ]);
-function isRecord(value) {
-    return typeof value === "object" && value !== null && !Array.isArray(value);
-}
 function isOpenApiDocument(value) {
     return isRecord(value) && ("swagger" in value || "openapi" in value);
 }

package/dist/plugin.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"plugin.d.ts","sourceRoot":"","sources":["../src/plugin.ts"],"names":[],"mappings":"AAaA,OAAO,KAAK,EAAE,UAAU,EAAE,UAAU,EAAE,MAAM,WAAW,CAAC;AAGxD,YAAY,EAAE,UAAU,EAAE,UAAU,EAAE,CAAC;AAMvC,MAAM,WAAW,cAAc;IAC7B,sCAAsC;IACtC,IAAI,EAAE,MAAM,GAAG,MAAM,CAAC;IACtB,sCAAsC;IACtC,IAAI,CAAC,EAAE,UAAU,CAAC;IAClB,8CAA8C;IAC9C,gBAAgB,CAAC,EAAE,OAAO,CAAC;IAC3B,gDAAgD;IAChD,iBAAiB,CAAC,EAAE,OAAO,CAAC;IAC5B,wCAAwC;IACxC,aAAa,CAAC,EAAE;QACd,UAAU,CAAC,EAAE,OAAO,CAAC;QACrB,OAAO,CAAC,EAAE,OAAO,CAAC;QAClB,SAAS,CAAC,EAAE,OAAO,CAAC;KACrB,CAAC;CACH;AAED;;;;;;;;;;;GAWG;AACH,wBAAsB,OAAO,CAC3B,OAAO,EAAE,cAAc,GACtB,OAAO,CAAC,OAAO,CAAC,MAAM,CAAC,CAwCzB"}
+{"version":3,"file":"plugin.d.ts","sourceRoot":"","sources":["../src/plugin.ts"],"names":[],"mappings":"AAaA,OAAO,KAAK,EAAE,UAAU,EAAE,UAAU,EAAE,MAAM,WAAW,CAAC;AAIxD,YAAY,EAAE,UAAU,EAAE,UAAU,EAAE,CAAC;AAEvC,MAAM,WAAW,cAAc;IAC7B,sCAAsC;IACtC,IAAI,EAAE,MAAM,GAAG,MAAM,CAAC;IACtB,sCAAsC;IACtC,IAAI,CAAC,EAAE,UAAU,CAAC;IAClB,8CAA8C;IAC9C,gBAAgB,CAAC,EAAE,OAAO,CAAC;IAC3B,gDAAgD;IAChD,iBAAiB,CAAC,EAAE,OAAO,CAAC;IAC5B,wCAAwC;IACxC,aAAa,CAAC,EAAE;QACd,UAAU,CAAC,EAAE,OAAO,CAAC;QACrB,OAAO,CAAC,EAAE,OAAO,CAAC;QAClB,SAAS,CAAC,EAAE,OAAO,CAAC;KACrB,CAAC;CACH;AAED;;;;;;;;;;;GAWG;AACH,wBAAsB,OAAO,CAC3B,OAAO,EAAE,cAAc,GACtB,OAAO,CAAC,OAAO,CAAC,MAAM,CAAC,CAwCzB"}

package/dist/plugin.js

@@ -3,9 +3,7 @@ import { detectCrudResources } from "./crud-detector.js";
 import { createCreateGenerator, createDeleteGenerator, createListGenerator, createReadGenerator, createStaticGenerator, createUpdateGenerator, } from "./generators.js";
 import { parseSpec } from "./parser.js";
 import { loadSeed } from "./seed.js";
-function isRecord(value) {
-    return typeof value === "object" && value !== null && !Array.isArray(value);
-}
+import { isRecord } from "./utils.js";
 /**
  * Create an OpenAPI plugin that auto-registers CRUD routes from a spec.
  *
@@ -26,7 +24,7 @@ export async function openapi(options) {
         : new Map();
     return {
         name: "@schmock/openapi",
-        version: "1.0.0",
+        version: "1.1.1",
         install(instance) {
             // Seed initial state — we need state to be initialized before registering routes
             // The state is populated via generator context.state on first request,

package/dist/seed.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"seed.d.ts","sourceRoot":"","sources":["../src/seed.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAGvD,MAAM,MAAM,UAAU,GAAG,OAAO,EAAE,GAAG,MAAM,GAAG;IAAE,KAAK,EAAE,MAAM,CAAA;CAAE,CAAC;AAEhE,MAAM,MAAM,UAAU,GAAG,MAAM,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC;AAEpD;;;;;;;GAOG;AACH,wBAAgB,QAAQ,CACtB,MAAM,EAAE,UAAU,EAClB,SAAS,EAAE,YAAY,EAAE,GACxB,GAAG,CAAC,MAAM,EAAE,OAAO,EAAE,CAAC,CAwCxB"}
+{"version":3,"file":"seed.d.ts","sourceRoot":"","sources":["../src/seed.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAGvD,MAAM,MAAM,UAAU,GAAG,OAAO,EAAE,GAAG,MAAM,GAAG;IAAE,KAAK,EAAE,MAAM,CAAA;CAAE,CAAC;AAEhE,MAAM,MAAM,UAAU,GAAG,MAAM,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC;AAEpD;;;;;;;GAOG;AACH,wBAAgB,QAAQ,CACtB,MAAM,EAAE,UAAU,EAClB,SAAS,EAAE,YAAY,EAAE,GACxB,GAAG,CAAC,MAAM,EAAE,OAAO,EAAE,CAAC,CAyDxB"}

package/dist/seed.js

@@ -20,7 +20,13 @@ export function loadSeed(config, resources) {
         else if (typeof source === "string") {
             // File path
             const content = readFileSync(source, "utf-8");
-            const parsed = JSON.parse(content);
+            let parsed;
+            try {
+                parsed = JSON.parse(content);
+            }
+            catch {
+                throw new Error(`Seed file "${source}" for resource "${resourceName}" contains invalid JSON`);
+            }
             if (!Array.isArray(parsed)) {
                 throw new Error(`Seed file "${source}" for resource "${resourceName}" must contain a JSON array`);
             }
@@ -30,10 +36,16 @@ export function loadSeed(config, resources) {
             source !== null &&
             "count" in source) {
             // Auto-generate from schema
+            const rawCount = source.count;
+            if (typeof rawCount !== "number" ||
+                !Number.isInteger(rawCount) ||
+                rawCount < 0) {
+                throw new Error(`Seed count for "${resourceName}" must be a non-negative integer, got: ${String(rawCount)}`);
+            }
             if (!resource?.schema) {
                 throw new Error(`Cannot auto-generate seed for "${resourceName}": no schema found in spec`);
             }
-            const items = generateSeedItems(resource.schema, source.count, resource.idParam);
+            const items = generateSeedItems(resource.schema, rawCount, resource.idParam);
             result.set(resourceName, items);
         }
     }

package/dist/utils.d.ts

@@ -0,0 +1,4 @@
+import type { JSONSchema7 } from "json-schema";
+export declare function isRecord(value: unknown): value is Record<string, unknown>;
+export declare function toJsonSchema(node: Record<string, unknown>): JSONSchema7;
+//# sourceMappingURL=utils.d.ts.map

package/dist/utils.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"utils.d.ts","sourceRoot":"","sources":["../src/utils.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,aAAa,CAAC;AAE/C,wBAAgB,QAAQ,CAAC,KAAK,EAAE,OAAO,GAAG,KAAK,IAAI,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAEzE;AAED,wBAAgB,YAAY,CAAC,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,WAAW,CAEvE"}

package/dist/utils.js

@@ -0,0 +1,6 @@
+export function isRecord(value) {
+    return typeof value === "object" && value !== null && !Array.isArray(value);
+}
+export function toJsonSchema(node) {
+    return Object.assign({}, node);
+}

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@schmock/openapi",
   "description": "OpenAPI/Swagger spec-driven auto-registration plugin for Schmock",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "type": "module",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",
@@ -38,8 +38,8 @@
   "devDependencies": {
     "@amiceli/vitest-cucumber": "^6.2.0",
     "@types/json-schema": "^7.0.15",
-    "@types/node": "^25.1.0",
+    "@types/node": "^25.2.1",
     "openapi-types": "^12.1.3",
-    "vitest": "^4.0.15"
+    "vitest": "^4.0.18"
   }
 }

package/src/crud-detector.ts

@@ -2,6 +2,7 @@
 
 import type { JSONSchema7 } from "json-schema";
 import type { ParsedPath } from "./parser.js";
+import { isRecord, toJsonSchema } from "./utils.js";
 
 export type CrudOperation = "list" | "create" | "read" | "update" | "delete";
 
@@ -110,8 +111,8 @@ function buildResource(
           const items = Array.isArray(listSchema.items)
             ? listSchema.items[0]
             : listSchema.items;
-          if (typeof items === "object" && items !== null) {
-            schema = schema ?? (items as JSONSchema7);
+          if (isRecord(items)) {
+            schema = schema ?? toJsonSchema(items);
           }
         }
       } else if (p.method === "POST") {

package/src/generators.ts

@@ -4,13 +4,10 @@ import { generateFromSchema } from "@schmock/schema";
 import type { JSONSchema7 } from "json-schema";
 import type { CrudResource } from "./crud-detector.js";
 import type { ParsedPath } from "./parser.js";
+import { isRecord } from "./utils.js";
 
 const COLLECTION_STATE_PREFIX = "openapi:collections:";
 
-function isRecord(value: unknown): value is Record<string, unknown> {
-  return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-
 function toTuple(status: number, body: unknown): [number, unknown] {
   return [status, body];
 }
@@ -159,9 +156,11 @@ export function createStaticGenerator(
     if (responseSchema) {
       try {
         return generateFromSchema({ schema: responseSchema });
-      } catch {
-        // Complex schemas (deep anyOf, circular refs) may fail generation.
-        // Fall back to an empty object rather than crashing.
+      } catch (error) {
+        console.warn(
+          `[@schmock/openapi] Schema generation failed for ${parsedPath.method} ${parsedPath.path}:`,
+          error instanceof Error ? error.message : error,
+        );
         return {};
       }
     }

package/src/normalizer.ts

@@ -1,17 +1,7 @@
 /// <reference path="../../../types/schmock.d.ts" />
 
 import type { JSONSchema7 } from "json-schema";
-
-function isRecord(value: unknown): value is Record<string, unknown> {
-  return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-
-function toJsonSchema(node: Record<string, unknown>): JSONSchema7 {
-  // Object.assign merges unknown-keyed properties into JSONSchema7.
-  // This is safe because the normalizer has already ensured the shape
-  // is valid JSON Schema 7 before calling this function.
-  return Object.assign<JSONSchema7, Record<string, unknown>>({}, node);
-}
+import { isRecord, toJsonSchema } from "./utils.js";
 
 /**
  * Normalize an OpenAPI schema to pure JSON Schema 7 that json-schema-faker understands.

package/src/parser.ts

@@ -5,6 +5,7 @@ import { toHttpMethod } from "@schmock/core";
 import type { JSONSchema7 } from "json-schema";
 import type { OpenAPI } from "openapi-types";
 import { normalizeSchema } from "./normalizer.js";
+import { isRecord } from "./utils.js";
 
 export interface ParsedSpec {
   title: string;
@@ -41,10 +42,6 @@ const HTTP_METHOD_KEYS = new Set([
   "options",
 ]);
 
-function isRecord(value: unknown): value is Record<string, unknown> {
-  return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-
 function isOpenApiDocument(value: unknown): value is OpenAPI.Document {
   return isRecord(value) && ("swagger" in value || "openapi" in value);
 }

package/src/plugin.ts

@@ -13,13 +13,10 @@ import {
 import { parseSpec } from "./parser.js";
 import type { SeedConfig, SeedSource } from "./seed.js";
 import { loadSeed } from "./seed.js";
+import { isRecord } from "./utils.js";
 
 export type { SeedConfig, SeedSource };
 
-function isRecord(value: unknown): value is Record<string, unknown> {
-  return typeof value === "object" && value !== null && !Array.isArray(value);
-}
-
 export interface OpenApiOptions {
   /** File path or inline spec object */
   spec: string | object;
@@ -60,7 +57,7 @@ export async function openapi(
 
   return {
     name: "@schmock/openapi",
-    version: "1.0.0",
+    version: "1.1.1",
 
     install(instance: Schmock.CallableMockInstance) {
       // Seed initial state — we need state to be initialized before registering routes

package/src/seed.ts

@@ -31,7 +31,14 @@ export function loadSeed(
     } else if (typeof source === "string") {
       // File path
       const content = readFileSync(source, "utf-8");
-      const parsed: unknown = JSON.parse(content);
+      let parsed: unknown;
+      try {
+        parsed = JSON.parse(content);
+      } catch {
+        throw new Error(
+          `Seed file "${source}" for resource "${resourceName}" contains invalid JSON`,
+        );
+      }
       if (!Array.isArray(parsed)) {
         throw new Error(
           `Seed file "${source}" for resource "${resourceName}" must contain a JSON array`,
@@ -44,6 +51,16 @@ export function loadSeed(
       "count" in source
     ) {
       // Auto-generate from schema
+      const rawCount = source.count;
+      if (
+        typeof rawCount !== "number" ||
+        !Number.isInteger(rawCount) ||
+        rawCount < 0
+      ) {
+        throw new Error(
+          `Seed count for "${resourceName}" must be a non-negative integer, got: ${String(rawCount)}`,
+        );
+      }
       if (!resource?.schema) {
         throw new Error(
           `Cannot auto-generate seed for "${resourceName}": no schema found in spec`,
@@ -51,7 +68,7 @@ export function loadSeed(
       }
       const items = generateSeedItems(
         resource.schema,
-        source.count,
+        rawCount,
         resource.idParam,
       );
       result.set(resourceName, items);

package/src/steps/openapi-seed.steps.ts

@@ -1,4 +1,5 @@
 import { resolve } from "node:path";
+import { writeFileSync } from "node:fs";
 import { describeFeature, loadFeature } from "@amiceli/vitest-cucumber";
 import { schmock } from "@schmock/core";
 import { expect } from "vitest";
@@ -7,9 +8,12 @@ import { openapi } from "../plugin";
 const feature = await loadFeature("../../features/openapi-seed.feature");
 const fixturesDir = resolve(import.meta.dirname, "../__fixtures__");
 
+const scratchDir = resolve(import.meta.dirname, "../__fixtures__");
+
 describeFeature(feature, ({ Scenario }) => {
   let mock: Schmock.CallableMockInstance;
   let response: Schmock.Response;
+  let specPath: string;
 
   Scenario("Seed with inline objects", ({ Given, When, Then }) => {
     Given("a mock with Petstore spec and inline seed data", async () => {
@@ -91,4 +95,53 @@ describeFeature(feature, ({ Scenario }) => {
       });
     },
   );
+
+  Scenario(
+    "Invalid seed count produces descriptive error",
+    ({ Given, Then }) => {
+      Given("a Petstore spec path", () => {
+        specPath = `${fixturesDir}/petstore-swagger2.json`;
+      });
+
+      Then(
+        'creating a mock with seed count "abc" should throw about non-negative integer',
+        async () => {
+          await expect(
+            openapi({
+              spec: specPath,
+              seed: { pets: { count: "abc" as any } },
+            }),
+          ).rejects.toThrow("non-negative integer");
+        },
+      );
+    },
+  );
+
+  Scenario(
+    "Malformed seed file produces descriptive error",
+    ({ Given, Then }) => {
+      Given("a Petstore spec path", () => {
+        specPath = `${fixturesDir}/petstore-swagger2.json`;
+      });
+
+      Then(
+        "creating a mock with malformed seed file should throw about invalid JSON",
+        async () => {
+          const badFile = resolve(scratchDir, "__bad-seed-temp.json");
+          writeFileSync(badFile, "NOT VALID JSON{{{");
+          try {
+            await expect(
+              openapi({
+                spec: specPath,
+                seed: { pets: badFile },
+              }),
+            ).rejects.toThrow("invalid JSON");
+          } finally {
+            const { unlinkSync } = await import("node:fs");
+            unlinkSync(badFile);
+          }
+        },
+      );
+    },
+  );
 });

package/src/utils.ts

@@ -0,0 +1,9 @@
+import type { JSONSchema7 } from "json-schema";
+
+export function isRecord(value: unknown): value is Record<string, unknown> {
+  return typeof value === "object" && value !== null && !Array.isArray(value);
+}
+
+export function toJsonSchema(node: Record<string, unknown>): JSONSchema7 {
+  return Object.assign<JSONSchema7, Record<string, unknown>>({}, node);
+}