tyelf22/free-skills

code-review-companion

Paste in a diff and get a structured review: bugs, edge cases, naming, and the one thing worth fixing before merge.

Vedi sorgente

Come usare

  1. Copia il comando e controlla il pacchetto prima di installarlo nel tuo ambiente agente.
  2. Controlla strumenti, credenziali e ambienti richiesti nel documento originale.
  3. Descrivi input e risultato desiderato; confronta l’output con le fonti.
npx skills add https://github.com/tyelf22/free-skills --skill code-review-companion

Prima di iniziare

  • L’installazione non include abbonamenti, quote API o accesso ai file.
  • Stelle e installazioni indicano attività, non un test indipendente di qualità o sicurezza.
Documento Skill originale

Contenuto dal repository con titoli, esempi, codice, tabelle, link e immagini preservati.

Code Review Companion

Paste in a diff and get a structured review: bugs, edge cases, naming, and the one thing worth fixing before merge.

{{input}} in the steps below is the user's input (Paste a diff or code snippet).

Steps

1. Scan for bugs

Read {{input}} line by line and flag anything that looks like a bug: logic errors, typos, off-by-ones, wrong comparisons. Quote the line.

2. Check edge cases

Identify unhandled inputs: null/undefined, empty collections, zero, negatives, unexpected types. List what's missing.

3. Review naming and clarity

Flag names, structures, or conditions that would confuse the next reader. Suggest a clearer alternative for each.

4. Pick the one fix that matters

Name the single issue that should block merging and explain its impact in one sentence.

Example

Paste a diff or code snippet: function getDiscount(user) { if (user.role = 'admin') return 1; return user.tier === 'gold' ? 0.2 : 0; }

Output:

CODE REVIEW

BUGS
• Line 2 uses `=` (assignment) not `===` — always sets role to 'admin' and returns true.

EDGE CASES
• No handling for missing `user.tier` — falls through to 0, which may be fine but isn't explicit.

NAMING
• `getDiscount` returning 1 (100%) for admins is surprising — consider `getDiscountRate` or a comment.

FIX FIRST
The `=` vs `===` bug — it grants every user an admin-level discount.

Correlati per attività

dallo stesso repository

Altri Skills

Tutti gli Skills