@ifc-lite/bcf 3.0.0 → 4.0.0

package/README.md

@@ -1,6 +1,6 @@
 # @ifc-lite/bcf
 
-BCF (BIM Collaboration Format) support for IFClite. Reads and writes BCF 2.1 and 3.0 files — the issue-tracking format every BIM tool speaks (Revit, Archicad, Solibri, BIMcollab, etc.).
+BCF (BIM Collaboration Format) support for IFClite. Reads and writes BCF 2.1 and 3.0 files — the topic-tracking format every BIM tool speaks (Revit, Archicad, Solibri, BIMcollab, etc.).
 
 ## Installation
 

package/dist/index.d.ts

@@ -2,7 +2,7 @@
  * @ifc-lite/bcf - BIM Collaboration Format support
  *
  * This package provides BCF (BIM Collaboration Format) support for IFC-Lite.
- * It implements BCF 2.1 and 3.0 specifications for issue tracking in BIM projects.
+ * It implements BCF 2.1 and 3.0 specifications for topic tracking in BIM projects.
  *
  * @see https://github.com/buildingSMART/BCF-XML
  * @see https://www.buildingsmart.org/standards/bsi-standards/bim-collaboration-format/
@@ -28,7 +28,7 @@ export declare function createBCFProject(options?: {
     version?: '2.1' | '3.0';
 }): BCFProject;
 /**
- * Create a new BCF topic (issue)
+ * Create a new BCF topic
  */
 export declare function createBCFTopic(options: {
     title: string;

package/dist/index.js

@@ -24,7 +24,7 @@ export function createBCFProject(options) {
     };
 }
 /**
- * Create a new BCF topic (issue)
+ * Create a new BCF topic
  */
 export function createBCFTopic(options) {
     return {

package/dist/reader-viewpoint-content.d.ts

@@ -0,0 +1,6 @@
+import type { BCFViewpoint } from './types.js';
+/**
+ * Parse viewpoint XML content
+ */
+export declare function parseViewpointContent(content: string, versionId: '2.1' | '3.0'): BCFViewpoint | null;
+//# sourceMappingURL=reader-viewpoint-content.d.ts.map

package/dist/reader-viewpoint-content.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"reader-viewpoint-content.d.ts","sourceRoot":"","sources":["../src/reader-viewpoint-content.ts"],"names":[],"mappings":"AAaA,OAAO,KAAK,EACV,YAAY,EAQb,MAAM,YAAY,CAAC;AAIpB;;GAEG;AACH,wBAAgB,qBAAqB,CAAC,OAAO,EAAE,MAAM,EAAE,SAAS,EAAE,KAAK,GAAG,KAAK,GAAG,YAAY,GAAG,IAAI,CAgCpG"}

package/dist/reader-viewpoint-content.js

@@ -0,0 +1,250 @@
+/* This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at https://mozilla.org/MPL/2.0/. */
+/**
+ * Parsing of a single BCFV viewpoint file's XML content: the two camera
+ * shapes, the shared point/direction primitive, and the geometry overlays
+ * (lines, clipping planes, bitmaps). Split out of reader.ts, which still owns
+ * the zip-level orchestration (locating .bcfv files, resolving snapshots) that
+ * calls into {@link parseViewpointContent}.
+ */
+import { extractElement } from './xml-text.js';
+import { parseFiniteFloat } from './numeric.js';
+import { parseComponents } from './reader-components.js';
+/**
+ * Parse viewpoint XML content
+ */
+export function parseViewpointContent(content, versionId) {
+    // Extract viewpoint GUID from root element (Guid can be anywhere in the tag)
+    const guidMatch = content.match(/<VisualizationInfo[^>]+Guid="([^"]+)"/);
+    const guid = guidMatch?.[1] || crypto.randomUUID?.() || `vp-${Date.now()}`;
+    // Parse perspective camera
+    const perspectiveCamera = parsePerspectiveCamera(content);
+    // Parse orthogonal camera
+    const orthogonalCamera = parseOrthogonalCamera(content);
+    // Parse components
+    const components = parseComponents(content, versionId);
+    // Parse lines
+    const lines = parseLines(content);
+    // Parse clipping planes
+    const clippingPlanes = parseClippingPlanes(content);
+    // Parse bitmaps
+    const bitmaps = parseBitmaps(content);
+    return {
+        guid,
+        perspectiveCamera,
+        orthogonalCamera,
+        components,
+        lines: lines.length > 0 ? lines : undefined,
+        clippingPlanes: clippingPlanes.length > 0 ? clippingPlanes : undefined,
+        bitmaps: bitmaps.length > 0 ? bitmaps : undefined,
+    };
+}
+/**
+ * Parse perspective camera from viewpoint content
+ */
+function parsePerspectiveCamera(content) {
+    const match = content.match(/<PerspectiveCamera>([\s\S]*?)<\/PerspectiveCamera>/);
+    if (!match)
+        return undefined;
+    const cameraContent = match[1];
+    const viewPoint = parsePoint(cameraContent, 'CameraViewPoint');
+    const direction = parseDirection(cameraContent, 'CameraDirection');
+    const upVector = parseDirection(cameraContent, 'CameraUpVector');
+    const fieldOfView = extractElement(cameraContent, 'FieldOfView');
+    if (!viewPoint || !direction || !upVector || !fieldOfView) {
+        return undefined;
+    }
+    // Same treatment as the coordinates: an unusable scalar is a missing one.
+    // `fieldOfView` is converted to radians and handed to the viewer camera,
+    // and the parser already drops the whole camera when the element is absent.
+    const fov = parseFiniteFloat(fieldOfView);
+    if (fov === undefined)
+        return undefined;
+    return {
+        cameraViewPoint: viewPoint,
+        cameraDirection: direction,
+        cameraUpVector: upVector,
+        fieldOfView: fov,
+        ...parseAspectRatio(cameraContent),
+    };
+}
+/**
+ * Parse the optional `<AspectRatio>` of either camera type.
+ *
+ * BCF 3.0's visinfo.xsd makes `AspectRatio` (a `PositiveDouble`) a REQUIRED
+ * child of both camera types; BCF 2.1 has no such element. It is read here for
+ * both, since the element's presence — not the archive's declared version — is
+ * what says whether there is a value to keep.
+ *
+ * Reading it matters beyond fidelity: the writer refuses to emit a 3.0 camera
+ * without one, so a 3.0 archive from another tool could otherwise be read and
+ * then not written back. Returned as a spread-able partial so an absent or
+ * unusable value leaves the property off entirely rather than setting it to
+ * `undefined`, and a non-positive value is dropped rather than carried into
+ * output the schema would reject.
+ */
+function parseAspectRatio(cameraContent) {
+    const raw = extractElement(cameraContent, 'AspectRatio');
+    if (!raw)
+        return {};
+    const value = parseFiniteFloat(raw);
+    if (value === undefined || !(value > 0))
+        return {};
+    return { aspectRatio: value };
+}
+/**
+ * Parse orthogonal camera from viewpoint content
+ */
+function parseOrthogonalCamera(content) {
+    const match = content.match(/<OrthogonalCamera>([\s\S]*?)<\/OrthogonalCamera>/);
+    if (!match)
+        return undefined;
+    const cameraContent = match[1];
+    const viewPoint = parsePoint(cameraContent, 'CameraViewPoint');
+    const direction = parseDirection(cameraContent, 'CameraDirection');
+    const upVector = parseDirection(cameraContent, 'CameraUpVector');
+    const viewToWorldScale = extractElement(cameraContent, 'ViewToWorldScale');
+    if (!viewPoint || !direction || !upVector || !viewToWorldScale) {
+        return undefined;
+    }
+    // `viewToWorldScale` becomes the orthographic half-height, which is the
+    // value `getOrthoSize()` hands back into a saved viewpoint — so a
+    // non-finite one persists past the session if it is allowed in (#2461).
+    const scale = parseFiniteFloat(viewToWorldScale);
+    if (scale === undefined)
+        return undefined;
+    return {
+        cameraViewPoint: viewPoint,
+        cameraDirection: direction,
+        cameraUpVector: upVector,
+        viewToWorldScale: scale,
+        ...parseAspectRatio(cameraContent),
+    };
+}
+/**
+ * Parse a 3D point from XML
+ */
+function parsePoint(content, elementName) {
+    const match = content.match(new RegExp(`<${elementName}>([\\s\\S]*?)<\\/${elementName}>`));
+    if (!match)
+        return undefined;
+    const x = extractElement(match[1], 'X');
+    const y = extractElement(match[1], 'Y');
+    const z = extractElement(match[1], 'Z');
+    if (x === undefined || y === undefined || z === undefined) {
+        return undefined;
+    }
+    // A coordinate that is not a real number is treated as a missing one.
+    // `parseFloat` has no out-of-band failure value — `"NaN"` parses to `NaN`
+    // and the well-formed literal `"1e999"` parses to `Infinity` — and from here
+    // the value reaches `Camera.setPosition`/`setTarget`, which store a pose
+    // verbatim by design. Once stored, a single non-finite coordinate spreads
+    // across the whole pose on the next gesture. Rejecting at the file boundary
+    // means it never gets there, and it costs no new branch: every caller
+    // already drops the thing it was parsing when a coordinate is missing
+    // (#2466).
+    const px = parseFiniteFloat(x);
+    const py = parseFiniteFloat(y);
+    const pz = parseFiniteFloat(z);
+    if (px === undefined || py === undefined || pz === undefined) {
+        return undefined;
+    }
+    return { x: px, y: py, z: pz };
+}
+/**
+ * Parse a 3D direction from XML
+ */
+function parseDirection(content, elementName) {
+    return parsePoint(content, elementName);
+}
+/**
+ * Parse lines
+ */
+function parseLines(content) {
+    const lines = [];
+    const linesMatch = content.match(/<Lines>([\s\S]*?)<\/Lines>/);
+    if (!linesMatch)
+        return lines;
+    const lineMatches = linesMatch[1].matchAll(/<Line>([\s\S]*?)<\/Line>/g);
+    for (const match of lineMatches) {
+        const startPoint = parsePoint(match[1], 'StartPoint');
+        const endPoint = parsePoint(match[1], 'EndPoint');
+        if (startPoint && endPoint) {
+            lines.push({ startPoint, endPoint });
+        }
+    }
+    return lines;
+}
+/**
+ * Parse clipping planes
+ */
+function parseClippingPlanes(content) {
+    const planes = [];
+    const planesMatch = content.match(/<ClippingPlanes>([\s\S]*?)<\/ClippingPlanes>/);
+    if (!planesMatch)
+        return planes;
+    const planeMatches = planesMatch[1].matchAll(/<ClippingPlane>([\s\S]*?)<\/ClippingPlane>/g);
+    for (const match of planeMatches) {
+        const location = parsePoint(match[1], 'Location');
+        const direction = parseDirection(match[1], 'Direction');
+        if (location && direction) {
+            planes.push({ location, direction });
+        }
+    }
+    return planes;
+}
+/**
+ * Parse bitmaps
+ *
+ * The two BCF versions diverge in shape (see writer.ts's writeBitmap/
+ * writeViewpointFiles for the write side of this):
+ * - BCF 3.0: entries sit inside a `<Bitmaps>` wrapper, and the per-entry
+ *   format element is named `<Format>`. No tag inside an entry shares the
+ *   entry's own name, so a plain non-greedy `<Bitmap>...</Bitmap>` match
+ *   is unambiguous.
+ * - BCF 2.1: entries sit DIRECTLY under `<VisualizationInfo>` (no wrapper),
+ *   and the format element is confusingly also named `<Bitmap>`, nested one
+ *   level inside the entry (`<Bitmap><Bitmap>PNG</Bitmap><Reference>...`).
+ *   A naive non-greedy `<Bitmap>...</Bitmap>` match on that shape terminates
+ *   at the FIRST `</Bitmap>` it sees -- the inner format tag's closing tag,
+ *   not the entry's -- and silently drops the rest of the entry. It must be
+ *   matched with an explicit two-level pattern instead.
+ */
+function parseBitmaps(content) {
+    const bitmaps = [];
+    const bitmapsMatch = content.match(/<Bitmaps>([\s\S]*?)<\/Bitmaps>/);
+    const pushBitmap = (format, body) => {
+        const reference = extractElement(body, 'Reference');
+        const location = parsePoint(body, 'Location');
+        const normal = parseDirection(body, 'Normal');
+        const up = parseDirection(body, 'Up');
+        const height = extractElement(body, 'Height');
+        const parsedHeight = height === undefined ? undefined : parseFiniteFloat(height);
+        if (format && reference && location && normal && up && parsedHeight !== undefined) {
+            bitmaps.push({
+                format: format.toUpperCase() === 'JPG' ? 'JPG' : 'PNG',
+                reference,
+                location,
+                normal,
+                up,
+                height: parsedHeight,
+            });
+        }
+    };
+    if (bitmapsMatch) {
+        // BCF 3.0 shape: <Bitmaps><Bitmap><Format>...</Format>...</Bitmap>...</Bitmaps>
+        for (const match of bitmapsMatch[1].matchAll(/<Bitmap>([\s\S]*?)<\/Bitmap>/g)) {
+            pushBitmap(extractElement(match[1], 'Format'), match[1]);
+        }
+    }
+    else {
+        // BCF 2.1 shape: <Bitmap><Bitmap>PNG</Bitmap>...</Bitmap>, unwrapped,
+        // directly under VisualizationInfo.
+        for (const match of content.matchAll(/<Bitmap>\s*<Bitmap>([\s\S]*?)<\/Bitmap>([\s\S]*?)<\/Bitmap>/g)) {
+            pushBitmap(match[1], match[2]);
+        }
+    }
+    return bitmaps;
+}
+//# sourceMappingURL=reader-viewpoint-content.js.map

package/dist/reader-viewpoint-content.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"reader-viewpoint-content.js","sourceRoot":"","sources":["../src/reader-viewpoint-content.ts"],"names":[],"mappings":"AAAA;;+DAE+D;AAE/D;;;;;;GAMG;AAEH,OAAO,EAAE,cAAc,EAAE,MAAM,eAAe,CAAC;AAW/C,OAAO,EAAE,gBAAgB,EAAE,MAAM,cAAc,CAAC;AAChD,OAAO,EAAE,eAAe,EAAE,MAAM,wBAAwB,CAAC;AAEzD;;GAEG;AACH,MAAM,UAAU,qBAAqB,CAAC,OAAe,EAAE,SAAwB;IAC7E,6EAA6E;IAC7E,MAAM,SAAS,GAAG,OAAO,CAAC,KAAK,CAAC,uCAAuC,CAAC,CAAC;IACzE,MAAM,IAAI,GAAG,SAAS,EAAE,CAAC,CAAC,CAAC,IAAI,MAAM,CAAC,UAAU,EAAE,EAAE,IAAI,MAAM,IAAI,CAAC,GAAG,EAAE,EAAE,CAAC;IAE3E,2BAA2B;IAC3B,MAAM,iBAAiB,GAAG,sBAAsB,CAAC,OAAO,CAAC,CAAC;IAE1D,0BAA0B;IAC1B,MAAM,gBAAgB,GAAG,qBAAqB,CAAC,OAAO,CAAC,CAAC;IAExD,mBAAmB;IACnB,MAAM,UAAU,GAAG,eAAe,CAAC,OAAO,EAAE,SAAS,CAAC,CAAC;IAEvD,cAAc;IACd,MAAM,KAAK,GAAG,UAAU,CAAC,OAAO,CAAC,CAAC;IAElC,wBAAwB;IACxB,MAAM,cAAc,GAAG,mBAAmB,CAAC,OAAO,CAAC,CAAC;IAEpD,gBAAgB;IAChB,MAAM,OAAO,GAAG,YAAY,CAAC,OAAO,CAAC,CAAC;IAEtC,OAAO;QACL,IAAI;QACJ,iBAAiB;QACjB,gBAAgB;QAChB,UAAU;QACV,KAAK,EAAE,KAAK,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,SAAS;QAC3C,cAAc,EAAE,cAAc,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,cAAc,CAAC,CAAC,CAAC,SAAS;QACtE,OAAO,EAAE,OAAO,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,SAAS;KAClD,CAAC;AACJ,CAAC;AAED;;GAEG;AACH,SAAS,sBAAsB,CAAC,OAAe;IAC7C,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,oDAAoD,CAAC,CAAC;IAClF,IAAI,CAAC,KAAK;QAAE,OAAO,SAAS,CAAC;IAE7B,MAAM,aAAa,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC;IAE/B,MAAM,SAAS,GAAG,UAAU,CAAC,aAAa,EAAE,iBAAiB,CAAC,CAAC;IAC/D,MAAM,SAAS,GAAG,cAAc,CAAC,aAAa,EAAE,iBAAiB,CAAC,CAAC;IACnE,MAAM,QAAQ,GAAG,cAAc,CAAC,aAAa,EAAE,gBAAgB,CAAC,CAAC;IACjE,MAAM,WAAW,GAAG,cAAc,CAAC,aAAa,EAAE,aAAa,CAAC,CAAC;IAEjE,IAAI,CAAC,SAAS,IAAI,CAAC,SAAS,IAAI,CAAC,QAAQ,IAAI,CAAC,WAAW,EAAE,CAAC;QAC1D,OAAO,SAAS,CAAC;IACnB,CAAC;IAED,0EAA0E;IAC1E,yEAAyE;IACzE,4EAA4E;IAC5E,MAAM,GAAG,GAAG,gBAAgB,CAAC,WAAW,CAAC,CAAC;IAC1C,IAAI,GAAG,KAAK,SAAS;QAAE,OAAO,SAAS,CAAC;IAExC,OAAO;QACL,eAAe,EAAE,SAAS;QAC1B,eAAe,EAAE,SAAS;QAC1B,cAAc,EAAE,QAAQ;QACxB,WAAW,EAAE,GAAG;QAChB,GAAG,gBAAgB,CAAC,aAAa,CAAC;KACnC,CAAC;AACJ,CAAC;AAED;;;;;;;;;;;;;;GAcG;AACH,SAAS,gBAAgB,CAAC,aAAqB;IAC7C,MAAM,GAAG,GAAG,cAAc,CAAC,aAAa,EAAE,aAAa,CAAC,CAAC;IACzD,IAAI,CAAC,GAAG;QAAE,OAAO,EAAE,CAAC;IACpB,MAAM,KAAK,GAAG,gBAAgB,CAAC,GAAG,CAAC,CAAC;IACpC,IAAI,KAAK,KAAK,SAAS,IAAI,CAAC,CAAC,KAAK,GAAG,CAAC,CAAC;QAAE,OAAO,EAAE,CAAC;IACnD,OAAO,EAAE,WAAW,EAAE,KAAK,EAAE,CAAC;AAChC,CAAC;AAED;;GAEG;AACH,SAAS,qBAAqB,CAAC,OAAe;IAC5C,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,kDAAkD,CAAC,CAAC;IAChF,IAAI,CAAC,KAAK;QAAE,OAAO,SAAS,CAAC;IAE7B,MAAM,aAAa,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC;IAE/B,MAAM,SAAS,GAAG,UAAU,CAAC,aAAa,EAAE,iBAAiB,CAAC,CAAC;IAC/D,MAAM,SAAS,GAAG,cAAc,CAAC,aAAa,EAAE,iBAAiB,CAAC,CAAC;IACnE,MAAM,QAAQ,GAAG,cAAc,CAAC,aAAa,EAAE,gBAAgB,CAAC,CAAC;IACjE,MAAM,gBAAgB,GAAG,cAAc,CAAC,aAAa,EAAE,kBAAkB,CAAC,CAAC;IAE3E,IAAI,CAAC,SAAS,IAAI,CAAC,SAAS,IAAI,CAAC,QAAQ,IAAI,CAAC,gBAAgB,EAAE,CAAC;QAC/D,OAAO,SAAS,CAAC;IACnB,CAAC;IAED,wEAAwE;IACxE,kEAAkE;IAClE,wEAAwE;IACxE,MAAM,KAAK,GAAG,gBAAgB,CAAC,gBAAgB,CAAC,CAAC;IACjD,IAAI,KAAK,KAAK,SAAS;QAAE,OAAO,SAAS,CAAC;IAE1C,OAAO;QACL,eAAe,EAAE,SAAS;QAC1B,eAAe,EAAE,SAAS;QAC1B,cAAc,EAAE,QAAQ;QACxB,gBAAgB,EAAE,KAAK;QACvB,GAAG,gBAAgB,CAAC,aAAa,CAAC;KACnC,CAAC;AACJ,CAAC;AAED;;GAEG;AACH,SAAS,UAAU,CAAC,OAAe,EAAE,WAAmB;IACtD,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,MAAM,CAAC,IAAI,WAAW,oBAAoB,WAAW,GAAG,CAAC,CAAC,CAAC;IAC3F,IAAI,CAAC,KAAK;QAAE,OAAO,SAAS,CAAC;IAE7B,MAAM,CAAC,GAAG,cAAc,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;IACxC,MAAM,CAAC,GAAG,cAAc,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;IACxC,MAAM,CAAC,GAAG,cAAc,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;IAExC,IAAI,CAAC,KAAK,SAAS,IAAI,CAAC,KAAK,SAAS,IAAI,CAAC,KAAK,SAAS,EAAE,CAAC;QAC1D,OAAO,SAAS,CAAC;IACnB,CAAC;IAED,sEAAsE;IACtE,0EAA0E;IAC1E,6EAA6E;IAC7E,yEAAyE;IACzE,0EAA0E;IAC1E,4EAA4E;IAC5E,sEAAsE;IACtE,sEAAsE;IACtE,WAAW;IACX,MAAM,EAAE,GAAG,gBAAgB,CAAC,CAAC,CAAC,CAAC;IAC/B,MAAM,EAAE,GAAG,gBAAgB,CAAC,CAAC,CAAC,CAAC;IAC/B,MAAM,EAAE,GAAG,gBAAgB,CAAC,CAAC,CAAC,CAAC;IAE/B,IAAI,EAAE,KAAK,SAAS,IAAI,EAAE,KAAK,SAAS,IAAI,EAAE,KAAK,SAAS,EAAE,CAAC;QAC7D,OAAO,SAAS,CAAC;IACnB,CAAC;IAED,OAAO,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC;AACjC,CAAC;AAED;;GAEG;AACH,SAAS,cAAc,CAAC,OAAe,EAAE,WAAmB;IAC1D,OAAO,UAAU,CAAC,OAAO,EAAE,WAAW,CAA6B,CAAC;AACtE,CAAC;AAED;;GAEG;AACH,SAAS,UAAU,CAAC,OAAe;IACjC,MAAM,KAAK,GAAc,EAAE,CAAC;IAC5B,MAAM,UAAU,GAAG,OAAO,CAAC,KAAK,CAAC,4BAA4B,CAAC,CAAC;IAC/D,IAAI,CAAC,UAAU;QAAE,OAAO,KAAK,CAAC;IAE9B,MAAM,WAAW,GAAG,UAAU,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,2BAA2B,CAAC,CAAC;IACxE,KAAK,MAAM,KAAK,IAAI,WAAW,EAAE,CAAC;QAChC,MAAM,UAAU,GAAG,UAAU,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,YAAY,CAAC,CAAC;QACtD,MAAM,QAAQ,GAAG,UAAU,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,UAAU,CAAC,CAAC;QAClD,IAAI,UAAU,IAAI,QAAQ,EAAE,CAAC;YAC3B,KAAK,CAAC,IAAI,CAAC,EAAE,UAAU,EAAE,QAAQ,EAAE,CAAC,CAAC;QACvC,CAAC;IACH,CAAC;IAED,OAAO,KAAK,CAAC;AACf,CAAC;AAED;;GAEG;AACH,SAAS,mBAAmB,CAAC,OAAe;IAC1C,MAAM,MAAM,GAAuB,EAAE,CAAC;IACtC,MAAM,WAAW,GAAG,OAAO,CAAC,KAAK,CAAC,8CAA8C,CAAC,CAAC;IAClF,IAAI,CAAC,WAAW;QAAE,OAAO,MAAM,CAAC;IAEhC,MAAM,YAAY,GAAG,WAAW,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,6CAA6C,CAAC,CAAC;IAC5F,KAAK,MAAM,KAAK,IAAI,YAAY,EAAE,CAAC;QACjC,MAAM,QAAQ,GAAG,UAAU,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,UAAU,CAAC,CAAC;QAClD,MAAM,SAAS,GAAG,cAAc,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,WAAW,CAAC,CAAC;QACxD,IAAI,QAAQ,IAAI,SAAS,EAAE,CAAC;YAC1B,MAAM,CAAC,IAAI,CAAC,EAAE,QAAQ,EAAE,SAAS,EAAE,CAAC,CAAC;QACvC,CAAC;IACH,CAAC;IAED,OAAO,MAAM,CAAC;AAChB,CAAC;AAED;;;;;;;;;;;;;;;;GAgBG;AACH,SAAS,YAAY,CAAC,OAAe;IACnC,MAAM,OAAO,GAAgB,EAAE,CAAC;IAChC,MAAM,YAAY,GAAG,OAAO,CAAC,KAAK,CAAC,gCAAgC,CAAC,CAAC;IAErE,MAAM,UAAU,GAAG,CAAC,MAA0B,EAAE,IAAY,EAAE,EAAE;QAC9D,MAAM,SAAS,GAAG,cAAc,CAAC,IAAI,EAAE,WAAW,CAAC,CAAC;QACpD,MAAM,QAAQ,GAAG,UAAU,CAAC,IAAI,EAAE,UAAU,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAG,cAAc,CAAC,IAAI,EAAE,QAAQ,CAAC,CAAC;QAC9C,MAAM,EAAE,GAAG,cAAc,CAAC,IAAI,EAAE,IAAI,CAAC,CAAC;QACtC,MAAM,MAAM,GAAG,cAAc,CAAC,IAAI,EAAE,QAAQ,CAAC,CAAC;QAC9C,MAAM,YAAY,GAAG,MAAM,KAAK,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,gBAAgB,CAAC,MAAM,CAAC,CAAC;QAEjF,IAAI,MAAM,IAAI,SAAS,IAAI,QAAQ,IAAI,MAAM,IAAI,EAAE,IAAI,YAAY,KAAK,SAAS,EAAE,CAAC;YAClF,OAAO,CAAC,IAAI,CAAC;gBACX,MAAM,EAAE,MAAM,CAAC,WAAW,EAAE,KAAK,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,KAAK;gBACtD,SAAS;gBACT,QAAQ;gBACR,MAAM;gBACN,EAAE;gBACF,MAAM,EAAE,YAAY;aACrB,CAAC,CAAC;QACL,CAAC;IACH,CAAC,CAAC;IAEF,IAAI,YAAY,EAAE,CAAC;QACjB,gFAAgF;QAChF,KAAK,MAAM,KAAK,IAAI,YAAY,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,+BAA+B,CAAC,EAAE,CAAC;YAC9E,UAAU,CAAC,cAAc,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,QAAQ,CAAC,EAAE,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QAC3D,CAAC;IACH,CAAC;SAAM,CAAC;QACN,sEAAsE;QACtE,oCAAoC;QACpC,KAAK,MAAM,KAAK,IAAI,OAAO,CAAC,QAAQ,CAClC,8DAA8D,CAC/D,EAAE,CAAC;YACF,UAAU,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QACjC,CAAC;IACH,CAAC;IAED,OAAO,OAAO,CAAC;AACjB,CAAC"}

