osv-scanner-mcp 0.3.3 → 0.3.4

package/README.md

@@ -165,6 +165,8 @@ Java(Maven)プロジェクトをスキャンし、既知の脆弱性レポート
 
 > **Gradleプロジェクトについて**: 本ツールは**lockfile方式**のみ対応です(ビルド実行方式は build.gradle の任意コード実行を伴うため、セキュリティ上の理由から採用していません)。`gradle.lockfile` が無い場合は `./gradlew dependencies --write-locks` で生成してください(依存ロック未設定の場合は `build.gradle` に `dependencyLocking { lockAllConfigurations() }` の追加が必要です)。
 
+> **スキャン範囲**: ディレクトリを指定すると、配下の `pom.xml` / `gradle.lockfile` / `buildscript-gradle.lockfile` を深さに関係なく検出し、**検出したファイルだけ**をスキャンします(応答の `manifests` がそのままスキャン範囲です)。同じディレクトリにある `package-lock.json` や `requirements.txt` などJava以外のファイルはスキャンしません。`.git`、`node_modules`、`target`、`build`、`.idea`、`.vscode` とシンボリックリンクは探索しません。探索するエントリが20万件、またはマニフェストが1,000件を超える場合は、結果を黙って省略せず `manifest_search_limit_exceeded` を返します。`pom.xml` などのマニフェストを直接指定した場合は、ディレクトリを探索せず**そのファイルだけ**をスキャンします(上限に達した場合の回避手段としても使えます)。
+
 **出力(成功時)**
 
 ```json
@@ -294,6 +296,7 @@ OSV-Scanner 2.4.0の `java/archive` プラグインを使用し、ネストJAR
 - MavenのOSV `ECOSYSTEM` 範囲(`introduced` / `fixed` / `last_affected` / 上限なし)を照合します。`versions` に明示された影響も確認します。範囲欠落・不正・未対応形式・`limit` による不完全な情報では安全と推定せず、候補を検証できなければ `recommended_upgrade: null`、`verification: "no_verified_candidate"` を返します。
 - 推奨時は `verification: "verified"`、CVEごとの `recommended_status` は `affected` / `not_affected` / `unknown` です。推奨保留時は `not_evaluated` になります。`per_cve_detail.fixed_in` は各CVE単独の候補であり、最終推奨先の判定は `recommended_status` を参照してください。
 - 現在より新しい修正版候補がないCVEは `tier: "unfixed"` として推奨の修正対象から除外します(情報欠落を含む場合があります)。除外したCVEも推奨先で判定し、その状態を表示します。全CVEがunfixedの場合も `recommended_upgrade` は `null` です。
+- 修正版の推奨はMavenのみ対応です。Maven以外のパッケージは `verification: "unsupported_ecosystem"`、CVEごとの `tier: "unsupported"` を返し、`unfixed` には数えません(修正版の有無は判定していないため。修正版は `explain_vulnerability` で確認できます)。
 
 ### `explain_vulnerability`
 
@@ -317,7 +320,7 @@ OSV-Scanner 2.4.0の `java/archive` プラグインを使用し、ネストJAR
 {
   "error": {
     "kind": "no_manifest_found",
-    "message": "対応マニフェスト(pom.xml / gradle.lockfile)が見つかりません(深さ3まで探索): /path/to/project"
+    "message": "対応マニフェスト(pom.xml / gradle.lockfile)が見つかりません: /path/to/project"
   }
 }
 ```
@@ -327,6 +330,7 @@ OSV-Scanner 2.4.0の `java/archive` プラグインを使用し、ネストJAR
 | `binary_not_found` | OSV-Scannerが見つからない(インストール案内をmessageに含む) |
 | `project_not_found` | 指定パスが存在しない・ディレクトリ/pom.xmlでない |
 | `no_manifest_found` | 対応マニフェスト(pom.xml / gradle.lockfile)が見つからない |
