Skip to content

Sync 02-example export/deck fixes into the skill scaffold - #25

Merged
tph-kds merged 16 commits into
mainfrom
feat/sync-example-fixes-scaffold
Aug 16, 2026
Merged

Sync 02-example export/deck fixes into the skill scaffold#25
tph-kds merged 16 commits into
mainfrom
feat/sync-example-fixes-scaffold

Conversation

@tph-kds

@tph-kds tph-kds commented Aug 16, 2026

Copy link
Copy Markdown
Owner

Vendors the proven export/deck layer from examples/02-example/ into the canonical skills/deckforge/starter-components/ so freshly generated decks get working PPTX export, image handling, charts, process blocks, and offline embedding.

Highlights

  • Vendored deck/export modules byte-for-byte from 02-example (43-file drift-guard, all VERBATIM blobs identical).
  • deck-types.ts rebuilt as a re-export shim (Block/SlideInteraction import binding fixed).
  • Rebuilt export/index.ts barrel + export-coverage audit.
  • Chart-spec vendored; docs mandate snapshot export, self-contained output, and asset-manifest image workflow.
  • Drift-guard (check_scaffold_sync.py) + est_scaffold_sync.py wired into
    pm run validate.
  • Embedded skill copy kept in sync (idempotent skills:sync).

Verification

  • Typecheck exit 0 (78 src + 16 test + 2 script files).
  • Vitest 16/16 suites, 457 tests pass.
  • Export preflight PASS (offline, zero missing assets); PPTX smoke export PASS (ppt/media 5 entries, 353,899 B).

pm run validate passes except a pre-existing est_release_gate failure (reproduces identically on a clean main clone; unrelated).

  • Final whole-branch review: APPROVED, no blockers.

Note:
pm run validate is checked locally only; CI status on this PR reflects the repo's pre-existing release-gate test.

tph-kds added 16 commits August 15, 2026 03:40
…nents

Regenerate examples/02-example/.agents/skills/deckforge to mirror the
vendored scaffold deck/ and export/ modules plus the rebuilt barrel and
barrel-coverage audit. Preserve two embedded-only system-prompt.md sections
(slot-positioning contract, export safety) into canonical before syncing.
@cursor

cursor Bot commented Aug 16, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@tph-kds
tph-kds merged commit eed3a71 into main Aug 16, 2026
16 of 31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant