Discuss Code

Discuss code critically before or instead of editing. Use when user asks for a compact opinion on code, a diff, snippet, design, naming, logic, simplicity, structure, or maintainability, and wants findings plus smallest useful fixes. For formal risk-ranked pull request review use code-reviewer; for architecture candidates use architecture-review; for actual cleanup edits use code-simplifier or refactoring.

flpbalada 9806ff6 2 files · 2.1 KB Updated

File contents

Discuss Code

Review code critically. Focus on real issues. Keep output compact.

Goal

  • Find concrete problems.
  • Explain impact plainly.
  • Suggest smallest useful fix.
  • Avoid praise unless there are no findings.

Rules

  • Findings first.
  • Highest impact first.
  • Max 5 findings.
  • No speculative issues without evidence.
  • Reference file and line when possible.
  • Show code only when it clarifies fix.

Check

  • Correctness.
  • Unneeded logic.
  • Duplication.
  • Weak abstraction.
  • Nesting and complexity.
  • Naming.
  • Dead code.
  • Missed simple pattern.

Flow

  1. Read user-provided code or target.
  2. Identify behavior and intent.
  3. Check for bugs and maintainability risks.
  4. Rank findings by impact.
  5. Suggest minimal fixes.

Output

Verdict: good | mixed | bad

## Finding N: short title
What: concrete issue
Why: impact or risk
Suggestion: smallest useful fix
Reference: `file:line` or snippet

If no findings:

Verdict: good
No findings.
Residual risk: [missing context or test gap]

Related skills

  • code-architecture-wrong-abstraction
  • typescript-best-practices
  • react-useeffect-avoid
  • naming-cheatsheet

flpbalada/fb-skills/tree/main/skills/_archive/discuss-code commit 9806ff6610

Frequently asked questions

npx skillmds@latest add flpbalada/discuss-code