senior-fullstack

フルスタック実装レビュー

使用するタイミング

機能がブラウザー、バックエンドサービス、データベースにまたがる場合や、既存のフルスタック実装に統合上の抜け漏れがないかレビューする場合に使用します。

ワークフロー

1. プロジェクトの実際の技術スタック、ビルドコマンド、規約を確認します。インターフェースの状態、API リクエスト、認可、データアクセス、レスポンス処理を通して、ユーザーの操作を 1 つ追跡します。
2. コードを変更する前に、リクエストとレスポンスの契約、権限チェック、入力検証、失敗時の状態を定義します。データベースのマイグレーションと外部への副作用は明示します。
3. 既存の依存関係を使用し、整合性のある最小限の範囲を実装します。インターフェースの表示、ドメインロジック、永続化を分離し、パラメーター化クエリとサーバー側の認可を使用します。
4. 変更した動作について、プロジェクトで定められたチェックを実行します。ルールにはユニットテスト、権限と永続化には統合テスト、操作全体にはブラウザーテストを実施します。実際に実行したチェックと、利用できなかった環境を記録します。
5. 必要に応じて、マイグレーション、ロールバック、リリースの計画を準備します。依存関係のインストール、破壊的なデータ変更、外部通知、デプロイには、それぞれの操作に応じた認可が必要です。

例

ユーザー：「請求設定ページを追加してください。」

アシスタント：既存の請求連携とアカウント権限を確認し、設定 API を定義して、読み込み中・空・エラーの状態を実装します。また、あるアカウントが別のアカウントの設定を変更できないことをテストします。プロバイダーへの書き込みとデプロイは、ローカルでの実装とは別に準備します。インターフェースのテストだけを目的として、有料サブスクリプションを作成しないでください。

同梱の参考資料

• 技術スタック確認チェックリスト
• 境界と契約のレビュー
• 実装と検証のワークフロー

ヘルパーの状態

残されている 3 つの Python ヘルパーは、未実装の過去のひな型です。これらは意図的にゼロ以外の終了コードで失敗し、ひな型の生成、品質分析、プロジェクトの認証は行いません。プロジェクトの実際のツールを使用してください。検出事項が空であることを、監査に問題がない根拠として報告しないでください。このスキルには、実行可能なインストール機能や本番環境へのデプロイ機能は含まれていません。

制限事項

• 技術スタックとプロバイダーの互換性は、実際のプロジェクトで確認する必要があります。
• 参考資料はレビュー用チェックリストであり、完全なアプリケーションのテンプレートではありません。
• 一部のテストに合格しても、セキュリティ、パフォーマンス、デプロイの準備が整っているとは限りません。
• コードレビューの依頼から、アカウント、請求、本番リソースを変更する権限があると判断しないでください。