@jini-ai/cms 0.4.1 → 0.5.0

package/dist/taxonomy/write-service.js

@@ -408,7 +408,7 @@ export async function assignTerms(required, _optional = {}) {
  * authorize -> validateAssignmentTarget -> write -> stamp -> outbox chokepoint as `assignTerms`,
  * and — like `assignTerms` — never produces a `taxonomy_revisions` row ( disclosed
  * narrowing applies equally to removal). Idempotent: unassigning a row that isn't there is a no-op,
- * not an error (`remove`'s 0-vs-1 return is intentionally not surfaced to the caller). Chosen over
+ * not an error. Only removed ids are returned, stamped, and emitted; an entirely absent set is a no-op. Chosen over
  * a "set terms" replace call — see the taxonomy plan's "What the docs settle" #3 for why replace
  * risks silently erasing a concurrent assignment.
  * See docs/decisions/DR-005-ordered-taxonomy-validation.md.
@@ -417,19 +417,24 @@ export async function unassignTerms(required, _optional = {}) {
     const { deps, principalId, contentType, contentId, termIds } = required;
     await authorizeTaxonomyManage(deps, principalId);
     await validateAssignmentTarget(deps, contentType, contentId, termIds);
+    const removedTermIds = [];
     for (const termId of termIds) {
-        await deps.entryTerms.remove({ contentType, contentId, termId });
+        if (await deps.entryTerms.remove({ contentType, contentId, termId }) > 0)
+            removedTermIds.push(termId);
     }
+    if (removedTermIds.length === 0)
+        return { removedTermIds };
     const now = kernelNowIso({ clock: deps.clock });
     await deps.stampWatermark({});
     await deps.outbox.enqueue({ event: {
             name: "taxonomy.terms_unassigned",
             contentType,
             contentId,
-            termIds,
+            termIds: removedTermIds,
             actorId: principalId,
             occurredAt: now,
         } });
+    return { removedTermIds };
 }
 /** Content-deletion event subscriber. Best-effort (a missed event leaves an orphaned
  * `entry_terms` row that is inert on read; a periodic/boot reconciliation sweep — modeled by

package/dist/trash/write-service.js

@@ -95,12 +95,16 @@ export function createTrashService(deps, optional = {}) {
                         priorMarker: marker.priorMarker ?? trashOptional.priorMarker ?? null,
                     };
                     await deps.repo.insert({ row });
-                    await notifyChanged(optional, {
-                        workspaceId: required.workspaceId,
-                        entityType: required.entityType,
-                        entityId: required.entityId,
-                        change: "trash",
-                    });
+                    // A successful noop may repair a missing index, but did not change the domain marker.
+                    // Repeated removal must not emit another recovery watermark or outbox event (F1487).
+                    if (!marker.noop) {
+                        await notifyChanged(optional, {
+                            workspaceId: required.workspaceId,
+                            entityType: required.entityType,
+                            entityId: required.entityId,
+                            change: "trash",
+                        });
+                    }
                     return marker;
                 }
             });
@@ -135,12 +139,15 @@ export function createTrashService(deps, optional = {}) {
                     if (!marker.ok)
                         return marker.reason === "not-found" ? "not-found" : "version-changed";
                     await deps.repo.deleteById({ workspaceId: required.workspaceId, id: row.id });
-                    await notifyChanged(optional, {
-                        workspaceId: required.workspaceId,
-                        entityType: required.entityType,
-                        entityId: required.entityId,
-                        change: "restore",
-                    });
+                    // An already-clear marker still needs index cleanup, without a duplicate notification.
+                    if (!marker.noop) {
+                        await notifyChanged(optional, {
+                            workspaceId: required.workspaceId,
+                            entityType: required.entityType,
+                            entityId: required.entityId,
+                            change: "restore",
+                        });
+                    }
                     return "restored";
                 }
             });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jini-ai/cms",
-  "version": "0.4.1",
+  "version": "0.5.0",
   "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": {
@@ -132,7 +132,7 @@
     "@jini-ai/http-kit": "0.4.0"
   },
   "dependencies": {
-    "@jini-ai/core": "0.4.0"
+    "@jini-ai/core": "^0.4.1"
   },
   "peerDependencies": {
     "sharp": "^0.35.3",