[{"data":1,"prerenderedAt":-1},["ShallowReactive",2],{"skill:01M71C6PFKHSVBPCES0VKGSD0Y":3,"skill-same:01M71C6PFKHSVBPCES0VKGSD0Y":64},{"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":28,"publishedAt":38,"linkOnly":39,"markdownContent":40,"markdownBody":41,"frontMatter":42,"refName":43,"commitSha":44,"skillDirectory":45,"forksCount":14,"repositoryHtmlUrl":46,"repositoryHomepageUrl":6,"ownerHtmlUrl":47,"ownerBlogUrl":6,"ownerDescription":48,"updatedAt":49,"install":50,"resources":63},"01M71C6PFKHSVBPCES0VKGSD0Y","code-review",null,"Standalone code review methodology for structured, severity-classified code assessment","yqecea\u002Fgem-swarm","yqecea","https:\u002F\u002Favatars.githubusercontent.com\u002Fu\u002F196189405?v=4","01M71C12KWH5QPTE12J69335KF","Yusuf","01M71C12MFS3BRGNE5SSHB4DY9",1,"Apache-2.0","https:\u002F\u002Fgithub.com\u002Fyqecea\u002Fgem-swarm\u002Fblob\u002Fmain\u002Fsrc\u002Fskills\u002Fshared\u002Fcode-review\u002FSKILL.md",[18,23],{"categoryNo":19,"categoryName":20,"taxonomyType":21,"taxonomyTypeDesc":22},"01JBSEEDCAT000000000000007","代码审查","SCENARIO","使用场景",{"categoryNo":24,"categoryName":25,"taxonomyType":26,"taxonomyTypeDesc":27},"01JBSEEDCAT000000000000030","开发工程师","PROFESSION","适用职业",[29,30,31,32,33,34,35,36,37],"code review","review","code","assessment","severity","methodology","structured","standalone","for","2026-11-10 10:12:30",false,"---\nname: code-review\ndescription: Standalone code review methodology for structured, severity-classified code assessment\n---\n\n# Code Review Skill\n\nActivate this skill when performing standalone code reviews via the runtime-specific Maestro review entrypoint or during orchestration quality gates (post-phase checks and final completion gate). This skill provides the methodology for scoping, executing, and reporting code reviews.\n\n## Scope Detection Protocol\n\nDetermine review scope using the following priority order:\n\n1. **User-specified paths**: If the user provides file paths or glob patterns, expand glob patterns using the `glob` tool to resolve them to concrete file paths before delegating to the `code-reviewer` agent\n2. **Staged changes**: If `git diff --staged` produces output, review staged changes\n3. **Last commit diff**: If no staged changes exist, review the last commit via `git diff HEAD~1`\n4. **Fallback**: If none of the above yield content, ask the user to specify scope\n\nAlways confirm the detected scope with the user before proceeding.\n\nIf scope is provided as file paths and a git diff is empty for some paths (for example, new unstaged files), include those files' current contents directly in review context so they are still reviewed.\n\n## Review Orchestration\n\n### Delegation Flow\n\n1. Detect review scope using the protocol above\n2. Gather the full diff content for the detected scope, and include current file contents when diff content is unavailable for scoped files\n3. Delegate to the `code-reviewer` agent with:\n   - The full diff content\n   - File paths involved\n   - Any user-provided focus areas or concerns\n4. Process the agent's Task Report\n5. Present findings to the user in the structured output format below\n\n### Context Enrichment\n\nWhen delegating to the `code-reviewer` agent, include:\n- The diff content (not just file names)\n- Surrounding context for modified sections (10 lines before\u002Fafter when available)\n- Project language and framework information (detected from package.json, Cargo.toml, go.mod, etc.)\n\n## Severity Classification\n\n### Critical\nIssues that could cause security vulnerabilities, data loss, or system crashes:\n- SQL\u002FNoSQL injection vectors\n- Authentication\u002Fauthorization bypasses\n- Unvalidated user input at system boundaries\n- Resource leaks (unclosed connections, file handles)\n- Race conditions with data corruption potential\n\n### Major\nIssues that cause bugs, design flaws, or significant maintainability problems:\n- Logic errors in business rules\n- Missing error handling on external calls\n- SOLID principle violations that impact extensibility\n- Incorrect API contracts or type mismatches\n- Missing null\u002Fundefined checks on external data\n\n### Minor\nIssues related to style, naming, or minor convention violations:\n- Naming inconsistencies\n- Code style deviations from project conventions\n- Suboptimal but correct implementations\n- Missing type annotations where inference is insufficient\n\n### Suggestion\nOptional improvements that enhance readability or maintainability:\n- Alternative patterns that improve clarity\n- Performance optimizations with marginal impact\n- Structural improvements for future extensibility\n\n## Output Format\n\nPresent findings in a structured table followed by a summary:\n\n```\n## Code Review Results\n\n**Scope**: [description of what was reviewed]\n**Files Reviewed**: [count]\n**Total Findings**: [count by severity]\n\n### Findings\n\n| # | Severity | File | Line | Description | Suggested Fix |\n|---|----------|------|------|-------------|---------------|\n| 1 | Critical | path\u002Fto\u002Ffile.ts | 42 | [description] | [fix] |\n| 2 | Major | path\u002Fto\u002Ffile.ts | 87 | [description] | [fix] |\n\n### Summary\n\n[1-2 paragraph summary of overall code quality, patterns observed, and priority actions]\n```\n\n## Verification Rule\n\nEvery finding **must**:\n- Reference a specific file and line number\n- Be verified against the actual code (not assumed from patterns)\n- Include a concrete suggested fix or action\n- Be classified with a severity that matches the classification criteria above\n\nDo NOT report:\n- Speculative issues based on assumptions about runtime behavior\n- Style preferences not established by the project's conventions\n- Issues in code outside the review scope\n\n## Review Scope Calibration\n\nCalibrate the depth and focus of review based on the type of change being reviewed:\n\n### Calibration Rules\n- **New files**: Full review across all dimensions — architecture fit, pattern compliance, security, naming conventions, error handling, testability, dependency direction\n- **Modified files (behavior change)**: Focus on the diff — correctness of new behavior, regression risk, contract compliance with existing interfaces, edge case handling in new code paths\n- **Modified files (refactoring)**: Focus on behavior preservation — verify same inputs produce same outputs, no unintended side effects introduced, no behavior changes disguised as refactoring\n- **Deleted files**: Dependency verification — confirm no remaining code imports from, references, or depends on the deleted files. Check for orphaned tests that tested the deleted code.\n- **Configuration changes**: Environment impact assessment — does this change affect production? Staging? Local development? All environments? Are there secrets or credentials involved?\n\n### Application\nWhen reviewing a diff that contains multiple change types (new files + modifications + deletions), apply the appropriate calibration to each file independently. Do not apply \"new file\" depth to a file that only had a minor modification.\n\n## Finding Deduplication Protocol\n\nWhen reviewing multiple files, identify and consolidate findings that share the same root cause.\n\n### Deduplication Rules\n- If the same pattern violation appears in 3+ files, report it **once** as a systemic finding with the list of all affected locations — not as N separate findings\n- A systemic finding includes: the pattern being violated, why it matters, the full list of affected file:line locations, and a single remediation recommendation that addresses all instances\n- Unique findings (appearing in only 1-2 files) are reported individually as normal\n\n### Deduplication Format\n```\n### Systemic Finding: [Pattern Violation Name]\n- **Severity**: [Critical | Major | Minor | Suggestion]\n- **Description**: [What the pattern violation is and why it matters]\n- **Affected Locations**:\n  - `path\u002Fto\u002Ffile1.ext:line` — [brief context]\n  - `path\u002Fto\u002Ffile2.ext:line` — [brief context]\n  - `path\u002Fto\u002Ffile3.ext:line` — [brief context]\n- **Remediation**: [Single recommendation that addresses all instances]\n```\n\nThis produces cleaner, more actionable review output by surfacing systemic issues as patterns rather than repeating the same finding across multiple files.\n","# Code Review Skill\n\nActivate this skill when performing standalone code reviews via the runtime-specific Maestro review entrypoint or during orchestration quality gates (post-phase checks and final completion gate). This skill provides the methodology for scoping, executing, and reporting code reviews.\n\n## Scope Detection Protocol\n\nDetermine review scope using the following priority order:\n\n1. **User-specified paths**: If the user provides file paths or glob patterns, expand glob patterns using the `glob` tool to resolve them to concrete file paths before delegating to the `code-reviewer` agent\n2. **Staged changes**: If `git diff --staged` produces output, review staged changes\n3. **Last commit diff**: If no staged changes exist, review the last commit via `git diff HEAD~1`\n4. **Fallback**: If none of the above yield content, ask the user to specify scope\n\nAlways confirm the detected scope with the user before proceeding.\n\nIf scope is provided as file paths and a git diff is empty for some paths (for example, new unstaged files), include those files' current contents directly in review context so they are still reviewed.\n\n## Review Orchestration\n\n### Delegation Flow\n\n1. Detect review scope using the protocol above\n2. Gather the full diff content for the detected scope, and include current file contents when diff content is unavailable for scoped files\n3. Delegate to the `code-reviewer` agent with:\n   - The full diff content\n   - File paths involved\n   - Any user-provided focus areas or concerns\n4. Process the agent's Task Report\n5. Present findings to the user in the structured output format below\n\n### Context Enrichment\n\nWhen delegating to the `code-reviewer` agent, include:\n- The diff content (not just file names)\n- Surrounding context for modified sections (10 lines before\u002Fafter when available)\n- Project language and framework information (detected from package.json, Cargo.toml, go.mod, etc.)\n\n## Severity Classification\n\n### Critical\nIssues that could cause security vulnerabilities, data loss, or system crashes:\n- SQL\u002FNoSQL injection vectors\n- Authentication\u002Fauthorization bypasses\n- Unvalidated user input at system boundaries\n- Resource leaks (unclosed connections, file handles)\n- Race conditions with data corruption potential\n\n### Major\nIssues that cause bugs, design flaws, or significant maintainability problems:\n- Logic errors in business rules\n- Missing error handling on external calls\n- SOLID principle violations that impact extensibility\n- Incorrect API contracts or type mismatches\n- Missing null\u002Fundefined checks on external data\n\n### Minor\nIssues related to style, naming, or minor convention violations:\n- Naming inconsistencies\n- Code style deviations from project conventions\n- Suboptimal but correct implementations\n- Missing type annotations where inference is insufficient\n\n### Suggestion\nOptional improvements that enhance readability or maintainability:\n- Alternative patterns that improve clarity\n- Performance optimizations with marginal impact\n- Structural improvements for future extensibility\n\n## Output Format\n\nPresent findings in a structured table followed by a summary:\n\n```\n## Code Review Results\n\n**Scope**: [description of what was reviewed]\n**Files Reviewed**: [count]\n**Total Findings**: [count by severity]\n\n### Findings\n\n| # | Severity | File | Line | Description | Suggested Fix |\n|---|----------|------|------|-------------|---------------|\n| 1 | Critical | path\u002Fto\u002Ffile.ts | 42 | [description] | [fix] |\n| 2 | Major | path\u002Fto\u002Ffile.ts | 87 | [description] | [fix] |\n\n### Summary\n\n[1-2 paragraph summary of overall code quality, patterns observed, and priority actions]\n```\n\n## Verification Rule\n\nEvery finding **must**:\n- Reference a specific file and line number\n- Be verified against the actual code (not assumed from patterns)\n- Include a concrete suggested fix or action\n- Be classified with a severity that matches the classification criteria above\n\nDo NOT report:\n- Speculative issues based on assumptions about runtime behavior\n- Style preferences not established by the project's conventions\n- Issues in code outside the review scope\n\n## Review Scope Calibration\n\nCalibrate the depth and focus of review based on the type of change being reviewed:\n\n### Calibration Rules\n- **New files**: Full review across all dimensions — architecture fit, pattern compliance, security, naming conventions, error handling, testability, dependency direction\n- **Modified files (behavior change)**: Focus on the diff — correctness of new behavior, regression risk, contract compliance with existing interfaces, edge case handling in new code paths\n- **Modified files (refactoring)**: Focus on behavior preservation — verify same inputs produce same outputs, no unintended side effects introduced, no behavior changes disguised as refactoring\n- **Deleted files**: Dependency verification — confirm no remaining code imports from, references, or depends on the deleted files. Check for orphaned tests that tested the deleted code.\n- **Configuration changes**: Environment impact assessment — does this change affect production? Staging? Local development? All environments? Are there secrets or credentials involved?\n\n### Application\nWhen reviewing a diff that contains multiple change types (new files + modifications + deletions), apply the appropriate calibration to each file independently. Do not apply \"new file\" depth to a file that only had a minor modification.\n\n## Finding Deduplication Protocol\n\nWhen reviewing multiple files, identify and consolidate findings that share the same root cause.\n\n### Deduplication Rules\n- If the same pattern violation appears in 3+ files, report it **once** as a systemic finding with the list of all affected locations — not as N separate findings\n- A systemic finding includes: the pattern being violated, why it matters, the full list of affected file:line locations, and a single remediation recommendation that addresses all instances\n- Unique findings (appearing in only 1-2 files) are reported individually as normal\n\n### Deduplication Format\n```\n### Systemic Finding: [Pattern Violation Name]\n- **Severity**: [Critical | Major | Minor | Suggestion]\n- **Description**: [What the pattern violation is and why it matters]\n- **Affected Locations**:\n  - `path\u002Fto\u002Ffile1.ext:line` — [brief context]\n  - `path\u002Fto\u002Ffile2.ext:line` — [brief context]\n  - `path\u002Fto\u002Ffile3.ext:line` — [brief context]\n- **Remediation**: [Single recommendation that addresses all instances]\n```\n\nThis produces cleaner, more actionable review output by surfacing systemic issues as patterns rather than repeating the same finding across multiple files.\n",{"name":5,"description":7},"main","923c514d4d5e15bbfd65ec7eaf949a997679aa3e","src\u002Fskills\u002Fshared\u002Fcode-review","https:\u002F\u002Fgithub.com\u002Fyqecea\u002Fgem-swarm","https:\u002F\u002Fgithub.com\u002Fyqecea","15 y.o Data Science,Student","2026-04-26 20:00:27",{"source":8,"skillName":5,"commands":51,"prompt":61,"zipAvailable":62},[52,55,58],{"tool":53,"command":54},"npx","npx skills add yqecea\u002Fgem-swarm --skill code-review",{"tool":56,"command":57},"bunx","bunx skills add yqecea\u002Fgem-swarm --skill code-review",{"tool":59,"command":60},"pnpm","pnpm dlx skills add yqecea\u002Fgem-swarm --skill code-review","安装技能，技能地址：https:\u002F\u002Fgithub.com\u002Fyqecea\u002Fgem-swarm\u002Fblob\u002Fmain\u002Fsrc\u002Fskills\u002Fshared\u002Fcode-review\u002FSKILL.md",true,[],[65,77,91,102,114,134],{"skillNo":66,"name":67,"nameZh":6,"description":68,"descriptionZh":6,"repositoryFullName":8,"ownerLogin":9,"ownerAvatarUrl":10,"ownerAccountNo":11,"ownerDisplayName":12,"repositoryNo":13,"starsCount":14,"licenseSpdxId":15,"licenseFileUrl":6,"sourceHtmlUrl":69,"categories":70,"keywords":71,"publishedAt":38},"01M71C74XVPP23ST10E8MYJZA0","validation","Cross-cutting validation methodology for verifying phase outputs and project integrity","https:\u002F\u002Fgithub.com\u002Fyqecea\u002Fgem-swarm\u002Fblob\u002Fmain\u002Fsrc\u002Fskills\u002Fshared\u002Fvalidation\u002FSKILL.md",[],[67,72,37,73,74,75,34,76],"validation methodology","project","outputs","phase","integrity",{"skillNo":78,"name":79,"nameZh":6,"description":80,"descriptionZh":6,"repositoryFullName":8,"ownerLogin":9,"ownerAvatarUrl":10,"ownerAccountNo":11,"ownerDisplayName":12,"repositoryNo":13,"starsCount":14,"licenseSpdxId":15,"licenseFileUrl":6,"sourceHtmlUrl":81,"categories":82,"keywords":83,"publishedAt":38},"01M71C72PTKRV481M975R3QDDE","session-management","Manages orchestration session state, tracking, and resumption","https:\u002F\u002Fgithub.com\u002Fyqecea\u002Fgem-swarm\u002Fblob\u002Fmain\u002Fsrc\u002Fskills\u002Fshared\u002Fsession-management\u002FSKILL.md",[],[84,85,86,87,88,89,90],"session management","session","management","session state","orchestration","state","tracking",{"skillNo":92,"name":93,"nameZh":6,"description":94,"descriptionZh":6,"repositoryFullName":8,"ownerLogin":9,"ownerAvatarUrl":10,"ownerAccountNo":11,"ownerDisplayName":12,"repositoryNo":13,"starsCount":14,"licenseSpdxId":15,"licenseFileUrl":6,"sourceHtmlUrl":95,"categories":96,"keywords":97,"publishedAt":38},"01M71C6ZY1EGM4SGVYT4B2HHD7","implementation-planning","Generates detailed implementation plans from finalized designs","https:\u002F\u002Fgithub.com\u002Fyqecea\u002Fgem-swarm\u002Fblob\u002Fmain\u002Fsrc\u002Fskills\u002Fshared\u002Fimplementation-planning\u002FSKILL.md",[],[98,99,100,101],"planning","implementation","implementation planning","plans",{"skillNo":103,"name":104,"nameZh":6,"description":105,"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":38},"01M71C6XQVGMDFJ50BP5SBDS76","execution","Phase execution methodology for orchestration workflows with error handling and completion protocols","https:\u002F\u002Fgithub.com\u002Fyqecea\u002Fgem-swarm\u002Fblob\u002Fmain\u002Fsrc\u002Fskills\u002Fshared\u002Fexecution\u002FSKILL.md",[],[104,37,34,109,110,88,75,111,112,113],"error","workflows","protocols","error handling","with",{"skillNo":115,"name":116,"nameZh":6,"description":117,"descriptionZh":6,"repositoryFullName":8,"ownerLogin":9,"ownerAvatarUrl":10,"ownerAccountNo":11,"ownerDisplayName":12,"repositoryNo":13,"starsCount":14,"licenseSpdxId":15,"licenseFileUrl":6,"sourceHtmlUrl":118,"categories":119,"keywords":126,"publishedAt":38},"01M71C6VY8APWMNK1XEJ3WP7A2","design-dialogue","Guides structured design conversations for complex engineering tasks","https:\u002F\u002Fgithub.com\u002Fyqecea\u002Fgem-swarm\u002Fblob\u002Fmain\u002Fsrc\u002Fskills\u002Fshared\u002Fdesign-dialogue\u002FSKILL.md",[120,123],{"categoryNo":121,"categoryName":122,"taxonomyType":21,"taxonomyTypeDesc":22},"01JBSEEDCAT000000000000015","视觉设计",{"categoryNo":124,"categoryName":125,"taxonomyType":26,"taxonomyTypeDesc":27},"01JBSEEDCAT000000000000032","设计师",[127,128,129,130,131,37,132,133,35],"dialogue","design","design dialogue","guides","conversations","tasks","engineering",{"skillNo":135,"name":136,"nameZh":6,"description":137,"descriptionZh":6,"repositoryFullName":8,"ownerLogin":9,"ownerAvatarUrl":10,"ownerAccountNo":11,"ownerDisplayName":12,"repositoryNo":13,"starsCount":14,"licenseSpdxId":15,"licenseFileUrl":6,"sourceHtmlUrl":138,"categories":139,"keywords":146,"publishedAt":38},"01M71C6R4RJWG3S2R26778TPD8","delegation","Agent delegation best practices for constructing effective subagent prompts with proper scoping","https:\u002F\u002Fgithub.com\u002Fyqecea\u002Fgem-swarm\u002Fblob\u002Fmain\u002Fsrc\u002Fskills\u002Fshared\u002Fdelegation\u002FSKILL.md",[140,143],{"categoryNo":141,"categoryName":142,"taxonomyType":21,"taxonomyTypeDesc":22},"01JBSEEDCAT000000000000010","后端与接口",{"categoryNo":144,"categoryName":145,"taxonomyType":21,"taxonomyTypeDesc":22},"01JBSEEDCAT000000000000023","Skill与智能体开发",[136,147,148,37,149,113,150],"subagent","scoping","best practices","prompts"]