@avocadostudio-ai/orchestrator-core 0.3.1 → 0.3.3

package/dist/ops/ops-engine.js

@@ -1,5 +1,5 @@
 import { z } from "zod";
-import { blockSchemas, operationSchema, validateBlockProps, findManifestSchemaIssue, isChrome, 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 } 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";
@@ -88,7 +88,17 @@ function remapRouteReference(value, fromSlug, toSlug) {
     }
     return value;
 }
-function rewriteRouteLinksInValue(input, fromSlug, toSlug) {
+/**
+ * Rewrite every route reference under `input` from `fromSlug` to `toSlug`.
+ *
+ * The walk is permissive by design — it remaps any string that starts with the
+ * old slug, whatever prop holds it — because a custom block ships no field
+ * metadata and its links still have to survive a rename. `mediaKeys` is the one
+ * exception: props that address a *resource* rather than a page. Renaming
+ * `/pricing` to `/plans` does not move `/pricing/demo.mp4`, and before this the
+ * walker rewrote that path too, quietly breaking the asset.
+ */
+function rewriteRouteLinksInValue(input, fromSlug, toSlug, mediaKeys) {
     if (typeof input === "string") {
         const mapped = remapRouteReference(input, fromSlug, toSlug);
         return { value: mapped, changed: mapped !== input };
@@ -96,7 +106,7 @@ function rewriteRouteLinksInValue(input, fromSlug, toSlug) {
     if (Array.isArray(input)) {
         let changed = false;
         const next = input.map((item) => {
-            const mapped = rewriteRouteLinksInValue(item, fromSlug, toSlug);
+            const mapped = rewriteRouteLinksInValue(item, fromSlug, toSlug, mediaKeys);
             if (mapped.changed)
                 changed = true;
             return mapped.value;
@@ -109,11 +119,8 @@ function rewriteRouteLinksInValue(input, fromSlug, toSlug) {
     let changed = false;
     const next = {};
     for (const [key, value] of Object.entries(source)) {
-        if (typeof value === "string" && key.toLowerCase().includes("href")) {
-            const mapped = remapRouteReference(value, fromSlug, toSlug);
-            if (mapped !== value)
-                changed = true;
-            next[key] = mapped;
+        if (mediaKeys.has(key)) {
+            next[key] = value;
             continue;
         }
         if (typeof value === "string" && key === "body") {
@@ -128,7 +135,7 @@ function rewriteRouteLinksInValue(input, fromSlug, toSlug) {
             next[key] = rewritten;
             continue;
         }
-        const mapped = rewriteRouteLinksInValue(value, fromSlug, toSlug);
+        const mapped = rewriteRouteLinksInValue(value, fromSlug, toSlug, mediaKeys);
         if (mapped.changed)
             changed = true;
         next[key] = mapped.value;
@@ -138,7 +145,7 @@ function rewriteRouteLinksInValue(input, fromSlug, toSlug) {
 function rewriteLinksToRenamedPage(page, fromSlug, toSlug) {
     let changed = false;
     const nextBlocks = page.blocks.map((block) => {
-        const mapped = rewriteRouteLinksInValue(block.props, fromSlug, toSlug);
+        const mapped = rewriteRouteLinksInValue(block.props, fromSlug, toSlug, getMediaFields(block.type));
         if (!mapped.changed)
             return block;
         changed = true;
@@ -208,16 +215,36 @@ export function classifyGuardrailError(reason) {
         lower.includes("required") ||
         lower.includes("unknown props") ||
         lower.includes("out of range") ||
-        lower.includes("must be")) {
+        lower.includes("must be") ||
+        // Every "already exists" this engine raises — a duplicate block id, a page
+        // slug that is taken — it raises *as* a schema_violation. This list did not
+        // know the word, so a plan one renamed id away from applying classified as
+        // `internal_error`, which the repair pass refuses to touch. Callers holding
+        // the thrown error should prefer its `category` over this function; the
+        // keyword is here for the paths that only ever had the string.
+        lower.includes("already exists")) {
         return "schema_violation";
     }
     return "internal_error";
 }
-export function formatValidationError(reason) {
-    return `${classifyGuardrailError(reason)}: ${reason}`;
+/**
+ * `category` is the one the failure was actually thrown with, for callers that
+ * still hold it. Without it this falls back to reading the category back out of
+ * the message, which is a guess — see `isRepairEligibleCategory`.
+ */
+export function formatValidationError(reason, category) {
+    return `${category ?? classifyGuardrailError(reason)}: ${reason}`;
+}
+/**
+ * The deterministic repair pass exists for plans that are structurally wrong
+ * but fixable — a bad prop name, an id that clashes. Anything else is either
+ * hopeless or not the planner's fault, and re-prompting it wastes a model call.
+ */
+export function isRepairEligibleCategory(category) {
+    return category === "schema_violation";
 }
 export function isDeterministicRepairEligible(reason) {
-    return classifyGuardrailError(reason) === "schema_violation";
+    return isRepairEligibleCategory(classifyGuardrailError(reason));
 }
 /**
  * Extracts structured fields from a planner schema_violation reason so the

package/dist/publish/publish-target-registry.js

@@ -10,7 +10,7 @@ import { DeployHookPublishTarget } from "./targets/deploy-hook.js";
  * handling requests:
  *
  * ```ts
- * import { registerPublishTarget } from "./publish/publish-target-registry.js"
+ * import { registerPublishTarget } from "@avocadostudio-ai/orchestrator-core"
  * registerPublishTarget(new S3PublishTarget())
  * ```
  *

package/dist/publish/publish-target.d.ts

@@ -6,7 +6,7 @@ import type { PublishTracker } from "../state/session-state.ts";
  * up writing content. Built-ins cover git + Vercel, a site-contract POST, and
  * a raw Vercel deploy hook. Third parties (S3, GitLab Pages, custom CI, etc.)
  * implement {@link PublishTarget} and register via
- * {@link registerPublishTarget} from `./publish-target-registry.js`.
+ * {@link registerPublishTarget} from `@avocadostudio-ai/orchestrator-core`.
  *
  * The route handler's job shrinks to: build {@link PublishContext}, call
  * {@link selectPublishTarget}, invoke `target.publish(ctx)`, save the returned

package/dist/state/session-state.js

@@ -3,6 +3,7 @@ import { dirname, resolve } from "node:path";
 import { demoPublishedPages, demoSiteConfig, ensureItemIds, IMAGE_PLACEHOLDER } from "@avocadostudio-ai/shared";
 import { toErrorDetail as _unifiedToErrorDetail } from "../errors.js";
 import { archiveMigratedJson, getStore, notePersistenceFailure, notePersistenceSuccess, readLegacyJson, resolveDbFile, resolveJsonMigrationTtlDays, sweepStaleMigrations, } from "./sqlite-store-singleton.js";
+import { discardPendingProposalsForSession, sweepExpiredProposals } from "../durable/durable-store-singleton.js";
 // Re-exported so callers that already depend on this module for state can ask
 // whether that state is actually being kept, without a second import path.
 export { persistenceHealth, persistenceWarning } from "./sqlite-store-singleton.js";
@@ -173,6 +174,9 @@ export function forgetSession(sessionKey) {
     pendingClarificationBySession.delete(sessionKey);
     chatHistoryBySession.delete(sessionKey);
     pendingApprovalPlanBySession.delete(sessionKey);
+    // Session ids are caller-supplied and routinely reused, so a pending proposal
+    // left behind here would rehydrate into the *next* session with this id.
+    discardPendingProposalsForSession(sessionKey);
     imageSourcePreferenceBySession.delete(sessionKey);
     publishStatusBySession.delete(sessionKey);
     capabilitiesBySession.delete(sessionKey);
@@ -1091,12 +1095,15 @@ const EPHEMERAL_MAP_CAP = 500; // max entries for maps without timestamps
  */
 export function evictStaleEphemeralMaps() {
     const now = Date.now();
-    // pendingApprovalPlanBySession: has createdAt — evict by age
+    // pendingApprovalPlanBySession: has createdAt — evict by age. The durable
+    // mirror carries the same TTL as `expiresAt`, so both halves age out together
+    // rather than the Map forgetting a plan the store would hand straight back.
     for (const [key, plan] of pendingApprovalPlanBySession) {
         if (now - new Date(plan.createdAt).getTime() > APPROVAL_PLAN_TTL_MS) {
             pendingApprovalPlanBySession.delete(key);
         }
     }
+    sweepExpiredProposals();
     // publishStatusBySession: has updatedAt — evict by age
     for (const [key, tracker] of publishStatusBySession) {
         if (now - new Date(tracker.updatedAt).getTime() > PUBLISH_STATUS_TTL_MS) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@avocadostudio-ai/orchestrator-core",
-  "version": "0.3.1",
+  "version": "0.3.3",
   "type": "module",
   "exports": {
     "./package.json": "./package.json",
@@ -23,8 +23,8 @@
     "openai": "^4.87.1",
     "sharp": "^0.34.5",
     "zod": "^4.3.6",
-    "@avocadostudio-ai/migration-sdk": "^0.3.1",
-    "@avocadostudio-ai/shared": "^0.3.1"
+    "@avocadostudio-ai/shared": "^0.3.3",
+    "@avocadostudio-ai/migration-sdk": "^0.3.3"
   },
   "devDependencies": {
     "@google/genai": "^1.46.0",