@voyantjs/catalog-authoring 0.104.1 → 0.105.0

package/dist/validate.d.ts

@@ -0,0 +1,17 @@
+import type { AuthoringIssue } from "./errors.js";
+import type { ProductGraphSpec } from "./spec.js";
+/**
+ * Category-aware shape validation. Runs before the builder (compose path) so a
+ * wrong-shape spec is rejected with descriptive, agent-recoverable issues
+ * instead of producing a malformed-but-bookable product.
+ *
+ * Keyed on `bookingMode` (the structural classifier that exists in the schema
+ * today; `supplyModel` is still proposed). See the voyantjs/platform authoring
+ * spec for the canonical per-category rules.
+ *
+ * Scope: the generic products graph — excursion (`date`/`date_time`), multi-day
+ * tour/package (`itinerary`), transfer (`transfer`). Other modes (`stay`,
+ * `open`, `other`) pass through leniently in v1.
+ */
+export declare function validateProductGraph(spec: ProductGraphSpec): AuthoringIssue[];
+//# sourceMappingURL=validate.d.ts.map

package/dist/validate.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"validate.d.ts","sourceRoot":"","sources":["../src/validate.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,aAAa,CAAA;AACjD,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,WAAW,CAAA;AAEjD;;;;;;;;;;;;GAYG;AACH,wBAAgB,oBAAoB,CAAC,IAAI,EAAE,gBAAgB,GAAG,cAAc,EAAE,CAyE7E"}

package/dist/validate.js

@@ -0,0 +1,83 @@
+/**
+ * Category-aware shape validation. Runs before the builder (compose path) so a
+ * wrong-shape spec is rejected with descriptive, agent-recoverable issues
+ * instead of producing a malformed-but-bookable product.
+ *
+ * Keyed on `bookingMode` (the structural classifier that exists in the schema
+ * today; `supplyModel` is still proposed). See the voyantjs/platform authoring
+ * spec for the canonical per-category rules.
+ *
+ * Scope: the generic products graph — excursion (`date`/`date_time`), multi-day
+ * tour/package (`itinerary`), transfer (`transfer`). Other modes (`stay`,
+ * `open`, `other`) pass through leniently in v1.
+ */
+export function validateProductGraph(spec) {
+    const issues = [];
+    const mode = spec.product.bookingMode;
+    const totalDays = spec.itineraries.reduce((n, i) => n + i.days.length, 0);
+    const allUnits = spec.options.flatMap((o) => o.units);
+    // Every bookable product needs at least one option with a unit.
+    if (spec.options.length === 0) {
+        issues.push({
+            code: "no_options",
+            field: "options",
+            message: "Product has no options, so it cannot be booked.",
+            fix: "Add at least one option with at least one unit.",
+        });
+    }
+    else if (allUnits.length === 0) {
+        issues.push({
+            code: "no_units",
+            field: "options[].units",
+            message: "No option has any units, so there is nothing to price or book.",
+            fix: "Add at least one unit (e.g. an 'Adult' person unit) to an option.",
+        });
+    }
+    if (mode === "date" || mode === "date_time") {
+        if (totalDays > 1) {
+            issues.push({
+                code: "excursion_multi_day",
+                field: "itineraries",
+                message: `A '${mode}' (excursion) product is single-day, but the spec has ${totalDays} itinerary days.`,
+                fix: "Use bookingMode 'itinerary' for a multi-day product, or reduce the itinerary to a single day.",
+            });
+        }
+        const roomUnit = allUnits.find((u) => u.unitType === "room");
+        if (roomUnit) {
+            issues.push({
+                code: "excursion_room_unit",
+                field: "options[].units[].unitType",
+                message: `Excursions price per person, but unit '${roomUnit.name}' has unitType 'room'.`,
+                fix: "Set the unit's unitType to 'person', or switch bookingMode to 'stay'/'itinerary' for room-based products.",
+            });
+        }
+    }
+    if (mode === "itinerary" && totalDays < 2) {
+        issues.push({
+            code: "tour_needs_days",
+            field: "itineraries",
+            message: `A multi-day ('itinerary') product needs at least 2 itinerary days; found ${totalDays}.`,
+            fix: "Add itinerary days, or use bookingMode 'date' for a single-day excursion.",
+        });
+    }
+    if (mode === "transfer") {
+        if (totalDays > 0) {
+            issues.push({
+                code: "transfer_no_days",
+                field: "itineraries",
+                message: `Transfers are point-to-point and take no itinerary days; found ${totalDays}.`,
+                fix: "Remove the itinerary days; model the journey via pickup/dropoff pricing instead.",
+            });
+        }
+        const badUnit = allUnits.find((u) => u.unitType !== "vehicle" && u.unitType !== "person");
+        if (badUnit) {
+            issues.push({
+                code: "transfer_unit_type",
+                field: "options[].units[].unitType",
+                message: `Transfer unit '${badUnit.name}' has unitType '${badUnit.unitType}'; transfers sell per vehicle or per seat.`,
+                fix: "Set the unit's unitType to 'vehicle' or 'person'.",
+            });
+        }
+    }
+    return issues;
+}

