pptd-utils 2026.10.2.dev3__tar.gz → 2026.10.2.dev4__tar.gz

{pptd_utils-2026.10.2.dev3 → pptd_utils-2026.10.2.dev4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: pptd-utils
-Version: 2026.10.2.dev3
+Version: 2026.10.2.dev4
 Summary: Offline PPTD v2 toolkit for converting editable presentations to PowerPoint
 Author: binaryify
 License: MIT

{pptd_utils-2026.10.2.dev3 → pptd_utils-2026.10.2.dev4}/app/src/Deck/context.tsx

@@ -66,5 +66,5 @@ function useDeckCtx(): DeckCtx {
   return ctx
 }
 
-export type { DeckCtx, DeckFeatures }
+export type { DeckCtx }
 export { Ctx, useDeckCtx }

{pptd_utils-2026.10.2.dev3 → pptd_utils-2026.10.2.dev4}/app/src/Renderer.tsx

@@ -59,7 +59,7 @@ export function PageView({ page, slide, onSelect, static: isStatic = false, edit
       {page.elements.map(el => {
         const anim = active.get(el.elementId)
         const hidden = groups.length > 0 && !visible.has(el.elementId) && page.animations?.some(a => a.elementId === el.elementId)
-        if (editingId === el.elementId) return null // editor overlay replaces it
+        if (editingId !== undefined && editingId === el.elementId) return null // editor overlay replaces it
         return <div
           key={el.elementId}
           className={act ? 'cursor-crosshair' : undefined}

{pptd_utils-2026.10.2.dev3 → pptd_utils-2026.10.2.dev4}/app/src/load.ts

@@ -90,6 +90,13 @@ export async function loadProject(src: FileSource): Promise<LoadedProject> {
   for (const rel of pptd.pages) {
     const page = yamlLoad(await src.read(base + rel)) as Page
     if (!page || !Array.isArray(page.elements)) throw new Error(`invalid page: ${rel}`)
+    const ids = new Set<string>()
+    for (const [index, el] of page.elements.entries()) {
+      if (typeof el?.elementId !== 'string' || !el.elementId.trim())
+        throw new Error(`${rel}: element ${index + 1}: elementId must be a nonempty string`)
+      if (ids.has(el.elementId)) throw new Error(`${rel}: duplicate elementId: ${el.elementId}`)
+      ids.add(el.elementId)
+    }
     pages.push(resolveSrc(page, base, urlOf) as Page)
     pagePaths.push(base + rel)
   }

pptd_utils-2026.10.2.dev4/app/test/load.test.ts

@@ -0,0 +1,17 @@
+import { expect, test } from 'bun:test'
+import { loadProject } from '../src/load'
+
+function load(elements: unknown[]) {
+  const files: Record<string, string> = {
+    'deck.pptd': JSON.stringify({ pages: ['slide.page'] }),
+    'slide.page': JSON.stringify({ elements }),
+  }
+  return loadProject({ list: async () => Object.keys(files), read: async p => files[p], url: p => p })
+}
+
+test('requires nonempty string element IDs, unique per page', async () => {
+  for (const elementId of [undefined, null, '', '  ', 42])
+    await expect(load([{ elementId }])).rejects.toThrow('elementId must be a nonempty string')
+  await expect(load([{ elementId: 'title' }, { elementId: 'title' }])).rejects.toThrow('duplicate elementId: title')
+  expect((await load([{ elementId: 'title' }, { elementId: 'body' }])).pages).toHaveLength(1)
+})

{pptd_utils-2026.10.2.dev3 → pptd_utils-2026.10.2.dev4}/pptd_utils/main.py

@@ -119,7 +119,13 @@ def build(pptd_path, out_path):
         slide = prs.slides.add_slide(prs.slide_layouts[6])
         _set_background(slide, page.get("background"), theme, root, prs)
         shape_ids = {}
-        for el in page.get("elements", []):
+        for index, el in enumerate(page.get("elements", []), 1):
+            element_id = el.get("elementId") if isinstance(el, dict) else None
+            if not isinstance(element_id, str) or not element_id.strip():
+                raise ValueError(f"{page_rel}: element {index}: elementId must be a nonempty string")
+            if element_id in shape_ids:
+                raise ValueError(f"{page_rel}: duplicate elementId: {element_id}")
+            shape_ids[element_id] = None
             shape = _render_element(slide, el, theme, root)
             if shape is not None:
                 shape_ids[el.get("elementId")] = (

{pptd_utils-2026.10.2.dev3 → pptd_utils-2026.10.2.dev4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "pptd-utils"
-version = "2026.10.2.dev3"
+version = "2026.10.2.dev4"
 description = "Offline PPTD v2 toolkit for converting editable presentations to PowerPoint"
 readme = "pptd_utils/README.md"
 requires-python = ">=3.9"

{pptd_utils-2026.10.2.dev3 → pptd_utils-2026.10.2.dev4}/skills/cowork-ppt/reference/pptd.md

@@ -508,6 +508,19 @@ elements:
 
 Common properties of all elements.
 
+**Every element must declare `elementId`: a nonempty string, unique within its page.**
+This is required for every element type, even without animations. IDs may repeat
+across different pages. Missing, blank, non-string, or duplicate IDs are invalid;
+the local app loader and `pptd_utils` exporter reject them. IDs are not generated
+automatically.
+
+```yaml
+- elementId: title-1
+  elementType: text
+  bounds: [100, 50, 760, 80]
+  content: {text: Annual Work Summary}
+```
+
 ```ts
 interface ElementBase {
   elementId: string;                                                      // constraint: unique within the same page; unique element ID

{pptd_utils-2026.10.2.dev3 → pptd_utils-2026.10.2.dev4}/skills/cowork-ppt/scripts/pptd_utils/main.py

@@ -119,7 +119,13 @@ def build(pptd_path, out_path):
         slide = prs.slides.add_slide(prs.slide_layouts[6])
         _set_background(slide, page.get("background"), theme, root, prs)
         shape_ids = {}
-        for el in page.get("elements", []):
+        for index, el in enumerate(page.get("elements", []), 1):
+            element_id = el.get("elementId") if isinstance(el, dict) else None
+            if not isinstance(element_id, str) or not element_id.strip():
+                raise ValueError(f"{page_rel}: element {index}: elementId must be a nonempty string")
+            if element_id in shape_ids:
+                raise ValueError(f"{page_rel}: duplicate elementId: {element_id}")
+            shape_ids[element_id] = None
             shape = _render_element(slide, el, theme, root)
             if shape is not None:
                 shape_ids[el.get("elementId")] = (

pptd_utils-2026.10.2.dev4/tests/element_ids.py

@@ -0,0 +1,30 @@
+"""Run: python3 tests/element_ids.py"""
+import sys
+import tempfile
+from pathlib import Path
+
+sys.path.insert(0, str(Path(__file__).resolve().parents[1]))
+from pptd_utils.main import build
+import yaml
+
+with tempfile.TemporaryDirectory() as tmp:
+    root = Path(tmp)
+    manifest = root / 'deck.pptd'
+    manifest.write_text(yaml.safe_dump({'version': 'v2', 'size': [960, 540], 'pages': ['slide.page']}))
+    for ids, error in [([None], 'nonempty string'), ([''], 'nonempty string'),
+                       (['  '], 'nonempty string'), ([42], 'nonempty string'),
+                       (['title', 'title'], 'duplicate elementId')]:
+        elements = [{'elementType': 'text', 'bounds': [0, 0, 100, 40],
+                     'content': {'text': 'Test'}, 'elementId': i} for i in ids]
+        (root / 'slide.page').write_text(yaml.safe_dump({'elements': elements}))
+        try:
+            build(manifest, root / 'deck.pptx')
+        except ValueError as exc:
+            assert error in str(exc), str(exc)
+        else:
+            raise AssertionError(f'accepted invalid IDs: {ids}')
+        assert not (root / 'deck.pptx').exists()
+    elements[1]['elementId'] = 'body'
+    (root / 'slide.page').write_text(yaml.safe_dump({'elements': elements}))
+    assert build(manifest, root / 'deck.pptx')[0] == 1
+print('element ID checks passed')