mcpField
R
review-hog-perspective-performance-reliability

review-hog-perspective-performance-reliability

github.com/posthogテスト・品質95147 回インストール今日更新
リポジトリ

review-hog-perspective-performance-reliability について

Performance & Reliability の観点から PR のコードを評価するスキルです。リソース効率、エラー復旧、スケーラビリティ、運用準備を確認し、ロジックとセキュリティは別の観点に委ねます。

主な機能

  1. 01リソース使用とクエリの確認
  2. 02エラー処理と再試行のレビュー
  3. 03スケーラビリティとキャッシュの確認
  4. 04可観測性と運用面の確認
  5. 05コード調査用コマンド

活用シーン

  1. 01PR のパフォーマンスボトルネックの確認
  2. 02エラー経路に潜む信頼性リスクの発見
  3. 03非同期処理、キャッシュ、タイムアウトの確認

インストール

コマンドをコピーしてターミナルで実行します。

$npx skills add https://github.com/posthog/ai-plugin --skill review-hog-perspective-performance-reliability

インストール後、Claude Code で/review-hog-perspective-performance-reliabilityを入力して読み込まれたか確認します。

詳細

レビューの観点：パフォーマンスと信頼性

パフォーマンスと信頼性の観点から PR の変更部分をレビューします。このコードは本番環境で良好なパフォーマンスを発揮し、安定して動作するでしょうか？リソース効率、エラー処理と復旧、スケーラビリティのパターン、運用準備に注目してください。

これは、同じ変更部分を並行してレビューする複数の独立した観点のひとつです。ロジックとセキュリティは別の観点で扱います。担当範囲に集中し、他の観点でも報告される可能性があるかどうかを気にせず、見つけたパフォーマンスまたは信頼性の問題をすべて報告してください（重複は後で別の重複排除ステップで解消します）。

主な調査対象

  1. リソース効率

    • N+1 クエリの問題を特定する
    • データベースインデックスの不足を確認する
    • 不要な再レンダリングを探す（フロントエンド）
    • メモリリークを探す
    • バンドルサイズとインポートを確認する

  2. エラー処理と復旧

    • 必要な箇所に try / catch ブロックがあることを確認する
    • エラーが握りつぶされていないか確認する
    • 失敗時の再試行ロジックを検証する
    • エラーバウンダリ（React）を確認する
    • エラーメッセージの品質を確認する

  3. スケーラビリティのパターン

    • キャッシュの機会が見落とされていないか確認する
    • ページネーションの実装を確認する
    • 非同期にすべき同期処理を探す
    • リソースプールの枯渇リスクを特定する
    • 必要な箇所にレート制限があることを確認する

  4. 運用準備

    • ログの網羅性を確認する
    • メトリクス／監視のフックを確認する
    • タイムアウト設定を検証する
    • ヘルスチェックの網羅性を確認する
    • クリーンアップハンドラーを確認する

調査コマンド

  • ループ内のクエリを検索する：rg "for.*in|while" --type py -A 10 | rg "query|select|fetch"
  • エラー処理を確認する：rg "try:|except:|catch|finally" --type py --type js -B 2 -A 5
  • 非同期処理を検索する：rg "async|await|Promise|then\(" --type js --type ts -A 3
  • キャッシュを確認する：rg "cache|memoize|memo|useMemo" --type py --type js -A 3
  • タイムアウトを検索する：rg "timeout|deadline|ttl" --type py --type js -A 2
  • ログを確認する：rg "logger|log\.|console\." --type py --type js

注目する箇所

主に次の箇所に注目してください。

  • パフォーマンスに影響するコアアプリケーションコード
  • データベースのクエリファイルと ORM の使用箇所
  • API エンドポイントとリクエストハンドラー
  • レンダリングロジックを含むフロントエンドコンポーネント
  • バックグラウンドジョブの処理と非同期タスク
  • キャッシュの実装
  • ファイル I/O とネットワーク処理
  • 設定／ビルドファイル（タイムアウト、上限、バンドル最適化の設定）

テスト以外のファイルのみを対象に問題を検出してください。文脈の把握を除き、ベンダー／サードパーティ製ファイルと生成ファイルは対象外とします。

他の観点に任せる事項

  • ロジックと正確性のエラー → ロジックと正確性
  • セキュリティの脆弱性と API 契約の変更 → 契約とセキュリティ
  • コードスタイルやフォーマット → PostHog Review の対象外

主な確認事項

  • このコードは負荷の高い状況でもスケールするでしょうか？
  • 適切な復旧処理を伴って、エラーが適切に処理されるでしょうか？
  • 十分な可観測性（ログ、メトリクス）があるでしょうか？
  • リソースは効率的に使われているでしょうか？
  • ボトルネックやパフォーマンスが急激に低下する箇所はないでしょうか？
  • システムは障害に対して十分な耐性があるでしょうか？

有効な指摘とは

パフォーマンスと信頼性に関する指摘は、次のいずれかに関連します。

  • パフォーマンスのボトルネック（N+1、インデックス不足など）
  • エラー処理または復旧処理の不足
  • スケーラビリティの制限
  • リソースの非効率な使用
  • 可観測性の不足
  • 運用上の安全策の不足
  • 信頼性に関する懸念
重大度の目安
  • 必ず修正：本番環境の停止や深刻な性能低下を引き起こす可能性があります
  • 修正を推奨：パフォーマンスに目立った影響がある、または信頼性のリスクがあります
  • 検討：軽微な最適化や、あると望ましい改善です

コメント

Markdown 対応