@erclx/canon 5.21.0 → 5.23.0

package/docs/workflow/skill-map.md

@@ -118,24 +118,20 @@ This page is the corpus the coverage claim is measured against: every name `cano
 
 ## Generate an artifact on demand
 
-| Skill                     | When to use                                                                                                       |
-| ------------------------- | ----------------------------------------------------------------------------------------------------------------- |
-| `canon:create-rule`       | For a project-specific governance rule the toolkit does not ship                                                  |
-| `canon:create-skill`      | For a new `SKILL.md`                                                                                              |
-| `canon:create-standard`   | For a new authoring convention                                                                                    |
-| `canon:draft-docs`        | For a brand-new `docs/*.md` page, drafted against `standards/docs.md`                                             |
-| `canon:draft-ready`       | For finished files a worker should copy, written as a ready folder with its overview, thin plan, and task         |
-| `canon:draft-context`     | For a brand-new `canon/context/<domain>.md` entry, drafted against `standards/context.md`                         |
-| `canon:draft-wireframes`  | For a brand-new `canon/wireframes/<surface>.md` file, drafted against `standards/wireframes.md`                   |
-| `canon:draft-wiki`        | For a brand-new wiki reference page on a subject owned outside the project, drafted against `standards/wiki.md`   |
-| `canon:draft-figure`      | For a hand-drawn figure inside an existing doc, drafted against `standards/figures.md` in Mermaid or freehand SVG |
-| `canon:draft-readme`      | For a project's `README.md`, drafted against `standards/readme.md`                                                |
-| `canon:bash-cli-script`   | For a small non-interactive shell wrapper, stopping at 100 lines in favor of TypeScript on Bun                    |
-| `canon:ci-workflow`       | For a GitHub Actions workflow file                                                                                |
-| `canon:draft-slides`      | For a deck, drafted as `.claude/SLIDES.md` and rendered to PowerPoint                                             |
-| `canon:draft-screencast`  | For a recording script with beats and defaults already seeded                                                     |
-| `canon:record-screencast` | For recording a screencast draft, then composing the take into a finished mp4 when the draft asks for a wrap      |
-| `canon:draft-identity`    | For a project's logo mark and its social card, drafted through `draft-and-pick`'s pick loop                       |
+| Skill                     | When to use                                                                                                                                    |
+| ------------------------- | ---------------------------------------------------------------------------------------------------------------------------------------------- |
+| `canon:create-rule`       | For a project-specific governance rule the toolkit does not ship                                                                               |
+| `canon:create-skill`      | For a new `SKILL.md`                                                                                                                           |
+| `canon:create-standard`   | For a new authoring convention                                                                                                                 |
+| `canon:draft-doc`         | For a brand-new `docs/` page, context entry, wireframe surface, wiki reference page, or `README.md`, drafted against the standard for its kind |
+| `canon:draft-ready`       | For finished files a worker should copy, written as a ready folder with its overview, thin plan, and task                                      |
+| `canon:draft-figure`      | For a hand-drawn figure inside an existing doc, drafted against `standards/figures.md` in Mermaid or freehand SVG                              |
+| `canon:bash-cli-script`   | For a small non-interactive shell wrapper, stopping at 100 lines in favor of TypeScript on Bun                                                 |
+| `canon:ci-workflow`       | For a GitHub Actions workflow file                                                                                                             |
+| `canon:draft-slides`      | For a deck, drafted as `.claude/SLIDES.md` and rendered to PowerPoint                                                                          |
+| `canon:draft-screencast`  | For a recording script with beats and defaults already seeded                                                                                  |
+| `canon:record-screencast` | For recording a screencast draft, then composing the take into a finished mp4 when the draft asks for a wrap                                   |
+| `canon:draft-identity`    | For a project's logo mark and its social card, drafted through `draft-and-pick`'s pick loop                                                    |
 
 ## Answer a question at any point
 

package/governance/rules/canon/604-scratch.md

@@ -7,4 +7,3 @@ description: Write temporary files to the scratch folder structure
 ## Temporary files
 
 - Write temporary files to `.canon/tmp/<slug>/<file>.md` at the current worktree's root, a nested folder with a kebab-slug tied to the topic rather than a flat `<slug>-<file>.md`. The slug must not take one of the five reserved names `runs`, `hooks`, `handoff`, `pr`, and `render`. A skill that resolves a write at the main root instead states so in its own body.
-- Write to `.claude/.tmp/<slug>/` instead in a project that carries no `.canon/` root, and `canon migrate records` moves it to the first spelling. <!-- canon-keep-record-root -->

package/governance/rules/claude/575-hooks.md

@@ -1,8 +1,7 @@
 ---
-description: State the hook stdin guard and the silencing rule for .claude/hooks scripts and their seeded copies
+description: State the hook stdin guard and the silencing rule for .claude/hooks scripts
 paths:
   - '.claude/hooks/**/*.sh'
