Content
63%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The core ElevenLabs guidance is excellent — executable code, practical model/voice-setting tables, and hard-won gotchas (SSML support per model, ellipsis warning, IVC vs clone API). Quality drops in the Remotion section, which duplicates library knowledge Claude already has and inlines what should be a separate reference, and the skill's outbound references point to files that do not exist in the bundle.
Suggestions
Move the Remotion integration (timing patterns, error handling, demo-video sync) into a separate reference file and keep only a short workflow + manifest example in SKILL.md, trimming the generic FadeInAudio/DelayedAudio/SafeAudio patterns Claude already knows.
Fix or remove the broken references: `reference.md` and `../provider-refresh-oct23/SKILL.md` do not exist in the bundle, so the 'full API docs' and v4-transport pointers currently dead-end.
Briefly define or link the environment-specific tooling invoked in the body (`tools/voiceover.py`, `/generate-voiceover`, `ElevenLabsTTS`) so those snippets are actionable outside the unnamed toolkit.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The TTS core is dense and high-value (model table, voice-settings table, SSML gotchas like the ellipsis warning), but the Remotion section inlines ~230 lines of generic patterns Claude already knows (FadeInAudio/interpolate fades, DelayedAudio with Sequence, delayRender error handling, playbackRate math). Fits "mostly efficient but includes some unnecessary explanation or could be tightened"; not 4 because the generic Remotion material is more than a minor trim. | 3 / 5 |
Actionability | Copy-paste-ready Python for TTS, instant voice cloning, sound effects, and music compose, plus concrete ffmpeg conversion and voice-ID tables — mostly executable. Minor gaps keep it below 5: `ElevenLabsTTS().execute(...)`, `python tools/voiceover.py`, and `/generate-voiceover` reference tooling that is never defined and not present in the bundle. | 4 / 5 |
Workflow Clarity | The Remotion integration gives an explicit Step 1–3 sequence with manifest-driven timing and an Error Handling section, and the TTS section has a genuine listen → adjust → regenerate feedback loop. Checkpoints are mostly present (SafeAudio continueRender, alignment_matches_text flag), but there is no validation step before committing scene durations to the composition, so it falls just under anchor 5. | 4 / 5 |
Progressive Disclosure | Section headers are clear and references are prominently signaled ("For full API docs, see [reference.md](reference.md)"), but no bundle files exist — both `reference.md` and `../provider-refresh-oct23/SKILL.md` are missing — and the large Remotion integration content that belongs in a separate reference file is fully inlined. This matches "references present but not clearly backed; content that should be separate is inline"; not 2 because the internal structure is genuinely well-organized, not minimal. | 3 / 5 |
Total | 14 / 20 Passed |