[{"data":1,"prerenderedAt":-1},["ShallowReactive",2],{"skill:01M71V43VQC1Z579KZSHB8PHK4":3,"skill-same:01M71V43VQC1Z579KZSHB8PHK4":79},{"skillNo":4,"name":5,"nameZh":6,"description":7,"descriptionZh":6,"repositoryFullName":8,"ownerLogin":9,"ownerAvatarUrl":10,"ownerAccountNo":11,"ownerDisplayName":6,"repositoryNo":12,"starsCount":13,"licenseSpdxId":14,"licenseFileUrl":6,"sourceHtmlUrl":15,"categories":16,"keywords":27,"publishedAt":48,"linkOnly":49,"markdownContent":50,"markdownBody":51,"frontMatter":52,"refName":59,"commitSha":60,"skillDirectory":61,"forksCount":13,"repositoryHtmlUrl":62,"repositoryHomepageUrl":6,"ownerHtmlUrl":63,"ownerBlogUrl":6,"ownerDescription":6,"updatedAt":64,"install":65,"resources":78},"01M71V43VQC1Z579KZSHB8PHK4","code-review",null,"Review code changes for correctness, test coverage, security, performance, and maintainability","strings77wzq\u002Fgoal-run","strings77wzq","https:\u002F\u002Favatars.githubusercontent.com\u002Fu\u002F184192027?v=4","01M71V4218PHCX3JS0WGM99C41","01M71V421WHQ3QPRVP9GWKQVGV",0,"MIT","https:\u002F\u002Fgithub.com\u002Fstrings77wzq\u002Fgoal-run\u002Fblob\u002Fmain\u002F.agent\u002Fskills\u002Fcode-review\u002FSKILL.md",[17,22],{"categoryNo":18,"categoryName":19,"taxonomyType":20,"taxonomyTypeDesc":21},"01JBSEEDCAT000000000000007","代码审查","SCENARIO","使用场景",{"categoryNo":23,"categoryName":24,"taxonomyType":25,"taxonomyTypeDesc":26},"01JBSEEDCAT000000000000030","开发工程师","PROFESSION","适用职业",[28,29,30,31,32,33,34,35,36,37,38,39,40,41,42,43,44,45,46,47],"review","code review","code","human code","changes","for","implementing","correctness","you","critical","as","do","files","use","coverage","assessment","pr","change","test","performance","2026-11-10 13:02:02",false,"---\nname: code-review\ndescription: Review code changes for correctness, test coverage, security, performance, and maintainability\nversion: '1.0.0'\nrisk: low\npermissions:\n  - read_files\nwhen_to_use: |\n  Use after implementing a change, before merging a PR, or whenever you need\n  an independent quality assessment of code changes.\nwhen_not_to_use: |\n  Do not use as a replacement for human code review on critical paths.\n  Do not use for changes that have not been tested.\n---\n\n# Code Review\n\nPerform a structured review of code changes against quality dimensions.\n\n## Workflow\n\n1. **Understand the change**\n   - Read the goal specification and implementation strategy\n   - Review the diff to understand what changed and why\n   - Run the test suite to confirm tests pass\n\n2. **Review dimensions**\n\n   a. **Correctness**\n   - Does the change correctly implement the goal?\n   - Are edge cases handled?\n   - Is error handling appropriate?\n   - Are there any off-by-one errors, null reference risks, or race conditions?\n\n   b. **Test Gaps**\n   - Does the change include adequate tests?\n   - Are edge cases tested?\n   - Are negative paths tested? (what happens on failure?)\n   - Is there test coverage for rollback\u002Frecovery?\n\n   c. **Security**\n   - Does the change introduce any OWASP Top 10 vulnerabilities?\n   - Are there injection risks (SQL, command, path)?\n   - Are secrets or credentials exposed?\n   - Is input validated at system boundaries?\n\n   d. **Performance**\n   - Are there obvious performance issues (N+1 queries, large allocations)?\n   - Is the algorithmic complexity appropriate for the data scale?\n   - Are there blocking operations that could be async?\n\n   e. **Maintainability**\n   - Is the code clear and self-documenting?\n   - Are names descriptive and consistent with the codebase?\n   - Is there unnecessary abstraction or duplication?\n   - Will a future developer understand this code?\n\n   f. **Observability**\n   - Are errors logged with enough context to debug?\n   - Are important state transitions observable?\n\n   g. **API Compatibility**\n   - Are public APIs changed? If so, is the change backward-compatible?\n   - Are deprecations properly signaled?\n\n3. **Produce review report**\n   - Summary: APPROVED \u002F CHANGES REQUESTED \u002F BLOCKED\n   - Per-dimension findings with severity (error\u002Fwarning\u002Finfo)\n   - Specific file:line references for each finding\n   - Recommendations for each issue\n\n## Required Outputs\n\n- Review report with all dimensions assessed\n- An overall verdict: APPROVED, CHANGES REQUESTED, or BLOCKED\n- For each finding: severity, location, and recommendation\n\n## Verification Expectations\n\n- Every finding must reference a specific file and line\n- BLOCKED verdict requires at least one error-level finding\n- APPROVED verdict means no error-level findings\n- Security findings are always at least warning severity\n\n## Safety Notes\n\n- This skill only reads files. It does not modify code.\n- If a security vulnerability is found, mark it as error severity.\n- Do not approve changes that delete or weaken existing tests.\n- If tests are missing for the changed behavior, request changes.\n","# Code Review\n\nPerform a structured review of code changes against quality dimensions.\n\n## Workflow\n\n1. **Understand the change**\n   - Read the goal specification and implementation strategy\n   - Review the diff to understand what changed and why\n   - Run the test suite to confirm tests pass\n\n2. **Review dimensions**\n\n   a. **Correctness**\n   - Does the change correctly implement the goal?\n   - Are edge cases handled?\n   - Is error handling appropriate?\n   - Are there any off-by-one errors, null reference risks, or race conditions?\n\n   b. **Test Gaps**\n   - Does the change include adequate tests?\n   - Are edge cases tested?\n   - Are negative paths tested? (what happens on failure?)\n   - Is there test coverage for rollback\u002Frecovery?\n\n   c. **Security**\n   - Does the change introduce any OWASP Top 10 vulnerabilities?\n   - Are there injection risks (SQL, command, path)?\n   - Are secrets or credentials exposed?\n   - Is input validated at system boundaries?\n\n   d. **Performance**\n   - Are there obvious performance issues (N+1 queries, large allocations)?\n   - Is the algorithmic complexity appropriate for the data scale?\n   - Are there blocking operations that could be async?\n\n   e. **Maintainability**\n   - Is the code clear and self-documenting?\n   - Are names descriptive and consistent with the codebase?\n   - Is there unnecessary abstraction or duplication?\n   - Will a future developer understand this code?\n\n   f. **Observability**\n   - Are errors logged with enough context to debug?\n   - Are important state transitions observable?\n\n   g. **API Compatibility**\n   - Are public APIs changed? If so, is the change backward-compatible?\n   - Are deprecations properly signaled?\n\n3. **Produce review report**\n   - Summary: APPROVED \u002F CHANGES REQUESTED \u002F BLOCKED\n   - Per-dimension findings with severity (error\u002Fwarning\u002Finfo)\n   - Specific file:line references for each finding\n   - Recommendations for each issue\n\n## Required Outputs\n\n- Review report with all dimensions assessed\n- An overall verdict: APPROVED, CHANGES REQUESTED, or BLOCKED\n- For each finding: severity, location, and recommendation\n\n## Verification Expectations\n\n- Every finding must reference a specific file and line\n- BLOCKED verdict requires at least one error-level finding\n- APPROVED verdict means no error-level findings\n- Security findings are always at least warning severity\n\n## Safety Notes\n\n- This skill only reads files. It does not modify code.\n- If a security vulnerability is found, mark it as error severity.\n- Do not approve changes that delete or weaken existing tests.\n- If tests are missing for the changed behavior, request changes.\n",{"name":5,"risk":53,"version":54,"description":7,"permissions":55,"when_to_use":57,"when_not_to_use":58},"low","1.0.0",[56],"read_files","Use after implementing a change, before merging a PR, or whenever you need\nan independent quality assessment of code changes.\n","Do not use as a replacement for human code review on critical paths.\nDo not use for changes that have not been tested.","main","ea8458ad5d16f32f8e464369b6dfb656b907bf0a",".agent\u002Fskills\u002Fcode-review","https:\u002F\u002Fgithub.com\u002Fstrings77wzq\u002Fgoal-run","https:\u002F\u002Fgithub.com\u002Fstrings77wzq","2026-05-31 10:20:34",{"source":8,"skillName":5,"commands":66,"prompt":76,"zipAvailable":77},[67,70,73],{"tool":68,"command":69},"npx","npx skills add strings77wzq\u002Fgoal-run --skill code-review",{"tool":71,"command":72},"bunx","bunx skills add strings77wzq\u002Fgoal-run --skill code-review",{"tool":74,"command":75},"pnpm","pnpm dlx skills add strings77wzq\u002Fgoal-run --skill code-review","安装技能，技能地址：https:\u002F\u002Fgithub.com\u002Fstrings77wzq\u002Fgoal-run\u002Fblob\u002Fmain\u002F.agent\u002Fskills\u002Fcode-review\u002FSKILL.md",true,[],[80,96,119,137,144,160],{"skillNo":81,"name":82,"nameZh":6,"description":83,"descriptionZh":6,"repositoryFullName":8,"ownerLogin":9,"ownerAvatarUrl":10,"ownerAccountNo":11,"ownerDisplayName":6,"repositoryNo":12,"starsCount":13,"licenseSpdxId":14,"licenseFileUrl":6,"sourceHtmlUrl":84,"categories":85,"keywords":89,"publishedAt":48},"01M71V4NKDTPEZPWE5NQX3AACH","my-skill","A custom GoalRun skill","https:\u002F\u002Fgithub.com\u002Fstrings77wzq\u002Fgoal-run\u002Fblob\u002Fmain\u002Ftemplates\u002Fskill\u002FSKILL.md",[86],{"categoryNo":87,"categoryName":88,"taxonomyType":20,"taxonomyTypeDesc":21},"01JBSEEDCAT000000000000023","Skill与智能体开发",[90,91,92,93,94,95,40],"my skill","my","custom","describe","when","read",{"skillNo":97,"name":98,"nameZh":6,"description":99,"descriptionZh":6,"repositoryFullName":8,"ownerLogin":9,"ownerAvatarUrl":10,"ownerAccountNo":11,"ownerDisplayName":6,"repositoryNo":12,"starsCount":13,"licenseSpdxId":14,"licenseFileUrl":6,"sourceHtmlUrl":100,"categories":101,"keywords":105,"publishedAt":48},"01M71V4K3CXN6KZBSJTZ8XN98B","tdd-change","Implement changes using strict test-driven development — red, green, refactor","https:\u002F\u002Fgithub.com\u002Fstrings77wzq\u002Fgoal-run\u002Fblob\u002Fmain\u002Fskills\u002Ftdd-change\u002FSKILL.md",[102],{"categoryNo":103,"categoryName":104,"taxonomyType":20,"taxonomyTypeDesc":21},"01JBSEEDCAT000000000000008","测试与质量",[106,45,107,108,42,109,110,111,112,33,113,114,37,115,116,39,40,117,118,41],"tdd","tdd change","development","green","test-driven","Existing","fixes","true","complete","tests","regression","test suite","only",{"skillNo":120,"name":121,"nameZh":6,"description":122,"descriptionZh":6,"repositoryFullName":8,"ownerLogin":9,"ownerAvatarUrl":10,"ownerAccountNo":11,"ownerDisplayName":6,"repositoryNo":12,"starsCount":13,"licenseSpdxId":14,"licenseFileUrl":6,"sourceHtmlUrl":123,"categories":124,"keywords":125,"publishedAt":48},"01M71V4G13PXZTXWRT067RS9WT","implementation-strategy","Plan implementation strategy for medium-to-high risk changes before writing code","https:\u002F\u002Fgithub.com\u002Fstrings77wzq\u002Fgoal-run\u002Fblob\u002Fmain\u002Fskills\u002Fimplementation-strategy\u002FSKILL.md",[],[126,127,128,39,94,112,33,129,113,130,131,40,118,41,132,133,30,134,135,136],"implementation strategy","strategy","implementation","updates","plan implementation","openspec","documentation","blast radius","auth","writing","risk",{"skillNo":138,"name":5,"nameZh":6,"description":7,"descriptionZh":6,"repositoryFullName":8,"ownerLogin":9,"ownerAvatarUrl":10,"ownerAccountNo":11,"ownerDisplayName":6,"repositoryNo":12,"starsCount":13,"licenseSpdxId":14,"licenseFileUrl":6,"sourceHtmlUrl":139,"categories":140,"keywords":143,"publishedAt":48},"01M71V4E2FSCAETRWN155NAG49","https:\u002F\u002Fgithub.com\u002Fstrings77wzq\u002Fgoal-run\u002Fblob\u002Fmain\u002Fskills\u002Fcode-review\u002FSKILL.md",[141,142],{"categoryNo":18,"categoryName":19,"taxonomyType":20,"taxonomyTypeDesc":21},{"categoryNo":23,"categoryName":24,"taxonomyType":25,"taxonomyTypeDesc":26},[28,30,29,46,47,45,35,31,33,34,113,36,37,38,39,40,41,42,43,44],{"skillNo":145,"name":146,"nameZh":6,"description":147,"descriptionZh":6,"repositoryFullName":8,"ownerLogin":9,"ownerAvatarUrl":10,"ownerAccountNo":11,"ownerDisplayName":6,"repositoryNo":12,"starsCount":13,"licenseSpdxId":14,"licenseFileUrl":6,"sourceHtmlUrl":148,"categories":149,"keywords":152,"publishedAt":48},"01M71V4CZGBGV3GZ4P6HW8STPR","valid-skill","A valid skill for testing purposes","https:\u002F\u002Fgithub.com\u002Fstrings77wzq\u002Fgoal-run\u002Fblob\u002Fmain\u002Fpackages\u002Fharness\u002Ftest\u002Ffixtures\u002Fskills\u002Fvalid-skill\u002FSKILL.md",[150,151],{"categoryNo":103,"categoryName":104,"taxonomyType":20,"taxonomyTypeDesc":21},{"categoryNo":87,"categoryName":88,"taxonomyType":20,"taxonomyTypeDesc":21},[153,154,155,156,33,39,40,41,157,95,158,159],"valid","valid skill","harness","static","write","Production","testing",{"skillNo":161,"name":162,"nameZh":6,"description":163,"descriptionZh":6,"repositoryFullName":8,"ownerLogin":9,"ownerAvatarUrl":10,"ownerAccountNo":11,"ownerDisplayName":6,"repositoryNo":12,"starsCount":13,"licenseSpdxId":14,"licenseFileUrl":6,"sourceHtmlUrl":164,"categories":165,"keywords":167,"publishedAt":48},"01M71V4C3HXW8CG0YCW5N0Y26Y","dangerous-skill","A skill with dangerous content","https:\u002F\u002Fgithub.com\u002Fstrings77wzq\u002Fgoal-run\u002Fblob\u002Fmain\u002Fpackages\u002Fharness\u002Ftest\u002Ffixtures\u002Fskills\u002Fdangerous-skill\u002FSKILL.md",[166],{"categoryNo":87,"categoryName":88,"taxonomyType":20,"taxonomyTypeDesc":21},[168,169,170,171,37,172,40,173,174,175,176],"dangerous","dangerous skill","a skill","with","commands","execute","shell","content","delete"]