@contrail/documents 1.21.0-alpha.fb2b49f → 1.21.0

package/CHANGELOG.md

@@ -28,6 +28,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 +64,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 +79,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,4 +1,4 @@
-import { DocumentElement, PositionDefinition } from '../types';
+import { DocumentElement } from '../types';
 import { ContainerElementType, ContainerKind } from './container-kind';
 import { DocumentContainer, MeasuredDocumentElement } from './document-container';
 export declare class DocumentContainerIndex {
@@ -18,8 +18,7 @@ export declare class DocumentContainerIndex {
     innermostContainerIdOf(documentElementId: string, elementType: ContainerElementType): string | null;
     memberIdsOf(containerId: string): readonly string[];
     memberDocumentElementsOf(containerId: string): DocumentElement[];
-    containerPathFor(documentElement: DocumentElement, kind: ContainerKind, at?: PositionDefinition): readonly string[];
-    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

@@ -44,28 +44,18 @@ class DocumentContainerIndex {
         }
         return members;
     }
-    containerPathFor(documentElement, kind, at) {
-        var _a;
+    containerPathFor(documentElement, kind) {
+        var _a, _b;
         if (!documentElement) {
             return [];
         }
-        const known = this.data.measuredById.get(documentElement.id);
-        const positioned = at ? Object.assign(Object.assign({}, documentElement), { position: at }) : documentElement;
         const measured = {
-            documentElement: positioned,
-            box: at || !known ? document_container_1.DocumentContainerUtil.measureBox(positioned, this.data.documentElementsById) : known.box,
-            drawOrder: (_a = known === null || known === void 0 ? void 0 : known.drawOrder) !== null && _a !== void 0 ? _a : Number.MAX_SAFE_INTEGER,
+            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);
     }
-    containerPathAtPoint(point, elementType) {
-        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);
-    }
     containersOfType(elementType) {
         var _a;
         return this.resolveAll((_a = this.data.containerIdsByElementType.get(elementType)) !== null && _a !== void 0 ? _a : []);

package/lib/containers/document-container.js

@@ -89,7 +89,7 @@ class DocumentContainerUtil {
             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 && !documentElement.position) {
+            if (!width && !height && !x && !y) {
                 return null;
             }
             return DocumentContainerUtil.toCoordinateBox(x, y, width, height);

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

@@ -52,6 +52,7 @@ 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;
@@ -59,6 +60,7 @@ export declare class DynamicTextUtil {
         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' &&
@@ -204,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) {
@@ -219,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 ||
@@ -235,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.fb2b49f",
+  "version": "1.21.0",
   "description": "Documents library for contrail platform",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",