ship-safe 8.0.0 → 9.0.0

package/cli/commands/watch.js

@@ -13,6 +13,7 @@
 import fs from 'fs';
 import path from 'path';
 import chalk from 'chalk';
+import { execFileSync } from 'child_process';
 import { SKIP_DIRS, SKIP_EXTENSIONS, SKIP_FILENAMES, SECRET_PATTERNS, SECURITY_PATTERNS } from '../utils/patterns.js';
 import { isHighEntropyMatch, getConfidence } from '../utils/entropy.js';
 import * as output from '../utils/output.js';
@@ -289,11 +290,13 @@ function showWatchStatus(rootPath) {
 // =============================================================================
 
 async function watchDeep(absolutePath, options = {}) {
-  const { buildOrchestrator } = await import('../agents/index.js');
+  const { buildOrchestratorAsync } = await import('../agents/index.js');
   const { ReconAgent } = await import('../agents/recon-agent.js');
 
-  const debounceMs = options.debounce || 1500;
-  const threshold = options.threshold || null;
+  const debounceMs    = options.debounce   || 1500;
+  const threshold     = options.threshold  || null;
+  const slackWebhook  = options.slack      || process.env.SHIP_SAFE_SLACK_WEBHOOK || null;
+  const prComments    = options.prComment  || false;
   const scoringEngine = new ScoringEngine();
 
   console.log();
@@ -301,7 +304,9 @@ async function watchDeep(absolutePath, options = {}) {
   console.log();
   console.log(chalk.cyan('  Running full agent scans on file changes'));
   console.log(chalk.gray(`  Debounce: ${debounceMs}ms`));
-  if (threshold) console.log(chalk.gray(`  Threshold: ${threshold}/100`));
+  if (threshold)   console.log(chalk.gray(`  Threshold: ${threshold}/100`));
+  if (slackWebhook) console.log(chalk.gray('  Slack:     notifications enabled'));
+  if (prComments)  console.log(chalk.gray('  PR:        inline comments enabled (requires gh CLI)'));
   console.log(chalk.gray('  Press Ctrl+C to stop'));
   console.log();
 
@@ -332,7 +337,7 @@ async function watchDeep(absolutePath, options = {}) {
     console.log(chalk.gray(`  [${timestamp}] ${files.length} file(s) changed — deep scanning...`));
 
     try {
-      const orchestrator = buildOrchestrator();
+      const orchestrator = await buildOrchestratorAsync(absolutePath, { quiet: true });
       const context = {
         rootPath: absolutePath,
         files,
@@ -391,6 +396,16 @@ async function watchDeep(absolutePath, options = {}) {
       if (threshold && scoreResult.score < threshold) {
         console.log(chalk.red.bold(`  ⚠ Score ${scoreResult.score} below threshold ${threshold}\n`));
       }
+
+      // ── Slack Notification ──────────────────────────────────────────────
+      if (slackWebhook && findings.length > 0) {
+        await postSlackAlert(slackWebhook, findings, scoreResult, absolutePath).catch(() => {});
+      }
+
+      // ── GitHub PR Inline Comments ────────────────────────────────────────
+      if (prComments && findings.length > 0) {
+        await postPRComments(findings, absolutePath).catch(() => {});
+      }
     } catch (err) {
       console.log(chalk.red(`  [${timestamp}] Scan error: ${err.message}\n`));
     }
@@ -431,6 +446,128 @@ async function watchDeep(absolutePath, options = {}) {
   }
 }
 
+// =============================================================================
+// SLACK NOTIFICATIONS
+// =============================================================================
+
+/**
+ * Post a security alert to a Slack webhook.
+ * Webhook URL can be set via --slack or SHIP_SAFE_SLACK_WEBHOOK env var.
+ */
+async function postSlackAlert(webhookUrl, findings, scoreResult, rootPath) {
+  const repoName  = path.basename(rootPath);
+  const criticals = findings.filter(f => f.severity === 'critical').length;
+  const highs     = findings.filter(f => f.severity === 'high').length;
+
+  const color = criticals > 0 ? 'danger' : highs > 0 ? 'warning' : 'good';
+  const emoji = criticals > 0 ? ':rotating_light:' : highs > 0 ? ':warning:' : ':shield:';
+
+  const topFindings = findings
+    .filter(f => f.severity === 'critical' || f.severity === 'high')
+    .slice(0, 5)
+    .map(f => `• *${f.severity.toUpperCase()}* ${f.title} — \`${f.file ? path.basename(f.file) : '?'}${f.line ? `:${f.line}` : ''}\``)
+    .join('\n');
+
+  const payload = {
+    attachments: [{
+      color,
+      fallback: `Ship Safe: ${findings.length} security finding(s) in ${repoName}`,
+      title:    `${emoji} Ship Safe — Security Alert`,
+      text:     `*${repoName}* — Score: *${scoreResult.score ?? '?'}/100* — ${findings.length} finding(s) (${criticals} critical, ${highs} high)`,
+      fields:   topFindings ? [{ title: 'Top Findings', value: topFindings, short: false }] : [],
+      footer:   'ship-safe watch --deep',
+      ts:       Math.floor(Date.now() / 1000),
+    }],
+  };
+
+  const res = await fetch(webhookUrl, {
+    method:  'POST',
+    headers: { 'Content-Type': 'application/json' },
+    body:    JSON.stringify(payload),
+    signal:  AbortSignal.timeout(10000),
+  });
+
+  if (!res.ok) {
+    console.log(chalk.yellow(`  [Slack] Notification failed: HTTP ${res.status}`));
+  }
+}
+
+// =============================================================================
+// GITHUB PR INLINE COMMENTS
+// =============================================================================
+
+/**
+ * Post inline security comments to the currently open PR (if any).
+ * Requires `gh` CLI to be installed and authenticated.
+ *
+ * Posts a review comment for each critical/high finding in a changed file.
+ */
+async function postPRComments(findings, rootPath) {
+  // Check if gh is available
+  try {
+    execFileSync('gh', ['--version'], { stdio: 'pipe' });
+  } catch {
+    console.log(chalk.gray('  [PR] gh CLI not found — skipping PR comments'));
+    return;
+  }
+
+  // Get current PR number
+  let prNumber;
+  try {
+    const prJson = execFileSync('gh', ['pr', 'view', '--json', 'number'], {
+      cwd: rootPath, encoding: 'utf-8', stdio: 'pipe',
+    });
+    prNumber = JSON.parse(prJson).number;
+  } catch {
+    return; // No open PR on this branch
+  }
+
+  // Get current commit SHA
+  let sha;
+  try {
+    sha = execFileSync('git', ['rev-parse', 'HEAD'], {
+      cwd: rootPath, encoding: 'utf-8', stdio: 'pipe',
+    }).trim();
+  } catch {
+    return;
+  }
+
+  const criticalOrHigh = findings.filter(f =>
+    (f.severity === 'critical' || f.severity === 'high') && f.file && f.line
+  ).slice(0, 10); // Max 10 comments per scan
+
+  for (const f of criticalOrHigh) {
+    const relFile = path.relative(rootPath, f.file).replace(/\\/g, '/');
+    const body = [
+      `**Ship Safe — ${f.severity.toUpperCase()} finding**`,
+      '',
+      `**${f.title}**`,
+      f.description || '',
+      '',
+      f.remediation ? `**Fix:** ${f.remediation}` : '',
+      '',
+      `_[${f.rule}] — detected by ship-safe watch_`,
+    ].filter(l => l !== undefined).join('\n');
+
+    try {
+      execFileSync('gh', [
+        'api',
+        `repos/{owner}/{repo}/pulls/${prNumber}/comments`,
+        '--method', 'POST',
+        '--field', `body=${body}`,
+        '--field', `commit_id=${sha}`,
+        '--field', `path=${relFile}`,
+        '--field', `line=${f.line}`,
+        '--field', 'side=RIGHT',
+      ], { cwd: rootPath, stdio: 'pipe' });
+    } catch { /* individual comment failure is non-fatal */ }
+  }
+
+  if (criticalOrHigh.length > 0) {
+    console.log(chalk.gray(`  [PR #${prNumber}] Posted ${criticalOrHigh.length} inline comment(s)`));
+  }
+}
+
 // =============================================================================
 // CONFIG WATCH — scanConfigFiles
 // =============================================================================

package/cli/providers/llm-provider.js

@@ -92,6 +92,9 @@ class AnthropicProvider extends BaseLLMProvider {
     this.baseUrl = options.baseUrl || 'https://api.anthropic.com/v1/messages';
   }
 
+  /** Whether this provider supports guaranteed-JSON tool-use output */
+  get supportsStructuredOutput() { return true; }
+
   async complete(systemPrompt, userPrompt, options = {}) {
     const response = await fetch(this.baseUrl, {
       method: 'POST',
@@ -101,7 +104,7 @@ class AnthropicProvider extends BaseLLMProvider {
         'content-type': 'application/json',
       },
       body: JSON.stringify({
-        model: this.model,
+        model: options.model || this.model,
         max_tokens: options.maxTokens || 2048,
         system: systemPrompt,
         messages: [{ role: 'user', content: userPrompt }],
@@ -109,12 +112,57 @@ class AnthropicProvider extends BaseLLMProvider {
     });
 
     if (!response.ok) {
-      throw new Error(`Anthropic API error: HTTP ${response.status}`);
+      const body = await response.text().catch(() => '');
+      throw new Error(`Anthropic API error: HTTP ${response.status} ${body.slice(0, 200)}`);
     }
 
     const data = await response.json();
     return data.content?.[0]?.text || '';
   }
+
+  /**
+   * Complete with guaranteed-JSON output via Anthropic tool-use API.
+   * The LLM is forced to call the named tool, so the response always matches
+   * the provided JSON Schema — no regex cleanup needed.
+   *
+   * @param {string} systemPrompt
+   * @param {string} userPrompt
+   * @param {string} toolName       — Name of the forced tool call
+   * @param {object} inputSchema    — JSON Schema for the tool's input
+   * @param {object} options        — { maxTokens, model }
+   * @returns {Promise<object|null>} — Parsed tool input object, or null on failure
+   */
+  async completeWithTools(systemPrompt, userPrompt, toolName, inputSchema, options = {}) {
+    const response = await fetch(this.baseUrl, {
+      method: 'POST',
+      headers: {
+        'x-api-key': this.apiKey,
+        'anthropic-version': '2023-06-01',
+        'content-type': 'application/json',
+      },
+      body: JSON.stringify({
+        model: options.model || this.model,
+        max_tokens: options.maxTokens || 2048,
+        system: systemPrompt,
+        messages: [{ role: 'user', content: userPrompt }],
+        tools: [{
+          name: toolName,
+          description: `Report ${toolName} results`,
+          input_schema: inputSchema,
+        }],
+        tool_choice: { type: 'tool', name: toolName },
+      }),
+    });
+
+    if (!response.ok) {
+      const body = await response.text().catch(() => '');
+      throw new Error(`Anthropic API error: HTTP ${response.status} ${body.slice(0, 200)}`);
+    }
+
+    const data = await response.json();
+    const toolUse = data.content?.find(b => b.type === 'tool_use');
+    return toolUse?.input ?? null;
+  }
 }
 
 // =============================================================================

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ship-safe",
-  "version": "8.0.0",
+  "version": "9.0.0",
   "description": "AI-powered multi-agent security platform. 22 agents scan 80+ attack classes including Hermes Agent deployments (ASI-01–ASI-10), tool registry poisoning, function-call injection, skill permission drift, and agent attestation. Ship Safe × Hermes Agent.",
   "main": "cli/index.js",
   "bin": {