agentsclimarketplace

Code review

Skill coreline-ai/claude-devflow-plugin/skills/code-review

Claude Code 개발 라이프사이클 스킬 번들: Plan·Parallelize·Test·Review·Ship | Claude Code dev lifecycle skill bundle: Plan, Parallelize, Test, Review, Ship

Install
npx -y skills add coreline-ai/claude-devflow-plugin --skill code-review

Assembled from the repository path, not quoted from the project. Check it against their README if it does not work.

One thing to look at

  • 1 stars1 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.

What its author says it does

Copied from the file, not written here

Dual-mode review skill — review implementation plans before coding (plan mode) or review code changes after implementation (code mode) with architecture, risk, and test analysis

SKILL.md

3.6 KB, as published. Nobody here has run it

Code Review

Overview

두 가지 모드를 제공하는 리뷰 스킬:

  • Plan mode: 코딩 전 구현 계획 리뷰
  • Code mode: 구현 후 코드 변경사항 리뷰

Inputs

  • $target: 모드 결정 기준
    • .md 파일 경로, implement_*, plan 키워드 → Plan mode
    • code, diff, 브랜치명, PR 번호 → Code mode
    • 모호하면 사용자에게 확인

Plan Mode

Process

  1. 대상 계획서와 관련 프로젝트 문서를 읽는다.
  2. 목표와 범위를 평문으로 재진술한다.
  3. references/plan-review-checklist.md를 기준으로 검토한다:
    • 아키텍처와 데이터 흐름
    • 소유권 경계, 계약, 의존성
    • 엣지케이스, 실패 모드, 보안, 성능, 롤백
    • 테스트 전략과 smoke 커버리지
  4. 우선순위가 매겨진 권고사항을 생성한다.
  5. 요청 시 계획서에 수용된 권고사항을 반영한다.

Plan Mode Output

  1. Verdict: ready / needs revision / blocked
  2. Top risks
  3. Architecture & contract notes
  4. Test coverage notes
  5. Required plan edits
  6. Recommended next action

Review Standards (Plan)

선호:

  • 암묵적 커플링보다 명시적 계약
  • 영리한 추상화보다 지루한 기술
  • 빅뱅 리라이트보다 점진적 변경
  • 수동 확인보다 결정적 테스트

경고:

  • 소유자가 불분명한 공유 파일
  • 후속 경로 없는 스텁
  • happy path만 커버하는 테스트
  • 숨겨진 마이그레이션/롤아웃 리스크
  • 관련 없는 기능이 섞인 계획
  • 정리로 위장한 범위 확장

Code Mode

Process

  1. 변경사항을 수집한다:
    • git diff (staged + unstaged)
    • git diff main...HEAD (브랜치 전체)
    • git show <commit> (특정 커밋)
  2. references/code-review-checklist.md를 기준으로 검토한다:
    • 정확성 (로직 에러, off-by-one, null 처리, 레이스 컨디션)
    • 보안 (입력 검증, 시크릿 노출, 인젝션, 안전하지 않은 eval)
    • 성능 (복잡도, 메모리, I/O 패턴, 캐싱)
    • 테스트 (커버리지, 엣지케이스, 회귀, 모킹 전략)
    • 스타일/컨벤션 (네이밍, 포맷, import 순서, 데드코드)
  3. 심각도별로 발견사항을 분류한다.

Code Mode Output

  1. Summary table: 파일별 변경 요약
  2. Findings: 심각도별 상세 발견사항
  3. Required fixes: 반드시 수정해야 하는 항목
  4. Suggestions: 권장 개선사항

Severity Levels

Level의미조치
critical버그, 보안 취약점, 데이터 손실 위험반드시 수정
warning잠재적 문제, 성능 이슈수정 권장
suggestion개선 가능한 부분고려
nit스타일, 포맷선택적

Keep looking

Skills are one crate of 328,083. Ordering is by how many stacks a row turns up in, so the top of any crate is what has actually been picked rather than what has the most stars.