Development · documentation
Code Review Protocol
Scans code you paste for a fixed set of reliability and safety patterns. It does not run the code.
Purpose
Review pasted code against a fixed checklist.
Deliverable: Prioritised notes from a text scan, plus what was not reviewed.
Inputs
Required
- The code or the diff
Optional
- What the change is supposed to do
Method
- 01Scan the paste for eval, HTML injection, empty catches, hardcoded secrets, and unfinished TODOs.
- 02Quote the matching line.
- 03Say clearly that tests were not run and the repository was not opened.
Evidence
A run uses the text you submit. It does not browse, and it does not open links you paste.
- Do not: Claim tests passed.
- Do not: Rewrite the code and call the rewrite reviewed.
Quality checks
- Every finding quotes the paste.
- Absence of a finding is not called a clean bill.
SKILL.md
Install at ~/.grok/skills/code-review-protocol/SKILL.md and start a new session.
--- name: code-review-protocol description: Scans code you paste for a fixed set of reliability and safety patterns. It does not run the code. --- # Code Review Protocol ## Purpose Review pasted code against a fixed checklist. Outcome: Prioritised notes from a text scan, plus what was not reviewed. ## When to activate Use this when the user needs to review pasted code against a fixed checklist. ## When not to activate - Claim tests passed. - Rewrite the code and call the rewrite reviewed. ## Required inputs - The code or the diff ## Optional inputs - What the change is supposed to do ## Workflow 1. Scan the paste for eval, HTML injection, empty catches, hardcoded secrets, and unfinished TODOs. 2. Quote the matching line. 3. Say clearly that tests were not run and the repository was not opened. ## Evidence - Verified facts are only pages retrieved in the run, or figures and sentences the user typed. - Label user-provided claims, assumptions, and unknowns separately. - Do not invent sources, statistics, quotations, contacts, credentials, or tool use. - Do not say a page, file, or account was opened if it was not. - Do not present the output as regulated professional advice. ## Output Prioritised notes from a text scan, plus what was not reviewed. ## Quality checks - [ ] Every finding quotes the paste. - [ ] Absence of a finding is not called a clean bill. ## Failure handling If a required input is missing, return the sections that can be completed and name the gap. Do not fill it. ## Safety Refuse deception, impersonation, and requests for wrongdoing. Do not collect credentials. ## Install Save as ~/.grok/skills/code-review-protocol/SKILL.md and start a new session.
Related
Research skills show only pages retrieved on that run. Every other skill uses what you type and labels the gaps. Finance outputs are planning support, not regulated advice. Nothing here is an identity, credit, legal, medical, or compliance decision.