@contrail/documents 1.21.0-alpha.fb70961 → 1.22.0-alpha.766acdd

package/CHANGELOG.md

@@ -5,6 +5,17 @@ All notable changes to `@contrail/documents` are documented here.
 Format follows [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
 Versioning follows [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.22.0] - 2026-10-10
+
+Also shipped on the 2.x line as 2.5.0.
+
+### Added
+
+- `clientCreatedOn` on `WriteTransactionRequest`, `DocumentTransactionPayload`, and
+  `DocumentTransactionMessage`: when the writing client created the transaction (ISO 8601, client
+  clock). The server stores it with the transaction and passes it through in the broadcast, so a
+  receiver can measure how long the edit took to reach it.
+
 ## [1.21.0] - 2026-09-26
 
 Maintenance release on the 1.x line; the same change ships on 2.x as 2.4.0.
@@ -28,6 +39,18 @@ Maintenance release on the 1.x line; the same change ships on 2.x as 2.4.0.
   member into a different cell. Anything deciding what an element overlaps or sits on wants the
   scaled one.
 
+### Removed
+
+- `DocumentContainerIndex.containerPathAtPoint`. It applied the anchor rule alone, because a bare
+  point has nothing to compare against a container, and returned the same shape as `containerIdsOf`,
+  so it read as a second way to ask one question. `containerPathFor(element, kind)` replaces it and
+  applies every rule. Callers that hold a position pass the element with that position on it. This
+  is the one removal a consumer has to act on.
+- `ContainerKind.layer`, `DocumentContainer.name` and `DocumentContainer.depth`, and
+  `DocumentContainerIndex.maxDepth`. The container API has not shipped in a released version, so
+  nothing outside this repository can be holding them. Read a container's name from
+  `documentElement.name`.
+
 ### Deprecated
 
 - `isContainerBinding()` in favour of `DynamicTextUtil.takesValueFromContainer()`. Same rule, named
@@ -52,6 +75,12 @@ Maintenance release on the 1.x line; the same change ships on 2.x as 2.4.0.
   `n/a`. `calculateSize` no longer throws on an element that reaches it without a `size`, which on
   the server failed the read for the whole document.
 
+- `min` and `max` over a date property rendered nothing at all. `Number` of a date string is `NaN`,
+  so every member was dropped; they now order by the instant each date names.
+- `uniqueArray` sorted the rendered text rather than the value, so `11/20/2026` came after
+  `01/15/2027` and `100` before `9`. It now orders by value, falling back to text order for values
+  that are neither a date nor a number.
+
 ### Changed
 
 - `anchorFor` and the index measure through the same factory call, so a caller holding an element
@@ -61,6 +90,15 @@ Maintenance release on the 1.x line; the same change ships on 2.x as 2.4.0.
   two containers that merely cross no longer both claim it.
 - `buildIndex` scans element types before measuring, so a document with no containers of the kinds
   asked for returns an empty index without a geometry pass.
+- **A section nests inside another only when it is drawn above it.** A section drawn over one it
+  encloses hides it, so the hidden section holds nothing and a `section.count` on it reads zero.
+  This changes existing boards wherever the outer section was drawn last.
+- `containerIdsOf` and `memberIdsOf` return copies. `readonly` is erased at runtime, and a caller
+  sorting what it got back reordered the index's own list for every later read.
+- A member list is a set of ids, so two elements sharing an id count once.
+- `min` and `max` no longer read an empty member as zero, and `uniqueArray` no longer emits one as
+  a blank entry. A text box narrower than its own padding measures without a fixed width.
+- A nullish entry in the element list is dropped rather than throwing.
 
 ## [1.20.0] - 2026-09-25
 

package/lib/containers/document-container-index.d.ts

@@ -1,10 +1,11 @@
-import { DocumentElement, PositionDefinition } from '../types';
-import { ContainerElementType } from './container-kind';
-import { DocumentContainer } from './document-container';
+import { DocumentElement } from '../types';
+import { ContainerElementType, ContainerKind } from './container-kind';
+import { DocumentContainer, MeasuredDocumentElement } from './document-container';
 export declare class DocumentContainerIndex {
     private readonly data;
     constructor(data: {
         documentElementsById: Map<string, DocumentElement>;
+        measuredById: Map<string, MeasuredDocumentElement>;
         containersById: Map<string, DocumentContainer>;
         containerPathsByDocumentElementId: Map<string, Map<string, string[]>>;
         memberIdsByContainerId: Map<string, string[]>;
@@ -17,7 +18,7 @@ export declare class DocumentContainerIndex {
     innermostContainerIdOf(documentElementId: string, elementType: ContainerElementType): string | null;
     memberIdsOf(containerId: string): readonly string[];
     memberDocumentElementsOf(containerId: string): DocumentElement[];
-    containerPathAtPoint(point: PositionDefinition, elementType: ContainerElementType): readonly string[];
+    containerPathFor(documentElement: DocumentElement, kind: ContainerKind): readonly string[];
     private containersOfType;
     private resolveAll;
     private containerPath;

package/lib/containers/document-container-index.js

@@ -9,6 +9,7 @@ class DocumentContainerIndex {
     static empty() {
         return new DocumentContainerIndex({
             documentElementsById: new Map(),
+            measuredById: new Map(),
             containersById: new Map(),
             containerPathsByDocumentElementId: new Map(),
             memberIdsByContainerId: new Map(),
@@ -43,13 +44,17 @@ class DocumentContainerIndex {
         }
         return members;
     }
-    containerPathAtPoint(point, elementType) {
+    containerPathFor(documentElement, kind) {
         var _a, _b;
-        const containers = this.containersOfType(elementType);
-        const covering = containers.filter((container) => document_container_1.DocumentContainerUtil.containsPoint(container.box, point));
-        const nests = (_b = (_a = containers[0]) === null || _a === void 0 ? void 0 : _a.kind.nests) !== null && _b !== void 0 ? _b : true;
-        const path = nests ? document_container_1.DocumentContainerUtil.chain(covering) : document_container_1.DocumentContainerUtil.topmost(covering);
-        return path.map((container) => container.id);
+        if (!documentElement) {
+            return [];
+        }
+        const measured = {
+            documentElement,
+            box: document_container_1.DocumentContainerUtil.measureBox(documentElement, this.data.documentElementsById),
+            drawOrder: (_b = (_a = this.data.measuredById.get(documentElement.id)) === null || _a === void 0 ? void 0 : _a.drawOrder) !== null && _b !== void 0 ? _b : Number.MAX_SAFE_INTEGER,
+        };
+        return document_container_1.DocumentContainerUtil.pathFor(measured, kind, this.containersOfType(kind.elementType)).map((container) => container.id);
     }
     containersOfType(elementType) {
         var _a;

package/lib/containers/document-container.d.ts

@@ -25,5 +25,9 @@ export declare class DocumentContainerUtil {
     static containsPoint(box: CoordinateBox, point: PositionDefinition): boolean;
     static encloses(outer: CoordinateBox, inner: CoordinateBox): boolean;
     static chain(candidates: DocumentContainer[]): DocumentContainer[];
+    static measureBox(documentElement: DocumentElement, documentElementsById: Map<string, DocumentElement>): CoordinateBox | null;
+    static pathFor(measured: MeasuredDocumentElement, kind: ContainerKind, containers: DocumentContainer[]): DocumentContainer[];
+    static enclosersOf(measured: MeasuredDocumentElement, containers: DocumentContainer[]): DocumentContainer[];
+    static candidatesFor(measured: MeasuredDocumentElement, containers: DocumentContainer[]): DocumentContainer[];
     static topmost(containers: DocumentContainer[]): DocumentContainer[];
 }

package/lib/containers/document-container.js

@@ -24,9 +24,12 @@ class DocumentContainerUtil {
         var _a, _b, _c, _d, _e;
         const position = documentElement === null || documentElement === void 0 ? void 0 : documentElement.position;
         const angle = (_a = documentElement === null || documentElement === void 0 ? void 0 : documentElement.rotate) === null || _a === void 0 ? void 0 : _a.angle;
-        if (!position || !angle || !newSize || !documentElement.size) {
+        if (!position) {
             return position;
         }
+        if (!angle || !newSize || !documentElement.size) {
+            return Object.assign({}, position);
+        }
         const scaleX = (_c = (_b = documentElement.scale) === null || _b === void 0 ? void 0 : _b.x) !== null && _c !== void 0 ? _c : 1;
         const scaleY = (_e = (_d = documentElement.scale) === null || _d === void 0 ? void 0 : _d.y) !== null && _e !== void 0 ? _e : 1;
         const cornerOffset = (size) => {
@@ -36,7 +39,8 @@ class DocumentContainerUtil {
         };
         const before = cornerOffset(documentElement.size);
         const after = cornerOffset(newSize);
-        return { x: position.x + before.x - after.x, y: position.y + before.y - after.y };
+        const moved = { x: position.x + before.x - after.x, y: position.y + before.y - after.y };
+        return Number.isFinite(moved.x) && Number.isFinite(moved.y) ? moved : Object.assign({}, position);
     }
     static resizeKeepingAnchor(dynamicTextElement, entityType, text) {
         const size = dynamic_text_util_1.DynamicTextUtil.calculateSize(dynamicTextElement, entityType, text);
@@ -79,6 +83,56 @@ class DocumentContainerUtil {
             path.unshift(topmostOf(enclosing));
         }
     }
+    static measureBox(documentElement, documentElementsById) {
+        var _a, _b;
+        try {
+            const start = ((_a = documentElement === null || documentElement === void 0 ? void 0 : documentElement.start) === null || _a === void 0 ? void 0 : _a.id) ? documentElementsById.get(documentElement.start.id) : undefined;
+            const end = ((_b = documentElement === null || documentElement === void 0 ? void 0 : documentElement.end) === null || _b === void 0 ? void 0 : _b.id) ? documentElementsById.get(documentElement.end.id) : undefined;
+            const { x, y, width, height } = document_element_factory_1.DocumentElementFactory.getBoundingClientRectScaled(documentElement, start, end);
+            if (!width && !height && !x && !y) {
+                return null;
+            }
+            return DocumentContainerUtil.toCoordinateBox(x, y, width, height);
+        }
+        catch (_c) {
+            return null;
+        }
+    }
+    static pathFor(measured, kind, containers) {
+        if (!containers.length || !measured.box) {
+            return [];
+        }
+        if (measured.documentElement.type === kind.elementType) {
+            return kind.nests ? DocumentContainerUtil.chain(DocumentContainerUtil.enclosersOf(measured, containers)) : [];
+        }
+        const candidates = DocumentContainerUtil.candidatesFor(measured, containers);
+        return kind.nests ? DocumentContainerUtil.chain(candidates) : DocumentContainerUtil.topmost(candidates);
+    }
+    static enclosersOf(measured, containers) {
+        return containers.filter((container) => {
+            var _a;
+            return container.id !== measured.documentElement.id &&
+                DocumentContainerUtil.encloses(container.box, measured.box) &&
+                measured.drawOrder > container.drawOrder &&
+                !((_a = measured.documentElement.elementIds) === null || _a === void 0 ? void 0 : _a.includes(container.id));
+        });
+    }
+    static candidatesFor(measured, containers) {
+        const anchor = DocumentContainerUtil.anchorOf(measured.documentElement, measured.box);
+        return containers.filter((container) => {
+            var _a;
+            if (container.id === measured.documentElement.id || measured.drawOrder <= container.drawOrder) {
+                return false;
+            }
+            if (!DocumentContainerUtil.containsPoint(container.box, anchor)) {
+                return false;
+            }
+            if (DocumentContainerUtil.encloses(measured.box, container.box)) {
+                return false;
+            }
+            return !((_a = measured.documentElement.elementIds) === null || _a === void 0 ? void 0 : _a.includes(container.id));
+        });
+    }
     static topmost(containers) {
         return containers.length
             ? [containers.reduce((topmost, container) => (container.drawOrder > topmost.drawOrder ? container : topmost))]

package/lib/containers/document-container.service.d.ts

@@ -5,9 +5,5 @@ export declare class DocumentContainerService {
     static buildIndex(documentElements: readonly DocumentElement[], kinds?: ContainerKind[]): DocumentContainerIndex;
     static isContainer(documentElement: DocumentElement, kinds?: ContainerKind[]): boolean;
     private static measureDocumentElements;
-    private static measure;
     private static collectContainers;
-    private static resolvePath;
-    private static enclosersOf;
-    private static candidateContainers;
 }

package/lib/containers/document-container.service.js

@@ -1,7 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.DocumentContainerService = void 0;
-const document_element_factory_1 = require("../document-element-factory");
 const container_kind_1 = require("./container-kind");
 const document_container_index_1 = require("./document-container-index");
 const document_container_1 = require("./document-container");
@@ -26,7 +25,7 @@ class DocumentContainerService {
         for (const measured of measuredDocumentElements) {
             const byElementType = new Map();
             for (const kind of kinds) {
-                byElementType.set(kind.elementType, this.resolvePath(measured, kind, (_a = containersByElementType.get(kind.elementType)) !== null && _a !== void 0 ? _a : []));
+                byElementType.set(kind.elementType, document_container_1.DocumentContainerUtil.pathFor(measured, kind, (_a = containersByElementType.get(kind.elementType)) !== null && _a !== void 0 ? _a : []));
             }
             pathsByMeasured.set(measured, byElementType);
         }
@@ -52,8 +51,13 @@ class DocumentContainerService {
             }
             containerPathsByDocumentElementId.set(documentElementId, pathsByElementType);
         }
+        const measuredById = new Map();
+        for (const measured of measuredDocumentElements) {
+            measuredById.set(measured.documentElement.id, measured);
+        }
         return new document_container_index_1.DocumentContainerIndex({
             documentElementsById,
+            measuredById,
             containersById,
             containerPathsByDocumentElementId,
             memberIdsByContainerId,
@@ -67,21 +71,9 @@ class DocumentContainerService {
         return documentElements.map((documentElement, drawOrder) => ({
             documentElement,
             drawOrder,
-            box: this.measure(documentElement, documentElementsById),
+            box: document_container_1.DocumentContainerUtil.measureBox(documentElement, documentElementsById),
         }));
     }
-    static measure(documentElement, documentElementsById) {
-        var _a, _b;
-        try {
-            const start = ((_a = documentElement === null || documentElement === void 0 ? void 0 : documentElement.start) === null || _a === void 0 ? void 0 : _a.id) ? documentElementsById.get(documentElement.start.id) : undefined;
-            const end = ((_b = documentElement === null || documentElement === void 0 ? void 0 : documentElement.end) === null || _b === void 0 ? void 0 : _b.id) ? documentElementsById.get(documentElement.end.id) : undefined;
-            const { x, y, width, height } = document_element_factory_1.DocumentElementFactory.getBoundingClientRectScaled(documentElement, start, end);
-            return document_container_1.DocumentContainerUtil.toCoordinateBox(x, y, width, height);
-        }
-        catch (_c) {
-            return null;
-        }
-    }
     static collectContainers(measuredDocumentElements, kind) {
         return measuredDocumentElements
             .filter((measured) => measured.documentElement.type === kind.elementType && measured.box)
@@ -93,40 +85,5 @@ class DocumentContainerService {
             drawOrder: measured.drawOrder,
         }));
     }
-    static resolvePath(measured, kind, containers) {
-        if (!containers.length || !measured.box) {
-            return [];
-        }
-        if (measured.documentElement.type === kind.elementType) {
-            return kind.nests ? document_container_1.DocumentContainerUtil.chain(this.enclosersOf(measured, containers)) : [];
-        }
-        const candidates = this.candidateContainers(measured, containers);
-        return kind.nests ? document_container_1.DocumentContainerUtil.chain(candidates) : document_container_1.DocumentContainerUtil.topmost(candidates);
-    }
-    static enclosersOf(measured, containers) {
-        return containers.filter((container) => {
-            var _a;
-            return container.id !== measured.documentElement.id &&
-                document_container_1.DocumentContainerUtil.encloses(container.box, measured.box) &&
-                measured.drawOrder > container.drawOrder &&
-                !((_a = measured.documentElement.elementIds) === null || _a === void 0 ? void 0 : _a.includes(container.id));
-        });
-    }
-    static candidateContainers(measured, containers) {
-        const anchor = document_container_1.DocumentContainerUtil.anchorOf(measured.documentElement, measured.box);
-        return containers.filter((container) => {
-            var _a;
-            if (container.id === measured.documentElement.id || measured.drawOrder <= container.drawOrder) {
-                return false;
-            }
-            if (!document_container_1.DocumentContainerUtil.containsPoint(container.box, anchor)) {
-                return false;
-            }
-            if (document_container_1.DocumentContainerUtil.encloses(measured.box, container.box)) {
-                return false;
-            }
-            return !((_a = measured.documentElement.elementIds) === null || _a === void 0 ? void 0 : _a.includes(container.id));
-        });
-    }
 }
 exports.DocumentContainerService = DocumentContainerService;

package/lib/types/document-transaction.d.ts

@@ -13,6 +13,7 @@ export type DocumentOperation = {
 export type DocumentElementPatch = Partial<Omit<DocumentElement, 'id' | 'documentId'>>;
 interface WriteTransactionRequestBase {
     clientTransactionId?: string;
+    clientCreatedOn?: string;
 }
 export interface InlineWriteTransactionRequest extends WriteTransactionRequestBase {
     operations: DocumentOperation[];
@@ -53,6 +54,7 @@ interface DocumentTransactionMessageBase {
     sequenceNumber: number;
     createdOn: string;
     clientTransactionId?: string;
+    clientCreatedOn?: string;
 }
 export type DocumentTransactionMessage = (DocumentTransactionMessageBase & {
     operations: DocumentOperation[];
@@ -68,6 +70,7 @@ export interface TransactionUploadTarget {
 }
 export type DocumentTransactionPayload = {
     clientTransactionId?: string;
+    clientCreatedOn?: string;
 } & ({
     operations: DocumentOperation[];
     operationsS3Key?: never;

package/lib/util/dynamic-text/dynamic-text-util.d.ts

@@ -52,10 +52,15 @@ export declare const HARD_CODED_TEXT: {
 };
 export declare function isContainerBinding(bindingProperty: string | null | undefined): boolean;
 export declare class DynamicTextUtil {
+    static componentsBoundTo(componentElements: DocumentElement[], entityType: string): DocumentElement[];
     static isDynamicText(documentElement: DocumentElement): boolean;
     static takesValueFromContainer(bindingProperty: string | null | undefined): boolean;
     static getDynamicTextValue(dynamicTextElement: DocumentElement, property: TypeProperty, propertyLevel: PropertyLevel, container: DynamicTextContainer | null, componentModels: any[], ownerReferenceModel?: any): string;
-    static calculateSize(dynamicTextElement: DocumentElement, entityType: any, text: string): import("../..").SizeDefinition;
+    static calculateSize(dynamicTextElement: DocumentElement, entityType: any, text: string): {
+        width: number;
+        height: number;
+    };
+    private static rankValue;
     static getTextFromFrameMemberValues(displayFunction: string, property: any, memberValues: any[]): string;
     private static getItemCount;
 }

package/lib/util/dynamic-text/dynamic-text-util.js

@@ -63,6 +63,12 @@ function isContainerBinding(bindingProperty) {
     return DynamicTextUtil.takesValueFromContainer(bindingProperty);
 }
 class DynamicTextUtil {
+    static componentsBoundTo(componentElements, entityType) {
+        if (CONTAINER_SOURCES.includes(entityType)) {
+            return componentElements;
+        }
+        return (componentElements !== null && componentElements !== void 0 ? componentElements : []).filter((element) => { var _a; return Object.keys((_a = element === null || element === void 0 ? void 0 : element.modelBindings) !== null && _a !== void 0 ? _a : {}).includes(entityType); });
+    }
     static isDynamicText(documentElement) {
         var _a;
         return ((documentElement === null || documentElement === void 0 ? void 0 : documentElement.type) === 'text' &&
@@ -167,16 +173,19 @@ class DynamicTextUtil {
         return text;
     }
     static calculateSize(dynamicTextElement, entityType, text) {
-        var _a, _b, _c, _d, _e;
-        let size = (_a = dynamicTextElement.size) !== null && _a !== void 0 ? _a : { width: 0, height: 0 };
-        const fontFamily = (_b = document_util_1.DocumentStyleUtil.getFirstFontFamily(dynamicTextElement === null || dynamicTextElement === void 0 ? void 0 : dynamicTextElement.text)) !== null && _b !== void 0 ? _b : document_util_1.DEFAULT_TEXT_FONT_FAMILY;
-        const fontSize = (_e = (dynamicTextElement.isTextTool
-            ? (_d = (_c = dynamicTextElement.style) === null || _c === void 0 ? void 0 : _c.font) === null || _d === void 0 ? void 0 : _d.size
-            : document_util_1.DocumentStyleUtil.getFirstFontSize(dynamicTextElement === null || dynamicTextElement === void 0 ? void 0 : dynamicTextElement.text, true))) !== null && _e !== void 0 ? _e : document_util_1.DEFAULT_TEXT_FONT_SIZE;
+        var _a, _b, _c, _d, _e, _f, _g, _h, _j, _k, _l;
+        let size = {
+            width: (_b = (_a = dynamicTextElement.size) === null || _a === void 0 ? void 0 : _a.width) !== null && _b !== void 0 ? _b : 0,
+            height: (_d = (_c = dynamicTextElement.size) === null || _c === void 0 ? void 0 : _c.height) !== null && _d !== void 0 ? _d : 0,
+        };
+        const fontFamily = (_f = document_util_1.DocumentStyleUtil.getFirstFontFamily((_e = dynamicTextElement === null || dynamicTextElement === void 0 ? void 0 : dynamicTextElement.text) !== null && _e !== void 0 ? _e : '')) !== null && _f !== void 0 ? _f : document_util_1.DEFAULT_TEXT_FONT_FAMILY;
+        const fontSize = (_j = (dynamicTextElement.isTextTool
+            ? (_h = (_g = dynamicTextElement.style) === null || _g === void 0 ? void 0 : _g.font) === null || _h === void 0 ? void 0 : _h.size
+            : document_util_1.DocumentStyleUtil.getFirstFontSize(dynamicTextElement === null || dynamicTextElement === void 0 ? void 0 : dynamicTextElement.text, true))) !== null && _j !== void 0 ? _j : document_util_1.DEFAULT_TEXT_FONT_SIZE;
         const fontSizePx = document_util_1.DocumentStyleUtil.toPx(fontSize, 4);
         const padding = dynamicTextElement.isTextTool ? document_util_1.TEXT_TOOL_PADDING : document_util_1.TEXT_BOX_PADDING;
-        const bold = document_util_1.DocumentStyleUtil.isBold(dynamicTextElement === null || dynamicTextElement === void 0 ? void 0 : dynamicTextElement.text, fontFamily);
-        const italic = document_util_1.DocumentStyleUtil.isItalic(dynamicTextElement === null || dynamicTextElement === void 0 ? void 0 : dynamicTextElement.text);
+        const bold = document_util_1.DocumentStyleUtil.isBold((_k = dynamicTextElement === null || dynamicTextElement === void 0 ? void 0 : dynamicTextElement.text) !== null && _k !== void 0 ? _k : '', fontFamily);
+        const italic = document_util_1.DocumentStyleUtil.isItalic((_l = dynamicTextElement === null || dynamicTextElement === void 0 ? void 0 : dynamicTextElement.text) !== null && _l !== void 0 ? _l : '');
         const measureTextSettings = {
             fontFamily,
             fontSize: fontSizePx,
@@ -201,6 +210,9 @@ class DynamicTextUtil {
         catch (e) { }
         return size;
     }
+    static rankValue(value, property) {
+        return (property === null || property === void 0 ? void 0 : property.propertyType) === types_1.PropertyType.Date ? Date.parse(value) : Number(value);
+    }
     static getTextFromFrameMemberValues(displayFunction, property, memberValues) {
         let text = '';
         if (memberValues.length === 0) {
@@ -216,11 +228,16 @@ class DynamicTextUtil {
             text = formatter.formatValueForProperty(memberValues.reduce((a, b) => a + b, 0), property);
         }
         else if (displayFunction === DynamicTextDisplayFunction.UNIQUE_ARRAY) {
-            text = Array.from(new Set(memberValues
-                .filter((e) => e !== null && e !== undefined && e !== '')
-                .flat()
-                .map((e) => formatter.formatValueForProperty(e, property))))
-                .sort()
+            const byValue = new Map();
+            for (const value of memberValues.filter((e) => e !== null && e !== undefined && e !== '').flat()) {
+                const formatted = formatter.formatValueForProperty(value, property);
+                if (!byValue.has(formatted)) {
+                    byValue.set(formatted, DynamicTextUtil.rankValue(value, property));
+                }
+            }
+            text = Array.from(byValue)
+                .sort(([leftText, leftRank], [rightText, rightRank]) => Number.isNaN(leftRank) || Number.isNaN(rightRank) ? leftText.localeCompare(rightText) : leftRank - rightRank)
+                .map(([formatted]) => formatted)
                 .join(', ');
         }
         else if (!displayFunction ||
@@ -232,15 +249,17 @@ class DynamicTextUtil {
         }
         else if (displayFunction === DynamicTextDisplayFunction.MIN ||
             displayFunction === DynamicTextDisplayFunction.MAX) {
-            const present = memberValues
+            const isDate = (property === null || property === void 0 ? void 0 : property.propertyType) === types_1.PropertyType.Date;
+            const ranked = memberValues
                 .filter((e) => e !== null && e !== undefined && e !== '')
-                .map(Number)
-                .filter((e) => !Number.isNaN(e));
-            if (!present.length) {
+                .map((value) => ({ value, rank: DynamicTextUtil.rankValue(value, property) }))
+                .filter(({ rank }) => !Number.isNaN(rank));
+            if (!ranked.length) {
                 return '';
             }
-            const extreme = displayFunction === DynamicTextDisplayFunction.MIN ? Math.min(...present) : Math.max(...present);
-            text = formatter.formatValueForProperty(extreme, property);
+            const wantsLower = displayFunction === DynamicTextDisplayFunction.MIN;
+            const extreme = ranked.reduce((best, candidate) => (wantsLower ? candidate.rank < best.rank : candidate.rank > best.rank) ? candidate : best);
+            text = formatter.formatValueForProperty(isDate ? extreme.value : extreme.rank, property);
         }
         return text;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contrail/documents",
-  "version": "1.21.0-alpha.fb70961",
+  "version": "1.22.0-alpha.766acdd",
   "description": "Documents library for contrail platform",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",