package/dist/reader.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"reader.d.ts","sourceRoot":"","sources":["../src/reader.ts"],"names":[],"mappings":"AAaA,OAAO,KAAK,EACV,UAAU,EAgBX,MAAM,YAAY,CAAC;AAyIpB;;;;;;GAMG;AACH,wBAAsB,OAAO,CAC3B,IAAI,EAAE,IAAI,GAAG,IAAI,GAAG,WAAW,GAAG,UAAU,EAC5C,MAAM,CAAC,EAAE;IAAE,eAAe,CAAC,EAAE,MAAM,CAAC;IAAC,UAAU,CAAC,EAAE,MAAM,CAAC;IAAC,gBAAgB,CAAC,EAAE,MAAM,CAAA;CAAE,GACpF,OAAO,CAAC,UAAU,CAAC,CA4CrB"}
+{"version":3,"file":"reader.d.ts","sourceRoot":"","sources":["../src/reader.ts"],"names":[],"mappings":"AAYA,OAAO,KAAK,EACV,UAAU,EASX,MAAM,YAAY,CAAC;AAyIpB;;;;;;GAMG;AACH,wBAAsB,OAAO,CAC3B,IAAI,EAAE,IAAI,GAAG,IAAI,GAAG,WAAW,GAAG,UAAU,EAC5C,MAAM,CAAC,EAAE;IAAE,eAAe,CAAC,EAAE,MAAM,CAAC;IAAC,UAAU,CAAC,EAAE,MAAM,CAAC;IAAC,gBAAgB,CAAC,EAAE,MAAM,CAAA;CAAE,GACpF,OAAO,CAAC,UAAU,CAAC,CA4CrB"}

package/dist/reader.js

@@ -7,9 +7,8 @@
  * Parses .bcfzip files into BCFProject structure
  */
 import JSZip from 'jszip';
-import { parseComponents } from './reader-components.js';
 import { extractElement, unescapeXml, parseLabels, parseXsBoolean } from './xml-text.js';
-import { parseFiniteFloat } from './numeric.js';
+import { parseViewpointContent } from './reader-viewpoint-content.js';
 /**
  * Resource caps guarding against a malicious (zip-bomb) .bcfzip: a tiny
  * compressed archive that expands to gigabytes, or one with a pathological
@@ -207,6 +206,18 @@ async function readTopics(zip, budget, versionId) {
         try {
             const topic = await readTopic(zip, topicGuid, budget, versionId);
             if (topic) {
+                // A second topic folder whose internal Topic/@Guid collides with one
+                // already parsed must not silently overwrite it in the map -- that
+                // would drop a whole topic with no signal the caller could ever act
+                // on (#3960). Keep the first occurrence (folder set iteration order
+                // is insertion order, so this is deterministic) and warn, the same
+                // way every other "skip this piece, keep going" decision in this
+                // function is already reported.
+                if (topics.has(topic.guid)) {
+                    console.warn(`Duplicate topic Guid ${topic.guid}: folder "${topicGuid}" collides with an ` +
+                        `already-read topic folder and is being dropped. Keeping the first one read.`);
+                    continue;
+                }
                 topics.set(topic.guid, topic);
             }
         }
@@ -253,6 +264,7 @@ async function readTopic(zip, topicFolder, budget, versionId) {
     const description = extractElement(topicContent, 'Description');
     const priority = extractElement(topicContent, 'Priority');
     const index = extractElement(topicContent, 'Index');
+    const parsedIndex = index ? Number.parseInt(index, 10) : undefined;
     const creationDate = extractElement(topicContent, 'CreationDate'); // required no-default in markup.xsd; leave undefined, don't fabricate
     const creationAuthor = extractElement(topicContent, 'CreationAuthor'); // same: required no-default, don't fabricate 'Unknown'
     const modifiedDate = extractElement(topicContent, 'ModifiedDate');
@@ -285,7 +297,9 @@ async function readTopic(zip, topicFolder, budget, versionId) {
         topicType,
         topicStatus,
         priority,
-        index: index ? parseInt(index, 10) : undefined,
+        // Preserve integer parsing: accepting fractions makes re-export throw.
+        // Guard malformed/overflowed tokens instead of storing NaN/Infinity (#3961).
+        index: Number.isFinite(parsedIndex) ? parsedIndex : undefined,
         creationDate,
         creationAuthor,
         modifiedDate,
@@ -637,241 +651,6 @@ async function parseViewpoints(zip, topicFolder, markupContent, budget, versionI
     }
     return viewpoints;
 }
-/**
- * Parse viewpoint XML content
- */
-function parseViewpointContent(content, versionId) {
-    // Extract viewpoint GUID from root element (Guid can be anywhere in the tag)
-    const guidMatch = content.match(/<VisualizationInfo[^>]+Guid="([^"]+)"/);
-    const guid = guidMatch?.[1] || crypto.randomUUID?.() || `vp-${Date.now()}`;
-    // Parse perspective camera
-    const perspectiveCamera = parsePerspectiveCamera(content);
-    // Parse orthogonal camera
-    const orthogonalCamera = parseOrthogonalCamera(content);
-    // Parse components
-    const components = parseComponents(content, versionId);
-    // Parse lines
-    const lines = parseLines(content);
-    // Parse clipping planes
-    const clippingPlanes = parseClippingPlanes(content);
-    // Parse bitmaps
-    const bitmaps = parseBitmaps(content);
-    return {
-        guid,
-        perspectiveCamera,
-        orthogonalCamera,
-        components,
-        lines: lines.length > 0 ? lines : undefined,
-        clippingPlanes: clippingPlanes.length > 0 ? clippingPlanes : undefined,
-        bitmaps: bitmaps.length > 0 ? bitmaps : undefined,
-    };
-}
-/**
- * Parse perspective camera from viewpoint content
- */
-function parsePerspectiveCamera(content) {
-    const match = content.match(/<PerspectiveCamera>([\s\S]*?)<\/PerspectiveCamera>/);
-    if (!match)
-        return undefined;
-    const cameraContent = match[1];
-    const viewPoint = parsePoint(cameraContent, 'CameraViewPoint');
-    const direction = parseDirection(cameraContent, 'CameraDirection');
-    const upVector = parseDirection(cameraContent, 'CameraUpVector');
-    const fieldOfView = extractElement(cameraContent, 'FieldOfView');
-    if (!viewPoint || !direction || !upVector || !fieldOfView) {
-        return undefined;
-    }
-    // Same treatment as the coordinates: an unusable scalar is a missing one.
-    // `fieldOfView` is converted to radians and handed to the viewer camera,
-    // and the parser already drops the whole camera when the element is absent.
-    const fov = parseFiniteFloat(fieldOfView);
-    if (fov === undefined)
-        return undefined;
-    return {
-        cameraViewPoint: viewPoint,
-        cameraDirection: direction,
-        cameraUpVector: upVector,
-        fieldOfView: fov,
-        ...parseAspectRatio(cameraContent),
-    };
-}
-/**
- * Parse the optional `<AspectRatio>` of either camera type.
- *
- * BCF 3.0's visinfo.xsd makes `AspectRatio` (a `PositiveDouble`) a REQUIRED
- * child of both camera types; BCF 2.1 has no such element. It is read here for
- * both, since the element's presence — not the archive's declared version — is
- * what says whether there is a value to keep.
- *
- * Reading it matters beyond fidelity: the writer refuses to emit a 3.0 camera
- * without one, so a 3.0 archive from another tool could otherwise be read and
- * then not written back. Returned as a spread-able partial so an absent or
- * unusable value leaves the property off entirely rather than setting it to
- * `undefined`, and a non-positive value is dropped rather than carried into
- * output the schema would reject.
- */
-function parseAspectRatio(cameraContent) {
-    const raw = extractElement(cameraContent, 'AspectRatio');
-    if (!raw)
-        return {};
-    const value = parseFiniteFloat(raw);
-    if (value === undefined || !(value > 0))
-        return {};
-    return { aspectRatio: value };
-}
-/**
- * Parse orthogonal camera from viewpoint content
- */
-function parseOrthogonalCamera(content) {
-    const match = content.match(/<OrthogonalCamera>([\s\S]*?)<\/OrthogonalCamera>/);
-    if (!match)
-        return undefined;
-    const cameraContent = match[1];
-    const viewPoint = parsePoint(cameraContent, 'CameraViewPoint');
-    const direction = parseDirection(cameraContent, 'CameraDirection');
-    const upVector = parseDirection(cameraContent, 'CameraUpVector');
-    const viewToWorldScale = extractElement(cameraContent, 'ViewToWorldScale');
-    if (!viewPoint || !direction || !upVector || !viewToWorldScale) {
-        return undefined;
-    }
-    // `viewToWorldScale` becomes the orthographic half-height, which is the
-    // value `getOrthoSize()` hands back into a saved viewpoint — so a
-    // non-finite one persists past the session if it is allowed in (#2461).
-    const scale = parseFiniteFloat(viewToWorldScale);
-    if (scale === undefined)
-        return undefined;
-    return {
-        cameraViewPoint: viewPoint,
-        cameraDirection: direction,
-        cameraUpVector: upVector,
-        viewToWorldScale: scale,
-        ...parseAspectRatio(cameraContent),
-    };
-}
-/**
- * Parse a 3D point from XML
- */
-function parsePoint(content, elementName) {
-    const match = content.match(new RegExp(`<${elementName}>([\\s\\S]*?)<\\/${elementName}>`));
-    if (!match)
-        return undefined;
-    const x = extractElement(match[1], 'X');
-    const y = extractElement(match[1], 'Y');
-    const z = extractElement(match[1], 'Z');
-    if (x === undefined || y === undefined || z === undefined) {
-        return undefined;
-    }
-    // A coordinate that is not a real number is treated as a missing one.
-    // `parseFloat` has no out-of-band failure value — `"NaN"` parses to `NaN`
-    // and the well-formed literal `"1e999"` parses to `Infinity` — and from here
-    // the value reaches `Camera.setPosition`/`setTarget`, which store a pose
-    // verbatim by design. Once stored, a single non-finite coordinate spreads
-    // across the whole pose on the next gesture. Rejecting at the file boundary
-    // means it never gets there, and it costs no new branch: every caller
-    // already drops the thing it was parsing when a coordinate is missing
-    // (#2466).
-    const px = parseFiniteFloat(x);
-    const py = parseFiniteFloat(y);
-    const pz = parseFiniteFloat(z);
-    if (px === undefined || py === undefined || pz === undefined) {
-        return undefined;
-    }
-    return { x: px, y: py, z: pz };
-}
-/**
- * Parse a 3D direction from XML
- */
-function parseDirection(content, elementName) {
-    return parsePoint(content, elementName);
-}
-/**
- * Parse lines
- */
-function parseLines(content) {
-    const lines = [];
-    const linesMatch = content.match(/<Lines>([\s\S]*?)<\/Lines>/);
-    if (!linesMatch)
-        return lines;
-    const lineMatches = linesMatch[1].matchAll(/<Line>([\s\S]*?)<\/Line>/g);
-    for (const match of lineMatches) {
-        const startPoint = parsePoint(match[1], 'StartPoint');
-        const endPoint = parsePoint(match[1], 'EndPoint');
-        if (startPoint && endPoint) {
-            lines.push({ startPoint, endPoint });
-        }
-    }
-    return lines;
-}
-/**
- * Parse clipping planes
- */
-function parseClippingPlanes(content) {
-    const planes = [];
-    const planesMatch = content.match(/<ClippingPlanes>([\s\S]*?)<\/ClippingPlanes>/);
-    if (!planesMatch)
-        return planes;
-    const planeMatches = planesMatch[1].matchAll(/<ClippingPlane>([\s\S]*?)<\/ClippingPlane>/g);
-    for (const match of planeMatches) {
-        const location = parsePoint(match[1], 'Location');
-        const direction = parseDirection(match[1], 'Direction');
-        if (location && direction) {
-            planes.push({ location, direction });
-        }
-    }
-    return planes;
-}
-/**
- * Parse bitmaps
- *
- * The two BCF versions diverge in shape (see writer.ts's writeBitmap/
- * writeViewpointFiles for the write side of this):
- * - BCF 3.0: entries sit inside a `<Bitmaps>` wrapper, and the per-entry
- *   format element is named `<Format>`. No tag inside an entry shares the
- *   entry's own name, so a plain non-greedy `<Bitmap>...</Bitmap>` match
- *   is unambiguous.
- * - BCF 2.1: entries sit DIRECTLY under `<VisualizationInfo>` (no wrapper),
- *   and the format element is confusingly also named `<Bitmap>`, nested one
- *   level inside the entry (`<Bitmap><Bitmap>PNG</Bitmap><Reference>...`).
- *   A naive non-greedy `<Bitmap>...</Bitmap>` match on that shape terminates
- *   at the FIRST `</Bitmap>` it sees -- the inner format tag's closing tag,
- *   not the entry's -- and silently drops the rest of the entry. It must be
- *   matched with an explicit two-level pattern instead.
- */
-function parseBitmaps(content) {
-    const bitmaps = [];
-    const bitmapsMatch = content.match(/<Bitmaps>([\s\S]*?)<\/Bitmaps>/);
-    const pushBitmap = (format, body) => {
-        const reference = extractElement(body, 'Reference');
-        const location = parsePoint(body, 'Location');
-        const normal = parseDirection(body, 'Normal');
-        const up = parseDirection(body, 'Up');
-        const height = extractElement(body, 'Height');
-        if (format && reference && location && normal && up && height) {
-            bitmaps.push({
-                format: format.toUpperCase() === 'JPG' ? 'JPG' : 'PNG',
-                reference,
-                location,
-                normal,
-                up,
-                height: parseFloat(height),
-            });
-        }
-    };
-    if (bitmapsMatch) {
-        // BCF 3.0 shape: <Bitmaps><Bitmap><Format>...</Format>...</Bitmap>...</Bitmaps>
-        for (const match of bitmapsMatch[1].matchAll(/<Bitmap>([\s\S]*?)<\/Bitmap>/g)) {
-            pushBitmap(extractElement(match[1], 'Format'), match[1]);
-        }
-    }
-    else {
-        // BCF 2.1 shape: <Bitmap><Bitmap>PNG</Bitmap>...</Bitmap>, unwrapped,
-        // directly under VisualizationInfo.
-        for (const match of content.matchAll(/<Bitmap>\s*<Bitmap>([\s\S]*?)<\/Bitmap>([\s\S]*?)<\/Bitmap>/g)) {
-            pushBitmap(match[1], match[2]);
-        }
-    }
-    return bitmaps;
-}
 /**
  * Convert Uint8Array to base64 string
  */