@outbuild-company/schedule-core 1.16.0 → 1.16.1

package/dist/index.cjs

@@ -11588,18 +11588,19 @@ var AUTOSCHEDULE_SCOPE = { ROOTS: "roots" };
 async function runPostMutation(deps, args) {
   recomputePromotedSummaryPonderators(deps);
   const progressInvalidation = resolveProgressInvalidation(deps.adapter);
-  const passes = resolvePasses(args, progressInvalidation);
+  const effectiveArgs = resolveTransactionAutoscheduleArgs(deps.adapter, args);
+  const passes = resolvePasses(effectiveArgs, progressInvalidation);
   if (passes.has(DERIVED_PASS.AUTOSCHEDULE)) {
-    await refreshMutatedParentBounds(deps.adapter, args);
+    await refreshMutatedParentBounds(deps.adapter, effectiveArgs);
   }
   const schedule = passes.has(DERIVED_PASS.AUTOSCHEDULE) ? await runAutoschedule(
     deps,
-    args.autoscheduleFrom,
-    args.preventSummaryTargetPullback
+    effectiveArgs.autoscheduleFrom,
+    effectiveArgs.preventSummaryTargetPullback
   ) : emptySchedule();
   await rollUpParentBounds(
     deps.adapter,
-    args,
+    effectiveArgs,
     schedule.scheduledIds,
     passes,
     progressInvalidation
@@ -11607,6 +11608,49 @@ async function runPostMutation(deps, args) {
   await runDerivedRecomputes(deps, passes, args.now);
   return schedule;
 }
+function resolveTransactionAutoscheduleArgs(adapter, args) {
+  const autoscheduleFrom = resolveTransactionAutoscheduleFrom(
+    adapter,
+    args.autoscheduleFrom
+  );
+  return autoscheduleFrom === args.autoscheduleFrom ? args : { ...args, autoscheduleFrom };
+}
+function resolveTransactionAutoscheduleFrom(adapter, requested) {
+  if (requested === AUTOSCHEDULE_SCOPE.ROOTS) {
+    return AUTOSCHEDULE_SCOPE.ROOTS;
+  }
+  const targets = collectLinkMutationTargets(adapter);
+  if (targets.size === 0) return requested;
+  if (requested !== null) targets.add(String(requested));
+  if (targets.size !== 1) return AUTOSCHEDULE_SCOPE.ROOTS;
+  const [target] = targets;
+  if (target === void 0 || !adapter.activityExists(target) || adapter.hasChildren(target)) {
+    return AUTOSCHEDULE_SCOPE.ROOTS;
+  }
+  return target;
+}
+function collectLinkMutationTargets(adapter) {
+  const targets = /* @__PURE__ */ new Set();
+  const journal = adapter.peekWriteCapture();
+  if (!journal) return targets;
+  for (const operation of journal.ops) {
+    if (operation.kind === WRITE_OPERATION_KIND.LINK_REMOVE) {
+      targets.add(String(operation.before.target));
+      continue;
+    }
+    if (operation.kind === WRITE_OPERATION_KIND.LINK_ADD) {
+      const link = adapter.getLink(operation.linkId);
+      if (link) targets.add(String(link.target));
+      continue;
+    }
+    if (operation.kind === WRITE_OPERATION_KIND.LINK_FIELD) {
+      targets.add(String(operation.before.target));
+      const link = adapter.getLink(operation.linkId);
+      if (link) targets.add(String(link.target));
+    }
+  }
+  return targets;
+}
 function recomputePromotedSummaryPonderators(deps) {
   const journal = deps.adapter.peekWriteCapture();
   if (!journal) return;
@@ -14752,12 +14796,20 @@ function collectInvalidLinkIdsForCycle(movedActivityIds, adapter) {
     const src = String(link.source);
     const tgt = String(link.target);
     if (!moved.has(src) && !moved.has(tgt)) continue;
-    if (isAncestor(tgt, src, adapter) || isAncestor(src, tgt, adapter)) {
+    if (isHierarchyInvalidLink(link, adapter)) {
       out.push(String(link.id));
     }
   }
   return out;
 }
+function isHierarchyInvalidLink(link, adapter) {
+  const source = String(link.source);
+  const target = String(link.target);
+  return isAncestor(target, source, adapter) || isAncestor(source, target, adapter);
+}
+function collectInvalidLinkIdsFromSnapshots(links, adapter) {
+  return [...links].filter(([, link]) => isHierarchyInvalidLink(link, adapter)).map(([linkId]) => String(linkId));
+}
 function isAncestor(descendantId, candidateAncestorId, adapter) {
   let cur = String(descendantId);
   const guard = /* @__PURE__ */ new Set();
@@ -14981,6 +15033,10 @@ async function dispatchActivityMove(action, options, deps) {
     }
   );
   const invalidLinkIds = parentChanged ? collectInvalidLinkIdsForCycle([action.activityId], adapter) : [];
+  const promotionInvalidLinkIds = collectInvalidLinkIdsFromSnapshots(
+    beforeLinks,
+    adapter
+  );
   const trackingEvent = {
     name: DISPATCH_TRACK_EVENT.ACTIVITY_MOVE,
     properties: {
@@ -14989,7 +15045,7 @@ async function dispatchActivityMove(action, options, deps) {
       after_sibling_id: action.afterSiblingId ?? null,
       before_sibling_id: action.beforeSiblingId ?? null,
       invalid_link_ids: [
-        .../* @__PURE__ */ new Set([...invalidLinkIds.map(String), ...beforeLinks.keys()])
+        .../* @__PURE__ */ new Set([...promotionInvalidLinkIds, ...invalidLinkIds.map(String)])
       ]
     }
   };
@@ -15169,6 +15225,10 @@ async function dispatchActivityIndent(action, options, deps) {
     if (shift.before !== void 0) beforeImage.correlativeId = shift.before;
     beforeSnap.set(shift.activityId, beforeImage);
   }
+  const promotionInvalidLinkIds = collectInvalidLinkIdsFromSnapshots(
+    beforeLinks,
+    adapter
+  );
   const invalidLinkIds = collectInvalidLinkIdsForCycle(succeededIds, adapter);
   for (const linkId of invalidLinkIds) {
     const link = adapter.getLink(linkId);
@@ -15199,7 +15259,9 @@ async function dispatchActivityIndent(action, options, deps) {
         activity_id: f.activityId,
         reason: f.reason
       })),
-      invalid_link_ids: [...beforeLinks.keys()]
+      invalid_link_ids: [
+        .../* @__PURE__ */ new Set([...promotionInvalidLinkIds, ...invalidLinkIds.map(String)])
+      ]
     }
   };
   if (action.eventSource !== void 0) {