feat(tool-routing): simplify test-claude to single command #27
Conversation
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
…very Update documentation to reflect the new manifest-driven route discovery: - Replace CLAUDE_PLUGINS_DIR with CLAUDE_PROJECT_ROOT - Document exact projectPath matching for local-scoped plugins - Add troubleshooting for "fewer sources than expected" issue - Update environment variables table The old filesystem-based discovery using CLAUDE_PLUGINS_DIR no longer applies. Discovery now uses `claude plugin list --json` which filters local-scoped plugins by exact projectPath match. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add Quick Validation section using test-claude for reliable, isolated testing that doesn't affect user's plugin state. This approach: - Installs all plugins with routes from local repo - Uses isolated CLAUDE_CONFIG_DIR - Avoids project path matching issues - Tests the actual discovery path Claude Code uses Also reorganized Step 3 into three options: - Option A: test-claude (recommended) - Option B: TOOL_ROUTING_ROUTES (explicit, no discovery) - Option C: user's enabled plugins (variable results) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Collapse the 6-step test-claude workflow into a single command: Before: ./bin/test-claude --setup ./bin/test-claude --install tool-routing ./bin/test-claude --install git ./bin/test-claude --install ci-cd-tools ./bin/test-claude --install dev-tools ./bin/test-claude After: ./bin/test-claude The script now: - Auto-detects all plugins with .claude-plugin/ directories - Sets up config and installs everything on first run - Skips setup on subsequent runs (config already exists) - Only flag is --clean to remove test config For validation without interactive session, use: ./bin/test-claude -p "" Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
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.
Summary
test-claudefrom 6 commands to 1.claude-plugin/directories--setup,--install,--status,--resetflags (only--cleanremains)Before
After
./bin/test-claude # setup + install all + run claudeFor validation without interactive session:
Test plan
./bin/test-claude --helpshows simplified usage./bin/test-claude --cleanremoves test config./bin/test-claude -p ""sets up config and exitsCLAUDE_CONFIG_DIR=... tool-routing listshows 5 sourcesCLAUDE_CONFIG_DIR=... tool-routing testpasses (32 tests)🤖 Generated with Claude Code