package/package.json

@@ -1,66 +1,50 @@
 {
   "name": "@voyantjs/catalog-authoring",
-  "version": "0.104.1",
+  "version": "0.105.0",
   "license": "Apache-2.0",
   "type": "module",
   "exports": {
-    ".": "./src/index.ts",
-    "./schema": "./src/schema.ts",
-    "./spec": "./src/spec.ts",
-    "./extension": "./src/extension.ts"
-  },
-  "scripts": {
-    "typecheck": "tsc --noEmit",
-    "lint": "biome check src/",
-    "test": "vitest run",
-    "build": "tsc -p tsconfig.json",
-    "clean": "rm -rf dist tsconfig.tsbuildinfo",
-    "prepack": "pnpm run build"
+    ".": {
+      "types": "./dist/index.d.ts",
+      "import": "./dist/index.js",
+      "default": "./dist/index.js"
+    },
+    "./schema": {
+      "types": "./dist/schema.d.ts",
+      "import": "./dist/schema.js",
+      "default": "./dist/schema.js"
+    },
+    "./spec": {
+      "types": "./dist/spec.d.ts",
+      "import": "./dist/spec.js",
+      "default": "./dist/spec.js"
+    },
+    "./extension": {
+      "types": "./dist/extension.d.ts",
+      "import": "./dist/extension.js",
+      "default": "./dist/extension.js"
+    }
   },
   "dependencies": {
-    "@voyantjs/core": "workspace:^",
-    "@voyantjs/db": "workspace:^",
-    "@voyantjs/hono": "workspace:^",
-    "@voyantjs/pricing": "workspace:^",
-    "@voyantjs/products": "workspace:^",
     "drizzle-orm": "^0.45.2",
     "hono": "^4.12.10",
-    "zod": "^4.3.6"
+    "zod": "^4.3.6",
+    "@voyantjs/core": "^0.104.1",
+    "@voyantjs/db": "^0.104.1",
+    "@voyantjs/hono": "^0.104.1",
+    "@voyantjs/pricing": "^0.104.1",
+    "@voyantjs/products": "^0.104.3"
   },
   "devDependencies": {
-    "@voyantjs/voyant-test-utils": "workspace:^",
-    "@voyantjs/voyant-typescript-config": "workspace:^",
-    "typescript": "^6.0.2"
+    "typescript": "^6.0.2",
+    "@voyantjs/voyant-test-utils": "^0.1.0",
+    "@voyantjs/voyant-typescript-config": "^0.1.0"
   },
   "files": [
     "dist"
   ],
   "publishConfig": {
-    "access": "public",
-    "exports": {
-      ".": {
-        "types": "./dist/index.d.ts",
-        "import": "./dist/index.js",
-        "default": "./dist/index.js"
-      },
-      "./schema": {
-        "types": "./dist/schema.d.ts",
-        "import": "./dist/schema.js",
-        "default": "./dist/schema.js"
-      },
-      "./spec": {
-        "types": "./dist/spec.d.ts",
-        "import": "./dist/spec.js",
-        "default": "./dist/spec.js"
-      },
-      "./extension": {
-        "types": "./dist/extension.d.ts",
-        "import": "./dist/extension.js",
-        "default": "./dist/extension.js"
-      }
-    },
-    "main": "./dist/index.js",
-    "types": "./dist/index.d.ts"
+    "access": "public"
   },
   "repository": {
     "type": "git",
@@ -74,5 +58,14 @@
       "@voyantjs/products",
       "@voyantjs/pricing"
     ]
-  }
-}
+  },
+  "scripts": {
+    "typecheck": "tsc --noEmit",
+    "lint": "biome check src/",
+    "test": "vitest run",
+    "build": "tsc -p tsconfig.json",
+    "clean": "rm -rf dist tsconfig.tsbuildinfo"
+  },
+  "main": "./dist/index.js",
+  "types": "./dist/index.d.ts"
+}