river-review-code

id	river-review-code
name	river-review-code
description	一般コード品質のレビューエージェント。デフォルトのフォールバック先。 可読性、保守性、型安全性、ロギング等の個別スキルへルーティングする。
category	midstream
phase	["midstream"]
severity	minor
applyTo	["src/**/*.{ts,tsx,js,jsx,mjs,cjs}","app/**/*.{ts,tsx,js,jsx,mjs,cjs}","lib/**/*.{ts,tsx,js,jsx,mjs,cjs}","packages/**/*.{ts,tsx,js,jsx,mjs,cjs}","scripts/**/*.{ts,tsx,js,jsx,mjs,cjs}","runners/**/*.{ts,tsx,js,jsx,mjs,cjs}"]
inputContext	["diff","fullFile"]
outputKind	["findings","actions"]
tags	["code-quality","default","entry","routing"]
version	0.1.0
license	MIT

Code Quality Review（一般コード品質レビュー）

コードの可読性、保守性、型安全性を検証する。他の専門エージェントに該当しない場合のデフォルトフォールバック先。

When to Use / いつ使うか

• 一般的なコード変更のレビュー時
• 他の専門エージェント（architecture, security, performance, testing）に該当しない場合
• コード品質の総合的なチェックが必要な場合

Routing / ルーティング

キーワード	スキルID	説明
型, TypeScript, strict	typescript-strict	TypeScript strict モード準拠
null, undefined, optional	typescript-nullcheck	null 安全性チェック
非同期, await, Promise	async-correctness	非同期処理の正しさ検証
型駆動, 設計	type-driven-design	型駆動設計
ログ, 監視	logging-observability	ロギング・可観測性
自動化, 境界	review-automation-boundary	レビュー自動化の境界
コメント, トリアージ	review-comment-triage	レビューコメント分類
幻覚的参照, 実在確認	hallucinated-reference	新規参照の実在確認
簡素化, 整理, simplify	SIMPLIFY 観点（本 skill 内）	品質クリーンアップ4観点
破壊的操作, undo, 回復支援	UX-SAFEGUARD 観点（本 skill 内）	操作の安全装置2観点

UI/コンポーネント系のルーティング（a11y, デザインシステム, Next.js App Router 境界等）は river-review-frontend に一元化済み（#1462）。本ルーターからは移設し、二重発火を避けている。

デフォルト動作

• キーワード指定なし → 以下のヒューリスティクスで判定:
  • .ts/.tsxファイル → TypeScript strict + nullチェック
  • コンポーネントファイル → river-review-frontend も参照（a11y・デザインシステム観点は frontend 側が担当）
  • 設定ファイル → 型駆動設計チェック

Checklist / チェックリスト

一般コードレビューでは以下を確認する:

可読性

• 関数・変数の命名が意図を表現しているか
• 意図を伝えない広すぎる名前（data / info / manager / handler / util / current）、共有されていない略語、同一概念の別名（または別概念の同名）がないか