@shbernal/ts-pptx 4.0.1 → 4.2.0

package/dist/{presentation-DGYlAWhl.js → presentation-4Ggq0_Xa.js}

@@ -1,13 +1,13 @@
-import { i as PackageReadError, l as warn, n as InvalidOptionError, o as UnsupportedFeatureError, t as InternalError } from "./errors-C4oENPcu.js";
-import { X as MIN_SLIDE_MASTER_ID, Y as MAX_SLIDE_ID } from "./units-internal-XJlf-hOl.js";
-import { D as emuToInches } from "./table-cell-margins-EVg0EwxT.js";
-import { $t as THEME_REL, At as NOTES_MASTER_CONTENT_TYPE, Ct as HYPERLINK_REL, Dt as MS_MEDIA_REL, E as EMBEDDED_FONT_SLOTS, Et as MODERN_COMMENT_AUTHORS_REL, Ft as OFFICE_DOCUMENT_REL, Ht as RELATIONSHIPS_CONTENT_TYPE, It as OFFICE_REL, Jt as SLIDE_MASTER_REL, Kt as SLIDE_LAYOUT_REL, Lt as PACKAGE_REL, Mt as NOTES_SLIDE_CONTENT_TYPE, Nt as NOTES_SLIDE_REL, Ot as MS_POWERPOINT_CONTENT, Qt as THEME_CONTENT_TYPE, S as COLOR_MAP_TOKENS, Ut as SHARED_PARTS, Wt as SLIDE_CONTENT_TYPE, Xt as TABLE_STYLES_CONTENT_TYPE, Yt as SLIDE_REL, Z as THEME_COLOR_SLOTS, Zt as TABLE_STYLES_REL, _t as CORE_PROPS_REL, at as CHARTEX_REL, ct as CHART_CONTENT_TYPE, dt as CHART_STYLE_REL, en as VIDEO_REL, fn as imageFormatForBytes, in as OOXML_NS, it as CHARTEX_CONTENT_TYPE, jt as NOTES_MASTER_REL, kt as MS_REL, lt as CHART_REL, ot as CHART_COLOR_STYLE_CONTENT_TYPE, pt as COMMENTS_REL, rn as XLSX_CONTENT_TYPE, rt as AUDIO_REL, st as CHART_COLOR_STYLE_REL, u as boolValue, ut as CHART_STYLE_CONTENT_TYPE, wt as IMAGE_REL, zt as PRESENTATION_MAIN_CONTENT_TYPE } from "./image-size-CvN2iCA4.js";
-import { Ct as FONT_DATA_CONTENT_TYPE, Ot as XML_DECL, Rt as encodeXmlAttrValue, St as voidEl, Tt as FONT_REL_TYPE, n as makeXmlNotesMaster, o as makeXmlNotesSlideSkeleton, wt as FONT_DATA_EXTENSION, xt as raw, yt as el } from "./notes-Dh12JAbn.js";
-import { t as pickDefined } from "./options-internal-WqgMf720.js";
-import { n as readZip, t as ZipWriter } from "./zip-CKd8ihcf.js";
-import { t as transitionSpeedForDuration } from "./transition-speed-kUEWDWTx.js";
-import { $ as placeholderInheritedColor, $t as relsPartNameFor, A as flattenAnimations, At as CSLD_AFTER_BG, B as readColorRef, Bt as SLIDE_AFTER_TRANSITION, Ct as styleRefFill, E as AutoShape, Et as themeElementsOfRoot, Ft as PRESENTATION_AFTER_SLD_ID_LST, G as resolveSlideColorContext, Gt as SPPR_LN_AFTER, H as resolveLayoutColorContext, Ht as SPPR_AFTER_XFRM, It as PRESENTATION_AFTER_SLD_MASTER_ID_LST, J as checkPositiveEmu, Jt as TXBODY_AFTER_BODYPR, K as resolveSlideThemeParts, Kt as SPPR_SCENE3D_AFTER, L as readPatternFill, Lt as RPR_FILL_AFTER, M as pruneSpids, Mt as LN_SLOTS, N as remapSpids, Nt as PRESENTATION_AFTER_EMBEDDED_FONT_LST, O as carryShapeAnimations, Ot as hasFillChoice, P as TextFrame, Pt as PRESENTATION_AFTER_NOTES_MASTER_ID_LST, Q as placeholderInheritedAnchor, Qt as relativePartName, R as readPictureFill, Rt as RUN_AFTER_RPR, St as resolveSchemeToken, T as Picture, Tt as substitutePhClr, U as resolveMasterColorContext, Ut as SPPR_EFFECT_AFTER, Vt as SLIDE_MASTER_AFTER_SLD_LAYOUT_ID_LST, W as resolveNotesColorContext, Wt as SPPR_FILL_AFTER, X as RUN_PROP_NAMES, Xt as partNameExtension, Y as LST_STYLE_LEVELS, Yt as TXBODY_AFTER_LSTSTYLE, Z as lstStyleLevelDefRPr, Zt as partNameToZipPath, _n as parseXml, _t as isA, an as boolAttr, at as slideDefinesProp, bn as serializeXml, bt as replaceColorElement, cn as descendantsByTag, ct as carriedDecorations, dn as getElements, dt as nextDrawingId, en as resolveRelativePartName, et as placeholderInheritedListStyles, fn as getOrAddChild, ft as nthShapeChild, gn as ownerDocumentOf, gt as fmtEntry, hn as numberValue, ht as spTreeOf, in as attr, it as slideDefinesColor, j as hasAnimations, jt as EMBEDDED_FONT_ENTRY_AFTER, k as enumerateSpids, l as wrapShapeElement, ln as firstChild, lt as collectElements, m as resolveTableCellTextStyle, mn as numberAttr, mt as reassignDrawingIds, n as buildShapes, nn as singleRelPart, nt as placeholderInheritedXfrm, on as childElements, ot as cSldName, pn as insertInOrder, pt as nvPrOf, q as checkFiniteEmu, qt as SPPR_SP3D_AFTER, r as findShapeByIdDeep, rn as zipPathToPartName, rt as placeholderOf, sn as createElement, st as cSldOf, t as GroupShape, tn as resolveSingleRel, tt as placeholderInheritedRunProps, u as GraphicFrame, un as firstChildElement, ut as firstShapeChild, vn as removeAttr, vt as parseClrMap, wt as styleRefLine, xn as setAttr, xt as resolveColor, yn as removeChildrenByQName, yt as parseClrScheme, z as readGradientFill, zt as SHAPE_AFTER_SPPR } from "./shapes-re0KyQk_.js";
-//#region src/read/opc/content-types.ts
+import { D as emuToInches } from "./enums-BP9wsb9Q.js";
+import { n as readZip, t as ZipWriter } from "./zip-7JTh61LX.js";
+import { $ as LST_STYLE_LEVELS, $t as TXBODY_AFTER_LSTSTYLE, A as enumerateSpids, At as themeElementsOfRoot, Bt as PRESENTATION_AFTER_SLD_MASTER_ID_LST, Cn as removeChildrenByQName, Ct as parseClrScheme, D as AutoShape, Dt as styleRefFill, E as Picture, Et as resolveSchemeToken, F as remapSpids, Ft as EMBEDDED_FONT_ENTRY_AFTER, Gt as SLIDE_MASTER_AFTER_SLD_LAYOUT_ID_LST, H as readGradientFill, Ht as RUN_AFTER_RPR, I as TextFrame, It as LN_SLOTS, J as resolveNotesColorContext, Jt as SPPR_FILL_AFTER, K as resolveLayoutColorContext, Kt as SPPR_AFTER_XFRM, Lt as PRESENTATION_AFTER_EMBEDDED_FONT_LST, M as hasAnimations, Mt as hasFillChoice, N as pruneSpids, Ot as styleRefLine, P as readBuildEffects, Pt as CSLD_AFTER_BG, Q as checkPositiveEmu, Qt as TXBODY_AFTER_BODYPR, Rt as PRESENTATION_AFTER_NOTES_MASTER_ID_LST, Sn as removeAttr, St as parseClrMap, Tn as setAttr, Tt as resolveColor, U as readPictureFill, Ut as SHAPE_AFTER_SPPR, V as readPatternFill, Vt as RPR_FILL_AFTER, W as readColorRef, Wt as SLIDE_AFTER_TRANSITION, X as resolveSlideThemeParts, Xt as SPPR_SCENE3D_AFTER, Y as resolveSlideColorContext, Yt as SPPR_LN_AFTER, Z as checkFiniteEmu, Zt as SPPR_SP3D_AFTER, _n as insertInOrder, _t as nvPrOf, an as resolveSingleRel, at as placeholderInheritedRunProps, bn as ownerDocumentOf, bt as fmtEntry, cn as attr, ct as slideDefinesColor, dn as createElement, dt as cSldOf, en as partNameExtension, et as RUN_PROP_NAMES, fn as descendantsByTag, ft as carriedDecorations, gn as getOrAddChild, gt as nthShapeChild, hn as getElements, ht as nextDrawingId, in as resolveRelativePartName, it as placeholderInheritedListStyles, j as flattenAnimations, k as carryShapeAnimations, kt as substitutePhClr, l as wrapShapeElement, ln as boolAttr, lt as slideDefinesProp, m as resolveTableCellTextStyle, mn as firstChildElement, mt as firstShapeChild, n as buildShapes, nn as relativePartName, nt as placeholderInheritedAnchor, on as singleRelPart, ot as placeholderInheritedXfrm, pn as firstChild, pt as collectElements, q as resolveMasterColorContext, qt as SPPR_EFFECT_AFTER, r as findShapeByIdDeep, rn as relsPartNameFor, rt as placeholderInheritedColor, sn as zipPathToPartName, st as placeholderOf, t as GroupShape, tn as partNameToZipPath, tt as lstStyleLevelDefRPr, u as GraphicFrame, un as childElements, ut as cSldName, vn as numberAttr, vt as reassignDrawingIds, wn as serializeXml, wt as replaceColorElement, xn as parseXml, xt as isA, yn as numberValue, yt as spTreeOf, zt as PRESENTATION_AFTER_SLD_ID_LST } from "./shapes-CAPOjU0Q.js";
+import { Ft as voidEl, It as XML_DECL, Kt as encodeXmlAttrValue, Mt as el, Pt as raw, a as makeXmlNotesMaster, gt as FONT_REL_TYPE, ht as FONT_DATA_EXTENSION, l as makeXmlNotesSlideSkeleton, mt as FONT_DATA_CONTENT_TYPE, n as makeXmlComments, t as makeXmlCommentAuthors } from "./comments-CKIZYRbZ.js";
+import { a as PackageReadError, n as InternalError, r as InvalidOptionError, s as UnsupportedFeatureError, u as warn } from "./entry-errors-JMthgJ2g.js";
+import { B as NOTES_SLIDE_CONTENT_TYPE, C as COMMENT_AUTHORS_REL, F as MS_MEDIA_REL, G as PACKAGE_REL, I as MS_POWERPOINT_CONTENT, L as MS_REL, M as IMAGE_REL, P as MODERN_COMMENT_AUTHORS_REL, Q as SLIDE_CONTENT_TYPE, Qt as THEME_COLOR_SLOTS, R as NOTES_MASTER_CONTENT_TYPE, S as COMMENT_AUTHORS_CONTENT_TYPE, T as CORE_PROPS_REL, Tt as EMBEDDED_FONT_SLOTS, U as OFFICE_DOCUMENT_REL, V as NOTES_SLIDE_REL, W as OFFICE_REL, X as RELATIONSHIPS_CONTENT_TYPE, Z as SHARED_PARTS, _ as CHART_REL, _n as imageFormatForBytes, at as TABLE_STYLES_REL, b as COMMENTS_CONTENT_TYPE, ct as VIDEO_REL, d as AUDIO_REL, dt as XLSX_CONTENT_TYPE, et as SLIDE_LAYOUT_REL, f as CHARTEX_CONTENT_TYPE, ft as OOXML_NS, g as CHART_CONTENT_TYPE, h as CHART_COLOR_STYLE_REL, it as TABLE_STYLES_CONTENT_TYPE, j as HYPERLINK_REL, m as CHART_COLOR_STYLE_CONTENT_TYPE, nt as SLIDE_MASTER_REL, on as boolValue, ot as THEME_CONTENT_TYPE, p as CHARTEX_REL, q as PRESENTATION_MAIN_CONTENT_TYPE, rt as SLIDE_REL, st as THEME_REL, v as CHART_STYLE_CONTENT_TYPE, x as COMMENTS_REL, xt as COLOR_MAP_TOKENS, y as CHART_STYLE_REL, z as NOTES_MASTER_REL } from "./chart-metadata-CwiTTcJq.js";
+import { X as MIN_SLIDE_MASTER_ID, Y as MAX_SLIDE_ID } from "./units-internal-BR3a4cIi.js";
+import { t as transitionSpeedForDuration } from "./transition-speed-CNJ0fSpQ.js";
+import { t as pickDefined } from "./options-internal-BWgmLPdx.js";
+// src/read/opc/content-types.ts
 /**
 * Overlay over `[Content_Types].xml`, used to resolve part content types at
 * load and to register new ones when parts are added. While clean, the original
@@ -57,7 +57,7 @@ var ContentTypes = class ContentTypes {
 	}
 	/**
 	* Drop the `Override` for `partName` if present (e.g. when a part is removed).
-	* `Default` rules by extension are left untouched — they may still serve other
+	* `Default` rules by extension are left untouched; they may still serve other
 	* parts. Marks the overlay dirty when something was removed.
 	*/
 	removeOverride(partName) {
@@ -78,10 +78,9 @@ var ContentTypes = class ContentTypes {
 		return lines.join("");
 	}
 };
-//#endregion
-//#region src/read/opc/part.ts
+// src/read/opc/part.ts
 const textDecoder$1 = new TextDecoder("utf-8");
-const textEncoder$4 = new TextEncoder();
+const textEncoder$5 = new TextEncoder();
 const XML_DECLARATION_RE = /^\uFEFF?<\?xml[^>]*\?>\s*/;
 /**
 * One OPC part. Holds the original bytes from the zip for its whole life;
@@ -147,11 +146,10 @@ var Part = class {
 		if (!this.#dirty) return this.#bytes;
 		let xml = serializeXml(this.dom);
 		if (!xml.startsWith("<?xml")) xml = (XML_DECLARATION_RE.exec(textDecoder$1.decode(this.#bytes))?.[0] ?? "<?xml version=\"1.0\" encoding=\"UTF-8\" standalone=\"yes\"?>\r\n") + xml;
-		return textEncoder$4.encode(xml);
+		return textEncoder$5.encode(xml);
 	}
 };
-//#endregion
-//#region src/read/opc/relationships.ts
+// src/read/opc/relationships.ts
 /**
 * Overlay over one `.rels` part, keyed by relationship id. While clean, the
 * `.rels` bytes pass through verbatim on save (byte-identical); once `add()`
@@ -302,17 +300,16 @@ var Relationships = class Relationships {
 		return lines.join("");
 	}
 };
-//#endregion
-//#region src/read/opc/package.ts
+// src/read/opc/package.ts
 const CONTENT_TYPES_ZIP_PATH = "[Content_Types].xml";
 const textDecoder = new TextDecoder("utf-8");
-const textEncoder$3 = new TextEncoder();
+const textEncoder$4 = new TextEncoder();
 /**
 * An OPC package loaded from `.pptx` bytes.
 *
 * Fidelity contract: parts that are never mutated (`Part.markDirty()`) are
 * written back byte-identically by `save()`. Whole-zip byte-identity is not
-* promised — zip metadata and compression may differ — but part bodies, the
+* promised (zip metadata and compression may differ), but part bodies, the
 * part-name set, and part order are preserved.
 */
 var OpcPackage = class OpcPackage {
@@ -357,8 +354,12 @@ var OpcPackage = class OpcPackage {
 		}
 		return new OpcPackage(parts, contentTypes, contentTypesBytes);
 	}
