$npx -y skills add Archive228/loopkit --skill owasp-reviewSecurity-review a diff against the OWASP Top 10. Use before merging anything that touches auth, input handling, queries, or external calls.
| 1 | # OWASP Review |
| 2 | Check the diff for each, with the exact line: |
| 3 | - **Injection** — any string-built SQL/shell/HTML. Demand parameterized queries / escaping. |
| 4 | - **Broken access control** — does it verify the user OWNS the resource, not just that they're logged in? |
| 5 | - **Auth** — secrets in code? tokens without expiry? password compare not constant-time? |
| 6 | - **SSRF** — user-controlled URL fetched server-side without an allowlist. |
| 7 | - **Sensitive data** — PII/secrets logged, returned in errors, or sent unencrypted. |
| 8 | - **Deserialization** — untrusted input into pickle/yaml.load/eval. |
| 9 | - **Dependency** — a new package with known CVEs or no maintenance. |
| 10 | Output: a list of {line, risk, fix}. If clean, say so explicitly. Never assume input is safe because "it comes from our frontend". |