@avocadostudio-ai/orchestrator-core 0.26.0 → 0.28.0

package/dist/handler/create-orchestrator.js

@@ -65,7 +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 { declareLibraryMount, observeLibraryMount } from "./library-mount.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";
 async function buildRuntime(config) {
@@ -538,9 +538,23 @@ export function createOrchestrator(config = {}) {
     // See `library-mount.ts` for why this is not simply a config value.
     if (config.previewUrl)
         declareLibraryMount(`${config.previewUrl.replace(/\/+$/, "")}${basePath}`);
+    /*
+     * Requests this process's own preview dispatches in-process (see
+     * `library-mount.ts`). Membership is the credential: a request that arrived
+     * over the network is a different object, so it can never be in the set, and
+     * nothing about it — no header, no URL — can put it there. The bypass covers
+     * the three GET draft reads the SDK makes and nothing else, so code that
+     * reaches the dispatcher can read the drafts the preview renders, but never
+     * edit or publish.
+     */
+    const inProcessRequests = new WeakSet();
+    const IN_PROCESS_READS = new Set(["/draft/pages", "/draft/slugs", "/draft/site-config"]);
     const handler = async function handler(request) {
         const url = new URL(request.url);
-        observeLibraryMount(`${url.origin}${basePath}`);
+        const inProcess = inProcessRequests.has(request);
+        // An in-process URL is synthetic; recording it would mislead every HTTP reader.
+        if (!inProcess)
+            observeLibraryMount(`${url.origin}${basePath}`);
         const path = stripBasePath(url.pathname, basePath);
         const cors = corsHeadersFor(request, config);
         if (request.method === "OPTIONS") {
@@ -566,9 +580,12 @@ export function createOrchestrator(config = {}) {
          * (the /auth exchange itself, and image reads an <img> tag cannot
          * authenticate) pass through inside checkAuth.
          */
-        const gate = await checkAuth({ request, path, auth: config.auth, cors, log: gateLogger });
-        if (gate.response)
-            return gate.response;
+        const trustedRead = inProcess && request.method === "GET" && IN_PROCESS_READS.has(path);
+        if (!trustedRead) {
+            const gate = await checkAuth({ request, path, auth: config.auth, cors, log: gateLogger });
+            if (gate.response)
+                return gate.response;
+        }
         if (request.method === "GET" && path === "/auth/status") {
             /*
              * `gateEnabled` answers one question for the editor's PasswordGate: "do I
@@ -2647,7 +2664,13 @@ export function createOrchestrator(config = {}) {
                 "Use the Fastify orchestrator at apps/orchestrator for the full surface (sites, the session directory, the agent surface, etc.)."
         }, { status: 405, cors });
     };
+    const dispatchInProcess = (request) => {
+        inProcessRequests.add(request);
+        return handler(request);
+    };
+    registerLibraryDispatch(basePath, dispatchInProcess);
     handler.dispose = async () => {
+        clearLibraryDispatch(dispatchInProcess);
         if (!runtimePromise)
             return;
         try {

package/dist/handler/library-mount.d.ts

@@ -1,6 +1,11 @@
 export declare const LIBRARY_MOUNT_KEY = "__avocado_library_mount__";
+export type LibraryDispatch = (request: Request) => Promise<Response>;
 /** Called at construction, from `previewUrl` + `basePath`. */
 export declare function declareLibraryMount(url: string): void;
 /** Called on each request, from the URL the runtime actually received. */
 export declare function observeLibraryMount(url: string): void;
 export declare function getLibraryMount(): string | null;
+/** Called at construction: this handler answers in-process draft reads. */
+export declare function registerLibraryDispatch(basePath: string, dispatch: LibraryDispatch): void;
+/** Called on dispose, and only clears the slot if it is still this handler's. */
+export declare function clearLibraryDispatch(dispatch: LibraryDispatch): void;

package/dist/handler/library-mount.js

@@ -43,3 +43,17 @@ export function getLibraryMount() {
     const s = slot();
     return s.observed ?? s.declared ?? null;
 }
+/** Called at construction: this handler answers in-process draft reads. */
+export function registerLibraryDispatch(basePath, dispatch) {
+    const s = slot();
+    s.basePath = basePath.replace(/\/+$/, "");
+    s.dispatch = dispatch;
+}
+/** Called on dispose, and only clears the slot if it is still this handler's. */
+export function clearLibraryDispatch(dispatch) {
+    const s = slot();
+    if (s.dispatch !== dispatch)
+        return;
+    delete s.dispatch;
+    delete s.basePath;
+}

package/dist/ops/ops-engine.js

@@ -766,6 +766,73 @@ function _requireListDiscriminator(manifestByType, blockType, listKey, item) {
     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" });
 }
+/*
+ * A list row must use the list's own field names.
+ *
+ * Integrations declare item schemas loose, so bookkeeping keys a CMS adds
+ * (`_type`, `_key`) ride through validation, and every field is optional. Such
+ * a schema accepts any object. A planner that had just written Avocado's
+ * built-in FAQ shape, `{ q, a }`, into a site whose accordion reads
+ * `{ title, body }` got an "applied" and a row that rendered empty, with the
+ * text present in the draft under names nothing reads.
+ *
+ * So a key is refused when the list neither declares it nor already holds it
+ * on another row: copying an existing row, undeclared keys and all, still
+ * works, and only a name the list has never seen is refused. The error names
+ * the fields the list does use, so the planner can correct itself. Lists with
+ * no declared item fields are left alone, since there is nothing to check
+ * against.
+ */
+/** Property names an item of `listKey` may carry, per the block's JSON Schema. */
+function _schemaItemKeys(definition, listKey) {
+    const props = definition.propsSchema?.properties;
+    const listSchema = props?.[listKey];
+    const keys = new Set();
+    const visit = (node) => {
+        if (!node || typeof node !== "object")
+            return;
+        const n = node;
+        for (const key of Object.keys(n.properties ?? {}))
+            keys.add(key);
+        for (const branch of [...(n.oneOf ?? []), ...(n.anyOf ?? []), ...(n.allOf ?? [])])
+            visit(branch);
+    };
+    visit(listSchema?.items);
+    return [...keys];
+}
+function _requireKnownItemKeys(manifestByType, blockType, listKey, list, item) {
+    const definition = manifestByType.get(blockType);
+    if (!definition)
+        return;
+    const listField = resolveManifestFieldMeta(definition).listFields[listKey];
+    if (!listField)
+        return;
+    const declared = new Set(Object.keys(listField.itemFields ?? {}));
+    for (const fields of Object.values(listField.itemFieldsByType ?? {})) {
+        for (const key of Object.keys(fields ?? {}))
+            declared.add(key);
+    }
+    if (listField.discriminator)
+        declared.add(listField.discriminator);
+    // The props schema names every item property, optional ones included; the
+    // field meta above names only what the panel offers.
+    for (const key of _schemaItemKeys(definition, listKey))
+        declared.add(key);
+    if (declared.size === 0)
+        return;
+    const known = new Set(declared);
+    for (const row of list) {
+        if (row && typeof row === "object" && !Array.isArray(row)) {
+            for (const key of Object.keys(row))
+                known.add(key);
+        }
+    }
+    const unknown = Object.keys(item).filter((key) => key !== "id" && !key.startsWith("_") && !known.has(key));
+    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" });
+}
 function _requireManifestComponent(manifestByType, blockType, operationName) {
     if (manifestByType.size === 0)
         return;
@@ -1449,6 +1516,7 @@ async function _applyOpsAtomicallyUnsafe(session, ops, options) {
                 // silently resolve to the first occurrence.
                 const newItem = structuredClone(op.item);
                 _requireListDiscriminator(manifestByType, block.type, op.listKey, newItem);
+                _requireKnownItemKeys(manifestByType, block.type, op.listKey, list, newItem);
                 if (typeof newItem.id !== "string" ||
                     newItem.id.length === 0 ||
                     _itemIndexById(nextList, newItem.id) !== -1) {
@@ -1476,6 +1544,7 @@ async function _applyOpsAtomicallyUnsafe(session, ops, options) {
                 // A patch must never clobber the stable item id (it's the addressing key).
                 const itemPatch = { ...op.patch };
                 delete itemPatch.id;
+                _requireKnownItemKeys(manifestByType, block.type, op.listKey, list, itemPatch);
                 const nextList = list.map((entry, idx) => {
                     if (idx !== targetIndex)
                         return entry;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@avocadostudio-ai/orchestrator-core",
-  "version": "0.26.0",
+  "version": "0.28.0",
   "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.26.0",
-    "@avocadostudio-ai/shared": "^0.26.0"
+    "@avocadostudio-ai/migration-sdk": "^0.28.0",
+    "@avocadostudio-ai/shared": "^0.28.0"
   },
   "devDependencies": {
     "@anthropic-ai/claude-agent-sdk": "^0.3.220",