refactor(knowledge): rename youtube skill to youtube-digest

[kyle-sexton]

What

Atomic rename of the knowledge plugin's youtube skill to youtube-digest (one of the shadowed-skill renames charted in docs/topics/shadowed-skill-renames/PLAN.md, PR #256):

• git mv plugins/knowledge/skills/youtube → plugins/knowledge/skills/youtube-digest (166 files, nearly all pure renames)
• Frontmatter name: youtube → youtube-digest; '/youtube' trigger token → '/youtube-digest'
• All internal ${CLAUDE_PLUGIN_ROOT}/skills/youtube/... paths → skills/youtube-digest/... (SKILL.md, context docs, templates, evals, and the generated recovery command in detect-recoverable-bootstrap.js)
• Slash references /knowledge:youtube → /knowledge:youtube-digest in the skill, its siblings (course-digest SKILL/context/evals/reference), and the knowledge README
• Bare `/youtube watch`/`/youtube resume` skill references in extraction JS comments and the generated continuation prompt → /youtube-digest ...
• Living docs: docs/MIGRATION-PLAYBOOK.md, docs/knowledge-integration-design.md skill-name mentions updated
• plugins/knowledge bumped 0.5.6 → 0.6.0 with a breaking-rename CHANGELOG entry

Why

Sibling skills follow a source+operation grammar (book-distill, course-digest); the platform noun alone named the source but not the operation.

Breaking

/knowledge:youtube no longer resolves — invoke /knowledge:youtube-digest. Model-invocation triggers are preserved: "youtube", "watch this YouTube video", and youtube.com/youtu.be URL sharing still route to the skill. In-flight .work/<watch-epic>/... slices are unaffected; resume via /knowledge:youtube-digest resume <video-slug>.

Deliberate skips

• Platform-name prose ("YouTube video", URL-matching text) unchanged everywhere
• Historical CHANGELOG entries untouched (skills/youtube/ and /knowledge:youtube remain only there)
• vendor/ untouched; marketplace.json renames map untouched; youtube kept as a discovery keyword in plugin.json/marketplace tags
• Hypothetical future standalone youtube plugin name kept as-is (README revisit condition, CATALOG-TAXONOMY graduation trigger, plugin-organization PLAN) — those name a plugin, not this skill
• Internal identifiers not derived from the skill name kept: @melodic/youtube-extraction package, {tmp}/youtube-* temp dirs, "youtube-extraction deps"

Verification

• git grep "knowledge:youtube" → only -digest form outside historical CHANGELOG entries
• git grep "skills/youtube/" → zero hits outside historical CHANGELOG entries
• All edited JSON (plugin.json, both evals.json, marketplace.json) parses

🤖 Generated with Claude Code

https://claude.ai/code/session_01YVApnNGoKxYMJKNQx1QK8C

Related

• Program contract: program: shadowed-skill-renames contract (naming grammar rollout) #280 (shadowed-skill-renames naming grammar rollout)
• Conventions PR: docs: codify skill-naming grammar and cross-plugin reference rules #256

No linked issue — program-level rename executed under #280.

[chatgpt-codex-connector]

💡 Codex Review

claude-code-plugins/plugins/knowledge/skills/youtube-digest/evals/evals.json

Line 2 in 5276881

"skill_name": "youtube",

Update the eval skill name after the rename

For eval runs that use skill_name to select or report the skill, this still declares youtube even though the skill was renamed to youtube-digest and the eval prompts below now invoke /knowledge:youtube-digest; those runs can target or attribute results to a removed skill instead of covering the new command. Please update this to youtube-digest.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[kyle-sexton]

Codex review finding VALID — evals.json still declared skill_name: "youtube". Fixed in 7425d82: now youtube-digest. (Review-body finding — no reactions endpoint on that surface; this reply is the classification record.)

[claude]

Claude finished @kyle-sexton's task in 1m 43s —— View job

---

PR Review

• Gather context and understand the PR scope
• Review content changes (non-rename diffs)
• Verify evals.json fix (7425d82)
• Check cross-references and completeness

---

This is a clean, well-scoped atomic rename. The Codex finding (stale skill_name: "youtube" in evals.json) was valid and correctly addressed in the follow-up commit 7425d82. No further issues found.

What I verified

Frontmatter / dispatch (SKILL.md)
SKILL.md line 2: name: youtube-digest ✓ — trigger description also references '/youtube-digest' and preserves the natural-language triggers (youtube, watch this YouTube video, URL patterns) so model-invocation routing is unbroken.

Evals fix
evals/evals.json line 2: "skill_name": "youtube-digest" ✓ — and all nine eval prompts now invoke /knowledge:youtube-digest ... ✓.

Generated recovery command
detect-recoverable-bootstrap.js line 112: emits skills/youtube-digest/extraction/run.mjs ✓.

Sibling cross-references

• course-digest/SKILL.md: routes YouTube URLs to /knowledge:youtube-digest ✓
• course-digest/evals/evals.json: eval youtube-url-routes-to-youtube-digest-skill correctly expects a redirect to /knowledge:youtube-digest ✓
• course-digest/context/storage-schema.md and reference/adapters/discovery-checklist.md: updated ✓

Context files (watch-pipeline.md, watch-queue.md, etc.)
All ${CLAUDE_PLUGIN_ROOT}/skills/youtube-digest/... paths confirmed correct; no stale skills/youtube/ paths appear outside the CHANGELOG.

/knowledge:youtube grep
Zero hits outside historical CHANGELOG entries ✓ — the deliberate skips (old CHANGELOG prose, vendor, marketplace renames map, youtube keyword tags) are appropriate and documented.

Version bump & CHANGELOG
plugin.json bumped 0.5.6 → 0.6.0 with a clear BREAKING label and resume instructions — correct semver for a breaking rename ✓.

Docs
docs/MIGRATION-PLAYBOOK.md and docs/knowledge-integration-design.md updated to youtube-digest ✓.

Verdict

Approve. The rename is complete and consistent. The only issue raised during review (Codex P2: stale skill_name in evals.json) was fixed before this review ran. No new concerns.