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.
インストールする前に、エージェントに与えられる指示の中身を確認できます。
Review local diffs across two independent, non-polluting axes:
Evaluate the diff against these 12 classic code smells:
| Code Smell | What It Is | How to Correct |
|---|---|---|
| Mysterious Name | Types, methods, or variables whose names do not reveal intent. | Rename to match domain glossary. |
| Duplicated Code | Same logic shape recurring across multiple files/handlers. | Extract to shared domain/application method. |
| Feature Envy | A method accessing another object's data more than its own. | Move method onto the owning data/aggregate. |
| Primitive Obsession | Raw string/int standing in for domain concepts (e.g. email, status). | Encapsulate in strongly typed value object/enum. |
| Data Clumps | Same 3+ parameters traveling together across methods. | Bundle into a record or parameter object. |
| Shotgun Surgery | One logical change forcing scattered edits across unrelated files. | Consolidate cohesion in owning module. |
| Divergent Change | One class modified for multiple unrelated reasons. | Split class by single responsibility. |
| Speculative Generality | Unused abstractions, generic hooks, or parameters not needed yet. | Delete; keep code minimal (YAGNI). |
| Message Chains | Long a.B.C.D property walks violating Demeter's Law. | Hide navigation behind a method on first object. |
| Middle Man | A class/handler that merely delegates without adding value. | Cut middle man; call target directly. |
| Repeated Switches | Repeated switch/if cascades on the same enum/type. | Replace with polymorphism or strategy pattern. |
| Refused Bequest | A subclass ignoring or overriding most inherited behavior. | Replace inheritance with composition. |
detect_changes_tool to get risk-scored change analysis.get_affected_flows_tool to find impacted execution paths.query_graph_tool with pattern="tests_for" to check test coverage.get_impact_radius_tool to understand blast radius.## Standards & Code Smells and ## Spec & Intent Fidelity.get_minimal_context(task="<your task>") before any other graph tool.detail_level="minimal" on all calls. Only escalate to "standard" when minimal is insufficient.まだレビューはありません。使ってみた感想をお寄せください。
概要と使いどころ
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.
日本語の概要は準備中です。原文の説明を表示しています。
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.
日本語の概要は準備中です。原文の説明を表示しています。
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.
日本語の概要は準備中です。原文の説明を表示しています。
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.
日本語の概要は準備中です。原文の説明を表示しています。
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.
日本語の概要は準備中です。原文の説明を表示しています。
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.
日本語の概要は準備中です。原文の説明を表示しています。