@shbernal/ts-pptx 4.1.0 → 4.2.0

package/dist/{notes-BcT571R4.js → comments-CKIZYRbZ.js}

@@ -1,25 +1,25 @@
-import { l as warn, n as InvalidOptionError, o as UnsupportedFeatureError, t as InternalError, u as warnOnce } from "./errors-Da2nXCJm.js";
-import { B as DEF_TEXT_SHADOW, G as MAX_FONT_SIZE_PT, I as DEF_SHAPE_LINE_COLOR, N as DEF_FONT_COLOR, S as transparencyToAlpha, U as LEVEL_MARGINS_EMU, W as LEVEL_PPR_TAIL, a as convertRotationDegrees, c as inch2Emu, f as opacityToAlpha, g as ptsToEmuLenient, h as positiveCoordinateEmu, i as convertArcAngle, l as lineWidthToEmu, n as clampRangedInput, p as percentToFixedPercent, q as SLDNUMFLDID, r as convertAngleUnits, s as getSmartParseNumber, u as mapStated, z as DEF_TEXT_GLOW } from "./units-internal-DFcUYjh3.js";
-import { F as EMU_PER_INCH, I as EMU_PER_POINT, P as ptToHundredths, b as FIXED_PCT_PER_PERCENT, c as SchemeColor, p as VALID_SHAPE_PRESETS, w as PERCENT_SCALE } from "./table-cell-margins-PDshga8T.js";
-import { Ct as FONT_REL, E as EMBEDDED_FONT_SLOTS, F as LINE_END_TYPES, G as TEXT_CAPS_TYPES, I as PATTERN_PRESETS, J as TEXT_STRIKE_TYPES, L as PRESET_LINE_DASHES, Mt as NOTES_MASTER_REL, W as TEXT_AUTONUM_SCHEMES, X as TEXT_UNDERLINE_TYPES, Xt as SLIDE_REL, Y as TEXT_TAB_ALIGN_TYPES, a as textAlignToken, an as OOXML_NS, c as checkEnumOrThrow, d as xsdBool, en as THEME_REL, f as xsdBoolIfTrue, h as stripHash, l as checkEnumOrWarn, m as splitRgbaHex, on as PML_ROOT_NS, p as isHexColor, s as lineCapToken, w as DEFAULT_COLOR_MAP, wt as HYPERLINK_REL, zt as PACKAGE_REL_NS } from "./image-size-DwDbakcl.js";
-import { n as setOrClear } from "./options-internal-WqgMf720.js";
-//#region src/xml-escape.ts
+import { d as warnOnce, n as InternalError, r as InvalidOptionError, s as UnsupportedFeatureError, u as warn } from "./entry-errors-JMthgJ2g.js";
+import { B as DEF_TEXT_SHADOW, G as MAX_FONT_SIZE_PT, I as DEF_SHAPE_LINE_COLOR, N as DEF_FONT_COLOR, S as transparencyToAlpha, U as LEVEL_MARGINS_EMU, W as LEVEL_PPR_TAIL, a as convertRotationDegrees, c as inch2Emu, f as opacityToAlpha, g as ptsToEmuLenient, h as positiveCoordinateEmu, i as convertArcAngle, l as lineWidthToEmu, n as clampRangedInput, p as percentToFixedPercent, q as SLDNUMFLDID, r as convertAngleUnits, s as getSmartParseNumber, u as mapStated, z as DEF_TEXT_GLOW } from "./units-internal-BR3a4cIi.js";
+import { F as EMU_PER_INCH, I as EMU_PER_POINT, P as ptToHundredths, b as FIXED_PCT_PER_PERCENT, f as VALID_SHAPE_PRESETS, s as SchemeColor, w as PERCENT_SCALE } from "./enums-BP9wsb9Q.js";
+import { A as FONT_REL, Ct as DEFAULT_COLOR_MAP, Ft as PATTERN_PRESETS, Gt as TEXT_CAPS_TYPES, Jt as TEXT_STRIKE_TYPES, K as PACKAGE_REL_NS, Lt as PRESET_LINE_DASHES, Pt as LINE_END_TYPES, Tt as EMBEDDED_FONT_SLOTS, Wt as TEXT_AUTONUM_SCHEMES, Xt as TEXT_UNDERLINE_TYPES, Yt as TEXT_TAB_ALIGN_TYPES, an as checkEnumOrWarn, c as SHOW_JUMPS, cn as xsdBoolIfTrue, dn as stripHash, ft as OOXML_NS, in as checkEnumOrThrow, j as HYPERLINK_REL, ln as isHexColor, pt as PML_ROOT_NS, rt as SLIDE_REL, s as textAlignToken, sn as xsdBool, st as THEME_REL, u as lineCapToken, un as splitRgbaHex, z as NOTES_MASTER_REL } from "./chart-metadata-CwiTTcJq.js";
+import { n as setOrClear } from "./options-internal-BWgmLPdx.js";
+// src/xml-escape.ts
 /**
 * pptx-ts: turning a caller's string into something that can sit inside XML.
 *
-* Three escapers with three notions of what is dangerous used to live here-and-there — one under
-* `gen/`, one under `read/oxml/`, and a fourth-of-a-job local one in `embedded-fonts.ts` — and the
+* Three escapers with three notions of what is dangerous used to live here-and-there (one under
+* `gen/`, one under `read/oxml/`, and a fourth-of-a-job local one in `embedded-fonts.ts`), and the
 * weakest was the one reached by the least-checked input. `serializeEmbeddedFontLst` puts
 * `font.typeface` straight from `pptx.embedFont({ typeface })` through an escaper that handled
 * `& < > "` and nothing else, so `embedFont({ typeface: 'Foo\v' })` wrote a vertical tab into
-* `<p:font typeface="…">` — a character XML 1.0 forbids outright, and exactly what the write side
-* strips at every other emission site — and `'Foo\nBar'` lost its newline to attribute-value
+* `<p:font typeface="…">`, a character XML 1.0 forbids outright and exactly what the write side
+* strips at every other emission site, and `'Foo\nBar'` lost its newline to attribute-value
 * normalisation.
 *
 * The copies had a real reason: `embedded-fonts.ts` is imported by `read/api/ops/`, so it must not
 * pull in `src/gen/`, and `read/oxml/dom.ts` is on the same side of that line. The answer to
 * "both halves need this" is a dependency-free root module, the same placement and rationale as
-* `hex-color.ts` — not a third, weaker copy.
+* `hex-color.ts`, not a third, weaker copy.
 *
 * This module imports nothing.
 */
