[{"data":1,"prerenderedAt":-1},["ShallowReactive",2],{"skill:01M6ESPPEPPR2BBCCYJ0AMVKJ1":3,"skill-same:01M6ESPPEPPR2BBCCYJ0AMVKJ1":82},{"skillNo":4,"name":5,"nameZh":6,"description":7,"descriptionZh":6,"repositoryFullName":8,"ownerLogin":9,"ownerAvatarUrl":10,"ownerAccountNo":11,"ownerDisplayName":12,"repositoryNo":13,"starsCount":14,"licenseSpdxId":15,"licenseFileUrl":6,"sourceHtmlUrl":16,"categories":17,"keywords":31,"publishedAt":48,"linkOnly":49,"markdownContent":50,"markdownBody":51,"frontMatter":52,"refName":53,"commitSha":54,"skillDirectory":55,"forksCount":56,"repositoryHtmlUrl":57,"repositoryHomepageUrl":58,"ownerHtmlUrl":59,"ownerBlogUrl":6,"ownerDescription":60,"updatedAt":61,"install":62,"resources":75},"01M6ESPPEPPR2BBCCYJ0AMVKJ1","sop-code-review",null,"Comprehensive code review workflow coordinating quality, security, performance, and documentation reviewers. 4-hour timeline for thorough multi-agent review.","ComeOnOliver\u002Fskillshub","ComeOnOliver","https:\u002F\u002Favatars.githubusercontent.com\u002Fu\u002F43215937?v=4","01M6E5JNFECJAAVFE0W7SMWWD2","Haoran Shu","01M6E5JNNDQS7AJZDYFQ8TPZCK",66,"MIT","https:\u002F\u002Fgithub.com\u002FComeOnOliver\u002Fskillshub\u002Fblob\u002Fmain\u002Fskills\u002Faiskillstore\u002Fmarketplace\u002Fdnyoussef\u002Fsop-code-review\u002FSKILL.md",[18,23,26],{"categoryNo":19,"categoryName":20,"taxonomyType":21,"taxonomyTypeDesc":22},"01JBSEEDCAT000000000000007","代码审查","SCENARIO","使用场景",{"categoryNo":24,"categoryName":25,"taxonomyType":21,"taxonomyTypeDesc":22},"01JBSEEDCAT000000000000023","Skill与智能体开发",{"categoryNo":27,"categoryName":28,"taxonomyType":29,"taxonomyTypeDesc":30},"01JBSEEDCAT000000000000030","开发工程师","PROFESSION","适用职业",[32,33,34,35,36,37,38,39,40,41,42,43,44,45,46,47],"code review","review","code","sop","sop code review","security","performance","workflow","documentation","timeline","multi-agent","quality","agent review","code review workflow","multi","comprehensive code review","2026-11-10 10:12:30",false,"---\nname: sop-code-review\ndescription: Comprehensive code review workflow coordinating quality, security, performance, and documentation reviewers. 4-hour timeline for thorough multi-agent review.\n---\n\n# SOP: Code Review Workflow\n\nComprehensive code review using specialized reviewers for different quality aspects.\n\n## Timeline: 4 Hours\n\n**Phases**:\n1. Automated Checks (30 min)\n2. Specialized Reviews (2 hours)\n3. Integration Review (1 hour)\n4. Final Approval (30 min)\n\n---\n\n## Phase 1: Automated Checks (30 minutes)\n\n### Quick Quality Checks\n\n**Parallel Automated Testing**:\n\n```javascript\n\u002F\u002F Initialize review swarm\nawait mcp__ruv-swarm__swarm_init({\n  topology: 'star',  \u002F\u002F Coordinator pattern for reviews\n  maxAgents: 6,\n  strategy: 'specialized'\n});\n\n\u002F\u002F Run all automated checks in parallel\nconst [lint, tests, coverage, build] = await Promise.all([\n  Task(\"Linter\", `\nRun linting checks:\n- ESLint for JavaScript\u002FTypeScript\n- Pylint for Python\n- RuboCop for Ruby\n- Check for code style violations\n\nStore results: code-review\u002F${prId}\u002Flint-results\n`, \"reviewer\"),\n\n  Task(\"Test Runner\", `\nRun test suite:\n- Unit tests\n- Integration tests\n- E2E tests (if applicable)\n- All tests must pass\n\nStore results: code-review\u002F${prId}\u002Ftest-results\n`, \"tester\"),\n\n  Task(\"Coverage Analyzer\", `\nCheck code coverage:\n- Overall coverage > 80%\n- New code coverage > 90%\n- No critical paths uncovered\n\nGenerate coverage report\nStore: code-review\u002F${prId}\u002Fcoverage-report\n`, \"reviewer\"),\n\n  Task(\"Build Validator\", `\nValidate build:\n- Clean build (no warnings)\n- Type checking passes\n- No broken dependencies\n- Bundle size within limits\n\nStore build results: code-review\u002F${prId}\u002Fbuild-status\n`, \"reviewer\")\n]);\n\n\u002F\u002F If any automated check fails, stop and request fixes\nif (hasFailures([lint, tests, coverage, build])) {\n  await Task(\"Review Coordinator\", `\nAutomated checks failed. Request fixes from author:\n${summarizeFailures([lint, tests, coverage, build])}\n\nStore feedback: code-review\u002F${prId}\u002Fautomated-feedback\n`, \"pr-manager\");\n  return; \u002F\u002F Stop review until fixed\n}\n```\n\n**Deliverables**:\n- All automated checks passing\n- Test results documented\n- Coverage report generated\n\n---\n\n## Phase 2: Specialized Reviews (2 hours)\n\n### Parallel Expert Reviews\n\n**Sequential coordination of parallel reviews**:\n\n```javascript\n\u002F\u002F Spawn specialized reviewers in parallel\nconst [codeQuality, security, performance, architecture, docs] = await Promise.all([\n  Task(\"Code Quality Reviewer\", `\nReview for code quality:\n\n**Readability**:\n- Clear, descriptive names (variables, functions, classes)\n- Appropriate function\u002Fmethod length (\u003C 50 lines)\n- Logical code organization\n- Minimal cognitive complexity\n\n**Maintainability**:\n- DRY principle (no code duplication)\n- SOLID principles followed\n- Clear separation of concerns\n- Proper error handling\n\n**Best Practices**:\n- Following language idioms\n- Proper use of design patterns\n- Appropriate comments (why, not what)\n- No code smells (magic numbers, long parameter lists)\n\nStore review: code-review\u002F${prId}\u002Fquality-review\nRating: 1-5 stars\n`, \"code-analyzer\"),\n\n  Task(\"Security Reviewer\", `\nReview for security issues:\n\n**Authentication & Authorization**:\n- Proper authentication checks\n- Correct authorization rules\n- No privilege escalation risks\n- Secure session management\n\n**Data Security**:\n- Input validation (prevent injection attacks)\n- Output encoding (prevent XSS)\n- Sensitive data encryption\n- No hardcoded secrets or credentials\n\n**Common Vulnerabilities** (OWASP Top 10):\n- SQL Injection prevention\n- XSS prevention\n- CSRF protection\n- Secure dependencies (no known vulnerabilities)\n\nStore review: code-review\u002F${prId}\u002Fsecurity-review\nSeverity: Critical\u002FHigh\u002FMedium\u002FLow for each finding\n`, \"security-manager\"),\n\n  Task(\"Performance Reviewer\", `\nReview for performance issues:\n\n**Algorithmic Efficiency**:\n- Appropriate time complexity (no unnecessary O(n²))\n- Efficient data structures chosen\n- No unnecessary iterations\n- Lazy loading where appropriate\n\n**Resource Usage**:\n- No memory leaks\n- Proper cleanup (connections, files, timers)\n- Efficient database queries (avoid N+1)\n- Batch operations where possible\n\n**Optimization Opportunities**:\n- Caching potential\n- Parallelization opportunities\n- Database index needs\n- API call optimization\n\nStore review: code-review\u002F${prId}\u002Fperformance-review\nImpact: High\u002FMedium\u002FLow for each finding\n`, \"perf-analyzer\"),\n\n  Task(\"Architecture Reviewer\", `\nReview for architectural consistency:\n\n**Design Patterns**:\n- Follows established patterns in codebase\n- Appropriate abstraction level\n- Proper dependency injection\n- Clean architecture principles\n\n**Integration**:\n- Fits well with existing code\n- No unexpected side effects\n- Backward compatibility maintained\n- API contracts respected\n\n**Scalability**:\n- Design supports future growth\n- No hardcoded limits\n- Stateless where possible\n- Horizontally scalable\n\nStore review: code-review\u002F${prId}\u002Farchitecture-review\nConcerns: Blocker\u002FMajor\u002FMinor for each finding\n`, \"system-architect\"),\n\n  Task(\"Documentation Reviewer\", `\nReview documentation:\n\n**Code Documentation**:\n- Public APIs documented (JSDoc\u002Fdocstring)\n- Complex logic explained\n- Non-obvious behavior noted\n- Examples provided where helpful\n\n**External Documentation**:\n- README updated (if needed)\n- API docs updated (if API changed)\n- Migration guide (if breaking changes)\n- Changelog updated\n\n**Tests as Documentation**:\n- Test names are descriptive\n- Test coverage demonstrates usage\n- Edge cases documented in tests\n\nStore review: code-review\u002F${prId}\u002Fdocs-review\nCompleteness: 0-100%\n`, \"api-docs\")\n]);\n\n\u002F\u002F Aggregate all reviews\nawait Task(\"Review Aggregator\", `\nAggregate specialized reviews:\n- Quality: ${codeQuality}\n- Security: ${security}\n- Performance: ${performance}\n- Architecture: ${architecture}\n- Documentation: ${docs}\n\nIdentify:\n- Blocking issues (must fix before merge)\n- High-priority suggestions\n- Nice-to-have improvements\n\nGenerate summary\nStore: code-review\u002F${prId}\u002Faggregated-review\n`, \"reviewer\");\n```\n\n**Deliverables**:\n- 5 specialized reviews completed\n- Issues categorized by severity\n- Aggregated review summary\n\n---\n\n## Phase 3: Integration Review (1 hour)\n\n### End-to-End Impact Assessment\n\n**Sequential Analysis**:\n\n```javascript\n\u002F\u002F Step 1: Integration Testing\nawait Task(\"Integration Tester\", `\nTest integration with existing system:\n- Does this change break any existing functionality?\n- Are all integration tests passing?\n- Does it play well with related modules?\n- Any unexpected side effects?\n\nRun integration test suite\nStore results: code-review\u002F${prId}\u002Fintegration-tests\n`, \"tester\");\n\n\u002F\u002F Step 2: Deployment Impact\nawait Task(\"DevOps Reviewer\", `\nAssess deployment impact:\n- Infrastructure changes needed?\n- Database migrations required?\n- Configuration updates needed?\n- Backward compatibility maintained?\n- Rollback plan clear?\n\nStore assessment: code-review\u002F${prId}\u002Fdeployment-impact\n`, \"cicd-engineer\");\n\n\u002F\u002F Step 3: User Impact\nawait Task(\"Product Reviewer\", `\nAssess user impact:\n- Does this change improve user experience?\n- Are there any user-facing changes?\n- Is UX\u002FUI consistent with design system?\n- Are analytics\u002Ftracking updated?\n\nStore assessment: code-review\u002F${prId}\u002Fuser-impact\n`, \"planner\");\n\n\u002F\u002F Step 4: Risk Assessment\nawait Task(\"Risk Analyzer\", `\nOverall risk assessment:\n- What's the blast radius of this change?\n- What's the worst-case failure scenario?\n- Do we have rollback procedures?\n- Should this be feature-flagged?\n- Monitoring and alerting adequate?\n\nStore risk assessment: code-review\u002F${prId}\u002Frisk-analysis\nRecommendation: Approve\u002FConditional\u002FReject\n`, \"reviewer\");\n```\n\n**Deliverables**:\n- Integration test results\n- Deployment impact assessment\n- User impact assessment\n- Risk analysis\n\n---\n\n## Phase 4: Final Approval (30 minutes)\n\n### Review Summary & Decision\n\n**Sequential Finalization**:\n\n```javascript\n\u002F\u002F Step 1: Generate Final Summary\nawait Task(\"Review Coordinator\", `\nGenerate final review summary:\n\n**Automated Checks**: ✅ All passing\n**Quality Review**: ${qualityScore}\u002F5\n**Security Review**: ${securityIssues} issues (${criticalCount} critical)\n**Performance Review**: ${perfIssues} issues (${highImpactCount} high-impact)\n**Architecture Review**: ${archConcerns} concerns (${blockerCount} blockers)\n**Documentation Review**: ${docsCompleteness}% complete\n**Integration Tests**: ${integrationStatus}\n**Deployment Impact**: ${deploymentImpact}\n**User Impact**: ${userImpact}\n**Risk Level**: ${riskLevel}\n\n**Blocking Issues**:\n${listBlockingIssues()}\n\n**Recommendations**:\n${generateRecommendations()}\n\n**Overall Decision**: ${decision} (Approve\u002FRequest Changes\u002FReject)\n\nStore final summary: code-review\u002F${prId}\u002Ffinal-summary\n`, \"pr-manager\");\n\n\u002F\u002F Step 2: Author Notification\nawait Task(\"Notification Agent\", `\nNotify PR author:\n- Review complete\n- Summary of findings\n- Action items (if any)\n- Next steps\n\nSend notification\nStore: code-review\u002F${prId}\u002Fauthor-notification\n`, \"pr-manager\");\n\n\u002F\u002F Step 3: Decision Actions\nif (decision === 'Approve') {\n  await Task(\"Merge Coordinator\", `\nApproved for merge:\n- Add \"approved\" label\n- Update PR status\n- Queue for merge (if auto-merge enabled)\n- Notify relevant teams\n\nStore: code-review\u002F${prId}\u002Fmerge-approval\n`, \"pr-manager\");\n} else if (decision === 'Request Changes') {\n  await Task(\"Feedback Coordinator\", `\nRequest changes:\n- Create detailed feedback comment\n- Label as \"changes-requested\"\n- Assign back to author\n- Schedule follow-up review\n\nStore: code-review\u002F${prId}\u002Fchange-request\n`, \"pr-manager\");\n} else {\n  await Task(\"Rejection Handler\", `\nReject PR:\n- Create detailed explanation\n- Suggest alternative approaches\n- Label as \"rejected\"\n- Close PR (or request fundamental rework)\n\nStore: code-review\u002F${prId}\u002Frejection\n`, \"pr-manager\");\n}\n```\n\n**Deliverables**:\n- Final review summary\n- Author notification\n- Decision and next steps\n\n---\n\n## Success Criteria\n\n### Review Quality\n- **Coverage**: All aspects reviewed (quality, security, performance, architecture, docs)\n- **Consistency**: Reviews follow established guidelines\n- **Actionability**: All feedback is specific and actionable\n- **Timeliness**: Reviews completed within 4 hours\n\n### Code Quality Gates\n- **Automated Tests**: 100% passing\n- **Code Coverage**: > 80% overall, > 90% for new code\n- **Linting**: 0 violations\n- **Security**: 0 critical issues, 0 high-severity issues\n- **Performance**: No high-impact performance regressions\n- **Documentation**: 100% of public APIs documented\n\n### Process Metrics\n- **Review Turnaround**: \u003C 4 hours (business hours)\n- **Author Satisfaction**: > 4\u002F5 (feedback is helpful)\n- **Defect Escape Rate**: \u003C 1% (issues found in production that should have been caught)\n\n---\n\n## Review Guidelines\n\n### What Reviewers Should Focus On\n\n**DO Review**:\n- Logic correctness\n- Edge case handling\n- Error handling robustness\n- Security vulnerabilities\n- Performance implications\n- Code clarity and maintainability\n- Test coverage and quality\n- API design and contracts\n- Documentation completeness\n\n**DON'T Nitpick**:\n- Personal style preferences (use automated linting)\n- Minor variable naming (unless truly confusing)\n- Trivial formatting (use automated formatting)\n- Subjective \"better\" ways (unless significantly better)\n\n### Giving Feedback\n\n**Effective Feedback**:\n- ✅ \"This function has O(n²) complexity. Consider using a hash map for O(n).\"\n- ✅ \"This input isn't validated. Add validation to prevent SQL injection.\"\n- ✅ \"This error isn't logged. Add error logging for debugging.\"\n\n**Ineffective Feedback**:\n- ❌ \"I don't like this.\"\n- ❌ \"This could be better.\"\n- ❌ \"Change this.\" (without explanation)\n\n**Tone**:\n- Be respectful and constructive\n- Assume good intent\n- Ask questions rather than make demands\n- Suggest, don't dictate (unless security\u002Fcritical issue)\n\n---\n\n## Agent Coordination Summary\n\n**Total Agents Used**: 12-15\n**Execution Pattern**: Star topology (coordinator with specialists)\n**Timeline**: 4 hours\n**Memory Namespaces**: code-review\u002F{pr-id}\u002F*\n\n**Key Agents**:\n1. reviewer - Lint, build, coordination\n2. tester - Test execution, integration testing\n3. code-analyzer - Code quality review\n4. security-manager - Security review\n5. perf-analyzer - Performance review\n6. system-architect - Architecture review\n7. api-docs - Documentation review\n8. cicd-engineer - Deployment impact\n9. planner - Product\u002Fuser impact\n10. pr-manager - Review coordination, notifications\n\n---\n\n## Usage\n\n```javascript\n\u002F\u002F Invoke this SOP skill for a PR\nSkill(\"sop-code-review\")\n\n\u002F\u002F Or execute with specific PR\nTask(\"Code Review Orchestrator\", `\nExecute comprehensive code review for PR #${prNumber}\nRepository: ${repoName}\nAuthor: ${authorName}\nChanges: ${changesSummary}\n`, \"pr-manager\")\n```\n\n---\n\n**Status**: Production-ready SOP\n**Complexity**: Medium (12-15 agents, 4 hours)\n**Pattern**: Star topology with specialized reviewers\n","# SOP: Code Review Workflow\n\nComprehensive code review using specialized reviewers for different quality aspects.\n\n## Timeline: 4 Hours\n\n**Phases**:\n1. Automated Checks (30 min)\n2. Specialized Reviews (2 hours)\n3. Integration Review (1 hour)\n4. Final Approval (30 min)\n\n---\n\n## Phase 1: Automated Checks (30 minutes)\n\n### Quick Quality Checks\n\n**Parallel Automated Testing**:\n\n```javascript\n\u002F\u002F Initialize review swarm\nawait mcp__ruv-swarm__swarm_init({\n  topology: 'star',  \u002F\u002F Coordinator pattern for reviews\n  maxAgents: 6,\n  strategy: 'specialized'\n});\n\n\u002F\u002F Run all automated checks in parallel\nconst [lint, tests, coverage, build] = await Promise.all([\n  Task(\"Linter\", `\nRun linting checks:\n- ESLint for JavaScript\u002FTypeScript\n- Pylint for Python\n- RuboCop for Ruby\n- Check for code style violations\n\nStore results: code-review\u002F${prId}\u002Flint-results\n`, \"reviewer\"),\n\n  Task(\"Test Runner\", `\nRun test suite:\n- Unit tests\n- Integration tests\n- E2E tests (if applicable)\n- All tests must pass\n\nStore results: code-review\u002F${prId}\u002Ftest-results\n`, \"tester\"),\n\n  Task(\"Coverage Analyzer\", `\nCheck code coverage:\n- Overall coverage > 80%\n- New code coverage > 90%\n- No critical paths uncovered\n\nGenerate coverage report\nStore: code-review\u002F${prId}\u002Fcoverage-report\n`, \"reviewer\"),\n\n  Task(\"Build Validator\", `\nValidate build:\n- Clean build (no warnings)\n- Type checking passes\n- No broken dependencies\n- Bundle size within limits\n\nStore build results: code-review\u002F${prId}\u002Fbuild-status\n`, \"reviewer\")\n]);\n\n\u002F\u002F If any automated check fails, stop and request fixes\nif (hasFailures([lint, tests, coverage, build])) {\n  await Task(\"Review Coordinator\", `\nAutomated checks failed. Request fixes from author:\n${summarizeFailures([lint, tests, coverage, build])}\n\nStore feedback: code-review\u002F${prId}\u002Fautomated-feedback\n`, \"pr-manager\");\n  return; \u002F\u002F Stop review until fixed\n}\n```\n\n**Deliverables**:\n- All automated checks passing\n- Test results documented\n- Coverage report generated\n\n---\n\n## Phase 2: Specialized Reviews (2 hours)\n\n### Parallel Expert Reviews\n\n**Sequential coordination of parallel reviews**:\n\n```javascript\n\u002F\u002F Spawn specialized reviewers in parallel\nconst [codeQuality, security, performance, architecture, docs] = await Promise.all([\n  Task(\"Code Quality Reviewer\", `\nReview for code quality:\n\n**Readability**:\n- Clear, descriptive names (variables, functions, classes)\n- Appropriate function\u002Fmethod length (\u003C 50 lines)\n- Logical code organization\n- Minimal cognitive complexity\n\n**Maintainability**:\n- DRY principle (no code duplication)\n- SOLID principles followed\n- Clear separation of concerns\n- Proper error handling\n\n**Best Practices**:\n- Following language idioms\n- Proper use of design patterns\n- Appropriate comments (why, not what)\n- No code smells (magic numbers, long parameter lists)\n\nStore review: code-review\u002F${prId}\u002Fquality-review\nRating: 1-5 stars\n`, \"code-analyzer\"),\n\n  Task(\"Security Reviewer\", `\nReview for security issues:\n\n**Authentication & Authorization**:\n- Proper authentication checks\n- Correct authorization rules\n- No privilege escalation risks\n- Secure session management\n\n**Data Security**:\n- Input validation (prevent injection attacks)\n- Output encoding (prevent XSS)\n- Sensitive data encryption\n- No hardcoded secrets or credentials\n\n**Common Vulnerabilities** (OWASP Top 10):\n- SQL Injection prevention\n- XSS prevention\n- CSRF protection\n- Secure dependencies (no known vulnerabilities)\n\nStore review: code-review\u002F${prId}\u002Fsecurity-review\nSeverity: Critical\u002FHigh\u002FMedium\u002FLow for each finding\n`, \"security-manager\"),\n\n  Task(\"Performance Reviewer\", `\nReview for performance issues:\n\n**Algorithmic Efficiency**:\n- Appropriate time complexity (no unnecessary O(n²))\n- Efficient data structures chosen\n- No unnecessary iterations\n- Lazy loading where appropriate\n\n**Resource Usage**:\n- No memory leaks\n- Proper cleanup (connections, files, timers)\n- Efficient database queries (avoid N+1)\n- Batch operations where possible\n\n**Optimization Opportunities**:\n- Caching potential\n- Parallelization opportunities\n- Database index needs\n- API call optimization\n\nStore review: code-review\u002F${prId}\u002Fperformance-review\nImpact: High\u002FMedium\u002FLow for each finding\n`, \"perf-analyzer\"),\n\n  Task(\"Architecture Reviewer\", `\nReview for architectural consistency:\n\n**Design Patterns**:\n- Follows established patterns in codebase\n- Appropriate abstraction level\n- Proper dependency injection\n- Clean architecture principles\n\n**Integration**:\n- Fits well with existing code\n- No unexpected side effects\n- Backward compatibility maintained\n- API contracts respected\n\n**Scalability**:\n- Design supports future growth\n- No hardcoded limits\n- Stateless where possible\n- Horizontally scalable\n\nStore review: code-review\u002F${prId}\u002Farchitecture-review\nConcerns: Blocker\u002FMajor\u002FMinor for each finding\n`, \"system-architect\"),\n\n  Task(\"Documentation Reviewer\", `\nReview documentation:\n\n**Code Documentation**:\n- Public APIs documented (JSDoc\u002Fdocstring)\n- Complex logic explained\n- Non-obvious behavior noted\n- Examples provided where helpful\n\n**External Documentation**:\n- README updated (if needed)\n- API docs updated (if API changed)\n- Migration guide (if breaking changes)\n- Changelog updated\n\n**Tests as Documentation**:\n- Test names are descriptive\n- Test coverage demonstrates usage\n- Edge cases documented in tests\n\nStore review: code-review\u002F${prId}\u002Fdocs-review\nCompleteness: 0-100%\n`, \"api-docs\")\n]);\n\n\u002F\u002F Aggregate all reviews\nawait Task(\"Review Aggregator\", `\nAggregate specialized reviews:\n- Quality: ${codeQuality}\n- Security: ${security}\n- Performance: ${performance}\n- Architecture: ${architecture}\n- Documentation: ${docs}\n\nIdentify:\n- Blocking issues (must fix before merge)\n- High-priority suggestions\n- Nice-to-have improvements\n\nGenerate summary\nStore: code-review\u002F${prId}\u002Faggregated-review\n`, \"reviewer\");\n```\n\n**Deliverables**:\n- 5 specialized reviews completed\n- Issues categorized by severity\n- Aggregated review summary\n\n---\n\n## Phase 3: Integration Review (1 hour)\n\n### End-to-End Impact Assessment\n\n**Sequential Analysis**:\n\n```javascript\n\u002F\u002F Step 1: Integration Testing\nawait Task(\"Integration Tester\", `\nTest integration with existing system:\n- Does this change break any existing functionality?\n- Are all integration tests passing?\n- Does it play well with related modules?\n- Any unexpected side effects?\n\nRun integration test suite\nStore results: code-review\u002F${prId}\u002Fintegration-tests\n`, \"tester\");\n\n\u002F\u002F Step 2: Deployment Impact\nawait Task(\"DevOps Reviewer\", `\nAssess deployment impact:\n- Infrastructure changes needed?\n- Database migrations required?\n- Configuration updates needed?\n- Backward compatibility maintained?\n- Rollback plan clear?\n\nStore assessment: code-review\u002F${prId}\u002Fdeployment-impact\n`, \"cicd-engineer\");\n\n\u002F\u002F Step 3: User Impact\nawait Task(\"Product Reviewer\", `\nAssess user impact:\n- Does this change improve user experience?\n- Are there any user-facing changes?\n- Is UX\u002FUI consistent with design system?\n- Are analytics\u002Ftracking updated?\n\nStore assessment: code-review\u002F${prId}\u002Fuser-impact\n`, \"planner\");\n\n\u002F\u002F Step 4: Risk Assessment\nawait Task(\"Risk Analyzer\", `\nOverall risk assessment:\n- What's the blast radius of this change?\n- What's the worst-case failure scenario?\n- Do we have rollback procedures?\n- Should this be feature-flagged?\n- Monitoring and alerting adequate?\n\nStore risk assessment: code-review\u002F${prId}\u002Frisk-analysis\nRecommendation: Approve\u002FConditional\u002FReject\n`, \"reviewer\");\n```\n\n**Deliverables**:\n- Integration test results\n- Deployment impact assessment\n- User impact assessment\n- Risk analysis\n\n---\n\n## Phase 4: Final Approval (30 minutes)\n\n### Review Summary & Decision\n\n**Sequential Finalization**:\n\n```javascript\n\u002F\u002F Step 1: Generate Final Summary\nawait Task(\"Review Coordinator\", `\nGenerate final review summary:\n\n**Automated Checks**: ✅ All passing\n**Quality Review**: ${qualityScore}\u002F5\n**Security Review**: ${securityIssues} issues (${criticalCount} critical)\n**Performance Review**: ${perfIssues} issues (${highImpactCount} high-impact)\n**Architecture Review**: ${archConcerns} concerns (${blockerCount} blockers)\n**Documentation Review**: ${docsCompleteness}% complete\n**Integration Tests**: ${integrationStatus}\n**Deployment Impact**: ${deploymentImpact}\n**User Impact**: ${userImpact}\n**Risk Level**: ${riskLevel}\n\n**Blocking Issues**:\n${listBlockingIssues()}\n\n**Recommendations**:\n${generateRecommendations()}\n\n**Overall Decision**: ${decision} (Approve\u002FRequest Changes\u002FReject)\n\nStore final summary: code-review\u002F${prId}\u002Ffinal-summary\n`, \"pr-manager\");\n\n\u002F\u002F Step 2: Author Notification\nawait Task(\"Notification Agent\", `\nNotify PR author:\n- Review complete\n- Summary of findings\n- Action items (if any)\n- Next steps\n\nSend notification\nStore: code-review\u002F${prId}\u002Fauthor-notification\n`, \"pr-manager\");\n\n\u002F\u002F Step 3: Decision Actions\nif (decision === 'Approve') {\n  await Task(\"Merge Coordinator\", `\nApproved for merge:\n- Add \"approved\" label\n- Update PR status\n- Queue for merge (if auto-merge enabled)\n- Notify relevant teams\n\nStore: code-review\u002F${prId}\u002Fmerge-approval\n`, \"pr-manager\");\n} else if (decision === 'Request Changes') {\n  await Task(\"Feedback Coordinator\", `\nRequest changes:\n- Create detailed feedback comment\n- Label as \"changes-requested\"\n- Assign back to author\n- Schedule follow-up review\n\nStore: code-review\u002F${prId}\u002Fchange-request\n`, \"pr-manager\");\n} else {\n  await Task(\"Rejection Handler\", `\nReject PR:\n- Create detailed explanation\n- Suggest alternative approaches\n- Label as \"rejected\"\n- Close PR (or request fundamental rework)\n\nStore: code-review\u002F${prId}\u002Frejection\n`, \"pr-manager\");\n}\n```\n\n**Deliverables**:\n- Final review summary\n- Author notification\n- Decision and next steps\n\n---\n\n## Success Criteria\n\n### Review Quality\n- **Coverage**: All aspects reviewed (quality, security, performance, architecture, docs)\n- **Consistency**: Reviews follow established guidelines\n- **Actionability**: All feedback is specific and actionable\n- **Timeliness**: Reviews completed within 4 hours\n\n### Code Quality Gates\n- **Automated Tests**: 100% passing\n- **Code Coverage**: > 80% overall, > 90% for new code\n- **Linting**: 0 violations\n- **Security**: 0 critical issues, 0 high-severity issues\n- **Performance**: No high-impact performance regressions\n- **Documentation**: 100% of public APIs documented\n\n### Process Metrics\n- **Review Turnaround**: \u003C 4 hours (business hours)\n- **Author Satisfaction**: > 4\u002F5 (feedback is helpful)\n- **Defect Escape Rate**: \u003C 1% (issues found in production that should have been caught)\n\n---\n\n## Review Guidelines\n\n### What Reviewers Should Focus On\n\n**DO Review**:\n- Logic correctness\n- Edge case handling\n- Error handling robustness\n- Security vulnerabilities\n- Performance implications\n- Code clarity and maintainability\n- Test coverage and quality\n- API design and contracts\n- Documentation completeness\n\n**DON'T Nitpick**:\n- Personal style preferences (use automated linting)\n- Minor variable naming (unless truly confusing)\n- Trivial formatting (use automated formatting)\n- Subjective \"better\" ways (unless significantly better)\n\n### Giving Feedback\n\n**Effective Feedback**:\n- ✅ \"This function has O(n²) complexity. Consider using a hash map for O(n).\"\n- ✅ \"This input isn't validated. Add validation to prevent SQL injection.\"\n- ✅ \"This error isn't logged. Add error logging for debugging.\"\n\n**Ineffective Feedback**:\n- ❌ \"I don't like this.\"\n- ❌ \"This could be better.\"\n- ❌ \"Change this.\" (without explanation)\n\n**Tone**:\n- Be respectful and constructive\n- Assume good intent\n- Ask questions rather than make demands\n- Suggest, don't dictate (unless security\u002Fcritical issue)\n\n---\n\n## Agent Coordination Summary\n\n**Total Agents Used**: 12-15\n**Execution Pattern**: Star topology (coordinator with specialists)\n**Timeline**: 4 hours\n**Memory Namespaces**: code-review\u002F{pr-id}\u002F*\n\n**Key Agents**:\n1. reviewer - Lint, build, coordination\n2. tester - Test execution, integration testing\n3. code-analyzer - Code quality review\n4. security-manager - Security review\n5. perf-analyzer - Performance review\n6. system-architect - Architecture review\n7. api-docs - Documentation review\n8. cicd-engineer - Deployment impact\n9. planner - Product\u002Fuser impact\n10. pr-manager - Review coordination, notifications\n\n---\n\n## Usage\n\n```javascript\n\u002F\u002F Invoke this SOP skill for a PR\nSkill(\"sop-code-review\")\n\n\u002F\u002F Or execute with specific PR\nTask(\"Code Review Orchestrator\", `\nExecute comprehensive code review for PR #${prNumber}\nRepository: ${repoName}\nAuthor: ${authorName}\nChanges: ${changesSummary}\n`, \"pr-manager\")\n```\n\n---\n\n**Status**: Production-ready SOP\n**Complexity**: Medium (12-15 agents, 4 hours)\n**Pattern**: Star topology with specialized reviewers\n",{"name":5,"description":7},"main","def8531e65114c0fca8fb8551c1871ee0eed705c","skills\u002Faiskillstore\u002Fmarketplace\u002Fdnyoussef\u002Fsop-code-review",24,"https:\u002F\u002Fgithub.com\u002FComeOnOliver\u002Fskillshub","https:\u002F\u002Fskillshub.wtf","https:\u002F\u002Fgithub.com\u002FComeOnOliver","AI & Blockchain Engineer\r\n","2026-06-24 22:07:19",{"source":8,"skillName":5,"commands":63,"prompt":73,"zipAvailable":74},[64,67,70],{"tool":65,"command":66},"npx","npx skills add ComeOnOliver\u002Fskillshub --skill sop-code-review",{"tool":68,"command":69},"bunx","bunx skills add ComeOnOliver\u002Fskillshub --skill sop-code-review",{"tool":71,"command":72},"pnpm","pnpm dlx skills add ComeOnOliver\u002Fskillshub --skill sop-code-review","安装技能，技能地址：https:\u002F\u002Fgithub.com\u002FComeOnOliver\u002Fskillshub\u002Fblob\u002Fmain\u002Fskills\u002Faiskillstore\u002Fmarketplace\u002Fdnyoussef\u002Fsop-code-review\u002FSKILL.md",true,[76],{"path":77,"resourceType":78,"resourceTypeDesc":79,"sizeBytes":80,"sourceHtmlUrl":81},"skill-report.json","REFERENCE","参考文档",11517,"https:\u002F\u002Fgithub.com\u002FComeOnOliver\u002Fskillshub\u002Fblob\u002Fmain\u002Fskills\u002Faiskillstore\u002Fmarketplace\u002Fdnyoussef\u002Fsop-code-review\u002Fskill-report.json",[83,93,103,112,118,125],{"skillNo":84,"name":85,"nameZh":6,"description":6,"descriptionZh":6,"repositoryFullName":8,"ownerLogin":9,"ownerAvatarUrl":10,"ownerAccountNo":11,"ownerDisplayName":12,"repositoryNo":13,"starsCount":14,"licenseSpdxId":15,"licenseFileUrl":6,"sourceHtmlUrl":86,"categories":87,"keywords":89,"publishedAt":48},"01M6FKJN13GMG4KY57J3QD8GY5","model-hierarchy-skill","https:\u002F\u002Fgithub.com\u002FComeOnOliver\u002Fskillshub\u002Fblob\u002Fmain\u002Fskills\u002Fzscole\u002Fmodel-hierarchy-skill\u002Fmodel-hierarchy-skill\u002FSKILL.md",[88],{"categoryNo":24,"categoryName":25,"taxonomyType":21,"taxonomyTypeDesc":22},[90,91,92],"model","model hierarchy","model hierarchy skill",{"skillNo":94,"name":95,"nameZh":6,"description":6,"descriptionZh":6,"repositoryFullName":8,"ownerLogin":9,"ownerAvatarUrl":10,"ownerAccountNo":11,"ownerDisplayName":12,"repositoryNo":13,"starsCount":14,"licenseSpdxId":15,"licenseFileUrl":6,"sourceHtmlUrl":96,"categories":97,"keywords":99,"publishedAt":48},"01M6FKJJJ6GCAMR4PPPRD12M5X","zoom-mcp","https:\u002F\u002Fgithub.com\u002FComeOnOliver\u002Fskillshub\u002Fblob\u002Fmain\u002Fskills\u002Fzoom\u002Fskills\u002Fzoom-mcp\u002FSKILL.md",[98],{"categoryNo":24,"categoryName":25,"taxonomyType":21,"taxonomyTypeDesc":22},[100,101,102],"MCP","zoom","zoom mcp",{"skillNo":104,"name":105,"nameZh":6,"description":6,"descriptionZh":6,"repositoryFullName":8,"ownerLogin":9,"ownerAvatarUrl":10,"ownerAccountNo":11,"ownerDisplayName":12,"repositoryNo":13,"starsCount":14,"licenseSpdxId":15,"licenseFileUrl":6,"sourceHtmlUrl":106,"categories":107,"keywords":108,"publishedAt":48},"01M6FKJE3KYT3KPEZMEEHYKA6K","zoom-apps-sdk","https:\u002F\u002Fgithub.com\u002FComeOnOliver\u002Fskillshub\u002Fblob\u002Fmain\u002Fskills\u002Fzoom\u002Fskills\u002Fzoom-apps-sdk\u002FSKILL.md",[],[109,110,111,101],"sdk","zoom apps sdk","apps",{"skillNo":113,"name":114,"nameZh":6,"description":6,"descriptionZh":6,"repositoryFullName":8,"ownerLogin":9,"ownerAvatarUrl":10,"ownerAccountNo":11,"ownerDisplayName":12,"repositoryNo":13,"starsCount":14,"licenseSpdxId":15,"licenseFileUrl":6,"sourceHtmlUrl":115,"categories":116,"keywords":117,"publishedAt":48},"01M6FKJ9EDH9Q5VWDF8HAJRSP0","windows","https:\u002F\u002Fgithub.com\u002FComeOnOliver\u002Fskillshub\u002Fblob\u002Fmain\u002Fskills\u002Fzoom\u002Fskills\u002Fwindows\u002FSKILL.md",[],[114],{"skillNo":119,"name":120,"nameZh":6,"description":6,"descriptionZh":6,"repositoryFullName":8,"ownerLogin":9,"ownerAvatarUrl":10,"ownerAccountNo":11,"ownerDisplayName":12,"repositoryNo":13,"starsCount":14,"licenseSpdxId":15,"licenseFileUrl":6,"sourceHtmlUrl":121,"categories":122,"keywords":124,"publishedAt":48},"01M6FKJ666BEEFC3AE9GJ72Q93","whiteboard","https:\u002F\u002Fgithub.com\u002FComeOnOliver\u002Fskillshub\u002Fblob\u002Fmain\u002Fskills\u002Fzoom\u002Fskills\u002Fwhiteboard\u002FSKILL.md",[123],{"categoryNo":24,"categoryName":25,"taxonomyType":21,"taxonomyTypeDesc":22},[120],{"skillNo":126,"name":127,"nameZh":6,"description":6,"descriptionZh":6,"repositoryFullName":8,"ownerLogin":9,"ownerAvatarUrl":10,"ownerAccountNo":11,"ownerDisplayName":12,"repositoryNo":13,"starsCount":14,"licenseSpdxId":15,"licenseFileUrl":6,"sourceHtmlUrl":128,"categories":129,"keywords":130,"publishedAt":48},"01M6FKJ21WFCDGTJQ2Q91T9DVW","websockets","https:\u002F\u002Fgithub.com\u002FComeOnOliver\u002Fskillshub\u002Fblob\u002Fmain\u002Fskills\u002Fzoom\u002Fskills\u002Fwebsockets\u002FSKILL.md",[],[127]]