@outbuild-company/schedule-core 1.3.0 → 1.4.0

package/dist/index.d.cts

@@ -260,6 +260,34 @@ interface FilterState {
     readonly dateRange?: FilterDateRange;
 }
 
+interface OrderRule {
+    /**
+     * Deliberately FilterField, not an alias of it: the columns a user can sort by
+     * are the columns a user can filter by, and the filter's registry already
+     * resolves units, baseline snapshots and critical-path values for all of them.
+     * A second field vocabulary would drift from this one on the first new column.
+     */
+    readonly field: FilterField;
+    readonly direction: 'asc' | 'desc';
+}
+/**
+ * The active ordering, as an ordered list of keys: rule 0 decides, rule 1 breaks
+ * its ties, and so on. An empty list means "no user ordering", which is stored
+ * as null rather than an empty state (mirroring isEmptyFilter).
+ */
+interface OrderState {
+    readonly rules: ReadonlyArray<OrderRule>;
+}
+/**
+ * One resequenced sibling group. Ordering is hierarchical: an activity only ever
+ * competes with its own siblings, so a change is expressed per parent rather
+ * than as a flat sequence over the whole schedule.
+ */
+interface BranchOrderChange {
+    readonly parentId: string;
+    readonly childIds: ReadonlyArray<string>;
+}
+
 type LinksBatchOperation = {
     kind: 'create';
     source: ActivityId;
@@ -385,6 +413,10 @@ type DispatchAction = {
     logic: 'and' | 'or';
     dateRange?: FilterDateRange;
     eventSource?: string;
+} | {
+    kind: 'sort-set';
+    rules: ReadonlyArray<OrderRule>;
+    eventSource?: string;
 } | {
     kind: 'sir-sync';
     activityId: ActivityId;
@@ -500,6 +532,15 @@ interface ChangeSet {
     calendars: ReadonlyArray<EntityChange<Calendar>>;
     trackingEvents: ReadonlyArray<TrackingEvent>;
     viewState?: ReadonlyArray<ViewStateChange>;
+    /**
+     * Sibling groups whose display sequence changed, each already in final order.
+     * Carried per branch rather than as a flat sequence because ordering never
+     * reparents: the renderer assigns each array and repaints once.
+     *
+     * Absent when nothing resequenced. Never a partial answer: a branch listed
+     * here holds all of its children.
+     */
+    order?: ReadonlyArray<BranchOrderChange>;
     effects?: ReadonlyArray<ScheduleEffect>;
     warnings?: ReadonlyArray<ConstraintWarning>;
 }
@@ -655,6 +696,8 @@ declare class ScheduleCore {
     getChildrenIds(parentId: ActivityId | '0'): ReadonlyArray<ActivityId>;
     getSelectedActivityIds(): string[];
     getHiddenActivityIds(): string[];
+    getActiveFilter(): Readonly<FilterState> | null;
+    getActiveOrder(): Readonly<OrderState> | null;
     getVisualOrderIds(): ReadonlyArray<ActivityId>;
     hasChild(parentId: ActivityId | '0'): boolean;
     getCalendar(id: CalendarId): Readonly<Calendar> | null;
@@ -672,7 +715,52 @@ declare class ScheduleCore {
     } | null>;
     isCriticalPathSettled(): boolean;
     whenCriticalPathSettled(): Promise<void>;
-    private _withReappliedFilter;
+    private _withReappliedViewState;
+    /**
+     * 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.
+     */
+    private _reapplyViewState;
+    /**
+     * 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.
+     */
+    private _emitTouchedBranchOrder;
+    /**
+     * 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.
+     */
+    private _reapplyActiveOrder;
     private _reapplyActiveFilter;
     private _recordScheduleMutation;
     private _startCriticalPathForCurrentRevision;
@@ -791,6 +879,7 @@ declare const DISPATCH_ACTION_KIND: {
     readonly SELECTION_REPLACE: "selection-replace";
     readonly VISIBILITY_SET: "visibility-set";
     readonly FILTER_SET: "filter-set";
+    readonly SORT_SET: "sort-set";
     readonly SIR_SYNC: "sir-sync";
     readonly ACTIVITY_LOOKAHEAD_SYNC: "activity-lookahead-sync";
 };
@@ -836,4 +925,4 @@ declare const DISPATCH_TRACK_EVENT: {
 };
 type DispatchTrackEvent = (typeof DISPATCH_TRACK_EVENT)[keyof typeof DISPATCH_TRACK_EVENT];
 
-export { ACTIVITY_TYPE, type ActivityCreter, type ActivityId, type ActivityType, type BackendActivityInput, type BackendCalendarInput, type BackendLinkInput, type BackendScheduleInput, type BackendSectorInput, type BaselinePoint, type BaselineVersion, CALENDAR_UNIT, COLUMN, CONSTRAINT_TYPE, CONSTRAINT_TYPE_TO_LABEL, CREATION_KIND, type Calendar, type CalendarId, type CalendarUnit, type CalendarWorktime, type ChangeSet, type ColumnName, type ConstraintType, type ConstraintWarning, type CoreActivity, type CreationKind, DEFAULT_HOURS_PER_DAY, DISPATCH_ACTION_KIND, DISPATCH_ACTION_KINDS, DISPATCH_TRACK_EVENT, type DispatchAction, type DispatchActionKind, type DispatchOptions, type DispatchRejectReason, type DispatchResult, type DispatchTrackEvent, type EntityChange, type FilterCriterion, type FilterDateRange, type FilterField, type FilterState, LINK_CODE_TO_TYPE, LINK_TYPE, LINK_TYPE_CODE, type Link, type LinkId, type LinkType, type LinkTypeCode, type LinksBatchOperation, NEW_ACTIVITY_DEFAULTS, type ParsedInput, type PastedActivityInput, type PastedLinkInput, type PendingRequest, type PersistedEntityIdentity, REJECTION_REASON, ROOT_PARENT_ID, ScheduleCore, type ScheduleCoreInput, type ScheduleCoreReporter, type ScheduleCoreStatus, type ScheduleEffect, type SectorMetadata, type StatusCriteria, type TrackingEvent, WORK_TIME_DIRECTION, type WorkTimeDirection, checkNoUpdatedLinks, computeExpectedProgress, expectedProgressFromBaseline, getActiveBaseline, getUnsavedActivities, isRootParent, normalizeParentKey, parseFromBackend, willRunCriticalPath, yieldToBrowser };
+export { ACTIVITY_TYPE, type ActivityCreter, type ActivityId, type ActivityType, type BackendActivityInput, type BackendCalendarInput, type BackendLinkInput, type BackendScheduleInput, type BackendSectorInput, type BaselinePoint, type BaselineVersion, type BranchOrderChange, CALENDAR_UNIT, COLUMN, CONSTRAINT_TYPE, CONSTRAINT_TYPE_TO_LABEL, CREATION_KIND, type Calendar, type CalendarId, type CalendarUnit, type CalendarWorktime, type ChangeSet, type ColumnName, type ConstraintType, type ConstraintWarning, type CoreActivity, type CreationKind, DEFAULT_HOURS_PER_DAY, DISPATCH_ACTION_KIND, DISPATCH_ACTION_KINDS, DISPATCH_TRACK_EVENT, type DispatchAction, type DispatchActionKind, type DispatchOptions, type DispatchRejectReason, type DispatchResult, type DispatchTrackEvent, type EntityChange, type FilterCriterion, type FilterDateRange, type FilterField, type FilterState, LINK_CODE_TO_TYPE, LINK_TYPE, LINK_TYPE_CODE, type Link, type LinkId, type LinkType, type LinkTypeCode, type LinksBatchOperation, NEW_ACTIVITY_DEFAULTS, type OrderRule, type OrderState, type ParsedInput, type PastedActivityInput, type PastedLinkInput, type PendingRequest, type PersistedEntityIdentity, REJECTION_REASON, ROOT_PARENT_ID, ScheduleCore, type ScheduleCoreInput, type ScheduleCoreReporter, type ScheduleCoreStatus, type ScheduleEffect, type SectorMetadata, type StatusCriteria, type TrackingEvent, WORK_TIME_DIRECTION, type WorkTimeDirection, checkNoUpdatedLinks, computeExpectedProgress, expectedProgressFromBaseline, getActiveBaseline, getUnsavedActivities, isRootParent, normalizeParentKey, parseFromBackend, willRunCriticalPath, yieldToBrowser };

package/dist/index.d.ts

@@ -260,6 +260,34 @@ interface FilterState {
     readonly dateRange?: FilterDateRange;
 }
 
+interface OrderRule {
+    /**
+     * Deliberately FilterField, not an alias of it: the columns a user can sort by
+     * are the columns a user can filter by, and the filter's registry already
+     * resolves units, baseline snapshots and critical-path values for all of them.
+     * A second field vocabulary would drift from this one on the first new column.
+     */
+    readonly field: FilterField;
+    readonly direction: 'asc' | 'desc';
+}
+/**
+ * The active ordering, as an ordered list of keys: rule 0 decides, rule 1 breaks
+ * its ties, and so on. An empty list means "no user ordering", which is stored
+ * as null rather than an empty state (mirroring isEmptyFilter).
+ */
+interface OrderState {
+    readonly rules: ReadonlyArray<OrderRule>;
+}
+/**
+ * One resequenced sibling group. Ordering is hierarchical: an activity only ever
+ * competes with its own siblings, so a change is expressed per parent rather
+ * than as a flat sequence over the whole schedule.
+ */
+interface BranchOrderChange {
+    readonly parentId: string;
+    readonly childIds: ReadonlyArray<string>;
+}
+
 type LinksBatchOperation = {
     kind: 'create';
     source: ActivityId;
@@ -385,6 +413,10 @@ type DispatchAction = {
     logic: 'and' | 'or';
     dateRange?: FilterDateRange;
     eventSource?: string;
+} | {
+    kind: 'sort-set';
+    rules: ReadonlyArray<OrderRule>;
+    eventSource?: string;
 } | {
     kind: 'sir-sync';
     activityId: ActivityId;
@@ -500,6 +532,15 @@ interface ChangeSet {
     calendars: ReadonlyArray<EntityChange<Calendar>>;
     trackingEvents: ReadonlyArray<TrackingEvent>;
     viewState?: ReadonlyArray<ViewStateChange>;
+    /**
+     * Sibling groups whose display sequence changed, each already in final order.
+     * Carried per branch rather than as a flat sequence because ordering never
+     * reparents: the renderer assigns each array and repaints once.
+     *
+     * Absent when nothing resequenced. Never a partial answer: a branch listed
+     * here holds all of its children.
+     */
+    order?: ReadonlyArray<BranchOrderChange>;
     effects?: ReadonlyArray<ScheduleEffect>;
     warnings?: ReadonlyArray<ConstraintWarning>;
 }
@@ -655,6 +696,8 @@ declare class ScheduleCore {
     getChildrenIds(parentId: ActivityId | '0'): ReadonlyArray<ActivityId>;
     getSelectedActivityIds(): string[];
     getHiddenActivityIds(): string[];
+    getActiveFilter(): Readonly<FilterState> | null;
+    getActiveOrder(): Readonly<OrderState> | null;
     getVisualOrderIds(): ReadonlyArray<ActivityId>;
     hasChild(parentId: ActivityId | '0'): boolean;
     getCalendar(id: CalendarId): Readonly<Calendar> | null;
@@ -672,7 +715,52 @@ declare class ScheduleCore {
     } | null>;
     isCriticalPathSettled(): boolean;
     whenCriticalPathSettled(): Promise<void>;
-    private _withReappliedFilter;
+    private _withReappliedViewState;
+    /**
+     * 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.
+     */
+    private _reapplyViewState;
+    /**
+     * 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.
+     */
+    private _emitTouchedBranchOrder;
+    /**
+     * 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.
+     */
+    private _reapplyActiveOrder;
     private _reapplyActiveFilter;
     private _recordScheduleMutation;
     private _startCriticalPathForCurrentRevision;
@@ -791,6 +879,7 @@ declare const DISPATCH_ACTION_KIND: {
     readonly SELECTION_REPLACE: "selection-replace";
     readonly VISIBILITY_SET: "visibility-set";
     readonly FILTER_SET: "filter-set";
+    readonly SORT_SET: "sort-set";
     readonly SIR_SYNC: "sir-sync";
     readonly ACTIVITY_LOOKAHEAD_SYNC: "activity-lookahead-sync";
 };
@@ -836,4 +925,4 @@ declare const DISPATCH_TRACK_EVENT: {
 };
 type DispatchTrackEvent = (typeof DISPATCH_TRACK_EVENT)[keyof typeof DISPATCH_TRACK_EVENT];
 
-export { ACTIVITY_TYPE, type ActivityCreter, type ActivityId, type ActivityType, type BackendActivityInput, type BackendCalendarInput, type BackendLinkInput, type BackendScheduleInput, type BackendSectorInput, type BaselinePoint, type BaselineVersion, CALENDAR_UNIT, COLUMN, CONSTRAINT_TYPE, CONSTRAINT_TYPE_TO_LABEL, CREATION_KIND, type Calendar, type CalendarId, type CalendarUnit, type CalendarWorktime, type ChangeSet, type ColumnName, type ConstraintType, type ConstraintWarning, type CoreActivity, type CreationKind, DEFAULT_HOURS_PER_DAY, DISPATCH_ACTION_KIND, DISPATCH_ACTION_KINDS, DISPATCH_TRACK_EVENT, type DispatchAction, type DispatchActionKind, type DispatchOptions, type DispatchRejectReason, type DispatchResult, type DispatchTrackEvent, type EntityChange, type FilterCriterion, type FilterDateRange, type FilterField, type FilterState, LINK_CODE_TO_TYPE, LINK_TYPE, LINK_TYPE_CODE, type Link, type LinkId, type LinkType, type LinkTypeCode, type LinksBatchOperation, NEW_ACTIVITY_DEFAULTS, type ParsedInput, type PastedActivityInput, type PastedLinkInput, type PendingRequest, type PersistedEntityIdentity, REJECTION_REASON, ROOT_PARENT_ID, ScheduleCore, type ScheduleCoreInput, type ScheduleCoreReporter, type ScheduleCoreStatus, type ScheduleEffect, type SectorMetadata, type StatusCriteria, type TrackingEvent, WORK_TIME_DIRECTION, type WorkTimeDirection, checkNoUpdatedLinks, computeExpectedProgress, expectedProgressFromBaseline, getActiveBaseline, getUnsavedActivities, isRootParent, normalizeParentKey, parseFromBackend, willRunCriticalPath, yieldToBrowser };
+export { ACTIVITY_TYPE, type ActivityCreter, type ActivityId, type ActivityType, type BackendActivityInput, type BackendCalendarInput, type BackendLinkInput, type BackendScheduleInput, type BackendSectorInput, type BaselinePoint, type BaselineVersion, type BranchOrderChange, CALENDAR_UNIT, COLUMN, CONSTRAINT_TYPE, CONSTRAINT_TYPE_TO_LABEL, CREATION_KIND, type Calendar, type CalendarId, type CalendarUnit, type CalendarWorktime, type ChangeSet, type ColumnName, type ConstraintType, type ConstraintWarning, type CoreActivity, type CreationKind, DEFAULT_HOURS_PER_DAY, DISPATCH_ACTION_KIND, DISPATCH_ACTION_KINDS, DISPATCH_TRACK_EVENT, type DispatchAction, type DispatchActionKind, type DispatchOptions, type DispatchRejectReason, type DispatchResult, type DispatchTrackEvent, type EntityChange, type FilterCriterion, type FilterDateRange, type FilterField, type FilterState, LINK_CODE_TO_TYPE, LINK_TYPE, LINK_TYPE_CODE, type Link, type LinkId, type LinkType, type LinkTypeCode, type LinksBatchOperation, NEW_ACTIVITY_DEFAULTS, type OrderRule, type OrderState, type ParsedInput, type PastedActivityInput, type PastedLinkInput, type PendingRequest, type PersistedEntityIdentity, REJECTION_REASON, ROOT_PARENT_ID, ScheduleCore, type ScheduleCoreInput, type ScheduleCoreReporter, type ScheduleCoreStatus, type ScheduleEffect, type SectorMetadata, type StatusCriteria, type TrackingEvent, WORK_TIME_DIRECTION, type WorkTimeDirection, checkNoUpdatedLinks, computeExpectedProgress, expectedProgressFromBaseline, getActiveBaseline, getUnsavedActivities, isRootParent, normalizeParentKey, parseFromBackend, willRunCriticalPath, yieldToBrowser };