@avocadostudio-ai/orchestrator-core 0.6.0 → 0.7.0

package/dist/ops/ops-engine.js

@@ -1,5 +1,5 @@
 import { z } from "zod";
-import { blockSchemas, operationSchema, validateBlockProps, findManifestSchemaIssue, isChrome, getMediaFields, isInBlockCatalogue, catalogueBlockTypes, mapSemanticThemeTokens, generateItemId, isRichTextDoc, fromMarkdown, mergeRichTextDoc } from "@avocadostudio-ai/shared";
+import { blockSchemas, operationSchema, validateBlockProps, findManifestSchemaIssue, isChrome, getMediaFields, isInBlockCatalogue, catalogueBlockTypes, mapSemanticThemeTokens, generateItemId, isRichTextDoc, fromMarkdown, mergeRichTextDoc, resolveManifestFieldMeta } from "@avocadostudio-ai/shared";
 import { normalizeRouteCandidate } from "../nlp/intent-helpers.js";
 import { pageIdFromSlug, pageTitleFromSlug } from "../nlp/plan-normalizer.js";
 import { OperationError, toErrorDetail as _unifiedToErrorDetail } from "../errors.js";
@@ -692,6 +692,49 @@ function _validateWithManifestIfPresent(manifestByType, blockType, nextProps) {
     }
     return propCheck.data;
 }
+/**
+ * A row added to a polymorphic list must say which shape it is.
+ *
+ * A list whose field meta carries a `discriminator` holds several shapes, and
+ * the row's value for that key is the only thing that says which. A row that
+ * omits it is not a row with one field missing: it is a row that corresponds to
+ * nothing the site can draw, and on a CMS that stores each row as its own
+ * document — `component` in Storyblok, `_type` in Sanity — it cannot be
+ * constructed at all. An adapter's only choices are to drop it or to guess.
+ *
+ * Dropping it is what happened. Asked for a statistics section, a planner added
+ * a card grid and filled it with four rows carrying every declared field and no
+ * `component`; the adapter skipped all four; **the page rendered the grid's
+ * heading over empty white space**, and the only trace anywhere was a
+ * `console.warn` in a dev server log. The manifest had the answer the whole
+ * time — the list declares its discriminator and names the types it admits.
+ *
+ * So reject it here, in the same place and the same category as an unknown
+ * prop, where the failure is loud and lands at the moment of the mistake with
+ * the admissible types in the message.
+ *
+ * Only omission is rejected. A value naming no branch is left alone on purpose:
+ * `itemFieldsByType` is derived from a schema that may enumerate fewer types
+ * than the CMS has, and `resolveListItemFields` already falls back to the union
+ * for exactly that case. Rejecting there would refuse rows a site can render.
+ */
+function _requireListDiscriminator(manifestByType, blockType, listKey, item) {
+    const definition = manifestByType.get(blockType);
+    if (!definition)
+        return;
+    const listField = resolveManifestFieldMeta(definition).listFields[listKey];
+    const discriminator = listField?.discriminator;
+    if (!discriminator)
+        return;
+    const value = item[discriminator];
+    if (typeof value === "string" && value.length > 0)
+        return;
+    if (typeof value === "number" || typeof value === "boolean")
+        return;
+    const admissible = Object.keys(listField.itemFieldsByType ?? {}).filter((t) => t.length > 0);
+    const choices = admissible.length > 0 ? `: ${admissible.join(", ")}` : "";
+    throw new OperationError(`A new ${listKey} row must set "${discriminator}", which names its type${choices}`, { category: "schema_violation" });
+}
 function _requireManifestComponent(manifestByType, blockType, operationName) {
     if (manifestByType.size === 0)
         return;
@@ -1207,6 +1250,7 @@ async function _applyOpsAtomicallyUnsafe(session, ops, options) {
                 // (id included), and a duplicate id would make every later id lookup
                 // silently resolve to the first occurrence.
                 const newItem = structuredClone(op.item);
+                _requireListDiscriminator(manifestByType, block.type, op.listKey, newItem);
                 if (typeof newItem.id !== "string" ||
                     newItem.id.length === 0 ||
                     _itemIndexById(nextList, newItem.id) !== -1) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@avocadostudio-ai/orchestrator-core",
-  "version": "0.6.0",
+  "version": "0.7.0",
   "type": "module",
   "exports": {
     "./package.json": "./package.json",
@@ -22,20 +22,8 @@
     "openai": "^4.87.1",
     "sharp": "^0.34.5",
     "zod": "^4.3.6",
-    "@avocadostudio-ai/migration-sdk": "^0.6.0",
-    "@avocadostudio-ai/shared": "^0.6.0"
-  },
-  "peerDependencies": {
-    "googleapis": "^171.4.0",
-    "@google/genai": "^1.46.0"
-  },
-  "peerDependenciesMeta": {
-    "googleapis": {
-      "optional": true
-    },
-    "@google/genai": {
-      "optional": true
-    }
+    "@avocadostudio-ai/migration-sdk": "^0.7.0",
+    "@avocadostudio-ai/shared": "^0.7.0"
   },
   "devDependencies": {
     "@anthropic-ai/claude-agent-sdk": "^0.3.220",
@@ -77,6 +65,22 @@
     "url": "https://github.com/avocadostudio-ai/avocado.git",
     "directory": "packages/orchestrator-core"
   },
+  "peerDependencies": {
+    "googleapis": "^171.4.0",
+    "@google/genai": "^1.46.0",
+    "@anthropic-ai/claude-agent-sdk": ">=0.3.0"
+  },
+  "peerDependenciesMeta": {
+    "googleapis": {
+      "optional": true
+    },
+    "@google/genai": {
+      "optional": true
+    },
+    "@anthropic-ai/claude-agent-sdk": {
+      "optional": true
+    }
+  },
   "scripts": {
     "typecheck": "tsc --noEmit",
     "build": "tsc -p tsconfig.build.json && node ./scripts/copy-assets.mjs",