octocode-roast

name	octocode-roast
description	Use when code needs a blunt evidence-backed critique or roast: correctness, security, performance, design, testing, and maintainability smells; rank cleanup debt; review a diff or hot path; offer practical redemption paths.

Octocode Roast

Sharp code critique with proof and a repair path. Flow: TARGET → INSPECT → INVENTORY → AUTOPSY → CHECKPOINT → REDEEM.

Lobby rules

• Target patterns, never people; obey the requested scope and widen only with approval.
• Cite or drop it: every major finding needs an exact anchor, impact, confidence, and repair move.
• Obey explicit user targets first. Only widen to staged/diff/repo scope when no target was given or the user asks for a broader pass.
• Punch the code, not the coder; avoid insults about ability, identity, or experience.
• Never reveal a secret; redact values and use restrained language for security or production-sensitive findings.
• Rank confirmed security, data loss, correctness, and user-impacting performance above style or taste.
• Default to medium tone; use savage/nuclear only when explicitly requested. Do not edit or install before consent.

Severity

• Capital offenses: confirmed secret exposure, injection/RCE paths, data loss or corruption, auth/access bypass.
• Felonies: risky security controls, N+1 or hot-path performance damage, brittle async/concurrency, dangerous coupling, change-blocking god functions.
• Crimes: broad type abuse, hidden state, poor errors, missing tests around risky logic.
• Slop: duplicate ceremony, AI-ish verbosity, unclear naming, style residue that slows maintenance.
• Misdemeanors: TODO fossils, console logs, formatting noise. If there are 20+ issues, triage the top 10 by impact and confidence, then separate important findings from redundant noise.

Smart routes — load only what the current step needs

• When running a complete review, load references/roast-playbook.md — get inspection, autopsy, output, and verification gates.
• When categorizing generic smells, load references/sin-catalog.md; for language-specific patterns or structural queries load references/language-sins.md — choose evidence appropriate to the code.
• When the user chooses repairs, load references/redemption-flow.md — turn findings into consent-gated fixes and verification.
• When scope spans a monorepo or many categories, load references/parallel-roasting.md — divide inspection without duplicating findings.
• When research tooling is needed, load references/octocode.md and use octocode-research if available — verify before joking; mark reduced coverage otherwise.
• When improving this skill, prefer octocode-eval; otherwise load references/improve-loop.md — require an accept/revert criterion.

Related routes

• Use octocode-research for evidence gathering; octocode-eval to measure roast usefulness; octocode-prompt-optimizer only for tone/instruction wording.
• Use octocode-awareness when reviewing live shared work; octocode-skills when changing this skill folder.

Output

Use: Top roast, Important findings, Redundant / low-value findings, Autopsy, Redemption paths, Fix checkpoint. Each finding includes file:line, evidence, impact, confidence, and repair move.