docs: fix false 'no test harness' claim and add missing Publishing section#102
Draft
github-actions[bot] wants to merge 1 commit into
Draft
Conversation
…ction
CONTRIBUTING.md stated 'There is no automated test harness for skills'
which directly contradicts the three-tier test suite in tests/. Replace
that paragraph with an accurate description of tiers 1–3, the run-tests.sh
entry point, and CI coverage.
README.md was missing the Publishing section that CONTRIBUTING.md
references ('See the Publishing section of the README'). Add a minimal
maintainer-focused section covering version bumps, tagging, and the
self-hosted marketplace approach.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
github-actions
Bot
added
automation
documentation
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.
What changed
CONTRIBUTING.md— Testing section rewrittenThe previous text stated:
This is factually wrong. The repo ships a three-tier test suite under
tests/(run-tests.sh,test-invariants.sh, three tier-1 skill tests, and tier-3 E2E scripts). The false statement would mislead contributors into skipping the test suite entirely.The replacement section describes:
./tests/run-tests.shentry point and common flagsgh aw validate, OAuth grep counts) preserved from the originalREADME.md— Publishing section addedCONTRIBUTING.mdends with:That anchor (
#publishing) did not exist inREADME.md, making it a broken cross-reference. The new section documents the four maintainer steps: version bump, tagging/release, the self-hosted marketplace approach, and directory listing updates.Why no other files changed
The catalog agent files, skill files, and lock files are correct as-is. The
fix(agent-team): fail loud on missing dispatch inputsbehavior introduced in the parent PR (#69) is already documented incatalog/agent-team/README.md(Limits and gotchas → "Input propagation") and in each agent's "Required input contract" section.Testing
grep -n "no automated"inCONTRIBUTING.mdno longer matchesgrep -n "Publishing"inREADME.mdreturns the new section headergh aw validatenot needed