-  - 'tooling/claude/seeds/.claude/hooks/**/*.sh'
 ---
 
 # Hook standards

package/governance/rules/claude/576-settings.md

@@ -1,8 +1,7 @@
 ---
-description: State the autoContinueAtUsageLimit inversion and the autoCompactWindow cap for .claude/settings.json and its seeded copy
+description: State the autoContinueAtUsageLimit inversion and the autoCompactWindow cap for .claude/settings.json
 paths:
   - '.claude/settings.json'
-  - 'tooling/claude/seeds/.claude/settings.json'
 ---
 
 # Settings standards

package/governance/rules/standards/architecture.md

@@ -3,7 +3,6 @@
 description: Shape and content rules for canon/ARCHITECTURE.md
 paths:
   - 'canon/ARCHITECTURE.md'
-  - '.claude/ARCHITECTURE.md'
 ---
 
 # Architecture standards

package/governance/rules/standards/context.md

@@ -3,7 +3,6 @@
 description: Shape and content rules for canon/context/<domain>.md entries
 paths:
   - 'canon/context/**'
-  - '.claude/context/**'
 ---
 
 # Context entry standards

package/governance/rules/standards/design.md

@@ -3,7 +3,6 @@
 description: Shape and content rules for canon/DESIGN.md
 paths:
   - 'canon/DESIGN.md'
-  - '.claude/DESIGN.md'
 ---
 
 # Design standards

package/governance/rules/standards/diagrams.md

@@ -3,7 +3,6 @@
 description: Shape and content rules for .canon/diagrams/<kind>.md files
 paths:
   - '.canon/diagrams/**'
-  - '.claude/DIAGRAMS.md'
 ---
 
 # Diagram standards
@@ -13,5 +12,3 @@ paths:
 - Follow the diagrams standard for shape and content rules for .canon/diagrams/<kind>.md files. It is the single source. Read it with `canon standards diagrams`.
 - Layout, budgets, accessibility, labels, and render verification inside the fence are a separate topic. The mermaid standard routes them.
 - A diagram entry carries structure and flow, not implementation. Read the standard before adding or revising a kind.
-- Write a new diagram to `.canon/diagrams/<kind>.md`, never to `.claude/DIAGRAMS.md`
-- Convert a `.claude/DIAGRAMS.md` left by an older install into per-kind entries before editing it

package/governance/rules/standards/requirements.md

@@ -3,7 +3,6 @@
 description: Shape and content rules for canon/REQUIREMENTS.md
 paths:
   - 'canon/REQUIREMENTS.md'
-  - '.claude/REQUIREMENTS.md'
 ---
 
 # Requirements standards

package/governance/rules/standards/wireframes.md

@@ -3,7 +3,6 @@
 description: Shape and content rules for canon/wireframes/<surface>.md files
 paths:
   - 'canon/wireframes/**'
-  - '.claude/wireframes/**'
 ---
 
 # Wireframe standards

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@erclx/canon",
   "type": "module",
