@bygghammar/cms-platform 0.92.0 → 0.92.1

package/CHANGELOG.md

@@ -8,6 +8,16 @@ matches `package.json` exactly. See
 Versions 0.1.0 through 0.78.0 predate this repository and were reconstructed
 from the published npm tarballs.
 
+## 0.92.1
+
+- Fix the page-builder save and publish RPC call by removing the unsupported
+  `_source_page_id` argument. The database supplies the source page internally,
+  preserving self-reveal protection. No migration is added or changed, and
+  platform contract v3 is unchanged.
+- Clarify publication RPC errors so an argument mismatch is not presented as
+  proof that migrations must be replayed. Include only a recognized diagnostic
+  code and request a schema comparison before applying changes.
+
 ## 0.92.0
 
 **Breaking: platform contract v3.** A site pinned to this package must call

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bygghammar/cms-platform",
-  "version": "0.92.0",
+  "version": "0.92.1",
   "description": "Shared CMS platform: blocks, block types, i18n core, UI primitives and design-token contract used by every Bygghammar/Voyara site.",
   "type": "module",
   "sideEffects": [

package/src/server/pages-core.ts

@@ -377,9 +377,9 @@ export async function getPageForEditCore({ context }: PagesDeps, key: string) {
 }
 
 /**
- * Codes and wording that mean "this installation has not run the publication
- * migration yet". `PGRST202`/`PGRST203` come from PostgREST, `42883`/`42P01`
- * from PostgreSQL itself.
+ * Codes and wording that mean "this installation has no such function, or none
+ * that takes these arguments". `PGRST202`/`PGRST203` come from PostgREST, which
+ * answers the same way for both; `42883`/`42P01` from PostgreSQL itself.
  */
 const FUNCTION_MISSING = new Set(["PGRST202", "PGRST203", "42883", "42P01"]);
 
@@ -423,10 +423,14 @@ async function publicationRpc(
   ) {
     // A slice of the generated migration manifest rather than a second copy of
     // the names: an operator sent after a file this package has stopped
-    // shipping is worse off than one told nothing.
+    // shipping is worse off than one told nothing. Only a code from the set
+    // above is repeated; the backend's own message is not.
     throw new Error(
-      `${whenFailed}: this database does not have ${fn}. Apply the migrations ` +
-        `${installedBy.join(" and ")} shipped with this package and try again.`,
+      `${whenFailed}: the database has no ${fn} that accepts this call` +
+        `${FUNCTION_MISSING.has(code) ? ` (${code})` : ""}. Either the migrations ` +
+        `${installedBy.join(" and ")} shipped with this package are not installed, ` +
+        `or the installed ${fn} takes different arguments than this package sends. ` +
+        `Compare the database's schema with this package before applying anything.`,
     );
   }
   throw new Error(message || whenFailed);
@@ -474,10 +478,10 @@ export async function savePageVersionCore(
       _note: input.note ?? null,
       _publish: Boolean(input.publish),
       _actor_email: actorEmail(context),
+      // No `_source_page_id`: the function takes none, and names `_page_id` as
+      // the source of every reveal itself, so a countdown pointing at the page
+      // being published is still refused rather than hiding it.
       _reveals: reveals,
-      // Named so a countdown pointing at the page being published is refused
-      // rather than hiding it until its own deadline.
-      _source_page_id: input.pageId,
       _audit: input.publish
         ? {
             description: