ts-visio 1.0.2 → 1.2.0

package/dist/ShapeReader.d.ts

@@ -5,5 +5,17 @@ export declare class ShapeReader {
     private parser;
     constructor(pkg: VisioPackage);
     readShapes(path: string): VisioShape[];
+    /**
+     * Returns every shape on the page flattened into a single array,
+     * including shapes nested inside groups at any depth.
+     */
+    readAllShapes(path: string): VisioShape[];
+    /**
+     * Find a single shape by ID anywhere in the page tree (including nested groups).
+     * Returns undefined if not found.
+     */
+    readShapeById(path: string, shapeId: string): VisioShape | undefined;
+    private gatherShapes;
+    private findShapeById;
     private parseShape;
 }

package/dist/ShapeReader.js

@@ -20,12 +20,68 @@ class ShapeReader {
             return [];
         }
         const parsed = this.parser.parse(content);
-        // Supports PageContents -> Shapes -> Shape or just Shapes -> Shape depending on structure
         const shapesData = parsed.PageContents?.Shapes?.Shape;
         if (!shapesData)
             return [];
-        const shapesArray = (0, VisioParsers_1.asArray)(shapesData);
-        return shapesArray.map(s => this.parseShape(s));
+        return (0, VisioParsers_1.asArray)(shapesData).map(s => this.parseShape(s));
+    }
+    /**
+     * Returns every shape on the page flattened into a single array,
+     * including shapes nested inside groups at any depth.
+     */
+    readAllShapes(path) {
+        let content;
+        try {
+            content = this.pkg.getFileText(path);
+        }
+        catch {
+            return [];
+        }
+        const parsed = this.parser.parse(content);
+        const shapesData = parsed.PageContents?.Shapes?.Shape;
+        if (!shapesData)
+            return [];
+        const result = [];
+        this.gatherShapes((0, VisioParsers_1.asArray)(shapesData), result);
+        return result;
+    }
+    /**
+     * Find a single shape by ID anywhere in the page tree (including nested groups).
+     * Returns undefined if not found.
+     */
+    readShapeById(path, shapeId) {
+        let content;
+        try {
+            content = this.pkg.getFileText(path);
+        }
+        catch {
+            return undefined;
+        }
+        const parsed = this.parser.parse(content);
+        const shapesData = parsed.PageContents?.Shapes?.Shape;
+        if (!shapesData)
+            return undefined;
+        return this.findShapeById((0, VisioParsers_1.asArray)(shapesData), shapeId);
+    }
+    gatherShapes(rawShapes, result) {
+        for (const s of rawShapes) {
+            result.push(this.parseShape(s));
+            if (s.Shapes?.Shape) {
+                this.gatherShapes((0, VisioParsers_1.asArray)(s.Shapes.Shape), result);
+            }
+        }
+    }
+    findShapeById(rawShapes, shapeId) {
+        for (const s of rawShapes) {
+            if (s['@_ID'] === shapeId)
+                return this.parseShape(s);
+            if (s.Shapes?.Shape) {
+                const found = this.findShapeById((0, VisioParsers_1.asArray)(s.Shapes.Shape), shapeId);
+                if (found)
+                    return found;
+            }
+        }
+        return undefined;
     }
     parseShape(s) {
         const shape = {

package/dist/VisioDocument.d.ts

@@ -17,5 +17,15 @@ export declare class VisioDocument {
      * Set a background page for a foreground page
      */
     setBackgroundPage(foregroundPage: Page, backgroundPage: Page): Promise<void>;
+    /**
+     * Find a page by name. Returns undefined if no page with that name exists.
+     */
+    getPage(name: string): Page | undefined;
+    /**
+     * Delete a page from the document.
+     * Removes the page XML, its relationships, the Content Types entry,
+     * and any BackPage references from other pages.
+     */
+    deletePage(page: Page): Promise<void>;
     save(filename?: string): Promise<Buffer>;
 }

package/dist/VisioDocument.js

@@ -67,6 +67,7 @@ class VisioDocument {
         const pageStub = {
             ID: newId,
             Name: name,
+            xmlPath: `visio/pages/page${newId}.xml`,
             Shapes: [],
             Connects: []
         };
@@ -76,10 +77,12 @@ class VisioDocument {
         if (!this._pageCache) {
             const internalPages = this.pageManager.load();
             this._pageCache = internalPages.map(p => {
-                // Adapter for VisioPage interface
                 const pageStub = {
                     ID: p.id.toString(),
                     Name: p.name,
+                    // Thread the relationship-resolved path so that loaded files
+                    // with non-sequential page filenames are handled correctly.
+                    xmlPath: p.xmlPath,
                     Shapes: [],
                     Connects: [],
                     isBackground: p.isBackground,
@@ -99,6 +102,7 @@ class VisioDocument {
         const pageStub = {
             ID: newId,
             Name: name,
+            xmlPath: `visio/pages/page${newId}.xml`,
             Shapes: [],
             Connects: [],
             isBackground: true
@@ -112,6 +116,21 @@ class VisioDocument {
         await this.pageManager.setBackgroundPage(foregroundPage.id, backgroundPage.id);
         this._pageCache = null;
     }
+    /**
+     * Find a page by name. Returns undefined if no page with that name exists.
+     */
+    getPage(name) {
+        return this.pages.find(p => p.name === name);
+    }
+    /**
+     * Delete a page from the document.
+     * Removes the page XML, its relationships, the Content Types entry,
+     * and any BackPage references from other pages.
+     */
+    async deletePage(page) {
+        await this.pageManager.deletePage(page.id);
+        this._pageCache = null;
+    }
     async save(filename) {
         return this.pkg.save(filename);
     }

package/dist/VisioPackage.d.ts

@@ -6,5 +6,6 @@ export declare class VisioPackage {
     load(buffer: Buffer | ArrayBuffer | Uint8Array): Promise<void>;
     updateFile(path: string, content: string | Buffer): void;
     save(filename?: string): Promise<Buffer>;
+    removeFile(path: string): void;
     getFileText(path: string): string;
 }

package/dist/VisioPackage.js

@@ -98,6 +98,13 @@ class VisioPackage {
         }
         return buffer;
     }
+    removeFile(path) {
+        if (!this.zip) {
+            throw new Error("Package not loaded");
+        }
+        this._files.delete(path);
+        this.zip.remove(path);
+    }
     getFileText(path) {
         const content = this._files.get(path);
         if (content === undefined) {

package/dist/core/MasterManager.js

@@ -24,7 +24,7 @@ class MasterManager {
         const parsed = this.parser.parse(content);
         let masterNodes = parsed.Masters ? parsed.Masters.Master : [];
         if (!Array.isArray(masterNodes)) {
-            masterNodes = [masterNodes];
+            masterNodes = masterNodes ? [masterNodes] : [];
         }
         this.masters = masterNodes.map((node) => ({
             id: node['@_ID'],

package/dist/core/MediaConstants.js

@@ -8,9 +8,18 @@ exports.MIME_TYPES = {
     'jpeg': 'image/jpeg',
     'gif': 'image/gif',
     'bmp': 'image/bmp',
-    'tiff': 'image/tiff'
+    'tiff': 'image/tiff',
+    'emf': 'image/emf',
+    'wmf': 'image/wmf',
+    'svg': 'image/svg+xml',
+    'ico': 'image/x-icon',
+    'tif': 'image/tiff',
+    'wdp': 'image/vnd.ms-photo'
 };
 exports.SUPPORTED_IMAGE_EXTENSIONS = Object.keys(exports.MIME_TYPES);
+// Extensions that are decoded as UTF-8 text; everything else is treated as binary
+// to prevent corruption of EMF, WMF, OLE, and other binary assets in .vsdx archives.
+const TEXT_EXTENSIONS = new Set(['xml', 'rels']);
 function isBinaryExtension(ext) {
-    return exports.SUPPORTED_IMAGE_EXTENSIONS.includes(ext.toLowerCase());
+    return !TEXT_EXTENSIONS.has(ext.toLowerCase());
 }

package/dist/core/MediaManager.js

@@ -1,30 +1,23 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.MediaManager = void 0;
-const fast_xml_parser_1 = require("fast-xml-parser");
 const MediaConstants_1 = require("./MediaConstants");
+const XmlHelper_1 = require("../utils/XmlHelper");
+const node_crypto_1 = require("node:crypto");
 class MediaManager {
     constructor(pkg) {
         this.pkg = pkg;
         this.deduplicationMap = new Map(); // hash -> paths
         this.indexed = false;
-        this.parser = new fast_xml_parser_1.XMLParser({
-            ignoreAttributes: false,
-            attributeNamePrefix: "@_"
-        });
-        this.builder = new fast_xml_parser_1.XMLBuilder({
-            ignoreAttributes: false,
-            attributeNamePrefix: "@_",
-            format: true
-        });
+        this.parser = (0, XmlHelper_1.createXmlParser)();
+        this.builder = (0, XmlHelper_1.createXmlBuilder)();
     }
     ensureIndex() {
         if (this.indexed)
             return;
-        const crypto = require('crypto');
         for (const [path, content] of this.pkg.filesMap.entries()) {
             if (path.startsWith('visio/media/') && Buffer.isBuffer(content)) {
-                const hash = crypto.createHash('sha1').update(content).digest('hex');
+                const hash = (0, node_crypto_1.createHash)('sha1').update(content).digest('hex');
                 // Store relative path suitable for relationships
                 const filename = path.split('/').pop();
                 if (filename) {
@@ -39,8 +32,7 @@ class MediaManager {
     }
     addMedia(name, data) {
         this.ensureIndex();
-        const crypto = require('crypto');
-        const hash = crypto.createHash('sha1').update(data).digest('hex');
+        const hash = (0, node_crypto_1.createHash)('sha1').update(data).digest('hex');
         if (this.deduplicationMap.has(hash)) {
             return this.deduplicationMap.get(hash);
         }
@@ -73,14 +65,14 @@ class MediaManager {
             parsed.Types.Default = [parsed.Types.Default];
         }
         const defaults = parsed.Types.Default;
-        const exists = defaults.some((d) => d['@_Extension']?.toLowerCase() === extension);
+        const exists = defaults.some((d) => d['@_Extension']?.toLowerCase() === extension.toLowerCase());
         if (!exists) {
             defaults.push({
                 '@_Extension': extension,
                 '@_ContentType': this.getContentType(extension)
             });
             // Array reference is already mutating object
-            const newXml = this.builder.build(parsed);
+            const newXml = (0, XmlHelper_1.buildXml)(this.builder, parsed);
             this.pkg.updateFile(ctPath, newXml);
         }
     }

package/dist/core/PageManager.d.ts

@@ -5,7 +5,7 @@ export interface PageEntry {
     relId: string;
     xmlPath: string;
     isBackground: boolean;
-    backPageId?: number;
+    backPageId?: string;
 }
 export declare class PageManager {
     private pkg;
@@ -21,6 +21,13 @@ export declare class PageManager {
      * Create a background page
      */
     createBackgroundPage(name: string): Promise<string>;
+    /**
+     * Delete a page and clean up all associated XML entries.
+     * Removes the page file, its .rels file, the entry in pages.xml,
+     * the relationship in pages.xml.rels, the Content Types override,
+     * and any BackPage references from other pages that pointed to it.
+     */
+    deletePage(pageId: string): Promise<void>;
     /**
      * Set a background page for a foreground page
      */

package/dist/core/PageManager.js

@@ -1,23 +1,16 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.PageManager = void 0;
-const fast_xml_parser_1 = require("fast-xml-parser");
 const VisioConstants_1 = require("./VisioConstants");
 const RelsManager_1 = require("./RelsManager");
+const XmlHelper_1 = require("../utils/XmlHelper");
 class PageManager {
     constructor(pkg) {
         this.pkg = pkg;
         this.pages = [];
         this.loaded = false;
-        this.parser = new fast_xml_parser_1.XMLParser({
-            ignoreAttributes: false,
-            attributeNamePrefix: "@_"
-        });
-        this.builder = new fast_xml_parser_1.XMLBuilder({
-            ignoreAttributes: false,
-            attributeNamePrefix: "@_",
-            format: true
-        });
+        this.parser = (0, XmlHelper_1.createXmlParser)();
+        this.builder = (0, XmlHelper_1.createXmlBuilder)();
         this.relsManager = new RelsManager_1.RelsManager(pkg);
     }
     load(force = false) {
@@ -72,7 +65,7 @@ class PageManager {
             const bgAttr = node['@_Background'];
             const isBackground = bgAttr === 'true' || bgAttr === '1' || bgAttr === true || bgAttr === 1;
             const backPageAttr = node['@_BackPage'];
-            const backPageId = backPageAttr ? parseInt(backPageAttr.toString()) : undefined;
+            const backPageId = backPageAttr ? backPageAttr.toString() : undefined;
             return {
                 id: parseInt(node['@_ID']),
                 name: node['@_Name'],
@@ -127,7 +120,7 @@ class PageManager {
             '@_PartName': `/${relativePath}`,
             '@_ContentType': VisioConstants_1.CONTENT_TYPES.VISIO_PAGE
         });
-        this.pkg.updateFile(ctPath, this.builder.build(parsedCt));
+        this.pkg.updateFile(ctPath, (0, XmlHelper_1.buildXml)(this.builder, parsedCt));
         // 4. Update Relationships (pages.xml -> new page file)
         // Source is "visio/pages/pages.xml", Target is "page{ID}.xml" (relative to source dir)
         const rId = await this.relsManager.ensureRelationship('visio/pages/pages.xml', fileName, VisioConstants_1.RELATIONSHIP_TYPES.PAGE);
@@ -142,9 +135,10 @@ class PageManager {
         parsedPages.Pages.Page.push({
             '@_ID': newId.toString(),
             '@_Name': name,
+            '@_NameU': name,
             'Rel': { '@_r:id': rId }
         });
-        this.pkg.updateFile(pagesPath, this.builder.build(parsedPages));
+        this.pkg.updateFile(pagesPath, (0, XmlHelper_1.buildXml)(this.builder, parsedPages));
         // Reload to include new page
         this.load(true);
         return newId.toString();
@@ -177,6 +171,7 @@ class PageManager {
         <Cell N="PageDrawSizeType" V="0"/>
     </PageSheet>
     <Shapes/>
+    <Connects/>
 </PageContents>`;
         this.pkg.updateFile(relativePath, pageContent);
         // 3. Update Content Types
@@ -191,7 +186,7 @@ class PageManager {
             '@_PartName': `/${relativePath}`,
             '@_ContentType': VisioConstants_1.CONTENT_TYPES.VISIO_PAGE
         });
-        this.pkg.updateFile(ctPath, this.builder.build(parsedCt));
+        this.pkg.updateFile(ctPath, (0, XmlHelper_1.buildXml)(this.builder, parsedCt));
         // 4. Update Relationships
         const rId = await this.relsManager.ensureRelationship('visio/pages/pages.xml', fileName, VisioConstants_1.RELATIONSHIP_TYPES.PAGE);
         // 5. Update Pages Index with Background="true"
@@ -205,13 +200,77 @@ class PageManager {
         parsedPages.Pages.Page.push({
             '@_ID': newId.toString(),
             '@_Name': name,
+            '@_NameU': name,
             '@_Background': '1', // Use '1' for boolean true attribute
             'Rel': { '@_r:id': rId }
         });
-        this.pkg.updateFile(pagesPath, this.builder.build(parsedPages));
+        this.pkg.updateFile(pagesPath, (0, XmlHelper_1.buildXml)(this.builder, parsedPages));
         this.load(true);
         return newId.toString();
     }
+    /**
+     * Delete a page and clean up all associated XML entries.
+     * Removes the page file, its .rels file, the entry in pages.xml,
+     * the relationship in pages.xml.rels, the Content Types override,
+     * and any BackPage references from other pages that pointed to it.
+     */
+    async deletePage(pageId) {
+        this.load();
+        const page = this.pages.find(p => p.id.toString() === pageId);
+        if (!page)
+            throw new Error(`Page ${pageId} not found`);
+        const pageFileName = page.xmlPath.split('/').pop(); // e.g. "page2.xml"
+        // 1. Remove the page XML file
+        try {
+            this.pkg.removeFile(page.xmlPath);
+        }
+        catch { /* already gone */ }
+        // 2. Remove the page's .rels file if it exists
+        const pageRelsPath = `visio/pages/_rels/${pageFileName}.rels`;
+        try {
+            this.pkg.removeFile(pageRelsPath);
+        }
+        catch { /* no rels file is fine */ }
+        // 3. Remove entry from pages.xml (and strip BackPage refs pointing to this page)
+        const pagesPath = 'visio/pages/pages.xml';
+        const pagesContent = this.pkg.getFileText(pagesPath);
+        const parsedPages = this.parser.parse(pagesContent);
+        let pageNodes = parsedPages.Pages.Page;
+        if (!Array.isArray(pageNodes))
+            pageNodes = pageNodes ? [pageNodes] : [];
+        // Remove deleted page and clean up BackPage refs on remaining pages
+        parsedPages.Pages.Page = pageNodes
+            .filter((n) => n['@_ID'] !== pageId)
+            .map((n) => {
+            if (n['@_BackPage'] === pageId) {
+                const copy = { ...n };
+                delete copy['@_BackPage'];
+                return copy;
+            }
+            return n;
+        });
+        this.pkg.updateFile(pagesPath, (0, XmlHelper_1.buildXml)(this.builder, parsedPages));
+        // 4. Remove relationship from pages.xml.rels
+        const pagesRelsPath = 'visio/pages/_rels/pages.xml.rels';
+        const pagesRelsContent = this.pkg.getFileText(pagesRelsPath);
+        const parsedPagesRels = this.parser.parse(pagesRelsContent);
+        let rels = parsedPagesRels.Relationships?.Relationship;
+        if (!Array.isArray(rels))
+            rels = rels ? [rels] : [];
+        parsedPagesRels.Relationships.Relationship = rels.filter((r) => r['@_Id'] !== page.relId);
+        this.pkg.updateFile(pagesRelsPath, (0, XmlHelper_1.buildXml)(this.builder, parsedPagesRels));
+        // 5. Remove Content Types override for the page file
+        const ctPath = '[Content_Types].xml';
+        const ctContent = this.pkg.getFileText(ctPath);
+        const parsedCt = this.parser.parse(ctContent);
+        let overrides = parsedCt.Types.Override;
+        if (!Array.isArray(overrides))
+            overrides = overrides ? [overrides] : [];
+        parsedCt.Types.Override = overrides.filter((o) => !o['@_PartName']?.endsWith(pageFileName));
+        this.pkg.updateFile(ctPath, (0, XmlHelper_1.buildXml)(this.builder, parsedCt));
+        // 6. Reload the page list to reflect the deletion
+        this.load(true);
+    }
     /**
      * Set a background page for a foreground page
      */
@@ -237,7 +296,7 @@ class PageManager {
         if (fgNode) {
             fgNode['@_BackPage'] = backgroundPageId.toString();
         }
-        this.pkg.updateFile(pagesPath, this.builder.build(parsedPages));
+        this.pkg.updateFile(pagesPath, (0, XmlHelper_1.buildXml)(this.builder, parsedPages));
         this.load(true);
     }
 }

package/dist/core/RelsManager.js

@@ -1,20 +1,13 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.RelsManager = void 0;
-const fast_xml_parser_1 = require("fast-xml-parser");
 const VisioConstants_1 = require("./VisioConstants");
+const XmlHelper_1 = require("../utils/XmlHelper");
 class RelsManager {
     constructor(pkg) {
         this.pkg = pkg;
-        this.parser = new fast_xml_parser_1.XMLParser({
-            ignoreAttributes: false,
-            attributeNamePrefix: "@_"
-        });
-        this.builder = new fast_xml_parser_1.XMLBuilder({
-            ignoreAttributes: false,
-            attributeNamePrefix: "@_",
-            format: true
-        });
+        this.parser = (0, XmlHelper_1.createXmlParser)();
+        this.builder = (0, XmlHelper_1.createXmlBuilder)();
     }
     getRelsPath(partPath) {
         // file.xml -> _rels/file.xml.rels
@@ -66,7 +59,7 @@ class RelsManager {
             '@_Target': target
         });
         // Save back
-        const newXml = this.builder.build(parsed);
+        const newXml = (0, XmlHelper_1.buildXml)(this.builder, parsed);
         this.pkg.updateFile(relsPath, newXml);
         return newId;
     }

package/dist/index.d.ts

@@ -1,8 +1,10 @@
 export { VisioPackage } from './VisioPackage';
-export { PageManager } from './PageManager';
+export { PageManager } from './core/PageManager';
+export type { PageEntry } from './core/PageManager';
 export { ShapeReader } from './ShapeReader';
 export { ShapeModifier } from './ShapeModifier';
 export { VisioDocument } from './VisioDocument';
 export { Page } from './Page';
 export { Shape } from './Shape';
+export type { ShapeData, ShapeHyperlink } from './Shape';
 export * from './types/VisioTypes';

package/dist/index.js

@@ -17,7 +17,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.Shape = exports.Page = exports.VisioDocument = exports.ShapeModifier = exports.ShapeReader = exports.PageManager = exports.VisioPackage = void 0;
 var VisioPackage_1 = require("./VisioPackage");
 Object.defineProperty(exports, "VisioPackage", { enumerable: true, get: function () { return VisioPackage_1.VisioPackage; } });
-var PageManager_1 = require("./PageManager");
+var PageManager_1 = require("./core/PageManager");
 Object.defineProperty(exports, "PageManager", { enumerable: true, get: function () { return PageManager_1.PageManager; } });
 var ShapeReader_1 = require("./ShapeReader");
 Object.defineProperty(exports, "ShapeReader", { enumerable: true, get: function () { return ShapeReader_1.ShapeReader; } });

package/dist/shapes/ConnectorBuilder.js

@@ -55,9 +55,8 @@ class ConnectorBuilder {
                 }
             }
         };
-        const topShapes = parsed.PageContents.Shapes ?
-            (Array.isArray(parsed.PageContents.Shapes.Shape) ? parsed.PageContents.Shapes.Shape : [parsed.PageContents.Shapes.Shape])
-            : [];
+        const rawShapes = parsed.PageContents.Shapes?.Shape;
+        const topShapes = Array.isArray(rawShapes) ? rawShapes : rawShapes ? [rawShapes] : [];
         mapHierarchy(topShapes, null);
         return shapeHierarchy;
     }
@@ -120,16 +119,16 @@ class ConnectorBuilder {
                 { '@_N': 'ShapePermeableX', '@_V': '0' },
                 { '@_N': 'ShapePermeableY', '@_V': '0' },
                 { '@_N': 'ShapeRouteStyle', '@_V': '1' },
-                { '@_N': 'ConFixedCode', '@_V': '0' }
+                { '@_N': 'ConFixedCode', '@_V': '0' },
+                { '@_N': 'BeginArrow', '@_V': beginArrow || '0' },
+                { '@_N': 'BeginArrowSize', '@_V': '2' },
+                { '@_N': 'EndArrow', '@_V': endArrow || '0' },
+                { '@_N': 'EndArrowSize', '@_V': '2' }
             ],
             Section: [
                 (0, StyleHelpers_1.createLineSection)({
                     color: '#000000',
-                    weight: '0.01',
-                    beginArrow: beginArrow || '0',
-                    beginArrowSize: '2',
-                    endArrow: endArrow || '0',
-                    endArrowSize: '2'
+                    weight: '0.01'
                 }),
                 {
                     '@_N': 'Geometry',

package/dist/shapes/ContainerBuilder.js

@@ -64,18 +64,20 @@ class ContainerBuilder {
             textXform.Cell = [];
         if (!Array.isArray(textXform.Cell))
             textXform.Cell = [textXform.Cell];
-        const upsertCell = (name, val, unit) => {
+        // Extract the shape's current height for use as the static @_V on TxtPinY.
+        const heightVal = shape.Cell?.find((c) => c['@_N'] === 'Height')?.['@_V'] ?? '1';
+        const upsertCell = (name, formula, unit, val) => {
             const idx = textXform.Cell.findIndex((c) => c['@_N'] === name);
-            const cell = { '@_N': name, '@_V': val, '@_U': unit };
+            const cell = { '@_N': name, '@_V': val, '@_F': formula, '@_U': unit };
             if (idx >= 0)
                 textXform.Cell[idx] = cell;
             else
                 textXform.Cell.push(cell);
         };
-        // Move text to top of shape: TxtPinY = Height, TxtLocPinY = TxtHeight
-        // Removed redundant '*1' from formulas
-        upsertCell('TxtPinY', 'Height', 'DY');
-        upsertCell('TxtLocPinY', 'TxtHeight', 'DY');
+        // Move text to top of shape: formula drives dynamic sizing; @_V is the static initial value.
+        // TxtHeight is Visio-computed, so its static value is left as '0'.
+        upsertCell('TxtPinY', 'Height', 'DY', heightVal);
+        upsertCell('TxtLocPinY', 'TxtHeight', 'DY', '0');
     }
     static makeList(shape, direction = 'vertical') {
         // 1. Convert basic container to List

package/dist/shapes/ForeignShapeBuilder.js

@@ -8,14 +8,17 @@ class ForeignShapeBuilder {
             '@_NameU': `Sheet.${id}`,
             '@_Name': `Sheet.${id}`,
             '@_Type': 'Foreign',
-            ForeignData: { '@_r:id': rId },
+            ForeignData: {
+                '@_ForeignType': 'Bitmap',
+                Rel: { '@_r:id': rId }
+            },
             Cell: [
                 { '@_N': 'PinX', '@_V': props.x.toString() },
                 { '@_N': 'PinY', '@_V': props.y.toString() },
                 { '@_N': 'Width', '@_V': props.width.toString() },
                 { '@_N': 'Height', '@_V': props.height.toString() },
-                { '@_N': 'LocPinX', '@_V': (props.width / 2).toString() },
-                { '@_N': 'LocPinY', '@_V': (props.height / 2).toString() }
+                { '@_N': 'LocPinX', '@_V': (props.width / 2).toString(), '@_F': 'Width*0.5' },
+                { '@_N': 'LocPinY', '@_V': (props.height / 2).toString(), '@_F': 'Height*0.5' }
             ],
             Section: [
                 // Foreign shapes typically have no border (LinePattern=0)
@@ -34,9 +37,9 @@ class ForeignShapeBuilder {
                     Cell: [{ '@_N': 'NoFill', '@_V': '1' }], // Images usually don't have a fill behind them
                     Row: [
                         { '@_T': 'MoveTo', '@_IX': '1', Cell: [{ '@_N': 'X', '@_V': '0' }, { '@_N': 'Y', '@_V': '0' }] },
-                        { '@_T': 'LineTo', '@_IX': '2', Cell: [{ '@_N': 'X', '@_V': props.width.toString() }, { '@_N': 'Y', '@_V': '0' }] },
-                        { '@_T': 'LineTo', '@_IX': '3', Cell: [{ '@_N': 'X', '@_V': props.width.toString() }, { '@_N': 'Y', '@_V': props.height.toString() }] },
-                        { '@_T': 'LineTo', '@_IX': '4', Cell: [{ '@_N': 'X', '@_V': '0' }, { '@_N': 'Y', '@_V': props.height.toString() }] },
+                        { '@_T': 'LineTo', '@_IX': '2', Cell: [{ '@_N': 'X', '@_V': props.width.toString(), '@_F': 'Width' }, { '@_N': 'Y', '@_V': '0' }] },
+                        { '@_T': 'LineTo', '@_IX': '3', Cell: [{ '@_N': 'X', '@_V': props.width.toString(), '@_F': 'Width' }, { '@_N': 'Y', '@_V': props.height.toString(), '@_F': 'Height' }] },
+                        { '@_T': 'LineTo', '@_IX': '4', Cell: [{ '@_N': 'X', '@_V': '0' }, { '@_N': 'Y', '@_V': props.height.toString(), '@_F': 'Height' }] },
                         { '@_T': 'LineTo', '@_IX': '5', Cell: [{ '@_N': 'X', '@_V': '0' }, { '@_N': 'Y', '@_V': '0' }] }
                     ]
                 }