mcpsec 0.2.1 → 0.3.0

package/README.md

@@ -14,6 +14,22 @@ MCP gives AI agents access to tools, files, databases, and APIs. A single malici
 
 mcpsec finds these problems before attackers do.
 
+## Real-World Results
+
+We tested mcpsec against 10 popular MCP servers (GitHub, Slack, Postgres, Brave Search, Puppeteer, etc.) configured exactly as their official README files recommend.
+
+**Score: 2/100**
+
+| Severity | Findings | Examples |
+|----------|----------|---------|
+| Critical | 3 | GitHub PAT, Slack bot token, and Postgres password in plain text config |
+| High | 1 | API key hardcoded in env block |
+| Medium | 1 | Unverified third-party npm package via `npx -y` |
+
+The only servers that passed clean were ones that don't require credentials (filesystem, memory, sqlite). Every server that needs an API key had it hardcoded in the config file - because that's what the docs tell you to do.
+
+Against deliberately vulnerable MCP server configs (from security research projects), mcpsec found **17 findings** including hardcoded AWS keys, Stripe live keys, SSRF endpoints, and supply chain risks.
+
 ## Quick Start
 
 ```bash
@@ -44,7 +60,7 @@ npx mcpsec scan --baseline --json
 ## Example Output
 
 ```
-  mcpsec - MCP Security Scanner v0.1.0
+  mcpsec - MCP Security Scanner v0.3.0
   ──────────────────────────────────────────────────
 
   Configurations Found
@@ -85,6 +101,41 @@ npx mcpsec scan --baseline --json
   Fix: Pin to a specific version: npx package@1.2.3
 ```
 
+## Registry Scanning
+
+Scan MCP servers directly from the official [MCP Registry](https://registry.modelcontextprotocol.io/) to assess their security before installing them.
+
+```bash
+# Scan top servers from the registry
+npx mcpsec scan --registry
+
+# Limit number of servers
+npx mcpsec scan --registry --limit 50
+
+# Search for servers by keyword
+npx mcpsec scan --registry --search "database"
+
+# Scan a specific server by name
+npx mcpsec scan --registry --server "filesystem"
+
+# Combine with output formats
+npx mcpsec scan --registry --json
+npx mcpsec scan --registry --sarif
+```
+
+### Registry-Specific Checks
+
+In addition to running the standard static analysis pipeline, registry mode performs supply-chain checks against npm:
+
+| Check | Category | Severity |
+|-------|----------|----------|
+| Package not found on npm | Supply Chain | High |
+| Package < 1 week old | Supply Chain | Medium |
+| Package < 100 weekly downloads | Supply Chain | Low |
+| Single maintainer on npm | Supply Chain | Info |
+| No repository URL in registry entry | Supply Chain | Low |
+| Repository URL returns 404 | Supply Chain | Medium |
+
 ## What It Detects
 
 ### Static Analysis (default)
@@ -234,6 +285,10 @@ Options:
   --json                   Output results as JSON
   --sarif                  Output results as SARIF 2.1.0 (for GitHub Code Scanning)
   --path <file>            Scan a specific config file
+  --registry               Scan servers from the official MCP registry
+  --limit <n>              Max servers to fetch from registry (default: 20)
+  --search <query>         Search registry servers by keyword
+  --server <name>          Scan a specific registry server by name
   --save-baseline [file]   Save scan results as baseline (default: .mcpsec-baseline.json)
   --baseline [file]        Compare scan against baseline and show diff
   --no-color               Disable colored output
@@ -269,11 +324,13 @@ src/
     injection-patterns.ts   Prompt injection / tool poisoning patterns
     url-validator.ts        SSRF detection (cloud metadata, private IPs)
     mcp-client.ts           MCP protocol client (stdio + HTTP)
+    registry-client.ts      MCP registry + npm API client
   scanner/
     config-scanner.ts       Config-level checks (transport, docker, supply chain)
     credential-scanner.ts   API key and credential detection
     tool-scanner.ts         Injection and command injection scanning
     live-scanner.ts         Live server tool/resource/prompt analysis
+    registry-scanner.ts     Registry supply-chain checks (npm age, downloads)
     report.ts               Score calculation and report output
     sarif.ts                SARIF 2.1.0 output for GitHub Code Scanning
     baseline.ts             Baseline save/load and diff engine
@@ -283,7 +340,7 @@ src/
 
 - [x] Cross-server tool shadowing detection
 - [x] GitHub Actions action (`uses: robdtaylor/sentinel-mcp@v1`)
-- [ ] MCP server registry scanning
+- [x] MCP server registry scanning
 - [x] Baseline / diff mode (track changes between scans)
 - [x] SARIF output format (`--sarif` for GitHub Code Scanning)
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mcpsec",
-  "version": "0.2.1",
+  "version": "0.3.0",
   "description": "Security scanner for MCP (Model Context Protocol) servers - detects tool poisoning, credential exposure, prompt injection, and SSRF",
   "license": "MIT",
   "author": "Rob Taylor <robdtaylor@users.noreply.github.com>",

package/src/cli/index.ts

@@ -13,6 +13,7 @@ import { discoverConfigs, scanConfigs } from '../scanner/config-scanner';
 import { credentialScanner } from '../scanner/credential-scanner';
 import { toolScanner } from '../scanner/tool-scanner';
 import { liveScanner } from '../scanner/live-scanner';
+import { registryScanner } from '../scanner/registry-scanner';
 import { generateReport, printReport, printReportJSON } from '../scanner/report';
 import { printReportSARIF } from '../scanner/sarif';
 import {
@@ -39,6 +40,10 @@ const HELP = `
     --json                   Output results as JSON
     --sarif                  Output results as SARIF 2.1.0 (for GitHub Code Scanning)
     --path <file>            Scan a specific config file
