rafter-code-review

Guides structured security code reviews using OWASP, ASVS, and MITRE CWE checklists.

27|13|Updated Jul 25, 2025
One-click install
npx skills add https://github.com/Raftersecurity/rafter-cli --skill rafter-code-review-raftersecurity
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: rafter-code-review
Source: https://github.com/Raftersecurity/rafter-cli/tree/main/python/rafter_cli/resources/skills/rafter-code-review
Command: npx skills add https://github.com/Raftersecurity/rafter-cli --skill rafter-code-review-raftersecurity

SYSTEM DOCUMENTATION & REQUIREMENTS

💡 This Skill includes references (resource) components.

What problem does it solve? Manual security review of code diffs is inconsistent: reviewers miss injection paths, broken object-level authorization, insecure deserialization, and LLM-specific risks because no single checklist covers every application type. This Skill provides question-driven walkthroughs keyed to the actual security surface of a diff, so reviewers systematically check what compilers and test suites cannot catch. ## Core Features & Use Cases - Category-based review walkthroughs: Separate reference docs for web applications (OWASP Top 10), APIs (OWASP API Security Top 10), LLM-integrated features (OWASP LLM Top 10), and CLI/library/IaC code (MITRE CWE Top 25 keyed by language). - Risk-tiered depth selection: An ASVS guide helps pick L1/L2/L3 verification depth based on the sensitivity of the data flows in the diff, with spot-check questions per category. - Single-finding investigation playbook: Canonical questions for reachability, authorization coverage, data flow, trust boundaries, concurrency, and secrets lifecycle when following up on one suspicious pattern. - Use Case: Before merging a pull request that adds a new REST endpoint touching user data, walk the API checklist to verify per-object authorization, rate limiting, and mass-assignment protections, citing file:line evidence for each finding. ## Quick Start Ask the AI to review the current diff for security issues using the rafter-code-review checklist matching the type of code changed.

Frequently Asked Questions about rafter-code-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I do a security code review on a pull request?▼

Identify the security surface of the diff (user input, SQL, shell, auth, secrets, file paths, deserialization, crypto, network endpoints), then walk the matching checklist: web-app, API, LLM, or CWE Top 25. Cite file:line evidence for each question and pair with automated SAST and secrets scanning.

What OWASP checklist should I use for REST API review?▼

Use the OWASP API Security Top 10 (2023), covered in the api.md reference. It walks broken object level authorization (BOLA), broken authentication, mass assignment, rate limiting, function-level authorization, SSRF, and unsafe third-party API consumption as per-endpoint questions.

How do I review LLM or AI agent code for security issues?▼

Walk the OWASP LLM Top 10 checklist in the llm.md reference. It covers prompt injection tracing, sensitive data in prompts, tool-call argument validation, excessive agency with human-in-the-loop gates, RAG corpus write access, and token budget limits.

When can I skip a full security review of a code change?▼

Skip the full walk when the diff touches none of the security surface: no user input, SQL, shell, auth, credentials, file paths, serialization, crypto, network calls, deletion, or dependency changes. Research or local-only code qualifies only if it truly has none of that surface.

Does this checklist replace automated SAST or secrets scanning?▼

No. The skill explicitly requires pairing manual review with automated scanning: rafter secrets for credential leaks and rafter run for SAST/SCA. The checklists catch design and context-dependent issues that scanners miss, and scanner findings are cross-referenced against the manual walk.

How do I choose between ASVS L1, L2, and L3 verification levels?▼

Pick the level matching the highest-sensitivity data flow in scope: L1 for low-value internal tools, L2 for apps handling PII, payments, or B2B tenant data, and L3 where compromise causes real harm such as financial transactions or healthcare records. Then spot-check three questions per ASVS category rather than walking all 280+ requirements.