@avocadostudio-ai/orchestrator-core 0.7.0 → 0.8.0

package/dist/http/publish-actions.d.ts

@@ -74,7 +74,7 @@ export declare function loadPublishedForDiff(opts: {
     logger: Logger;
 }): Promise<{
     pages: PageDoc[];
-    siteConfig: SiteConfig | null;
+    siteConfig: SiteConfig | null | undefined;
     source: PublishedPagesSource;
 }>;
 /**

package/dist/http/publish-actions.js

@@ -68,7 +68,8 @@ async function readPublishedJsonFile() {
 export async function loadPublishedForDiff(opts) {
     const { siteOrigin, logger } = opts;
     let remotePages = null;
-    let remoteSiteConfig = null;
+    /** `undefined` = the site did not say. See `PublishedContent`. */
+    let remoteSiteConfig;
     if (siteOrigin) {
         try {
             const res = await fetch(`${siteOrigin.replace(/\/+$/, "")}/api/editor/pages`, {
@@ -78,7 +79,13 @@ export async function loadPublishedForDiff(opts) {
                 const data = (await res.json());
                 if (Array.isArray(data.pages)) {
                     remotePages = data.pages;
-                    remoteSiteConfig = data.siteConfig ?? null;
+                    /*
+                     * Absent and `null` are different answers and must stay different.
+                     * `null` is the site saying it publishes no header config; absent is
+                     * an editor API that does not report one at all, which is most of
+                     * them — an Astro site returns `{pages}` and nothing else.
+                     */
+                    remoteSiteConfig = "siteConfig" in data ? (data.siteConfig ?? null) : undefined;
                 }
             }
             else {
@@ -89,17 +96,29 @@ export async function loadPublishedForDiff(opts) {
             logger.warn({ siteOrigin, err: String(err) }, "publish/diff: site fetch failed, falling back");
         }
     }
-    // Hot path: remote returned both pages and siteConfig.
-    if (remotePages && remoteSiteConfig) {
+    // Hot path: the remote answered about both.
+    if (remotePages && remoteSiteConfig !== undefined) {
         return { pages: remotePages, siteConfig: remoteSiteConfig, source: "site" };
     }
-    // Either the remote didn't run, didn't include siteConfig (older SDK or
-    // site dev not yet restarted), or didn't include pages. Read the JSON file
-    // to backfill the missing pieces — without it, a missing siteConfig would
-    // be misread as "header added · all fields" on every publish-diff load.
+    // Either the remote didn't run, didn't mention siteConfig, or didn't include
+    // pages. Read the JSON file to backfill the missing pieces — without it, a
+    // missing siteConfig would be misread as "header added · all fields" on
+    // every publish-diff load.
     const fromFile = await readPublishedJsonFile();
     const pages = remotePages ?? fromFile.pages;
-    const siteConfig = remoteSiteConfig ?? fromFile.siteConfig;
+    /*
+     * The file is `apps/site`'s published content, so it may only answer for
+     * `apps/site`. When the remote supplied the pages it is the authority on its
+     * own header config, and a site that did not mention one has not said it has
+     * none — it has said nothing, which `publishDiffAction` reads as "no header
+     * change" rather than as a change.
+     *
+     * Backfilling here instead is what made every diff for a site whose editor
+     * API returns only `{pages}` report the bundled demo's name, logo, navGroups
+     * and navLabels as the "before" — seven changed fields, all of them
+     * describing a site nobody had asked about.
+     */
+    const siteConfig = remotePages ? undefined : (remoteSiteConfig ?? fromFile.siteConfig);
     if (pages)
         return { pages, siteConfig, source: remotePages ? "site" : "file" };
     logger.warn("publish/diff: falling back to in-memory publishedPages — diff may be inaccurate");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@avocadostudio-ai/orchestrator-core",
-  "version": "0.7.0",
+  "version": "0.8.0",
   "type": "module",
   "exports": {
     "./package.json": "./package.json",
@@ -22,8 +22,8 @@
     "openai": "^4.87.1",
     "sharp": "^0.34.5",
     "zod": "^4.3.6",
-    "@avocadostudio-ai/migration-sdk": "^0.7.0",
-    "@avocadostudio-ai/shared": "^0.7.0"
+    "@avocadostudio-ai/shared": "^0.8.0",
+    "@avocadostudio-ai/migration-sdk": "^0.8.0"
   },
   "devDependencies": {
     "@anthropic-ai/claude-agent-sdk": "^0.3.220",
@@ -58,12 +58,7 @@
   "license": "Apache-2.0",
   "homepage": "https://docs.avocadostudio.dev",
   "bugs": {
-    "url": "https://github.com/avocadostudio-ai/avocado/issues"
-  },
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/avocadostudio-ai/avocado.git",
-    "directory": "packages/orchestrator-core"
+    "url": "https://docs.avocadostudio.dev"
   },
   "peerDependencies": {
     "googleapis": "^171.4.0",