@jini-ai/cms 0.5.0 → 0.5.1

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 0.5.1 — 2026-10-06
+
+- Navigation: the `menus_update_menu_tree` schema declares the `entryType`/`lastKnownHref` hints on entry targets, and `updateMenuTree` carries a stored hint forward when the same item still targets the same entry and the submitted target omits it, so echoing a read no longer marks the menu changed.
+
 ## 0.5.0 — 2026-10-05
 
 ### BREAKING

package/dist/navigation/agent-tools.js

@@ -84,7 +84,15 @@ const NAV_TARGET_SCHEMA = {
             type: "object",
             additionalProperties: false,
             required: ["kind", "entryId"],
-            properties: { kind: { const: "entryRef" }, entryId: { type: "string", description: "Target entry id (e.g. a page or post)." } },
+            properties: {
+                kind: { const: "entryRef" },
+                entryId: { type: "string", description: "Target entry id (e.g. a page or post)." },
+                // Declared so a read can be echoed back unchanged: these were stored on page links but absent
+                // here, so every agent save dropped them (dry run 2026-10-05). The save also keeps the stored
+                // value when omitted for the same item id + entryId (`carryForwardEntryHints`).
+                entryType: { type: "string", description: "Content-type hint, e.g. 'page' or 'post'. Copy it unchanged from the read; omit for a new link." },
+                lastKnownHref: { type: "string", description: "The entry's last known public URL, e.g. '/about'. Copy it unchanged from the read; omit for a new link." },
+            },
         },
         {
             type: "object",
@@ -207,7 +215,7 @@ export const menusAgentToolCatalog = [
         name: "menus_update_menu_tree",
         description: "Replaces an existing menu's whole item tree (and optionally its title/slug) in one version-guarded write. " +
             "Always a whole-tree replace — there is no per-item patch operation, so the submitted tree must include every " +
-            "item the menu keeps. To delete a menu, use trash_item with entityType 'menu' (it moves to the Trash and can be restored).",
+            "item the menu keeps, each with its fields copied unchanged from the read. To delete a menu, use trash_item with entityType 'menu' (it moves to the Trash and can be restored).",
         sideEffects: "mutates-durable-state",
         authorization: { permission: "admin.menus.update" },
         inputSchema: {

package/dist/navigation/menu-service.d.ts

@@ -198,6 +198,18 @@ export interface UpdateMenuTreeRequired {
 export interface UpdateMenuTreeOptional {
     limits?: TreeValidationLimits | undefined;
 }
+/**
+ * Whole-tree saves arrive from callers that may not know an item's stored hint fields (an agent
+ * echoing a read through a schema, an older editor build). Dropping them on every save made an
+ * untouched menu look changed (dry run 2026-10-05: ~30 header page links lost `entryType: "page"`
+ * on one assistant save and the menu showed pending-to-live). So a hint is kept when the same item
+ * id still points at the same entry and the submitted target leaves that hint out; a re-pointed
+ * link or an explicitly submitted hint is taken as sent.
+ */
+export declare function carryForwardEntryHints({ previous, next }: {
+    previous: readonly NavItemNode[];
+    next: readonly NavItemNode[];
+}, _optional?: {}): NavItemNode[];
 /**
  * Replaces a menu's whole item tree, guarded by optimistic concurrency on the
  * entry `version` (matches the host's `updatePost` pattern): `expectedVersion` must equal the version

package/dist/navigation/menu-service.js

@@ -327,6 +327,44 @@ export async function createMenu(required, optional = {}) {
     }));
     return { menu };
 }
+/** The stored `entryRef` hint fields a whole-tree save carries forward when the submitted target omits them. */
+const ENTRY_HINT_KEYS = ["entryType", "lastKnownHref"];
+function entryTargetsById(nodes, into = new Map()) {
+    for (const node of nodes) {
+        if (node.target?.kind === "entryRef")
+            into.set(node.id, node.target);
+        if (node.children)
+            entryTargetsById(node.children, into);
+    }
+    return into;
+}
+function withStoredHints(target, stored) {
+    if (target.kind !== "entryRef" || !stored || stored.entryId !== target.entryId)
+        return target;
+    const merged = { ...target };
+    for (const key of ENTRY_HINT_KEYS) {
+        if (merged[key] === undefined && stored[key] !== undefined)
+            merged[key] = stored[key];
+    }
+    return merged;
+}
+/**
+ * Whole-tree saves arrive from callers that may not know an item's stored hint fields (an agent
+ * echoing a read through a schema, an older editor build). Dropping them on every save made an
+ * untouched menu look changed (dry run 2026-10-05: ~30 header page links lost `entryType: "page"`
+ * on one assistant save and the menu showed pending-to-live). So a hint is kept when the same item
+ * id still points at the same entry and the submitted target leaves that hint out; a re-pointed
+ * link or an explicitly submitted hint is taken as sent.
+ */
+export function carryForwardEntryHints({ previous, next }, _optional = {}) {
+    const stored = entryTargetsById(previous);
+    const walk = (nodes) => nodes.map((node) => ({
+        ...node,
+        target: withStoredHints(node.target, stored.get(node.id)),
+        children: node.children ? walk(node.children) : undefined,
+    }));
+    return walk(next);
+}
 /**
  * Replaces a menu's whole item tree, guarded by optimistic concurrency on the
  * entry `version` (matches the host's `updatePost` pattern): `expectedVersion` must equal the version
@@ -369,7 +407,10 @@ export async function updateMenuTree(required, optional = {}) {
             throw new MenuConflictError({ message: `slug '${slug}' already exists` });
         }
     }
-    const items = validateAndCloneTree({ items: input.items }, optional.limits);
+    const items = carryForwardEntryHints({
+        previous: existing.doc.items,
+        next: validateAndCloneTree({ items: input.items }, optional.limits),
+    });
     const menu = {
         ...existing,
         title,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jini-ai/cms",
-  "version": "0.5.0",
+  "version": "0.5.1",
   "description": "Content-model capability for Jini-hosted products: framework-free contracts, ports, and domain services in `/core` and per-domain subpaths; Node-bound adapters in domain-specific subpaths.",
   "license": "Apache-2.0",
   "repository": {