メインコンテンツまでスキップ

Reviewer Lens Taxonomy(レビュアー・レンズの分類)

Reviewer Lens は、既存のレビュー機構を「評価目的」の切り口で整理するための説明語です。新しいレジストリやスキーマ列ではなく、既存の実装領域別ルーターreview team ロールを評価目的へ写像する語彙として使います。

この文書の位置づけ

本文書は写像のための説明ドキュメントであり、コードやスキーマは変更しません。Lens はスキーマの enum やレジストリ ID へ昇格させず、doc レベルの語彙にとどめます。詳細な検討の経緯は Issue #1545 を参照してください。

由来 / Provenance

Lens という整理軸は、3 つの外部フレームワークから概念を輸入しています。名指しの参照(nominative)にとどめ、各フレームワークの推奨や提携は主張しません。

  • g-stack — 目的別の専門レビュー(CEO / engineering / design / DevEx など)
  • Superpowers — 仕様に対するレビューとトレーサビリティ
  • Matt Pocock skills — alignment 成果物の整合性レビュー
一次ソース未確認

上記 3 フレームワークの一次ソースは、本文書の作成時点で取得できていません。したがって内容は Issue #1545 の要約を正とします。原典を取得できた場合は、Lens 定義の過不足を再検証してください。

Lens とは何か / What a Lens Is

Lens は「どの評価関数でレビューするか」を表す切り口です。River Review には、すでに 2 つのテキソノミーが並存しています。

  • 実装領域別ルーター — skills/agent-skills/river-review-{architecture,code,docs,frontend,performance,security,testing} の 7 スキル
  • review team ロール — src/lib/reviewer-orchestrator.mjsREVIEWER_ROLES(6 ロール)

Lens は 3 つ目のテキソノミーではありません。上記 2 者を評価目的へ写像する読み替えの語です。新しいレビュアーを無制限に増やさず、目的・適用条件・期待成果が明確な場合にのみ Lens を足す方針をとります。

写像表 / Mapping Table

各 Lens を、既存のルーター・review team ロール・skills/registry.yaml の registry skill へ写像します。

Lens主な問い実装領域別ルーターreview team ロールregistry skill(代表例)充足度
engineering設計・保守性・既存構造・拡張性は妥当かriver-review-codebug-hunterexisting-pattern-conformance / async-correctness / nullability-contractCovered
security認証・権限・入力・データ保護に問題はないかriver-review-securitysecurity-scannersecurity-basic / secret-credential-scan / trust-boundaries-authzCovered
qa受入条件・境界値・異常系・回帰を検証したかriver-review-testingtest-gaptest-existence / coverage-gap / flaky-testCovered
designUX・視認性・操作性・a11y に問題はないかriver-review-frontendfrontend-reviewerdesign-system-component-reuse / design-token-enforcement / a11y-accessible-nameCovered
architecture境界・依存・データフロー・契約を壊さないかriver-review-architecture(専任ロールなし)architecture-boundaries / architecture-traceability / data-flow-state-ownershipCovered
operability監視・障害対応・設定・デプロイは安全かriver-review-performance(部分)ci-cd-reviewer(部分)operability-slo / failure-modes-observability / availability-architectureCovered
devexAPI・CLI・導入・デバッグ体験を損なわないかriver-review-docs(部分)(専任ロールなし)api-design / openapi-contract / api-compatibility / doc-hygieneCovered
releasemigration・互換性・rollback は安全か(専任ルーターなし)dependency-reviewer(部分)migration-rollout-rollback / migration-safety / api-versioning-compatCovered
product要求・利用価値・事業ルールに適合するか(専任ルーターなし)(専任ロールなし)(該当なし)Gap

充足度の読み方は次のとおりです。

  • Covered — ルーター・ロール・registry skill のいずれかの組み合わせで、評価目的を満たす写像先が存在する。
  • Gap — どの写像先も存在せず、必要になった時点で追加を検討する。
充足度は registry skill 込みで再計算済み(#1545 Phase 1.5 / U2)

本表の初版は 7 ルーターと 6 ロールだけを写像先として数えており、skills/registry.yaml の registry skill を勘定に入れていませんでした。そのため operability / release / devex / architecture が実態より低い Partial と表示され、「Gap が多いので Lens を増やすべきだ」という誤った投資判断を誘発する状態にありました。再計算後の真の Gap は product の 1 件のみです。専任ロールが不在でも registry skill が評価目的を満たすなら Covered と読みます。

横断 Lens: rationale / Cross-Cutting Lens

rationale(根拠追跡性)は、写像表の 1 行ではなく既存 Lens を横断する読み替えとして扱います(Issue #1783 Phase 0 の案 C)。How をコード、What をテスト、Why を Issue / Plan / PR、Why not をコメント / ADR へ配置する原則を、engineering / qa / architecture / product の各 Lens に重ねて適用します。

独立した review gate は追加しません。finding 分類 13 コードの一覧と写像先は、リポジトリ内の docs/review/rationale-traceability.md に一元化しています。severity の語彙は .claude/rules/review-core.md、レビュー方針は pages/reference/review-policy.md が引き続き正本であり、同ドキュメントはその派生物として写像を書くにとどまります。この Lens も他の Lens と同じく doc レベルの語彙であり、スキーマの enum やレジストリ ID へ昇格させません。

Gap の扱い / Handling Gaps

Gap の Lens(現時点では product のみ)は、真のギャップが運用で観測された場合にのみ registry skill として追加します。先回りしてレビュアーを増やすことはしません。product / design のようにコード外の成果物を必要とする Lens は、artifact 入力契約経由で成果物を受け取ります。

設計決定 / Design Decisions

  • Lens 単位の有効性集計(Issue #1545 Phase 3)では、lens を feedback entry の保存フィールドにしない。集計時に skillId から本文書の写像表経由で導出する。
  • 理由は、保存すると Issue #1574 契約4(content-addressed candidate ID)の冪等性と契約5(clusterKey 互換)が写像表の改訂で壊れるためである。

PlanGate との境界 / Boundary with PlanGate

Lens の語彙定義は River Review を SSoT とします。River Review は「レビュー観点の定義・実行・記憶」を担い、GO / NO-GO・停止・承認・merge は担いません。PlanGate(Issue #851)は River Review の findings を利用する側で、Lens を mode / risk の入力として参照します。二重定義を避けるため、Lens の語彙は本文書に一元化します。

関連ドキュメントは次のとおりです。

人間レビューの要点 / Notes for Human Review

  • 本文書は写像のみを扱い、新しい語彙・スキーマ・レジストリを増やさない。
  • 一次ソースは未確認のため、原典取得後に Lens 定義を再検証する。
  • review team ロールは SKILL.md と reviewer-orchestrator.mjs の二重管理である。本文書は写像にとどめ、二重管理を悪化させない。