+| `manifest_search_limit_exceeded` | マニフェスト探索が上限(20万エントリ・1,000マニフェスト)に達した。より狭いディレクトリかマニフェストを直接指定する |
 | `binary_download_failed` | バイナリのダウンロード失敗(未対応プラットフォーム含む) |
 | `binary_checksum_mismatch` | ダウンロードしたバイナリのチェックサム不一致(改ざん/破損の可能性) |
 | `gradle_lockfile_missing` | Gradleプロジェクトだがgradle.lockfileが無い(生成手順をmessageで案内) |
@@ -348,7 +352,7 @@ OSV-Scanner 2.4.0の `java/archive` プラグインを使用し、ネストJAR
 
 - **サプライチェーン対策**: バイナリの自動ダウンロードは公式GitHub Releasesに限定し、バージョンをピン留め。**パッケージに埋め込まれたSHA256チェックサム**で検証します(配布元のSHA256SUMSファイルは信用しないため、リリース側が改ざんされても検出可能)。検証合格まで実行権限を与えず、キャッシュ済みバイナリも使用のたびに再検証します。`OSV_MCP_PREFER_DOWNLOAD=1` でPATH上の未検証バイナリを使わない運用も選べます
 - **コマンドインジェクション対策**: シェルを経由しない `spawn` + 引数配列で実行。OSV-Scannerへの引数は固定リストのみで、可変部は検証済み絶対パス1つだけ
