code-review

코드 리뷰 — 버그, 로직 오류, dead code 검출

خيارات التثبيت

يُحدَّد Prompt الذي يراجع المصدر أولًا بشكل افتراضي. يمكنك التبديل إلى أمر مباشر أو تنزيل نسخة محلية.

التثبيت باستخدام Codex أو Claude انسخ هذا Prompt والصقه في Codex أو Claude أو مساعد آخر ليراجع صفحة Skill ويثبّتها لك.

يتجاوز الأمر المباشر Prompt المخصّص للمراجعة. افحص المصدر قبل تشغيله.

npx skills add https://github.com/JiHong88/suneditor --skill code-review

يبقى الأمر في سطر واحد. مرّر أفقيًا لمراجعته كاملًا قبل النسخ.

تفضّل نسخة محلية؟ نزّل الملفات المتاحة حاليًا لدى SkillsMP.

اقرأ SKILL.md وأي ملفات مرافقة يعرضها SkillsMP قبل أن تقرر التثبيت.

SKILL.md

تعليمات المصدر · معاينة للقراءة فقط

name	code-review
description	코드 리뷰 — 버그, 로직 오류, dead code 검출

Review the specified file(s) or current git diff for bugs, logic errors, and dead code.

Target: $ARGUMENTS (file path, or blank for current diff)

Steps:

If arguments provided, read those files. Otherwise run git diff to get current changes.
Exclude test/dev/ files from review — they are local dev-only and not review targets.
Analyze for:
- Logic bugs (off-by-one, null access, race conditions)
- Dead code (unreachable branches, unused variables)
- Missing edge cases
Report findings with file path and line numbers.
Do NOT fix anything — report only. Let the user decide what to fix.

Review guidelines — avoid false positives:

Null/guard checks: If a null check already exists upstream in the call chain (caller side), do NOT flag the callee for lacking its own null guard. Only flag if there is a realistic unguarded path.
Intentional patterns: If code looks intentional and consistent with surrounding patterns, do NOT report it as a bug. If unsure, note it as a confirmation question (e.g. "의도된 동작인지 확인: ...") rather than a bug.
Severity filtering: Only report issues that could actually cause incorrect behavior or crashes. Do not report style preferences, hypothetical future risks, or "would be nice" improvements.
staged/unstaged diff: Do not flag git staging state differences as findings.

수동 코드 수정 후 changes.md에 누락된 변경사항 추가

커밋 메시지를 자동 생성하고 커밋한다. 커밋 요청 시 반드시 이 스킬을 참조한다.

코드 수정 후 lint, 빌드, 검증, 테스트 파이프라인 실행

name	code-review
description	코드 리뷰 — 버그, 로직 오류, dead code 검출

Review the specified file(s) or current git diff for bugs, logic errors, and dead code.

Target: $ARGUMENTS (file path, or blank for current diff)

Steps:

If arguments provided, read those files. Otherwise run git diff to get current changes.
Exclude test/dev/ files from review — they are local dev-only and not review targets.
Analyze for:
- Logic bugs (off-by-one, null access, race conditions)
- Dead code (unreachable branches, unused variables)
- Missing edge cases
Report findings with file path and line numbers.
Do NOT fix anything — report only. Let the user decide what to fix.

Review guidelines — avoid false positives:

Null/guard checks: If a null check already exists upstream in the call chain (caller side), do NOT flag the callee for lacking its own null guard. Only flag if there is a realistic unguarded path.
Intentional patterns: If code looks intentional and consistent with surrounding patterns, do NOT report it as a bug. If unsure, note it as a confirmation question (e.g. "의도된 동작인지 확인: ...") rather than a bug.
Severity filtering: Only report issues that could actually cause incorrect behavior or crashes. Do not report style preferences, hypothetical future risks, or "would be nice" improvements.
staged/unstaged diff: Do not flag git staging state differences as findings.