-  "version": "5.21.0",
+  "version": "5.23.0",
   "description": "Runs your agent's work and keeps its code good, on one setup for every repo.",
   "license": "MIT",
   "bin": {

package/scripts/manage-sandbox.sh

@@ -190,10 +190,10 @@ provision_sandbox() {
   (cd "$SANDBOX" && configure_sandbox_git_credentials)
 }
 
-# Runs the real installer rather than copying the source tree. The two copies
-# these replaced each reimplemented an installer's selection rules, so a change
-# to what install produces left the sandbox provisioning the old shape and no
-# scenario could observe the difference.
+# Runs the real installer rather than copying the source tree. The copies these
+# replaced, for gov rules and for seeds, each reimplemented an installer's
+# selection rules, so a change to what install produces left the sandbox
+# provisioning the old shape and no scenario could observe the difference.
 run_sandbox_install() {
   local label="$1"
   shift
@@ -223,10 +223,9 @@ inject_gov_rules() {
 }
 
 inject_seeds() {
-  local seeds_source="$PROJECT_ROOT/tooling/claude/seeds"
-  if [ -d "$seeds_source" ]; then
-    cp -r "$seeds_source/." "$SANDBOX/"
-  fi
+  [ ! -d "$PROJECT_ROOT/tooling/claude/seeds" ] && return
+
+  run_sandbox_install "seeds" claude init "$SANDBOX"
 }
 
 commit_environment_setup() {

package/src/claude/cases/authoring.ts

@@ -31,7 +31,7 @@ export const AUTHORING_CASES: readonly SkillCase[] = [
   {
     prompt:
       'Write a brand-new docs page for the capture command, nothing under docs/ covers it yet.',
-    expect: 'draft-docs',
+    expect: 'draft-doc',
   },
   {
     prompt:
@@ -40,22 +40,22 @@ export const AUTHORING_CASES: readonly SkillCase[] = [
   },
   {
     prompt: 'This project has no README.md at all, write one from scratch.',
-    expect: 'draft-readme',
+    expect: 'draft-doc',
   },
   {
     prompt:
       'Write a context entry for the payments domain, there is no canon/context page for it yet.',
-    expect: 'draft-context',
+    expect: 'draft-doc',
   },
   {
     prompt:
       'Draft a wireframe for the settings panel, nothing under canon/wireframes covers that surface yet.',
-    expect: 'draft-wireframes',
+    expect: 'draft-doc',
   },
   {
     prompt:
       'Write a wiki reference page for Claude Code output styles, no page covers that subject yet.',
-    expect: 'draft-wiki',
+    expect: 'draft-doc',
   },
   {
     prompt: 'Say what that dense answer actually means in plain terms.',

package/src/commands/slides.ts

@@ -119,10 +119,10 @@ export function register(program: Command): void {
 
 /**
  * The converter loads on demand, so no other command resolves pptxgenjs,
- * jszip, or the token module at startup. A fallback, a refused link or chart,
- * or a token the master fell back on reports one line each and keeps the exit
- * at 0, as an unrecognized layout does on the markdown path, since the deck is
- * written either way. A malformed `deck.json` refuses before any slide is laid
+ * jszip, or the token module at startup. A fallback, a refused link, chart, or
+ * effect, or a token the master fell back on reports one line each and keeps
+ * the exit at 0, as an unrecognized layout does on the markdown path, since the
+ * deck is written either way. A malformed `deck.json` refuses before any slide is laid
  * out and exits 1.
  */
 async function renderHtmlFolder(
@@ -163,6 +163,16 @@ async function renderHtmlFolder(
       `${GREY}│${NC} ${RED}✗${NC} slide ${slide} ${selector} chart left out: ${message}\n`,
     )
   }
+  for (const { slide, message } of result.refusedMotion) {
+    process.stderr.write(
+      `${GREY}│${NC} ${RED}✗${NC} slide ${slide} motion left out: ${message}\n`,
+    )
+  }
+  for (const { path, message } of result.refusedFonts) {
+    process.stderr.write(
+      `${GREY}│${NC} ${RED}✗${NC} font ${path} not embedded: ${message}\n`,
+    )
+  }
   for (const notice of result.notices) {
     process.stderr.write(`${GREY}│${NC} ${RED}✗${NC} ${notice}\n`)
   }

package/src/commands/sync.ts

@@ -168,6 +168,16 @@ function renderCheck(report: CheckReport): void {
     logInfo('Move the content yourself. No sync command touches these.')
   }
 
+  if (report.unmovedSurfaces.length > 0) {
+    logStep('Surfaces at the old .claude/ root')
+    for (const entry of report.unmovedSurfaces) {
+      logWarn(`${entry.rel} (moves to ${entry.movesTo})`)
+    }
+    logInfo(
+      'Rules glob the canon/ spelling only. Run `canon migrate surface-roots` to move these.',
+    )
+  }
+
   renderSeeds(report)
 
   if (report.superseded.length > 0) {

package/src/commands/tasks.ts

@@ -42,6 +42,7 @@ import {
 import {
   type Finding,
   type FolderClaim,
+  type WideToken,
   type Unplaced,
   type Untested,
   type ValidateOutcome,
@@ -1411,6 +1412,16 @@ function reportValidation(
       for (const claim of outcome.claims) logWarn(describeClaim(claim))
     }
 
+    // The width limit is a heuristic over one preview pane, so a wide token
+    // reports and moves no exit code, and a measure failing on a cell that
+    // fits would teach a reader to skip it.
+    logStep('Wide tokens')
+    if (outcome.wide.length === 0) {
+      logInfo('every code span fits the preview pane')
+    } else {
+      for (const token of outcome.wide) logWarn(describeWide(token))
+    }
+
     // Unplaced is the normal state between a session filing a task and an
     // orchestrator placing its row, so it moves no exit code. It still reports,
     // since it is the only local detector for a row a hand-edit dropped or a
@@ -1436,6 +1447,7 @@ function reportValidation(
         findings: outcome.findings,
         untested: outcome.untested,
         claims: outcome.claims,
+        wide: outcome.wide,
         unplaced: outcome.unplaced,
       })}\n`,
     )
@@ -1457,6 +1469,10 @@ function describeClaim(claim: FolderClaim): string {
   return `${claim.group}: ${claim.subject} ${claim.message}`
 }
 
+function describeWide(token: WideToken): string {
+  return `${token.group}: ${token.subject} ${token.message}`
+}
+
 function describeUnplaced(task: Unplaced): string {
   return `${task.subject} ${task.message}`
 }

package/src/gov/citations.ts

@@ -266,8 +266,7 @@ export function collectCitations(text: string): RawCitation[] {
  *
  * A rule spelling a whole path there is declaring that exact artifact, and a
  * body line naming it again is naming what the rule governs rather than
- * pointing a reader somewhere. `governance/rules/standards/diagrams.md` tells
- * its reader to convert a `.claude/DIAGRAMS.md` left by an older install, which
+ * pointing a reader somewhere. Such a path names a file a target holds, so it
  * is correctly absent from this tree and correctly named in the rule.
  */
 function governedPaths(text: string): string[] {

package/src/migrate/skill-names.ts

@@ -33,6 +33,13 @@ import { defineRenameRules, type RenameRules } from '@/migrate/rename'
  * its name collided with `docs-sync`, which owns `docs/`. `claude-docs` is
  * retargeted onto `context-fold` for the same reason `claude-ui-test` was.
  *
+ * Five rows merge skills rather than rename one. `draft-docs`,
+ * `draft-context`, `draft-wireframes`, `draft-readme`, and `draft-wiki` ran
+ * one procedure five times over, differing only in the standard each read and
+ * the catalog each checked, so they became `draft-doc`, which picks the kind
+ * from where the document lands. Every retired name lands on the one skill, so
+ * a target citing any of them resolves after a single sweep.
+ *
  * Every name takes two words. Ten of these would have landed as a bare single
  * word under a plain strip, and a bare word such as `review` or `docs` is a
  * substring of ordinary prose with no token left for a later sweep to find.
@@ -79,6 +86,11 @@ export const SKILL_NAME_MAP: Readonly<Record<string, string>> = {
   'canon-slides-draft': 'draft-slides',
   'canon-record': 'record-screencast',
   'canon-frames-read': 'read-frames',
+  'draft-docs': 'draft-doc',
+  'draft-context': 'draft-doc',
+  'draft-wireframes': 'draft-doc',
+  'draft-readme': 'draft-doc',
+  'draft-wiki': 'draft-doc',
 }
 
 /**

package/src/slides/convert/deck.ts

@@ -1,5 +1,5 @@
-import { existsSync, readFileSync, statSync } from 'node:fs'
-import { basename, join } from 'node:path'
+import { existsSync, readFileSync, realpathSync, statSync } from 'node:fs'
+import { basename, isAbsolute, join, relative, resolve } from 'node:path'
 
 /**
  * What belongs to the deck rather than to one slide, read from `deck.json`
@@ -27,6 +27,18 @@ export interface DeckConfig {
   readonly slideNumbers: boolean
   /** Absolute, resolved against the deck folder. */
   readonly mark?: string
+  /** The faces the package step embeds, in the order the deck lists them. */
+  readonly fonts: readonly DeckFont[]
+}
+
+export interface DeckFont {
+  readonly family: string
+  readonly weight: number
+  readonly style: 'normal' | 'italic'
+  /** Absolute, resolved against the deck folder and kept inside it. */
+  readonly path: string
+  /** The path as `deck.json` wrote it, which a refusal names. */
+  readonly source: string
 }
 
 export type DeckRead =
@@ -43,8 +55,10 @@ const DECK_FIELDS = new Set([
   'footer',
   'slideNumbers',
   'mark',
+  'fonts',
 ])
 const BAND_FIELDS = new Set<string>(['show', ...BAND_SLOTS])
+const FONT_FIELDS = new Set(['family', 'weight', 'style', 'path'])
 
 class FieldError extends Error {
   constructor(
@@ -133,7 +147,73 @@ function parse(raw: unknown, folder: string): DeckConfig {
       throw new FieldError('mark', `mark ${mark} does not exist`)
     }
   }
-  return { title, header, footer, slideNumbers, mark }
+  const fonts = fontList(raw.fonts, folder)
+  return { title, header, footer, slideNumbers, mark, fonts }
+}
+
+function isWithin(folder: string, path: string): boolean {
+  const inside = relative(folder, path)
+  return inside !== '' && !inside.startsWith('..') && !isAbsolute(inside)
+}
+
+/**
+ * A face is read from the deck folder only, so a `deck.json` cannot point the
+ * export at a file elsewhere on the machine and carry it out inside the deck.
+ */
+function fontList(value: unknown, folder: string): DeckFont[] {
+  if (value === undefined) return []
+  if (!Array.isArray(value)) {
+    throw new FieldError('fonts', 'fonts must be a list')
+  }
+  return value.map((entry: unknown, index): DeckFont => {
+    const field = `fonts[${index}]`
+    if (!isRecord(entry)) {
+      throw new FieldError(field, `${field} must be an object`)
+    }
+    rejectUnknown(entry, FONT_FIELDS, `${field}.`)
+    const family = stringField(entry.family, `${field}.family`)
+    if (!family) {
+      throw new FieldError(`${field}.family`, `${field}.family is required`)
+    }
+    const weight = entry.weight ?? 400
+    if (
+      typeof weight !== 'number' ||
+      !Number.isInteger(weight) ||
+      weight < 1 ||
+      weight > 1000
+    ) {
+      throw new FieldError(
+        `${field}.weight`,
+        `${field}.weight must be a whole number from 1 to 1000`,
+      )
+    }
+    const style = entry.style ?? 'normal'
+    if (style !== 'normal' && style !== 'italic') {
+      throw new FieldError(
+        `${field}.style`,
+        `${field}.style must be normal or italic`,
+      )
+    }
+    const source = stringField(entry.path, `${field}.path`)
+    if (!source) {
+      throw new FieldError(`${field}.path`, `${field}.path is required`)
+    }
+    const path = resolve(folder, source)
+    const outside = new FieldError(
+      `${field}.path`,
+      `${field}.path ${source} lies outside the deck folder`,
+    )
+    if (!isWithin(folder, path)) throw outside
+    if (!existsSync(path) || !statSync(path).isFile()) {
+      throw new FieldError(
+        `${field}.path`,
+        `${field}.path ${path} does not exist`,
+      )
+    }
+    // A link inside the folder can still lead out of it.
+    if (!isWithin(realpathSync(folder), realpathSync(path))) throw outside
+    return { family, weight, style, path, source }
+  })
 }
 
 export function readDeck(folder: string): DeckRead {

package/src/slides/convert/export.ts

@@ -42,13 +42,23 @@ import {
   type WalkedSlide,
   walkSlide,
 } from '@/slides/convert/walk'
+import { embedFonts, type FontNotice } from '@/slides/package/fonts'
+import {
+  type EntranceSpec,
+  entranceShapes,
+  type MotionNotice,
+  type SlideMotionEdit,
+  writeMotion,
+} from '@/slides/package/motion'
 
 /**
  * Turns a folder of HTML slides into one editable deck. Chromium lays each
  * slide out at 1280 by 720 in the project's tokens, `walkSlide` reads what it
  * placed, `planSlide` maps the records onto shapes, and an element whose CSS has
  * no mapping lands as a screenshot of itself. The first slide's computed tokens
- * build the masters, and `deck.json` sets the bands they carry.
+ * build the masters, and `deck.json` sets the bands they carry. Once pptxgenjs
+ * has written the package, `src/slides/package/` adds the transitions,
+ * entrances, and embedded faces it declares nothing for.
  *
  * Every browser reference this feature adds lives here, and the command reaches
  * it through a dynamic import so no other command resolves the engine.
@@ -94,6 +104,10 @@ export type ExportResult =
       readonly fallbacks: readonly SlideFallback[]
       readonly refusedLinks: readonly SlideRefusedLink[]
       readonly refusedCharts: readonly RefusedChart[]
+      /** Transitions and entrances left out, each with the reason. */
+      readonly refusedMotion: readonly MotionNotice[]
+      /** Faces `deck.json` lists that the deck does not carry, with the reason. */
+      readonly refusedFonts: readonly FontNotice[]
       /** What the deck master could not take from the project's tokens. */
       readonly notices: readonly string[]
     }
@@ -172,6 +186,7 @@ export async function exportHtmlDeck(
   const refusedLinks: SlideRefusedLink[] = []
   const refusedCharts: RefusedChart[] = []
   const patches: Patch[] = []
+  const motions: SlideMotionEdit[] = []
   const notices: string[] = []
 
   try {
@@ -227,9 +242,8 @@ export async function exportHtmlDeck(
         slide.background = { color: walked.background.hex }
       }
       if (walked.meta.isHidden) slide.hidden = true
-      for (const [opIndex, op] of plan.ops.entries()) {
-        const name = `canon-${slideNumber}-${opIndex + 1}`
-        await draw(page, slide, { slide: slideNumber, name }, op, patches)
+      for (const op of plan.ops) {
+        await draw(page, slide, slideNumber, op, patches)
       }
       for (const record of walked.charts) {
         const chart = planChart(record, chartTheme)
@@ -241,6 +255,24 @@ export async function exportHtmlDeck(
       }
       drawOverrides(slide, deck, theme, walked.meta)
       if (walked.meta.notes) slide.addNotes(walked.meta.notes)
+      const { transition } = walked.meta
+      const roots = new Set(
+        walked.entrances.flatMap(({ record }) =>
+          record === null ? [] : [record],
+        ),
+      )
+      const entrances = walked.entrances.map(
+        ({ record, ...spec }): EntranceSpec => ({
+          ...spec,
+          shapes: entranceShapes(record, walked.records, plan.names, roots),
+        }),
+      )
+      if (transition || entrances.length > 0) {
+        motions.push({
+          slide: slideNumber,
+          motion: { ...(transition ? { transition } : {}), entrances },
+        })
+      }
     }
     if (drifted.length > 0) {
       const noun = drifted.length === 1 ? 'slide' : 'slides'
@@ -262,12 +294,32 @@ export async function exportHtmlDeck(
   if (!(written instanceof Uint8Array)) {
     return refused('export-failed', 'pptxgenjs returned no buffer')
   }
-  const patched = await applyPatches(written, patches)
+  // A deck needing no edit keeps the bytes pptxgenjs wrote.
+  let packaged: Uint8Array = written
+  const refusedMotion: MotionNotice[] = []
+  const refusedFonts: FontNotice[] = []
+  if (patches.length > 0 || motions.length > 0 || deck.fonts.length > 0) {
+    const zip = await JSZip.loadAsync(written)
+    await applyPatches(zip, patches)
+    refusedMotion.push(...(await writeMotion(zip, motions)))
+    const faces = deck.fonts.map((font) => ({
+      family: font.family,
+      weight: font.weight,
+      style: font.style,
+      path: font.source,
+      bytes: readFileSync(font.path),
+    }))
+    refusedFonts.push(...(await embedFonts(zip, faces)))
+    packaged = await zip.generateAsync({
+      type: 'nodebuffer',
+      compression: 'DEFLATE',
+    })
+  }
 
   mkdirSync(outDir, { recursive: true })
   const fileName = `${basename(sourceDir)}.pptx`
   const pptxPath = join(outDir, fileName)
-  writeFileSync(pptxPath, patched)
+  writeFileSync(pptxPath, packaged)
 
   let mirrorPath: string | undefined
   if (options.mirror) {
@@ -284,6 +336,8 @@ export async function exportHtmlDeck(
     fallbacks,
     refusedLinks,
     refusedCharts,
+    refusedMotion,
+    refusedFonts,
     notices,
   }
 }
@@ -437,10 +491,11 @@ const base64 = (bytes: Buffer | string): string =>
 async function draw(
   page: Page,
   slide: PptxGenJS.Slide,
-  target: { readonly slide: number; readonly name: string },
+  slideNumber: number,
   op: DrawOp,
   patches: Patch[],
 ): Promise<void> {
+  const target = { slide: slideNumber, name: op.options.objectName ?? '' }
   if (op.kind === 'shape') {
     slide.addShape(op.shape, op.options)
   } else if (op.kind === 'text') {
@@ -448,18 +503,13 @@ async function draw(
   } else if (op.kind === 'table') {
     slide.addTable(op.rows, op.options)
   } else if (op.kind === 'image') {
-    slide.addImage({
-      ...op.options,
-      ...imageSource(op.src),
-      objectName: target.name,
-    })
+    slide.addImage({ ...op.options, ...imageSource(op.src) })
     if (op.radius > 0) patches.push({ ...target, radius: op.radius })
     if (op.frame) slide.addShape(op.radius > 0 ? 'roundRect' : 'rect', op.frame)
   } else if (op.kind === 'svg') {
     slide.addImage({
       ...op.options,
       data: `image/svg+xml;base64,${base64(op.markup)}`,
-      objectName: target.name,
     })
     patches.push({ ...target, png: await screenshot(page, op.id) })
   } else {
@@ -472,14 +522,13 @@ async function draw(
  * Two things pptxgenjs cannot write, fixed in the package it wrote. An SVG's
  * PNG fallback comes out as the SVG's own bytes under a `.png` name in Node, so
  * a viewer that cannot draw SVG gets a broken image, and a picture takes no
- * corner radius. Each patched picture is found by the object name `draw` gave
- * it, which is the only handle the written XML keeps.
+ * corner radius. Each patched picture is found by the object name `planSlide`
+ * gave it, which is the only handle the written XML keeps.
  */
 async function applyPatches(
-  deck: Uint8Array,
+  zip: JSZip,
   patches: readonly Patch[],
-): Promise<Buffer> {
-  const zip = await JSZip.loadAsync(deck)
+): Promise<void> {
   const bySlide = Map.groupBy(patches, (patch) => patch.slide)
   for (const [slideNumber, slidePatches] of bySlide) {
     const slidePath = `ppt/slides/slide${slideNumber}.xml`
@@ -504,7 +553,6 @@ async function applyPatches(
     }
     zip.file(slidePath, xml)
   }
-  return zip.generateAsync({ type: 'nodebuffer', compression: 'DEFLATE' })
 }
 
 function pictureXml(xml: string, name: string): string | undefined {