forge-security-review

Forge セキュリティレビュー

Forge に特化したホワイトボックスセキュリティレビューを実施し、悪用可能性、影響、証拠、修正ガイダンスを添えて検証済みの検出結果を報告します。

トークン効率を重視したデフォルト設定

トークン使用量を削減するため、デフォルトでマニフェストに基づくルーティングを使用します。すべてのルールファイルを最初に読み込まないでください。

ルールアセット

レビュー用ルールは、このスキルの assets/security-rules/ にパッケージされています。

• グローバルベースライン：assets/security-rules/_global-forge.mdc
• カテゴリーインデックス：assets/security-rules/forge-*/_index-*.mdc
• カテゴリー詳細チェック：assets/security-rules/forge-*/*.mdc

実行上の必須事項

このスキルが起動された場合：

1. このスキルのディレクトリから、まず静的解析を実行します。

• scripts/run_static_analysis.sh <forge-project-root-directory>
• Windows では .ps1 スクリプトを使用します

2. 詳細なコードレビューを始める前に、manifest.yml を読み込みます。
3. 最初に assets/security-rules/_global-forge.mdc を読み込みます。
4. マニフェストとコードの兆候に基づき、関連するカテゴリーインデックスルールのみを読み込みます。
5. 実際のコードパターンによって該当する検出ヒューリスティックが起動された場合に限り、詳細なサブルールを読み込みます。
6. 以下の項目について、証拠に基づくセキュリティレビューを実施します。
   • 認証／認可
   • インジェクションと入力検証
   • テナント分離とテナント間の情報漏えい
   • シークレットとストレージ
   • 外向き通信／リモート接続／CSP およびマニフェストの権限
   • 公開エントリーポイント（Web トリガー）
   • エージェントおよびその他の Forge セキュリティリスク
7. ユーザーが修正を明示的に依頼しない限り、アプリのコードを変更しません。
8. スキャン出力と生成されたアーティファクトはすべて security-audit-artifacts/ に書き込みます。

ルールのルーティングワークフロー

フェーズ 1：偵察（必須）

まず manifest.yml を読み込み、以下を抽出します。

• permissions.scopes
• permissions.external.fetch
• permissions.content.scripts
• modules（resolver/webtrigger/scheduledTrigger/rovo/etc.）
• remotes
• app.runtime.name

実行マップを作成します。

• UI モジュール → ブリッジ呼び出し → resolver／関数
• 外部エントリーポイント（Web トリガー、イベント、スケジュール）
• api.asUser() と api.asApp() の経路
• 外向き fetch の宛先

フェーズ 2：インデックスルールの選択（二段階読み込み）

必ず最初に読み込みます。

• assets/security-rules/_global-forge.mdc

次に、関連するカテゴリのインデックスルールのみを読み込みます。

シグナル	読み込むルール
スコープの有意な使用、変更、または asApp() の使用	assets/security-rules/forge-authn-authz/_index-authn-authz.mdc
webtrigger または scheduledTrigger モジュール	assets/security-rules/forge-webtrigger-entrypoints/_index-webtrigger-entrypoints.mdc
permissions.external.fetch または remotes	assets/security-rules/forge-egress-remotes/_index-egress-remotes.mdc
SQL API、または信頼できない入力がリゾルバーのシンクに到達する場合	assets/security-rules/forge-injection/_index-injection.mdc
マルチテナントのパターン、モジュール／グローバル状態、キャッシュの再利用	assets/security-rules/forge-tenant-isolation/_index-tenant-isolation.mdc
認証情報／トークン／シークレットの取り扱い	assets/security-rules/forge-secrets-storage/_index-secrets-storage.mdc
安全でない CSP、またはスコープ／設定の誤構成が疑われる場合	assets/security-rules/forge-manifest-config/_index-manifest-config.mdc
Rovo モジュール／アクション	assets/security-rules/forge-rovo-agents/_index-rovo-agents.mdc
基本的なログ記録／エラー処理／静的解析に関する懸念	assets/security-rules/forge-auditing/_index-auditing.mdc
依存関係／パッケージのリスクレビュー	assets/security-rules/forge-misc/_index-misc.mdc

サブルールの方針：

• インデックスを読んだ後は、コードで確認された検出ヒューリスティックに一致するサブルールのみを読み込みます。
• カテゴリ内のすべてのサブルールを事前に読み込まないでください。

フェーズ 3：分析と検証

読み込んだ各カテゴリについて、以下を行います。

1. 到達可能なエントリポイントを列挙します。
2. ソース → 検証／認可 → シンクを追跡します。
3. 根拠を示して悪用可能性を確認します。
4. 確認済みの指摘事項を CVSS v3.1 で評価します。

集中的なレビューのモード

ユーザーが範囲を絞ったレビュー（たとえば、認可のみ）を依頼した場合は、以下を読み込みます。

• グローバルベースライン
• 依頼されたカテゴリのインデックス
• そのカテゴリ内で一致するサブルールのみ

範囲外で明らかな重大な指摘事項が確認された場合も、引き続き言及します。

レビューワークフロー

1. 実行マップを作成します。
   • UI Kit／Custom UI のエントリポイント
   • ブリッジの呼び出しとリゾルバーハンドラー
   • api.asUser() ／ api.asApp() の呼び出し経路
   • 外部への送信／リモート、およびトリガーのエントリポイント
2. 各指摘事項について、ソース → 検証／認可 → シンクを追跡します。
3. 確認済みの脆弱性として分類する前に、悪用可能性を検証します。
4. 悪用できないものの強化が望ましい事項は、独立した「検証が必要」セクションに記載します。
5. 各問題について、ファイル単位の根拠と実用的なテストの手がかりを提示します。

静的解析モード

ユーザーがフルスキャンを依頼した場合は、以下から始まる完全なワークフローを実行します。

• assets/security-rules/forge-auditing/static-analysis-forge.mdc

利用可能な場合に想定されるツール：Semgrep、npm audit、Snyk、gitleaks。

出力要件

• Markdown 形式のセキュリティ監査レポートを提示します。
• 確認済みの悪用可能な指摘事項を、CVSS v3.1 の深刻度と影響度に基づいて並べます。
• 確認済みの各指摘事項について、以下を含めます。
  • CVSS ベクターと基本スコア
  • 深刻度の区分
  • 悪用可能性と影響
  • ファイルの根拠とソースからシンクまでの追跡
  • CWE マッピング
  • 具体的なコマンドを含む、再現可能な PoC／テスト手順
• 前提条件と根拠の不足を記載します。
• 脆弱性が存在する場合、スキャナーの検出件数だけを報告しないでください。

トリガーフレーズの例

• 「この Forge アプリのセキュリティをレビューしてください」
• 「Forge アプリのホワイトボックスセキュリティ監査を実施してください」
• 「このアプリに認可バイパスやテナント分離の問題がないか確認してください」
• 「この Forge コードベースの静的解析を全面的に実施してください」