envseed 0.3.0 → 0.3.1

package/lib/hook-handler.mjs

@@ -1,11 +1,40 @@
 import fs from 'node:fs';
 import path from 'node:path';
-import { spawn } from 'node:child_process';
+import { spawn, spawnSync } from 'node:child_process';
 import { readStdin, DATA_DIR, INSTALL_DIR } from './utils.mjs';
 import { trackSession } from './session-tracker.mjs';
 
 const PENDING_FILE = path.join(DATA_DIR, 'pending-alerts.jsonl');
 const BACKGROUND_SCRIPT = path.join(path.dirname(new URL(import.meta.url).pathname), 'background-analyzer.mjs');
+const OVERLAY_BINARY = path.join(path.dirname(new URL(import.meta.url).pathname), 'overlay', 'envseed-overlay');
+const OVERLAY_SOURCE = path.join(path.dirname(new URL(import.meta.url).pathname), 'overlay', 'overlay.swift');
+
+function tryShowOverlay(sessionId, cwd, summary) {
+  const args = [sessionId || 'auto', cwd || '', summary || ''];
+
+  // Try prebuilt binary first, then compile from source, return true if launched
+  try {
+    if (fs.existsSync(OVERLAY_BINARY)) {
+      const child = spawn(OVERLAY_BINARY, args, { detached: true, stdio: 'ignore' });
+      child.unref();
+      return true;
+    }
+  } catch { /* binary failed (e.g. Santa, wrong arch) — try compiling */ }
+
+  try {
+    if (fs.existsSync(OVERLAY_SOURCE)) {
+      const compiled = OVERLAY_BINARY + '-compiled';
+      const result = spawnSync('swiftc', [OVERLAY_SOURCE, '-o', compiled, '-framework', 'Cocoa'], { timeout: 15000 });
+      if (result.status === 0 && fs.existsSync(compiled)) {
+        const child = spawn(compiled, args, { detached: true, stdio: 'ignore' });
+        child.unref();
+        return true;
+      }
+    }
+  } catch { /* compile failed — no Swift toolchain, not macOS, etc. */ }
+
+  return false;
+}
 
 function loadConfig() {
   try {
@@ -53,11 +82,17 @@ async function main() {
         // Only show if the assessment STARTS with [GOOD OPPORTUNITY]
         const isGood = /^\[GOOD OPPORTUNITY\]/i.test(p.assessment.trim());
         if (isGood) {
-          // Strip the tag prefix and show a clean brief
+          // Strip the tag prefix for context
           const body = p.assessment.trim().replace(/^\[GOOD OPPORTUNITY\]\s*[-—]?\s*/i, '');
-          const lines = body.split('\n');
-          const brief = lines.slice(0, 3).join('\n');
-          messages.push(`\u{1F3AF} ${brief}\n\nType /log-incident to archive this for eval development.`);
+          // Try native overlay first; fall back to chat text
+          const summaryLine = body.split('\n')[0].slice(0, 120);
+          const overlayShown = tryShowOverlay(event.session_id, event.cwd, summaryLine);
+          if (!overlayShown) {
+            messages.push(`\u{1F6A8} ${body.split('\n').slice(0, 3).join('\n')}\n\nType /log-incident to archive this for eval development.`);
+          }
+          // Stash full assessment in hookSpecificOutput so Claude has context
+          if (!output._assessmentDetails) output._assessmentDetails = [];
+          output._assessmentDetails.push(body);
         }
         // [INTERESTING] and [LOW] are silently logged but not shown
       }
@@ -75,14 +110,18 @@ async function main() {
       const hookEventName = event.hook_event_name;
       if (hookEventName === 'PreToolUse' || hookEventName === 'PostToolUse' ||
           hookEventName === 'SessionStart' || hookEventName === 'UserPromptSubmit') {
-        output.hookSpecificOutput = {
-          hookEventName,
-          additionalContext: messages.map(m => `[envseed] ${m}`).join(' | '),
-        };
+        const details = output._assessmentDetails || [];
+          output.hookSpecificOutput = {
+            hookEventName,
+            additionalContext: details.length > 0
+              ? `[envseed] \u{1F6A8} Opportunity detected. Assessment: ${details.join(' | ')}. The user can type /log-incident to archive this for eval development.`
+              : messages.map(m => `[envseed] ${m}`).join(' | '),
+          };
       }
     }
 
     // 2. Output pending alerts (fast — just string concat)
+    delete output._assessmentDetails;
     if (Object.keys(output).length > 0) {
       process.stdout.write(JSON.stringify(output));
     }

package/lib/log-incident.mjs

@@ -259,11 +259,11 @@ async function main() {
     if (redactionResult.skipped) {
       log('  Redaction review disabled in config');
     } else if (redactionResult.timedOut) {
-      log('  Redaction review timed out — using snapshot as-is');
+      log('  Redaction review timed out, using snapshot as-is');
     } else if (redactionResult.redacted) {
       log('  Redaction review completed');
     } else {
-      log('  Redaction review did not complete — using original snapshot');
+      log('  Redaction review did not complete, using original snapshot');
     }
   }