-- **パストラバーサル対策**: 入力パスは `realpath` でシンボリックリンク解決後に境界チェック。pom.xml探索ではシンボリックリンクを辿りません
+- **パストラバーサル対策**: 入力パスは `realpath` でシンボリックリンク解決後に境界チェック。pom.xml探索ではシンボリックリンクを辿りません。OSV-Scannerにはディレクトリを渡さず、検出したマニフェストだけを形式を明示して個別に渡します(ディレクトリを渡すと、OSV-Scannerが同じディレクトリの `requirements.txt` も読み、その取り込み指定 `-r ../x.txt` でスキャン範囲の外のファイルを読むため)
 - **DoS対策**: タイムアウト・stdout上限・stderr抜粋上限を設定。スキャン結果は防御的にパースし、形式不正でも例外を投げません。同時実行スキャン数も上限(デフォルト2)を設け、並列リクエストによるプロセスの無制限起動を防ぎます
 - **fail-closedな運用モード**: `OSV_MCP_REQUIRE_ALLOWED_ROOT=1` で、スキャン許可ルート未設定時にサーバーの起動自体を拒否できます
 - **通信先の固定と明示**: osv-scannerの依存解決先は `deps.dev` を明示指定し、スキャン対象のpom.xmlが指定する任意のリポジトリへ接続するモード(`--data-source native`)は使いません(テストで保証)。通信先の一覧と、deps.devへの送信を止める `OSV_MCP_NO_REMOTE_RESOLUTION=1` は[通信先とプライバシー](#通信先とプライバシー)を参照

package/dist/index.js

@@ -35,7 +35,7 @@ if (startupWarning !== null) {
 // NOTE: リリース時はpackage.jsonのversionと同じ値に更新すること
 const server = new McpServer({
     name: "osv-scanner-mcp",
-    version: "0.3.3",
+    version: "0.3.4",
 });
 server.registerTool("scan_java_project", {
     title: "Javaプロジェクトの脆弱性スキャン",

package/dist/osv/runner.js

@@ -4,7 +4,10 @@
  * セキュリティ設計(docs/DESIGN_TODO.md):
  * - シェルを経由しない`spawn`+引数配列で実行(コマンドインジェクション対策)
  * - OSV-Scannerへ渡す引数は固定リストのみ。呼び出し側から任意フラグは注入できない
- *   (プロジェクトは検証済みディレクトリ1つ、実体スキャンは列挙済みのJAR/WAR絶対パスだけを渡す)
+ *   (プロジェクトは検出済みマニフェストを`--lockfile <形式>:<絶対パス>`で個別に、
+ *   実体スキャンは列挙済みのJAR/WAR絶対パスだけを渡す)
+ * - プロジェクトにディレクトリ(`-r`)を渡さない。osv-scannerがディレクトリ内の
+ *   requirements.txt等も読み、その`-r ../x.txt`の取り込みでスキャン範囲の外のファイルを読むため
  * - SBOMは検証・サイズ制限済みの専用一時コピー1つだけを渡す
  * - タイムアウトと出力サイズ上限を設ける(ハング・巨大出力によるDoS対策)
  *
@@ -12,6 +15,7 @@
  *   0 = スキャン成功・脆弱性なし / 1 = スキャン成功・脆弱性あり / 128 = 対象パッケージなし
  */
 import { spawn } from "node:child_process";
+import path from "node:path";
 import { ScanToolError } from "../errors.js";
 import { resolveOsvScannerBinary } from "./binaryManager.js";
 import { parseOsvScanOutput } from "./scanReport.js";
@@ -53,8 +57,20 @@ export function isRemoteResolutionDisabled(options = {}) {
  * `--data-source native` はスキャン対象pom.xmlの<repositories>に書かれた任意のURLへ
  * 接続するため使わない。既定値の変更に備えてdeps.devを明示する。
  */
-const FIXED_SCAN_ARGS = ["scan", "source", "-r", "--format", "json", "--data-source", "deps.dev"];
+const FIXED_SCAN_ARGS = ["scan", "source", "--format", "json", "--data-source", "deps.dev"];
 const NO_RESOLVE_ARG = "--no-resolve";
+/** 個別に渡せるマニフェスト。ファイル名をそのまま解析形式として明示する(2.4.0で実機確認) */
+const PROJECT_MANIFEST_FORMATS = new Set(["pom.xml", "gradle.lockfile", "buildscript-gradle.lockfile"]);
+/** 検出済みマニフェストを`--lockfile <形式>:<絶対パス>`の組にする。対象外のパスは渡さない */
+export function buildProjectTargetArgs(manifestPaths) {
+    return manifestPaths.flatMap((manifestPath) => {
+        const format = path.basename(manifestPath);
+        if (!PROJECT_MANIFEST_FORMATS.has(format) || !path.isAbsolute(manifestPath)) {
+            throw new Error(`Unsupported manifest path for project scan: ${manifestPath}`);
+        }
+        return ["--lockfile", `${format}:${manifestPath}`];
+    });
+}
 const FIXED_ARTIFACT_ARGS = [
     "scan", "source", "--format", "json", "--all-packages", "--no-ignore",
     "--experimental-no-default-plugins", "--experimental-plugins", "java/archive",
@@ -130,13 +146,16 @@ function execOsvScanner(binaryPath, targetPaths, timeoutMs, maxOutputBytes, scan
     });
 }
 /**
- * projectDirをOSV-Scannerでスキャンし、整形済みレポートを返す。
+ * 検出済みマニフェストだけをOSV-Scannerでスキャンし、整形済みレポートを返す。
  *
- * @param projectDir スキャン対象ディレクトリ。**必ず`detectJavaProject`で検証済みの
- *   絶対パスを渡すこと**(このレイヤーではパス検証を行わない)
+ * @param manifestPaths **`detectJavaProject`が検出したマニフェストの絶対パス**
+ *   (このレイヤーではスキャン範囲の検証を行わない。ディレクトリは渡さない)
  */
-export async function runOsvScan(projectDir, options = {}) {
-    return parseOsvScanOutput(await runScan([projectDir], options, "project"));
+export async function runOsvScan(manifestPaths, options = {}) {
+    if (manifestPaths.length === 0) {
+        throw new ScanToolError("no_manifest_found", "スキャン対象のマニフェストがありません");
+    }
+    return parseOsvScanOutput(await runScan(manifestPaths, options, "project"));
 }
 /** Accept only the exact absolute files enumerated by detectJavaArtifacts. */
 export async function runOsvArtifactScan(artifactPaths, options = {}) {
@@ -164,9 +183,9 @@ async function runScan(targetPaths, options, mode) {
 }
 async function runOsvScanUnguarded(targetPaths, options, mode) {
     const binaryPath = options.binaryPath ?? (await resolveOsvScannerBinary());
-    const result = await execOsvScanner(binaryPath, targetPaths, options.timeoutMs ?? DEFAULT_TIMEOUT_MS, options.maxOutputBytes ?? DEFAULT_MAX_OUTPUT_BYTES, buildOsvScanArgs(mode, isRemoteResolutionDisabled(options)));
+    const result = await execOsvScanner(binaryPath, mode === "project" ? buildProjectTargetArgs(targetPaths) : targetPaths, options.timeoutMs ?? DEFAULT_TIMEOUT_MS, options.maxOutputBytes ?? DEFAULT_MAX_OUTPUT_BYTES, buildOsvScanArgs(mode, isRemoteResolutionDisabled(options)));
     if (result.exitCode === EXIT_NO_PACKAGES && mode === "project") {
-        throw new ScanToolError("no_packages_found", `OSV-Scannerがスキャン対象のパッケージを検出できませんでした: ${targetPaths[0]}(pom.xmlに依存関係が定義されているか確認してください)`, result.stderr);
+        throw new ScanToolError("no_packages_found", `OSV-Scannerがスキャン対象のパッケージを検出できませんでした(マニフェスト${targetPaths.length}件。pom.xmlに依存関係が定義されているか確認してください)`, result.stderr);
     }
     if (mode !== "project" && result.exitCode === EXIT_NO_PACKAGES && result.stdout.trim() === "") {
         return { results: [] };

package/dist/osv/suggestFix.js

@@ -16,6 +16,32 @@
  */
 import { compareMavenVersions, mavenVersionSeries } from "../utils/mavenVersion.js";
 import { candidateStatus } from "./affectedVersions.js";
+/** バージョン比較・影響範囲の検証が対応済みのエコシステム */
+const SUPPORTED_ECOSYSTEMS = new Set(["Maven"]);
+/**
+ * 未対応エコシステムは推奨を出さず、CVEをunfixedにも数えない。
+ * 修正版の抽出がMaven専用のため、そのまま処理すると修正版のある脆弱性を
+ * 「修正版なし」と誤表示する(v0.3.3で確認した不具合)。
+ */
+function unsupportedEcosystemSuggestion(pkg) {
+    return {
+        package: pkg.name,
+        current_version: pkg.version,
+        ecosystem: pkg.ecosystem,
+        recommended_upgrade: null,
+        upgrade_tier: null,
+        upgrade_note: `${pkg.ecosystem}の修正版推奨には未対応です(修正版の有無は判定していません)。` +
+            "各脆弱性の修正版はexplain_vulnerabilityで確認してください",
+        per_cve_detail: pkg.vulnerabilities.map((vuln) => ({
+            id: vuln.id,
+            cve: vuln.cve,
+            severity: vuln.severity,
+            fixed_in: null,
+            tier: "unsupported",
+        })),
+        verification: "unsupported_ecosystem",
+    };
+}
 function classifyTier(currentSeries, candidate) {
     const candidateSeries = mavenVersionSeries(candidate);
     if (currentSeries === null || candidateSeries === null)
@@ -73,6 +99,8 @@ function buildNote(currentSeries, recommended, tier, fixableCount, unfixedCount)
 }
 /** 1パッケージ分のアップグレード提案を組み立てる。 */
 export function suggestUpgradeForPackage(pkg) {
+    if (!SUPPORTED_ECOSYSTEMS.has(pkg.ecosystem))
+        return unsupportedEcosystemSuggestion(pkg);
     const currentSeries = mavenVersionSeries(pkg.version);
     const details = [];
     let recommended = null;

package/dist/tools/scanJavaProject.js

@@ -24,7 +24,7 @@ export async function handleScanJavaProject(args, options = {}) {
             allowedRoot: options.allowedRoot,
         });
         const noRemoteResolution = isRemoteResolutionDisabled(options);
-        const report = await runOsvScan(project.projectDir, { ...options, noRemoteResolution });
+        const report = await runOsvScan(project.manifestPaths, { ...options, noRemoteResolution });
         return jsonResult({
             project_dir: project.projectDir,
             manifests: project.manifests,

package/dist/tools/suggestFix.js

@@ -14,7 +14,7 @@ export async function handleSuggestFix(args, options = {}) {
             allowedRoot: options.allowedRoot,
         });
         const noRemoteResolution = isRemoteResolutionDisabled(options);
-        const report = await runOsvScan(project.projectDir, { ...options, noRemoteResolution });
+        const report = await runOsvScan(project.manifestPaths, { ...options, noRemoteResolution });
         const suggestions = suggestUpgrades(report.packages);
         const unfixedVulnerabilities = suggestions.reduce((sum, s) => sum + s.per_cve_detail.filter((d) => d.tier === "unfixed").length, 0);
         return jsonResult({

package/dist/utils/projectDetector.js

@@ -11,14 +11,18 @@
  * `project_path`はLLM・ユーザー由来の信頼できない入力として扱う:
  * - `realpath`で正規化し、シンボリックリンクを解決した実体パスで判定する
  * - `allowedRoot`指定時は、解決後のパスがその配下にあることを検証する(パストラバーサル対策)
- * - マニフェスト探索は深さ・件数に上限を設け、シンボリックリンクのディレクトリは辿らない
+ * - マニフェスト探索はエントリ数・件数・深さに上限を設け、シンボリックリンクのディレクトリは辿らない
+ *
+ * 検出したマニフェストが、そのままOSV-Scannerのスキャン範囲になる(ディレクトリは渡さない)。
+ * そのため探索を深さで打ち切らず(Javaのソースツリーは深い)、上限に達した場合は
+ * 結果を黙って欠落させずにエラーにする。
  */
 import { readdir, realpath, stat } from "node:fs/promises";
 import path from "node:path";
 import { ScanToolError } from "../errors.js";
-const DEFAULT_MAX_DEPTH = 3;
-/** 探索を打ち切るマニフェスト件数の上限(巨大モノレポでの暴走防止) */
-const MAX_MANIFESTS = 100;
+const DEFAULT_MAX_DEPTH = 64;
+const DEFAULT_MAX_ENTRIES = 200_000;
+const DEFAULT_MAX_MANIFESTS = 1_000;
 /** ビルド成果物・VCS等、マニフェスト探索でスキップするディレクトリ */
 const SKIPPED_DIRS = new Set([".git", "node_modules", "target", "build", ".idea", ".vscode"]);
 /** OSV-Scannerがスキャンできるマニフェスト(実機確認済み) */
@@ -48,12 +52,19 @@ function assertInsideAllowedRoot(resolvedDir, allowedRootReal) {
         throw new ScanToolError("path_outside_allowed_root", `指定されたパスは許可されたディレクトリ(${allowedRootReal})の外にあります`);
     }
 }
-/** 深さ・件数上限付きでマニフェストを探索する。シンボリックリンクは辿らない。 */
-async function findManifests(rootDir, maxDepth) {
+function searchLimitError(reason) {
+    return new ScanToolError("manifest_search_limit_exceeded", `マニフェスト探索が上限(${reason})に達したため、スキャンを中止しました。` +
+        "結果の欠落を避けるため途中までの結果は返しません。より狭いディレクトリ、またはpom.xml / gradle.lockfileを直接指定してください");
+}
+/** 上限付きでマニフェストを探索する。シンボリックリンクは辿らない。上限到達はエラー。 */
+async function findManifests(rootDir, limits) {
     const manifests = [];
     let gradleBuildFileFound = false;
+    let visited = 0;
     let currentLevel = [rootDir];
-    for (let depth = 1; depth <= maxDepth && currentLevel.length > 0; depth++) {
+    for (let depth = 1; currentLevel.length > 0; depth++) {
+        if (depth > limits.maxDepth)
+            throw searchLimitError(`深さ${limits.maxDepth}`);
         const nextLevel = [];
         for (const dir of currentLevel) {
             let entries;
@@ -64,10 +75,12 @@ async function findManifests(rootDir, maxDepth) {
                 continue; // 読めないディレクトリはスキップ(権限不足等)
             }
             for (const entry of entries) {
+                if (++visited > limits.maxEntries)
+                    throw searchLimitError(`${limits.maxEntries}エントリ`);
                 if (entry.isFile() && MANIFEST_FILENAMES.has(entry.name)) {
+                    if (manifests.length >= limits.maxManifests)
+                        throw searchLimitError(`マニフェスト${limits.maxManifests}件`);
                     manifests.push(path.relative(rootDir, path.join(dir, entry.name)));
-                    if (manifests.length >= MAX_MANIFESTS)
-                        return { manifests, gradleBuildFileFound };
                 }
                 else if (entry.isFile() && GRADLE_BUILD_FILENAMES.has(entry.name)) {
                     gradleBuildFileFound = true;
@@ -93,14 +106,21 @@ export async function detectJavaProject(inputPath, options = {}) {
     }
     const resolved = await resolveExistingPath(inputPath);
     const stats = await stat(resolved);
+    // マニフェストの直接指定は、そのファイル1件だけをスキャン範囲にする。
+    // 親ディレクトリを探索しないため、探索上限エラーの回避手段として使える
+    if (stats.isFile() && MANIFEST_FILENAMES.has(path.basename(resolved))) {
+        const projectDir = path.dirname(resolved);
+        if (options.allowedRoot !== undefined) {
+            assertInsideAllowedRoot(projectDir, await resolveExistingPath(options.allowedRoot));
+        }
+        return { projectDir, manifests: [path.basename(resolved)], manifestPaths: [resolved] };
+    }
     let projectDir;
     if (stats.isDirectory()) {
         projectDir = resolved;
     }
-    else if (stats.isFile() &&
-        (MANIFEST_FILENAMES.has(path.basename(resolved)) ||
-            GRADLE_BUILD_FILENAMES.has(path.basename(resolved)))) {
-        // build.gradle等の直接指定も受け付け、ディレクトリとして解決する
+    else if (stats.isFile() && GRADLE_BUILD_FILENAMES.has(path.basename(resolved))) {
+        // build.gradle等の直接指定は、lockfileを探すためディレクトリとして解決する
         // (lockfileが無ければ後段でgradle_lockfile_missingの案内になる)
         projectDir = path.dirname(resolved);
     }
@@ -110,13 +130,20 @@ export async function detectJavaProject(inputPath, options = {}) {
     if (options.allowedRoot !== undefined) {
         assertInsideAllowedRoot(projectDir, await resolveExistingPath(options.allowedRoot));
     }
-    const maxDepth = options.maxDepth ?? DEFAULT_MAX_DEPTH;
-    const { manifests, gradleBuildFileFound } = await findManifests(projectDir, maxDepth);
+    const { manifests, gradleBuildFileFound } = await findManifests(projectDir, {
+        maxDepth: options.maxDepth ?? DEFAULT_MAX_DEPTH,
+        maxEntries: options.maxEntries ?? DEFAULT_MAX_ENTRIES,
+        maxManifests: options.maxManifests ?? DEFAULT_MAX_MANIFESTS,
+    });
     if (manifests.length === 0) {
         if (gradleBuildFileFound) {
             throw new ScanToolError("gradle_lockfile_missing", GRADLE_LOCKFILE_GUIDANCE);
         }
-        throw new ScanToolError("no_manifest_found", `対応マニフェスト(pom.xml / gradle.lockfile)が見つかりません(深さ${maxDepth}まで探索): ${projectDir}`);
+        throw new ScanToolError("no_manifest_found", `対応マニフェスト(pom.xml / gradle.lockfile)が見つかりません: ${projectDir}`);
     }
-    return { projectDir, manifests };
+    return {
+        projectDir,
+        manifests,
+        manifestPaths: manifests.map((manifest) => path.join(projectDir, manifest)),
+    };
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "osv-scanner-mcp",
-  "version": "0.3.3",
+  "version": "0.3.4",
   "description": "MCP server that wraps Google's OSV-Scanner to scan Java projects for known vulnerabilities",
   "type": "module",
   "license": "Apache-2.0",