✨ Update Claude plugin to v0.3.0 with full CLI support#205
Conversation
- Remove MCP references - the plugin is now purely skills-based - Update CLI commands to match current Vizzly CLI structure - Add review commands: approve, reject, comment - Add cloud build commands: status, builds, comparisons - Add account commands: login, logout, whoami, orgs, projects - Add project commands: project:select, project:list, project:token - Update setup instructions to use project:select workflow - Add CLI Integration section to README
This comment has been minimized.
This comment has been minimized.
PR Review: ✨ Update Claude plugin to v0.3.0 with full CLI supportOverall AssessmentThis is a well-structured update that significantly improves the Claude plugin by documenting all available CLI commands. The changes align well with the current CLI codebase (v0.27.0) and provide comprehensive coverage of new functionality. ✅ Strengths1. Accurate CLI Command DocumentationAll commands documented in the plugin match the actual CLI implementation:
2. Improved Authentication FlowThe setup instructions correctly reflect the modern workflow:
3. Comprehensive CoverageThe
4. Consistent Messaging
🔍 Issues & Suggestions1. Command Syntax Inconsistency (Minor)In vizzly approve <comparison-id>
vizzly reject <comparison-id>
vizzly comment <build-id> "msg"However, the actual CLI requires:
Suggested fix: vizzly approve <comparison-id> # Approve a comparison
vizzly reject <comparison-id> --reason "explanation" # Reject with reason (required)
vizzly comment <build-id> "message" # Add comment to build2. Missing Command Context (Enhancement)The **For local TDD:**
vizzly tdd status # Show current test status
vizzly baselines # List and query baselines
**For cloud builds:**
vizzly status <build-id> # Check specific build status (requires build ID from CI)
vizzly builds # List recent builds
vizzly comparisons # Query and search comparisons3. Incomplete Command Examples (Minor)In Consider showing how users find the comparison ID: 4. Test Plan Incomplete (Process)The PR test plan has unchecked items. Before merging, please verify:
🔒 Security ConsiderationsNo security concerns. The plugin only documents CLI commands without adding new execution paths or handling sensitive data differently. 🎯 Test CoverageNo test changes needed - this is purely documentation. However, consider adding integration tests in the future to verify plugin documentation stays in sync with CLI commands. 📝 Suggestions for Future PRs
✨ RecommendationApprove with minor fixes This PR significantly improves the Claude plugin's usefulness by documenting the full CLI surface area. The only required changes are fixing the command syntax inconsistencies (#1 above). The other suggestions are enhancements that can be addressed in follow-up PRs. Great work modernizing the authentication flow and removing MCP complexity! Files to update before merge:
|
- Fix reject command syntax: -r/--reason flag is required - Add context for when to use local vs cloud commands - Show how to find comparison IDs with vizzly comparisons -b <build-id> - Use short flags (-r, -m, -b) matching CLI conventions - Add comparisons filtering examples (--name, --status)
1c10592 to
2d724e0
Compare
This comment has been minimized.
This comment has been minimized.
Vizzly - Visual Test ResultsCLI Reporter - 8 changes need review
Changes needing review (8)fullscreen-viewer · Firefox · 1920×1080 · 0.1% diff bulk-accept-dialog · Firefox · 1920×1080 · 0.5% diff viewer-slide-mode · Firefox · 1920×1080 · 0.5% diff filter-failed-only · Firefox · 375×704 · 14.7% diff viewer-zoomed-100 · Firefox · 1920×1080 · 0.4% diff fullscreen-viewer · Firefox · 375×667 · 188.3% diff ...and 2 more in Vizzly.
|






Summary
vizzly approve,vizzly reject,vizzly commentvizzly status,vizzly builds,vizzly comparisonsvizzly login,vizzly logout,vizzly whoami,vizzly orgs,vizzly projectsvizzly project:select,vizzly project:list,vizzly project:token,vizzly project:removevizzly project:selectworkflow instead of deprecated commandsTest plan
claude --plugin-dir ./vizzly-plugin/plugins/vizzlyvizzly --helpoutput