+    --registry               Scan servers from the official MCP registry
+    --limit <n>              Max servers to fetch from registry (default: 20)
+    --search <query>         Search registry servers by keyword
+    --server <name>          Scan a specific registry server by name
     --save-baseline [file]   Save scan results as baseline (default: ${DEFAULT_BASELINE_PATH})
     --baseline [file]        Compare scan against baseline and show diff
     --no-color               Disable colored output
@@ -50,6 +55,11 @@ const HELP = `
     mcpsec scan --live
     mcpsec scan --json
     mcpsec scan --path ~/.cursor/mcp.json
+    mcpsec scan --registry
+    mcpsec scan --registry --limit 50
+    mcpsec scan --registry --search "database"
+    mcpsec scan --registry --server "filesystem"
+    mcpsec scan --registry --json
     mcpsec scan --save-baseline
     mcpsec scan --baseline
     mcpsec scan --baseline --json
@@ -65,7 +75,7 @@ async function main() {
   }
 
   if (command === '--version' || command === '-v') {
-    console.log('mcpsec v0.2.0');
+    console.log('mcpsec v0.3.0');
     process.exit(0);
   }
 
@@ -82,6 +92,15 @@ async function main() {
   const pathIndex = args.indexOf('--path');
   const specificPath = pathIndex !== -1 ? args[pathIndex + 1] : undefined;
 
+  // Registry flags
+  const registryMode = args.includes('--registry');
+  const limitIndex = args.indexOf('--limit');
+  const registryLimit = limitIndex !== -1 ? parseInt(args[limitIndex + 1], 10) || 20 : 20;
+  const searchIndex = args.indexOf('--search');
+  const registrySearch = searchIndex !== -1 ? args[searchIndex + 1] : undefined;
+  const serverIndex = args.indexOf('--server');
+  const registryServer = serverIndex !== -1 ? args[serverIndex + 1] : undefined;
+
   // Baseline flags
   const saveBaselineIndex = args.indexOf('--save-baseline');
   const saveBaselineMode = saveBaselineIndex !== -1;
@@ -106,8 +125,45 @@ async function main() {
 
   // Discover configs
   let configs: MCPConfigFile[];
+  const allFindings: Finding[] = [];
+
+  if (registryMode) {
+    // Registry scanning mode
+    if (!jsonOutput && !sarifOutput) {
+      const label = registryServer
+        ? `server "${registryServer}"`
+        : registrySearch
+          ? `"${registrySearch}" (limit: ${registryLimit})`
+          : `top ${registryLimit} servers`;
+      process.stderr.write(`\n\x1b[1m🌐 Registry Scan: ${label}\x1b[0m\n`);
+      process.stderr.write(`\x1b[2m   Fetching from registry.modelcontextprotocol.io...\x1b[0m\n`);
+    }
+
+    try {
+      const result = await registryScanner.scanRegistry({
+        limit: registryLimit,
+        search: registrySearch,
+        server: registryServer,
+      });
+
+      configs = result.configs;
+      allFindings.push(...result.findings);
+
+      if (configs.length === 0) {
+        if (!jsonOutput && !sarifOutput) {
+          console.error('  No servers found matching your criteria.');
+        }
+        process.exit(0);
+      }
 
-  if (specificPath) {
+      if (!jsonOutput && !sarifOutput) {
+        process.stderr.write(`\x1b[2m   Found ${configs.length} server(s), running security analysis...\x1b[0m\n`);
+      }
+    } catch (err) {
+      console.error(`Registry fetch failed: ${(err as Error).message}`);
+      process.exit(1);
+    }
+  } else if (specificPath) {
     // Scan a specific file
     const { existsSync, readFileSync } = await import('fs');
     if (!existsSync(specificPath)) {
@@ -140,23 +196,18 @@ async function main() {
     configs = discoverConfigs();
   }
 
-  // Run all scanners
-  const allFindings: Finding[] = [];
-
-  // Config-level checks
+  // Run standard scanners (config, credential, tool)
   const configFindings = scanConfigs(configs);
   allFindings.push(...configFindings);
 
-  // Credential scanner
   const credFindings = await credentialScanner.scan(configs);
   allFindings.push(...credFindings);
 
-  // Tool/injection scanner
   const toolFindings = await toolScanner.scan(configs);
   allFindings.push(...toolFindings);
 
-  // Live server scanner (connects to running servers)
-  if (liveMode) {
+  // Live server scanner (connects to running servers -- not used in registry mode)
+  if (liveMode && !registryMode) {
     if (!jsonOutput) {
       process.stderr.write('\n\x1b[1m🔴 Live Server Scan\x1b[0m\n');
     }

package/src/lib/registry-client.ts

@@ -0,0 +1,188 @@
+/**
+ * Sentinel MCP - Registry Client
+ *
+ * HTTP client for fetching MCP server listings from public registries
+ * and npm package metadata for supply-chain analysis.
+ */
+
+// ============================================================================
+// Types
+// ============================================================================
+
+export interface RegistryServer {
+  name: string;
+  description: string;
+  packages: { type: string; name: string }[];
+  repository?: string;
+  version?: string;
+}
+
+export interface NpmPackageInfo {
+  exists: boolean;
+  created?: string;
+  weeklyDownloads?: number;
+  maintainers?: number;
+  repository?: string;
+}
+
+// ============================================================================
+// Official MCP Registry
+// ============================================================================
+
+const REGISTRY_BASE = 'https://registry.modelcontextprotocol.io/v0.1';
+
+/**
+ * Fetch server listings from the official MCP registry
+ */
+export async function fetchRegistryServers(options: {
+  limit?: number;
+  search?: string;
+  server?: string;
+}): Promise<RegistryServer[]> {
+  const { limit = 20, search, server } = options;
+
+  let url = `${REGISTRY_BASE}/servers`;
+  const params = new URLSearchParams();
+
+  if (search) {
+    params.set('q', search);
+  }
+
+  if (params.toString()) {
+    url += `?${params.toString()}`;
+  }
+
+  const response = await fetch(url);
+
+  if (!response.ok) {
+    throw new Error(`Registry API error: ${response.status} ${response.statusText}`);
+  }
+
+  const data = await response.json();
+
+  // The API returns { servers: [{ server: {...}, _meta: {...} }, ...] }
+  const rawEntries: any[] = Array.isArray(data)
+    ? data
+    : (data.servers || data.items || data.results || []);
+
+  // Each entry wraps the actual server data in a `server` key
+  let servers: RegistryServer[] = rawEntries.map((entry: any) => {
+    const s = entry.server || entry;
+    return {
+      name: s.name || s.id || '',
+      description: s.description || '',
+      packages: normalizePackages(s),
+      repository: (typeof s.repository === 'object' ? s.repository?.url : s.repository) || undefined,
+      version: s.version || s.latest_version || undefined,
+    };
+  });
+
+  // Filter to a specific server name if requested
+  if (server) {
+    const needle = server.toLowerCase();
+    servers = servers.filter(
+      (s) => s.name.toLowerCase() === needle || s.name.toLowerCase().includes(needle)
+    );
+  }
+
+  return servers.slice(0, limit);
+}
+
+/**
+ * Normalize the various package formats from the registry API.
+ *
+ * The official registry uses:
+ *   packages: [{ registryType: "npm", name: "@scope/pkg" }]
+ *   or: packages: [{ registryType: "oci", identifier: "docker.io/img:tag" }]
+ *   or: remotes: [{ type: "streamable-http", url: "..." }] (no installable package)
+ */
+function normalizePackages(entry: any): { type: string; name: string }[] {
+  // Direct packages array
+  if (Array.isArray(entry.packages)) {
+    return entry.packages
+      .map((p: any) => ({
+        type: p.registryType || p.registry_type || p.type || 'npm',
+        name: p.name || p.identifier || p.package_name || '',
+      }))
+      .filter((p: { name: string }) => p.name);
+  }
+
+  // npm field directly
+  if (entry.npm) {
+    return [{ type: 'npm', name: entry.npm }];
+  }
+
+  // pypi field directly
+  if (entry.pypi) {
+    return [{ type: 'pypi', name: entry.pypi }];
+  }
+
+  // package_name field
+  if (entry.package_name) {
+    return [{ type: 'npm', name: entry.package_name }];
+  }
+
+  return [];
+}
+
+// ============================================================================
+// npm Registry
+// ============================================================================
+
+const NPM_REGISTRY = 'https://registry.npmjs.org';
+const NPM_API = 'https://api.npmjs.org';
+
+/**
+ * Fetch package metadata from npm for supply-chain analysis
+ */
+export async function fetchNpmPackageInfo(packageName: string): Promise<NpmPackageInfo | null> {
+  try {
+    // Fetch package metadata
+    const metaResponse = await fetch(`${NPM_REGISTRY}/${encodeURIComponent(packageName)}`);
+
+    if (metaResponse.status === 404) {
+      return { exists: false };
+    }
+
+    if (!metaResponse.ok) {
+      return null; // API error, can't determine
+    }
+
+    const meta = await metaResponse.json();
+
+    // Extract creation date
+    const created = meta.time?.created || undefined;
+
+    // Extract maintainer count
+    const maintainers = Array.isArray(meta.maintainers) ? meta.maintainers.length : undefined;
+
+    // Extract repository URL
+    const repository = typeof meta.repository === 'string'
+      ? meta.repository
+      : meta.repository?.url || undefined;
+
+    // Fetch weekly download count (separate API)
+    let weeklyDownloads: number | undefined;
+    try {
+      const dlResponse = await fetch(
+        `${NPM_API}/downloads/point/last-week/${encodeURIComponent(packageName)}`
+      );
+      if (dlResponse.ok) {
+        const dlData = await dlResponse.json();
+        weeklyDownloads = dlData.downloads;
+      }
+    } catch {
+      // Download stats unavailable, not critical
+    }
+
+    return {
+      exists: true,
+      created,
+      weeklyDownloads,
+      maintainers,
+      repository,
+    };
+  } catch {
+    return null; // Network error
+  }
+}

package/src/scanner/registry-scanner.ts

@@ -0,0 +1,297 @@
+/**
+ * Sentinel MCP - Registry Scanner
+ *
+ * Fetches MCP servers from public registries, converts them to synthetic
+ * MCPConfigFile objects for the existing scanner pipeline, and runs
+ * registry-specific supply-chain checks (npm package age, downloads, etc).
+ */
+
+import type { Finding, MCPConfigFile, Scanner } from '../lib/types';
+import { fetchRegistryServers, fetchNpmPackageInfo } from '../lib/registry-client';
+import type { RegistryServer } from '../lib/registry-client';
+
+// ============================================================================
+// Registry Scanner
+// ============================================================================
+
+export interface RegistryScanOptions {
+  limit?: number;
+  search?: string;
+  server?: string;
+}
+
+/**
+ * Convert a registry server entry into a synthetic MCPConfigFile
+ * so existing scanners (credential, tool, config) can process it unchanged.
+ */
+export function toSyntheticConfig(server: RegistryServer): MCPConfigFile {
+  const servers: Record<string, any> = {};
+
+  // Build a synthetic server config based on the package type
+  const npmPkg = server.packages.find((p) => p.type === 'npm');
+  const pypiPkg = server.packages.find((p) => p.type === 'pypi');
+  const ociPkg = server.packages.find((p) => p.type === 'oci');
+
+  if (npmPkg) {
+    servers[server.name] = {
+      command: 'npx',
+      args: ['-y', npmPkg.name],
+    };
+  } else if (pypiPkg) {
+    servers[server.name] = {
+      command: 'uvx',
+      args: [pypiPkg.name],
+    };
+  } else if (ociPkg) {
+    servers[server.name] = {
+      command: 'docker',
+      args: ['run', '-i', ociPkg.name],
+    };
+  } else {
+    // Remote-only or unknown -- create a placeholder
+    servers[server.name] = {
+      command: 'unknown',
+      args: [],
+    };
+  }
+
+  return {
+    path: `registry://modelcontextprotocol.io/${server.name}`,
+    client: 'MCP Registry',
+    servers,
+    raw: server,
+  };
+}
+
+/**
+ * Run registry-specific supply-chain checks against npm packages.
+ * These checks go beyond what static config analysis catches.
+ */
+async function checkNpmSupplyChain(
+  server: RegistryServer,
+  configPath: string,
+): Promise<Finding[]> {
+  const findings: Finding[] = [];
+  const npmPkg = server.packages.find((p) => p.type === 'npm');
+
+  if (!npmPkg) return findings;
+
+  const info = await fetchNpmPackageInfo(npmPkg.name);
+
+  if (!info) return findings; // API unavailable, skip
+
+  // Package not found on npm
+  if (!info.exists) {
+    findings.push({
+      id: 'REG-NPM-404',
+      severity: 'high',
+      category: 'supply-chain',
+      title: `npm package not found: ${npmPkg.name}`,
+      description: `The npm package "${npmPkg.name}" listed for "${server.name}" does not exist on the npm registry. This may indicate a typosquat risk or a removed package.`,
+      server: server.name,
+      configFile: configPath,
+      evidence: `package: ${npmPkg.name}`,
+      remediation: 'Verify the package name is correct. Do not install packages that cannot be found on npm.',
+    });
+    return findings; // No point checking further
+  }
+
+  // Package less than 1 week old
+  if (info.created) {
+    const createdDate = new Date(info.created);
+    const ageMs = Date.now() - createdDate.getTime();
+    const ageDays = ageMs / (1000 * 60 * 60 * 24);
+
+    if (ageDays < 7) {
+      findings.push({
+        id: 'REG-NPM-NEW',
+        severity: 'medium',
+        category: 'supply-chain',
+        title: `Very new npm package: ${npmPkg.name}`,
+        description: `The npm package "${npmPkg.name}" was created ${Math.floor(ageDays)} day(s) ago. New packages have not been vetted by the community and may pose supply-chain risks.`,
+        server: server.name,
+        configFile: configPath,
+        evidence: `created: ${info.created}`,
+        remediation: 'Wait for the package to establish a track record before installing. Review the source code manually.',
+      });
+    }
+  }
+
+  // Low weekly downloads
+  if (info.weeklyDownloads !== undefined && info.weeklyDownloads < 100) {
+    findings.push({
+      id: 'REG-NPM-LOW-DL',
+      severity: 'low',
+      category: 'supply-chain',
+      title: `Low download count: ${npmPkg.name}`,
+      description: `The npm package "${npmPkg.name}" has only ${info.weeklyDownloads} weekly downloads. Low adoption means fewer eyes reviewing the code for security issues.`,
+      server: server.name,
+      configFile: configPath,
+      evidence: `weekly downloads: ${info.weeklyDownloads}`,
+      remediation: 'Review the package source code before installing. Consider well-established alternatives.',
+    });
+  }
+
+  // Single maintainer
+  if (info.maintainers !== undefined && info.maintainers <= 1) {
+    findings.push({
+      id: 'REG-NPM-SOLE',
+      severity: 'info',
+      category: 'supply-chain',
+      title: `Single maintainer: ${npmPkg.name}`,
+      description: `The npm package "${npmPkg.name}" has only ${info.maintainers} maintainer(s). Single-maintainer packages are more susceptible to account takeover attacks.`,
+      server: server.name,
+      configFile: configPath,
+      evidence: `maintainers: ${info.maintainers}`,
+      remediation: 'Monitor the package for unexpected changes. Consider pinning to a specific verified version.',
+    });
+  }
+
+  return findings;
+}
+
+/**
+ * Check registry-level metadata for supply-chain signals
+ */
+function checkRegistryMetadata(
+  server: RegistryServer,
+  configPath: string,
+): Finding[] {
+  const findings: Finding[] = [];
+
+  // No repository URL
+  if (!server.repository) {
+    findings.push({
+      id: 'REG-NO-REPO',
+      severity: 'low',
+      category: 'supply-chain',
+      title: `No repository URL: ${server.name}`,
+      description: `The registry entry for "${server.name}" has no linked source repository. Without a repo, it's harder to audit the server's code.`,
+      server: server.name,
+      configFile: configPath,
+      evidence: 'repository: not provided',
+      remediation: 'Prefer MCP servers that link to a public source repository for auditability.',
+    });
+  }
+
+  // No packages at all
+  if (server.packages.length === 0) {
+    findings.push({
+      id: 'REG-NO-PKG',
+      severity: 'low',
+      category: 'supply-chain',
+      title: `No install packages listed: ${server.name}`,
+      description: `The registry entry for "${server.name}" has no npm or PyPI package listed. The installation method is unclear.`,
+      server: server.name,
+      configFile: configPath,
+      evidence: 'packages: []',
+      remediation: 'Check the server documentation for manual installation instructions.',
+    });
+  }
+
+  return findings;
+}
+
+/**
+ * Check if repository URL is reachable (HTTP HEAD request)
+ */
+async function checkRepositoryReachable(
+  server: RegistryServer,
+  configPath: string,
+): Promise<Finding[]> {
+  if (!server.repository) return [];
+
+  try {
+    const response = await fetch(server.repository, {
+      method: 'HEAD',
+      redirect: 'follow',
+      signal: AbortSignal.timeout(5000),
+    });
+
+    if (response.status === 404) {
+      return [{
+        id: 'REG-REPO-404',
+        severity: 'medium',
+        category: 'supply-chain',
+        title: `Repository not found: ${server.name}`,
+        description: `The repository URL for "${server.name}" returns 404. The source code may have been deleted or moved.`,
+        server: server.name,
+        configFile: configPath,
+        evidence: `repository: ${server.repository} (HTTP ${response.status})`,
+        remediation: 'Do not use MCP servers whose source repository has been removed.',
+      }];
+    }
+  } catch {
+    // Timeout or network error - don't flag, could be transient
+  }
+
+  return [];
+}
+
+// ============================================================================
+// Exported Scanner
+// ============================================================================
+
+export const registryScanner: Scanner & {
+  scanRegistry(options: RegistryScanOptions): Promise<{
+    configs: MCPConfigFile[];
+    findings: Finding[];
+  }>;
+} = {
+  name: 'Registry Scanner',
+
+  /**
+   * Standard Scanner interface - not used for registry mode,
+   * but satisfies the interface for type compatibility.
+   */
+  async scan(_configs: MCPConfigFile[]): Promise<Finding[]> {
+    return [];
+  },
+
+  /**
+   * Main entry point: fetch from registry, build synthetic configs,
+   * and run registry-specific checks.
+   */
+  async scanRegistry(options: RegistryScanOptions): Promise<{
+    configs: MCPConfigFile[];
+    findings: Finding[];
+  }> {
+    // Fetch servers from registry
+    const servers = await fetchRegistryServers(options);
+
+    if (servers.length === 0) {
+      return { configs: [], findings: [] };
+    }
+
+    // Convert to synthetic configs
+    const configs = servers.map(toSyntheticConfig);
+
+    // Run registry-specific checks in parallel
+    const allFindings: Finding[] = [];
+    let findingCounter = 0;
+
+    for (const server of servers) {
+      const configPath = `registry://modelcontextprotocol.io/${server.name}`;
+
+      // Metadata checks (synchronous)
+      const metaFindings = checkRegistryMetadata(server, configPath);
+      allFindings.push(...metaFindings);
+
+      // npm supply-chain checks (async)
+      const npmFindings = await checkNpmSupplyChain(server, configPath);
+      allFindings.push(...npmFindings);
+
+      // Repository reachability check (async)
+      const repoFindings = await checkRepositoryReachable(server, configPath);
+      allFindings.push(...repoFindings);
+    }
+
+    // Assign unique IDs
+    for (const finding of allFindings) {
+      findingCounter++;
+      finding.id = `${finding.id}-${findingCounter}`;
+    }
+
+    return { configs, findings: allFindings };
+  },
+};

package/src/scanner/report.ts

@@ -6,7 +6,7 @@
 
 import type { ScanReport, ScanSummary, Finding, MCPConfigFile, ScanStatus } from '../lib/types';
 
-const VERSION = '0.2.0';
+const VERSION = '0.3.0';
 
 // ============================================================================
 // Score Calculation