@superdoc/sdk 2.13.0-next.12 → 2.13.0-next.13

package/dist/agent/actions.cjs

@@ -641,8 +641,8 @@ function evaluateChecks(pre, post, checks) {
         else if (check.kind === 'tracked-change-count-delta') {
             results.push({
                 check,
-                passed: post.trackedChanges.length - pre.trackedChanges.length === check.delta,
-                detail: `pre=${pre.trackedChanges.length} post=${post.trackedChanges.length}`,
+                passed: post.counts.trackedChanges - pre.counts.trackedChanges === check.delta,
+                detail: `pre=${pre.counts.trackedChanges} post=${post.counts.trackedChanges}`,
             });
         }
         else if (check.kind === 'revision-changed') {
@@ -1544,11 +1544,25 @@ async function runDeleteBlocks(doc, args) {
         const changeMode = parseChangeMode(args.changeMode);
         const executedOperations = [];
         const deleted = [];
+        const addressedTrackedChangeIds = new Set();
+        let allTrackedEffectsAddressed = true;
         for (const target of targets) {
             try {
                 const result = await deleteFn({ target: { kind: 'block', nodeType: target.nodeType, nodeId: target.nodeId } }, changeMode ? { changeMode } : undefined);
                 executedOperations.push({ operationId: 'doc.blocks.delete', result: compactOpResult(result) });
                 deleted.push({ nodeId: target.nodeId, nodeType: target.nodeType });
+                if (changeMode === 'tracked') {
+                    // One block transaction can create multiple logical review entities
+                    // (for example, text plus its section break). Exact receipt refs let
+                    // us fail closed on missing entities. Older handles omit refs, and
+                    // adjacent deletions can coalesce into one review entity, so their
+                    // truthful fallback is the positive observed inventory delta rather
+                    // than one assumed entity per operation.
+                    const refIds = trackedChangeRefIds(result);
+                    allTrackedEffectsAddressed &&= refIds.size > 0;
+                    for (const entityId of refIds)
+                        addressedTrackedChangeIds.add(entityId);
+                }
             }
             catch (err) {
                 errors$1.push({
@@ -1562,14 +1576,37 @@ async function runDeleteBlocks(doc, args) {
         // Tracked deletions leave the block in place until the revision is decided,
         // so the block count cannot move — count the structural revisions instead.
         // Direct deletions must show one fewer block of each deleted node type.
+        const observedTrackedChangeDelta = post.counts.trackedChanges - pre.counts.trackedChanges;
+        // A later deletion can coalesce the batch and replace entity identities
+        // returned by earlier operation receipts. For a multi-operation batch the
+        // final inventory is therefore the authoritative count. A single operation
+        // keeps using its addressed refs so fan-out remains verifiable even when
+        // the inventory listing is capped.
+        const expectedTrackedChangeDelta = deleted.length > 1
+            ? observedTrackedChangeDelta
+            : allTrackedEffectsAddressed
+                ? addressedTrackedChangeIds.size
+                : Math.max(observedTrackedChangeDelta, addressedTrackedChangeIds.size);
         const checks = changeMode === 'tracked'
-            ? [{ kind: 'tracked-change-count-delta', delta: deleted.length }]
+            ? [
+                {
+                    kind: 'tracked-change-count-delta',
+                    delta: expectedTrackedChangeDelta,
+                },
+            ]
             : [...new Set(deleted.map((entry) => entry.nodeType))].map((nodeType) => ({
                 kind: 'block-count-delta',
                 nodeType,
                 delta: -deleted.filter((entry) => entry.nodeType === nodeType).length,
             }));
         const verification = evaluateChecks(pre, post, checks);
+        if (changeMode === 'tracked' && observedTrackedChangeDelta <= 0) {
+            verification[0] = {
+                ...verification[0],
+                passed: false,
+                detail: `pre=${pre.counts.trackedChanges} post=${post.counts.trackedChanges}; no tracked review entity was created`,
+            };
+        }
         // Compare against the DEDUPED target list: two selectors naming the same
         // block are one deletion, and counting the raw selectors would report a
         // fully applied call as `partial` with no error to explain it. Selectors
@@ -4513,6 +4550,15 @@ function compactOpResult(result) {
         compact.stepCount = result.steps.length;
     return compact;
 }
+function trackedChangeRefIds(result) {
+    const refs = asRecord(result)?.trackedChangeRefs;
+    if (!Array.isArray(refs))
+        return new Set();
+    return new Set(refs.flatMap((entry) => {
+        const entityId = asString(asRecord(entry)?.entityId);
+        return entityId ? [entityId] : [];
+    }));
+}
 function parseScopedReplaceArgs(args, changeMode) {
     if (args.action !== 'replace_text')
         return null;

package/dist/agent/actions.js

@@ -638,8 +638,8 @@ function evaluateChecks(pre, post, checks) {
         else if (check.kind === 'tracked-change-count-delta') {
             results.push({
                 check,
-                passed: post.trackedChanges.length - pre.trackedChanges.length === check.delta,
-                detail: `pre=${pre.trackedChanges.length} post=${post.trackedChanges.length}`,
+                passed: post.counts.trackedChanges - pre.counts.trackedChanges === check.delta,
+                detail: `pre=${pre.counts.trackedChanges} post=${post.counts.trackedChanges}`,
             });
         }
         else if (check.kind === 'revision-changed') {
@@ -1541,11 +1541,25 @@ async function runDeleteBlocks(doc, args) {
         const changeMode = parseChangeMode(args.changeMode);
         const executedOperations = [];
         const deleted = [];
+        const addressedTrackedChangeIds = new Set();
+        let allTrackedEffectsAddressed = true;
         for (const target of targets) {
             try {
                 const result = await deleteFn({ target: { kind: 'block', nodeType: target.nodeType, nodeId: target.nodeId } }, changeMode ? { changeMode } : undefined);
                 executedOperations.push({ operationId: 'doc.blocks.delete', result: compactOpResult(result) });
                 deleted.push({ nodeId: target.nodeId, nodeType: target.nodeType });
+                if (changeMode === 'tracked') {
+                    // One block transaction can create multiple logical review entities
+                    // (for example, text plus its section break). Exact receipt refs let
+                    // us fail closed on missing entities. Older handles omit refs, and
+                    // adjacent deletions can coalesce into one review entity, so their
+                    // truthful fallback is the positive observed inventory delta rather
+                    // than one assumed entity per operation.
+                    const refIds = trackedChangeRefIds(result);
+                    allTrackedEffectsAddressed &&= refIds.size > 0;
+                    for (const entityId of refIds)
+                        addressedTrackedChangeIds.add(entityId);
+                }
             }
             catch (err) {
                 errors.push({
@@ -1559,14 +1573,37 @@ async function runDeleteBlocks(doc, args) {
         // Tracked deletions leave the block in place until the revision is decided,
         // so the block count cannot move — count the structural revisions instead.
         // Direct deletions must show one fewer block of each deleted node type.
+        const observedTrackedChangeDelta = post.counts.trackedChanges - pre.counts.trackedChanges;
+        // A later deletion can coalesce the batch and replace entity identities
+        // returned by earlier operation receipts. For a multi-operation batch the
+        // final inventory is therefore the authoritative count. A single operation
+        // keeps using its addressed refs so fan-out remains verifiable even when
+        // the inventory listing is capped.
+        const expectedTrackedChangeDelta = deleted.length > 1
+            ? observedTrackedChangeDelta
+            : allTrackedEffectsAddressed
+                ? addressedTrackedChangeIds.size
+                : Math.max(observedTrackedChangeDelta, addressedTrackedChangeIds.size);
         const checks = changeMode === 'tracked'
-            ? [{ kind: 'tracked-change-count-delta', delta: deleted.length }]
+            ? [
+                {
+                    kind: 'tracked-change-count-delta',
+                    delta: expectedTrackedChangeDelta,
+                },
+            ]
             : [...new Set(deleted.map((entry) => entry.nodeType))].map((nodeType) => ({
                 kind: 'block-count-delta',
                 nodeType,
                 delta: -deleted.filter((entry) => entry.nodeType === nodeType).length,
             }));
         const verification = evaluateChecks(pre, post, checks);
+        if (changeMode === 'tracked' && observedTrackedChangeDelta <= 0) {
+            verification[0] = {
+                ...verification[0],
+                passed: false,
+                detail: `pre=${pre.counts.trackedChanges} post=${post.counts.trackedChanges}; no tracked review entity was created`,
+            };
+        }
         // Compare against the DEDUPED target list: two selectors naming the same
         // block are one deletion, and counting the raw selectors would report a
         // fully applied call as `partial` with no error to explain it. Selectors
@@ -4510,6 +4547,15 @@ function compactOpResult(result) {
         compact.stepCount = result.steps.length;
     return compact;
 }
+function trackedChangeRefIds(result) {
+    const refs = asRecord(result)?.trackedChangeRefs;
+    if (!Array.isArray(refs))
+        return new Set();
+    return new Set(refs.flatMap((entry) => {
+        const entityId = asString(asRecord(entry)?.entityId);
+        return entityId ? [entityId] : [];
+    }));
+}
 function parseScopedReplaceArgs(args, changeMode) {
     if (args.action !== 'replace_text')
         return null;