docs: fix stale Testing section and broken Publishing link in CONTRIBUTING#66
Draft
github-actions[bot] wants to merge 1 commit intomainfrom
Draft
Conversation
…UTING Two gaps surfaced by an update-docs pass: - Testing section said "There is no automated test harness for skills" but the repo ships a three-tier suite (tests/run-tests.sh) that CI runs on every PR. Replaced with the actual run commands, tier table, and a link to tests/README.md for full details. - Publishing section linked to README.md#publishing, an anchor that doesn't exist in the README. Replaced the dead link with self-contained steps: version bump in plugin.json, GitHub Release, and registry updates on claude-plugins.dev and ClaudePluginHub. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
github-actions
Bot
added
automation
documentation
This was referenced Apr 22, 2026
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
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.
Two documentation gaps found during an
update-docspass onCONTRIBUTING.md.What changed
Testing section (lines 75–99)
The previous text opened with "There is no automated test harness for skills" — accurate when the file was first written, but the repo now ships a full three-tier test suite (
tests/run-tests.sh) that CI executes on every PR.Replaced with:
./tests/run-tests.sh,--verboseflag)tests/README.mdtests/README.mdfor full detailsPublishing section (lines 128–136)
The previous text was a single sentence: "See the Publishing section of the README for the steps..." — but
README.mdhas no## Publishingsection, making this a broken cross-reference.Replaced with self-contained steps:
.claude-plugin/plugin.jsonv<version>(marketplace JSON referencesmain, so installations auto-update)Why no other files changed
README.mdandtests/README.mdare accurate. The skill files (SKILL.md) are intentionally stable instruction sets; their "Out of scope" lists describe what the skill doesn't do, which is still correct. No other doc drift was found.Testing
No code changes — documentation only. Verify by reading the rendered Markdown on this PR.