docs: retire the docs-hosted copy of the update-docs skill - #1182
Merged
Conversation
The shared skill now lives in pipecat-ai/pipecat, published through the pipecat-dev-skills marketplace that already distributes that repo's other skills (pipecat-ai/pipecat#5471). That is the right home: six of the ten skills there are org-generic rather than pipecat-specific, so Cloud and client developers want the same plugin, and it is the repo that actually gets maintained. This directory was the first attempt at the same goal, hosting the skill here and leaving a pointer stub in pipecat. That stub would have shipped through the marketplace, since the plugin manifest sources skills from ./.claude/skills/*, giving anyone who updated the plugin a skill whose entire content was a path into a repo they may not have checked out. Nothing references these files any more: pipecat's workflow and skill are clean of the hosted path, and pipecat-flows still reads its own local copy. llms.txt and llms-full.txt are unchanged, confirming .md files here were never indexed. Reverts the content of #1180 and #1181. The proportionality rules #1181 strengthened are preserved in the pipecat copy, verbatim apart from Prettier normalising one emphasis marker.
Contributor
|
🔍 Mintlify preview for this branch: https://daily-docs-retire-hosted-skill.mintlify.site |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Removes
automation/update-docs/. Reverts the content of #1180 and #1181, now that pipecat-ai/pipecat#5471 has merged and the shared skill lives there instead.Why the location moved
#1180 hosted the canonical skill here and left a pointer stub in pipecat. That was wrong in a way I only caught afterwards:
pipecat's.claude-plugin/marketplace.jsonsources skills from./.claude/skills/*, so the stub would have shipped through the marketplace — anyone updatingpipecat-devwould have got a skill whose entire content was a path into a repo they may not have checked out.pipecat is also the better home on the merits. Six of its ten published skills (
cleanup,code-review,pr-description,pr-submit,squash-commits,prose-review) are generic rather than pipecat-specific, so that marketplace is already the org's shared dev tooling — a Cloud or client developer wants the same plugin. And it's the repo that reliably gets maintained.Nothing is lost
The generic-skill / per-repo-profile split from #1180 is intact — it moved wholesale into pipecat, along with
PROFILE_CONTRACT.md. The proportionality rules #1181 strengthened are preserved verbatim, apart from Prettier having normalised*facts*to_facts_.Verified before deleting
automation/update-docs— pipecat's workflow,SKILL.md, andPROFILE_CONTRACT.mdare all clean of the hosted path, andpipecat-flowsreads its own local copyautomation/README.mditselfllms.txtandllms-full.txtregenerate byte-identical, confirming.mdfiles underautomation/were never indexedVerification
mint broken-links --check-anchors --check-redirectsclean,docs-meta-lint.mjs0 errors.Step A3 of the rollout plan. Next up is A5 — proving the merged skill end to end with a
workflow_dispatchrun — and A6, onboardingpipecat-cloud.🤖 Generated with Claude Code