-
.claude/commands/gh-milestones/test-plan.mdcreated -
.claude/commands/gh-milestones/README.mdcreated/updated -
docs/dev/testing/HIGH_LEVEL_TESTING_GUIDELINES.mdupdated - Files have correct frontmatter (description, argument-hint)
- Command specification is complete
- All 6 steps are documented
- Progressive disclosure pattern is enforced
- Example output is provided
- Testing documentation references are correct
After this PR is merged and a new Claude Code session starts:
# Verify command appears in available commands
/help
# Should show: /gh-milestones:test-plan# Test with milestone name
/gh-milestones:test-plan ls-test-improvement
# Test with milestone number
/gh-milestones:test-plan 14The command should:
- Load
@docs/dev/testing/README.mdfirst - Fetch milestone information via
gh api - Identify milestone type correctly
- Load only relevant testing docs (2-4 docs, not all 8)
- Generate test plan at
docs/implementation/<milestone>-test-plan.md - Include all required sections:
- Test Strategy Overview
- Unit Tests
- Integration Tests
- Test Data & Fixtures
- Success Criteria
- References
- Keep context usage under 5000 tokens
- Provide actionable checklist
- Include Toyota andon cord reminder
Test with different milestone types:
- SDK feature milestone
- CLI feature milestone
- Infrastructure milestone
- Documentation milestone
Monitor which testing docs get loaded:
- Should NOT load all 8 docs from
docs/dev/testing/ - Should load HIGH_LEVEL_TESTING_GUIDELINES.md always
- Should load 1-3 additional relevant docs based on feature type
Review generated test plan document:
- Contains concrete, specific test cases
- References actual codebase patterns
- Follows CRUD lifecycle pattern where applicable
- Includes cleanup strategies
- Specifies environment variables needed
- References Toyota andon cord principle
All checks must pass:
- Command file exists with correct structure
- README documents the command
- HIGH_LEVEL_TESTING_GUIDELINES.md references command
- Post-merge: Command invocable
- Post-merge: Generated test plans follow standards
- Post-merge: Context usage <5000 tokens verified
- Slash commands are loaded at CLI startup, not dynamically
- Testing must occur after PR merge in a new Claude Code session
- Context window monitoring is critical for progressive disclosure
- Generated test plans should be reviewed for quality and specificity