본문으로 건너뛰기
cccskills
무료GitHub에 공개

review-changes

Load when asked to review a local diff/change set for correctness, regressions, blast radius, affected flows, and missing tests using the knowledge graph; use `review-pr` for intent/checklist/merge-gate evidence.

한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.

설치 방법 보기

포함된 파일(1)

  • SKILL.md3.3 KB

SKILL.md(원문)

설치하기 전에 에이전트에게 전달되는 지시 내용을 확인할 수 있습니다.

<!-- ABOUTME: Two-axis risk-aware diff review using Fowler smells, flow, impact, and coverage evidence. --> <!-- ABOUTME: Evaluates Standards and Spec Fidelity independently so clean code does not mask missed requirements. -->

Review Changes: Two-Axis Code Review

Review local diffs across two independent, non-polluting axes:

  1. Axis 1 (Standards & Smells): Does the code adhere to Clean Architecture, repo rules, and the Fowler Smells Baseline?
  2. Axis 2 (Spec & Intent Fidelity): Does the diff faithfully satisfy the requested behavior without scope creep or missed edge cases?

The Fowler 12-Smell Baseline

Evaluate the diff against these 12 classic code smells:

Code SmellWhat It IsHow to Correct
Mysterious NameTypes, methods, or variables whose names do not reveal intent.Rename to match domain glossary.
Duplicated CodeSame logic shape recurring across multiple files/handlers.Extract to shared domain/application method.
Feature EnvyA method accessing another object's data more than its own.Move method onto the owning data/aggregate.
Primitive ObsessionRaw string/int standing in for domain concepts (e.g. email, status).Encapsulate in strongly typed value object/enum.
Data ClumpsSame 3+ parameters traveling together across methods.Bundle into a record or parameter object.
Shotgun SurgeryOne logical change forcing scattered edits across unrelated files.Consolidate cohesion in owning module.
Divergent ChangeOne class modified for multiple unrelated reasons.Split class by single responsibility.
Speculative GeneralityUnused abstractions, generic hooks, or parameters not needed yet.Delete; keep code minimal (YAGNI).
Message ChainsLong a.B.C.D property walks violating Demeter's Law.Hide navigation behind a method on first object.
Middle ManA class/handler that merely delegates without adding value.Cut middle man; call target directly.
Repeated SwitchesRepeated switch/if cascades on the same enum/type.Replace with polymorphism or strategy pattern.
Refused BequestA subclass ignoring or overriding most inherited behavior.Replace inheritance with composition.

Review Workflow

  1. Run detect_changes_tool to get risk-scored change analysis.
  2. Run get_affected_flows_tool to find impacted execution paths.
  3. For each high-risk symbol, run query_graph_tool with pattern="tests_for" to check test coverage.
  4. Run get_impact_radius_tool to understand blast radius.
  5. Report findings under two distinct headings: ## Standards & Code Smells and ## Spec & Intent Fidelity.

Token Efficiency Rules

  • ALWAYS start with get_minimal_context(task="<your task>") before any other graph tool.
  • Use detail_level="minimal" on all calls. Only escalate to "standard" when minimal is insufficient.
  • Target: complete any review/debug/refactor task in ≤5 tool calls and ≤800 total output tokens.

리뷰

아직 리뷰가 없습니다. 사용해 본 소감을 남겨 주세요.

댓글

로그인 없이 쓸 수 있습니다. 사용 후기, 질문, 작동하지 않은 환경 등을 편하게 남겨 주세요.

아직 댓글이 없습니다. 첫 댓글을 남겨 보세요.

보내면 바로 공개됩니다. 이메일 주소나 전화번호 같은 개인 정보는 쓰지 마세요. URL은 2개까지입니다.

0 / 1,000자

이 양식은 reCAPTCHA로 보호되며 Google 개인정보처리방침 및 서비스 약관이 적용됩니다.

같은 저장소의 스킬

Load for Blazor/MudBlazor UI changes involving forms, dialogs, focus, keyboard navigation, landmarks, ARIA, color contrast, RTL-safe styling, or WCAG 2.2 AA tests; not for backend-only changes.

한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.

islamu-ngo/Event82026년 10월 20일 업데이트

Load when a task asks to research, verify, compare, or look up framework/package behavior, release notes, standards, RFCs, CVEs, or unfamiliar APIs; use repository evidence first, then official docs, and not for codebase navigation alone.

한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.

islamu-ngo/Event82026년 10월 20일 업데이트

aspire

무료

Load for Aspire AppHost work: start/stop/wait resources, inspect dashboard/logs/traces, add integrations/resources, rebuild a service, run isolated worktrees, or diagnose Aspire orchestration; not for plain dotnet apps, container-only deployments, or cloud deployment.

한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.

islamu-ngo/Event82026년 10월 20일 업데이트

Load for authentication or authorization changes involving BFF cookies/tokens, JWT validation, claims/user ID extraction, policies, handler access checks, impersonation, or 401/403 bugs; not for UI affordance gating alone.

한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.

islamu-ngo/Event82026년 10월 20일 업데이트

Load for Blazor BFF server work involving YARP proxy routes, cookie sessions, access-token forwarding/refresh, downstream API calls, BFF handlers, or browser-to-API auth failures; not for client component rendering or API JWT validation alone.

한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.

islamu-ngo/Event82026년 10월 20일 업데이트

Load when editing Blazor `.razor.css`, scoped selectors, `::deep`, BEM class names, RTL/logical CSS properties, or styling nested MudBlazor components; not for global design tokens or non-Blazor CSS.

한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.

islamu-ngo/Event82026년 10월 20일 업데이트

islamu-ngo의 스킬 모두 보기

이 스킬의 문제 신고