docs: fix outdated testing section, broken publishing link, agent-team input table#138
Draft
github-actions[bot] wants to merge 1 commit into
Draft
Conversation
…m input table Three documentation gaps addressed: - CONTRIBUTING.md: replace the incorrect "no automated test harness" claim with the actual tier-1/tier-2/tier-3 structure from tests/README.md, including a quick-reference table mapping changed areas to test commands. - README.md: add missing #publishing section (CONTRIBUTING.md linked to it but the anchor didn't exist). - catalog/agent-team/README.md: expand Files table with required/optional input columns; document that pr_number is intentionally optional on first impl (blank → new branch, real value → push to existing PR); improve manual-dispatch examples in "Kicking off a task" step 4; sharpen the Input propagation gotcha to distinguish required vs optional inputs. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.
Three documentation gaps addressed by this PR, all identified during the post-commit docs sweep triggered by d688265.
Changes
CONTRIBUTING.md— Testing section rewriteThe previous version stated "There is no automated test harness for skills", which has been incorrect since the
tests/directory was added. This misleads contributors into thinking the only verification path is a manual skill walkthrough.Replaced with the actual tier structure (tier-2 grep invariants <1s, tier-1 Claude-invoked assertions ~4-5min, tier-3 manual E2E) and a quick-reference table mapping each changed area to the correct test command.
README.md— Add missing#publishingsectionCONTRIBUTING.mdline 115 links toREADME.md#publishingbut that anchor did not exist. Added a brief maintainer-only publishing section covering version bumping, GitHub Release creation, and registry submission steps.catalog/agent-team/README.md— Input table and manual-dispatch clarityTriggered by the recent fix to agent-team
workflow_dispatchinputs (#69):pr_numberis the one intentionally optional input (blank → new PR branch; real value → push to existing PR).gh workflow run --fieldexamples for planner, implementer, and reviewer.Testing
No code changes — documentation only. Verified by reading the modified files against the source-of-truth (
implementer-agent.mdfrontmatter,tests/run-tests.sh,tests/README.md).🤖 Generated with [Claude Code]((claude.com/redacted)