+	/**
+	* The part at `partName`, or `undefined`. Takes `null`, as a part-name getter such as
+	* `Picture.imagePartName` reads when there is no part to name, and misses on it.
+	*/
 	part(partName) {
-		return this.parts.get(partName);
+		return partName === null ? void 0 : this.parts.get(partName);
 	}
 	partsByContentType(contentType) {
 		if (!this.#partsByContentType) {
@@ -403,7 +404,7 @@ var OpcPackage = class OpcPackage {
 	* later `save()` neither re-emits it nor flushes a stale `.rels` for it. A relationship set
 	* belongs to its part: one left behind would come back as the relationships of the next part
 	* added under the same name. Low-level: it does not touch references *to* this part
-	* (dangling rels are the caller's concern) nor cascade to parts this one referenced — see
+	* (dangling rels are the caller's concern) nor cascade to parts this one referenced; see
 	* {@link Presentation.removeSlide} for the slide-aware variant that unwires the presentation
 	* and prunes orphans.
 	*/
@@ -477,21 +478,20 @@ var OpcPackage = class OpcPackage {
 		for (const relationships of this.#relationshipsCache.values()) {
 			if (!relationships.isDirty) continue;
 			const relsPartName = relsPartNameFor(relationships.sourcePartName);
-			const bytes = textEncoder$3.encode(relationships.serialize());
+			const bytes = textEncoder$4.encode(relationships.serialize());
 			this.#parts.set(relsPartName, new Part(relsPartName, RELATIONSHIPS_CONTENT_TYPE, bytes));
 			this.#partsByContentType = null;
 		}
 	}
 };
-//#endregion
-//#region src/read/api/slide-background.ts
+// src/read/api/slide-background.ts
 /**
 * Read a slide's effective background (`p:cSld/p:bg`), decoded into a typed union.
 *
 * A background lives on the slide itself, or is inherited from its slideLayout,
-* else its slideMaster — the {@link SlideBackground.source} field records which.
+* else its slideMaster; the {@link SlideBackground.source} field records which.
 * The writer authors solid-colour, gradient, pattern and image backgrounds (all FAITHFUL);
-* theme-indexed (`p:bgRef`) backgrounds are read-only for imported decks — surface
+* theme-indexed (`p:bgRef`) backgrounds are read-only for imported decks. Surface
 * them so a codegen pass can carry the part rather than fake it. A
 * `themeRef` additionally resolves its `idx` through the slide theme's `fmtScheme`
 * to a concrete {@link BackgroundFill} in `resolvedFill`, so a consumer can ask what
@@ -534,13 +534,13 @@ function decodeBackgroundFill(container, ctx, rels) {
 /**
 * Resolve a `p:bgRef`'s `idx` into the concrete {@link BackgroundFill} it renders
 * as, through the slide theme's `fmtScheme`. A `p:bgRef` is a
-* `CT_StyleMatrixReference` — the same shape as a shape's `a:fillRef` — so
+* `CT_StyleMatrixReference` (the same shape as a shape's `a:fillRef`), so
 * {@link styleRefFill} builds the indexed `bgFillStyleLst`/`fillStyleLst` entry with
 * its `phClr` substituted by the bgRef's colour, and the resulting fill element is
 * decoded like a `p:bgPr`. `null` when the theme has no `fmtScheme`, the entry is
 * absent, or the colour cannot be resolved.
 *
-* `themeRels` — not the owning slide/layout/master's rels — is what an `a:blipFill`
+* `themeRels`, not the owning slide/layout/master's rels, is what an `a:blipFill`
 * entry's `r:embed` resolves against: the fill element is lifted out of the *theme*
 * part, so its relationship id is scoped to that part. Several stock Office themes
 * (Ion, Facet, …) put a picture in the third `bgFillStyleLst` slot, so a deck using
@@ -581,14 +581,13 @@ function readSlideBackground(bg, source, ctx, rels, themeRels) {
 		source
 	};
 }
-//#endregion
-//#region src/read/api/ops/part-index.ts
+// src/read/api/ops/part-index.ts
 /**
 * Walking a deck's master/layout spine by partname.
 *
 * The `p:sldMasterIdLst` / `p:sldLayoutIdLst` entries hold relationship ids, not partnames, so
 * every traversal has to resolve through the owning part's relationships. These are the
-* source-side reads that do it — used against a *foreign* presentation as often as against this
+* source-side reads that do it, used against a *foreign* presentation as often as against this
 * one, which is why they take the deck rather than living on it.
 */
 /**
@@ -628,8 +627,7 @@ function idListTargets(lst, qname, resolve) {
 	}
 	return out;
 }
-//#endregion
-//#region src/read/api/chrome.ts
+// src/read/api/chrome.ts
 /** Read `<a:latin>`/`<a:ea>`/`<a:cs>` typefaces of a theme font family; empty string ⇒ `null`. */
 function readThemeFontFace(font) {
 	const face = (child) => {
@@ -723,7 +721,7 @@ function placeholderShapes(spTree) {
 	return getElements(spTree, "p:sp").filter((sp) => placeholderOf(sp) !== null);
 }
 /**
-* One placeholder shape (`p:sp` with a `p:ph`) of a slide master or slide layout —
+* One placeholder shape (`p:sp` with a `p:ph`) of a slide master or slide layout:
 * the definition a slide's placeholder of the same `type`/`idx` inherits geometry
 * and text style from. Geometry getters read the placeholder's *own* `a:xfrm`,
 * which a master/layout placeholder carries explicitly (so it round-trips, unlike a
@@ -749,7 +747,7 @@ var Placeholder = class {
 	* `xfrm()` + `emuFrom`; and,
 	* the one that showed, the text frame was built with **no** inheritance context, so
 	* `Run.resolvedSizePt`, `resolvedFontFace`, `resolvedColor` and `resolvedBold` resolved
-	* through `SlideMaster.shapes` and came back `null` through `SlideMaster.placeholders` —
+	* through `SlideMaster.shapes` and came back `null` through `SlideMaster.placeholders`:
 	* two views of one element disagreeing about the same run.
 	*/
 	shape;
@@ -817,7 +815,7 @@ var Placeholder = class {
 	}
 };
 /**
-* The shared body of a slide master and a slide layout — the two template tiers a slide
+* The shared body of a slide master and a slide layout, the two template tiers a slide
 * inherits from.
 *
 * Both are an OPC part holding a `p:cSld` with a shape tree and a resolvable theme context, and
@@ -835,7 +833,7 @@ var Placeholder = class {
 *
 * `Slide` deliberately does not extend this. It is a shape host too, but it is constructed from
 * a `Presentation` rather than an `OpcPackage`, reaches its relationships through that, and
-* reports an unnamed slide as `null` where a template tier reports `''` — three differences in
+* reports an unnamed slide as `null` where a template tier reports `''`. That is three differences in
 * the same handful of members, which is more contortion than the sharing is worth.
 */
 var TemplatePart = class {
@@ -860,7 +858,7 @@ var TemplatePart = class {
 		return (cSld && attr(cSld, "name")) ?? "";
 	}
 	/**
-	* Every shape in this tier's `p:cSld/p:spTree`, in document order — the same `AnyShape`
+	* Every shape in this tier's `p:cSld/p:spTree`, in document order: the same `AnyShape`
 	* union `Slide.shapes` returns, so shape-walking code applies unchanged.
 	*
 	* {@link placeholders} is the filtered view of the same tree.
@@ -869,7 +867,7 @@ var TemplatePart = class {
 		const spTree = this.spTree();
 		return spTree ? buildShapes(spTree, this) : [];
 	}
-	/** The shape anywhere in this tier's tree with the given drawing id, or `undefined`. */
+	/** The shape anywhere in this tier's tree with the given drawing id, or `undefined` (always, for `null`). */
 	shapeByIdDeep(id) {
 		return findShapeByIdDeep(this.shapes, id);
 	}
@@ -930,7 +928,7 @@ var SlideMaster = class extends TemplatePart {
 	}
 	/**
 	* The master's *own* background (`p:cSld/p:bg`), decoded, or `null` when it defines
-	* none. Scoped to the master's own element — a slide's *effective* background,
+	* none. Scoped to the master's own element. A slide's *effective* background,
 	* resolved through the slide → layout → master chain, is `Slide.background`.
 	*/
 	get background() {
@@ -939,21 +937,21 @@ var SlideMaster = class extends TemplatePart {
 		const themePart = this.relTarget(THEME_REL);
 		return readSlideBackground(bg, "master", this.themeContext(), this.relationships, themePart ? this.opc.relationshipsFor(themePart.partName) : null);
 	}
-	/** A master's context is its own `p:clrMap` plus its theme — no walk beyond this part. */
+	/** A master's context is its own `p:clrMap` plus its theme; no walk beyond this part. */
 	resolveThemeContext() {
 		return resolveMasterColorContext(this.opc, this.partName);
 	}
 };
 /**
-* A deck's slide layout (`slideLayoutN.xml`) as a modeled object — the gallery entry
+* A deck's slide layout (`slideLayoutN.xml`) as a modeled object: the gallery entry
 * a slide binds to. Exposes its {@link name}, its {@link type} (import-only; the
 * writer authors none), its {@link master} (and, through it, the {@link theme}), its
 * `shapes` and the `placeholders` subset of them, and its own {@link background}.
 * Reachable from `Slide.layout` and {@link SlideMaster.layouts}.
 *
 * The write API authors non-placeholder shapes into a layout: every
-* `defineSlideMaster({ objects })` member that is not a `placeholder` — a `rect`, a
-* `line`, an `image`, a `chart`, a `text` box — lands in this tier's tree.
+* `defineSlideMaster({ objects })` member that is not a `placeholder` (a `rect`, a
+* `line`, an `image`, a `chart`, a `text` box) lands in this tier's tree.
 */
 var SlideLayout = class extends TemplatePart {
 	/**
@@ -968,7 +966,7 @@ var SlideLayout = class extends TemplatePart {
 	/**
 	* Whether this layout draws the master's non-placeholder shapes
 	* (`p:sldLayout/@showMasterSp`). `xsd:boolean` defaulting to `true`, so an absent
-	* attribute means shown — the layout-tier counterpart of `Slide.showMasterSp`,
+	* attribute means shown. This is the layout-tier counterpart of `Slide.showMasterSp`,
 	* and the one PowerPoint actually writes on a section-divider or full-bleed
 	* layout. A renderer resolving what to paint under a slide's own shapes has to
 	* consult both: the slide's flag, then its layout's.
@@ -988,7 +986,7 @@ var SlideLayout = class extends TemplatePart {
 	}
 	/**
 	* The layout's *own* background (`p:cSld/p:bg`), decoded, or `null` when it defines
-	* none. A layout usually authors one — an explicit fill (`p:bgPr`) or a
+	* none. A layout usually authors one: an explicit fill (`p:bgPr`) or a
 	* theme-indexed reference (`p:bgRef`). The slide's *effective* background is
 	* `Slide.background`.
 	*/
@@ -1003,8 +1001,7 @@ var SlideLayout = class extends TemplatePart {
 		return resolveLayoutColorContext(this.opc, this.partName);
 	}
 };
-//#endregion
-//#region src/read/api/notes.ts
+// src/read/api/notes.ts
 /**
 * One placeholder shape (`p:sp`) of a notes slide. A notes slide holds a fixed set:
 * the slide thumbnail (`type="sldImg"`), the notes body (`type="body"`), and the
@@ -1012,8 +1009,8 @@ var SlideLayout = class extends TemplatePart {
 * `a:xfrm` and are `null` when it inherits geometry from the notesMaster (the case
 * on an authored deck, whose `sldImg`/`sldNum` carry an empty `p:spPr`).
 *
-* A notes placeholder IS a {@link Placeholder} — identity, geometry and the escape hatch are
-* the same twelve members read off the same `p:sp`, down to `p:ph` being found the same way —
+* A notes placeholder IS a {@link Placeholder}. Identity, geometry and the escape hatch are
+* the same twelve members read off the same `p:sp`, down to `p:ph` being found the same way,
 * so it extends one rather than repeating it. Two things are its own: the flattened
 * {@link text} convenience, and a {@link textFrame} that threads the notesMaster inheritance
 * context onto the body frame.
@@ -1025,7 +1022,7 @@ var NotesPlaceholder = class extends Placeholder {
 	* runs round-trip a slide's authored notes; the `sldNum` placeholder's frame
 	* holds the slide-number `a:fld`, whose value surfaces through `TextFrame.text`.
 	*
-	* Overrides {@link Placeholder.textFrame} for the inheritance context alone — see below.
+	* Overrides {@link Placeholder.textFrame} for the inheritance context alone (see below).
 	*/
 	get textFrame() {
 		const txBody = firstChild(this.sp, "p:txBody");
@@ -1051,7 +1048,7 @@ var NotesPlaceholder = class extends Placeholder {
 	}
 };
 /**
-* A slide's speaker-notes slide (`notesSlideN.xml`) as a modeled object — the deep
+* A slide's speaker-notes slide (`notesSlideN.xml`) as a modeled object: the deep
 * companion to {@link Slide.notesText}. Exposes the notes
 * shape tree as {@link NotesPlaceholder}s ({@link body}/{@link slideImage}/
 * {@link slideNumber}) plus the body {@link textFrame} the older getters flatten.
@@ -1071,7 +1068,7 @@ var NotesSlide = class {
 	get partName() {
 		return this.part.partName;
 	}
-	/** The notes part's own relationships — a notes hyperlink resolves through these. */
+	/** The notes part's own relationships; a notes hyperlink resolves through these. */
 	get relationships() {
 		return this.opc.relationshipsFor(this.partName);
 	}
@@ -1079,7 +1076,7 @@ var NotesSlide = class {
 	* The notes shape tree as full shapes, in document order.
 	*
 	* A notes slide holds nothing but its three placeholders, so {@link placeholders} is the
-	* view to read. This one exists because a notes slide is a {@link ShapeHost} — which is
+	* view to read. This one exists because a notes slide is a {@link ShapeHost}, which is
 	* what lets a {@link NotesPlaceholder} be the same {@link AutoShape} view of its `p:sp`
 	* that a master or layout placeholder is, rather than a second reading of one.
 	*/
@@ -1087,7 +1084,7 @@ var NotesSlide = class {
 		const spTree = this.#spTree();
 		return spTree ? buildShapes(spTree, this) : [];
 	}
-	/** The shape anywhere in the notes tree with the given drawing id, or `undefined`. */
+	/** The shape anywhere in the notes tree with the given drawing id, or `undefined` (always, for `null`). */
 	shapeByIdDeep(id) {
 		return findShapeByIdDeep(this.shapes, id);
 	}
@@ -1131,8 +1128,7 @@ var NotesSlide = class {
 		return spTreeOf(this.part.dom.documentElement);
 	}
 };
-//#endregion
-//#region src/read/api/ops/deck-target.ts
+// src/read/api/ops/deck-target.ts
 /**
 * `ppt/_rels/presentation.xml.rels` -- the deck's own relationship part.
 *
@@ -1147,8 +1143,7 @@ var NotesSlide = class {
 function presentationRels(dest) {
 	return dest.opc.relationshipsFor(dest.presentationPart.partName);
 }
-//#endregion
-//#region src/read/api/ops/master-registry.ts
+// src/read/api/ops/master-registry.ts
 /**
 * Slide-master bookkeeping in `presentation.xml` and in a master's own
 * `p:sldLayoutIdLst`: registering a master so renderers treat it as active,
@@ -1185,7 +1180,7 @@ function registerMaster(dest, masterPartName) {
 /**
 * Move the `p:sldMasterId` entries for `masterPartNames` to the front of
 * `p:sldMasterIdLst`, preserving their existing relative order, so the first of
-* them becomes the deck's `Designs(1)` — the theme PowerPoint's Design tab shows.
+* them becomes the deck's `Designs(1)`, the theme PowerPoint's Design tab shows.
 * Reorders the id list only; relationships, ids, and all other parts are left as
 * they are. A no-op when the named masters already lead (idempotent re-call).
 */
@@ -1215,7 +1210,7 @@ function promoteMasters(dest, masterPartNames) {
 * unique across the WHOLE presentation, not just within their own list. A duplicate
 * anywhere in this space makes PowerPoint report the file as corrupt on open
 * ("found a problem with content" / 0x80070570), even though LibreOffice silently
-* tolerates it — so an imported master AND each of its layouts must take an id past
+* tolerates it. So an imported master AND each of its layouts must take an id past
 * the max of both the master-id list and every layout-id list. Recomputed per
 * allocation so ids appended earlier in the same import are counted.
 *
@@ -1271,20 +1266,19 @@ function addLayoutToMaster(dest, masterPartName, layoutPartName) {
 	lst.appendChild(entry);
 	masterPart.markDirty();
 }
-//#endregion
-//#region src/read/api/ops/copy-traversal.ts
+// src/read/api/ops/copy-traversal.ts
 /**
 * What a copy traversal does with one relationship: skip it, carry it as an external link, or
 * follow it into the part it names.
 *
 * Notes pull in a notesMaster and its own theme, which an imported page does not need, so the
-* notes relationship is never followed — `carryNotes` picks it up afterwards for the pages
+* notes relationship is never followed; `carryNotes` picks it up afterwards for the pages
 * that asked. A master's relationships to its layouts are skipped so the copy stays lean:
 * copied layouts re-link themselves, and skipping the list is also what keeps one edited
 * layout from disqualifying every other layout that shares its master.
 *
-* Each caller still owns its own extra arms — a copy's overrides and its jump-link rule, the reuse
-* check's byte comparison — which is why this answers only the part they share.
+* Each caller still owns its own extra arms (a copy's overrides and its jump-link rule, the reuse
+* check's byte comparison), which is why this answers only the part they share.
 * @param part - the part the relationship belongs to
 * @param rel - the relationship being considered
 */
@@ -1293,13 +1287,12 @@ function copyTraversalStep(part, rel) {
 	if (part.contentType === "application/vnd.openxmlformats-officedocument.presentationml.slideMaster+xml" && rel.type === "http://schemas.openxmlformats.org/officeDocument/2006/relationships/slideLayout") return "skip";
 	return rel.targetMode === "External" ? "external" : "recurse";
 }
-//#endregion
-//#region src/read/api/ops/page-owned.ts
+// src/read/api/ops/page-owned.ts
 /**
 * What a page copy may share with the page it was copied from, and what it must own.
 *
-* Duplicating a page — `cloneSlide`, `importSlide` of the same page twice,
-* `importSlides` naming it twice — copies the page part and then has to decide,
+* Duplicating a page (`cloneSlide`, `importSlide` of the same page twice,
+* `importSlides` naming it twice) copies the page part and then has to decide,
 * for every part hanging off it, whether the copy points at the same part or gets
 * one of its own. Sharing is the cheap answer and is right for most of the graph:
 * a layout, a master, a theme, an image are deck-wide assets PowerPoint itself
@@ -1308,7 +1301,7 @@ function copyTraversalStep(part, rel) {
 * It is not right for the parts a page *owns*. A chart and its embedded workbook,
 * a SmartArt diagram's five parts, an OLE embedding, a notes slide: PowerPoint
 * treats each as belonging to exactly one page, and a package where two slides
-* resolve to one of them is one it refuses to open at all — the "the file or
+* resolve to one of them is one it refuses to open at all: the "the file or
 * directory is corrupted and unreadable" (`0x80070570`) dialog, with the whole
 * deck rejected rather than the duplicate page. The schema validator accepts such
 * a package: nothing in ECMA-376 says a chart part has one referrer, so this rule
@@ -1338,14 +1331,13 @@ function copyTraversalStep(part, rel) {
 const SHARED_BY_PAGE_COPIES = new Set(SHARED_PARTS.map((kind) => kind.relType));
 /**
 * Whether a page copy may point at this relationship's target rather than taking
-* a copy of it. Everything not listed is owned by the page and copied — see the
+* a copy of it. Everything not listed is owned by the page and copied; see the
 * module comment for why that is the safe default.
 */
 function isSharedByPageCopies(relType) {
 	return SHARED_BY_PAGE_COPIES.has(relType);
 }
-//#endregion
-//#region src/read/api/ops/part-reuse.ts
+// src/read/api/ops/part-reuse.ts
 /**
 * Whether the destination already holds `sourcePartName`'s subgraph, part for part and
 * byte for byte, in a state an imported slide can bind to.
@@ -1426,8 +1418,7 @@ function bytesEqual(a, b) {
 	for (let i = 0; i < a.length; i++) if (a[i] !== b[i]) return false;
 	return true;
 }
-//#endregion
-//#region src/read/api/ops/part-copy.ts
+// src/read/api/ops/part-copy.ts
 /** Open an ownership scope for one page copy. */
 function newOwnedScope() {
 	return /* @__PURE__ */ new Map();
@@ -1441,7 +1432,7 @@ function copyTarget(ctx) {
 * it would throw, found before anything in the destination moves.
 *
 * This is what lets a refused import leave the deck byte-identical. The copy changes the deck as it
-* goes — parts added, a master registered in `presentation.xml`, a layout linked into its master —
+* goes (parts added, a master registered in `presentation.xml`, a layout linked into its master),
 * so a failure it met halfway would leave all of that behind. An import therefore runs the same
 * traversal twice: first with a plan in its context, which throws what the copy would and writes
 * nothing, then for real. The dry run used to be a separate walk of the source, transcribed from the
@@ -1452,8 +1443,8 @@ function copyTarget(ctx) {
 * A plan stands in for the destination package as a {@link CopyTarget}. It hands out the partnames
 * the copy will be given, keeps its own copy of each source's registry so a part it planned is found
 * again as the copy would find it, and remembers the notes master it would install in a deck that has
-* none. Destination bookkeeping that adds no part — registering a master, linking a layout into it —
-* is not recorded.
+* none. Destination bookkeeping that adds no part, such as registering a master or linking a layout
+* into it, is not recorded.
 */
 var CopyPlan = class {
 	api;
@@ -1547,7 +1538,7 @@ var CopyPlan = class {
 * registry idempotence is right for a theme, master, layout or image, where a
 * second copy is waste; applied to the slide it made a repeated import of the
 * same source page return the first copy's partname, and the caller then wired a
-* second `p:sldId` to a part that already had one — a package PowerPoint refuses
+* second `p:sldId` to a part that already had one: a package PowerPoint refuses
 * to open (0x80070570), with nothing in the read model to show for it. So the
 * page goes into the selection plan `copyPart` already honours for a batch, which
 * re-materializes exactly the named pages and leaves the dedup of their
@@ -1570,7 +1561,7 @@ function copySlidePart(ctx, sourcePartName) {
 * per source package via the copy registry. Relationship ids are preserved so
 * the copied part body's `r:id`/`r:embed` references stay valid; targets are
 * rewritten to the freshly-allocated partnames. Notes relationships are
-* dropped. A copied `slideMaster` does not drag in all its sibling layouts —
+* dropped. A copied `slideMaster` does not drag in all its sibling layouts;
 * each imported `slideLayout` wires itself into the master instead (see
 * {@link linkLayoutIntoMaster}).
 *
@@ -1580,13 +1571,13 @@ function copySlidePart(ctx, sourcePartName) {
 * a `slide → slide` link may not leave the selection is enforced as each link is
 * followed (see {@link rebuildRels}). Imports run this traversal as a
 * {@link CopyPlan} first, which throws that and every other failure the copy
-* could meet — so once copying starts there is nothing left to throw, and a
+* could meet, so once copying starts there is nothing left to throw, and a
 * rejected batch never leaves a half-copied deck behind.
 *
 * Idempotence stops at the page's own parts. Reaching a page opens an
 * {@link OwnedScope}, and everything under it that {@link isSharedByPageCopies}
-* does not clear for sharing — a chart, a diagram, an OLE embedding, each with
-* its own subtree — is copied into that scope instead of the registry, so the
+* does not clear for sharing (a chart, a diagram, an OLE embedding, each with
+* its own subtree) is copied into that scope instead of the registry, so the
 * next copy of the page gets parts of its own. Pass `owned` to open the scope
 * from outside, as the rebinding import paths do for the page they build
 * themselves.
@@ -1694,7 +1685,7 @@ function rebuildRels(ctx, options) {
 * The page's shared targets are carried as the source wrote them. Each owned one is repointed at a
 * fresh copy, and the subtree under it is copied the same way, so a chart's workbook and
 * user-shapes drawing come along while the image inside that drawing stays shared. A relationship
-* *back* to the source page — a notes slide names the slide it annotates — is repointed at the
+* *back* to the source page (a notes slide names the slide it annotates) is repointed at the
 * clone, which is what makes the copied notes belong to it.
 *
 * Unlike an import, a clone keeps its page's notes, and nothing here consults the copy registry or
@@ -1769,19 +1760,18 @@ function linkLayoutIntoMaster(ctx, layoutSourceRels, layoutPartName) {
 	if (!masterPartName) return;
 	addLayoutToMaster(ctx.dest, masterPartName, layoutPartName);
 }
-//#endregion
-//#region src/read/api/ops/notes-master.ts
+// src/read/api/ops/notes-master.ts
 /**
 * Carrying speaker notes and the deck's single notes master across an import or an append.
 *
 * A `notesSlide` is not self-contained: it back-references the slide it annotates and forward-
 * references the deck's notes master. `CT_NotesMasterIdList` allows at most one master per
-* presentation, so a deck that already has one keeps it — the destination's notes styling wins —
+* presentation, so a deck that already has one keeps it (the destination's notes styling wins),
 * and a source master is copied only into a deck that has none. That single rule is why both the
 * `Presentation`-to-`Presentation` path and the generator-XML path funnel through
 * `registerNotesMaster`: they differ only in how the *part* comes into being.
 */
-const textEncoder$2 = new TextEncoder();
+const textEncoder$3 = new TextEncoder();
 /**
 * Carry the source slide's speaker notes onto the just-imported slide (the
 * `importNotes` option). The slide copy itself dropped the `notesSlide` rel, so
@@ -1789,7 +1779,7 @@ const textEncoder$2 = new TextEncoder();
 * `slide → notesSlide` rel on the new slide, and rebuilds the copied notesSlide's
 * own relationships:
 *
-* - its `slide` back-rel is repointed at the new slide (`newSlidePartName`) — the
+* - its `slide` back-rel is repointed at the new slide (`newSlidePartName`); the
 *   source slide is *not* copied (that would be circular);
 * - its `notesMaster` rel is resolved through {@link ensureNotesMaster}, which
 *   reuses this deck's notesMaster when it has one and copies the source's only
@@ -1838,7 +1828,7 @@ function carryNotes(dest, source, ctx, sourceSlidePartName, newSlidePartName) {
 */
 function addNotesSlidePart(dest, xml, slidePartName, notesMasterPartName) {
 	const notesPartName = dest.opc.reservePartNameLike("/ppt/notesSlides/notesSlide1.xml");
-	dest.opc.addPart(notesPartName, NOTES_SLIDE_CONTENT_TYPE, textEncoder$2.encode(xml));
+	dest.opc.addPart(notesPartName, NOTES_SLIDE_CONTENT_TYPE, textEncoder$3.encode(xml));
 	const notesRels = dest.opc.relationshipsFor(notesPartName);
 	if (notesMasterPartName) notesRels.addWithId("rId1", NOTES_MASTER_REL, relativePartName(notesPartName, notesMasterPartName));
 	notesRels.addWithId("rId2", SLIDE_REL, relativePartName(notesPartName, slidePartName));
@@ -1876,8 +1866,8 @@ function presentationRoot(dest) {
 * allows. Returns the partname, so callers can use it as a rel target.
 *
 * Split out of {@link ensureNotesMaster} because the two ways a notesMaster arrives
-* — copied from another `Presentation`, or authored by a generator and injected by
-* {@link Presentation.appendSlides} — differ only in how the *part* is created, not in how it is
+* (copied from another `Presentation`, or authored by a generator and injected by
+* {@link Presentation.appendSlides}) differ only in how the *part* is created, not in how it is
 * registered.
 */
 function registerNotesMaster(dest, notesMasterPartName) {
@@ -1921,7 +1911,7 @@ function installNotesMaster(dest, masterBytes, themeBytes, themeContentType = TH
 * theme its `.rels` requires (the normal write path emits that as `theme2.xml`).
 */
 function ensureNotesMasterFromXml(dest, master) {
-	return existingNotesMaster(dest) ?? installNotesMaster(dest, textEncoder$2.encode(master.xml), textEncoder$2.encode(master.themeXml));
+	return existingNotesMaster(dest) ?? installNotesMaster(dest, textEncoder$3.encode(master.xml), textEncoder$3.encode(master.themeXml));
 }
 /**
 * Resolve the notesMaster a *newly authored* notes slide should bind to
@@ -1941,7 +1931,7 @@ function ensureNotesMasterFromXml(dest, master) {
 * arrangement PowerPoint writes (`theme1.xml` for the master, `theme2.xml` for the
 * notes master).
 *
-* Throws when the deck has no theme to clone — a package that malformed cannot be
+* Throws when the deck has no theme to clone: a package that malformed cannot be
 * given a valid notesMaster, and silently emitting one without a resolvable theme
 * is what PowerPoint reports as a repair prompt rather than a bad edit.
 * @param {Presentation} dest - the deck being authored onto
@@ -1954,10 +1944,9 @@ function ensureNotesMasterForAuthoring(dest, slidePartName) {
 	const themePartName = resolveSlideThemeParts(dest.opc, slidePartName).themePartName;
 	const themePart = themePartName ? dest.opc.part(themePartName) : void 0;
 	if (!themePart) throw new PackageReadError("package/part-missing", `addNotes: no theme reachable from ${slidePartName} to bind a new notes master to`);
-	return installNotesMaster(dest, textEncoder$2.encode(makeXmlNotesMaster()), themePart.serialize(), themePart.contentType);
+	return installNotesMaster(dest, textEncoder$3.encode(makeXmlNotesMaster()), themePart.serialize(), themePart.contentType);
 }
-//#endregion
-//#region src/read/api/ops/notes-author.ts
+// src/read/api/ops/notes-author.ts
 /**
 * Authoring speaker notes onto a slide of a *loaded* deck ({@link Slide.addNotes}).
 *
@@ -1969,8 +1958,8 @@ function ensureNotesMasterForAuthoring(dest, slidePartName) {
 * The part it builds is the generator's, not a second design: the fixed three-
 * placeholder frame comes from {@link makeXmlNotesSlideSkeleton}, and the body
 * paragraphs carry the same elements and attributes the write path emits for a plain
-* `pptx.addSlide().addNotes(text)`. `test/read/add-notes.test.js` asserts that
-* equivalence, so the two cannot drift apart unnoticed — the same guarantee
+* `pptx.addSlide().addNotes(text)`. `test/read/add-notes.test.ts` asserts that
+* equivalence, so the two cannot drift apart unnoticed: the same guarantee
 * `src/ooxml/` gives the constants both halves share.
 *
 * The one difference is empty-element *spelling*: the write path emits `a:rPr` as an
@@ -2005,7 +1994,7 @@ function notesParagraphsXml(text) {
 /**
 * Replace a notes body's paragraphs with `text`, in the body's own document.
 *
-* The path taken when the slide *already* has a notes part — an imported one, or one
+* The path taken when the slide *already* has a notes part: an imported one, or one
 * an earlier `addNotes` wrote. Only the `a:p` children are replaced, so the part's
 * geometry, its `a:bodyPr`/`a:lstStyle`, and its other two placeholders survive. The
 * body's hyperlink relationships are deliberately *not* pruned: a caller replacing
@@ -2047,7 +2036,7 @@ function notesBodyTxBody(root) {
 * When the slide already has a notes part, only the body paragraphs are rewritten
 * (see {@link replaceNotesParagraphs}). When it has none, the part is built and
 * wired: `slide → notesSlide`, and on the new part the two relationships a notes
-* slide always carries — `notesMaster` as rId1 and a `slide` back-reference as rId2,
+* slide always carries: `notesMaster` as rId1 and a `slide` back-reference as rId2,
 * the ids and order the write path reserves.
 *
 * The notes master is resolved through {@link ensureNotesMasterForAuthoring}: this
@@ -2077,8 +2066,7 @@ function authorNotes(slide, text) {
 	slide.relationships.add(NOTES_SLIDE_REL, relativePartName(slide.partName, partName));
 	return partName;
 }
-//#endregion
-//#region src/read/api/comments.ts
+// src/read/api/comments.ts
 /** The slide → modern-comments-part relationship type (2018 schema). Only this module reads one. */
 const MODERN_COMMENTS_REL_TYPE = MS_REL + "2018/10/relationships/comments";
 /** Content type of the modern per-slide comments part (`ppt/comments/modernComment_*.xml`). */
@@ -2199,8 +2187,7 @@ function commentSchema(opc) {
 	if (opc.partsByContentType("application/vnd.openxmlformats-officedocument.presentationml.comments+xml").length > 0) return "legacy";
 	return "none";
 }
-//#endregion
-//#region src/read/api/tags.ts
+// src/read/api/tags.ts
 /** The owner → tags-part relationship type. Only this module follows one. */
 const TAGS_REL_TYPE = OFFICE_REL + "tags";
 /**
@@ -2223,8 +2210,7 @@ function readTagsForPart(opc, ownerPartName) {
 	}
 	return out;
 }
-//#endregion
-//#region src/read/api/transition.ts
+// src/read/api/transition.ts
 /**
 * Typed read/edit model for a slide's `p:transition` (slide-show transition).
 *
@@ -2235,7 +2221,7 @@ function readTagsForPart(opc, ownerPartName) {
 *   when the duration matches a coarse `spd` speed bucket; and
 * - **wrapped** `<mc:AlternateContent>` carrying a `<mc:Choice Requires="p14">`
 *   whose `p:transition` adds the precise `p14:dur` (milliseconds), plus a base
-*   `<mc:Fallback>` without it — emitted for an off-bucket exact duration.
+*   `<mc:Fallback>` without it. This form is emitted for an off-bucket exact duration.
 *
 * The getter prefers the `p14` Choice (so `durationMs` is recovered); the setter
 * emits the `mc:AlternateContent` form when `durationMs` is set and the bare form
@@ -2250,7 +2236,6 @@ function typeElement(transition) {
 }
 /** Namespace prefix for a URI, defaulting to `p` for the base presentationml namespace. */
 function prefixFor(uri) {
-	if (!uri) return "p";
 	for (const [prefix, ns] of Object.entries(OOXML_NS)) if (ns === uri) return prefix;
 	return "p";
 }
@@ -2413,8 +2398,7 @@ function removeTransition(root) {
 	const found = findTransition(root);
 	if (found) root.removeChild(found.outer);
 }
-//#endregion
-//#region src/read/api/slide-build.ts
+// src/read/api/slide-build.ts
 /**
 * Create `qname` in the parent's document and append it.
 *
@@ -2484,8 +2468,7 @@ function buildPicture(doc, spec) {
 	appendSpPr(pic, spec);
 	return pic;
 }
-//#endregion
-//#region src/read/api/slide.ts
+// src/read/api/slide.ts
 /** Recognize a common raster format from its leading bytes, through the shared registry. */
 function sniffImageType(bytes) {
 	const format = imageFormatForBytes(bytes);
@@ -2542,8 +2525,8 @@ var Slide = class {
 	* `fmtScheme` + `fontScheme`, plus the layout/master roots), walked once from
 	* slide → layout → master → theme and cached on this proxy. Backs the read-model
 	* resolved getters ({@link Shape.resolvedFill}, `Run.resolvedColor`,
-	* `Run.resolvedSizePt`, `Run.resolvedFontFace`) so a `schemeClr` token — including
-	* one delivered through a shape's `p:style` `fillRef`/`lnRef` — resolves to a
+	* `Run.resolvedSizePt`, `Run.resolvedFontFace`) so a `schemeClr` token (including
+	* one delivered through a shape's `p:style` `fillRef`/`lnRef`) resolves to a
 	* literal hex, and a placeholder run's inherited size/typeface (a `+mj-*`/`+mn-*`
 	* font token included) resolves to a literal value. The maps/roots are empty when
 	* the theme chain is incomplete, in which case tokens simply stay unresolved.
@@ -2593,7 +2576,7 @@ var Slide = class {
 	/**
 	* Whether this slide draws the master's non-placeholder shapes
 	* (`p:sld/@showMasterSp`). `xsd:boolean` defaulting to `true`, so an absent
-	* attribute means shown — the same shape as {@link hidden}.
+	* attribute means shown, the same shape as {@link hidden}.
 	*
 	* This is the switch that decides whether {@link SlideMaster.shapes} belongs on
 	* *this* slide. A slide the author gave a full-bleed image or a section divider
@@ -2620,7 +2603,7 @@ var Slide = class {
 	* Set or clear the slide's show transition. Assigning `null` removes it.
 	* Writing a transition with `durationMs` emits the `mc:AlternateContent` form
 	* (a `p14` Choice carrying `p14:dur` plus a base `mc:Fallback`); otherwise the
-	* bare `<p:transition>` is written. The node is inserted at its schema slot —
+	* bare `<p:transition>` is written. The node is inserted at its schema slot,
 	* after `p:clrMapOvr`, before `p:timing`/`p:extLst`. Marks the slide part dirty.
 	*
 	* The new transition is built from the current one before that is removed: it keeps the sound
@@ -2646,6 +2629,15 @@ var Slide = class {
 		return root ? hasAnimations(root) : false;
 	}
 	/**
+	* The slide's build effects: every time node in its `p:timing` with a `presetClass`, in document
+	* order, with the sequence it plays in, its preset, trigger, duration, step delay and target shape
+	* id. `[]` when the slide has no timing. Read-only: see {@link BuildEffect}.
+	*/
+	get animations() {
+		const root = this.part.dom.documentElement;
+		return root ? readBuildEffects(root) : [];
+	}
+	/**
 	* @internal The sorted, de-duplicated set of shape ids (`spid`) referenced by
 	* the slide's animations (`<p:spTgt spid>` and `<p:bldP spid>`). Exposed for the
 	* import paths' spid-coherence checks.
@@ -2675,14 +2667,14 @@ var Slide = class {
 	/**
 	* Flatten the slide's build animations: remove the `<p:timing>` block so the
 	* slide renders and edits as its final static state, with every shape shown at
-	* once. Gated like {@link hasAnimations} — a `<p:timing>` that is purely a media
+	* once. Gated like {@link hasAnimations}: a `<p:timing>` that is purely a media
 	* loop (no `<p:bldP>` or `presetID`) is preserved so media playback survives.
 	* Marks the slide part dirty and returns `true` only when a timing block was
 	* removed.
 	*
 	* This removes click-through staging only; it does not delete shapes. If a slide
 	* animated alternating states over the same region, the flattened render shows
-	* them all at once — keep that distinct from removing staged/duplicate shapes.
+	* them all at once. Keep that distinct from removing staged/duplicate shapes.
 	*/
 	flattenAnimations() {
 		const root = this.part.dom.documentElement;
@@ -2698,7 +2690,7 @@ var Slide = class {
 		return spTree ? buildShapes(spTree, this) : [];
 	}
 	/**
-	* All text on the slide, flattened in document order — the read-model
+	* All text on the slide, flattened in document order: the read-model
 	* counterpart to {@link TextFrame.text} one level up. Every text-bearing shape
 	* contributes its text, recursing into groups ({@link GroupShape}) and reading
 	* table cells ({@link GraphicFrame.table}) and SmartArt node text
@@ -2741,12 +2733,12 @@ var Slide = class {
 		return blocks.join("\n");
 	}
 	/**
-	* The slide's speaker-note text — the companion to {@link text} for the notes
+	* The slide's speaker-note text, the companion to {@link text} for the notes
 	* that {@link text} deliberately excludes. Delegates to {@link notesSlide}: finds
 	* the notes body placeholder (`p:ph` `type="body"`) and flattens its text frame
 	* the same way {@link TextFrame.text} does (paragraphs joined by `\n`).
 	*
-	* Returns `null` when the slide has **no notes slide part at all** — distinct
+	* Returns `null` when the slide has **no notes slide part at all**, distinct
 	* from `''`, which means a notes slide exists but its body is empty (PowerPoint
 	* often attaches an empty notes slide to every slide). Only the body placeholder
 	* is read; the slide-thumbnail (`sldImg`) and slide-number (`sldNum`)
@@ -2757,14 +2749,14 @@ var Slide = class {
 		return notes ? notes.text : null;
 	}
 	/**
-	* The slide's speaker-note body as a navigable {@link TextFrame} — the rich
+	* The slide's speaker-note body as a navigable {@link TextFrame}: the rich
 	* companion to {@link notesText}, which flattens the same body to a plain string.
 	* Walk `paragraphs → runs` to recover per-run formatting (bold/italic/underline,
 	* colour, size, face) and any notes hyperlink that {@link notesText} discards.
 	*
 	* `null` when the slide has **no notes slide part** (the same boundary
 	* {@link notesText} returns `null` at), and also when a notes part exists but
-	* carries no body-placeholder text frame — there is then no frame to hand back
+	* carries no body-placeholder text frame; there is then no frame to hand back
 	* (whereas {@link notesText} reports `''` for that empty-body case). Convenience
 	* for `notesSlide.textFrame`; see {@link notesSlide} for the whole modeled notes
 	* slide (its `sldImg`/`sldNum` placeholders and their geometry).
@@ -2776,9 +2768,9 @@ var Slide = class {
 	* The slide's speaker-notes slide (`notesSlideN.xml`) as a modeled
 	* {@link NotesSlide}, or `null` when the slide has no notes slide part. Beyond the
 	* body text {@link notesText}/{@link notesTextFrame} surface, this exposes the
-	* notes slide's three placeholders — the slide thumbnail (`slideImage`), the notes
-	* body, and the slide-number field (`slideNumber`) — each with its geometry and
-	* (where present) a navigable text frame.
+	* notes slide's three placeholders, each with its geometry and (where present) a
+	* navigable text frame: the slide thumbnail (`slideImage`), the notes body, and the
+	* slide-number field (`slideNumber`).
 	*
 	* The writer authors all three placeholders but leaves `sldImg`/`sldNum` with an
 	* empty `p:spPr`, so on an authored deck their geometry reads `null` while an
@@ -2834,7 +2826,7 @@ var Slide = class {
 	* carries its body text, `created` timestamp, marker position (EMU), its author
 	* resolved through the deck-wide {@link Presentation.modernCommentAuthors}
 	* registry (`@authorId` GUID → name/initials), and its reply thread nested under
-	* `replies`. Read-only — the writer authors the legacy schema; see
+	* `replies`. Read-only: the writer authors the legacy schema; see
 	* {@link Presentation.commentSchema} to tell which schema a deck uses.
 	*/
 	get modernComments() {
@@ -2843,7 +2835,7 @@ var Slide = class {
 	/**
 	* The slide's programmatic tags (`p:custDataLst/p:tags` on the slide part,
 	* resolved to `ppt/tags/tagN.xml`) as `{ name, val }` string pairs, `[]` when it
-	* has none. Host/add-in metadata with no visible rendering and no writer —
+	* has none. Host/add-in metadata with no visible rendering and no writer:
 	* read-only, preserved byte-for-byte on round-trip. Deck-level tags are
 	* {@link Presentation.tags}.
 	*/
@@ -2853,7 +2845,7 @@ var Slide = class {
 	/**
 	* The slide's effective background (`p:cSld/p:bg`), decoded into a typed
 	* {@link SlideBackground}. Resolved through the inheritance chain: the slide's
-	* own `p:bg` wins; failing that the slideLayout's, then the slideMaster's — the
+	* own `p:bg` wins; failing that the slideLayout's, then the slideMaster's. The
 	* result's `source` records which supplied it. `null` when nothing in the chain
 	* defines a background.
 	*
@@ -2899,20 +2891,24 @@ var Slide = class {
 	* `slide.slideNumber`, or added to a master defined with `slideNumber`; its geometry and run formatting
 	* read off the returned {@link AutoShape} the same as any placeholder.
 	*
-	* Scoped to the slide's *own* shape tree — a slide number inherited purely from
+	* Scoped to the slide's *own* shape tree: a slide number inherited purely from
 	* the master's `p:hf`/placeholder (with no shape on the slide) is a master-level
 	* concern this getter does not resolve.
 	*/
 	get slideNumberPlaceholder() {
 		return this.placeholder("sldNum") ?? null;
 	}
-	/** The first top-level shape with the given drawing id (`p:cNvPr/@id`), or `undefined`. */
+	/**
+	* The first top-level shape with the given drawing id (`p:cNvPr/@id`), or `undefined`. Takes
+	* `null`, as a shape's `id` reads when its `p:cNvPr` has no `@id`, and misses on it.
+	*/
 	shapeById(id) {
+		if (id === null) return void 0;
 		return this.shapes.find((shape) => shape.id === id);
 	}
 	/**
 	* The first shape *anywhere* in the slide's shape tree with the given drawing id
-	* (`p:cNvPr/@id`), descending into groups — unlike {@link shapeById}, which scans
+	* (`p:cNvPr/@id`), descending into groups, unlike {@link shapeById}, which scans
 	* only top-level shapes. Walked pre-order (a group is visited before its children),
 	* matching how the writer allocates ids. `undefined` when no shape carries that id.
 	*
@@ -2931,8 +2927,9 @@ var Slide = class {
 	/**
 	* The first placeholder of the given type (`p:ph/@type`, e.g. `title`,
 	* `ctrTitle`, `subTitle`, `body`), optionally narrowed by `idx`. Returns
-	* `undefined` when none match. Only `p:sp` shapes can be placeholders, so the
-	* result is an {@link AutoShape}.
+	* `undefined` when none match. Only {@link AutoShape}s are searched: a filled
+	* picture, table or chart placeholder is not found here, though its own
+	* `placeholder` getter still reads it.
 	*/
 	placeholder(type, idx) {
 		return this.shapes.find((shape) => {
@@ -3012,7 +3009,7 @@ var Slide = class {
 	* afterwards through {@link notesTextFrame}.
 	*
 	* The counterpart to the read side's `notesText`/`notesTextFrame`/`notesSlide`
-	* getters, and the way to annotate a slide that has **no notes part at all** —
+	* getters, and the way to annotate a slide that has **no notes part at all**:
 	* the state an `importSlide` without `{ importNotes: true }` leaves behind, and
 	* the one a `notesTextFrame` edit cannot reach because there is no frame to
 	* hand back. Calling it on a slide that already has notes replaces the body
@@ -3067,8 +3064,7 @@ var Slide = class {
 		return spTreeOf(this.part.dom.documentElement);
 	}
 };
-//#endregion
-//#region src/read/api/document-properties.ts
+// src/read/api/document-properties.ts
 /** `dc:title`-style core-property fields, keyed by output field → element qname. */
 const CORE_FIELDS = [
 	["title", "dc:title"],
@@ -3138,7 +3134,7 @@ function readExtendedProperties(opc) {
 * number; real types (`r4`/`r8`/`decimal`) → number; `bool` → boolean;
 * `filetime`/`date` → the **raw string** (see {@link CoreProperties} on
 * timestamps). Any unrecognized `vt:` element falls back to its text content as a
-* string — lossy but never throws.
+* string, which is lossy but never throws.
 */
 function decodeValue(vt) {
 	const text = vt.textContent ?? "";
@@ -3195,13 +3191,12 @@ function readCustomProperties(opc) {
 	}
 	return out;
 }
-//#endregion
-//#region src/read/api/ops/embedded-fonts.ts
+// src/read/api/ops/embedded-fonts.ts
 /**
 * Merging embedded font faces into a deck's `p:embeddedFontLst`.
 *
-* Fonts arrive two ways — copied out of another `Presentation`, or handed over as raw bytes by a
-* generator during an append — and the two differ only in how the binary part comes into being.
+* Fonts arrive two ways (copied out of another `Presentation`, or handed over as raw bytes by a
+* generator during an append), and the two differ only in how the binary part comes into being.
 * That difference is confined to a per-face thunk, so both paths share one merge core: entries
 * join by typeface, faces de-dupe by slot, and the thunk runs only for a face actually being
 * added, so a face the deck already embeds never mints an orphan part.
@@ -3263,7 +3258,7 @@ function readEmbeddedFontEntries(deck) {
 * every face this deck would copy names a part that is actually there.
 *
 * Every import that carries fonts calls it before it changes anything, because the carry runs
-* last — after the page or the masters are already in the deck — and a face it cannot copy would
+* last (after the page or the masters are already in the deck), and a face it cannot copy would
 * throw with parts added and no way back. A face whose `r:id` names no relationship and a face
 * whose relationship names no part are the same failure to a caller, the face's binary is not in
 * the package, so both are `package/part-missing`.
@@ -3346,7 +3341,7 @@ function carryGeneratedEmbeddedFonts(dest, fonts) {
 }
 /**
 * Merge normalized {@link IncomingEmbeddedFont} entries into this deck's
-* `p:embeddedFontLst` — the shared core of {@link carryEmbeddedFonts} (import-side)
+* `p:embeddedFontLst`: the shared core of {@link carryEmbeddedFonts} (import-side)
 * and {@link carryGeneratedEmbeddedFonts} (append-side). Entries merge by `typeface`,
 * faces de-dupe by slot (a face this deck already embeds is left as is). For each newly
 * added face the `fntdata` Default is ensured, the binary part is created via the face's
@@ -3396,10 +3391,128 @@ function mergeEmbeddedFontEntries(dest, entries) {
 		presPart.markDirty();
 	}
 }
-//#endregion
-//#region src/read/api/ops/slide-size.ts
+// src/read/api/ops/comments.ts
+/**
+* Carrying legacy review comments onto a deck: a comments part per slide, and the deck-wide
+* author registry (`ppt/commentAuthors.xml`) every comment names its author in by `@authorId`.
+* `appendSlides` builds the comments part from a generator's comments; `importSlide` and
+* `importSlides` copy the source's part with the page and then renumber it here.
+*
+* The registry is the part that makes this more than a copy. The destination may already have
+* one, with its own ids and its own per-author numbering, so an incoming comment's author is
+* looked up there by name and initials (the key the writer resolves authors by) and its `idx`
+* continues that author's `lastIdx`. An author the destination does not have is added under the
+* next free id.
+*/
+const textEncoder$2 = new TextEncoder();
+/** The deck's comment-author registry part, created and related from the presentation when it has none. */
+function commentAuthorsPart(dest) {
+	const existing = singleRelPart(dest.opc, dest.presentationPart.partName, COMMENT_AUTHORS_REL);
+	if (existing) return existing;
+	const partName = dest.opc.reservePartNameLike("/ppt/commentAuthors.xml");
+	const part = dest.opc.addPart(partName, COMMENT_AUTHORS_CONTENT_TYPE, textEncoder$2.encode(makeXmlCommentAuthors([])));
+	presentationRels(dest).add(COMMENT_AUTHORS_REL, relativePartName(dest.presentationPart.partName, partName));
+	return part;
+}
+/**
+* The deck's author registry, open for registering authors: each call to `register` finds or adds
+* the author by name and initials, takes that author's next `idx`, and marks the part dirty.
+*/
+function authorRegistry(dest) {
+	const part = commentAuthorsPart(dest);
+	const root = part.dom.documentElement;
+	if (!root) throw new PackageReadError("package/part-has-no-root", `${part.partName} has no document element to register comment authors in`);
+	const authors = getElements(root, "p:cmAuthor");
+	return (name, initials) => {
+		let author = authors.find((a) => attr(a, "name") === name && attr(a, "initials") === initials);
+		if (!author) {
+			const id = Math.max(-1, ...authors.map((a) => numberValue(attr(a, "id")) ?? -1)) + 1;
+			author = createElement(part.dom, "p:cmAuthor");
+			setAttr(author, "id", String(id));
+			setAttr(author, "name", name);
+			setAttr(author, "initials", initials);
+			setAttr(author, "lastIdx", "0");
+			setAttr(author, "clrIdx", String(id));
+			root.appendChild(author);
+			authors.push(author);
+		}
+		const idx = (numberValue(attr(author, "lastIdx")) ?? 0) + 1;
+		setAttr(author, "lastIdx", String(idx));
+		part.markDirty();
+		return {
+			authorId: numberValue(attr(author, "id")) ?? 0,
+			idx
+		};
+	};
+}
 /**
-* ts-pptx: slide-size preconditions for the import surface
+* Add a comments part holding `comments`, registering each author in the deck's
+* `commentAuthors.xml`. The slide's own relationship *to* the part is left to the caller, as
+* `addNotesSlidePart` leaves the notes one, because the append path adds it after every id
+* the slide body names.
+* @param dest - the deck the part is added to
+* @param comments - the slide's comments, in order
+* @returns the comments part's name
+*/
+function addCommentsPart(dest, comments) {
+	const register = authorRegistry(dest);
+	const meta = new Map(comments.map((comment) => [comment, register(comment.author, comment.initials)]));
+	const partName = dest.opc.reservePartNameLike("/ppt/comments/comment1.xml");
+	dest.opc.addPart(partName, COMMENTS_CONTENT_TYPE, textEncoder$2.encode(makeXmlComments(comments, meta)));
+	return partName;
+}
+/**
+* Renumber the comments part a page copy brought from `source` against this deck's registry
+* (#235). The copy keeps each `@authorId` and `@idx` as the source numbered them, which on this
+* deck names no author, or the wrong one. Each comment's author is resolved in the source's
+* registry and registered in this deck's, and the comment takes the id and the next `idx` it gets
+* there. A comment whose author the source itself does not register is left as it is.
+*
+* Runs once per page copy, after the copy: a comments part is owned by its page (see
+* `page-owned.ts`), so every copy has a part of its own to renumber.
+* @param dest - the deck the page was copied into
+* @param source - the deck it was copied from
+* @param slidePartName - the copied page in `dest`
+*/
+function carryCommentAuthors(dest, source, slidePartName) {
+	const part = singleRelPart(dest.opc, slidePartName, COMMENTS_REL);
+	const root = part?.dom.documentElement;
+	if (!part || !root) return;
+	const comments = getElements(root, "p:cm");
+	if (comments.length === 0) return;
+	const sourceAuthors = new Map(readCommentAuthors(source.opc, source.presentationPart.partName).map((a) => [a.id, a]));
+	const register = authorRegistry(dest);
+	const renumber = (cm) => {
+		const author = sourceAuthors.get(numberValue(attr(cm, "authorId")));
+		if (!author) return;
+		const { authorId, idx } = register(author.name, author.initials);
+		setAttr(cm, "authorId", String(authorId));
+		setAttr(cm, "idx", String(idx));
+	};
+	comments.forEach(renumber);
+	part.markDirty();
+}
+/**
+* Reset every author's `lastIdx` in the deck's registry to 0, for a deck with no comments left
+* (#244). `fromTemplate` strips the template's slides, and their comments with them, but the
+* registry keeps counting from the comments it removed, so the next comment would be numbered
+* after one the deck no longer has. `lastIdx` is the author's highest used index (ECMA-376
+* §19.4.2), and in an empty deck that is none. The authors themselves stay, with their ids and
+* colours, so a comment re-authored under the same name lands on the same entry.
+* @param dest - the deck whose slides were all removed
+*/
+function resetCommentAuthorIndexes(dest) {
+	const part = singleRelPart(dest.opc, dest.presentationPart.partName, COMMENT_AUTHORS_REL);
+	const root = part?.dom.documentElement;
+	if (!part || !root) return;
+	const authors = getElements(root, "p:cmAuthor").filter((author) => attr(author, "lastIdx") !== "0");
+	if (authors.length === 0) return;
+	for (const author of authors) setAttr(author, "lastIdx", "0");
+	part.markDirty();
+}
+// src/read/api/ops/slide-size.ts
+/**
+* pptx-ts: slide-size preconditions for the import surface
 *
 * Every import entry point starts by asking whether the two decks agree on canvas size, and
 * the five that asked wrote out the same comparison, the same `unknown`-printing formatter
@@ -3409,9 +3522,9 @@ function mergeEmbeddedFontEntries(dest, entries) {
 * except `importSlide`, and only when a rescale had been asked for.
 *
 * The two conditions are separated here, and the codes now mean what their names say:
-* - `import/slide-size-unknown` — a deck does not declare `p:sldSz`, so there is no size to
+* - `import/slide-size-unknown`: a deck does not declare `p:sldSz`, so there is no size to
 *   compare and no rescale to compute. Nothing the caller passes can make it comparable.
-* - `import/slide-size-mismatch` — both sizes are known and they differ. This one *is*
+* - `import/slide-size-mismatch`: both sizes are known and they differ. This one *is*
 *   answerable, which is why each caller supplies the escape hatch it offers.
 */
 /** A size as the error messages print it. */
@@ -3422,7 +3535,7 @@ function fmt(size) {
 * Both decks' sizes, or a throw when either deck does not declare one.
 *
 * Returned rather than asserted because every caller that needs both sizes known needs the
-* values too — to compute a rescale — and TypeScript cannot narrow two parameters from one
+* values too, to compute a rescale, and TypeScript cannot narrow two parameters from one
 * assertion signature.
 *
 * @param target - this deck's slide size
@@ -3447,19 +3560,18 @@ function requireEqualSlideSize(target, incoming, api, hint = "") {
 	if (known.widthEmu === source.widthEmu && known.heightEmu === source.heightEmu) return;
 	throw new InvalidOptionError("import/slide-size-mismatch", `${api} requires equal slide sizes${hint ? ` (${hint})` : ""}; target is ${fmt(target)}, source is ${fmt(incoming)}`);
 }
-/** Whether both sizes are known and equal — the question a caller with a rescale asks first. */
+/** Whether both sizes are known and equal: the question a caller with a rescale asks first. */
 function slideSizesMatch(target, incoming) {
 	return !!target && !!incoming && target.widthEmu === incoming.widthEmu && target.heightEmu === incoming.heightEmu;
 }
-//#endregion
-//#region src/read/api/ops/append-slides.ts
+// src/read/api/ops/append-slides.ts
 /**
-* ts-pptx: the body of {@link Presentation.appendSlides}
+* pptx-ts: the body of {@link Presentation.appendSlides}
 *
 * The hybrid "generate onto an existing deck" path: a slide producer authors slides, they are
-* spliced into this package under fresh partnames, and every dependency each one carries —
-* images, embedded and online media, hyperlinks, speaker notes, charts, intra-batch slide links
-* — is wired into its `.rels`.
+* spliced into this package under fresh partnames, and every dependency each one carries
+* (images, embedded and online media, hyperlinks, speaker notes, review comments, charts,
+* intra-batch slide links) is wired into its `.rels`.
 *
 * Here rather than on the class for the reason `presentation.ts` states about the four import
 * bodies: the *contract* is the doc comment a caller reads and stays on the public method, while
@@ -3472,12 +3584,12 @@ function slideSizesMatch(target, incoming) {
 */
 const textEncoder$1 = new TextEncoder();
 /**
-* Resolve `options.layout` — a layout *name* or a {@link LayoutHandle} — against this deck's
-* gallery, with no silent fallback: an unusable value is one of three named errors rather than a
+* Resolve `options.layout` (a layout *name* or a {@link LayoutHandle}) against this deck's
+* gallery, with no silent fallback: an unusable value is one of four named errors rather than a
 * default layout, because a slide bound to the wrong layout is a deck that looks authored.
 *
 * Split out because it is the part of the append most likely to be wanted by a second caller, and
-* because it is the only place `layout/ambiguous-name`, `layout/not-found` and
+* because it is the only place `layout/missing`, `layout/ambiguous-name`, `layout/not-found` and
 * `layout/foreign-handle` are raised.
 *
 * @param deck - the destination deck, whose `layouts()` gallery is searched
@@ -3486,6 +3598,8 @@ const textEncoder$1 = new TextEncoder();
 */
 function resolveLayoutTarget(deck, layout, api = "appendSlides") {
 	const gallery = deck.layouts();
+	const names = () => gallery.map((l) => JSON.stringify(l.name)).join(", ") || "(none)";
+	if (layout == null) throw new InvalidOptionError("layout/missing", `${api}: options.layout is required, a layout name or a LayoutHandle from layouts(); available: ${names()}`);
 	if (typeof layout !== "string") {
 		if (!gallery.some((l) => l.partName === layout.partName)) throw new InvalidOptionError("layout/foreign-handle", `${api}: layout ${layout.partName} does not belong to this presentation`);
 		return layout;
@@ -3493,10 +3607,7 @@ function resolveLayoutTarget(deck, layout, api = "appendSlides") {
 	const matches = gallery.filter((l) => l.name === layout);
 	if (matches.length > 1) throw new InvalidOptionError("layout/ambiguous-name", `${api}: layout name ${JSON.stringify(layout)} is ambiguous (${matches.length} layouts share it); pass a LayoutHandle from layouts() instead`);
 	const [only] = matches;
-	if (!only) {
-		const names = gallery.map((l) => JSON.stringify(l.name)).join(", ");
-		throw new InvalidOptionError("layout/not-found", `${api}: no layout named ${JSON.stringify(layout)}; available: ${names || "(none)"}`);
-	}
+	if (!only) throw new InvalidOptionError("layout/not-found", `${api}: no layout named ${JSON.stringify(layout)}; available: ${names()}`);
 	return only;
 }
 /** Image parts behind the slide body's `image` rels, each keeping the id the body was built with. */
@@ -3528,8 +3639,20 @@ function wireAvMedia(deck, rels, partName, slide) {
 	}
 }
 /**
-* Online (external-link) video: two External rels share the link Target — the ECMA video rel and
-* the MS-2007 media rel — with no media binary part and no content-type entry. The poster image is
+* A transition's start sound: one audio part behind the `audio` rel the body's `p:snd r:embed`
+* names. Registered as a Default extension entry, as {@link wireAvMedia} does.
+*/
+function wireTransitionSound(deck, rels, partName, slide) {
+	const sound = slide.transitionSound;
+	if (!sound) return;
+	const mediaPartName = deck.opc.reserveMediaPartName(sound.extn, "media");
+	deck.opc.contentTypes.ensureDefault(sound.extn, sound.contentType);
+	deck.opc.addPart(mediaPartName, sound.contentType, sound.bytes);
+	rels.addWithId(`rId${sound.rId}`, AUDIO_REL, relativePartName(partName, mediaPartName));
+}
+/**
+* Online (external-link) video: two External rels share the link Target (the ECMA video rel and
+* the MS-2007 media rel), with no media binary part and no content-type entry. The poster image is
 * wired by {@link wireMedia}, which runs first.
 */
 function wireOnlineMedia(rels, slide) {
@@ -3544,7 +3667,7 @@ function wireHyperlinks(rels, slide) {
 }
 /**
 * Speaker notes. The notes part carries its own rel namespace, independent of the slide's:
-* rId1 = notesMaster, rId2 = the slide it annotates, hyperlinks from rId3 — the order the
+* rId1 = notesMaster, rId2 = the slide it annotates, hyperlinks from rId3. That is the order the
 * generator's body was serialized against, so these are added by explicit id rather than left to
 * auto-numbering.
 *
@@ -3552,12 +3675,13 @@ function wireHyperlinks(rels, slide) {
 * so {@link wireAutoIdRels} adds it once every id the body does name is claimed.
 * @returns the notes part's name, or `null` when the slide has no notes
 */
-function wireNotes(deck, partName, slide, sourceNotesMaster) {
+function wireNotes(deck, partName, slide, sourceNotesMaster, partBySourceNumber, index) {
 	if (!slide.notes) return null;
 	const notesMasterPartName = sourceNotesMaster ? ensureNotesMasterFromXml(deck, sourceNotesMaster) : null;
 	const notesPartName = addNotesSlidePart(deck, slide.notes.xml, partName, notesMasterPartName);
 	const notesRels = deck.opc.relationshipsFor(notesPartName);
 	for (const h of slide.notes.hyperlinks) notesRels.addWithId(`rId${h.rId}`, HYPERLINK_REL, h.target, "External");
+	wireSlideLinks(notesRels, notesPartName, slide.notes.slideLinks ?? [], partBySourceNumber, index);
 	return notesPartName;
 }
 /**
@@ -3567,7 +3691,7 @@ function wireNotes(deck, partName, slide, sourceNotesMaster) {
 *
 * A chartEx chart differs in every other coordinate: the `chartEx{N}.xml` name family, the MS
 * chartex content type, the MS `chartEx` rel off the slide, and two mandatory sidecars at
-* rId2/rId3 — without which PowerPoint reports the deck as corrupt (schema-valid but unopenable).
+* rId2/rId3, without which PowerPoint reports the deck as corrupt (schema-valid but unopenable).
 * `c.chartEx` carries both sidecar bodies; the generator built them, so nothing here reaches into
 * the emitters.
 */
@@ -3581,6 +3705,11 @@ function wireCharts(deck, rels, partName, slide) {
 		deck.opc.addPart(embeddingPartName, XLSX_CONTENT_TYPE, c.embeddingBytes);
 		const chartRels = deck.opc.relationshipsFor(chartPartName);
 		chartRels.addWithId("rId1", PACKAGE_REL, relativePartName(chartPartName, embeddingPartName));
+		for (const picture of c.pictures ?? []) {
+			const mediaPartName = deck.opc.reserveMediaPartName(picture.extn);
+			deck.opc.addPart(mediaPartName, picture.contentType, picture.bytes);
+			chartRels.addWithId(`rId${picture.rId}`, IMAGE_REL, relativePartName(chartPartName, mediaPartName));
+		}
 		if (c.chartEx) {
 			const colorsPartName = deck.opc.reservePartNameLike("/ppt/charts/colors1.xml");
 			deck.opc.addPart(colorsPartName, CHART_COLOR_STYLE_CONTENT_TYPE, textEncoder$1.encode(c.chartEx.colorsXml));
@@ -3593,12 +3722,12 @@ function wireCharts(deck, rels, partName, slide) {
 	}
 }
 /**
-* Intra-batch slide-to-slide links: a `slide:N` in the source is repointed at the Nth appended
-* slide's new partname, which is why pass 1 places every slide before any of this runs. A link out
+* Intra-batch slide-to-slide links, from a slide or from its notes: a `slide:N` in the source is
+* repointed at the Nth appended slide's new partname, which is why pass 1 places every slide before any of this runs. A link out
 * of the batch was refused before pass 1 by {@link requireLinksInBatch}.
 */
-function wireSlideLinks(rels, partName, slide, partBySourceNumber, index) {
-	for (const link of slide.slideLinks) {
+function wireSlideLinks(rels, partName, links, partBySourceNumber, index) {
+	for (const link of links) {
 		const targetPartName = partBySourceNumber.get(link.sourceSlideNumber);
 		if (!targetPartName) throw new InternalError("import/slide-link-not-placed", `appendSlides: slide ${index} links to source slide ${link.sourceSlideNumber}, which was checked against the batch but has no appended slide`);
 		rels.addWithId(`rId${link.rId}`, SLIDE_REL, relativePartName(partName, targetPartName));
@@ -3611,12 +3740,13 @@ function wireSlideLinks(rels, partName, slide, partBySourceNumber, index) {
 */
 function requireLinksInBatch(slides) {
 	slides.forEach((slide, index) => {
-		for (const link of slide.slideLinks) if (link.sourceSlideNumber > slides.length) throw new InvalidOptionError("import/unresolved-slide-link", `appendSlides: slide ${index} links to source slide ${link.sourceSlideNumber}, which is not among the appended slides`);
+		for (const link of [...slide.slideLinks, ...slide.notes?.slideLinks ?? []]) if (link.sourceSlideNumber > slides.length) throw new InvalidOptionError("import/unresolved-slide-link", `appendSlides: slide ${index} links to source slide ${link.sourceSlideNumber}, which is not among the appended slides`);
 	});
 }
 /**
-* The first of pass 2's two phases: every rel the slide body names by id — media, embedded and
-* online media, hyperlinks, charts, slide links — each added with the rId the generator wrote.
+* The first of pass 2's two phases: every rel the slide body names by id (media, embedded and
+* online media, a transition sound, hyperlinks, charts, slide links), each added with the rId the
+* generator wrote.
 * Also creates the notes part, whose own rels are fixed ids too.
 *
 * A new rel the body references by id belongs here, and nowhere after {@link wireAutoIdRels}.
@@ -3626,24 +3756,26 @@ function wireFixedIdRels(deck, rels, placed, sourceNotesMaster, partBySourceNumb
 	const { slide, partName } = placed;
 	wireMedia(deck, rels, partName, slide);
 	wireAvMedia(deck, rels, partName, slide);
+	wireTransitionSound(deck, rels, partName, slide);
 	wireOnlineMedia(rels, slide);
 	wireHyperlinks(rels, slide);
-	const notesPartName = wireNotes(deck, partName, slide, sourceNotesMaster);
+	const notesPartName = wireNotes(deck, partName, slide, sourceNotesMaster, partBySourceNumber, index);
 	wireCharts(deck, rels, partName, slide);
-	wireSlideLinks(rels, partName, slide, partBySourceNumber, index);
+	wireSlideLinks(rels, partName, slide.slideLinks, partBySourceNumber, index);
 	return notesPartName;
 }
 /**
-* The second phase: the rels the slide body names by no id — its notes and its layout — numbered
-* by `add()`, which takes one past the highest id present.
+* The second phase: the rels the slide body names by no id (its notes, its comments and its
+* layout), numbered by `add()`, which takes one past the highest id present.
 *
 * Strictly after {@link wireFixedIdRels}. The generator numbers a slide's rels from rId1, so an
 * auto id taken before a fixed one is claimed is routinely that fixed one: the notes rel used to
 * be added in the middle of the first phase, and a slide carrying notes and a chart or a slide
 * link threw `relationship/duplicate-id`.
 */
-function wireAutoIdRels(rels, partName, notesPartName, layoutPartName) {
+function wireAutoIdRels(rels, partName, notesPartName, commentsPartName, layoutPartName) {
 	if (notesPartName) rels.add(NOTES_SLIDE_REL, relativePartName(partName, notesPartName));
+	if (commentsPartName) rels.add(COMMENTS_REL, relativePartName(partName, commentsPartName));
 	rels.add(SLIDE_LAYOUT_REL, relativePartName(partName, layoutPartName));
 }
 /**
@@ -3651,7 +3783,7 @@ function wireAutoIdRels(rels, partName, notesPartName, layoutPartName) {
 * {@link Presentation.appendSlides}.
 */
 async function appendSlides(deck, source, options) {
-	const target = resolveLayoutTarget(deck, options.layout);
+	const target = resolveLayoutTarget(deck, options?.layout);
 	const extracted = await source.extractSlides(pickDefined(options, ["onMediaError"]));
 	requireEqualSlideSize(deck.slideSize, {
 		widthEmu: extracted.widthEmu,
@@ -3675,15 +3807,16 @@ async function appendSlides(deck, source, options) {
 	placed.forEach((placedSlide, i) => {
 		const rels = deck.opc.relationshipsFor(placedSlide.partName);
 		const notesPartName = wireFixedIdRels(deck, rels, placedSlide, extracted.notesMaster, partBySourceNumber, i);
-		wireAutoIdRels(rels, placedSlide.partName, notesPartName, target.partName);
+		const comments = placedSlide.slide.comments;
+		const commentsPartName = comments?.length ? addCommentsPart(deck, comments) : null;
+		wireAutoIdRels(rels, placedSlide.partName, notesPartName, commentsPartName, target.partName);
 		const at = options.at === void 0 ? void 0 : options.at + i;
 		added.push(deck.insertSlidePart(placedSlide.part, at));
 	});
 	carryGeneratedEmbeddedFonts(deck, extracted.embeddedFonts || []);
 	return added;
 }
-//#endregion
-//#region src/read/api/ops/prune.ts
+// src/read/api/ops/prune.ts
 /**
 * Content types that are shared deck chrome, not owned by any one slide: the `deck` rows of
 * {@link SHARED_PARTS}, reached through the presentation → master → layout → theme graph or named
@@ -3815,8 +3948,7 @@ function isReferenced(pres, partName) {
 function relationshipOwners(pres) {
 	return [...[...pres.opc.parts.keys()].filter((partName) => !partName.endsWith(".rels")), "/"];
 }
-//#endregion
-//#region src/read/api/ops/import-memo.ts
+// src/read/api/ops/import-memo.ts
 var ImportMemo = class {
 	#registries = /* @__PURE__ */ new Map();
 	#rescaled = /* @__PURE__ */ new Map();
@@ -3840,14 +3972,13 @@ var ImportMemo = class {
 		for (const partName of gone) this.#rescaled.delete(partName);
 	}
 };
-//#endregion
-//#region src/read/api/ops/flatten.ts
+// src/read/api/ops/flatten.ts
 /**
 * Baking a slide's theme dependencies into the slide itself, for `Presentation.importSlide` /
 * `importShape`.
 *
 * A slide imported into another deck is rebound to the *destination* master, so everything it
-* resolved symbolically against its own theme silently re-resolves against a different one — its
+* resolved symbolically against its own theme silently re-resolves against a different one: its
 * scheme colours change, its style-matrix fills change, its placeholders snap to different
 * geometry and type sizes. The `theme: 'preserve'` mode exists to stop that: each pass here
 * resolves a dependency against the **source** theme (via `read/oxml/theme.ts` and
@@ -3856,25 +3987,25 @@ var ImportMemo = class {
 *
 * What gets flattened, and why each one matters:
 *
-* 1. **`a:schemeClr` tokens** — rewritten to the literal `a:srgbClr` the source `clrMap` +
+* 1. **`a:schemeClr` tokens**: rewritten to the literal `a:srgbClr` the source `clrMap` +
 *    `clrScheme` produce. Child transforms (`lumMod`/`shade`/`alpha`/…) are carried through
 *    untouched, so tints and shades render identically; only the *base* reference is swapped,
 *    never the transform math.
-* 2. **Style-matrix refs** — a shape's `p:style` (`lnRef`/`fillRef`/`effectRef`) indexes the
+* 2. **Style-matrix refs**: a shape's `p:style` (`lnRef`/`fillRef`/`effectRef`) indexes the
 *    theme `fmtScheme`; the indexed entry is resolved into an explicit `spPr` fill/line/effect
 *    (with its `phClr` substituted) and the ref neutralized so it cannot re-resolve. The
-*    `fontRef` is deliberately left intact so its font can re-bind to the destination theme —
-*    the "normalize fonts on attach" bonus.
-* 3. **Placeholder-inherited run colour, size and weight** — a run that sets none of its own
+*    `fontRef` is deliberately left intact so its font can re-bind to the destination theme
+*    (the "normalize fonts on attach" bonus).
+* 3. **Placeholder-inherited run colour, size and weight**: a run that sets none of its own
 *    takes them from the source placeholder → layout → master text-style chain. Rebinding
 *    replaces that chain, so each is resolved from the source styles and written onto the run.
-*    Typeface (`a:latin`) is deliberately *not* baked — it re-binds along with `fontRef`.
+*    Typeface (`a:latin`) is deliberately *not* baked; it re-binds along with `fontRef`.
 *    A shape that is not a placeholder takes the same properties from the presentation's
 *    `p:defaultTextStyle` instead, which the rebind swaps for the destination's, so those
 *    are baked from the source one. A table cell's text takes its size from the master's
 *    `p:otherStyle`, and its weight too where the table style states none, so those are
 *    baked from the source master.
-* 4. **Placeholder-inherited geometry** — a placeholder with no own `a:xfrm` takes position and
+* 4. **Placeholder-inherited geometry**: a placeholder with no own `a:xfrm` takes position and
 *    size from the matching source layout/master placeholder; without baking it, a title snaps
 *    to the destination default and often clips off-canvas.
 *
@@ -3897,7 +4028,7 @@ var ImportMemo = class {
 * 4. bake each placeholder run's effective colour and size/weight (inherited from
 *    the source layout/master text styles) explicitly onto the run, each other
 *    shape's run the ones it takes from the source `p:defaultTextStyle`, and each
-*    table cell's run the ones it takes from the source `p:otherStyle`;
+*    table cell's run the ones it takes from the source `p:otherStyle` and theme;
 * 5. rewrite every remaining `a:schemeClr` to its literal `a:srgbClr`.
 *
 * Steps run in this order so the inherited/materialized backgrounds are present
@@ -3917,7 +4048,7 @@ function flattenSlide(slideRoot, ctx) {
 	resolveSchemeColors(slideRoot, ctx);
 }
 /**
-* Flatten a single lifted shape's theme dependencies in place — the shape-scoped
+* Flatten a single lifted shape's theme dependencies in place: the shape-scoped
 * subset of {@link flattenSlide} used by `Presentation.importShape` `preserve`
 * mode. It runs every pass that resolves a *shape's* theme references against the
 * source theme (style-matrix refs, placeholder-inherited geometry/colour/size/
@@ -3938,7 +4069,7 @@ function flattenSlide(slideRoot, ctx) {
 *
 * `shapeRoot` must be an element whose *descendants* include the lifted shape
 * (the passes match via `getElementsByTagNameNS`, which excludes the root element
-* itself) — the caller wraps the imported `p:sp`/`p:pic`/`p:graphicFrame`/`p:grpSp`
+* itself), so the caller wraps the imported `p:sp`/`p:pic`/`p:graphicFrame`/`p:grpSp`
 * in a throwaway container before calling. The caller marks the part dirty.
 */
 function flattenShape(shapeRoot, ctx) {
@@ -3955,7 +4086,7 @@ function flattenShape(shapeRoot, ctx) {
 }
 /**
 * Restyle a rebound slide in place (the `theme: 'restyle'` mode): the exact
-* inverse of {@link flattenSlide}. It bakes *nothing* — leaving every
+* inverse of {@link flattenSlide}. It bakes *nothing*: leaving every
 * `a:schemeClr`/style-matrix ref and `p:bg` `bgRef` symbolic is the whole point,
 * so they re-resolve against the *destination* master's `clrMap` + theme once the
 * slide is rebound. The single mutation is dropping the slide's own colour-map
@@ -3978,14 +4109,14 @@ function restyleSlide(slideRoot) {
 * destination theme; a literal matching no slot (the common case) is left exactly
 * as authored. Transform children (`lumMod`/`shade`/`alpha`/…) are carried across.
 *
-* Plain `restyle` cannot re-brand a literal — it carries no theme reference. This
+* Plain `restyle` cannot re-brand a literal, since it carries no theme reference. This
 * re-introduces one by matching the literal's RGB against the slots the *source*
 * theme defined, emitting the scheme token that routes through the source `clrMap`
 * (so a literal equal to the source `accent1` slot becomes `schemeClr accent1`,
 * which the destination `clrMap`/theme then re-resolve). Slots are matched in
 * `clrScheme` order, so the first slot defining a given RGB wins when several share
 * it. Opt-in because it deliberately reinterprets authored literals as theme
-* colours — visual QA territory, the same caveat as the rest of `restyle`.
+* colours: visual QA territory, the same caveat as the rest of `restyle`.
 */
 function remapLiteralColors(slideRoot, ctx) {
 	const slotByHex = reverseClrScheme(ctx.clrScheme);
@@ -4074,7 +4205,7 @@ function resolvePlaceholderRunColors(slideRoot, ctx) {
 * `resolve` is asked once per *list level* per shape and memoized, because walking the
 * layout/master style chain is the expensive half and every paragraph at the same level gets
 * the same answer. `write` then runs per run, and owns the decision to skip a run the slide
-* already fixes — the two passes ask that question differently (one colour check versus three
+* already fixes. The two passes ask that question differently (one colour check versus three
 * independent property checks), so it stays on their side of the boundary rather than becoming
 * a predicate parameter only one caller could use.
 *
@@ -4169,7 +4300,7 @@ function resolvePlaceholderGeometry(slideRoot, ctx) {
 * We resolve the effective anchor from the source chain and write it explicitly.
 * Shapes whose `a:bodyPr` already fixes `@anchor`, and placeholders with no
 * resolvable inherited anchor, are left untouched. Other `a:bodyPr` knobs (insets,
-* autofit) are left as authored — they have sane defaults a plain shape keeps.
+* autofit) are left as authored; they have sane defaults a plain shape keeps.
 */
 function resolvePlaceholderBodyPr(shapeRoot, ctx) {
 	forEachPlaceholderTextBody(shapeRoot, ctx, (txBody, type, idx) => {
@@ -4182,21 +4313,20 @@ function resolvePlaceholderBodyPr(shapeRoot, ctx) {
 }
 /**
 * Bake the placeholder-inherited list style onto a lifted shape's text body (the
-* {@link flattenShape} placeholder-demotion path). A placeholder's per-level
-* *paragraph* formatting — indent (`marL`/`indent`), bullets (`a:buChar`/
-* `a:buAutoNum`/`a:buNone`), alignment, and the level `a:defRPr` — is inherited
-* from the source layout/master placeholder `a:lstStyle` and the master
-* `p:txStyles` category style. Run colour/size are baked onto each run by the
-* sibling passes, but paragraph-level defaults live here; once
-* {@link demotePlaceholders} strips the `p:ph` the shape stops inheriting them, so
-* bullets and indents would vanish. We materialize the effective list style onto
-* the shape's `p:txBody/a:lstStyle`.
+* {@link flattenShape} placeholder-demotion path). A placeholder inherits its per-level
+* *paragraph* formatting from the source layout/master placeholder `a:lstStyle` and
+* the master `p:txStyles` category style: indent (`marL`/`indent`), bullets
+* (`a:buChar`/`a:buAutoNum`/`a:buNone`), alignment, and the level `a:defRPr`. Run
+* colour/size are baked onto each run by the sibling passes, but paragraph-level
+* defaults live here; once {@link demotePlaceholders} strips the `p:ph` the shape
+* stops inheriting them, so bullets and indents would vanish. We materialize the
+* effective list style onto the shape's `p:txBody/a:lstStyle`.
 *
 * Resolution is per *level*, most specific tier wins whole: the slide's own level
 * (if it defines one) is kept verbatim, else the first source tier that defines it
 * (layout placeholder, then master placeholder, then master category style) is
 * cloned. This is a whole-element overlay, not the per-attribute merge PowerPoint
-* does — but explicit paragraph `a:pPr` on the slide's own runs travels with the
+* does, but explicit paragraph `a:pPr` on the slide's own runs travels with the
 * shape and still wins, so this only supplies defaults for the inherited case
 * (paragraphs that set no `a:pPr` of their own). Scheme colours in the cloned
 * levels are resolved by the later `resolveSchemeColors` pass.
@@ -4223,7 +4353,7 @@ function resolvePlaceholderListStyle(shapeRoot, ctx) {
 * Strip the `p:ph` marker from every placeholder shape in a lifted subtree so it
 * becomes a self-contained ordinary shape (the {@link flattenShape} path only). By
 * the time this runs, the shape's inherited geometry, colour, run size, anchor, and
-* list style are all baked explicitly, so it no longer needs — and must not keep —
+* list style are all baked explicitly, so it no longer needs (and must not keep)
 * its placeholder identity: a surviving `p:ph` would re-resolve against the *host*
 * deck's layout/master placeholder of the same type/idx (wrong inheritance, or a
 * fallback when the host has none) and could collide with the host slide's own
@@ -4241,7 +4371,7 @@ function demotePlaceholders(shapeRoot) {
 * {@link resolvePlaceholderRunColors}: for each placeholder run that sets no
 * `sz`/`b`/`i` of its own (nor at paragraph/text-body level on the *slide*),
 * resolve the value it would inherit from the source `slideLayout`/`slideMaster`
-* text styles — per paragraph list level — and write it explicitly onto the run's
+* text styles, per paragraph list level, and write it explicitly onto the run's
 * `a:rPr`. Each property resolves independently up the chain. Typeface (`a:latin`)
 * is left to re-bind to the destination theme, as the colour pass does for the `fontRef`.
 */
@@ -4294,25 +4424,26 @@ function resolveDefaultTextStyleRuns(root, ctx) {
 	}
 }
 /**
-* Bake what a table cell's runs take from the source master's `p:otherStyle`: size, and bold and
-* italic where the cell's table style states neither, per paragraph level, wherever the run, its
-* paragraph and the cell's own list style state none of them.
+* Bake what a table cell's runs take from the source deck: size, and bold and italic where the
+* cell's table style states neither, from the master's `p:otherStyle` per paragraph level, and the
+* colour from the table style's text style, else `tx1`, resolved against the source theme. Each is
+* written wherever the run, its paragraph and the cell's own list style state none of it.
 *
 * Cell text resolves through `p:otherStyle`, not `p:defaultTextStyle` (`TableCell.textFrame`), and
-* the rebind swaps the source master for the destination's, so without this a cell's text changes
-* size between two decks whose masters differ there. PowerPoint's Keep Source Formatting paste
-* writes nothing onto the cells; it keeps them on a copy of the source master instead
-* (`test/read/fixtures/authoring/probe-table-text-paste.ps1`), which is the look this pins.
+* the rebind swaps the source master and theme for the destination's, so without this a cell's text
+* changes size and colour between two decks whose masters or themes differ. PowerPoint's Keep Source
+* Formatting paste writes nothing onto the cells; it keeps them on a copy of the source master
+* instead (`test/read/fixtures/authoring/probe-table-text-paste.ps1`), which is the look this pins.
 *
-* The table style travels with the slide, so what it states is left to it. Where the table names a
-* style the source does not define, a built-in PowerPoint draws from its own definition, and bold
-* and italic are left alone because what that style states is unknown. A cell's colour and face do
-* not come from `p:otherStyle` at all, so they are not visited.
+* Only the table's style id travels with the slide, and PowerPoint resolves that id against its own
+* built-in gallery, never a definition in the package. The source's `tableStyles.xml` says what the
+* style states, so bold and italic it states are left to it. Where the source does not define the
+* style, bold, italic and colour are left alone because what the style states is unknown. The face
+* re-binds to the destination theme, as it does for every other text tier.
 */
 function resolveTableCellRuns(root, ctx) {
 	const txStyles = ctx.masterRoot ? firstChild(ctx.masterRoot, "p:txStyles") : null;
 	const otherStyle = txStyles && firstChild(txStyles, "p:otherStyle");
-	if (!otherStyle) return;
 	for (const tbl of descendantsByTag(root, OOXML_NS.a, "tbl")) {
 		const tblPr = firstChild(tbl, "a:tblPr");
 		const styleId = (tblPr && firstChild(tblPr, "a:tableStyleId"))?.textContent?.trim() || null;
@@ -4335,25 +4466,51 @@ function resolveTableCellRuns(root, ctx) {
 				const txBody = firstChild(tc, "a:txBody");
 				if (!txBody) return;
 				const styled = style ? resolveTableCellTextStyle(style, flags, rowIndex, colIndex, rows.length, colCount, ctx) : null;
+				const color = styleUnknown ? null : tableCellTextColor(styled, ownerDocumentOf(tbl), ctx);
 				const cellLst = firstChild(txBody, "a:lstStyle");
 				for (const p of getElements(txBody, "a:p")) {
 					const runs = [...getElements(p, "a:r"), ...getElements(p, "a:fld")];
 					if (runs.length === 0) continue;
 					const pPr = firstChild(p, "a:pPr");
 					const level = (pPr && numberValue(attr(pPr, "lvl"))) ?? 0;
-					const defRPr = lstStyleLevelDefRPr(otherStyle, level);
-					if (!defRPr) continue;
+					const defRPr = otherStyle && lstStyleLevelDefRPr(otherStyle, level);
 					const props = {
-						sz: attr(defRPr, "sz"),
-						b: styleUnknown || styled?.bold != null ? null : attr(defRPr, "b"),
-						i: styleUnknown || styled?.italic != null ? null : attr(defRPr, "i")
+						sz: defRPr ? attr(defRPr, "sz") : null,
+						b: !defRPr || styleUnknown || styled?.bold != null ? null : attr(defRPr, "b"),
+						i: !defRPr || styleUnknown || styled?.italic != null ? null : attr(defRPr, "i")
 					};
-					for (const run of runs) writeRunProps(run, props, pPr, cellLst, level);
+					for (const run of runs) {
+						writeRunProps(run, props, pPr, cellLst, level);
+						if (color && !slideDefinesColor(run, pPr, cellLst, level)) writeRunColor(run, color);
+					}
 				}
 			});
 		});
 	}
 }
+/**
+* The colour a cell's text paints on the source deck: what its table style's text style names,
+* else `tx1`, as `TableCell.textFrame` resolves it. The transforms are rebuilt as elements so
+* {@link writeRunColor} re-emits them.
+*/
+function tableCellTextColor(styled, doc, ctx) {
+	if (styled?.color) {
+		const transforms = styled.color.transforms.map(({ name, value }) => {
+			const t = createElement(doc, `a:${name}`);
+			if (value !== null) setAttr(t, "val", value);
+			return t;
+		});
+		return {
+			hex: styled.color.hex,
+			transforms
+		};
+	}
+	const tx1 = resolveSchemeToken("tx1", ctx);
+	return tx1 ? {
+		hex: tx1,
+		transforms: []
+	} : null;
+}
 /** The source deck's `a:tblStyle` with this id, or `null` when the source defines none. */
 function sourceTableStyle(ctx, styleId) {
 	if (!ctx.tableStyles) return null;
@@ -4448,15 +4605,14 @@ function neutralizeRef(ref) {
 	setAttr(ref, "idx", "0");
 	for (const child of childElements(ref)) ref.removeChild(child);
 }
-//#endregion
-//#region src/read/api/ops/carried-rels.ts
+// src/read/api/ops/carried-rels.ts
 /**
 * Relationship rewriting for content carried between packages.
 *
 * When a shape or decoration is deep-copied out of a source deck, every relationship
 * reference it carries (`r:embed` on a picture fill, `r:id` on a chart frame, `r:link` on a
 * linked image, …) still names an id in the *source* part's relationship list. Left alone
-* those ids either dangle or — worse — silently resolve to whatever the destination part
+* those ids either dangle or, worse, silently resolve to whatever the destination part
 * happens to have under the same id. This rewrites each one to a fresh relationship on the
 * destination part, copying the referenced part into the destination package on first sight.
 *
@@ -4516,8 +4672,7 @@ function carryRel(ctx, sourceRels, id, destPartName, destRels, relIdMap, owned)
 	if (!ownsIt) relIdMap.set(key, newId);
 	return newId;
 }
-//#endregion
-//#region src/read/api/ops/flatten-context.ts
+// src/read/api/ops/flatten-context.ts
 /**
 * The background the slide effectively inherits from its source subgraph: the layout's
 * `p:bg`, else the master's. Returns `null` when the slide carries its own `p:bg` (it stays
@@ -4550,8 +4705,7 @@ function sourceFlattenContext(sourceOpc, slidePartName) {
 		tableStyles: sourceOpc.partsByContentType("application/vnd.openxmlformats-officedocument.presentationml.tableStyles+xml")[0]?.dom.documentElement ?? null
 	};
 }
-//#endregion
-//#region src/read/api/ops/table-styles.ts
+// src/read/api/ops/table-styles.ts
 const TABLE_STYLES_DEFAULT_GUID = "{5C22544A-7EE6-4342-B048-85BDC9FD1C3A}";
 const textEncoder = new TextEncoder();
 /**
@@ -4560,7 +4714,7 @@ const textEncoder = new TextEncoder();
 * table resolves its `@tableStyleId` against the *destination* `tableStyles` and
 * silently falls back when that id is absent; the source `<a:tblStyle>` is itself
 * symbolic (scheme colours), so copying it verbatim under the **same** id lets the
-* table keep its structure while re-branding to this deck's theme — no table-XML
+* table keep its structure while re-branding to this deck's theme, with no table-XML
 * rewrite needed. Idempotent: an id this deck already defines is left as-is, and a
 * referenced id with no source definition is skipped. Creates and wires a
 * `tableStyles.xml` part if this deck has none.
@@ -4587,7 +4741,7 @@ function copySourceTableStyles(dest, sourceOpc, slideRoot) {
 }
 /**
 * Append each source `<a:tblStyle>` this deck does not already define, keyed by
-* `styleId` — the shared core of {@link copySourceTableStyles} (which offers only
+* `styleId`: the shared core of {@link copySourceTableStyles} (which offers only
 * the styles a restyled slide references) and {@link carryTableStyles} (which
 * offers the source's whole list). Destination-wins per id: a style this deck
 * already defines is left untouched, so a re-call adds nothing. Returns whether
@@ -4609,14 +4763,14 @@ function addTableStyles(destList, styles) {
 * Copy the source deck's **whole** `tableStyles.xml` into this deck: every
 * `<a:tblStyle>` it defines, plus its `a:tblStyleLst@def` (the default table
 * style). Presentation-level, so a separate traversal from the master/layout copy
-* chain — and whole-deck, since `tableStyles.xml` does not record which style
+* chain, and whole-deck, since `tableStyles.xml` does not record which style
 * belongs to which master. See `ImportSlideMastersOptions.tableStyles`.
 *
 * Styles union by `styleId` (destination-wins per id, via {@link addTableStyles}),
 * but `def` is **source-wins**: a caller asking for the source's table styling
 * wants its default too, and the destination's `def` is typically a generator stub
 * rather than a deliberate choice. Carrying the styles without the `def` would be
-* the worse half-measure — the standard default GUID is one most templates also
+* the worse half-measure: the standard default GUID is one most templates also
 * define, so a new table would silently resolve to the wrong style rather than
 * visibly to none. Idempotent: a re-call re-adds nothing and re-sets `def` to the
 * same value.
@@ -4653,26 +4807,25 @@ function ensureTableStylesPart(dest) {
 	dest.opc.relationshipsFor(dest.presentationPart.partName).add(TABLE_STYLES_REL, relativePartName(dest.presentationPart.partName, partName));
 	return part;
 }
-//#endregion
-//#region src/read/api/ops/import-slide.ts
+// src/read/api/ops/import-slide.ts
 /**
 * The three ways a slide can be brought across from another deck.
 *
 * {@link Presentation.importSlide} picks one by `options.theme`, and the difference between
 * them is entirely about what happens to the slide's *look*:
 *
-* - **`copy`** — not here. It copies the source theme subgraph wholesale via `copyPart`, so
+* - **`copy`**: not here. It copies the source theme subgraph wholesale via `copyPart`, so
 *   the slide keeps its look by keeping its own theme; nothing needs rewriting.
-* - **`preserve`** ({@link importSlidePreserve}) — rebind to this deck's master, then bake
+* - **`preserve`** ({@link importSlidePreserve}): rebind to this deck's master, then bake
 *   the source theme into the slide XML. The slide looks the same as it did; it just no
 *   longer depends on the theme it came from.
-* - **`restyle`** ({@link importSlideRestyle}) — rebind and bake *nothing*, so the slide's
+* - **`restyle`** ({@link importSlideRestyle}): rebind and bake *nothing*, so the slide's
 *   symbolic theme references re-resolve against the destination theme and it re-brands.
 *
 * The two share {@link importSlideRebind} and are exact inverses on top of it: preserve
 * resolves every reference it can, restyle resolves none. Any baking added to the shared
 * rebind would break restyle by pinning the slide to its source look, which is why the
-* rebind carries no theme handling of its own — not even the inherited background.
+* rebind carries no theme handling of its own, not even the inherited background.
 *
 * These were private methods on `Presentation`, where they made the read model's own surface
 * (open a deck, walk slides, read shapes) hard to see: the import machinery was roughly two
@@ -4703,17 +4856,17 @@ function importSlidePreserve(dest, ctx, source, sourceSlide, carryGraphics) {
 }
 /**
 * Import a slide in `restyle` mode: rebind it to this deck's master/layout and then
-* {@link restyleSlide} it — drop its colour map override but bake *nothing*, so its symbolic
+* {@link restyleSlide} it: drop its colour map override but bake *nothing*, so its symbolic
 * theme references re-resolve against the destination theme and the slide re-brands.
 *
 * The deliberate inverse of `preserve`: no flatten, no inherited-background bake, no
-* placeholder colour/size/geometry bake — every one of those would pin the slide to its
+* placeholder colour/size/geometry bake. Every one of those would pin the slide to its
 * source look, the opposite of re-branding. Carried decorations are left symbolic too, so
 * they re-brand along with the slide.
 *
 * With `remapLiterals` it additionally force-remaps the slide's source-theme literal colours
-* back to symbolic scheme colours and copies any referenced source table style into this deck
-* — the two things plain `restyle` cannot re-brand.
+* back to symbolic scheme colours and copies any referenced source table style into this deck:
+* the two things plain `restyle` cannot re-brand.
 * @param {Presentation} dest - the destination deck
 * @param {ImportContext} ctx - the open import out of the source package
 * @param {Presentation} source - the source deck
@@ -4739,11 +4892,11 @@ function importSlideRestyle(dest, ctx, source, sourceSlide, carryGraphics, remap
 /**
 * The rebind shared by `preserve` and `restyle`: copy the slide bytes into a fresh part,
 * rebuild its relationships (drop notes, repoint the `slideLayout` rel at this deck's
-* existing layout, copy every other internal target — media/charts — and pass externals
+* existing layout, copy every other internal target (media/charts), and pass externals
 * through), and optionally bake the source master/layout decorations onto the slide. Returns
 * the new part, its name, and its live root element for the caller's mode-specific pass.
 *
-* This carries *no* theme baking of its own — not even the inherited background.
+* This carries *no* theme baking of its own, not even the inherited background.
 * `preserve` adds that via {@link flattenSlide}'s context; `restyle` must not, so the
 * background stays symbolic and re-brands.
 * @param {Presentation} dest - the destination deck
@@ -4804,8 +4957,8 @@ function rebind(dest, ctx, source, sourceSlide, carryGraphics) {
 /**
 * Bake the source `slideLayout`/`slideMaster` shape-tree decorations onto the imported slide
 * (the `carryMasterGraphics` path). Every shape on those trees *except* placeholders is
-* deep-copied into the slide's `p:spTree` ahead of its own content — master decorations
-* first, then layout, then the slide's shapes — so document (z-)order keeps the master
+* deep-copied into the slide's `p:spTree` ahead of its own content (master decorations
+* first, then layout, then the slide's shapes), so document (z-)order keeps the master
 * furthest back. Each decoration's media and other relationship targets are copied into this
 * package and its `r:embed`/`r:id`/… references rewritten to fresh slide-local ids. The
 * injected shapes are left for the caller's {@link flattenSlide} pass to resolve any theme
@@ -4847,7 +5000,7 @@ function carryMasterGraphics(ctx, slideRoot, newPartName, slidePartName, owned)
 /**
 * The partname of the layout this deck's slides should attach to in `preserve` mode: the
 * first layout of the first slide master. Throws when the deck has no master/layout to attach
-* to (a deck ts-pptx always provides).
+* to (a deck pptx-ts always provides).
 * @param {Presentation} dest - the destination deck
 * @return {string} partname of the destination layout
 */
@@ -4858,8 +5011,7 @@ function destinationLayoutPartName(dest) {
 	if (!layoutPartName) throw new InvalidOptionError("import/destination-missing-layout", "importSlide with theme 'preserve' or 'restyle' requires a slide layout in the destination deck");
 	return layoutPartName;
 }
-//#endregion
-//#region src/read/api/ops/rescale.ts
+// src/read/api/ops/rescale.ts
 /** Build the EMU transform mapping source-canvas coordinates onto the target canvas (see {@link ImportSlideOptions.rescale}). */
 function computeRescale(source, target, mode) {
 	if (mode === "stretch") return {
@@ -4878,7 +5030,7 @@ function computeRescale(source, target, mode) {
 }
 /**
 * Rescale every top-level transform on a `p:spTree`: shapes/pictures/connectors
-* (`p:spPr/a:xfrm`), groups (`p:grpSpPr/a:xfrm` — only the group's own off/ext, so
+* (`p:spPr/a:xfrm`), groups (`p:grpSpPr/a:xfrm`, only the group's own off/ext, so
 * its children remap via the unchanged `chOff`/`chExt` rather than being recursed
 * into), and graphic frames (`p:graphicFrame/p:xfrm`, plus any inner table grid).
 * Placeholders that inherit geometry carry no `a:xfrm` and are left to inherit from
@@ -4927,8 +5079,8 @@ function rescaleXfrm(xfrm, t) {
 	}
 }
 /**
-* Scale a graphic-frame table's intrinsic dimensions — column widths by `sx`, row
-* heights by `sy` — so the table resizes with the slide (PowerPoint derives table
+* Scale a graphic-frame table's intrinsic dimensions (column widths by `sx`, row
+* heights by `sy`) so the table resizes with the slide (PowerPoint derives table
 * geometry from the grid, not the frame `a:ext`). No-op for non-table frames.
 */
 function rescaleTableGrid(graphicFrame, t) {
@@ -4946,8 +5098,7 @@ function rescaleTableGrid(graphicFrame, t) {
 		if (h !== null) setAttr(tr, "h", String(Math.max(0, Math.round(h * t.sy))));
 	}
 }
-//#endregion
-//#region src/read/api/ops/rescale-import.ts
+// src/read/api/ops/rescale-import.ts
 /**
 * Rescaling an imported slide's geometry onto the destination canvas.
 *
@@ -4964,8 +5115,8 @@ function rescaleTableGrid(graphicFrame, t) {
 * graphicFrame transform and table grid on the slide; in `copy` mode also
 * rescales the imported layout and master shape trees (resolved via the
 * slide → layout → master rel chain) so inherited placeholder/background geometry
-* stays aligned. `preserve`/`restyle` rebind to this deck's own master/layout —
-* already the destination size — so only the slide is touched. Geometry only:
+* stays aligned. `preserve`/`restyle` rebind to this deck's own master/layout,
+* already the destination size, so only the slide is touched. Geometry only:
 * font sizes and line widths are left as authored.
 */
 function rescaleImportedGeometry(dest, rescaledParts, slidePartName, theme, source, target, mode) {
@@ -5017,8 +5168,7 @@ function rescalePartGeometry(dest, rescaledParts, partName, transform, mode) {
 	rescaleSpTree(spTree, transform);
 	part.markDirty();
 }
-//#endregion
-//#region src/read/api/presentation-imports.ts
+// src/read/api/presentation-imports.ts
 /** Body of {@link Presentation.importSlide}, whose doc comment carries the contract. */
 function importSlide(deck, source, index, options = {}) {
 	const sourceSlide = source.slides[index];
@@ -5036,6 +5186,7 @@ function importSlide(deck, source, index, options = {}) {
 	const newPart = deck.opc.part(newPartName);
 	if (!newPart) throw new InternalError("import/part-went-missing", `Imported slide part went missing: ${newPartName}`);
 	if (sizesDiffer && options.rescale && target && incoming) rescaleImportedGeometry(deck, deck.rescaledParts, newPartName, options.theme, incoming, target, options.rescale === true ? "fit" : options.rescale);
+	carryCommentAuthors(deck, source, newPartName);
 	const slide = deck.insertSlidePart(newPart, options.at);
 	if (options.importNotes) carryNotes(deck, source, importCtx, sourceSlide.partName, newPartName);
 	if (options.embedFonts) carryEmbeddedFonts(deck, source, importCtx);
@@ -5090,6 +5241,7 @@ function importSlides(deck, requests) {
 			destPart
 		};
 	});
+	for (const request of planned) carryCommentAuthors(deck, request.source, request.destPartName);
 	for (const request of planned) {
 		const mode = rescaleBySource.get(request.source.opc);
 		if (!mode) continue;
@@ -5283,14 +5435,13 @@ function importShapes(deck, target, source, shapeIndices, options = {}) {
 	}
 	return result;
 }
-//#endregion
-//#region src/read/api/presentation.ts
+// src/read/api/presentation.ts
 /**
 * Read-model entry point: `Presentation` wraps an `OpcPackage` and exposes a
 * navigable, typed view of the deck (slides → shapes → text), backed by the
 * live DOM so the same nodes can later be mutated.
 *
-* The class holds the model and the deck-level plumbing every operation needs — the part
+* The class holds the model and the deck-level plumbing every operation needs: the part
 * index, the slide list, the copy registry that lets repeated imports from one source share
 * what they already copied, and the `p:sldIdLst` wiring. The operations themselves live in
 * `ops/`, one job per module, taking this deck as an argument. The slide-import machinery
@@ -5300,11 +5451,11 @@ function importShapes(deck, target, source, shapeIndices, options = {}) {
 * The four import entry points went the same way, and {@link appendSlides} after them: their
 * contracts stay here as the doc comments a caller reads, and their bodies live in
 * `presentation-imports.ts` and `ops/append-slides.ts`. Those bodies are the one thing in this
-* directory that is not independent of this class — they reach back for {@link importContext},
+* directory that is not independent of this class. They reach back for {@link importContext},
 * {@link insertSlidePart} and {@link rescaledParts}, which {@link cloneSlide} reads too and so
 * could not travel with them. Those three are `@internal` for that and nothing else.
 */
-/** Content type of the main part in a `.potx` template package — flipped to {@link PRESENTATION_MAIN_CONTENT_TYPE} by {@link Presentation.fromTemplate}. */
+/** Content type of the main part in a `.potx` template package, flipped to {@link PRESENTATION_MAIN_CONTENT_TYPE} by {@link Presentation.fromTemplate}. */
 const PRESENTATION_TEMPLATE_MAIN_CONTENT_TYPE = "application/vnd.openxmlformats-officedocument.presentationml.template.main+xml";
 var Presentation = class Presentation {
 	opc;
@@ -5358,6 +5509,7 @@ var Presentation = class Presentation {
 			if (mainPart.contentType === PRESENTATION_TEMPLATE_MAIN_CONTENT_TYPE) pres.opc.contentTypes.ensureRegistered(mainPart.partName, PRESENTATION_MAIN_CONTENT_TYPE);
 		}
 		while (pres.slides.length > 0) pres.removeSlide(0);
+		resetCommentAuthorIndexes(pres);
 		return pres;
 	}
 	/** The main presentation part (`/ppt/presentation.xml`), resolved via the package `officeDocument` relationship. */
@@ -5409,7 +5561,7 @@ var Presentation = class Presentation {
 	/**
 	* The deck's embedded font families (`p:embeddedFontLst` in `presentation.xml`),
 	* `[]` when it embeds none. Each entry names the typeface and resolves every
-	* embedded face's `r:id` to the absolute partname of its `.fntdata` binary — the
+	* embedded face's `r:id` to the absolute partname of its `.fntdata` binary: the
 	* read counterpart to the write-side `pptx.embedFont` / `importSlide({ embedFonts })`
 	* carry. An entry whose `p:font` has no `@typeface`, or a face whose `r:id` is
 	* missing or dangling, is skipped (faithful degradation, no throw). Read-only.
@@ -5439,7 +5591,7 @@ var Presentation = class Presentation {
 	* `ppt/authors.xml`), `[]` when the deck carries no modern comments. Unlike the
 	* legacy {@link commentAuthors}, each entry is keyed by a GUID `id` (plus
 	* `userId`/`providerId`); each slide's {@link Slide.modernComments} resolves its
-	* `@authorId` against this list. Read-only — the writer emits legacy comments.
+	* `@authorId` against this list. Read-only: the writer emits legacy comments.
 	*/
 	get modernCommentAuthors() {
 		return readModernCommentAuthors(this.opc, this.presentationPart.partName);
@@ -5469,7 +5621,7 @@ var Presentation = class Presentation {
 	* The deck's extended document properties (`docProps/app.xml`): the producing
 	* application and its version, the company, and the flat `TitlesOfParts` vector.
 	* `{}` when the deck carries no extended-properties part. The read counterpart of
-	* the write-side `pptx.company` setter — the only one of the four this library
+	* the write-side `pptx.company` setter, the only one of the four this library
 	* writes from a caller's value; the other three it states about itself.
 	*
 	* A deliberate subset of the part: see {@link ExtendedProperties} for why the
@@ -5491,7 +5643,7 @@ var Presentation = class Presentation {
 	* The deck-level programmatic tags (`p:custDataLst/p:tags` on `presentation.xml`,
 	* resolved to `ppt/tags/tagN.xml`) as `{ name, val }` string pairs, `[]` when the
 	* deck carries none. These are add-in/host metadata with no visible rendering and
-	* no writer — read-only, preserved byte-for-byte on round-trip. Per-slide tags are
+	* no writer. They are read-only, preserved byte-for-byte on round-trip. Per-slide tags are
 	* {@link Slide.tags}.
 	*/
 	get tags() {
@@ -5510,7 +5662,7 @@ var Presentation = class Presentation {
 	* charts, SmartArt diagrams and OLE embeddings, each with the subtree under it
 	* (a chart's embedded workbook comes along; the image inside its user-shapes
 	* drawing stays shared). PowerPoint refuses to open a deck where two slides
-	* resolve to one chart or diagram, so this is not tidiness — see
+	* resolve to one chart or diagram, so this is not tidiness; see
 	* `page-owned.ts` for the rule and the evidence behind it.
 	*/
 	cloneSlide(index, options = {}) {
@@ -5528,7 +5680,7 @@ var Presentation = class Presentation {
 	* `p:sldId` entry and the presentation→slide relationship are dropped, the slide
 	* part and its `.rels` are deleted, and any part the slide *privately* owned
 	* (its notes slide, slide-only media, charts/embeddings) that no remaining part
-	* references is pruned too — recursively. Shared deck chrome (layout, master,
+	* references is pruned too, recursively. Shared deck chrome (layout, master,
 	* theme, …) is never pruned, so the deck stays renderable; removing every slide
 	* leaves a valid master/layout-only package (a template shell). The slide also
 	* leaves every custom show and section it was in, which stay even when that
@@ -5568,9 +5720,9 @@ var Presentation = class Presentation {
 	* Append a copy of `source.slides[index]` to this presentation and return it.
 	*
 	* Unlike {@link cloneSlide} (same-deck duplicate), this copies a slide across
-	* a package boundary: it brings the connected sub-graph the slide depends on —
-	* its `slideLayout` → `slideMaster` → `theme`, plus any media, charts, and
-	* embeddings — into this package under fresh partnames, rewriting every
+	* a package boundary: it brings the connected sub-graph the slide depends on
+	* (its `slideLayout` → `slideMaster` → `theme`, plus any media, charts, and
+	* embeddings) into this package under fresh partnames, rewriting every
 	* partname, relationship id, and content-type registration so the result is a
 	* self-consistent OPC package. Parts of this (target) package that are not
 	* touched stay byte-identical, matching `cloneSlide`'s fidelity contract.
@@ -5578,7 +5730,7 @@ var Presentation = class Presentation {
 	* Only the layout(s) actually used by imported slides are copied; the imported
 	* master's `p:sldLayoutIdLst` is pruned to exactly those, mirroring how
 	* PowerPoint's "Reuse Slides" brings a slide across. Parts shared by repeated
-	* imports from the same source deck are copied once and reused — every part
+	* imports from the same source deck are copied once and reused: every part
 	* except the page itself. Importing the same source slide twice yields two
 	* independent copies over one shared layout/master/theme, which is what lets a
 	* deck show one page twice (verbatim, then edited).
@@ -5587,7 +5739,7 @@ var Presentation = class Presentation {
 	* into the slide XML and the slide is bound to this deck's existing
 	* master/layout; with `{ theme: 'restyle' }` the slide is bound to this deck's
 	* master/layout with its theme references left symbolic, so it re-brands to the
-	* destination palette — see {@link ImportSlideOptions}.
+	* destination palette. See {@link ImportSlideOptions}.
 	*
 	* By default the source slide size must equal this presentation's. Pass
 	* `{ rescale: 'fit' | 'stretch' }` to rescale the imported geometry onto this
@@ -5606,7 +5758,7 @@ var Presentation = class Presentation {
 	* Import selected pages from one or more loaded source presentations as one
 	* batch. Each imported page lands at its `outputIndex` in the complete
 	* destination slide list after the batch, and the returned array is parallel
-	* to `requests` — `result[i]` is the page `requests[i]` asked for, whatever
+	* to `requests`: `result[i]` is the page `requests[i]` asked for, whatever
 	* order the output positions were given in.
 	*
 	* Everything is checked before a single byte of this deck moves: every request
@@ -5619,13 +5771,13 @@ var Presentation = class Presentation {
 	* The batch also decides what a `slide → slide` link means: an internal link
 	* on a selected page must target another **selected** page (or one this deck
 	* already contains via an earlier import from that source), and is rewritten
-	* to the fresh partname — importing page 3 of 10 does not drag pages 1–2
+	* to the fresh partname. Importing page 3 of 10 does not drag pages 1–2
 	* across as dependencies, and never strands the link. This mirrors
 	* `appendSlides`' `import/unresolved-slide-link`, which the write side already
 	* enforces for generator decks.
 	*
 	* One request is one output page, so naming the same source page in several
-	* requests is how you ask for several independent copies of it — the page part
+	* requests is how you ask for several independent copies of it; the page part
 	* is the one thing an import never shares, exactly as in {@link importSlide}.
 	* Everything under it (layout, master, theme, media) is still copied once and
 	* shared. Where such a page is the target of a `slide → slide` link, the link
@@ -5636,7 +5788,7 @@ var Presentation = class Presentation {
 	* Speaker notes travel per request: `{ importNotes: true }` carries that page's
 	* `notesSlide` part across, wired to the new page and bound to a single
 	* `notesMaster` under the same 0..1 rule {@link importSlide} and
-	* {@link appendSlides} follow — the destination's own master wins when it has
+	* {@link appendSlides} follow: the destination's own master wins when it has
 	* one. Notes are part of the up-front dry run too, so a batch that would fail
 	* carrying them is refused with the deck still byte-identical. A page named in
 	* several requests gets its own copy of its notes each time, as of everything
@@ -5663,9 +5815,9 @@ var Presentation = class Presentation {
 	}
 	/**
 	* Graft slide master(s) from another open package into this one and return what
-	* was copied. Unlike {@link importSlide} — which brings a master across only as
+	* was copied. Unlike {@link importSlide}, which brings a master across only as
 	* the dependency of an imported *slide* and prunes it to the one layout that
-	* slide uses — this copies a master together with its **whole** layout family
+	* slide uses, this copies a master together with its **whole** layout family
 	* and attaches it to no slide: the master and its layouts land in this deck's
 	* layout gallery (PowerPoint's *Insert ▸ New Slide* / *Layout* picker) without
 	* changing any existing slide.
@@ -5697,7 +5849,7 @@ var Presentation = class Presentation {
 	}
 	/**
 	* The deck's slide masters, in `p:sldMasterIdLst` order, as modeled
-	* {@link SlideMaster}s — the typed read model over the shared chrome (each
+	* {@link SlideMaster}s: the typed read model over the shared chrome (each
 	* master's colour map, theme, placeholders, and the layouts built on it). Walk
 	* `pres.masters()[i].layouts` for the rich layout model, or use {@link layouts}
 	* for the flat {@link LayoutHandle} gallery {@link appendSlides} binds to.
@@ -5707,7 +5859,7 @@ var Presentation = class Presentation {
 		return slideMasterPartNames(this).map((partName) => this.opc.part(partName)).filter((part) => part !== void 0).map((part) => new SlideMaster(this.opc, part));
 	}
 	/**
-	* The deck's slide layouts, in master then layout order — the gallery a new
+	* The deck's slide layouts, in master then layout order: the gallery a new
 	* slide can bind to. Each {@link LayoutHandle} addresses one layout for
 	* {@link appendSlides}; the `name` is its `p:cSld@name`. Read-only enumeration:
 	* it copies nothing and leaves the package byte-identical.
@@ -5730,12 +5882,12 @@ var Presentation = class Presentation {
 	/**
 	* Append generator-produced slides onto this deck, binding each to an existing
 	* layout, and return the new {@link Slide}s. This is the hybrid
-	* "generate-onto-existing" path: the deck's masters, layouts, theme — and every
-	* other untouched part — stay **byte-identical** (only `presentation.xml`, its
+	* "generate-onto-existing" path: the deck's masters, layouts, theme, and every
+	* other untouched part stay **byte-identical** (only `presentation.xml`, its
 	* `.rels`, `[Content_Types].xml`, and the freshly-added slide/media parts
 	* change), because the existing chrome is never regenerated.
 	*
-	* `source` is any slide producer (a `TsPptx` instance); its authored slides
+	* `source` is any slide producer (a `PptxTs` instance); its authored slides
 	* are serialized via {@link SlideSource.extractSlides} and spliced in under
 	* fresh partnames, with each slide's `slideLayout` relationship pointed at the
 	* layout named by `options.layout` and its image/hyperlink relationships rebuilt
@@ -5745,13 +5897,13 @@ var Presentation = class Presentation {
 	* Charts and internal slide-to-slide hyperlinks are carried across: chart parts
 	* (chart XML + `.rels` + embedded workbook) are injected under fresh names, and a
 	* `slide:N` link is repointed at the Nth appended slide's new partname. A chartEx
-	* (Office 2016 — waterfall, funnel, treemap, ...) chart carries too, as its own
+	* (Office 2016: waterfall, funnel, treemap, ...) chart carries too, as its own
 	* `chartEx{N}.xml` part behind the MS chartEx rel, with the style and color-style
 	* sidecars PowerPoint requires alongside it.
 	*
 	* The generator's presentation-level embedded fonts (`pptx.embedFont`) are also
 	* carried into this deck and merged into its `p:embeddedFontLst`, de-duped by
-	* typeface + face slot — so author-side embedded fonts survive the append onto a
+	* typeface + face slot, so author-side embedded fonts survive the append onto a
 	* template that may itself already embed fonts.
 	*
 	* Embedded audio/video is carried too: the media part backs the ECMA audio/video rel
@@ -5762,24 +5914,28 @@ var Presentation = class Presentation {
 	* Speaker notes ride across too: a slide authored with `addNotes` gets a `notesSlide`
 	* part wired back to it, with its hyperlink rels preserved. A notes slide must bind to
 	* a notes master, and a template commonly has none, so the generator's is installed
-	* (with the theme its `.rels` needs) — but only when this deck has none of its own, so
+	* (with the theme its `.rels` needs), but only when this deck has none of its own, so
 	* an existing notes master and its styling always win.
 	*
+	* Review comments (`addComment`) ride across as a comments part per slide. Their
+	* authors merge into this deck's `commentAuthors.xml` by name and initials: an author
+	* the deck already has keeps its id and continues its numbering, and a new one is added.
+	*
 	* Limitations:
 	* - An internal link to a source slide outside the appended batch throws (its
 	*   target has no counterpart in the destination).
 	* - Appended slides are concrete absolute-positioned content with no placeholder
 	*   inheritance from the bound layout; the binding governs theme/`clrMap`
 	*   resolution and the "based on" link, not placeholder geometry. Author with
-	*   concrete colours — any `schemeClr` re-resolves against the destination theme.
+	*   concrete colours: any `schemeClr` re-resolves against the destination theme.
 	* - Source and destination slide sizes must match (no geometry rescale).
 	*/
 	async appendSlides(source, options) {
 		return appendSlides(this, source, options);
 	}
 	/**
-	* Copy one shape — an autoshape, picture, table/chart graphic frame, connector,
-	* or group — from `source.shapes[shapeIndex]` onto `target`, returning the new
+	* Copy one shape (an autoshape, picture, table/chart graphic frame, connector,
+	* or group) from `source.shapes[shapeIndex]` onto `target`, returning the new
 	* {@link Shape}. `target` must be a slide of *this* presentation; `source` may
 	* belong to any open presentation.
 	*
@@ -5790,7 +5946,7 @@ var Presentation = class Presentation {
 	* so they cannot collide with the host. With `theme: 'preserve'` (default) the
 	* shape's theme references are baked to literals against the *source* theme so it
 	* renders the same on a foreign host; `restyle` leaves them symbolic to re-brand;
-	* `copy` brings the XML across untouched — see {@link ImportShapeOptions}.
+	* `copy` brings the XML across untouched. See {@link ImportShapeOptions}.
 	*
 	* Differing slide sizes need `{ rescale }` (see {@link ImportShapeOptions.rescale});
 	* a lifted `preserve` placeholder is baked self-contained and demoted to a plain
@@ -5822,7 +5978,7 @@ var Presentation = class Presentation {
 	}
 	/**
 	* @internal The parts whose geometry a rescale has already rewritten, and in which mode.
-	* Exposed for `presentation-imports.ts` as a live map the callee adds to — the memo is what
+	* Exposed for `presentation-imports.ts` as a live map the callee adds to. The memo is what
 	* makes a layout or master shared across repeated imports from one source scale exactly
 	* once, so like the copy registry it has to outlive the call that fills it.
 	*/
@@ -5832,7 +5988,7 @@ var Presentation = class Presentation {
 	/**
 	* @internal Open an import out of `source`: this deck as the destination, paired with
 	* the copy registry for that package (created on first use). The registry is held on
-	* the class rather than in the context because it must outlive any one call — that is
+	* the class rather than in the context because it must outlive any one call: that is
 	* what makes a second import from the same source reuse the layout, master, theme, and
 	* media it already copied instead of duplicating them, and it is why the import bodies
 	* in `presentation-imports.ts` call back here rather than carrying it with them.
@@ -5879,7 +6035,7 @@ var Presentation = class Presentation {
 	* A slide id one past the highest existing, but at least ST_SlideId's minimum.
 	*
 	* The module named and enforced that minimum and not the maximum, so a deck near the ceiling
-	* got an out-of-range `p:sldId/@id` written with no diagnostic — the package then needs
+	* got an out-of-range `p:sldId/@id` written with no diagnostic. The package then needs
 	* repair, for a reason nothing reported. Past the ceiling there is no "next" id to hand out,
 	* so the allocation falls back to the lowest id in range that the deck is not already using;
 	* a deck holding every one of the two billion has no legal answer and says so.
@@ -5902,7 +6058,6 @@ var Presentation = class Presentation {
 		return this.opc.save();
 	}
 };
-//#endregion
 export { Slide as a, NotesSlide as c, SlideMaster as d, Theme as f, ContentTypes as g, Part as h, readExtendedProperties as i, Placeholder as l, Relationships as m, readCoreProperties as n, readTagsForPart as o, OpcPackage as p, readCustomProperties as r, NotesPlaceholder as s, Presentation as t, SlideLayout as u };
 
-//# sourceMappingURL=presentation-DGYlAWhl.js.map
+//# sourceMappingURL=presentation-4Ggq0_Xa.js.map