@avocadostudio-ai/orchestrator-core 0.28.0 → 0.29.1

package/dist/handler/create-orchestrator.js

@@ -21,7 +21,7 @@ import { mkdir, writeFile, readFile } from "node:fs/promises";
 import { resolve, basename } from "node:path";
 import { randomUUID } from "node:crypto";
 import { z } from "zod";
-import { operationSchema, blockManifestSchema, siteConfigSchema, declareBlockCatalogue, undeclaredBlockTypes, EDITOR_PROTOCOL_VERSION, withLoopbackSpellings, editorAllowHeaders, compareDraftSecret } from "@avocadostudio-ai/shared";
+import { operationSchema, blockManifestSchema, siteConfigSchema, declareBlockCatalogue, undeclaredBlockTypes, EDITOR_PROTOCOL_VERSION, withLoopbackSpellings, editorAllowHeaders, compareDraftSecret, buildBlockManifest } from "@avocadostudio-ai/shared";
 import { keyFileHint } from "../env-file-location.js";
 import { chatRequestBodySchema } from "../nlp/intent-detection.js";
 import { applyOpsAtomically, pickFocusBlockId, pickUpdatedSlug, toErrorDetail, classifyGuardrailError } from "../ops/ops-engine.js";
@@ -65,6 +65,7 @@ import { resolveCapabilities } from "../cms/adapter.js";
 import { mediaSourceFromUnknown } from "../cms/media-sources.js";
 import { extractAccessToken, isAccessGateEnabled, isValidAccessToken, mintAccessToken, verifyAccessPassword } from "../http/access-tokens.js";
 import { editorCredentials } from "../http/editor-credentials.js";
+import { OperationError } from "../errors.js";
 import { clearLibraryDispatch, declareLibraryMount, observeLibraryMount, registerLibraryDispatch } from "./library-mount.js";
 import { checkAuth, resolveAuth, CLOSED_HINT } from "./auth.js";
 import { setSiteAssetLister, invalidateSiteAssets } from "../state/site-assets.js";
@@ -1605,6 +1606,19 @@ export function createOrchestrator(config = {}) {
                         manifest = pm.data;
                 }
             }
+            /*
+             * No manifest in the request: use this mount's own. The editor sends one
+             * with every chat turn, but an MCP agent or a script calling /ops sends
+             * none, and without it every manifest check in the ops engine is skipped:
+             * read-only fields, fixed blocks, list discriminators, and list rows in a
+             * shape nothing renders. Library mode is the one host that already knows
+             * the site's blocks, the same ones `GET /blocks/manifest` serves.
+             */
+            if (!manifest) {
+                const own = buildBlockManifest();
+                if (own.blocks.length > 0)
+                    manifest = own;
+            }
             /*
              * Validate-only, before anything is snapshotted or bumped. This branch
              * did not exist: `dryRun` was parsed out of the body by nobody and the
@@ -1703,7 +1717,10 @@ export function createOrchestrator(config = {}) {
             }
             catch (error) {
                 const reason = toErrorDetail(error);
-                return jsonResponse({ error: reason, errorCode: classifyGuardrailError(reason) }, { status: 400, cors });
+                // An OperationError already knows its category; the text classifier is
+                // for errors that do not, and called a refused list row "internal_error".
+                const category = error instanceof OperationError ? error.category : undefined;
+                return jsonResponse({ error: reason, errorCode: category ?? classifyGuardrailError(reason) }, { status: 400, cors });
             }
         }
         /*

package/dist/ops/ops-engine.js

@@ -831,7 +831,7 @@ function _requireKnownItemKeys(manifestByType, blockType, listKey, list, item) {
     if (unknown.length === 0)
         return;
     throw new OperationError(`${listKey} rows have no field ${unknown.map((k) => `"${k}"`).join(", ")}. ` +
-        `Use the list's own fields: ${[...declared].join(", ")}`, { category: "schema_violation" });
+        `Use the list's own fields: ${[...declared].filter((key) => !key.startsWith("_")).join(", ")}`, { category: "schema_violation" });
 }
 function _requireManifestComponent(manifestByType, blockType, operationName) {
     if (manifestByType.size === 0)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@avocadostudio-ai/orchestrator-core",
-  "version": "0.28.0",
+  "version": "0.29.1",
   "type": "module",
   "exports": {
     "./package.json": "./package.json",
@@ -22,8 +22,8 @@
     "openai": "^4.87.1",
     "sharp": "^0.35.4",
     "zod": "^4.3.6",
-    "@avocadostudio-ai/migration-sdk": "^0.28.0",
-    "@avocadostudio-ai/shared": "^0.28.0"
+    "@avocadostudio-ai/migration-sdk": "^0.29.1",
+    "@avocadostudio-ai/shared": "^0.29.1"
   },
   "devDependencies": {
     "@anthropic-ai/claude-agent-sdk": "^0.3.220",