@outbuild-company/schedule-core 1.8.0 → 1.9.1

package/dist/index.js

@@ -399,16 +399,10 @@ var FIELD_REGISTRY = {
     valueKind: "reference",
     extract: (activity) => activity.calendarId === null ? null : String(activity.calendarId)
   },
-  // Normalized to string like calendarId. It is stored as a number, so it used
-  // to reach compareValues through the numeric branch while its sibling
-  // reference went through the string one — the same field kind comparing two
-  // different ways. localeCompare with numeric:true keeps the digit ordering.
   subcontractId: {
     valueKind: "reference",
     extract: (activity) => activity.subcontractId === null ? null : String(activity.subcontractId)
   },
-  // Declared boolean, not enum: it is one, and saying so is what lets the enum
-  // branch demand an order without inventing one for true/false.
   isCritical: {
     valueKind: "boolean",
     extract: (activity) => activity.isCritical
@@ -544,7 +538,6 @@ function evaluateVisibleIds(input) {
 function seedFilterProjection(input) {
   const { activities, parentOf, filter, context } = input;
   const matchedIds = /* @__PURE__ */ new Set();
-  const pinnedVisibleIds = /* @__PURE__ */ new Set();
   const visibleReferenceCounts = /* @__PURE__ */ new Map();
   const parentIds = /* @__PURE__ */ new Map();
   const remainingChildren = /* @__PURE__ */ new Map();
@@ -560,17 +553,11 @@ function seedFilterProjection(input) {
       );
     }
     const isMatch = matchesFilter(activity, filter, context);
-    const isPinned = input.pinnedVisibleIds?.has(activityId) === true;
-    visibleReferenceCounts.set(
-      activityId,
-      (isMatch ? 1 : 0) + (isPinned ? 1 : 0)
-    );
+    visibleReferenceCounts.set(activityId, isMatch ? 1 : 0);
     if (isMatch) matchedIds.add(activityId);
-    if (isPinned) pinnedVisibleIds.add(activityId);
   }
   return {
     matchedIds,
-    pinnedVisibleIds,
     visibleReferenceCounts,
     parentIds,
     remainingChildren
@@ -612,7 +599,6 @@ function buildFilterProjection(input) {
     filter: input.filter,
     context: input.context,
     matchedIds: seed2.matchedIds,
-    pinnedVisibleIds: seed2.pinnedVisibleIds,
     visibleIds,
     visibleReferenceCounts: seed2.visibleReferenceCounts
   };
@@ -670,8 +656,6 @@ var OPERATORS_BY_KIND = {
   date: /* @__PURE__ */ new Set(["after", "before"]),
   "id-array": /* @__PURE__ */ new Set(["someOf", "notSomeOf"]),
   enum: /* @__PURE__ */ new Set(["someOf", "notSomeOf"]),
-  // Same membership operators as enum: splitting the kinds was about ordering,
-  // not about filtering, and these two filter exactly like an enum.
   boolean: /* @__PURE__ */ new Set(["someOf", "notSomeOf"]),
   reference: /* @__PURE__ */ new Set(["someOf", "notSomeOf"])
 };
@@ -851,7 +835,7 @@ function getChildrenInVisualOrder(parentId, adapter) {
   children.sort(
     userOrder === null ? compareActivitiesInVisualOrder : (a, b) => userOrder(a, b) || compareActivitiesInVisualOrder(a, b)
   );
-  return children.map((a) => String(a.id));
+  return children.map((activity) => String(activity.id));
 }
 function iterateInVisualOrder(adapter) {
   const result = [];
@@ -972,107 +956,61 @@ function dispatchSortSet(action, deps) {
   return { ok: true, changes };
 }
 
-// src/columns/text/constants.ts
-var TEXT = "text";
-
-// src/columns/description/constants.ts
-var DESCRIPTION = "description";
-
-// src/columns/duration/constants.ts
-var DURATION = "duration";
-
-// src/columns/progress/constants.ts
-var PROGRESS = "progress";
-
-// src/columns/cost/constants.ts
-var COST = "cost";
-
-// src/columns/used-cost/constants.ts
-var USED_COST = "used_cost";
-
-// src/columns/hh-work-time/constants.ts
-var HH_WORK_TIME = "hhWorkTime";
-
-// src/columns/start-date/constants.ts
-var START_DATE = "start_date";
-
-// src/columns/end-date/constants.ts
-var END_DATE = "end_date";
-
-// src/columns/constraint-type/constants.ts
-var CONSTRAINT_TYPE = "constraint_type";
-
-// src/columns/constraint-date/constants.ts
-var CONSTRAINT_DATE = "constraint_date";
-
-// src/columns/calendar-id/constants.ts
-var CALENDAR_ID = "calendar_id";
-
-// src/columns/custom-id/constants.ts
-var CUSTOM_ID = "custom_id";
-
-// src/columns/subcontract-id/constants.ts
-var SUBCONTRACT_ID = "subcontractId";
-
-// src/columns/responsables/constants.ts
-var RESPONSABLES = "responsables";
-
-// src/columns/tags/constants.ts
-var TAGS = "tags";
-
-// src/columns/predecessors/constants.ts
-var CUSTOM_PREDECESSORS = "custom_predecessors";
-
-// src/columns/successors/constants.ts
-var CUSTOM_SUCESSORS = "custom_sucessors";
-
-// src/columns/real-work/constants.ts
-var REAL_WORK = "real_work";
-
-// src/columns/real-cost/constants.ts
-var REAL_COST = "real_cost";
-
-// src/columns/expected-progress/constants.ts
-var EXPECTED_PROGRESS = "expected_progress";
-
-// src/columns/expected-progress-base/constants.ts
-var EXPECTED_PROGRESS_BASE = "expected_progress_base";
-
-// src/columns/start-base/constants.ts
-var START_BASE = "start_base";
-
-// src/columns/end-base/constants.ts
-var END_BASE = "end_base";
-
-// src/columns/duration-base/constants.ts
-var DURATION_BASE = "duration_base";
-
-// src/columns/cost-base/constants.ts
-var COST_BASE = "cost_base";
-
-// src/columns/work-base/constants.ts
-var WORK_BASE = "work_base";
-
-// src/columns/calendar-duration/constants.ts
-var CALENDAR_DURATION = "calendarDuration";
-
-// src/columns/critical-path/constants.ts
-var IS_CRITICAL = "is_critical";
-var EARLY_START = "earlyStart";
-var EARLY_FINISH = "earlyFinish";
-var LATE_START = "lateStart";
-var LATE_FINISH = "lateFinish";
-var FREE_SLACK = "freeSlack";
-var TOTAL_SLACK = "totalSlack";
-
-// src/columns/correlative-id/constants.ts
-var CORRELATIVE_ID = "correlative_id";
+// src/constants/columns.ts
+var EDITABLE = {
+  TEXT: "text",
+  DESCRIPTION: "description",
+  DURATION: "duration",
+  PROGRESS: "progress",
+  COST: "cost",
+  USED_COST: "used_cost",
+  HH_WORK_TIME: "hhWorkTime",
+  START_DATE: "start_date",
+  END_DATE: "end_date",
+  CONSTRAINT_TYPE: "constraint_type",
+  CONSTRAINT_DATE: "constraint_date",
+  CALENDAR_ID: "calendar_id",
+  CUSTOM_ID: "custom_id",
+  SUBCONTRACT_ID: "subcontractId",
+  RESPONSABLES: "responsables",
+  TAGS: "tags",
+  CUSTOM_PREDECESSORS: "custom_predecessors",
+  CUSTOM_SUCESSORS: "custom_sucessors"
+};
+var READ_ONLY = {
+  REAL_WORK: "real_work",
+  REAL_COST: "real_cost",
+  EXPECTED_PROGRESS: "expected_progress",
+  EXPECTED_PROGRESS_BASE: "expected_progress_base",
+  START_BASE: "start_base",
+  END_BASE: "end_base",
+  DURATION_BASE: "duration_base",
+  COST_BASE: "cost_base",
+  WORK_BASE: "work_base",
+  CALENDAR_DURATION: "calendarDuration",
+  IS_CRITICAL: "is_critical",
+  EARLY_START: "earlyStart",
+  EARLY_FINISH: "earlyFinish",
+  LATE_START: "lateStart",
+  LATE_FINISH: "lateFinish",
+  FREE_SLACK: "freeSlack",
+  TOTAL_SLACK: "totalSlack",
+  CORRELATIVE_ID: "correlative_id",
+  UNIQUE_CORRELATIVE_ID: "unique_correlative_id",
+  PONDERATOR: "ponderator",
+  STATUS: "status"
+};
+var COLUMN = { ...EDITABLE, ...READ_ONLY };
 
-// src/columns/unique-correlative-id/constants.ts
-var UNIQUE_CORRELATIVE_ID = "unique_correlative_id";
+// src/shared/assert-never.ts
+function assertNever(value, message) {
+  throw new Error(message);
+}
 
-// src/columns/ponderator/constants.ts
-var PONDERATOR = "ponderator";
+// src/autoscheduler/utils/cancellation.ts
+function createIsCurrent(id, getCurrentId) {
+  return () => getCurrentId() === id;
+}
 
 // src/types.ts
 var WORK_TIME_DIRECTION = {
@@ -1091,71 +1029,8 @@ var STATUS = {
   DOING: "Doing"
 };
 
-// src/columns/status/constants.ts
-var STATUS_COLUMN = "status";
-var DEFAULT_STATUS_CRITERIA = "Baseline";
-function resolveStatusCriteria(value) {
-  return value === "Actual" ? "Actual" : DEFAULT_STATUS_CRITERIA;
-}
-
-// src/constants/columns.ts
-var EDITABLE = {
-  TEXT,
-  DESCRIPTION,
-  DURATION,
-  PROGRESS,
-  COST,
-  USED_COST,
-  HH_WORK_TIME,
-  START_DATE,
-  END_DATE,
-  CONSTRAINT_TYPE,
-  CONSTRAINT_DATE,
-  CALENDAR_ID,
-  CUSTOM_ID,
-  SUBCONTRACT_ID,
-  RESPONSABLES,
-  TAGS,
-  CUSTOM_PREDECESSORS,
-  CUSTOM_SUCESSORS
-};
-var READ_ONLY = {
-  REAL_WORK,
-  REAL_COST,
-  EXPECTED_PROGRESS,
-  EXPECTED_PROGRESS_BASE,
-  START_BASE,
-  END_BASE,
-  DURATION_BASE,
-  COST_BASE,
-  WORK_BASE,
-  CALENDAR_DURATION,
-  IS_CRITICAL,
-  EARLY_START,
-  EARLY_FINISH,
-  LATE_START,
-  LATE_FINISH,
-  FREE_SLACK,
-  TOTAL_SLACK,
-  CORRELATIVE_ID,
-  UNIQUE_CORRELATIVE_ID,
-  PONDERATOR,
-  STATUS: STATUS_COLUMN
-};
-var COLUMN = { ...EDITABLE, ...READ_ONLY };
-
-// src/shared/assert-never.ts
-function assertNever(value, message) {
-  throw new Error(message);
-}
-
-// src/autoscheduler/utils/cancellation.ts
-function createIsCurrent(id, getCurrentId) {
-  return () => getCurrentId() === id;
-}
-
 // src/constraints/index.ts
-var CONSTRAINT_TYPE2 = {
+var CONSTRAINT_TYPE = {
   ASAP: "asap",
   ALAP: "alap",
   SNET: "snet",
@@ -2072,7 +1947,7 @@ function processConstraints(activityIds, adapter, plans) {
     const activity = adapter.getActivity(id);
     if (!activity) continue;
     const type = activity.constraintType;
-    if (!type || type === CONSTRAINT_TYPE2.ASAP || type === CONSTRAINT_TYPE2.ALAP) {
+    if (!type || type === CONSTRAINT_TYPE.ASAP || type === CONSTRAINT_TYPE.ALAP) {
       continue;
     }
     const bounds = computeConstraintBounds(activity, adapter);
@@ -2121,14 +1996,14 @@ function validateMilestoneConstraint(activity, successorStart) {
   const { constraintType, constraintDate } = activity;
   if (!constraintType || !constraintDate) return successorStart;
   switch (constraintType) {
-    case CONSTRAINT_TYPE2.MSO:
-    case CONSTRAINT_TYPE2.MFO:
+    case CONSTRAINT_TYPE.MSO:
+    case CONSTRAINT_TYPE.MFO:
       return new Date(constraintDate);
-    case CONSTRAINT_TYPE2.SNET:
-    case CONSTRAINT_TYPE2.FNET:
+    case CONSTRAINT_TYPE.SNET:
+    case CONSTRAINT_TYPE.FNET:
       return constraintDate > successorStart ? new Date(constraintDate) : successorStart;
-    case CONSTRAINT_TYPE2.SNLT:
-    case CONSTRAINT_TYPE2.FNLT:
+    case CONSTRAINT_TYPE.SNLT:
+    case CONSTRAINT_TYPE.FNLT:
       return constraintDate < successorStart ? new Date(constraintDate) : successorStart;
     default:
       return successorStart;
@@ -2257,7 +2132,7 @@ async function asapPass(orderedIds, links, adapter, options, isCurrent, plans) {
       });
       plan.earliestSchedulingStart = linkStart;
       plan.earliestSchedulingEnd = plan.endDate;
-    } else if (activity.constraintType === CONSTRAINT_TYPE2.MSO) {
+    } else if (activity.constraintType === CONSTRAINT_TYPE.MSO) {
       if (plan.earliestStart) {
         plan.startDate = plan.earliestStart;
         plan.endDate = plan.earliestEnd ?? adapter.calculateEndDate({
@@ -2266,7 +2141,7 @@ async function asapPass(orderedIds, links, adapter, options, isCurrent, plans) {
           task: activity
         });
       }
-    } else if (activity.constraintType === CONSTRAINT_TYPE2.MFO) {
+    } else if (activity.constraintType === CONSTRAINT_TYPE.MFO) {
       if (plan.latestStart) {
         plan.startDate = plan.latestStart;
         plan.endDate = plan.latestEnd ?? adapter.calculateEndDate({
@@ -2292,7 +2167,7 @@ async function alapPass(reversedIds, links, asapPlans, adapter, _options, isCurr
   const activityCount = allActivities.length;
   const alapActivityIds = [];
   for (const activity of allActivities) {
-    if (activity.constraintType === CONSTRAINT_TYPE2.ALAP) {
+    if (activity.constraintType === CONSTRAINT_TYPE.ALAP) {
       alapActivityIds.push(activity.id);
     }
   }
@@ -2339,7 +2214,7 @@ async function alapPass(reversedIds, links, asapPlans, adapter, _options, isCurr
     sliceStart = yieldResult.newSliceStart;
     const activity = adapter.getActivity(activityId);
     if (!activity) continue;
-    if (activity.constraintType !== CONSTRAINT_TYPE2.ALAP) continue;
+    if (activity.constraintType !== CONSTRAINT_TYPE.ALAP) continue;
     if (!activity.autoScheduling) continue;
     if (activity.progress === COMPLETED_PROGRESS) continue;
     if (isSummary(activity.type, adapter.getChildren(activityId).length > 0))
@@ -2848,9 +2723,7 @@ function applyLinkCreates(links, deps) {
       break;
     }
     duplicateKeys.set(key, linkId);
-    if (existingGraphHasCycle && // ponytail: preserve legacy sequential semantics; optimize with dynamic
-    // SCC reachability only if corrupt cyclic schedules become measurable.
-    wouldIntroduceCycle(edges, {
+    if (existingGraphHasCycle && wouldIntroduceCycle(edges, {
       source: operation.source,
       target: operation.target
     })) {
@@ -3328,7 +3201,7 @@ function widenScopeToAlapActivities(group, allLinks, adapter) {
   const widenedAlapKeys = /* @__PURE__ */ new Set();
   for (const activity of adapter.getAllActivities()) {
     const activityKey = String(activity.id);
-    const isOutOfComponentAlap = activity.constraintType === CONSTRAINT_TYPE2.ALAP && !idSet.has(activityKey);
+    const isOutOfComponentAlap = activity.constraintType === CONSTRAINT_TYPE.ALAP && !idSet.has(activityKey);
     if (!isOutOfComponentAlap) continue;
     widenedAlapKeys.add(activityKey);
     idSet.add(activityKey);
@@ -3690,7 +3563,7 @@ var descriptionPipeline = {
 
 // src/columns/shared/activity-predicates.ts
 function effectiveConstraintType(activity) {
-  return activity.constraintType ?? CONSTRAINT_TYPE2.ASAP;
+  return activity.constraintType ?? CONSTRAINT_TYPE.ASAP;
 }
 var PROGRESS_COMPLETE = COMPLETED_PROGRESS;
 function isCompletedActivity(activity) {
@@ -4084,8 +3957,8 @@ var costPipeline = {
     }
     return canEdit();
   },
-  parseInput(rawValue, activity) {
-    if (String(rawValue).trim() === "") return parsed(activity.cost);
+  parseInput(rawValue) {
+    if (String(rawValue).trim() === "") return parsed(0);
     const numericValue = parseFloat(String(rawValue));
     if (Number.isNaN(numericValue)) return parseError("not_a_number");
     return parsed(numericValue);
@@ -4134,8 +4007,8 @@ var usedCostPipeline = {
     }
     return canEdit();
   },
-  parseInput(rawValue, activity) {
-    if (String(rawValue).trim() === "") return parsed(activity.usedCost);
+  parseInput(rawValue) {
+    if (String(rawValue).trim() === "") return parsed(0);
     const numericValue = parseFloat(String(rawValue));
     if (Number.isNaN(numericValue)) return parseError("not_a_number");
     return parsed(numericValue);
@@ -4428,7 +4301,7 @@ function checkIsWorkingDay(newDate, activity, ctx) {
   return null;
 }
 function detectEndDateConstraintWarning(activity, newEndDate) {
-  const cannotViolateStartPin = effectiveConstraintType(activity) === CONSTRAINT_TYPE2.MSO;
+  const cannotViolateStartPin = effectiveConstraintType(activity) === CONSTRAINT_TYPE.MSO;
   if (cannotViolateStartPin) return void 0;
   const projectedDate = pickProjectedDateForConstraint(
     effectiveConstraintType(activity),
@@ -4478,7 +4351,7 @@ function calculateNewDuration(activity, newEndDate, ctx) {
 
 // src/columns/constraint-type/constraint-type-pipeline.ts
 var VALID_CONSTRAINT_TYPES = new Set(
-  Object.values(CONSTRAINT_TYPE2)
+  Object.values(CONSTRAINT_TYPE)
 );
 var constraintTypePipeline = {
   targetField: "constraintType",
@@ -5259,6 +5132,12 @@ function runExpectedProgressBase(state, now, defaultBaseCalendarId = null) {
   return computeExpectedProgress(rootIds, now, adapter, defaultBaseCalendarId);
 }
 
+// src/columns/status/constants.ts
+var DEFAULT_STATUS_CRITERIA = "Baseline";
+function resolveStatusCriteria(value) {
+  return value === "Actual" ? "Actual" : DEFAULT_STATUS_CRITERIA;
+}
+
 // src/columns/status/compute.ts
 var roundToTwoDecimals = (value) => Math.round(value * 100) / 100;
 function resolveExpectedForStatus(activity, criteria) {
@@ -5286,7 +5165,7 @@ function applyStatusPass(state, criteria) {
       resolveExpectedForStatus(activity, criteria)
     );
     if (activity.status !== status) {
-      setActivityFieldDynamic(state, id, STATUS_COLUMN, status);
+      setActivityFieldDynamic(state, id, COLUMN.STATUS, status);
       changed.push(id);
     }
   });
@@ -9059,10 +8938,10 @@ var FreeFloat = class {
     this.durationCache = /* @__PURE__ */ new Map();
   }
   getFreeFloat() {
-    const PROGRESS2 = Number(this.activity.progress);
+    const PROGRESS = Number(this.activity.progress);
     const CONSTRAINT = this.getConstraintType();
     this.calculateFreeFloatOfLink(CONSTRAINT);
-    const activityFreeFloat = PROGRESS2 === 100 ? 0 : this.getFreeFloatBaseConstraint(CONSTRAINT);
+    const activityFreeFloat = PROGRESS === 100 ? 0 : this.getFreeFloatBaseConstraint(CONSTRAINT);
     return {
       activityFreeFloat,
       linksFreeFloat: this.linksFreeFloatObject
@@ -9221,11 +9100,11 @@ var FreeFloat = class {
     };
   }
   calculateMsoMfo(constraint) {
-    const CONSTRAINT_DATE2 = this.activity.constraint_date || /* @__PURE__ */ new Date();
+    const CONSTRAINT_DATE = this.activity.constraint_date || /* @__PURE__ */ new Date();
     const calculation = this.forwardMap.get(this.activity.id);
     const DATE_TO_CALCULATE = constraint === CONSTRAINT_TYPES.MSO ? calculation?.es : calculation?.ef;
     const freeSlack = this.calculationWithCalendar(
-      CONSTRAINT_DATE2,
+      CONSTRAINT_DATE,
       DATE_TO_CALCULATE
     );
     return Math.max(0, this.parseHoursToDays(freeSlack));
@@ -11288,7 +11167,7 @@ function buildParentMutations(input) {
   if (wasPromotableLeaf) {
     fields.type = PROMOTION_TARGET_TYPE;
     if ((input.promotionSource ?? "create") === "create") {
-      fields.constraintType = CONSTRAINT_TYPE2.ASAP;
+      fields.constraintType = CONSTRAINT_TYPE.ASAP;
       fields.constraintDate = null;
       fields.progress = 0;
     }
@@ -12429,8 +12308,6 @@ function dispatchSirSync(action, deps) {
     id,
     kind: "updated",
     fields: { pendingRequestIds: { before, after } },
-    // ISSUE-025: `after` must be the COMPLETE post-change entity; a partial
-    // object crashes any consumer that projects the full snapshot.
     after: structuredCloneActivity(updated)
   };
   return {
@@ -17909,16 +17786,6 @@ var ScheduleState = class _ScheduleState {
     this._viewState.setActiveOrder(nextOrder);
     this.invalidateDerivedOrder();
   }
-  /**
-   * Rebuild-on-demand, memoized: at most one build per invalidation, never one
-   * per comparison. getChildrenInVisualOrder asks for this once and then sorts a
-   * whole sibling group with it, and collectBranchOrder walks every branch, so a
-   * build per read would be thousands per dispatch.
-   *
-   * The factory reads hoursPerDay and the collation locale at build time, so a
-   * rebuilt comparator always reflects the current context — that is what keeps
-   * ordering and filtering from seeing two different hoursPerDay.
-   */
   getOrderComparator() {
     const activeOrder = this._viewState.getActiveOrder();
     if (activeOrder === null) {
@@ -17929,12 +17796,6 @@ var ScheduleState = class _ScheduleState {
     this._cache.orderComparator = this._buildOrderComparator(activeOrder);
     return this._cache.orderComparator;
   }
-  /**
-   * Both order caches, always together. The comparator decides the sequence, so
-   * a rebuilt comparator with a surviving sequence would paint the old order.
-   * invalidateVisualOrderIds stays separate on purpose: a structural change
-   * moves rows without touching the rules, so the comparator is still valid.
-   */
   invalidateDerivedOrder() {
     this._cache.orderComparator = null;
     this._cache.visualOrderIds = null;
@@ -18117,8 +17978,8 @@ var ScheduleState = class _ScheduleState {
     const key = String(activityId);
     if (this._viewState.isChecked(key) || !this._viewState.isVisible(key)) {
       this._captureViewStateOnce();
-      this._viewState.forget(key);
     }
+    this._viewState.forget(key);
     this._activities.delete(activityId);
     if (this._lastStartDate.has(String(activityId))) {
       this._captureLastStartDateOnce();
@@ -21043,29 +20904,6 @@ var ScheduleCore = class _ScheduleCore {
   _undo = new UndoRecorder();
   _opQueue = Promise.resolve();
   _scheduleRevision = 0;
-  /**
-   * Revision counting mutations that have been ADMITTED, whether or not they
-   * have changed anything yet. `_scheduleRevision` counts the ones that
-   * actually did.
-   *
-   * INVARIANT: `_pendingScheduleRevision >= _scheduleRevision`, and equality
-   * means no mutation is in flight.
-   *
-   * The two exist separately because the useful instant and the knowable
-   * instant are not the same one. A critical-path calculation becomes garbage
-   * the moment the next mutation is admitted, but whether that mutation is
-   * substantive is only knowable once its ChangeSet exists, which is hundreds
-   * of milliseconds later at project scale. Measured live on a 12268-activity
-   * project: the cancellation flag flipped 929 ms into a 930 ms calculation,
-   * 812 into 812 and 785 into 785, because the job's apply block queues behind
-   * the very dispatch whose completion cancels it. Cancellation and job end
-   * were the same event, so every discarded job ran the whole calculation.
-   *
-   * Splitting the counter buys the early signal without moving the meaning of
-   * `_scheduleRevision`, whose three readers (`isCriticalPathSettled`,
-   * `createDateGesturePreview` and the job's own `isCurrent`) are all written
-   * against "the state actually changed".
-   */
   _pendingScheduleRevision = 0;
   _criticalPathRevision = -1;
   _activeCriticalPath = null;
@@ -21178,13 +21016,6 @@ var ScheduleCore = class _ScheduleCore {
     this.assertReady();
     return readAllActivities(this.coreRuntime.state);
   }
-  /**
-   * Read del plan de indent (ISSUE-080): los padres destino DISTINTOS bajo los
-   * que quedarían las actividades si se indentaran ahora. Es exactamente la
-   * planificación que ejecuta el handler (`planIndentMoves`, autoridad única),
-   * expuesta para que la frontera pueda correr el protocolo de conversión a
-   * madre (chequeo backend + modal) ANTES de despachar. Solo lecturas.
-   */
   getIndentTargetParentIds(activityIds) {
     this.assertReady();
     const plan = planIndentMoves(
@@ -21337,7 +21168,6 @@ var ScheduleCore = class _ScheduleCore {
     this.assertReady();
     return this._saveTracker.hasUnsavedChanges();
   }
-  /** Read-only revision; moves once per substantive mutation. */
   getScheduleRevision() {
     return this._scheduleRevision;
   }
@@ -21383,24 +21213,10 @@ var ScheduleCore = class _ScheduleCore {
     );
     return workPromise;
   }
-  /**
-   * Marks a mutation as admitted and invalidates the calculation in flight.
-   *
-   * Called BEFORE `_enqueue`, which is the point of the whole thing: the wait
-   * in the operation queue is part of the window a running calculation wastes,
-   * and it is the longest part of it when several mutations are already
-   * queued.
-   *
-   * `dispatchChangesSchedulingState` is a pure function of the action, so this
-   * decision needs no state and cannot be wrong about the action's nature. What
-   * it cannot know yet is whether the mutation will produce anything, which is
-   * what `_settleScheduleMutation` reconciles afterwards.
-   */
   _beginScheduleMutation(action) {
     if (!dispatchChangesSchedulingState(action)) return false;
     return this._admitScheduleMutation();
   }
-  /** Undo and redo have no action to classify: reaching them IS the mutation. */
   _admitScheduleMutation() {
     this._pendingScheduleRevision++;
     if (this._activeCriticalPath) {
@@ -21408,30 +21224,6 @@ var ScheduleCore = class _ScheduleCore {
     }
     return true;
   }
-  /**
-   * Closes a mutation admitted by `_beginScheduleMutation`, on EVERY exit path.
-   *
-   * `changedState` false means the mutation was admitted and produced nothing
-   * (rejected, non-substantive, or rolled back). The pending revision walks
-   * back so the invariant holds and `isCriticalPathSettled` keeps telling the
-   * truth, and the calculation this mutation killed for nothing is re-armed.
-   *
-   * The re-arm goes through the queue and that is not incidental. Started
-   * outside it, the calculation would run immediately while the next queued
-   * mutation is still waiting, and that mutation would kill it again on
-   * admission: a cancel-and-restart treadmill burning a fresh project-wide deep
-   * copy per lap. Inside the queue the restart cannot happen until the queue
-   * drains, so at most one calculation is armed per settled mutation. The burst
-   * scenario will NOT catch a regression here, because there every dispatch
-   * succeeds and this path is never taken.
-   *
-   * It re-arms through `recomputeCriticalPath` rather than enqueueing the job
-   * directly, because the queue slot must NOT await the job: the job's own
-   * apply block needs a later slot on this same queue, so awaiting it from
-   * inside a slot deadlocks the core. `recomputeCriticalPath` already has the
-   * shape that returns the job's promise out of the slot instead of awaiting
-   * it, and it keeps `_criticalPathReady` pointing at the live calculation.
-   */
   _settleScheduleMutation(admitted, changedState) {
     if (!admitted) return;
     if (changedState) return;
@@ -21577,11 +21369,7 @@ var ScheduleCore = class _ScheduleCore {
     return operation;
   }
   isCriticalPathSettled() {
-    return this._criticalPathRevision === this._scheduleRevision && // An admitted mutation has not landed yet, so whatever is applied now is
-    // about to be stale. Without this term `whenCriticalPathSettled` would
-    // spin: it would keep arming calculations that `isCurrent` kills on their
-    // first check, each one paying the project-wide deep copy first.
-    this._pendingScheduleRevision === this._scheduleRevision && this._activeCriticalPath === null;
+    return this._criticalPathRevision === this._scheduleRevision && this._pendingScheduleRevision === this._scheduleRevision && this._activeCriticalPath === null;
   }
   async whenCriticalPathSettled() {
     while (!this.isCriticalPathSettled()) {
@@ -21597,21 +21385,6 @@ var ScheduleCore = class _ScheduleCore {
     const merged = this._reapplyViewState(result.changes);
     return merged === null ? result : { ...result, changes: merged };
   }
-  /**
-   * The single place the two view-state passes are chained. Dispatch, undo and
-   * redo all route through here: when this existed only inside the dispatch
-   * path, undo and redo reapplied the filter and forgot the order, so an edit
-   * that moved a row and was then undone left the row in its new position.
-   *
-   * The sequence between the passes is indifferent. Filter and order are
-   * orthogonal projections over the same tree — the filter decides which rows
-   * exist on screen and never reads the order; the order sequences every
-   * sibling group from the hierarchy index and never reads visibility. Either
-   * sequence produces the same ChangeSet.
-   *
-   * Both passes answer null while their state is inactive, so an unfiltered,
-   * unsorted schedule pays two property reads.
-   */
   _reapplyViewState(changes) {
     const withFilter = this._reapplyActiveFilter(changes);
     const sequenced = this._emitTouchedBranchOrder(withFilter ?? changes);
@@ -21619,58 +21392,8 @@ var ScheduleCore = class _ScheduleCore {
       sequenced.changes ?? withFilter ?? changes,
       sequenced.touchedParents
     );
-    const reapplied = withOrder ?? sequenced.changes ?? withFilter;
-    const finalChanges = reapplied ?? changes;
-    const createdIds = finalChanges.activities.filter((change) => change.kind === "created").map((change) => String(change.id));
-    if (createdIds.length === 0) return reapplied;
-    const createdAndAncestors = new Set(createdIds);
-    const filterProjection = this.coreRuntime.state.getFilterProjection();
-    for (const createdId of createdIds) {
-      if (filterProjection !== null) {
-        filterProjection.pinnedVisibleIds.add(createdId);
-      }
-      let activityId = createdId;
-      while (activityId !== null && activityId !== ROOT_PARENT_ID) {
-        createdAndAncestors.add(activityId);
-        if (filterProjection !== null) {
-          filterProjection.visibleIds.add(activityId);
-          filterProjection.visibleReferenceCounts.set(
-            activityId,
-            (filterProjection.visibleReferenceCounts.get(activityId) ?? 0) + 1
-          );
-        }
-        activityId = this.coreRuntime.state.getParentId(activityId);
-      }
-    }
-    const createdVisibility = applyVisibleSet(
-      this.coreRuntime.state,
-      createdAndAncestors,
-      createdAndAncestors
-    );
-    if (createdVisibility.length === 0) return reapplied;
-    return {
-      ...finalChanges,
-      viewState: [...finalChanges.viewState ?? [], ...createdVisibility]
-    };
+    return withOrder ?? sequenced.changes ?? withFilter;
   }
-  /**
-   * Emits `order` for the branches this mutation resequenced, when no user order
-   * is active.
-   *
-   * The contract is that `order` reports a CHANGED SEQUENCE, not the presence of
-   * a sort. Tying emission to the cause instead of the effect is what produced
-   * the undo bug and, later, the reparent ones: without an active order a move,
-   * an indent, an outdent or the undo of any of them rearranged rows and said
-   * nothing, so an incremental consumer kept the old sequence. The undo of a
-   * reparent was the worst of them — it carried neither `order` nor a single
-   * correlativeId, so the position was not recoverable by any consumer.
-   *
-   * The touched branches are derived from the ChangeSet rather than accumulated
-   * in the state: an entity whose parentId or correlativeId moved, plus the
-   * parents of created and deleted rows, is exactly the set of branches whose
-   * sequence can differ. That keeps this linear in the blast radius, adds
-   * nothing to the write path, and cannot leak across dispatches.
-   */
   _emitTouchedBranchOrder(changes) {
     const adapter = this.coreRuntime.state;
     const touchedParents = collectResequencedParents(
@@ -21687,15 +21410,6 @@ var ScheduleCore = class _ScheduleCore {
     if (order.length === 0) return { changes: null, touchedParents };
     return { changes: { ...changes, order }, touchedParents };
   }
-  /**
-   * Re-sequences the grid after any mutation that could have changed a value the
-   * active order sorts by. Ordering is a view over the data, so an edit that
-   * moves a row past its sibling must move the row, exactly as the filter makes
-   * a no-longer-matching row disappear.
-   *
-   * Production only re-sorts after a bar drag; diverging from that is a
-   * deliberate product decision, not an oversight.
-   */
   _reapplyActiveOrder(changes, resequencedParents) {
     const adapter = this.coreRuntime.state;
     if (adapter.getActiveOrder() === null) return null;
@@ -21738,8 +21452,7 @@ var ScheduleCore = class _ScheduleCore {
         activities: adapter.getAllActivities(),
         parentOf: (activityId) => adapter.getParentId(activityId),
         filter,
-        context,
-        ...projectionIsCurrent ? { pinnedVisibleIds: projection.pinnedVisibleIds } : {}
+        context
       });
       adapter.setFilterProjection(rebuiltProjection);
       visibleIds = rebuiltProjection.visibleIds;
@@ -21795,11 +21508,7 @@ var ScheduleCore = class _ScheduleCore {
         this.coreRuntime.baseCalendars
       )
     );
-    const isCurrent = () => !cancellation.cancelled && this._status !== SCHEDULE_CORE_STATUS.DESTROYED && this._scheduleRevision === revision && // A mutation admitted but not yet settled is enough to make this result
-    // garbage. Without this term the calculation only learns it is obsolete
-    // when the mutation finishes, which measured live is the same instant the
-    // job itself ends.
-    this._pendingScheduleRevision === this._scheduleRevision;
+    const isCurrent = () => !cancellation.cancelled && this._status !== SCHEDULE_CORE_STATUS.DESTROYED && this._scheduleRevision === revision && this._pendingScheduleRevision === this._scheduleRevision;
     const promise = (async () => {
       const calculatedFields = await computeCriticalPathFields(
         isolatedState,
@@ -22149,6 +21858,6 @@ var CREATION_KIND = {
   ROOT: "root"
 };
 
-export { ACTIVITY_TYPE, CALENDAR_UNIT, COLUMN, CONSTRAINT_TYPE2 as CONSTRAINT_TYPE, CONSTRAINT_TYPE_TO_LABEL, CREATION_KIND, DEFAULT_HOURS_PER_DAY, DISPATCH_ACTION_KIND, DISPATCH_ACTION_KINDS, DISPATCH_TRACK_EVENT, LINK_CODE_TO_TYPE, LINK_TYPE, LINK_TYPE_CODE, NEW_ACTIVITY_DEFAULTS, REJECTION_REASON, ROOT_PARENT_ID, SCHEDULE_CORE_STATUS, STATUS, ScheduleCore, WORK_TIME_DIRECTION, checkNoUpdatedLinks, computeExpectedProgress, expectedProgressFromBaseline, getActiveBaseline, getUnsavedActivities, isRootParent, normalizeParentKey, parseFromBackend, willRunCriticalPath, yieldToBrowser };
+export { ACTIVITY_TYPE, CALENDAR_UNIT, COLUMN, CONSTRAINT_TYPE, CONSTRAINT_TYPE_TO_LABEL, CREATION_KIND, DEFAULT_HOURS_PER_DAY, DISPATCH_ACTION_KIND, DISPATCH_ACTION_KINDS, DISPATCH_TRACK_EVENT, LINK_CODE_TO_TYPE, LINK_TYPE, LINK_TYPE_CODE, NEW_ACTIVITY_DEFAULTS, REJECTION_REASON, ROOT_PARENT_ID, SCHEDULE_CORE_STATUS, STATUS, ScheduleCore, WORK_TIME_DIRECTION, checkNoUpdatedLinks, computeExpectedProgress, expectedProgressFromBaseline, getActiveBaseline, getUnsavedActivities, isRootParent, normalizeParentKey, parseFromBackend, willRunCriticalPath, yieldToBrowser };
 //# sourceMappingURL=index.js.map
 //# sourceMappingURL=index.js.map