{"id":"ee5e3deb-cb2e-411b-8465-f57275d42fb5","shortId":"7BhH3Y","kind":"skill","title":"git-review","tagline":"Review code changes, pull requests, or merge requests for bugs, code quality, security, and maintainability. Use when asked to review a PR, MR, diff, or set of changes. Also handles GitHub PRs and GitLab MRs.","description":"## 1. Fetch the changes\n\nIf a GitHub PR URL or number is provided, fetch details and diff via `gh`:\n\n```\ngh pr view <number-or-url> --json title,body,additions,deletions,files,reviews,statusCheckRollup,headRefName,baseRefName\ngh pr diff <number-or-url>\n```\n\nIf `gh` cannot access the repository, a GitLab MR is referenced, or no URL is given, ask the user to paste the diff directly. A local diff also works:\n\n```\ngit diff main...<branch>\n```\n\n## 2. Check CI status\n\nIf fetched via `gh`, check `statusCheckRollup`. If any checks are failing or still pending, inform the user and stop — do not review until CI is green.\n\n## 3. Review the changes\n\nReview the diff thoroughly and critically. Do not just validate what is there — look for what is wrong or missing.\n\nCover the following areas, skipping any that are not relevant:\n\n- **Bugs and logic errors:** incorrect assumptions, edge cases, off-by-one errors, unhandled states\n- **Code quality:** readability, naming, unnecessary complexity, duplication\n- **Security:** injection risks, exposed secrets, improper input handling, unsafe defaults\n- **Performance:** unnecessary work, inefficient patterns, missing caching where appropriate\n- **Correctness:** does the implementation match the intent described in the title or description\n\nGroup feedback by file or concern. For each issue, state what the problem is and suggest a concrete fix. If something is genuinely good or well-handled, you may note it briefly, but focus on what needs attention.\n\nBe direct. Do not soften criticism or add filler. If the changes look fine, say so plainly.\n\n## 4. Post feedback\n\nIf the review was fetched via `gh` and there are issues, post a review comment:\n\n```\ngh pr review <number> --request-changes --body \"...\"\n```\n\nIf the changes look good, say so and leave approval to the user.","tags":["git","review","dotfiles","grimlink","agent-skills","ai-tools","bash","homebrew","laravel-valet","macos","magento2","nodejs"],"capabilities":["skill","source-grimlink","skill-git-review","topic-agent-skills","topic-ai-tools","topic-bash","topic-dotfiles","topic-git","topic-homebrew","topic-laravel-valet","topic-macos","topic-magento2","topic-nodejs","topic-php","topic-terminal"],"categories":["dotfiles"],"synonyms":[],"warnings":[],"endpointUrl":"https://skills.sh/GrimLink/dotfiles/git-review","protocol":"skill","transport":"skills-sh","auth":{"type":"none","details":{"cli":"npx skills add GrimLink/dotfiles","source_repo":"https://github.com/GrimLink/dotfiles","install_from":"skills.sh"}},"qualityScore":"0.454","qualityRationale":"deterministic score 0.45 from registry signals: · indexed on github topic:agent-skills · 8 github stars · SKILL.md body (1,821 chars)","verified":false,"liveness":"unknown","lastLivenessCheck":null,"agentReviews":{"count":0,"score_avg":null,"cost_usd_avg":null,"success_rate":null,"latency_p50_ms":null,"narrative_summary":null,"summary_updated_at":null},"enrichmentModel":"deterministic:skill-github:v1","enrichmentVersion":1,"enrichedAt":"2026-05-18T19:08:54.795Z","embedding":null,"createdAt":"2026-05-18T13:14:34.753Z","updatedAt":"2026-05-18T19:08:54.795Z","lastSeenAt":"2026-05-18T19:08:54.795Z","tsv":"'1':39 '2':106 '3':136 '4':280 'access':77 'add':270 'addit':64 'also':32,101 'appropri':210 'approv':314 'area':163 'ask':21,90 'assumpt':175 'attent':262 'baserefnam':70 'bodi':63,304 'briefli':256 'bug':13,170 'cach':208 'cannot':76 'case':177 'chang':6,31,42,139,274,303,307 'check':107,114,118 'ci':108,133 'code':5,14,185 'comment':297 'complex':190 'concern':229 'concret':241 'correct':211 'cover':160 'critic':145,268 'default':201 'delet':65 'describ':218 'descript':223 'detail':53 'diff':27,55,73,96,100,104,142 'direct':97,264 'duplic':191 'edg':176 'error':173,182 'expos':195 'fail':120 'feedback':225,282 'fetch':40,52,111,287 'file':66,227 'filler':271 'fine':276 'fix':242 'focus':258 'follow':162 'genuin':246 'gh':57,58,71,75,113,289,298 'git':2,103 'git-review':1 'github':34,45 'gitlab':37,81 'given':89 'good':247,309 'green':135 'group':224 'handl':33,199,251 'headrefnam':69 'implement':214 'improp':197 'incorrect':174 'ineffici':205 'inform':124 'inject':193 'input':198 'intent':217 'issu':232,293 'json':61 'leav':313 'local':99 'logic':172 'look':153,275,308 'main':105 'maintain':18 'match':215 'may':253 'merg':10 'miss':159,207 'mr':26,82 'mrs':38 'name':188 'need':261 'note':254 'number':49 'off-by-on':178 'one':181 'past':94 'pattern':206 'pend':123 'perform':202 'plain':279 'post':281,294 'pr':25,46,59,72,299 'problem':236 'provid':51 'prs':35 'pull':7 'qualiti':15,186 'readabl':187 'referenc':84 'relev':169 'repositori':79 'request':8,11,302 'request-chang':301 'review':3,4,23,67,131,137,140,285,296,300 'risk':194 'say':277,310 'secret':196 'secur':16,192 'set':29 'skill' 'skill-git-review' 'skip':164 'soften':267 'someth':244 'source-grimlink' 'state':184,233 'status':109 'statuscheckrollup':68,115 'still':122 'stop':128 'suggest':239 'thorough':143 'titl':62,221 'topic-agent-skills' 'topic-ai-tools' 'topic-bash' 'topic-dotfiles' 'topic-git' 'topic-homebrew' 'topic-laravel-valet' 'topic-macos' 'topic-magento2' 'topic-nodejs' 'topic-php' 'topic-terminal' 'unhandl':183 'unnecessari':189,203 'unsaf':200 'url':47,87 'use':19 'user':92,126,317 'valid':149 'via':56,112,288 'view':60 'well':250 'well-handl':249 'work':102,204 'wrong':157","prices":[{"id":"d3336610-c72f-44a4-afd3-3f1155b4aceb","listingId":"ee5e3deb-cb2e-411b-8465-f57275d42fb5","amountUsd":"0","unit":"free","nativeCurrency":null,"nativeAmount":null,"chain":null,"payTo":null,"paymentMethod":"skill-free","isPrimary":true,"details":{"org":"GrimLink","category":"dotfiles","install_from":"skills.sh"},"createdAt":"2026-05-18T13:14:34.753Z"}],"sources":[{"listingId":"ee5e3deb-cb2e-411b-8465-f57275d42fb5","source":"github","sourceId":"GrimLink/dotfiles/git-review","sourceUrl":"https://github.com/GrimLink/dotfiles/tree/main/skills/git-review","isPrimary":false,"firstSeenAt":"2026-05-18T13:14:34.753Z","lastSeenAt":"2026-05-18T19:08:54.795Z"}],"details":{"listingId":"ee5e3deb-cb2e-411b-8465-f57275d42fb5","quickStartSnippet":null,"exampleRequest":null,"exampleResponse":null,"schema":null,"openapiUrl":null,"agentsTxtUrl":null,"citations":[],"useCases":[],"bestFor":[],"notFor":[],"kindDetails":{"org":"GrimLink","slug":"git-review","github":{"repo":"GrimLink/dotfiles","stars":8,"topics":["agent-skills","ai-tools","bash","dotfiles","git","homebrew","laravel-valet","macos","magento2","nodejs","php","terminal","zsh"],"license":null,"html_url":"https://github.com/GrimLink/dotfiles","pushed_at":"2026-05-12T18:40:37Z","description":"🔧 My DEV env setup","skill_md_sha":"37c2c07ee09e527095c65bf330ac7674644f7008","skill_md_path":"skills/git-review/SKILL.md","default_branch":"main","skill_tree_url":"https://github.com/GrimLink/dotfiles/tree/main/skills/git-review"},"layout":"multi","source":"github","category":"dotfiles","frontmatter":{"name":"git-review","description":"Review code changes, pull requests, or merge requests for bugs, code quality, security, and maintainability. Use when asked to review a PR, MR, diff, or set of changes. Also handles GitHub PRs and GitLab MRs."},"skills_sh_url":"https://skills.sh/GrimLink/dotfiles/git-review"},"updatedAt":"2026-05-18T19:08:54.795Z"}}