sync hooks with evolver #558 (non-git workspace notice)

Brings in the session-start non-git notice + shared isGitWorkspace() that
landed in @evomap/evolver after the previous sync. The plugin's bundled hook
scripts now match evolver main (and the published 1.87.4 npm package).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: autogame-17

hooks/_runtimePaths.js

@@ -19,6 +19,7 @@
 const fs = require('fs');
 const path = require('path');
 const os = require('os');
+const { spawnSync } = require('child_process');
 
 function isEvolverPackageJson(filePath) {
   try {
@@ -267,4 +268,24 @@ function findMemoryGraph(evolverRoot) {
   return path.join(userDir, 'memory_graph.jsonl');
 }
 
-module.exports = { findEvolverRoot, findMemoryGraph, resolveProjectDir, resolveWorkspaceId };
+// Is `dir` inside a git work tree? Cheap, no-shell `git rev-parse`. Returns
+// false on any error (git missing, not a repo, timeout) and never throws — the
+// session-start hook uses this only to decide whether to surface a one-line
+// "evolver needs a git workspace" notice, so a false negative just suppresses
+// the notice rather than breaking anything.
+function isGitWorkspace(dir) {
+  try {
+    const res = spawnSync('git', ['rev-parse', '--is-inside-work-tree'], {
+      cwd: dir,
+      encoding: 'utf8',
+      timeout: 5000,
+      stdio: ['ignore', 'pipe', 'pipe'],
+      shell: false,
+    });
+    return res.status === 0 && typeof res.stdout === 'string' && res.stdout.trim() === 'true';
+  } catch {
+    return false;
+  }
+}
+
+module.exports = { findEvolverRoot, findMemoryGraph, resolveProjectDir, resolveWorkspaceId, isGitWorkspace };

hooks/evolver-session-start.js

@@ -7,9 +7,20 @@ const fs = require('fs');
 const path = require('path');
 const os = require('os');
 
-const { findEvolverRoot, findMemoryGraph, resolveProjectDir, resolveWorkspaceId } = require('./_runtimePaths');
+const { findEvolverRoot, findMemoryGraph, resolveProjectDir, resolveWorkspaceId, isGitWorkspace } = require('./_runtimePaths');
 const { filterRelevantOutcomes } = require('./_memoryFiltering');
 
+// One-line notice shown (throttled) when the workspace is not a git repo.
+// Evolver derives every outcome from the git diff, so in a non-git folder the
+// session-end hook records nothing — silently, unless we say so here. We surface
+// it in session-start's additionalContext (injected as opening context, which
+// does NOT trigger an extra inference round, unlike a stop-hook systemMessage).
+const NON_GIT_NOTICE =
+  '[Evolver] This folder is not a git repository, so evolution memory is inactive ' +
+  '(outcomes are derived from git diffs). Run `git init` here, or open a git project, ' +
+  'to enable recall and recording.';
+const NON_GIT_NOTICE_TTL_MS = 30 * 60 * 1000; // once per 30 min per folder
+
 // Return up to `n` of the current workspace's most-recent entries, in
 // chronological (oldest-first) order.
 //
@@ -94,18 +105,14 @@ function getDedupStatePath() {
   return path.join(dir, 'session-start-state.json');
 }
 
-function shouldSkipInjection() {
-  // Only apply dedup when explicitly enabled (set by Kiro adapter) OR when
-  // we detect a per-prompt-firing platform via PROMPT_SUBMIT heuristic in
-  // stdin. The stdin is drained in main(), so we rely on env flag here.
-  const dedupEnabled = String(process.env.EVOLVER_SESSION_START_DEDUP || '').toLowerCase() === '1'
-    || String(process.env.EVOLVER_SESSION_START_DEDUP || '').toLowerCase() === 'true';
-  if (!dedupEnabled) return false;
-
-  const ttlMs = Number(process.env.EVOLVER_SESSION_START_DEDUP_TTL_MS) || (30 * 60 * 1000);
-  const key = process.cwd();
+// TTL throttle keyed by an arbitrary string, persisted in session-start-state
+// .json. Returns true if `key` fired within the last `ttlMs` (caller should
+// suppress); otherwise records "now" for `key` and returns false. Best-effort:
+// a state read/write failure just means no throttling (fail open). Shared by
+// the Kiro per-prompt dedup and the non-git notice so both age out of the same
+// file (entries older than 24h are pruned on write).
+function throttled(key, ttlMs) {
   const statePath = getDedupStatePath();
-
   let state = {};
   try {
     if (fs.existsSync(statePath)) {
@@ -115,9 +122,7 @@ function shouldSkipInjection() {
 
   const now = Date.now();
   const last = state[key];
-  if (typeof last === 'number' && now - last < ttlMs) {
-    return true;
-  }
+  if (typeof last === 'number' && now - last < ttlMs) return true;
 
   state[key] = now;
   try {
@@ -130,53 +135,70 @@ function shouldSkipInjection() {
     fs.writeFileSync(tmp, JSON.stringify(state), 'utf8');
     fs.renameSync(tmp, statePath);
   } catch { /* best-effort */ }
-
   return false;
 }
 
+function shouldSkipInjection() {
+  // Only apply dedup when explicitly enabled (set by Kiro adapter) OR when
+  // we detect a per-prompt-firing platform via PROMPT_SUBMIT heuristic in
+  // stdin. The stdin is drained in main(), so we rely on env flag here.
+  const dedupEnabled = String(process.env.EVOLVER_SESSION_START_DEDUP || '').toLowerCase() === '1'
+    || String(process.env.EVOLVER_SESSION_START_DEDUP || '').toLowerCase() === 'true';
+  if (!dedupEnabled) return false;
+
+  const ttlMs = Number(process.env.EVOLVER_SESSION_START_DEDUP_TTL_MS) || (30 * 60 * 1000);
+  return throttled(process.cwd(), ttlMs);
+}
+
 function main() {
   if (shouldSkipInjection()) {
     process.stdout.write(JSON.stringify({}));
     return;
   }
 
-  const evolverRoot = findEvolverRoot();
-  const graphPath = findMemoryGraph(evolverRoot);
+  const currentDir = resolveProjectDir();
 
-  if (!graphPath) {
-    process.stdout.write(JSON.stringify({}));
-    return;
+  // Non-git notice: evolver records nothing in a non-git folder (outcomes come
+  // from git diffs), so tell the user — once per folder per TTL — instead of
+  // failing silently. Emitted regardless of whether any memory exists below.
+  const parts = [];
+  if (!isGitWorkspace(currentDir) && !throttled('nongit:' + currentDir, NON_GIT_NOTICE_TTL_MS)) {
+    parts.push(NON_GIT_NOTICE);
   }
 
+  const evolverRoot = findEvolverRoot();
+  const graphPath = findMemoryGraph(evolverRoot);
+
   // Scope to the current workspace BEFORE trimming to the most-recent window,
   // so other projects sharing the user-level fallback graph can't crowd this
   // workspace's outcomes out of view. When the workspace id can't be resolved,
   // belongsToWorkspace() falls back to "show it" — no regression vs. the old
   // unscoped behavior.
-  const currentDir = resolveProjectDir();
-  const currentId = resolveWorkspaceId(evolverRoot, currentDir);
-  const recent = readRecentWorkspaceEntries(graphPath, currentId, currentDir, 5);
-  const filtered = filterRelevantOutcomes(recent);
+  if (graphPath) {
+    const currentId = resolveWorkspaceId(evolverRoot, currentDir);
+    const recent = readRecentWorkspaceEntries(graphPath, currentId, currentDir, 5);
+    const filtered = filterRelevantOutcomes(recent);
+    if (filtered.length > 0) {
+      const successCount = filtered.filter(e => e.outcome && e.outcome.status === 'success').length;
+      const failCount = filtered.filter(e => e.outcome && e.outcome.status === 'failed').length;
+      parts.push([
+        `[Evolution Memory] Recent ${filtered.length} outcomes (${successCount} success, ${failCount} failed):`,
+        ...filtered.map(formatOutcome),
+        '',
+        'Use successful approaches. Avoid repeating failed patterns.',
+      ].join('\n'));
+    }
+  }
 
-  if (filtered.length === 0) {
+  if (parts.length === 0) {
     process.stdout.write(JSON.stringify({}));
     return;
   }
 
-  const successCount = filtered.filter(e => e.outcome && e.outcome.status === 'success').length;
-  const failCount = filtered.filter(e => e.outcome && e.outcome.status === 'failed').length;
-
-  const lines = filtered.map(formatOutcome);
-  const summary = [
-    `[Evolution Memory] Recent ${filtered.length} outcomes (${successCount} success, ${failCount} failed):`,
-    ...lines,
-    '',
-    'Use successful approaches. Avoid repeating failed patterns.',
-  ].join('\n');
-
+  const out = parts.join('\n\n');
   process.stdout.write(JSON.stringify({
-    agent_message: summary,
-    additionalContext: summary,
+    agent_message: out,
+    additionalContext: out,
   }));
 }