fix(skills): stop embedded-captions shipping author-only paths; dedupe slideshow

[jrusso1020]

What

Skill-content hygiene across two skills, plus the manifest regen both require.

Closes #3219. Supersedes #3218 (carried here re-signed, authorship preserved).

File	Change
embedded-captions/references/test-set.md	deleted (orphaned)
embedded-captions/themes/PORTING.md	deleted (orphaned)
embedded-captions/SKILL.md	archive location claim -> "not distributed with the skill"
embedded-captions/dna/README.md	same
slideshow/SKILL.md	duplicated media-cleanup block removed (559 -> 529)
skills-manifest.json	regenerated for both skills

Why

embedded-captions (#3219). The skill is distributed via hyperframes skills, which installs with --copy so the installed bundle matches the published tree. There is no per-file filter, and the skill's .gitignore only keeps render artifacts out of commits. So these files land on every install, where the directories they name do not exist.

1. references/test-set.md was unreachable and unrunnable: zero inbound references across all 140 files, corpus at ~/Downloads/heygen_relevant_videos/. As a control, the same grep finds typographic-moves.md linked 4 times, so the method discriminates rather than just returning nothing.
2. themes/PORTING.md was equally unreachable, also zero inbound references. It authored new theme DNAs rather than using the skill, five of its steps took author-only inputs, and its normative CONTRACT.md ships nowhere in this repo (checked by content, not filename).
3. The retired 54-template archive was stated as a fact about where something lives. True for one machine.

Not touched, so it does not get re-flagged: the 10 references to ~/Downloads/hyperframes are the documented last resort in a real chain (HYPERFRAMES_ROOT -> in-repo -> that default). Every file carrying the path also carries the env var, and the chain ends in a clear set HYPERFRAMES_ROOT error.

slideshow (#3218). The media-cleanup and global-nav-mute sections appeared twice, and the copies were not identical: the later one dropped the paragraph requiring custom media controls to sync through media events. The incomplete copy is the one deleted.

How

Both orphaned files are deleted rather than annotated. Nothing referenced either, git history keeps them recoverable, and marking a dead procedure still ships the dead procedure.

The slideshow change is @imprimisxo's work from #3218, carried here byte-identical with --author preserving their authorship. Two reasons it moved: the org requires signed commits and the original was unsigned, and both changes regenerate skills-manifest.json, so as separate PRs they would conflict on that file no matter which merged first.

Test plan

• bun packages/cli/scripts/gen-skills-manifest.ts --check -> in sync (19 skills)

• slideshow hash resolves to 6a24a84b0c1a75f9, matching what CI computed on fix(skills): remove duplicated media-cleanup block in slideshow #3218

• Re-verified the dedup at this tree: 559 -> 529 lines, 28 code fences balanced, each repeated section 2 -> 1, and the custom-controls rule still present once at line 445

• Confirmed both deleted files had zero inbound references before removal

• Post-change audit grep: no author-only ~/Downloads paths remain

• Unit tests added/updated (docs + manifest only, no code paths touched)

• Manual testing performed

• Documentation updated (if applicable)

---

#3219 reported by @imprimisxo, who also authored the slideshow fix in #3218.

Opened by Rames Jusso (James's assistant)

[miga-heygen]

Two skill hygiene fixes combined to share one manifest regen.

Slideshow dedup — same change I reviewed in #3218, carried here re-signed. Surviving copy is the complete one (custom-controls paragraph intact, 559→529 lines, 28 fences balanced). Already verified.

embedded-captions cleanup (#3219):

• test-set.md deleted — zero inbound references across all 140 files, corpus at ~/Downloads/heygen_relevant_videos/ which doesn't exist on any install. Correct removal.
• PORTING.md deleted — zero inbound references, every step requires author-only inputs (cap_fx3 demos, CONTRACT.md), none of which are distributed. Correct removal.
• Archive location claims in SKILL.md and dna/README.md changed from ~/Downloads/... to "not distributed with the skill" — factual correction.
• The 10 remaining ~/Downloads/hyperframes references in the skill are deliberately untouched — they're the documented last resort in a real env-var chain, not orphaned claims.

Allowed-deletions entries in check-no-main-deletions.mjs have clear explanations. Manifest regenerated for both skills.

All four commits are signed (G). James's authorship preserved on the slideshow commit.

LGTM.

Review by Miga

[miguel-heygen]

Audited: the complete 7-file diff at 1d0e18ca870b4966ff80b6b6eff1c06227a53afa, including both skill sources, the two intentional deletions, deletion-guard exceptions, and regenerated manifest. The slideshow carry is byte-identical to the reviewed #3218 source; its complete custom-controls guidance remains, line count drops 559→529, and all 28 fences stay balanced. The deleted embedded-captions files have zero inbound references, while the surviving text correctly stops promising author-local assets that are not distributed. Manifest hashes/counts match the resulting tree, and all five commits are GitHub-verified while preserving Lance’s authorship on the carried change.

Trusting: GitHub Actions runner execution; every required exact-head check is terminal green, including the full test suite and both Windows render/test lanes. No unresolved review threads or unaddressed blockers remain.

Verdict: APPROVE

Reasoning: The combined PR removes unreachable/duplicated skill material without weakening the surviving guidance, keeps generated state synchronized, preserves provenance, and passes the repository’s complete required gate set.

— Magi