roblox-code-review

Roblox コードレビュー

Roblox のコードレビューを適切なドメインスキルに振り分け、構造化されたレポートを作成します。変更内容に応じて該当する観点を適用し、毎回すべてを確認する必要はありません。

読み込むタイミング

• Roblox／Luau コードのコードレビューを依頼された場合
• セキュリティ、パフォーマンス、ネットワーク、収益化、データ永続化の監査を依頼された場合
• リモート、データ保存、コード構成に関する Roblox のベストプラクティスについて質問された場合

クイックリファレンス

振り分け：各観点で読み込むスキル

観点	読み込むスキル
セキュリティ監査	roblox-security
リモートの検証	roblox-networking
データ永続化	roblox-data
サーバー間の状態	roblox-server-data
収益化	roblox-monetization
パフォーマンス	roblox-performance
Luau の正確性	roblox-luau-core、roblox-luau-types
アーキテクチャ	roblox-architecture

静的解析の限界

• 静的スキャンでは、数値を使った require 呼び出し、動的コードのマーカー、難読化を思わせる名前、クリーンアップ用トークン、重複ファイルを検出できます。これらは出所やレビュー上の判断材料であり、断定的な結論ではありません。
• 重大度を判断する前に、該当する関数、権限境界、到達可能性、ライフサイクルを追跡してください。
• 検索だけで判明した事項は、観察された実行時の動作、テスト結果、入手できない証拠と区別して記載してください。
• 垂直スライスレビュー： プレイヤー向けの変更では、入力 → UI／ワールドのフィードバック → リモートまたはシミュレーション → 権威ある状態 → 永続化 → クリーンアップの流れを追跡してください。実際の経路をレビューしてください。ファイルやクラスが同時に存在するだけでは、実行時に接続されている証拠にはなりません。

出力形式

1. READY / NOT READY
2. 重大なブロッカー（セキュリティ、データ損失、クラッシュ）
3. 警告（リーク、ボトルネック、非推奨 API）
4. 未検証のリスクと入手できない証拠
5. 具体的な修正方法を含む指摘事項

重大度：Critical / High / Medium / Low。各指摘事項には、ファイルと行番号、問題点、影響、最小限の適切な修正を記載してください。詳細なチェックは、振り分け先のドメインスキルが担当します。