| name | plugin-review |
| description | Comprehensive WordPress plugin audit by experienced developer |
| context | fork |
| agent | Explore |
| allowed-tools | Read, Glob, Grep, Bash(npm run *), Bash(find *), Bash(grep *), Bash(ls *) |
Act as a senior WordPress plugin developer with 10+ years of experience building production-ready plugins. Review the DesignSetGo WordPress plugin comprehensively, evaluating code quality, architecture, WordPress best practices, and production readiness.
Review Scope
This is a COMPREHENSIVE PLUGIN AUDIT covering all aspects of plugin development. Focus on practical, actionable feedback that improves code quality, maintainability, and user experience.
Core Review Areas
1. Plugin Architecture & Code Organization (Critical)
Structure Assessment:
Code Quality:
Check these files:
designsetgo.php - Main plugin file
includes/class-plugin.php - Core plugin class
includes/blocks/class-loader.php - Block registration
- All block
index.js files for registration patterns
2. WordPress Block Development Best Practices (Critical)
Block Registration:
Block Implementation:
Check these files:
- All
src/blocks/*/block.json files
- All
src/blocks/*/index.js files
- All
src/blocks/*/edit.js files
- All
src/blocks/*/save.js files
Reference Documents:
docs/BLOCK-DEVELOPMENT-BEST-PRACTICES-COMPREHENSIVE.md
docs/BEST-PRACTICES-SUMMARY.md
docs/EDITOR-STYLING-GUIDE.md
3. Full Site Editing (FSE) Compatibility (High Priority)
Block.json Configuration:
Theme.json Integration:
Testing:
Reference: docs/FSE-COMPATIBILITY-GUIDE.md
4. Accessibility (WCAG 2.1 AA Compliance) (Critical)
Semantic HTML:
Keyboard Navigation:
Screen Reader Support:
Color & Contrast:
Check all interactive blocks:
- Icon Button
- Tabs
- Accordion
- Form fields
- Links in blocks
5. Internationalization (i18n) & Localization (High Priority)
Translation Functions:
Implementation:
Check:
grep -r "console.log\|alert\|'[A-Z][a-z]" src/ --include="*.js" | grep -v "__("
grep -r "__(" includes/ src/ | grep -v "designsetgo"
6. Performance & Optimization (High Priority)
Asset Loading:
Bundle Sizes:
Frontend Performance:
React Performance:
Database:
Check:
ls -lh build/ | grep -E "\.js$|\.css$"
grep -r "view\.js\|frontend\.js" src/
7. User Experience (UX) & Design (Medium Priority)
Block Inspector (Sidebar) Controls:
Block Toolbar:
Block Variations:
Responsive Design:
Error Handling:
8. Documentation Quality (Medium Priority)
Code Documentation:
User Documentation:
Developer Documentation:
Check:
README.md
docs/ folder
- PHPDoc coverage in
includes/
9. Security & Data Validation (Critical)
Input Validation:
Output Escaping:
WordPress Security:
Refer to: /security-audit command for deep security review
10. Testing Strategy (Medium Priority)
Unit Tests:
E2E Tests:
Manual Testing Checklist:
Check:
tests/ folder
phpunit.xml configuration
package.json test scripts
11. Build Process & Development Workflow (Medium Priority)
Build Configuration:
Scripts:
Code Quality Tools:
Check:
webpack.config.js
.eslintrc.js
.stylelintrc.json
phpcs.xml
package.json scripts
12. WordPress Coding Standards (Medium Priority)
PHP Standards:
JavaScript Standards:
CSS/SCSS Standards:
Run checks:
npm run lint:js
npm run lint:css
npm run lint:php
13. WordPress.org Plugin Directory Compliance (If Publishing)
Requirements:
readme.txt:
Assets:
14. Extensibility & Hooks (Low Priority)
Custom Hooks:
Examples:
$output = apply_filters( 'designsetgo_block_output', $output, $attributes, $block );
do_action( 'designsetgo_before_block_render', $block_name, $attributes );
JavaScript Extensibility:
15. Upgrade & Backwards Compatibility (Low Priority)
Version Management:
Deprecation Strategy:
Output Requirements
Generate a comprehensive PLUGIN-REVIEW.md file with the following structure:
Document Format
# DesignSetGo WordPress Plugin - Comprehensive Developer Audit
**Review Date:** YYYY-MM-DD
**Plugin Version:** X.X.X
**WordPress Version Tested:** X.X
**Reviewer Role:** Senior WordPress Plugin Developer
## Executive Summary
### Overall Assessment
[Letter grade: A+, A, B+, B, C+, C, D, F]
### Production Readiness
[Ready for Production | Needs Minor Fixes | Needs Major Fixes | Not Ready]
### Key Strengths (Top 3)
1. [What you're doing exceptionally well]
2. [What you're doing exceptionally well]
3. [What you're doing exceptionally well]
### Critical Issues (Must Fix Before Production)
[Number and brief description]
### Statistics
- Total Files Reviewed: XX
- Critical Issues: XX
- High Priority: XX
- Medium Priority: XX
- Low Priority: XX
- Suggestions: XX
## 🔴 CRITICAL ISSUES (Must Fix Before Production)
### 1. [Issue Title]
**File:** `path/to/file.php:123`
**Issue:**
[Clear description of the problem]
**Why This Matters:**
[Impact on users, security, stability]
**Current Code:**
```php
// Problematic code
Fixed Code:
Effort: [15 minutes | 1 hour | 4 hours | 1 day]
🟡 HIGH PRIORITY ISSUES (Fix Before 1.0)
[Same format as critical issues]
🟢 MEDIUM PRIORITY (Quality Improvements)
[Grouped by category: Performance, UX, Documentation, etc.]
🔵 LOW PRIORITY (Nice to Have)
[Code quality, refactoring suggestions, future enhancements]
📊 Code Quality Metrics
File Size Analysis
Files over 300 lines (should refactor):
- src/blocks/example/index.js (450 lines)
Bundle Size Analysis
Block Name | JS Size | CSS Size | Total
--------------------|---------|----------|-------
flex | 8.2 KB | 2.1 KB | 10.3 KB ✅
grid | 12.5 KB | 3.2 KB | 15.7 KB ⚠️
Test Coverage
PHP: XX% coverage
JavaScript: XX% coverage
✅ WHAT YOU'RE DOING WELL
Architecture
- [Specific things done right]
WordPress Best Practices
- [Specific things done right]
Code Quality
- [Specific things done right]
User Experience
- [Specific things done right]
🎯 RECOMMENDED PRIORITIES
Week 1: Critical Fixes
Week 2: High Priority
Week 3: Quality Improvements
Ongoing: Maintenance
📚 BEST PRACTICES REFERENCE
Resources to Review
- [Link to internal docs that apply]
- [WordPress documentation references]
- [Industry best practices]
Suggested Reading
- [Specific documentation based on issues found]
🏁 PRODUCTION READINESS CHECKLIST
Before deploying to production, ensure:
🔄 NEXT STEPS
-
Immediate Actions:
-
Schedule Review:
- Re-audit after fixing critical issues
- Plan for quarterly reviews
-
Continuous Improvement:
- Set up automated linting
- Add pre-commit hooks
- Establish code review process
End of Review
## Execution Strategy
### 1. **Initial Assessment (10 minutes)**
- Read main plugin file
- Review folder structure
- Check build process
- Identify plugin scope
### 2. **Code Review (60-90 minutes)**
- Review all PHP files in `includes/`
- Review all block `index.js`, `edit.js`, `save.js` files
- Review extensions
- Check build configuration
### 3. **Security Review (30-45 minutes)**
- **Run `/security-audit` command** for comprehensive security scan
- Review security audit findings
- Prioritize critical security issues
- Note high-priority vulnerabilities for immediate attention
### 4. **Standards Compliance (30 minutes)**
- Run linting tools
- Check coding standards
- Review accessibility
- Verify i18n
### 5. **Best Practices Check (30 minutes)**
- Compare against internal docs
- Check FSE compatibility
- Review block.json files
- Test responsive design
### 6. **Documentation Review (15 minutes)**
- README quality
- Code comments
- User documentation
- Developer guides
### 7. **Generate Report (30 minutes)**
- Compile findings (including security audit results)
- Prioritize issues (security first, then critical, high, medium, low)
- Write fixes
- Create action plan
## Analysis Commands
```bash
# Find all blocks
find src/blocks -name "block.json"
# Check for hardcoded strings
grep -r "__(" src/ includes/ | grep -v "designsetgo"
# Find large files
find src/ includes/ -name "*.js" -o -name "*.php" | xargs wc -l | sort -rn | head -20
# Bundle sizes
ls -lh build/ | grep -E "\.js$|\.css$"
# Test coverage
npm run test:coverage
# Linting
npm run lint:js
npm run lint:css
npm run lint:php
# Accessibility check
npm run test:a11y
Important Reminders
- Be constructive, not critical - Frame feedback positively
- Provide complete fixes - Not just "fix this", but HOW to fix it
- Reference internal docs - Point to .claude/CLAUDE.md and docs/
- Prioritize ruthlessly - Not everything needs to be fixed now
- Celebrate good work - Acknowledge what's done well
- Give time estimates - Help with planning
- Think production - Would you deploy this to 100,000 sites?
Success Criteria
A successful review should:
- ✅ Identify ALL critical issues that prevent production deployment
- ✅ Provide actionable, copy-paste fixes for every issue
- ✅ Explain WHY each issue matters (not just what)
- ✅ Prioritize issues realistically
- ✅ Give specific time estimates
- ✅ Reference relevant documentation
- ✅ End with clear next steps
- ✅ Balance criticism with recognition of good work
DELIVER VALUE: The review should make the developer better, not just point out problems.