@@ -51,13 +51,13 @@ function hasIllegalXmlChars(value) {
 *
 * ELEMENT TEXT ONLY. A literal tab, carriage return or line feed is left alone here because in
 * character data it *is* the content (`<a:t>` line breaks depend on it). Inside an attribute value
-* the same three characters are destroyed by the parser rather than preserved — see
+* the same three characters are destroyed by the parser rather than preserved; see
 * {@link encodeXmlAttrValue}, which every attribute-emitting path must use instead.
 * @param {string | number} xml - value to encode (numbers are stringified, as callers pass counts/sizes)
 * @returns {string} escaped XML
 */
 function encodeXmlEntities(xml) {
-	if (typeof xml === "undefined" || xml == null) return "";
+	if (xml == null) return "";
 	return xml.toString().replace(ILLEGAL_XML_CHARS_G, "").replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/"/g, "&quot;").replace(/'/g, "&apos;");
 }
 /**
@@ -78,8 +78,7 @@ function encodeXmlEntities(xml) {
 function encodeXmlAttrValue(xml) {
 	return encodeXmlEntities(xml).replace(/\t/g, "&#9;").replace(/\n/g, "&#10;").replace(/\r/g, "&#13;");
 }
-//#endregion
-//#region src/gen/utils.ts
+// src/gen/utils.ts
 /**
 * pptx-ts: shared generator helpers.
 *
@@ -176,11 +175,11 @@ function getDuplicateObjectNames(names) {
 *
 * Counting is the normal case: a slide's rels are minted in order, so one past the
 * total is free. It is not enough on its own, because a rel can arrive on a slide
-* carrying an id minted somewhere else — auto-paging re-registers a repeated header
+* carrying an id minted somewhere else: auto-paging re-registers a repeated header
 * row's hyperlink on each overflow slide under the id stamped on the shared
 * hyperlink object (`gen/define/hyperlinks.ts`). That id is then held but not
 * counted-to, so the next mint could hand out the same number and the part would
-* declare two `Relationship` elements sharing an `Id` — invalid OPC, and PowerPoint
+* declare two `Relationship` elements sharing an `Id`. That is invalid OPC, and PowerPoint
 * resolves such a reference to whichever comes first. Hence the scan: start where
 * the count says, then step over anything the slide already holds.
 * @param {PresSlideInternal} target - the slide to use
@@ -237,7 +236,7 @@ function mediaSlideKey(target) {
 * `embeddings`.
 *
 * The trailing number counts a target's media parts from one, and it is read *before* the
-* push that consumes the name — so this must be called in the object literal being pushed,
+* push that consumes the name, so this must be called in the object literal being pushed,
 * not hoisted above an earlier push, which would rename every part after it. Eight sites
 * spelled the rule out by hand with `_relsMedia.length + 1` inline; naming it is what lets
 * the off-by-one convention be stated once and read.
@@ -252,7 +251,7 @@ function nextMediaTarget(target, kind, extn, dir = "media") {
 }
 /**
 * The `path` a media rel carries when its bytes came inline as `data` and there is no file to
-* name — `preencoded.png`, `preencoded.mp4`. Nothing loads it: the media pass tells it apart
+* name: `preencoded.png`, `preencoded.mp4`. Nothing loads it: the media pass tells it apart
 * with {@link isPreencodedPath}.
 *
 * The spelling still reaches a deck. An image object with no alt text falls back to its path for
@@ -271,8 +270,7 @@ function preencodedPath(extn) {
 function isPreencodedPath(path) {
 	return path.startsWith("preencoded.");
 }
-//#endregion
-//#region src/ooxml/xml-decl.ts
+// src/ooxml/xml-decl.ts
 /**
 * The XML prolog every OOXML part this library writes begins with, including the parts the read
 * side regenerates when it serializes a package it has changed.
@@ -283,94 +281,12 @@ function isPreencodedPath(path) {
 * import: this module imports nothing, and no `ooxml/` module imports from `gen/` or `read/`.
 */
 const XML_DECL = "<?xml version=\"1.0\" encoding=\"UTF-8\" standalone=\"yes\"?>";
-//#endregion
-//#region src/embedded-fonts.ts
-/**
-* Shared model + serializer for PresentationML embedded fonts, used by both the
-* write side (author-side `pptx.embedFont`, `gen/pres/presentation.ts`) and the read side
-* (the `embedFonts` carry in `importSlide`, `importSlides`, `importSlideMasters` and
-* `appendSlides`, `src/read/api/ops/embedded-fonts.ts`).
-*
-* Embedded fonts are three coordinated pieces (ECMA-376 transitional):
-*  1. binary font parts `/ppt/fonts/fontN.fntdata` — the **raw** TTF/OTF bytes
-*     (PresentationML does not obfuscate embedded fonts), content type
-*     `application/x-fontdata` via a single `fntdata` Default extension;
-*  2. one `font` relationship per face from `presentation.xml`;
-*  3. a `p:embeddedFontLst` in `presentation.xml`, at index 7 of the
-*     `CT_Presentation` child sequence (after `smartTags`, before `custShowLst`
-*     … `defaultTextStyle`).
-*
-* This module owns only the OOXML-shape knowledge; rId allocation and part
-* placement stay with each caller (their packaging models differ).
-*
-* It must not import from `src/gen/`, because `read/api/ops/embedded-fonts.ts` imports it. What
-* it needs from elsewhere -- the attribute escaper, the `font` relationship type and the face slot
-* list -- comes from modules that import nothing outside `src/ooxml/` themselves.
-*/
-/** Extension for the binary font parts (`/ppt/fonts/fontN.fntdata`). */
-const FONT_DATA_EXTENSION = "fntdata";
-/** Content type for a `.fntdata` part — one `Default` covers every font part. */
-const FONT_DATA_CONTENT_TYPE = "application/x-fontdata";
-/** Relationship type from `presentation.xml` to a font part (one per face). */
-const FONT_REL_TYPE = FONT_REL;
-/**
-* Flatten `fonts` into one ordered face list, assigning sequential part indices
-* (1-based) and relationship ids (starting at `firstRId`). Faces are emitted in
-* `fonts` order, each font's faces in {@link EMBEDDED_FONT_SLOTS} order. Faces
-* without `bytes` are skipped (nothing to write).
-*/
-function flattenEmbeddedFaces(fonts, firstRId) {
-	const flat = [];
-	let partIndex = 0;
-	let rId = firstRId;
-	fonts.forEach((font, fontIndex) => {
-		for (const slot of EMBEDDED_FONT_SLOTS) {
-			const face = font.faces.find((f) => f.slot === slot);
-			if (!face || !face.bytes) continue;
-			flat.push({
-				fontIndex,
-				slot,
-				bytes: face.bytes,
-				partIndex: ++partIndex,
-				rId: rId++
-			});
-		}
-	});
-	return flat;
-}
-/**
-* Serialize a `<p:embeddedFontLst>` from `fonts` and a `(fontIndex, slot) → rId`
-* lookup (typically backed by {@link flattenEmbeddedFaces}). Returns `''` when no
-* font has any face with an allocated rId, so callers can emit nothing. Assumes
-* the enclosing document declares the `p:` and `r:` prefixes (presentation.xml
-* does), so no namespace declarations are emitted here.
-*/
-function serializeEmbeddedFontLst(fonts, rIdForFace) {
-	const entries = [];
-	fonts.forEach((font, fontIndex) => {
-		const faceXml = [];
-		for (const slot of EMBEDDED_FONT_SLOTS) {
-			const rId = rIdForFace(fontIndex, slot);
-			if (rId === void 0) continue;
-			faceXml.push(`<p:${slot} r:id="rId${rId}"/>`);
-		}
-		if (faceXml.length === 0) return;
-		let fontAttrs = `typeface="${encodeXmlAttrValue(font.typeface)}"`;
-		if (font.panose !== void 0) fontAttrs += ` panose="${encodeXmlAttrValue(font.panose)}"`;
-		if (font.pitchFamily !== void 0) fontAttrs += ` pitchFamily="${font.pitchFamily}"`;
-		if (font.charset !== void 0) fontAttrs += ` charset="${font.charset}"`;
-		entries.push(`<p:embeddedFont><p:font ${fontAttrs}/>${faceXml.join("")}</p:embeddedFont>`);
-	});
-	if (entries.length === 0) return "";
-	return `<p:embeddedFontLst>${entries.join("")}</p:embeddedFontLst>`;
-}
-//#endregion
-//#region src/gen/oxml/el.ts
+// src/gen/oxml/el.ts
 /**
 * pptx-ts: XML element builder (write-side substrate)
 *
 * The write path historically concatenated template strings and called
-* `encodeXmlEntities` by hand at every interpolation — the source of escaping,
+* `encodeXmlEntities` by hand at every interpolation, the source of escaping,
 * attribute-order and child-sequence bugs. This builder centralizes escaping
 * while reproducing today's exact byte layout, so a migration onto it can be
 * gated on strict byte-identity (`pnpm run byte-identity:check`) rather than on
@@ -383,7 +299,7 @@ function serializeEmbeddedFontLst(fonts, rIdForFace) {
 * 1. Self-closing is decided by ARITY, never by child value. `el()` always emits
 *    a paired tag; `voidEl()` always self-closes. This matters because
 *    `encodeXmlEntities(undefined)` returns `''`, so a value-based rule would
-*    turn today's `<dc:title></dc:title>` into `<dc:title/>` — a silent byte
+*    turn today's `<dc:title></dc:title>` into `<dc:title/>`, a silent byte
 *    regression on every optional-text element in the tree.
 *
 *    Escaping is split to match: attribute values go through `encodeXmlAttrValue`
@@ -413,7 +329,7 @@ function openTag(name, attrs, fmt) {
 	return out;
 }
 /**
-* Paired element: `<name …>children</name>` — emitted even when `children` is
+* Paired element: `<name …>children</name>`, emitted even when `children` is
 * empty or `undefined`. Text children are escaped; `raw()` children are not.
 */
 function el(name, attrs, children, fmt) {
@@ -426,7 +342,7 @@ function el(name, attrs, children, fmt) {
 	return out + (fmt?.closePrefix ?? "") + "</" + name + ">";
 }
 /**
-* Self-closing element: `<name …/>`. `fmt.closePrefix` goes before the `/>` — a
+* Self-closing element: `<name …/>`. `fmt.closePrefix` goes before the `/>`. A
 * handful of emitters write `<a:avLst />` with a space there, and that space is
 * byte-significant like any other.
 */
@@ -445,8 +361,7 @@ function voidEl(name, attrs, fmt) {
 function elOrVoid(name, attrs, childXml, fmt) {
 	return childXml ? el(name, attrs, raw(childXml), fmt) : voidEl(name, attrs, fmt);
 }
-//#endregion
-//#region src/gen/drawingml/color.ts
+// src/gen/drawingml/color.ts
 /**
 * pptx-ts: DrawingML color elements
 *
@@ -467,7 +382,7 @@ function elOrVoid(name, attrs, childXml, fmt) {
 */
 const ALPHA_TAG = "<a:alpha";
 /**
-* Emit a bare DrawingML color element — either `<a:schemeClr>` (scheme color) or `<a:srgbClr>`
+* Emit a bare DrawingML color element: either `<a:schemeClr>` (scheme color) or `<a:srgbClr>`
 * (hex RGB). This is a low-level primitive: it is the shared building block for every color
 * context (solid fills, gradient stops, `<a:alpha>` on shadows/glow, line fills, highlight,
 * underline, patterns, …), so it is intentionally NOT limited to solid fills and cannot be
@@ -475,7 +390,7 @@ const ALPHA_TAG = "<a:alpha";
 *
 * When you specifically want a *solid fill*, prefer `genXmlColorSelection` (`fill.ts`), which
 * wraps this in `<a:solidFill>…</a:solidFill>` and also handles `alpha`/`transparency` and the
-* gradient/pattern/image fill types — reach for `createColorElement` directly only when you
+* gradient/pattern/image fill types. Reach for `createColorElement` directly only when you
 * need the raw color element (e.g. inside `<a:ln>`, `<a:gs>`, an effect, or a highlight).
 * @param {string|SCHEME_COLORS} colorStr - hex RGB (e.g. "FFFF00") or a scheme color constant (e.g. SchemeColor.ACCENT1)
 * @param {string} [innerElements] - additional elements that adjust the color, nested inside the color element
@@ -490,7 +405,7 @@ function createColorElement(colorStr, innerElements) {
 		warnOnce("color/empty-string", `An empty string is not a color. "${DEF_FONT_COLOR}" used instead: this position requires one, so there is nothing to inherit. Omit the whole option if you meant to paint nothing.`);
 		colorStr = DEF_FONT_COLOR;
 	}
-	let colorVal = stripHash(colorStr || "");
+	let colorVal = stripHash(colorStr);
 	const rgba = splitRgbaHex(colorVal);
 	if (rgba.alpha !== void 0) {
 		if (!innerElements?.includes(ALPHA_TAG)) innerElements = alphaEl(Math.round(rgba.alpha * PERCENT_SCALE)) + (innerElements || "");
@@ -521,8 +436,8 @@ function isColorValue(value) {
 *
 * `''` is not a spelling of "no paint": omission already spells that, and `'inherit'` and
 * `'none'` spell the other two states a paint can be in. It reaches the library from the
-* caller's own missing value — an unset template field, `row.accent` on a row that has none
-* — so the useful answer is to say so and then behave exactly as omitting the option would
+* caller's own missing value (an unset template field, `row.accent` on a row that has none),
+* so the useful answer is to say so and then behave exactly as omitting the option would
 * have. What it must not do is coerce to {@link DEF_FONT_COLOR}, which paints visible black
 * on a shape the caller expected to keep the theme's paint.
 *
@@ -536,7 +451,7 @@ function isColorValue(value) {
 */
 function rejectEmptyColor(value, option) {
 	if (value !== "") return false;
-	warnOnce("color/empty-string", `\`${option}\` is an empty string, which is not a color — the option was ignored, exactly as omitting it would be. Pass 6-digit hex RGB (e.g. "FF3399") or a SchemeColor value.`);
+	warnOnce("color/empty-string", `\`${option}\` is an empty string, which is not a color, so the option was ignored, exactly as omitting it would be. Pass 6-digit hex RGB (e.g. "FF3399") or a SchemeColor value.`);
 	return true;
 }
 /**
@@ -544,7 +459,7 @@ function rejectEmptyColor(value, option) {
 *
 * The `x ?? DEFAULT` and `x || DEFAULT` this replaces disagreed about `''`: the first passed
 * it through to be painted black or dropped, the second quietly resolved it to the default.
-* Here it is one rule — an empty string is a missing value, reported and then resolved the
+* Here it is one rule: an empty string is a missing value, reported and then resolved the
 * way an absent option is.
 * @param value - the caller's colour option, possibly absent or empty
 * @param fallback - what an absent option resolves to at this site
@@ -565,126 +480,7 @@ function namedColorOr(value, fallback, option) {
 function alphaEl(fixedPct) {
 	return voidEl("a:alpha", { val: fixedPct });
 }
-//#endregion
-//#region src/gen/drawingml/effect.ts
-/**
-* pptx-ts: DrawingML effects (glow and shadow)
-*
-* Emit the `<a:glow>` / `<a:outerShdw>` / `<a:innerShdw>` children of an
-* `<a:effectLst>`, and normalize the user-facing `shadow` options into the
-* internal shape every emit site reads.
-* @see http://officeopenxml.com/drwSp-effects.php
-*/
-/**
-* Creates `a:glow` element
-* @param {TextGlowProps} options glow properties
-* @param {TextGlowProps} defaults defaults for unspecified properties in `opts`
-* @see http://officeopenxml.com/drwSp-effects.php
-* { size: 8, color: 'FFFFFF', opacity: 0.75 };
-*/
-function createGlowElement(options, defaults) {
-	const opts = {
-		...defaults,
-		...options
-	};
-	const size = positiveCoordinateEmu(opts.size, "glow/size-out-of-range", "glow.size");
-	const color = namedColorOr(opts.color, DEF_FONT_COLOR, "glow.color");
-	const opacity = opacityToAlpha(opts.opacity ?? 0);
-	return el("a:glow", { rad: size }, raw(createColorElement(color, alphaEl(opacity))));
-}
-/**
-* Creates an `a:outerShdw`/`a:innerShdw` element for a text run, shape, image, or chart.
-* Returns the shadow element only (no wrapping `a:effectLst`) so callers can either combine
-* it with other effects (e.g. glow) inside one `a:effectLst`, or wrap a lone shadow via
-* {@link createShadowEffectLst}.
-*
-* Colors go through {@link createColorElement}, so scheme colors (e.g. `accent1`) are honored
-* — earlier per-site copies hardcoded `a:srgbClr` and silently emitted invalid OOXML for them.
-* @param {ShadowPropsInternal} options shadow properties
-* @param {ShadowPropsInternal} defaults defaults for unspecified properties in `options`
-* @see http://officeopenxml.com/drwSp-effects.php
-* @returns {string} XML string, or '' when type is 'none'
-*/
-function createShadowElement(options, defaults) {
-	const opts = {
-		...defaults,
-		...options
-	};
-	if (opts.type === "none") return "";
-	const type = opts.type || "outer";
-	const blur = positiveCoordinateEmu(opts.blur ?? 0, "shadow/blur-out-of-range", "shadow.blur");
-	const offset = positiveCoordinateEmu(opts.offset ?? 0, "shadow/offset-out-of-range", "shadow.offset");
-	const angle = convertAngleUnits(opts.angle ?? 0, "shadow.angle");
-	const opacity = Math.round((opts._alpha ?? .75) * PERCENT_SCALE);
-	const color = namedColorOr(opts.color, DEF_FONT_COLOR, "shadow.color");
-	const attrs = {
-		...type === "outer" ? {
-			sx: 1e5,
-			sy: 1e5,
-			kx: 0,
-			ky: 0,
-			algn: "bl",
-			rotWithShape: xsdBool(opts.rotateWithShape)
-		} : {},
-		blurRad: blur,
-		dist: offset,
-		dir: angle
-	};
-	return el(`a:${type}Shdw`, attrs, raw(createColorElement(color, alphaEl(opacity))));
-}
-/**
-* Wraps a lone shadow in the `a:effectLst` that CT_ShapeProperties (shapes/images) and chart
-* marker/data-point properties require. Returns a self-closing `<a:effectLst/>` when there is
-* no shadow (missing/non-object options, or `type: 'none'`), matching the "no effects" element
-* PowerPoint emits. Use this instead of {@link createShadowElement} when the shadow is the only
-* effect; use `createShadowElement` directly when combining with other effects (e.g. glow).
-* @param {ShadowPropsInternal} options shadow properties
-* @param {ShadowPropsInternal} defaults defaults for unspecified properties in `options`
-* @returns {string} `<a:effectLst>…</a:effectLst>` or `<a:effectLst/>`
-*/
-function createShadowEffectLst(options, defaults) {
-	if (!options || typeof options !== "object") return voidEl("a:effectLst");
-	return elOrVoid("a:effectLst", null, createShadowElement(options, defaults));
-}
-/**
-* Normalize a caller's `shadow` into the internal shape every emit site reads: a legal `type`, a
-* whole-degree `angle` inside `ST_PositiveFixedAngle`, a `_alpha` derived from `transparency` or
-* from an RGBA colour's alpha byte, and a colour with no `#` and no alpha byte.
-*
-* **Pure.** It used to normalize the caller's object *in place* and return it, and its four call
-* sites split on which of the two contracts they used — two assigned the result, two discarded it
-* and relied on the mutation. That made the obvious cleanup a trap: making it pure without
-* converting every caller silently drops `transparency` and RGBA alpha on every text run, because
-* `_alpha` is what `genXmlTextRunProperties` reads and nothing else writes it. The value now
-* reaches the emitter as a value rather than as a side effect, and a shadow literal shared across
-* two shapes comes back untouched.
-* @param shadow - the caller's shadow options
-* @returns a fresh normalized bag, or `undefined` when the caller stated no shadow object
-*/
-function normalizeShadowOptions(shadow) {
-	if (!shadow || typeof shadow !== "object") return;
-	const corrected = { ...shadow };
-	if (corrected.type !== "outer" && corrected.type !== "inner" && corrected.type !== "none") {
-		warn("shadow/invalid-type", "shadow.type options are `outer`, `inner` or `none`.");
-		corrected.type = "outer";
-	}
-	if (corrected.angle !== void 0) corrected.angle = Math.round(clampRangedInput(corrected.angle, 0, 359, "shadow/angle-out-of-range", "shadow.angle", "shadow/angle-non-finite"));
-	if (corrected.transparency !== void 0) corrected._alpha = 1 - clampRangedInput(corrected.transparency, 0, 100, "shadow/transparency-out-of-range", "shadow.transparency") / 100;
-	if (corrected.color) {
-		if (corrected.color.startsWith("#")) {
-			warn("shadow/color-has-hash", "shadow.color should not include hash (#) character, , e.g. \"FF0000\"");
-			corrected.color = stripHash(corrected.color);
-		}
-		const rgba = splitRgbaHex(corrected.color);
-		if (rgba.alpha !== void 0) {
-			if (corrected._alpha === void 0) corrected._alpha = rgba.alpha;
-			corrected.color = rgba.rgb;
-		}
-	}
-	return corrected;
-}
-//#endregion
-//#region src/gen/drawingml/src-rect.ts
+// src/gen/drawingml/src-rect.ts
 /**
 * pptx-ts: the explicit `<a:srcRect>` source crop
 *
@@ -697,9 +493,10 @@ function normalizeShadowOptions(shadow) {
 * points, which never measure an image.
 */
 /**
-* The four insets, defaulted and checked. Rejects anything that is not a percentage and
-* any pair that would leave no source area at all — a degenerate `<a:srcRect>` renders as
-* an empty shape rather than as an error, so it has to fail here.
+* The four insets, defaulted and checked. Rejects a non-finite inset, one over 100, and any
+* pair that would leave no source area at all. A degenerate `<a:srcRect>` renders as an
+* empty shape rather than as an error, so it has to fail here. A negative inset is accepted:
+* DrawingML's `ST_Percentage` is signed, and it is how a source is padded inside its box.
 * @param crop - edge insets in percent; omitted edges default to 0
 * @param label - the option's owner, as an error message names it (`addImage`, `image fill`)
 * @param where - trailing context for an error message, or `''`
@@ -711,17 +508,18 @@ function normalizeImageCrop(crop, label, where) {
 		r: crop.r ?? 0,
 		b: crop.b ?? 0
 	};
-	for (const [name, val] of Object.entries(edges)) if (typeof val !== "number" || !Number.isFinite(val) || val < 0 || val > 100) throw new InvalidOptionError("image/crop-inset-out-of-range", `${label} crop.${name} must be a percentage between 0 and 100 (got ${String(val)})${where}.`);
+	for (const [name, val] of Object.entries(edges)) if (typeof val !== "number" || !Number.isFinite(val) || val > 100) throw new InvalidOptionError("image/crop-inset-out-of-range", `${label} crop.${name} must be a finite percentage no greater than 100 (got ${String(val)})${where}.`);
 	if (edges.l + edges.r >= 100) throw new InvalidOptionError("image/crop-insets-exceed-extent", `${label} crop: left+right insets (${edges.l}%+${edges.r}%) must be < 100%${where}.`);
 	if (edges.t + edges.b >= 100) throw new InvalidOptionError("image/crop-insets-exceed-extent", `${label} crop: top+bottom insets (${edges.t}%+${edges.b}%) must be < 100%${where}.`);
 	return edges;
 }
 /**
-* Build an explicit `<a:srcRect>` crop from percentage edge insets (0–100), emitted verbatim.
-* Each inset is the percent of the *source* image trimmed from that edge; values serialize in
-* 1000ths of a percent (DrawingML `ST_Percentage`, where 100% = 100000). Unlike `sizing: 'crop'`
-* — which derives the rect from displayed inches and the natural pixel size — this maps a
-* sub-region of the source directly, so it works for SVG and other unmeasurable formats.
+* Build an explicit `<a:srcRect>` crop from percentage edge insets, emitted verbatim.
+* Each inset is the percent of the *source* image trimmed from that edge, or padded onto it
+* when negative; values serialize in 1000ths of a percent (DrawingML `ST_Percentage`, where
+* 100% = 100000). Unlike `sizing: 'crop'`, which derives the rect from displayed inches and the
+* natural pixel size, this maps a sub-region of the source directly, so it works for SVG and
+* other unmeasurable formats.
 * @param crop - edge insets in percent; omitted edges default to 0
 * @param label - the option's owner, as an error message names it
 * @param where - trailing context for an error message, or `''`
@@ -738,7 +536,7 @@ function genXmlImageCropRect(crop, label, where) {
 	});
 }
 /**
-* `<a:stretch><a:fillRect/></a:stretch>` — the blipFill child that scales the source to the
+* `<a:stretch><a:fillRect/></a:stretch>`: the blipFill child that scales the source to the
 * shape's box, which every picture-bearing emitter writes and none parameterizes.
 *
 * After an `<a:srcRect/>`, it completes the canonical picture fill ECMA-376 gives
@@ -750,8 +548,7 @@ function genXmlImageCropRect(crop, label, where) {
 * it from every shape, table and chart emitter, and must not pull the image-format tables in.
 */
 const STRETCH_FILL_RECT = el("a:stretch", null, raw(voidEl("a:fillRect")));
-//#endregion
-//#region src/gen/drawingml/fill.ts
+// src/gen/drawingml/fill.ts
 /**
 * pptx-ts: DrawingML fills
 *
@@ -851,7 +648,7 @@ function genXmlImageFill(props) {
 * `type: 'none'` needs in order to mean anything (`{ type: 'none', color }` must stay
 * transparent), so it is the rule everywhere rather than one kind's exception.
 *
-* With no `type`, the first sub-object present selects the kind, in declaration order —
+* With no `type`, the first sub-object present selects the kind, in declaration order:
 * `gradient`, then `pattern`, then `image`. Setting two without a `type` is already a
 * contradiction; the order exists so the answer is stated rather than emergent.
 *
@@ -873,15 +670,15 @@ function resolveFillKind(props) {
 /**
 * {@link resolveFillKind} for a stroke, which has one kind fewer.
 *
-* `<a:ln>`'s paint child is `EG_LineFillProperties` — `a:noFill`, `a:solidFill`, `a:gradFill`,
-* `a:pattFill` — with no `a:blipFill` among them, so a picture stroke has no OOXML expression
+* `<a:ln>`'s paint child is `EG_LineFillProperties` (`a:noFill`, `a:solidFill`, `a:gradFill`,
+* `a:pattFill`), with no `a:blipFill` among them, so a picture stroke has no OOXML expression
 * and `ShapeLineProps` subtracts `image` from the fill props it inherits. This is where the
 * subtraction is enforced at run time, for the JS caller TypeScript cannot stop: the request
 * is refused rather than painted as nothing, because emitting a blipFill inside `a:ln` is a
 * package PowerPoint reports as needing repair, and emitting nothing is a stroke the caller
 * asked to be a picture and got in the theme colour without being told.
 *
-* Every stroke site resolves through here — the two `define/` rebuilds and the emitter — so
+* Every stroke site resolves through here (the two `define/` rebuilds and the emitter), so
 * the refusal names the call the caller made rather than surfacing at serialization time.
 */
 function resolveLineKind(props) {
@@ -897,7 +694,7 @@ function resolveLineKind(props) {
 * a fill child for an empty props object would paint them default black.
 *
 * "Said nothing" is narrower than "is empty". `'inherit'` is the explicit spelling of the
-* same silence and is false here. `'none'` is a statement — `<a:noFill/>` — and is true.
+* same silence and is false here. `'none'` is a statement (`<a:noFill/>`) and is true.
 * A `'solid'` kind needs a `color` to be saying anything; every other kind carries its
 * payload in its own sub-object, so its presence is the statement.
 *
@@ -916,7 +713,7 @@ function fillNamesPaint(props) {
 /**
 * Reject the empty-colour spelling of a missing value, in either shape a fill can wear.
 *
-* `''` and `{ color: '' }` say the same thing — the caller had no colour — and only the
+* `''` and `{ color: '' }` say the same thing (the caller had no colour), and only the
 * second used to reach {@link createColorElement} and paint black. Both fold back onto
 * *absence* here, which is not the same as "emit nothing": what an absent fill means is the
 * surface's own rule, and a text box's absent fill is `<a:noFill/>`. So a site whose absence
@@ -933,8 +730,8 @@ function rejectEmptyFill(props, option) {
 /**
 * A solid paint for {@link genXmlColorSelection}, carrying `transparency` only when one was set.
 *
-* Every caller assembles this out of an optional source — `border.transparency`,
-* `opts.transparency` — and writing the key with an `undefined` in it would be a second spelling
+* Every caller assembles this out of an optional source (`border.transparency`,
+* `opts.transparency`), and writing the key with an `undefined` in it would be a second spelling
 * of "no transparency": one that reads the same here, but not in a bag that is spread over
 * another. So the key is simply absent, which is this codebase's one spelling of unset.
 * @param color - the colour to paint, already resolved against whatever default applies
@@ -949,7 +746,7 @@ function solidPaint(color, transparency) {
 /**
 * Emit the fill group a set of fill/line props asks for, dispatching on
 * {@link resolveFillKind}. Returns '' for props that name nothing to paint, so a caller
-* with an optional paint can hand this whatever it has — including `undefined`.
+* with an optional paint can hand this whatever it has, including `undefined`.
 * @param {Color | ShapeFillProps | ShapeLineProps | undefined} props fill props
 * @returns XML string
 */
@@ -989,8 +786,204 @@ function genXmlColorSelection(props) {
 	}
 	return outText;
 }
-//#endregion
-//#region src/gen/drawingml/line.ts
+// src/embedded-fonts.ts
+/**
+* Shared model + serializer for PresentationML embedded fonts, used by both the
+* write side (author-side `pptx.embedFont`, `gen/pres/presentation.ts`) and the read side
+* (the `embedFonts` carry in `importSlide`, `importSlides`, `importSlideMasters` and
+* `appendSlides`, `src/read/api/ops/embedded-fonts.ts`).
+*
+* Embedded fonts are three coordinated pieces (ECMA-376 transitional):
+*  1. binary font parts `/ppt/fonts/fontN.fntdata`, holding the **raw** TTF/OTF bytes
+*     (PresentationML does not obfuscate embedded fonts), content type
+*     `application/x-fontdata` via a single `fntdata` Default extension;
+*  2. one `font` relationship per face from `presentation.xml`;
+*  3. a `p:embeddedFontLst` in `presentation.xml`, at index 7 of the
+*     `CT_Presentation` child sequence (after `smartTags`, before `custShowLst`
+*     … `defaultTextStyle`).
+*
+* This module owns only the OOXML-shape knowledge; rId allocation and part
+* placement stay with each caller (their packaging models differ).
+*
+* It must not import from `src/gen/`, because `read/api/ops/embedded-fonts.ts` imports it. What
+* it needs from elsewhere -- the attribute escaper, the `font` relationship type and the face slot
+* list -- comes from modules that import nothing outside `src/ooxml/` themselves.
+*/
+/** Extension for the binary font parts (`/ppt/fonts/fontN.fntdata`). */
+const FONT_DATA_EXTENSION = "fntdata";
+/** Content type for a `.fntdata` part. One `Default` covers every font part. */
+const FONT_DATA_CONTENT_TYPE = "application/x-fontdata";
+/** Relationship type from `presentation.xml` to a font part (one per face). */
+const FONT_REL_TYPE = FONT_REL;
+/**
+* Flatten `fonts` into one ordered face list, assigning sequential part indices
+* (1-based) and relationship ids (starting at `firstRId`). Faces are emitted in
+* `fonts` order, each font's faces in {@link EMBEDDED_FONT_SLOTS} order. Faces
+* without `bytes` are skipped (nothing to write).
+*/
+function flattenEmbeddedFaces(fonts, firstRId) {
+	const flat = [];
+	let partIndex = 0;
+	let rId = firstRId;
+	fonts.forEach((font, fontIndex) => {
+		for (const slot of EMBEDDED_FONT_SLOTS) {
+			const face = font.faces.find((f) => f.slot === slot);
+			if (!face || !face.bytes) continue;
+			flat.push({
+				fontIndex,
+				slot,
+				bytes: face.bytes,
+				partIndex: ++partIndex,
+				rId: rId++
+			});
+		}
+	});
+	return flat;
+}
+/**
+* Serialize a `<p:embeddedFontLst>` from `fonts` and a `(fontIndex, slot) → rId`
+* lookup (typically backed by {@link flattenEmbeddedFaces}). Returns `''` when no
+* font has any face with an allocated rId, so callers can emit nothing. Assumes
+* the enclosing document declares the `p:` and `r:` prefixes (presentation.xml
+* does), so no namespace declarations are emitted here.
+*/
+function serializeEmbeddedFontLst(fonts, rIdForFace) {
+	const entries = [];
+	fonts.forEach((font, fontIndex) => {
+		const faceXml = [];
+		for (const slot of EMBEDDED_FONT_SLOTS) {
+			const rId = rIdForFace(fontIndex, slot);
+			if (rId === void 0) continue;
+			faceXml.push(`<p:${slot} r:id="rId${rId}"/>`);
+		}
+		if (faceXml.length === 0) return;
+		let fontAttrs = `typeface="${encodeXmlAttrValue(font.typeface)}"`;
+		if (font.panose !== void 0) fontAttrs += ` panose="${encodeXmlAttrValue(font.panose)}"`;
+		if (font.pitchFamily !== void 0) fontAttrs += ` pitchFamily="${font.pitchFamily}"`;
+		if (font.charset !== void 0) fontAttrs += ` charset="${font.charset}"`;
+		entries.push(`<p:embeddedFont><p:font ${fontAttrs}/>${faceXml.join("")}</p:embeddedFont>`);
+	});
+	if (entries.length === 0) return "";
+	return `<p:embeddedFontLst>${entries.join("")}</p:embeddedFontLst>`;
+}
+// src/gen/drawingml/effect.ts
+/**
+* pptx-ts: DrawingML effects (glow and shadow)
+*
+* Emit the `<a:glow>` / `<a:outerShdw>` / `<a:innerShdw>` children of an
+* `<a:effectLst>`, and normalize the user-facing `shadow` options into the
+* internal shape every emit site reads.
+* @see http://officeopenxml.com/drwSp-effects.php
+*/
+/**
+* Creates `a:glow` element
+* @param {TextGlowProps} options glow properties
+* @param {TextGlowProps} defaults defaults for unspecified properties in `opts`
+* @see http://officeopenxml.com/drwSp-effects.php
+* { size: 8, color: 'FFFFFF', opacity: 0.75 };
+*/
+function createGlowElement(options, defaults) {
+	const opts = {
+		...defaults,
+		...options
+	};
+	const size = positiveCoordinateEmu(opts.size, "glow/size-out-of-range", "glow.size");
+	const color = namedColorOr(opts.color, DEF_FONT_COLOR, "glow.color");
+	const opacity = opacityToAlpha(opts.opacity ?? 0);
+	return el("a:glow", { rad: size }, raw(createColorElement(color, alphaEl(opacity))));
+}
+/**
+* Creates an `a:outerShdw`/`a:innerShdw` element for a text run, shape, image, or chart.
+* Returns the shadow element only (no wrapping `a:effectLst`) so callers can either combine
+* it with other effects (e.g. glow) inside one `a:effectLst`, or wrap a lone shadow via
+* {@link createShadowEffectLst}.
+*
+* Colors go through {@link createColorElement}, so scheme colors (e.g. `accent1`) are honored;
+* earlier per-site copies hardcoded `a:srgbClr` and silently emitted invalid OOXML for them.
+* @param {ShadowPropsInternal} options shadow properties
+* @param {ShadowPropsInternal} defaults defaults for unspecified properties in `options`
+* @see http://officeopenxml.com/drwSp-effects.php
+* @returns {string} XML string, or '' when type is 'none'
+*/
+function createShadowElement(options, defaults) {
+	const opts = {
+		...defaults,
+		...options
+	};
+	if (opts.type === "none") return "";
+	const type = opts.type || "outer";
+	const blur = positiveCoordinateEmu(opts.blur ?? 0, "shadow/blur-out-of-range", "shadow.blur");
+	const offset = positiveCoordinateEmu(opts.offset ?? 0, "shadow/offset-out-of-range", "shadow.offset");
+	const angle = convertAngleUnits(opts.angle ?? 0, "shadow.angle");
+	const opacity = Math.round((opts._alpha ?? .75) * PERCENT_SCALE);
+	const color = namedColorOr(opts.color, DEF_FONT_COLOR, "shadow.color");
+	const attrs = {
+		...type === "outer" ? {
+			sx: 1e5,
+			sy: 1e5,
+			kx: 0,
+			ky: 0,
+			algn: "bl",
+			rotWithShape: xsdBool(opts.rotateWithShape)
+		} : {},
+		blurRad: blur,
+		dist: offset,
+		dir: angle
+	};
+	return el(`a:${type}Shdw`, attrs, raw(createColorElement(color, alphaEl(opacity))));
+}
+/**
+* Wraps a lone shadow in the `a:effectLst` that CT_ShapeProperties (shapes/images) and chart
+* marker/data-point properties require. Returns a self-closing `<a:effectLst/>` when there is
+* no shadow (missing/non-object options, or `type: 'none'`), matching the "no effects" element
+* PowerPoint emits. Use this instead of {@link createShadowElement} when the shadow is the only
+* effect; use `createShadowElement` directly when combining with other effects (e.g. glow).
+* @param {ShadowPropsInternal} options shadow properties
+* @param {ShadowPropsInternal} defaults defaults for unspecified properties in `options`
+* @returns {string} `<a:effectLst>…</a:effectLst>` or `<a:effectLst/>`
+*/
+function createShadowEffectLst(options, defaults) {
+	if (!options || typeof options !== "object") return voidEl("a:effectLst");
+	return elOrVoid("a:effectLst", null, createShadowElement(options, defaults));
+}
+/**
+* Normalize a caller's `shadow` into the internal shape every emit site reads: a legal `type`, a
+* whole-degree `angle` inside `ST_PositiveFixedAngle`, a `_alpha` derived from `transparency` or
+* from an RGBA colour's alpha byte, and a colour with no `#` and no alpha byte.
+*
+* **Pure.** It used to normalize the caller's object *in place* and return it, and its four call
+* sites split on which of the two contracts they used: two assigned the result, two discarded it
+* and relied on the mutation. That made the obvious cleanup a trap: making it pure without
+* converting every caller silently drops `transparency` and RGBA alpha on every text run, because
+* `_alpha` is what `genXmlTextRunProperties` reads and nothing else writes it. The value now
+* reaches the emitter as a value rather than as a side effect, and a shadow literal shared across
+* two shapes comes back untouched.
+* @param shadow - the caller's shadow options
+* @returns a fresh normalized bag, or `undefined` when the caller stated no shadow object
+*/
+function normalizeShadowOptions(shadow) {
+	if (!shadow || typeof shadow !== "object") return void 0;
+	const corrected = { ...shadow };
+	if (corrected.type !== "outer" && corrected.type !== "inner" && corrected.type !== "none") {
+		warn("shadow/invalid-type", "shadow.type options are `outer`, `inner` or `none`.");
+		corrected.type = "outer";
+	}
+	if (corrected.angle !== void 0) corrected.angle = Math.round(clampRangedInput(corrected.angle, 0, 359, "shadow/angle-out-of-range", "shadow.angle", "shadow/angle-non-finite"));
+	if (corrected.transparency !== void 0) corrected._alpha = 1 - clampRangedInput(corrected.transparency, 0, 100, "shadow/transparency-out-of-range", "shadow.transparency") / 100;
+	if (corrected.color) {
+		if (corrected.color.startsWith("#")) {
+			warn("shadow/color-has-hash", "shadow.color should not include hash (#) character, , e.g. \"FF0000\"");
+			corrected.color = stripHash(corrected.color);
+		}
+		const rgba = splitRgbaHex(corrected.color);
+		if (rgba.alpha !== void 0) {
+			if (corrected._alpha === void 0) corrected._alpha = rgba.alpha;
+			corrected.color = rgba.rgb;
+		}
+	}
+	return corrected;
+}
+// src/gen/drawingml/line.ts
 /**
 * Every key `BorderProps` defines.
 *
@@ -1090,7 +1083,7 @@ function lineEndEl(tag, value, label) {
 *
 * TypeScript's excess-property check already rejects a stray key on a border written
 * *inline* at the call site. It deliberately does not fire when the border is built as
-* a variable first (`const b = {...}; addTable(rows, { border: b })`) — a variable may
+* a variable first (`const b = {...}; addTable(rows, { border: b })`), since a variable may
 * legitimately be a supertype. That exemption covers precisely the reuse pattern a
 * shared grid style encourages, so the mistake reaches here typed as valid and the
 * wrong-named value is discarded during generation. A `.pptx` gives no other signal:
@@ -1100,7 +1093,7 @@ function lineEndEl(tag, value, label) {
 function warnUnknownBorderKeys(border) {
 	for (const key in border) {
 		if (!Object.hasOwn(border, key) || BORDER_KEYS.includes(key)) continue;
-		warnOnce("border/unknown-key", `border: unknown option \`${key}\` is ignored — valid keys are ${BORDER_KEYS.join(", ")}. Line thickness is \`width\`, in points.`, {
+		warnOnce("border/unknown-key", `border: unknown option \`${key}\` is ignored. Valid keys are ${BORDER_KEYS.join(", ")}. Line thickness is \`width\`, in points.`, {
 			received: key,
 			valid: BORDER_KEYS
 		});
@@ -1114,8 +1107,8 @@ function warnUnknownBorderKeys(border) {
 * than taking the default: `lineWidthToEmu` refuses `NaN` and clamps `Infinity` to the top of
 * `ST_LineWidth`, as it does for every other line width.
 *
-* Every `BorderProps` the library reads — table cell borders, table-style regions, and
-* every chart border — resolves its width through here, so this doubles as the one
+* Every `BorderProps` the library reads (table cell borders, table-style regions, and
+* every chart border) resolves its width through here, so this doubles as the one
 * place that can vet a border's shape for all of them ({@link warnUnknownBorderKeys}).
 * @param {BorderProps} border - border properties (may carry `width`)
 * @param {number} defaultPt - width to use when `width` is not a number
@@ -1128,7 +1121,7 @@ function resolveBorderWidth(border, defaultPt) {
 }
 /**
 * A shape outline with its defaults applied: the kind `resolveLineKind` answers, no transparency, a
-* 1pt width, a solid dash, and on a solid stroke the default colour.
+* 1pt width, a solid dash, and on a solid stroke the default colour. An inherited line takes none of them.
 *
 * Shapes, line-shaped text boxes and connectors each merged these themselves, and the connector
 * disagreed about zero: `line.width: 0` took the 1pt default on a shape and a text box, and wrote no
@@ -1146,6 +1139,10 @@ function resolveBorderWidth(border, defaultPt) {
 */
 function withLineDefaults(line, colorLabel) {
 	const kind = resolveLineKind(line);
+	if (kind === "inherit") return {
+		...line,
+		type: kind
+	};
 	const resolved = {
 		...line,
 		type: kind,
@@ -1172,7 +1169,7 @@ function createLineCap(lineCap) {
 *
 * `<a:ln>` takes almost the same fill group as a shape interior, so a stroke can be a gradient
 * or a pattern as well as a solid color. Which one is asked for is {@link resolveLineKind}'s
-* answer, shared with the shape interior — the stroke side used to infer `gradient` from its
+* answer, shared with the shape interior. The stroke side used to infer `gradient` from its
 * sub-object while the interior inferred nothing, so the same `{ gradient }` spelling painted a
 * gradient outline and a black fill. "Almost" is the picture fill: `EG_LineFillProperties` has
 * no `a:blipFill`, and `resolveLineKind` is where that is refused.
@@ -1196,20 +1193,20 @@ function genXmlLineFill(line) {
 * Four builders used to turn a `BorderProps` into a line element, each with its own
 * defaults, and the differences between them were only visible by reading all four: a
 * chart data point's outline, a chart-area or plot-area border, the `<c:spPr>` line the
-* plot and chart space share, and a table cell's six edges. They agree on the shape —
-* `w` and `cap` on the element, a solid paint from `color`/`transparency`, then an
-* optional dash and whatever the schema allows after it — and disagree only on what fills
+* plot and chart space share, and a table cell's six edges. They agree on the shape
+* (`w` and `cap` on the element, a solid paint from `color`/`transparency`, then an
+* optional dash and whatever the schema allows after it) and disagree only on what fills
 * those slots. Those are this function's parameters, so each caller is now a named
 * argument set rather than a re-derivation.
 *
 * Two things are deliberately *not* parameters. A stroke that paints nothing is spelled
 * differently by each caller (a bare `<a:ln><a:noFill/></a:ln>` for a chart, an
 * attribute-carrying `w="0"` rule for a table cell, and nothing at all for a data point,
-* whose caller decides whether to emit it) — see {@link noStrokeLine} for the shared
+* whose caller decides whether to emit it); see {@link noStrokeLine} for the shared
 * spelling of the first. And the colour default is taken on `||`, not `??`, so an empty
 * `color` reads as *unstated* rather than reaching the colour validator as a bad value:
 * three of the four callers used to hand `''` straight through, which warns
-* `color/invalid-value` and paints `DEF_FONT_COLOR` — black — instead of the border
+* `color/invalid-value` and paints `DEF_FONT_COLOR` (black) instead of the border
 * default they had named a line earlier. The chart-border builder already read it this way,
 * and `dataBorder` is the only one of the other three a caller can actually reach with an
 * empty string; the rest resolve their colour in the definition step first.
@@ -1238,7 +1235,7 @@ function borderLine(name, border, spec) {
 * The `<a:ln>` that paints no stroke at all: an explicit `<a:noFill/>` on a bare element.
 *
 * This is the chart side's spelling of "no line" and is a different statement from a
-* {@link borderLine} whose paint is absent — an omitted `a:ln` inherits the theme's
+* {@link borderLine} whose paint is absent: an omitted `a:ln` inherits the theme's
 * `a:lnRef`, where this overrides it. A table cell spells the same intent with the
 * attributes still on the element and `w="0"`, which is why that one is not this.
 * @returns the `<a:ln><a:noFill/></a:ln>` element XML
@@ -1246,13 +1243,12 @@ function borderLine(name, border, spec) {
 function noStrokeLine() {
 	return el("a:ln", null, raw(voidEl("a:noFill")));
 }
-//#endregion
-//#region src/gen/define/hyperlinks.ts
+// src/gen/define/hyperlinks.ts
 /**
 * The relationship record one hyperlink serializes to.
 *
-* `data` distinguishes the two targets a hyperlink rel can have — `'slide'` for an internal
-* slide-to-slide link, `'dummy'` for an external URL — and `Target` is stored RAW, because
+* `data` distinguishes the two targets a hyperlink rel can have (`'slide'` for an internal
+* slide-to-slide link, `'dummy'` for an external URL), and `Target` is stored RAW, because
 * every emitter escapes it on the way out (see the note on `SlideRel.Target`).
 *
 * Four sites built this record by hand: the two below, `addImage`'s own hyperlink, and the
@@ -1276,7 +1272,8 @@ function hyperlinkRel(rId, hyperlink) {
 * definer and the run emitter each checked their own subset, and the shape path checked nothing. An
 * image refused an action-only link that a shape accepted, and nothing refused `url` together with
 * `slide`: that wrote two `<a:hlinkClick>` into one `<p:cNvPr>`, which allows one, over a slide-typed
-* relationship whose target was built from the URL.
+* relationship whose target was built from the URL. An `action` beside a `url` or a `slide` was
+* dropped without a word, so it is refused too.
 *
 * A `slide` has to be a 1-based slide number. `-1` and `1.5` were written as the targets
 * `slide-1.xml` and `slide1.5.xml`, parts no deck has. Whether the numbered slide exists is only
@@ -1288,17 +1285,23 @@ function hyperlinkRel(rId, hyperlink) {
 function validateHyperlink(hyperlink, call) {
 	if (typeof hyperlink !== "object" || hyperlink === null) throw new InvalidOptionError("hyperlink/not-an-object", `${call}: \`hyperlink\` option should be an object. Ex: \`hyperlink:{url:'https://github.com'}\``);
 	const { url, slide, action } = hyperlink;
-	if (url && slide) throw new InvalidOptionError("hyperlink/conflicting-targets", `${call}: \`hyperlink\` takes one destination but states both \`url\` and \`slide\`.`);
+	const stated = [
+		"url",
+		"slide",
+		"action"
+	].filter((key) => hyperlink[key]);
+	if (stated.length > 1) throw new InvalidOptionError("hyperlink/conflicting-targets", `${call}: \`hyperlink\` takes one destination but states ${stated.map((key) => `\`${key}\``).join(" and ")}.`);
 	if (slide != null && !(Number.isInteger(slide) && slide >= 1)) throw new InvalidOptionError("hyperlink/invalid-slide", `${call}: \`hyperlink.slide\` is a 1-based slide number, got ${typeof slide === "string" ? JSON.stringify(slide) : String(slide)}.`);
 	if (!url && !slide && !action) throw new InvalidOptionError("hyperlink/missing-target", `${call}: \`hyperlink\` requires either \`url\`, \`slide\`, or \`action\``);
+	if (action && !SHOW_JUMPS.includes(action)) throw new InvalidOptionError("hyperlink/invalid-action", `${call}: \`hyperlink.action\` is one of ${SHOW_JUMPS.join(", ")}, got ${JSON.stringify(action)}.`);
 }
 /**
 * Mint a fresh rel id for `hyperlink`, register it on `target`, and stamp the id back onto
 * the hyperlink so the emitter can write `r:id`.
 *
 * The id comes from {@link getNewRelId}, which skips every id already held on the slide.
-* `addImage` used to increment the image's own id instead, which is how an SVG picture — a
-* pair that already consumes two ids — ended up sharing the second of them with its
+* `addImage` used to increment the image's own id instead, which is how an SVG picture (a
+* pair that already consumes two ids) ended up sharing the second of them with its
 * hyperlink and emitting a duplicate `Relationship Id`.
 * @returns the allocated relationship id
 */
@@ -1310,9 +1313,7 @@ function registerHyperlinkRel(target, hyperlink) {
 }
 /** The objects one level of a text or table-cell tree holds: a table row is itself an array. */
 function hyperlinkTreeLevel(text) {
-	if (typeof text === "string" || typeof text === "number") return [];
-	if (Array.isArray(text)) return text;
-	return typeof text === "object" ? [text] : [];
+	return Array.isArray(text) ? text : [text];
 }
 /**
 * Parses text/text-objects from `addText()` and `addTable()` methods; creates 'hyperlink'-type Slide Rels for each hyperlink found
@@ -1321,7 +1322,7 @@ function hyperlinkTreeLevel(text) {
 * refused link on a second run used to throw after the first run's relationship was already on
 * the slide, left pointing at a shape that was never added.
 * @param {PresSlideInternal} target - slide object that any hyperlinks will be be added to
-* @param {number | string | TextProps | TextProps[] | TableCellInternal[][]} text - text to parse
+* @param {SlideObject | TextProps | TextProps[] | TableCellInternal[][]} text - text to parse
 * @param call - the method the objects were authored through, for a refused hyperlink's message
 */
 function createHyperlinkRels(target, text, call = "addText") {
@@ -1366,8 +1367,7 @@ function registerHyperlinkTree(target, tree) {
 		}
 	});
 }
-//#endregion
-//#region src/gen/drawingml/bullet.ts
+// src/gen/drawingml/bullet.ts
 /** A bullet `characterCode`: four hex digits naming a code point. */
 const CHARACTER_CODE = /^[0-9A-Fa-f]{4}$/;
 /** `ST_TextBulletStartAtNum`'s range. */
@@ -1412,13 +1412,12 @@ function buAutoNumEl(type, startAt, label) {
 		startAt: startAt === void 0 ? null : clampRangedInput(Math.round(startAt), START_AT_MIN, START_AT_MAX, "bullet/start-at-out-of-range", `${label} \`numberStartAt\``, "bullet/start-at-not-a-number")
 	});
 }
-//#endregion
-//#region src/gen/drawingml/clamp.ts
+// src/gen/drawingml/clamp.ts
 /**
 * pptx-ts: DrawingML value clamps
 *
 * Clamp font/character/line spacing values into their ST_* schema ranges and convert to the
-* units the attributes expect — hundredths of a point for most of them, thousandths of a
+* units the attributes expect: hundredths of a point for most of them, thousandths of a
 * percent for the one measure the caller states as a multiple rather than a size.
 * Out-of-range values make PowerPoint report the package as needing repair.
 */
@@ -1462,6 +1461,16 @@ const CHAR_SPACING = {
 	min: -4e5,
 	max: 4e5
 };
+const KERNING = {
+	code: "text/kerning-out-of-range",
+	nonFiniteCode: "coord/non-finite",
+	label: "kerning",
+	range: "0-4000pt",
+	convert: ptToHundredths,
+	perUnit: 100,
+	min: 0,
+	max: 4e5
+};
 const PARA_MARGIN = {
 	code: "text/paragraph-margin-out-of-range",
 	nonFiniteCode: "coord/non-finite",
@@ -1535,9 +1544,13 @@ function clampFontSizeSz(fontSizePts, label) {
 function clampCharSpacingSpc(charSpacingPts) {
 	return clampWithWarn(charSpacingPts, CHAR_SPACING);
 }
+/** Clamp a kerning threshold (points) into ST_TextNonNegativePoint (0..4000pt); returns hundredths for the `kern` attribute. */
+function clampKerningKern(kerningPts) {
+	return clampWithWarn(kerningPts, KERNING);
+}
 /**
 * Clamp a paragraph's left margin (points) into ST_TextMargin (0..4032pt, i.e. 0..51206400 EMU);
-* returns EMU for `a:pPr/@marL`. The type is unsigned — a negative margin is not a narrower one,
+* returns EMU for `a:pPr/@marL`. The type is unsigned. A negative margin is not a narrower one;
 * it is a value PowerPoint reports as needing repair.
 */
 function clampParaMarginEmu(marginPts) {
@@ -1587,13 +1600,12 @@ function clampLineSpacingPts(lineSpacingPts) {
 function clampLineSpacingMultiplePct(multiple) {
 	return clampWithWarn(multiple, LINE_SPACING_MULTIPLE);
 }
-//#endregion
-//#region src/gen/drawingml/math.ts
+// src/gen/drawingml/math.ts
 /**
 * pptx-ts: DrawingML native equations (OMML)
 *
 * Wrap raw OMML markup into the `<a14:m>` marker PowerPoint uses for editable
-* equations — as a standalone display paragraph (`genXmlMathParagraph`) or as an
+* equations, either as a standalone display paragraph (`genXmlMathParagraph`) or as an
 * inline run that flows between the surrounding `<a:r>` runs (`genXmlInlineMath`).
 */
 /** The `<a14:m>` marker, carrying both namespace declarations so the supplied OMML needs none. */
@@ -1616,7 +1628,7 @@ function a14Math(mathXml) {
 * @returns {string} an `<a:p>` math paragraph
 */
 function genXmlMathParagraph(omml) {
-	const trimmed = (omml || "").trim();
+	const trimmed = omml.trim();
 	const paraPr = el("m:oMathParaPr", null, raw(voidEl("m:jc", { "m:val": "centerGroup" })));
 	return el("a:p", null, [a14Math(trimmed.startsWith("<m:oMathPara") ? trimmed : trimmed.includes("<m:oMath") ? el("m:oMathPara", null, [paraPr, trimmed].map(raw)) : el("m:oMathPara", null, [paraPr, el("m:oMath", null, raw(trimmed))].map(raw))), voidEl("a:endParaRPr", { lang: "en-US" })].map(raw));
 }
@@ -1625,7 +1637,7 @@ function genXmlMathParagraph(omml) {
 *
 * Unlike the display form ({@link genXmlMathParagraph}), an inline equation is *not* its own
 * paragraph: PowerPoint authors an `<a14:m>` marker wrapping a bare `<m:oMath>` (no `<m:oMathPara>`,
-* no `<m:oMathParaPr>`/`<m:jc>`) that flows between the surrounding `<a:r>` runs in one `<a:p>` —
+* no `<m:oMathParaPr>`/`<m:jc>`) that flows between the surrounding `<a:r>` runs in one `<a:p>`,
 * pinned by the `math-omml-inline.pptx` oracle. The `mc:AlternateContent` envelope stays at the
 * shape level (see `objectHasMath`), so nothing wraps the run itself. We declare the `a14` and `m`
 * namespaces on the `<a14:m>` element so the supplied OMML needs none of its own, and accept the
@@ -1636,12 +1648,11 @@ function genXmlMathParagraph(omml) {
 * @returns {string} an `<a14:m>` inline equation run
 */
 function genXmlInlineMath(omml) {
-	const trimmed = (omml || "").trim();
+	const trimmed = omml.trim();
 	const oMathMatch = trimmed.match(/<m:oMath[\s>][\s\S]*<\/m:oMath>/);
 	return a14Math(oMathMatch ? oMathMatch[0] : el("m:oMath", null, raw(trimmed)));
 }
-//#endregion
-//#region src/gen/drawingml/text-run.ts
+// src/gen/drawingml/text-run.ts
 /**
 * `<a:extLst>` marking a hyperlink whose color should follow the text color rather than the
 * theme's hyperlink color. The leading spaces are byte-significant: this block was authored as
@@ -1649,7 +1660,7 @@ function genXmlInlineMath(omml) {
 * with `openPrefix`/`closePrefix` rather than being tidied away.
 */
 const HLINK_TEXT_COLOR_EXT = el("a:extLst", null, raw(el("a:ext", { uri: "{A12FA001-AC4F-418D-AE19-62706E023703}" }, raw(voidEl("ahyp:hlinkClr", {
-	"xmlns:ahyp": "http://schemas.microsoft.com/office/drawing/2018/hyperlinkcolor",
+	"xmlns:ahyp": OOXML_NS.ahyp,
 	val: "tx"
 }, { openPrefix: "   " })), {
 	openPrefix: "  ",
@@ -1663,7 +1674,7 @@ const HLINK_TEXT_COLOR_EXT = el("a:extLst", null, raw(el("a:ext", { uri: "{A12FA
 *
 * Two call sites downstream of the emitter act on "this paragraph has a bullet" rather than on
 * "this paragraph says something about its bullet", and both were plain truthiness tests until
-* `'inherit'` — a truthy string that draws nothing — joined the union. Left as truthiness they
+* `'inherit'`, a truthy string that draws nothing, joined the union. Left as truthiness they
 * would have broken the line grouping (every run starting its own paragraph) and eaten a real
 * leading `•` from text that emits no `a:buChar` to duplicate it.
 * @param {TextPropsOptions['bullet']} bullet - the option value
@@ -1677,12 +1688,12 @@ function emitsBulletMarkup(bullet) {
 *
 * Three inputs, because the option has three states and so does the attribute: an explicit
 * number is clamped and written, `'inherit'` writes nothing, and an absent option falls back to
-* whatever the `bullet` arm decided — which is also `null` for `bullet: 'inherit'`, the one arm
+* whatever the `bullet` arm decided, which is also `null` for `bullet: 'inherit'`, the one arm
 * that states nothing on its own.
 *
 * BOTH arms go through the clamp. The fallback used to be returned verbatim on the grounds that
 * the emitter computed it, but the emitter computes it from the caller's `bullet.indent` and
-* `indentLevel`, so it is caller input one multiplication later — and it reached `@marL` at
+* `indentLevel`, so it is caller input one multiplication later, and it reached `@marL` at
 * ~6700x the `ST_TextMargin` ceiling. The fallback arrives in EMU rather than points because the
 * bullet arms do their arithmetic there; the round trip through the clamp's own unit is exact
 * for any value in range, so a paragraph that was already legal emits the same bytes.
@@ -1700,7 +1711,7 @@ function resolveParagraphMargin(option, fallbackEmu, clamp) {
 * The paragraph's outline level, or `0` when the caller named none or named one `a:p/@lvl`
 * cannot hold.
 *
-* Resolved once because the level is read twice — for the attribute, and as the multiplier on
+* Resolved once because the level is read twice: for the attribute, and as the multiplier on
 * the bullet arms' default `@marL`. Validating it for one use and multiplying a margin by it in
 * the other is how `indentLevel: 1e6` reached `marL="342900342900"` under a warning that said
 * the value was being ignored.
@@ -1715,8 +1726,8 @@ function resolveIndentLevel(level) {
 }
 /**
 * The `@marL` a bulleted paragraph defaults to: the bullet's own hang, plus one more of it for
-* every outline level. Two arms of the bullet chain write it — `bullet: {…}` and `bullet: true`
-* — and each had the expression spelled out, which is how one of them would have been fixed
+* every outline level. Two arms of the bullet chain write it (`bullet: {…}` and `bullet: true`),
+* and each had the expression spelled out, which is how one of them would have been fixed
 * and the other left behind.
 * @param {number} bulletMarLEmu - the hang one level is worth, in EMU
 * @param {number} indentLevel - the resolved outline level
@@ -1759,7 +1770,7 @@ function genXmlParagraphProperties(textObj, isDefault) {
 			const bullet = opts.bullet;
 			const bulletImage = bullet.image;
 			const isPictureBullet = !!(bulletImage && (bulletImage.path || bulletImage.data));
-			if (opts.bullet?.indent) bulletMarL = ptsToEmuLenient(opts.bullet.indent);
+			if (opts.bullet.indent) bulletMarL = ptsToEmuLenient(opts.bullet.indent);
 			defaultMarL = bulletHangMarL(bulletMarL, indentLevel);
 			defaultIndent = -bulletMarL;
 			rejectEmptyColor(opts.bullet.color, "bullet.color");
@@ -1784,7 +1795,7 @@ function genXmlParagraphProperties(textObj, isDefault) {
 			defaultMarL = bulletHangMarL(bulletMarL, indentLevel);
 			defaultIndent = -bulletMarL;
 			strXmlBullet = buCharEl(void 0, "&#x2022;", "bullet");
-		} else if (!opts.bullet) {
+		} else {
 			defaultMarL = 0;
 			defaultIndent = 0;
 			indentBeforeMarL = true;
@@ -1806,6 +1817,17 @@ function genXmlParagraphProperties(textObj, isDefault) {
 	return paragraphPropXml;
 }
 /**
+* `underline` in its object form. `true` is shorthand for a single line; `false` states nothing, as
+* omitting the option does. Every `a:rPr` writer reads it through here, so the shorthand holds for a
+* table cell and a slide number as it does for a text run.
+* @param underline - the option as the caller wrote it
+* @returns the object form, or `undefined` when it states nothing
+*/
+function underlineProps(underline) {
+	if (underline === true) return { style: "sng" };
+	return typeof underline === "object" && underline !== null ? underline : void 0;
+}
+/**
 * Generate XML Text Run Properties (`a:rPr`)
 * @param {ObjectOptions|TextPropsOptions} opts - text options
 * @param {boolean} isDefault - whether these are the default text run properties
@@ -1814,32 +1836,33 @@ function genXmlParagraphProperties(textObj, isDefault) {
 function genXmlTextRunProperties(opts, isDefault) {
 	let runProps = "";
 	const runPropsTag = isDefault ? "a:defRPr" : "a:rPr";
-	const underline = typeof opts.underline === "object" && opts.underline?.style ? checkEnumOrWarn(opts.underline.style, TEXT_UNDERLINE_TYPES, "text/invalid-underline", "text `underline.style`") : opts.hyperlink ? "sng" : null;
-	const baseline = opts.baseline ? Math.round(opts.baseline * 50) : opts.subscript ? -4e4 : opts.superscript ? 3e4 : null;
+	const underlineOpts = underlineProps(opts.underline);
+	const underline = underlineOpts?.style ? checkEnumOrWarn(underlineOpts.style, TEXT_UNDERLINE_TYPES, "text/invalid-underline", "text `underline.style`") : opts.hyperlink ? "sng" : null;
+	const baseline = opts.baseline ? Math.round(opts.baseline * 1e3) : opts.subscript ? -4e4 : opts.superscript ? 3e4 : null;
 	const attrs = {
 		lang: opts.lang ? opts.lang : "en-US",
 		altLang: opts.lang ? "en-US" : null,
 		sz: mapStated(opts.fontSize, clampFontSizeSz) ?? null,
-		b: xsdBoolIfTrue(opts?.bold),
-		i: xsdBoolIfTrue(opts?.italic),
-		strike: opts?.strike ? typeof opts.strike === "string" ? checkEnumOrWarn(opts.strike, TEXT_STRIKE_TYPES, "text/invalid-strike", "text `strike`") : "sngStrike" : null,
-		cap: opts?.caps ? checkEnumOrWarn(opts.caps, TEXT_CAPS_TYPES, "text/invalid-caps", "text `caps`") : null,
+		b: xsdBoolIfTrue(opts.bold),
+		i: xsdBoolIfTrue(opts.italic),
+		strike: opts.strike ? typeof opts.strike === "string" ? checkEnumOrWarn(opts.strike, TEXT_STRIKE_TYPES, "text/invalid-strike", "text `strike`") : "sngStrike" : null,
+		cap: opts.caps ? checkEnumOrWarn(opts.caps, TEXT_CAPS_TYPES, "text/invalid-caps", "text `caps`") : null,
 		u: underline,
 		baseline,
 		spc: mapStated(opts.charSpacing, clampCharSpacingSpc) ?? null,
-		kern: opts.charSpacing ? 0 : null,
+		kern: opts.kerning === 0 ? 0 : mapStated(opts.kerning, clampKerningKern) ?? (opts.charSpacing ? 0 : null),
 		dirty: "0"
 	};
 	const hasShadow = !!opts.shadow && opts.shadow.type !== "none";
-	if (opts.color || opts.fontFace || opts.outline || opts.glow || hasShadow || typeof opts.underline === "object" && opts.underline.color) {
+	if (opts.color || opts.fontFace || opts.fontFaceEA || opts.outline || opts.glow || hasShadow || underlineOpts?.color) {
 		if (opts.outline && typeof opts.outline === "object") runProps += el("a:ln", { w: lineWidthToEmu(mapStated(opts.outline.size, (size) => size) ?? .75, "outline.size") }, raw(genXmlColorSelection(namedColorOr(opts.outline.color, "FFFFFF", "outline.color"))));
 		rejectEmptyColor(opts.color, "color");
 		if (opts.color) runProps += genXmlColorSelection(solidPaint(opts.color, opts.transparency));
 		if (opts.glow || hasShadow) runProps += el("a:effectLst", null, [opts.glow ? raw(createGlowElement(opts.glow, DEF_TEXT_GLOW)) : null, hasShadow ? raw(createShadowElement(opts.shadow, DEF_TEXT_SHADOW)) : null]);
 		rejectEmptyColor(opts.highlight, "highlight");
 		if (opts.highlight) runProps += el("a:highlight", null, raw(createColorElement(opts.highlight)));
-		if (typeof opts.underline === "object") rejectEmptyColor(opts.underline.color, "underline.color");
-		if (typeof opts.underline === "object" && opts.underline.color) runProps += el("a:uFill", null, raw(genXmlColorSelection(opts.underline.color)));
+		rejectEmptyColor(underlineOpts?.color, "underline.color");
+		if (underlineOpts?.color) runProps += el("a:uFill", null, raw(genXmlColorSelection(underlineOpts.color)));
 		if (opts.fontFace) {
 			runProps += voidEl("a:latin", {
 				typeface: opts.fontFace,
@@ -1848,7 +1871,7 @@ function genXmlTextRunProperties(opts, isDefault) {
 			});
 			if (opts.fontFaceEA) runProps += voidEl("a:ea", { typeface: opts.fontFaceEA });
 			runProps += voidEl("a:cs", { typeface: opts.fontFace });
-		}
+		} else if (opts.fontFaceEA) runProps += voidEl("a:ea", { typeface: opts.fontFaceEA });
 	}
 	if (opts.hyperlink) {
 		try {
@@ -1856,7 +1879,12 @@ function genXmlTextRunProperties(opts, isDefault) {
 		} catch (err) {
 			throw new InternalError("hyperlink/not-validated", `A run reached the emitter with a hyperlink its definer should have refused: ${err.message}`);
 		}
-		if (opts.hyperlink.url || opts.hyperlink.slide) {
+		if (opts.hyperlink.action && !opts.hyperlink.url && !opts.hyperlink.slide && !opts._shapeHyperlink) runProps += voidEl("a:hlinkClick", {
+			"r:id": "",
+			action: `ppaction://hlinkshowjump?jump=${opts.hyperlink.action}`,
+			tooltip: opts.hyperlink.tooltip ?? ""
+		});
+		else if (opts.hyperlink.url || opts.hyperlink.slide) {
 			const link = opts.hyperlink;
 			const linkAttrs = opts.hyperlink.url ? {
 				"r:id": `rId${link._rId}`,
@@ -1926,7 +1954,7 @@ function groupRunsIntoLines(arrTextObjects, opts) {
 			textObj.options.breakLine = false;
 		}
 		arrTexts.push(textObj);
-		if (arrTexts.length > 0 && textObj.options.breakLine) {
+		if (textObj.options.breakLine) {
 			if (idx + 1 < arrTextObjects.length) {
 				arrLines.push(arrTexts);
 				arrTexts = [];
@@ -1937,7 +1965,7 @@ function groupRunsIntoLines(arrTextObjects, opts) {
 	return arrLines;
 }
 /**
-* The options a run inherits from its shape — exactly the keys `genXmlTextRunProperties`
+* The options a run inherits from its shape: exactly the keys `genXmlTextRunProperties`
 * reads off a run's bag, in the order it reads them.
 *
 * This used to be `Object.entries(opts)` filtered on `!textOptions[key]`, which had two
@@ -1963,6 +1991,7 @@ const RUN_INHERITABLE_OPTIONS = [
 	"subscript",
 	"superscript",
 	"charSpacing",
+	"kerning",
 	"color",
 	"transparency",
 	"fontFace",
@@ -2024,8 +2053,9 @@ const PARAGRAPH_FALSY_IS_UNSTATED = [
 /**
 * Render each grouped line to an `<a:p>` paragraph: paragraph props, inherited run options,
 * text runs (and inline/display math), and the closing endParaRPr.
+* @param slideColor - the colour a run marked `_slideColor` is written in
 */
-function renderTextParagraphsXml(arrLines, slideObj, opts) {
+function renderTextParagraphsXml(arrLines, slideObj, opts, slideColor) {
 	let strSlideXml = "";
 	arrLines.forEach((line) => {
 		const firstRun = line[0];
@@ -2057,6 +2087,7 @@ function renderTextParagraphsXml(arrLines, slideObj, opts) {
 				if (key === "color" && textObj.options.hyperlink) return;
 				setOrClear(textOptions, key, textOptions[key] ?? shapeOptions[key]);
 			});
+			setOrClear(textOptions, "_shapeHyperlink", slideObj._type !== "tablecell" && textOptions.hyperlink !== void 0 && textOptions.hyperlink === shapeOptions.hyperlink ? true : void 0);
 			let _textRunObj = textObj;
 			if (idx === 0 && emitsBulletMarkup(line[0]?.options.bullet) && typeof textObj.text === "string") {
 				const _stripped = textObj.text.replace(/^[\u2022\u25E6\u25AA\u25AB\u25CF\u25CB\u2023\u2043\u2219]\s*/, "");
@@ -2067,7 +2098,16 @@ function renderTextParagraphsXml(arrLines, slideObj, opts) {
 			}
 			const isEmptyBreakArtifact = _textRunObj.text === "" && line.length > 1;
 			if (_textRunObj.math) paraXml += genXmlInlineMath(_textRunObj.math);
-			else if (!isEmptyBreakArtifact) paraXml += genXmlTextRun(_textRunObj);
+			else if (!isEmptyBreakArtifact) {
+				const runOpts = _textRunObj.options;
+				paraXml += genXmlTextRun(runOpts._slideColor && runOpts.color === void 0 ? {
+					..._textRunObj,
+					options: {
+						...runOpts,
+						color: slideColor
+					}
+				} : _textRunObj);
+			}
 			if (!textObj.text && opts.fontSize || textObj.options.fontSize) {
 				reqsClosingFontSize = true;
 				closingFontSize = closingFontSize ?? textObj.options.fontSize;
@@ -2097,8 +2137,7 @@ function renderTextParagraphsXml(arrLines, slideObj, opts) {
 	});
 	return strSlideXml;
 }
-//#endregion
-//#region src/gen/drawingml/geometry.ts
+// src/gen/drawingml/geometry.ts
 /**
 * pptx-ts: DrawingML geometry
 *
@@ -2107,8 +2146,8 @@ function renderTextParagraphsXml(arrLines, slideObj, opts) {
 * shape and image code paths so path/preset emission stays in one place.
 */
 /**
-* Several `<a:custGeom>` children are emitted as `<a:avLst />` — with a space before the
-* slash — where the rest of the tree writes `<a:avLst/>`. Byte-significant, so it stays.
+* Several `<a:custGeom>` children are emitted as `<a:avLst />` (with a space before the
+* slash) where the rest of the tree writes `<a:avLst/>`. Byte-significant, so it stays.
 */
 const SPACE_BEFORE_SLASH = { closePrefix: " " };
 /**
@@ -2122,12 +2161,12 @@ const BEZ_INDENT = {
 };
 /**
 * The 17 guide-formula operations ECMA-376 Part 1 §20.1.9.11 (`a:gd/@fmla`) defines.
-* The set is closed — the spec enumerates every operation with its arity — so an
+* The set is closed (the spec enumerates every operation with its arity), so an
 * unrecognized leading token is always a caller mistake, not a newer dialect.
 *
 * Only the operation is checked. The operands are deliberately left uninterpreted:
 * they may be literals, adjust names, or other guide names resolved in declaration
-* order, and validating them would mean re-implementing the geometry engine — which
+* order, and validating them would mean re-implementing the geometry engine, which
 * is exactly what this passthrough exists to avoid.
 */
 const GEOM_GUIDE_OPS = /* @__PURE__ */ new Set([
@@ -2164,8 +2203,8 @@ const RECT_RADIUS_ADJ1_SHAPES = /* @__PURE__ */ new Set(["round2SameRect", "roun
 *
 * An unknown `prst` value corrupts the package: PowerPoint shows the "needs
 * repair" dialog and drops the shape. So this is the safety net for every
-* `prstGeom` emitter — `addShape`, and the `shape` option on `addText`/`addImage`
-* — and it is also called during `addShape` normalization, so the caller sees
+* `prstGeom` emitter (`addShape`, and the `shape` option on `addText`/`addImage`),
+* and it is also called during `addShape` normalization, so the caller sees
 * their own typo at the API boundary rather than at serialization time.
 * @param {string} shapeName - the resolved preset geometry name
 * @throws {InvalidOptionError} `shape/unknown-preset` when the name is not a preset
@@ -2253,8 +2292,9 @@ function isArcPoint(point) {
 * `<a:ahLst>`) and guide formulas (`guides` → `<a:gdLst>`) that make the shape connectable and
 * editable. Shared by the shape and image code paths so that path emission stays in one place.
 * Points and connection-site/handle positions are authored in the object's own inch/EMU space
-* (0..cx, 0..cy) — not slide-relative and not normalized; a bare non-`Coord` string is passed
-* through verbatim as a guide-name reference (matching `ST_AdjCoordinate`'s number-or-name union).
+* (0..cx, 0..cy), not slide-relative and not normalized; a bare non-`Coord` string, in a path
+* point as in a site or handle, is passed through verbatim as a guide-name reference (matching
+* `ST_AdjCoordinate`'s number-or-name union).
 * @param {ObjectOptions} options - object options carrying `points`/`guides`/`connectionSites`/`adjustHandles`
 * @param {number} cx - object width (EMU), used as the path viewport width
 * @param {number} cy - object height (EMU), used as the path viewport height
@@ -2273,20 +2313,24 @@ function genXmlCustGeom(options, cx, cy, layout) {
 		if (typeof v === "string" && /^[A-Za-z_]/.test(v.trim())) return v;
 		return String(getSmartParseNumber(v, axis, layout));
 	};
+	/** A path coordinate: required, so an absent one writes `0` as it always has; a guide name passes through. */
+	const pathCoord = (v, axis) => adjCoord(v, axis) ?? "0";
+	/** An `ST_AdjAngle` in degrees, or a guide name (any string) emitted verbatim. */
+	const pathAngle = (v, name) => typeof v === "string" ? v : convertArcAngle(v, name);
 	/** `<a:pt>` in the object's own EMU space. */
 	const pt = (x, y) => voidEl("a:pt", {
-		x: getSmartParseNumber(x, "X", layout),
-		y: getSmartParseNumber(y, "Y", layout)
+		x: pathCoord(x, "X"),
+		y: pathCoord(y, "Y")
 	}, SPACE_BEFORE_SLASH);
 	const nodes = [];
 	points?.forEach((point, i) => {
 		if (isArcPoint(point)) {
-			if ("x" in point || "y" in point) warn("geometry/arc-node-point-ignored", "freeform arc node: x/y are ignored — an arcTo end point is computed from stAng/swAng and the radii.");
+			if ("x" in point || "y" in point) warn("geometry/arc-node-point-ignored", "freeform arc node: x/y are ignored because an arcTo end point is computed from stAng/swAng and the radii.");
 			nodes.push(voidEl("a:arcTo", {
-				hR: getSmartParseNumber(point.curve.hR, "Y", layout),
-				wR: getSmartParseNumber(point.curve.wR, "X", layout),
-				stAng: convertArcAngle(point.curve.stAng, "stAng"),
-				swAng: convertArcAngle(point.curve.swAng, "swAng")
+				hR: pathCoord(point.curve.hR, "Y"),
+				wR: pathCoord(point.curve.wR, "X"),
+				stAng: pathAngle(point.curve.stAng, "stAng"),
+				swAng: pathAngle(point.curve.swAng, "swAng")
 			}, SPACE_BEFORE_SLASH));
 		} else if ("curve" in point) switch (point.curve.type) {
 			case "cubic":
@@ -2380,7 +2424,7 @@ function genXmlCustGeom(options, cx, cy, layout) {
 	].map(raw));
 }
 /**
-* `<a:prstGeom prst="rect"><a:avLst/></a:prstGeom>` — the unadjusted rectangle every emitter
+* `<a:prstGeom prst="rect"><a:avLst/></a:prstGeom>`: the unadjusted rectangle every emitter
 * that draws a plain box writes, seven of them by hand.
 *
 * `fmt` carries the caller's `openPrefix`, because indentation before this element is
@@ -2389,8 +2433,16 @@ function genXmlCustGeom(options, cx, cy, layout) {
 function prstGeomRect(fmt) {
 	return el("a:prstGeom", { prst: "rect" }, raw(voidEl("a:avLst")), fmt);
 }
-//#endregion
-//#region src/gen/slide/objects/shared.ts
+// src/gen/slide/objects/shared.ts
+/**
+* The colour text the definer marked `_slideColor` is written in on this page: the slide's
+* {@link Slide.color}, else black. Read here, at write time, so an assignment made after the text
+* was added still reaches it (#88). A layout or the master has no `color` and gives black.
+* @param page - the page being written
+*/
+function slideTextColor(page) {
+	return "color" in page && page.color || "000000";
+}
 /**
 * The `<p:cNvPr>` every shape renderer writes: self-closing for most shape kinds, paired around a
 * hyperlink or media-action child for the others.
@@ -2491,7 +2543,7 @@ function xfrmEl(tag, frame, attrs = null, fmt) {
 * A group's `<a:xfrm>`: the box, then an IDENTITY child coordinate space (`chOff`/`chExt` equal
 * to `off`/`ext`).
 *
-* The identity is the contract `docs/groups.md` and the measured-fit solver both rest on — a
+* The identity is the contract `docs/groups.md` and the measured-fit solver both rest on: a
 * group never scales its children, so a grouped shape's authored size is its rendered size.
 * Three group emitters spelled the four-child block out.
 * @param frame - the box, in EMU
@@ -2551,13 +2603,13 @@ function spTreeEl(children) {
 *
 * The child order is the point. `CT_GraphicalObjectFrame` sequences
 * `nvGraphicFramePr` → `xfrm` → `graphic`, and PowerPoint reports a frame that gets it wrong as
-* a corrupt file (0x80070570) rather than as a bad element — a failure the five emitters that
+* a corrupt file (0x80070570) rather than as a bad element, a failure the five emitters that
 * build one (chart, table, OLE, zoom, 3D model) each had to be right about separately.
 *
 * `nvGraphicFramePr` stays the caller's to build and is *not* folded in here. The five differ in
-* every part of it — an empty `<p:cNvGraphicFramePr>` for a chart against four different
+* every part of it (an empty `<p:cNvGraphicFramePr>` for a chart against four different
 * `<a:graphicFrameLocks>` default sets, a `<p:nvPr>` that is empty for three and carries a
-* placeholder (and, for a table, a `p14:modId` extension) for the other two — and each of those
+* placeholder (and, for a table, a `p14:modId` extension) for the other two), and each of those
 * differences is deliberate and documented where it is made.
 *
 * @param opts.nvGraphicFramePr - the already-serialized `<p:nvGraphicFramePr>`
@@ -2580,7 +2632,7 @@ function graphicFrameEl(opts) {
 *
 * Fixed rather than taken from the caller's `objectLock`, and that is true of every construct
 * that emits one: the fallback picture is what a consumer *without* the feature draws, while the
-* caller's locks belong on the `mc:Choice` frame — the object PowerPoint itself manipulates. The
+* caller's locks belong on the `mc:Choice` frame, the object PowerPoint itself manipulates. The
 * two element types also accept different flags, so folding a `graphicFrameLocks` set onto a
 * `picLocks` would warn about every flag they do not share. Routed through `genXmlObjectLock` so
 * attribute order comes from `PICTURE_LOCK_ATTRS` (`gen/drawingml/locks.ts`) rather than from a literal, and a flag
@@ -2608,22 +2660,21 @@ function previewPicBody(previewRid, frame, outline = false, prefix = { ns: "p" }
 		outline ? raw(el("a:ln", { w: "3175" }, raw(el("a:solidFill", null, raw(voidEl("a:prstClr", { val: "ltGray" })))))) : null
 	]);
 }
-//#endregion
-//#region src/gen/opc/part-paths.ts
+// src/gen/opc/part-paths.ts
 /**
 * Where the parts of a written package live.
 *
 * Every part is named at least three times: the assembler writes it, `[Content_Types].xml`
 * declares an `Override` for it, and one or more `.rels` parts target it. Each of those was
-* its own template literal in its own module, and the three spellings differ — a zip entry
+* its own template literal in its own module, and the three spellings differ (a zip entry
 * name, a leading-slash part name, a relationship target resolved against the part that
-* states it — so nothing tied them together. A part written but not declared is a
+* states it), so nothing tied them together. A part written but not declared is a
 * PowerPoint repair, and so is a relationship pointing at a part nobody wrote.
 *
 * The canonical spelling here is the zip entry name: `ppt/slides/slide1.xml`, no leading
-* slash, which is what {@link ZipWriter.add} takes. The other two are derivations —
+* slash, which is what {@link ZipWriter.add} takes. The other two are derivations:
 * {@link overrideName} for the content-types declaration, {@link targetFromPresentation} and
-* {@link targetFromPptSubpart} for a relationship target — so a path can only be spelled once
+* {@link targetFromPptSubpart} for a relationship target. So a path can only be spelled once
 * and then transformed, never re-typed in a second dialect.
 *
 * Indices are 1-based throughout, matching the OPC naming rather than the arrays that drive it.
@@ -2644,7 +2695,7 @@ function slideLayoutPath(n) {
 }
 /**
 * The `n`-th notes-slide part. One is written for every slide, empty notes included, so the
-* index is the slide's own — see the assembler, which keeps the two enumerations in step.
+* index is the slide's own; see the assembler, which keeps the two enumerations in step.
 */
 function notesSlidePath(n) {
 	return `ppt/notesSlides/notesSlide${n}.xml`;
@@ -2676,14 +2727,14 @@ function overrideName(path) {
 	return `/${path}`;
 }
 /**
-* `path` as a relationship target stated by a part directly under `ppt/` — presentation.xml,
+* `path` as a relationship target stated by a part directly under `ppt/`: presentation.xml,
 * whose rels reach `slides/slide1.xml` with no traversal at all.
 */
 function targetFromPresentation(path) {
 	return path.slice(4);
 }
 /**
-* `path` as a relationship target stated by a part one level below `ppt/` — a slide, a layout,
+* `path` as a relationship target stated by a part one level below `ppt/`: a slide, a layout,
 * a notes slide, the master: `../slides/slide1.xml`.
 */
 function targetFromPptSubpart(path) {
@@ -2699,7 +2750,7 @@ function targetFromPptSubpart(path) {
 *
 * It exists because the slide-link target used to be spelled `slide${n}.xml` unconditionally, by
 * a function that also serves layouts and the master. A relationship target resolves against the
-* part that states it, so from `ppt/slideLayouts/` that named `ppt/slideLayouts/slide1.xml` — a
+* part that states it, so from `ppt/slideLayouts/` that named `ppt/slideLayouts/slide1.xml`, a
 * part no package has. Deriving the target from the emitting part's own path means the question
 * is asked once, by the only code that can answer it.
 */
@@ -2707,14 +2758,13 @@ function targetBetweenPptSubparts(fromPath, path) {
 	const dir = fromPath.slice(0, fromPath.lastIndexOf("/") + 1);
 	return path.startsWith(dir) ? path.slice(dir.length) : targetFromPptSubpart(path);
 }
-//#endregion
-//#region src/gen/opc/rels.ts
+// src/gen/opc/rels.ts
 /**
 * pptx-ts: shared `Relationship` writers.
 *
 * A `.rels` part is mostly one-off entries, written where the part that needs
 * them is built. This module holds the ones **more than one part** writes, so
-* that the attribute set stays a single fact — a `Relationship` whose `Type` and
+* that the attribute set stays a single fact. A `Relationship` whose `Type` and
 * `TargetMode` disagree between two writers does not throw, it produces a link
 * PowerPoint silently drops.
 */
@@ -2726,7 +2776,7 @@ function targetBetweenPptSubparts(fromPath, path) {
 * passed `rId3` or `3`, with seven more written inline in `gen/slide/`. The attribute set and
 * its order are byte-significant, so one writer is the only way they stay one fact.
 *
-* `target` is passed **unescaped** — `voidEl` escapes it on the way out. `gen/slide/object.ts`
+* `target` is passed **unescaped**; `voidEl` escapes it on the way out. `gen/slide/object.ts`
 * depends on that: it computes the escaped form separately to compare a rel against the bytes
 * already emitted, and the two escapers must not drift.
 * @param {string | number} id - the relationship id; a number gets the `rId` prefix
@@ -2756,7 +2806,7 @@ function relationshipsEl(rels, fmt) {
 /**
 * A complete `.rels` part: the XML declaration, a CRLF, then {@link relationshipsEl}.
 *
-* The embedded workbook's parts are the exception and use {@link relationshipsEl} directly —
+* The embedded workbook's parts are the exception and use {@link relationshipsEl} directly:
 * they follow the declaration with nothing, and two of them end with a bare newline.
 */
 function relationshipsPart(rels, fmt) {
@@ -2774,14 +2824,13 @@ function relationshipsPart(rels, fmt) {
 function externalHyperlinkRel(rId, target) {
 	return relationshipEl(rId, HYPERLINK_REL, target, { targetMode: "External" });
 }
-//#endregion
-//#region src/gen/drawingml/list-style.ts
+// src/gen/drawingml/list-style.ts
 /**
 * pptx-ts: the `<a:lvlNpPr>` list-style level and the theme-font `<a:defRPr>` inside it.
 *
-* A deck states its text defaults in four places — the presentation's `p:defaultTextStyle`, the
-* notesMaster's `p:notesStyle`, and the slide master's built-in and configured `p:txStyles` —
-* and all four write the same level element around the same run-property block: a `solidFill`
+* A deck states its text defaults in four places: the presentation's `p:defaultTextStyle`, the
+* notesMaster's `p:notesStyle`, and the slide master's built-in and configured `p:txStyles`.
+* All four write the same level element around the same run-property block: a `solidFill`
 * over one colour, then the `latin`/`ea`/`cs` triple pointing at one theme font family. They
 * differed in the size, in whether `b`/`i` were carried, and in whether the colour and the
 * Latin face were overridable, which is what the parameters here are.
@@ -2818,8 +2867,7 @@ function lvlPPr(level, attrs, children) {
 		...LEVEL_PPR_TAIL
 	}, children);
 }
-//#endregion
-//#region src/gen/slide/notes.ts
+// src/gen/slide/notes.ts
 /**
 * Collect the speaker-notes runs for a slide (flattened across any number of `addNotes()` calls).
 * @param {PresSlideInternal} slide - the slide object
@@ -2844,8 +2892,8 @@ function getNotesRuns(slide) {
 * can be written twice or gain notes between writes. Each notes-part writer computes this once and
 * hands the result to both the body and the rels file, which is what keeps the two in agreement.
 *
-* Only external `url` hyperlinks are supported; any other target is left off the copy, and a
-* `slide` target says so.
+* A `url` takes a hyperlink relationship and a `slide` a slide relationship, as on a slide; a show
+* jump (`action`) takes none.
 * @param {PresSlideInternal} slide - the slide object
 * @return {NotesSlideContent} the notes hyperlink relationships and the runs referencing them
 */
@@ -2857,15 +2905,7 @@ function buildNotesSlideRels(slide) {
 		rels,
 		runs: getNotesRuns(slide).map((run) => {
 			const hyperlink = run.options?.hyperlink;
-			if (!hyperlink || !run.options) return run;
-			if (!hyperlink.url) {
-				if (hyperlink.slide) warn("notes/hyperlink-slide-unsupported", "notes hyperlinks support `url` only (ignoring `slide` target)");
-				const { hyperlink: _unsupported, ...options } = run.options;
-				return {
-					...run,
-					options
-				};
-			}
+			if (!hyperlink || !run.options || !(hyperlink.url || hyperlink.slide)) return run;
 			lastRid++;
 			rels.push(hyperlinkRel(lastRid, hyperlink));
 			const linked = {
@@ -2909,8 +2949,8 @@ function genXmlNotesParagraphs(runs) {
 /**
 * PowerPoint's creation-id extension, on the `p:cSld` of both notes parts.
 *
-* The value is a fixed literal, and that is the point of naming it: it looks like data — a
-* per-deck identifier PowerPoint would vary — so left inline in two places, someone eventually
+* The value is a fixed literal, and that is the point of naming it: it looks like data (a
+* per-deck identifier PowerPoint would vary), so left inline in two places, someone eventually
 * changes one of them. It is deliberately constant, which is what keeps the two notes parts
 * byte-identical across builds.
 */
@@ -3158,8 +3198,8 @@ function makeXmlNotesSlide(slide, notes) {
 /**
 * The notes slide's fixed frame, with the only two slide-dependent strings passed in.
 *
-* A notes slide is the same three placeholders every time — the thumbnail (`sldImg`),
-* the notes body, and the slide-number field — so everything except the body's
+* A notes slide is the same three placeholders every time: the thumbnail (`sldImg`),
+* the notes body, and the slide-number field. So everything except the body's
 * paragraphs and the cached slide number is a constant. It is a parameter of its own
 * because the read model authors notes onto a *loaded* deck (`Slide.addNotes`), where
 * there is no `PresSlideInternal` to serialize but the part must come out identical:
@@ -3238,7 +3278,7 @@ function makeXmlNotesSlideSkeleton(bodyParagraphsXml, slideNum) {
 * @return {string} XML
 */
 function makeXmlNotesSlideRel(notes, slideNumber) {
-	const hlinkRels = notes.rels.map((rel) => externalHyperlinkRel(rel.rId, rel.Target)).join("");
+	const hlinkRels = notes.rels.map((rel) => rel.data === "slide" ? relationshipEl(rel.rId, SLIDE_REL, targetFromPptSubpart(slidePath(Number(rel.Target)))) : externalHyperlinkRel(rel.rId, rel.Target)).join("");
 	return XML_DECL + relationshipsEl([
 		relationshipEl(1, NOTES_MASTER_REL, targetFromPptSubpart(NOTES_MASTER_PATH)),
 		relationshipEl(2, SLIDE_REL, targetFromPptSubpart(slidePath(slideNumber))),
@@ -3259,7 +3299,95 @@ function makeXmlNotesMasterRel() {
 		closePrefix: "\n		"
 	});
 }
-//#endregion
-export { noStrokeLine as $, graphicFrameEl as A, getDuplicateObjectNames as At, genXmlParagraphProperties as B, targetBetweenPptSubparts as C, FONT_DATA_CONTENT_TYPE as Ct, cNvPrEl as D, serializeEmbeddedFontLst as Dt, FALLBACK_PICTURE_LOCKS as E, flattenEmbeddedFaces as Et, assertKnownPreset as F, nextMediaTarget as Ft, clampParaMarginInchesEmu as G, renderTextParagraphsXml as H, genXmlCustGeom as I, preencodedPath as It, createHyperlinkRels as J, buAutoNumEl as K, genXmlPresetGeom as L, validateObjectName as Lt, previewPicBody as M, getUuid as Mt, spTreeEl as N, isHyperlinkRel as Nt, cNvPrHyperlink as O, XML_DECL as Ot, xfrmEl as P, isPreencodedPath as Pt, createLineCap as Q, prstGeomRect as R, encodeXmlAttrValue as Rt, slidePath as S, voidEl as St, targetFromPresentation as T, FONT_REL_TYPE as Tt, clampFontSizeSz as U, groupRunsIntoLines as V, clampParaIndentInchesEmu as W, validateHyperlink as X, registerHyperlinkRel as Y, borderLine as Z, fontPath as _, namedColorOr as _t, makeXmlNotesSlideRel as a, fillNamesPaint as at, relsPath as b, elOrVoid as bt, themeFontDefRPr as c, rejectEmptyFill as ct, relationshipsEl as d, genXmlImageCropRect as dt, resolveBorderDash as et, relationshipsPart as f, createShadowEffectLst as ft, commentPath as g, isColorValue as gt, SLIDE_MASTER_PATH as h, createColorElement as ht, makeXmlNotesSlide as i, withLineDefaults as it, grpXfrmEl as j, getNewRelId as jt, genXmlShapeLine as k, bracedGuid as kt, externalHyperlinkRel as l, resolveFillKind as lt, PRESENTATION_PATH as m, alphaEl as mt, makeXmlNotesMaster as n, strokeDash as nt, makeXmlNotesSlideSkeleton as o, genXmlColorSelection as ot, NOTES_MASTER_PATH as p, normalizeShadowOptions as pt, buCharEl as q, makeXmlNotesMasterRel as r, strokePaint as rt, lvlPPr as s, genXmlPatternFill as st, buildNotesSlideRels as t, resolveDash as tt, relationshipEl as u, STRETCH_FILL_RECT as ut, notesSlidePath as v, rejectEmptyColor as vt, targetFromPptSubpart as w, FONT_DATA_EXTENSION as wt, slideLayoutPath as x, raw as xt, overrideName as y, el as yt, genXmlNormAutofit as z };
+// src/gen/slide/comments.ts
+/**
+* pptx-ts: comment parts
+*
+* Resolve every slide's comments into a deck-wide author registry with per-author
+* numbering, then emit the presentation-level `commentAuthors.xml` and each
+* slide's `comments/commentN.xml` part.
+*/
+/**
+* Walk every slide's comments and build the deck-wide author registry + per-comment numbering.
+*
+* Legacy comments number each comment with a *per-author* 1-based `idx`; an author's `lastIdx` is the
+* highest `idx` it used. Authors are keyed by `name`+`initials` and assigned ids in first-appearance
+* order (slide order, then insertion order). `clrIdx` mirrors `id` (one colour slot per author).
+* @param {PresSlideInternal[]} slides - all presentation slides
+* @return {ResolvedComments} the author list and per-comment metadata
+*/
+function resolveCommentAuthors(slides) {
+	const byKey = /* @__PURE__ */ new Map();
+	const authors = [];
+	const perAuthorCount = /* @__PURE__ */ new Map();
+	const meta = /* @__PURE__ */ new Map();
+	slides.forEach((slide) => {
+		slide._comments.forEach((comment) => {
+			const key = comment.author + "\0" + comment.initials;
+			let author = byKey.get(key);
+			if (!author) {
+				author = {
+					id: authors.length,
+					name: comment.author,
+					initials: comment.initials,
+					lastIdx: 0,
+					clrIdx: authors.length
+				};
+				byKey.set(key, author);
+				authors.push(author);
+				perAuthorCount.set(author.id, 0);
+			}
+			const idx = (perAuthorCount.get(author.id) ?? 0) + 1;
+			perAuthorCount.set(author.id, idx);
+			author.lastIdx = idx;
+			meta.set(comment, {
+				authorId: author.id,
+				idx
+			});
+		});
+	});
+	return {
+		authors,
+		meta
+	};
+}
+/**
+* Creates the presentation-level `ppt/commentAuthors.xml` part (`<p:cmAuthorLst>`).
+* @param {ResolvedCommentAuthor[]} authors - resolved deck-wide author registry
+* @return {string} XML
+*/
+function makeXmlCommentAuthors(authors) {
+	const cmAuthors = authors.map((author) => voidEl("p:cmAuthor", {
+		id: author.id,
+		name: author.name,
+		initials: author.initials,
+		lastIdx: author.lastIdx,
+		clrIdx: author.clrIdx
+	})).join("");
+	return XML_DECL + "\r\n" + el("p:cmAuthorLst", { ...PML_ROOT_NS }, raw(cmAuthors));
+}
+/**
+* Creates a per-slide comments part `ppt/comments/comment{N}.xml` (`<p:cmLst>`).
+* @param {SlideComment[]} comments - the slide's comments, in order
+* @param {Map} meta - per-comment `authorId`/`idx`, from {@link resolveCommentAuthors} on a write
+* and from the destination's registry on an append
+* @return {string} XML
+*/
+function makeXmlComments(comments, meta) {
+	const cms = comments.map((comment) => {
+		const m = meta.get(comment);
+		if (!m) throw new InternalError("comment/not-resolved", "makeXmlComments: a comment has no author metadata");
+		return el("p:cm", {
+			authorId: m.authorId,
+			dt: comment.date || null,
+			idx: m.idx
+		}, [raw(voidEl("p:pos", {
+			x: Math.round(inch2Emu(comment.x)),
+			y: Math.round(inch2Emu(comment.y))
+		})), raw(el("p:text", null, comment.text))]);
+	}).join("");
+	return XML_DECL + "\r\n" + el("p:cmLst", { ...PML_ROOT_NS }, raw(cms));
+}
+export { buAutoNumEl as $, cNvPrEl as A, namedColorOr as At, genXmlCustGeom as B, getUuid as Bt, relsPath as C, resolveFillKind as Ct, targetFromPptSubpart as D, alphaEl as Dt, targetBetweenPptSubparts as E, genXmlImageCropRect as Et, previewPicBody as F, voidEl as Ft, groupRunsIntoLines as G, validateObjectName as Gt, prstGeomRect as H, isPreencodedPath as Ht, slideTextColor as I, XML_DECL as It, clampFontSizeSz as J, renderTextParagraphsXml as K, encodeXmlAttrValue as Kt, spTreeEl as L, bracedGuid as Lt, genXmlShapeLine as M, el as Mt, graphicFrameEl as N, elOrVoid as Nt, targetFromPresentation as O, createColorElement as Ot, grpXfrmEl as P, raw as Pt, clampParaMarginInchesEmu as Q, xfrmEl as R, getDuplicateObjectNames as Rt, overrideName as S, rejectEmptyFill as St, slidePath as T, STRETCH_FILL_RECT as Tt, genXmlNormAutofit as U, nextMediaTarget as Ut, genXmlPresetGeom as V, isHyperlinkRel as Vt, genXmlParagraphProperties as W, preencodedPath as Wt, clampParaIndentInchesEmu as X, clampParaIndentEmu as Y, clampParaMarginEmu as Z, PRESENTATION_PATH as _, flattenEmbeddedFaces as _t, makeXmlNotesMaster as a, createLineCap as at, fontPath as b, genXmlColorSelection as bt, makeXmlNotesSlideRel as c, resolveDash as ct, themeFontDefRPr as d, withLineDefaults as dt, buCharEl as et, externalHyperlinkRel as f, createShadowEffectLst as ft, NOTES_MASTER_PATH as g, FONT_REL_TYPE as gt, relationshipsPart as h, FONT_DATA_EXTENSION as ht, buildNotesSlideRels as i, borderLine as it, cNvPrHyperlink as j, rejectEmptyColor as jt, FALLBACK_PICTURE_LOCKS as k, isColorValue as kt, makeXmlNotesSlideSkeleton as l, strokeDash as lt, relationshipsEl as m, FONT_DATA_CONTENT_TYPE as mt, makeXmlComments as n, registerHyperlinkRel as nt, makeXmlNotesMasterRel as o, noStrokeLine as ot, relationshipEl as p, normalizeShadowOptions as pt, underlineProps as q, resolveCommentAuthors as r, validateHyperlink as rt, makeXmlNotesSlide as s, resolveBorderDash as st, makeXmlCommentAuthors as t, createHyperlinkRels as tt, lvlPPr as u, strokePaint as ut, SLIDE_MASTER_PATH as v, serializeEmbeddedFontLst as vt, slideLayoutPath as w, solidPaint as wt, notesSlidePath as x, genXmlPatternFill as xt, commentPath as y, fillNamesPaint as yt, assertKnownPreset as z, getNewRelId as zt };
 
-//# sourceMappingURL=notes-BcT571R4.js.map
+//# sourceMappingURL=comments-CKIZYRbZ.js.map