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.

ソースを見る

使い方

  1. インストールコマンドをコピーし、自分の環境へ導入する前にソースを確認します。
  2. 必要なツール、認証情報、対応環境を原文で確認します。
  3. 入力と成果物を指定し、出力を資料と照合します。
npx skills add https://github.com/tyelf22/free-skills --skill code-review-companion

開始前の確認

  • 導入にはモデル契約、API枠、ファイルへのアクセス許可は含まれません。
  • スター数や導入数は活動の指標であり、独立した品質・安全性試験ではありません。
リポジトリの原文

見出し、例、コード、表、リンク、参照画像を含む原文を表示しています。

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.

タスクに関連する項目

同じリポジトリから

関連する Skills

すべての Skills