diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 6d936b6..fe5b873 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -36,6 +36,8 @@ Found a bug or have an enhancement? Great! Please: - Describe the current vs desired behavior - Include examples if relevant - Keep changes focused and minimal +- Consider adding validation phases for complex commands +- Use extended thinking for sophisticated analysis scenarios ### 3. Bug Reports Open an issue with: @@ -73,6 +75,31 @@ Keep them simple: - [ ] Handles common edge cases - [ ] Clear, actionable output - [ ] Under 100 lines +- [ ] Includes validation phase for complex commands +- [ ] No emojis in git-related output + +## Advanced Command Features + +For complex commands, consider implementing: + +### Validation Phases +Commands like `/refactor` and `/implement` should include validation: +``` +/refactor validate # Check completeness, find loose ends +/implement validate # Verify integration completeness +``` + +### Extended Thinking +Use `` blocks for sophisticated analysis in: +- Complex architectural refactoring +- Security vulnerability detection +- Multi-step problem solving + +### Command Integration +Minimal, pragmatic suggestions for natural workflow: +- Suggest `/test` after major changes +- Suggest `/commit` at logical checkpoints +- Avoid over-engineering command chains ## What We're Looking For @@ -80,11 +107,14 @@ Keep them simple: - Commands that automate tedious tasks - Cross-platform compatibility improvements - Real-world workflow optimizations +- Validation phases for complex operations +- Pragmatic command integration **No:** - Framework-specific tools (unless very popular) - Commands requiring external dependencies - Overly complex multi-step wizards +- Over-engineered command orchestration ## Issue Templates @@ -115,9 +145,10 @@ When creating issues, please use these templates: ## Community Standards 1. **Professional Communication** - Clear, concise, technical -2. **No Emojis in Code** - Keep commands clean and professional +2. **No Emojis in Code** - Keep commands, commits, PRs, and issues clean and professional 3. **Respect Time** - Quick reviews, fast merges for good contributions 4. **Test Before Submit** - Ensure your command works on major platforms +5. **Clean Architecture** - Follow clean code principles, no over-engineering ## Continuous Improvement diff --git a/README.md b/README.md index 49fbc5c..ccce32e 100644 --- a/README.md +++ b/README.md @@ -35,7 +35,7 @@ Professional commands for Claude Code CLI that save 2-3 hours per week on repeti 🚧 **Active Development Notice**: CCPlugins is continuously evolving based on real-world usage. We thoroughly test each command and refine them as we discover gaps and opportunities. This ensures you're always getting battle-tested, production-ready tools that solve actual developer problems. -CCPlugins is a curated set of 25 professional commands that extend Claude Code CLI with enterprise-grade development workflows. These commands leverage Claude's contextual understanding while providing structured, predictable outcomes optimized for Opus 4 and Sonnet 4 models. +CCPlugins is a curated set of 24 professional commands that extend Claude Code CLI with enterprise-grade development workflows. These commands leverage Claude's contextual understanding while providing structured, predictable outcomes optimized for Opus 4 and Sonnet 4 models. ## Quick Links @@ -76,7 +76,7 @@ python uninstall.py ``` ## Commands -25 professional commands optimized for Claude Code CLI's native capabilities. +24 professional commands optimized for Claude Code CLI's native capabilities with enhanced validation and refinement phases. ### 🚀 Development Workflow @@ -86,15 +86,15 @@ python uninstall.py /format # Auto-detect and apply project formatter /scaffold feature-name # Generate complete features from patterns /test # Run tests with intelligent failure analysis -/implement url/path/feature # Import and adapt code from any source (GitHub, CodePen, local) -/refactor # Intelligent code restructuring with cross-session continuity +/implement url/path/feature # Import and adapt code from any source with validation phase +/refactor # Intelligent code restructuring with validation & de-para mapping ``` ### 🛡️ Code Quality & Security ```bash /review # Multi-agent analysis (security, performance, quality, architecture) -/security-scan # Contextual vulnerability analysis with dependency health +/security-scan # Vulnerability analysis with extended thinking & remediation tracking /predict-issues # Proactive problem detection with timeline estimates /remove-comments # Clean obvious comments, preserve valuable docs /fix-imports # Repair broken imports after refactoring @@ -123,6 +123,26 @@ python uninstall.py ``` +## Enhanced Features + +### 🔍 Validation & Refinement +Complex commands now include validation phases to ensure completeness: +```bash +/refactor validate # Find remaining old patterns, verify 100% migration +/implement validate # Check integration completeness, find loose ends +``` + +### 🧠 Extended Thinking +Advanced analysis for complex scenarios: +- **Refactoring**: Deep architectural analysis for large-scale changes +- **Security**: Sophisticated vulnerability detection with chain analysis + +### 🔗 Pragmatic Command Integration +Natural workflow suggestions without over-engineering: +- Suggests `/test` after major changes +- Recommends `/commit` at logical checkpoints +- Maintains user control, no automatic execution + ## Real World Example ### Before `/cleanproject`: diff --git a/commands/cleanproject.md b/commands/cleanproject.md index b5f2694..eea4dde 100644 --- a/commands/cleanproject.md +++ b/commands/cleanproject.md @@ -43,6 +43,7 @@ git commit -m "Pre-cleanup checkpoint" || echo "No changes to commit" - Include "Generated with Claude Code" or similar messages - Modify git config or user credentials - Add any AI/assistant attribution to the commit +- Use emojis in commits, PRs, or git-related content I'll identify cleanup targets using native tools: - **Glob tool** to find temporary and debug files diff --git a/commands/commit.md b/commands/commit.md index 771fc7f..15a3452 100644 --- a/commands/commit.md +++ b/commands/commit.md @@ -72,5 +72,6 @@ The commit message will be concise, meaningful, and follow your project's conven - Include "Generated with Claude Code" or similar messages - Modify git config or user credentials - Add any AI/assistant attribution to the commit +- Use emojis in commits, PRs, or git-related content The commit will use only your existing git user configuration, maintaining full ownership and authenticity of your commits. \ No newline at end of file diff --git a/commands/implement.md b/commands/implement.md index 27fe634..b3320d7 100644 --- a/commands/implement.md +++ b/commands/implement.md @@ -52,7 +52,37 @@ Based on my analysis, I'll create an implementation plan: - Design integration approach - Break work into testable chunks -I'll write this plan to `implement/plan.md` with clear checkpoints for continuity across sessions. +I'll write this plan to `implement/plan.md`: + +```markdown +# Implementation Plan - [timestamp] + +## Source Analysis +- **Source Type**: [URL/Local/Description] +- **Core Features**: [identified features to implement] +- **Dependencies**: [required libraries/frameworks] +- **Complexity**: [estimated effort] + +## Target Integration +- **Integration Points**: [where it connects] +- **Affected Files**: [files to modify/create] +- **Pattern Matching**: [how to adapt to project style] + +## Implementation Tasks +[Prioritized checklist with progress tracking] + +## Validation Checklist +- [ ] All features implemented +- [ ] Tests written and passing +- [ ] No broken functionality +- [ ] Documentation updated +- [ ] Integration points verified +- [ ] Performance acceptable + +## Risk Mitigation +- **Potential Issues**: [identified risks] +- **Rollback Strategy**: [git checkpoints] +``` ## Phase 3: Intelligent Adaptation @@ -137,6 +167,7 @@ When you return and run `/implement` or `/implement resume`: /implement # Auto-detects and resumes /implement resume # Explicit resume /implement status # Check progress +/implement validate # Validate integration completeness ``` ## Execution Guarantee @@ -148,12 +179,53 @@ When you return and run `/implement` or `/implement resume`: 3. **Write plan** - Full implementation plan in `implement/plan.md` 4. **Show plan** - Present summary before implementing 5. **Execute systematically** - Follow plan with updates +6. **Validate integration** - Run validation when requested **I will NEVER:** - Start implementing without a written plan - Skip source or project analysis - Bypass session file creation - Begin coding before showing the plan +- Use emojis in commits, PRs, or git-related content + +## Phase 6: Implementation Validation + +When you run `/implement validate` after implementation: + +**Integration Analysis:** +1. **Coverage Check** - Verify all planned features implemented +2. **Integration Points** - Validate all connections work +3. **Test Coverage** - Ensure new code is tested +4. **TODO Scan** - Find any leftover TODOs +5. **Documentation** - Check if docs reflect changes + +**Validation Report:** +``` +IMPLEMENTATION VALIDATION +├── Features Implemented: 12/12 (100%) +├── Integration Points: 8/10 (2 pending) +├── Test Coverage: 87% +├── Build Status: Passing +└── Documentation: Needs update + +PENDING ITEMS: +- API endpoint /users/profile not connected +- WebSocket integration incomplete +- Missing tests for error scenarios +- README needs feature documentation + +ENHANCEMENT OPPORTUNITIES: +1. Add error boundary for new components +2. Implement caching for API calls +3. Add performance monitoring +4. Create usage examples +``` + +**Validation Actions:** +- Complete missing integrations +- Generate comprehensive test suite +- Update all affected documentation +- Create migration guide if breaking changes ## Command Suggestions diff --git a/commands/refactor.md b/commands/refactor.md index 93114b5..68b9774 100644 --- a/commands/refactor.md +++ b/commands/refactor.md @@ -99,10 +99,34 @@ Based on analysis, I'll create a structured plan: **Plan Structure:** I'll create a detailed plan in `refactor/plan.md`: -- Prioritized refactoring tasks -- Risk assessment for each change -- Checkpoints for validation -- Rollback points if needed + +```markdown +# Refactor Plan - [timestamp] + +## Initial State Analysis +- **Current Architecture**: [description of existing patterns] +- **Problem Areas**: [specific issues found] +- **Dependencies**: [external/internal dependencies] +- **Test Coverage**: [current coverage %] + +## Refactoring Tasks +[Prioritized list with risk levels] + +## Validation Checklist +- [ ] All old patterns removed +- [ ] No broken imports +- [ ] All tests passing +- [ ] Build successful +- [ ] Type checking clean +- [ ] No orphaned code +- [ ] Documentation updated + +## De-Para Mapping +| Before | After | Status | +|--------|-------|--------| +| OldService.method() | NewService.method() | Pending | +| /api/v1/* | /api/v2/* | Pending | +``` ## Phase 3: Incremental Execution @@ -182,8 +206,46 @@ Continuing from checkpoint... /refactor resume # Continue existing session /refactor status # Check progress without continuing /refactor new # Start fresh (archives existing) +/refactor validate # Validate completeness and find loose ends ``` +## Phase 6: Validation & Refinement + +When you run `/refactor validate` after completing refactoring: + +**Deep Validation Analysis:** +1. **Coverage Check** - Find all remaining old patterns +2. **Import Verification** - Detect broken or orphaned imports +3. **Build & Test** - Run full build and test suite +4. **Type Checking** - Verify type safety if applicable +5. **Dead Code Detection** - Identify removable legacy code + +**De-Para Mapping:** +``` +MIGRATION STATUS REPORT +├── Patterns Migrated: 45/48 (94%) +├── Files Updated: 67/70 +├── Tests Status: 3 failing +└── Build Status: Passing + +PENDING MIGRATIONS: +- src/legacy/UserHelper.js → Still using old pattern +- api/v1/routes.js → Mixed patterns detected +- tests/old-api.test.js → Needs update + +SUGGESTED REFINEMENTS: +1. Remove 12 orphaned files +2. Consolidate duplicate utilities +3. Update 3 missed import paths +4. Optimize bundle size (-15KB possible) +``` + +**Validation Actions:** +- Generate comprehensive de-para documentation +- Create migration guide for team +- Fix remaining issues automatically +- Ensure 100% pattern consistency + ## Safety Guarantees **Protection Measures:** @@ -197,6 +259,7 @@ Continuing from checkpoint... - Modify git configuration - Break working functionality - Make changes without validation +- Use emojis in commits, PRs, or git-related content ## Command Integration @@ -210,9 +273,10 @@ When appropriate, I may suggest using other commands: 1. **Setup session** - Check/create state files FIRST 2. **Deep analysis** - Use extended thinking for complex scenarios -3. **Write plan** - Document all changes in `refactor_plan.md` +3. **Write plan** - Document all changes in `refactor/plan.md` 4. **Get confirmation** - Show plan summary before starting 5. **Execute incrementally** - Follow plan with checkpoints +6. **Validate completeness** - Run validation phase when requested **I will NEVER:** - Start refactoring without a written plan diff --git a/commands/review.md b/commands/review.md index 4643028..00fd496 100644 --- a/commands/review.md +++ b/commands/review.md @@ -39,5 +39,6 @@ After review, I'll ask: "Create GitHub issues for critical findings?" - Include "Generated with Claude Code" in any output - Modify git config or repository settings - Add any AI/assistant signatures or watermarks +- Use emojis in commits, PRs, issues, or git-related content This focuses on real problems that impact your application's reliability and maintainability. \ No newline at end of file diff --git a/commands/session-end.md b/commands/session-end.md index cffd9e2..0b4e63a 100644 --- a/commands/session-end.md +++ b/commands/session-end.md @@ -37,5 +37,6 @@ I'll update the appropriate CLAUDE.md file with: - Include "Generated with Claude Code" or similar messages - Modify git config or user credentials - Add any AI/assistant attribution to the commit +- Use emojis in commits, PRs, or git-related content I'll preserve this summary in your memory system, ensuring continuity for future sessions and seamless handoffs to team members. This integrates with Claude Code CLI's native memory management for persistent context. \ No newline at end of file diff --git a/commands/todos-to-issues.md b/commands/todos-to-issues.md index 8eeb8d0..2f927f5 100644 --- a/commands/todos-to-issues.md +++ b/commands/todos-to-issues.md @@ -146,5 +146,6 @@ I'll handle rate limits and show you a summary of all created issues. - Include "Generated with Claude Code" in issue descriptions - Modify repository settings or permissions - Add any AI/assistant signatures or watermarks +- Use emojis in issues, PRs, or git-related content This helps convert your development notes into trackable work items. \ No newline at end of file