moflo 4.9.20 → 4.9.22

package/dist/src/cli/init/executor.js

@@ -23,65 +23,64 @@ import { errorDetail } from '../shared/utils/error-detail.js';
 /**
  * Skills to copy based on configuration. Exported for integrity tests.
  */
+// Skills installed into a consumer's `<root>/.claude/skills/` by `flo init`.
+// Every entry must be a moflo-quality, consumer-runnable skill — verified to
+// reference moflo CLI/MCP tools (not upstream `claude-flow` / `agentic-flow`
+// cruft). New additions MUST pass the same audit, otherwise the drift-guard
+// test (skills-classification-drift.test.ts) fails. See INTERNAL_SKILLS for
+// skills that ship in the tarball but are deliberately NOT installed.
 export const SKILLS_MAP = {
     core: [
-        'swarm-orchestration',
-        'swarm-advanced',
-        'sparc-methodology',
-        'hooks-automation',
-        'pair-programming',
-        'verification-quality',
-        'stream-chain',
-        'skill-builder',
+        'eldar',
+        'guidance',
+        'flo-simplify',
         'reasoningbank-intelligence',
     ],
-    browser: ['browser'],
-    github: [
-        'github-code-review',
-        'github-multi-repo',
-        'github-project-management',
-        'github-release-management',
-        'github-workflow-automation',
+    memory: [
+        'memory-patterns',
+        'memory-optimization',
+        'vector-search',
+    ],
+    spells: [
+        'spell-builder',
+        'spell-schedule',
+        'connector-builder',
     ],
 };
+// Skills that ship in the npm tarball (under `node_modules/moflo/.claude/skills/`)
+// but are deliberately NOT copied into consumer projects by `flo init`. Strictly
+// moflo-internal dev tooling. The drift-guard test asserts every dir under
+// `.claude/skills/` is classified in either SKILLS_MAP or INTERNAL_SKILLS (plus
+// the special `flo` + `fl` install path handled in moflo-init.ts).
+export const INTERNAL_SKILLS = [
+    'publish', // moflo's own /publish workflow — not consumer-relevant
+    'reset-epic', // moflo's own epic test-data reset — would torch a consumer's repo
+];
 /**
  * Commands to copy based on configuration
  */
-const COMMANDS_MAP = {
-    core: ['claude-flow-help.md', 'claude-flow-swarm.md', 'claude-flow-memory.md', 'simplify.md'],
-    analysis: [],
-    automation: [],
-    github: ['github'],
-    hooks: ['hooks'],
-    monitoring: [],
-    optimization: [],
-    sparc: ['sparc'],
-};
+// Empty after #949 retired the last shipped slash commands. moflo's user-facing
+// functionality now lives entirely in `.claude/skills/` (canonical) and
+// `.claude/agents/` (subagent surface). Kept as an empty-record placeholder
+// because callers iterate `Object.entries(COMMANDS_MAP)` to gate per-category
+// install — see init-copy-maps.test.ts for the iteration contract.
+const COMMANDS_MAP = {};
 /**
  * Agents to copy based on configuration. Exported for integrity tests.
+ *
+ * Each value is a directory name under `.claude/agents/` that ships in the
+ * moflo package. After #932 retired ~50 ruflo-aspirational agents, the set
+ * is narrowed to actual development specialties Claude is likely to invoke.
  */
 export const AGENTS_MAP = {
     core: ['core'],
-    consensus: ['consensus'],
-    github: ['github'],
-    hiveMind: ['hive-mind'],
-    sparc: ['sparc'],
-    swarm: ['swarm'],
-    browser: ['browser'], // agent-browser integration
-    // V3-specific agents
-    v3: ['v3'],
-    optimization: ['optimization'],
-    testing: ['testing'],
     analysis: ['analysis'],
     architecture: ['architecture'],
+    custom: ['custom'],
     development: ['development'],
     devops: ['devops'],
     documentation: ['documentation'],
-    specialized: ['specialized'],
-    goal: ['goal'],
-    sona: ['sona'],
-    data: ['data'],
-    custom: ['custom'],
+    security: ['security'],
 };
 /**
  * Directory structure to create
@@ -1509,39 +1508,24 @@ npx moflo swarm monitor
 ### Core Development (5)
 \`coder\`, \`reviewer\`, \`tester\`, \`planner\`, \`researcher\`
 
-### V3 Specialized (4)
-\`security-architect\`, \`security-auditor\`, \`memory-specialist\`, \`performance-engineer\`
-
-### Swarm Coordination (5)
-\`hierarchical-coordinator\`, \`mesh-coordinator\`, \`adaptive-coordinator\`, \`collective-intelligence-coordinator\`, \`swarm-memory-manager\`
-
-### Consensus & Distributed (7)
-\`byzantine-coordinator\`, \`raft-manager\`, \`gossip-coordinator\`, \`consensus-builder\`, \`crdt-synchronizer\`, \`quorum-manager\`, \`security-manager\`
-
-### Performance & Optimization (5)
-\`perf-analyzer\`, \`performance-benchmarker\`, \`task-orchestrator\`, \`memory-coordinator\`, \`smart-agent\`
-
-### GitHub & Repository (9)
-\`github-modes\`, \`pr-manager\`, \`code-review-swarm\`, \`issue-tracker\`, \`release-manager\`, \`workflow-automation\`, \`project-board-sync\`, \`repo-architect\`, \`multi-repo-swarm\`
-
-### SPARC Methodology (6)
-\`sparc-coord\`, \`sparc-coder\`, \`specification\`, \`pseudocode\`, \`architecture\`, \`refinement\`
+### Code Analysis (2)
+\`code-analyzer\`, \`analyst\`
 
-### Specialized Development (8)
-\`backend-dev\`, \`mobile-dev\`, \`ml-developer\`, \`cicd-engineer\`, \`api-docs\`, \`system-architect\`, \`code-analyzer\`, \`base-template-generator\`
+### Specialized Development (5)
+\`backend-dev\`, \`frontend-dev\`, \`database-dev\`, \`cicd-engineer\`, \`api-docs\`
 
-### Testing & Validation (2)
-\`tdd-london-swarm\`, \`production-validator\`
+### Security (1)
+\`security-auditor\`
 
 ### Agent Routing by Task
-| Task Type | Recommended Agents | Topology |
-|-----------|-------------------|----------|
-| Bug Fix | researcher, coder, tester | mesh |
-| New Feature | coordinator, architect, coder, tester, reviewer | hierarchical |
-| Refactoring | architect, coder, reviewer | mesh |
-| Performance | researcher, perf-engineer, coder | hierarchical |
-| Security | security-architect, auditor, reviewer | hierarchical |
-| Docs | researcher, api-docs | mesh |
+| Task Type | Recommended Agents |
+|-----------|--------------------|
+| Bug Fix | researcher, coder, tester |
+| New Feature | planner, coder, tester, reviewer |
+| Refactoring | coder, reviewer |
+| Security | security-auditor, reviewer |
+| GitHub workflow | coder, reviewer (with \`gh\` CLI) |
+| Docs | researcher, api-docs |
 
 ---
 

package/dist/src/cli/init/helpers-generator.js

@@ -207,7 +207,7 @@ var path = require('path');
 var PROJECT_DIR = (process.env.CLAUDE_PROJECT_DIR || process.cwd()).replace(/^\\/([a-z])\\//i, '$1:/');
 var STATE_FILE = path.join(PROJECT_DIR, '.claude', 'workflow-state.json');
 
-var STATE_DEFAULTS = { tasksCreated: false, taskCount: 0, memorySearched: false, memorySearchedBy: {}, memoryRequired: true, learningsStored: false, testsRun: false, simplifyRun: false, interactionCount: 0, sessionStart: null, lastBlockedAt: null };
+var STATE_DEFAULTS = { tasksCreated: false, taskCount: 0, memorySearched: false, memorySearchedBy: {}, memoryRequired: true, learningsStored: false, testsRun: false, simplifyRun: false, interactionCount: 0, sessionStart: null, lastBlockedAt: null, lastNamespaceHint: '', lastNamespaceHintEmittedBy: {}, flMode: null, swarmInitialized: false, hiveInitialized: false };
 
 function readState() {
   try {
@@ -255,7 +255,7 @@ function writeState(s) {
 
 // Load moflo.yaml gate config (defaults: all enabled)
 function loadGateConfig() {
-  var defaults = { memory_first: true, task_create_first: true, context_tracking: true, testing_gate: true, simplify_gate: true, learnings_gate: true };
+  var defaults = { memory_first: true, task_create_first: true, context_tracking: true, testing_gate: true, simplify_gate: true, learnings_gate: true, swarm_invocation_gate: true };
   try {
     var yamlPath = path.join(PROJECT_DIR, 'moflo.yaml');
     if (fs.existsSync(yamlPath)) {
@@ -266,6 +266,7 @@ function loadGateConfig() {
       if (/testing_gate:\\s*false/i.test(content)) defaults.testing_gate = false;
       if (/simplify_gate:\\s*false/i.test(content)) defaults.simplify_gate = false;
       if (/learnings_gate:\\s*false/i.test(content)) defaults.learnings_gate = false;
+      if (/swarm_invocation_gate:\\s*false/i.test(content)) defaults.swarm_invocation_gate = false;
     }
   } catch (e) { /* use defaults */ }
   return defaults;
@@ -278,6 +279,83 @@ var EXEMPT = ['.claude/', '.claude\\\\', 'CLAUDE.md', 'MEMORY.md', 'workflow-sta
 var DANGEROUS = ['rm -rf /', 'format c:', 'del /s /q c:\\\\', ':(){:|:&};:', 'mkfs.', '> /dev/sda'];
 var DIRECTIVE_RE = /^(yes|no|yeah|yep|nope|sure|ok|okay|correct|right|exactly|perfect)\\b/i;
 var TASK_RE = /\\b(fix|bug|error|implement|add|create|build|write|refactor|debug|test|feature|issue|security|optimi)\\b/i;
+
+// Namespace classification (#931). Hint stored on workflow-state and emitted
+// once by check-before-agent at Agent-spawn time — was emitted on every prompt
+// before, costing ~40 tokens × every prompt × every consumer.
+//
+// SYNC: these regexes + classifyNamespaceHint + applyPromptStateReset are
+// duplicated verbatim in bin/gate.cjs (canonical, synced to consumer
+// .claude/helpers/gate.cjs by post-install-bootstrap). Any edit MUST be
+// applied to both — this template is the fallback for the flo-init path
+// where source helpers cannot be located, so it must keep parity.
+var NS_LEARNINGS_RE = /\\b(remember|recall|insight|lesson learned|gotcha|post.?mortem)\\b|we (decid|agree|chose|said)/;
+var NS_TEST_RE = /\\b(test|spec|coverage|tested|test case|test cases|tests for|spec for)\\b/;
+var NS_EXPLICIT = [
+  { pattern: /\\b(pattern|convention|best practice|style|coding rule)\\b/, ns: 'patterns', label: 'code patterns and conventions' },
+  { pattern: /\\b(code.?map|file structure|project structure|directory)\\b/, ns: 'code-map', label: 'codebase navigation' },
+];
+var NS_PATTERN_RES = [/\\b(template|example|similar to|how do we|how should)\\b/];
+var NS_DOMAIN_RES = [
+  /\\b(guidance|guide|docs|documentation|rules|how-to)\\b/,
+  /\\b(architecture|design|domain|tenant|migrat|schema|deploy)/,
+  /\\b(rule|requirement|constraint|compliance)\\b/,
+];
+var NS_NAV_RES = [
+  /\\b(find|where|which file|look up|locate|endpoint|route|url|path)\\b/,
+  /\\b(class|function|method|component|service|entity|module)\\b/,
+];
+
+// Detect whether the current prompt invoked /fl or /flo with a swarm/hive flag
+// (#952). When set, check-before-agent BLOCKS the Agent spawn until the matching
+// MCP init has been recorded — the user explicitly opted in to the protected
+// coordination surface, so falling back to raw Agent dispatch silently regresses
+// headline moflo product capability.
+//
+// SYNC: duplicated verbatim in bin/gate.cjs.
+function detectFlMode(promptText) {
+  var p = promptText || '';
+  if (!/^\\s*\\/(?:fl|flo)\\b/i.test(p)) return null;
+  if (/(?:^|\\s)(?:-s|--swarm)\\b/.test(p)) return 'swarm';
+  if (/(?:^|\\s)(?:-h|--hive)\\b/.test(p)) return 'hive';
+  return null;
+}
+
+function classifyNamespaceHint(promptText) {
+  var lower = (promptText || '').toLowerCase();
+  if (NS_TEST_RE.test(lower)) return 'Memory namespace hint: use "tests" for test inventory and coverage lookups.';
+  if (NS_LEARNINGS_RE.test(lower)) return 'Memory namespace hint: use "learnings" for user-directed decisions and distilled insights.';
+  for (var i = 0; i < NS_EXPLICIT.length; i++) {
+    if (NS_EXPLICIT[i].pattern.test(lower)) return 'Memory namespace hint: use "' + NS_EXPLICIT[i].ns + '" for ' + NS_EXPLICIT[i].label + '.';
+  }
+  for (var j = 0; j < NS_DOMAIN_RES.length; j++) {
+    if (NS_DOMAIN_RES[j].test(lower)) return 'Memory namespace hint: search "guidance" and "learnings" for domain rules and project decisions.';
+  }
+  for (var k = 0; k < NS_PATTERN_RES.length; k++) {
+    if (NS_PATTERN_RES[k].test(lower)) return 'Memory namespace hint: use "patterns" for code patterns and conventions.';
+  }
+  for (var m = 0; m < NS_NAV_RES.length; m++) {
+    if (NS_NAV_RES[m].test(lower)) return 'Memory namespace hint: use "code-map" for codebase navigation.';
+  }
+  return '';
+}
+
+function applyPromptStateReset(state, promptText) {
+  state.memorySearched = false;
+  state.memorySearchedBy = {};
+  var DIRECTIVE_MAX_LEN = 20;
+  var escaped = /^@@\\s*/.test(promptText || '');
+  state.memoryRequired = !escaped && (promptText || '').length >= 4 && (TASK_RE.test(promptText || '') || (promptText || '').length > DIRECTIVE_MAX_LEN);
+  state.lastNamespaceHint = classifyNamespaceHint(promptText);
+  // Per-actor emission tracking — fresh window each prompt so subagents that
+  // spawn their own agents still see the hint on their first check-before-agent.
+  state.lastNamespaceHintEmittedBy = {};
+  // #952 — derive flMode from the user prompt and reset the matching init
+  // flag. Each /fl invocation must call its protected MCP init.
+  state.flMode = detectFlMode(promptText);
+  state.swarmInitialized = false;
+  state.hiveInitialized = false;
+}
 var TEST_RUNNER_RE = /(?:^|[^a-z])(?:npm|yarn|pnpm|bun)\\s+(?:run\\s+)?(?:test|t)(?:[:\\s]|$)|\\b(?:npx|pnpx)\\s+(?:vitest|jest|mocha|ava|tap|jasmine|pytest)\\b|(?:^|;|&&|\\|\\|)\\s*(?:vitest|jest|pytest|mocha|jasmine|tap|ava)\\s|\\b(?:cargo|go|deno|dotnet|mvn)\\s+test\\b|\\bgradle\\w*\\s+test\\b/i;
 var EDIT_RESET_SKIP_BOTH_RE = /\\.(md|markdown|txt|rst|adoc|lock|gitignore)$|(?:^|[\\\\\\/])(CHANGELOG(?:\\.md)?|\\.env\\.example|package-lock\\.json|pnpm-lock\\.yaml|yarn\\.lock|bun\\.lockb)$/i;
 // Test files: invalidate testsRun but preserve simplifyRun (#908) — /simplify
@@ -290,6 +368,8 @@ switch (command) {
     // Advisory only — agent spawning is never blocked.
     // Memory-first enforcement happens at the scan/read gate layer.
     // SubagentStart hook injects guidance directive into subagent context.
+    // #931 — TaskCreate REMINDER + namespace hint moved here from
+    // prompt-reminder so they emit only when Claude is about to spawn an Agent.
     var s = readState();
     if (config.task_create_first && !s.tasksCreated) {
       process.stdout.write('REMINDER: Use TaskCreate before spawning agents. Task tool is blocked until then.\\n');
@@ -297,6 +377,63 @@ switch (command) {
     if (config.memory_first && s.memoryRequired && !s.memorySearched) {
       process.stdout.write('REMINDER: Search memory (mcp__moflo__memory_search) before spawning agents.\\n');
     }
+    if (s.lastNamespaceHint) {
+      // Per-actor single-shot — each session_id emits the hint at most once
+      // per prompt. Subagents that spawn their own agents still see it on
+      // their first check-before-agent because their session_id is its own
+      // bucket. Falls back to a _legacy_ bucket when HOOK_SESSION_ID is
+      // missing (older Claude Code, direct CLI). The map clears on every
+      // new prompt via applyPromptStateReset.
+      var sid = process.env.HOOK_SESSION_ID || '';
+      var emittedBy = s.lastNamespaceHintEmittedBy || {};
+      var bucket = sid || '_legacy_';
+      if (!emittedBy[bucket]) {
+        process.stdout.write(s.lastNamespaceHint + '\\n');
+        emittedBy[bucket] = true;
+        s.lastNamespaceHintEmittedBy = emittedBy;
+        writeState(s);
+      }
+    }
+    // #952 — when /fl was invoked with -s/-h, the protected MCP init must run
+    // BEFORE any Agent spawn. Hard block: the user explicitly opted in to
+    // moflo's coordination surface, so silently dispatching Agent calls
+    // without mcp__moflo__swarm_init / mcp__moflo__hive-mind_init is the
+    // failure mode this gate exists to prevent (CLAUDE.md "⛔ Protected
+    // functionality"). Other Agent uses remain advisory.
+    if (config.swarm_invocation_gate) {
+      if (s.flMode === 'swarm' && !s.swarmInitialized) {
+        process.stderr.write('BLOCKED: /fl was invoked with -s/--swarm but mcp__moflo__swarm_init has not been called.\\n');
+        process.stderr.write('Run mcp__moflo__swarm_init first, then mcp__moflo__agent_spawn for each role, then dispatch Agent.\\n');
+        process.stderr.write('See .claude/skills/fl/execution-modes.md "SWARM mode" and CLAUDE.md "⛔ Protected functionality".\\n');
+        process.stderr.write('Disable via moflo.yaml: gates: swarm_invocation_gate: false\\n');
+        process.exit(2);
+      }
+      if (s.flMode === 'hive' && !s.hiveInitialized) {
+        process.stderr.write('BLOCKED: /fl was invoked with -h/--hive but mcp__moflo__hive-mind_init has not been called.\\n');
+        process.stderr.write('Run mcp__moflo__hive-mind_init first, then dispatch Agent or hive-mind workers.\\n');
+        process.stderr.write('See .claude/skills/fl/execution-modes.md "HIVE-MIND mode" and CLAUDE.md "⛔ Protected functionality".\\n');
+        process.stderr.write('Disable via moflo.yaml: gates: swarm_invocation_gate: false\\n');
+        process.exit(2);
+      }
+    }
+    break;
+  }
+  case 'record-swarm-init': {
+    // #952 — wired to mcp__moflo__swarm_init PostToolUse.
+    var s = readState();
+    if (!s.swarmInitialized) {
+      s.swarmInitialized = true;
+      writeState(s);
+    }
+    break;
+  }
+  case 'record-hive-init': {
+    // #952 — wired to mcp__moflo__hive-mind_init PostToolUse.
+    var s = readState();
+    if (!s.hiveInitialized) {
+      s.hiveInitialized = true;
+      writeState(s);
+    }
     break;
   }
   case 'check-before-scan': {
@@ -363,7 +500,8 @@ switch (command) {
     break;
   }
   case 'record-skill-run': {
-    if ((process.env.TOOL_INPUT_skill || '') === 'simplify') {
+    var skName = (process.env.TOOL_INPUT_skill || '');
+    if (skName === 'simplify' || skName === 'flo-simplify') {
       var s = readState();
       if (!s.simplifyRun) {
         s.simplifyRun = true;
@@ -397,7 +535,7 @@ switch (command) {
     var s = readState();
     var missing = [];
     if (config.testing_gate && !s.testsRun) missing.push('tests have not run since the last code edit (run npm test, vitest, jest, pytest, or similar)');
-    if (config.simplify_gate && !s.simplifyRun) missing.push('/simplify has not run since the last code edit');
+    if (config.simplify_gate && !s.simplifyRun) missing.push('/flo-simplify has not run since the last code edit');
     if (config.learnings_gate && !s.learningsStored) missing.push('learnings have not been stored (call mcp__moflo__memory_store)');
     if (missing.length === 0) break;
     process.stderr.write('BLOCKED: gh pr create requires the following before opening a PR:\\n');
@@ -422,18 +560,14 @@ switch (command) {
     break;
   }
   case 'prompt-reminder': {
+    // Full per-prompt reset (first UserPromptSubmit hook via prompt-hook.mjs).
+    // Owns interactionCount + Context warnings. TaskCreate REMINDER and
+    // namespace hint moved to check-before-agent (#931).
     var s = readState();
-    s.memorySearched = false;
-    // Wipe per-actor memory tracking too — a new user prompt is a fresh window
-    // for both parent AND any subagents the parent may spawn during this turn.
-    s.memorySearchedBy = {};
-    // learningsStored is session-scoped — once stored, it stays true until session reset.
-    // Resetting per-prompt caused false blocks when PR creation was on a later prompt.
     var prompt = process.env.CLAUDE_USER_PROMPT || '';
-    s.memoryRequired = prompt.length >= 4 && !DIRECTIVE_RE.test(prompt) && (TASK_RE.test(prompt) || prompt.length > 80);
+    applyPromptStateReset(s, prompt);
     s.interactionCount = (s.interactionCount || 0) + 1;
     writeState(s);
-    if (!s.tasksCreated) console.log('REMINDER: Use TaskCreate before spawning agents. Task tool is blocked until then.');
     if (config.context_tracking) {
       var ic = s.interactionCount;
       if (ic > 30) console.log('Context: CRITICAL. Commit, store learnings, suggest new session.');
@@ -442,12 +576,27 @@ switch (command) {
     }
     break;
   }
+  case 'prompt-state-reset': {
+    // Defensive safety-net (second UserPromptSubmit hook). Idempotent state
+    // reset only — no interactionCount increment, no emission. Ensures the
+    // per-prompt reset still happens if prompt-hook.mjs throws (#931). Skip
+    // the disk write when prompt-reminder already wrote the byte-identical
+    // post-reset state (the normal no-exception path).
+    var s = readState();
+    var prompt = process.env.CLAUDE_USER_PROMPT || '';
+    var before = JSON.stringify(s);
+    applyPromptStateReset(s, prompt);
+    if (JSON.stringify(s) !== before) writeState(s);
+    break;
+  }
   case 'compact-guidance': {
     console.log('Pre-Compact: Check CLAUDE.md for rules. Use memory search to recover context after compact.');
     break;
   }
   case 'session-reset': {
-    writeState({ tasksCreated: false, taskCount: 0, memorySearched: false, memorySearchedBy: {}, memoryRequired: true, learningsStored: false, testsRun: false, simplifyRun: false, interactionCount: 0, sessionStart: new Date().toISOString(), lastBlockedAt: null });
+    // Derive from STATE_DEFAULTS so adding a new state field requires only one
+    // edit (the defaults object).
+    writeState(Object.assign({}, STATE_DEFAULTS, { sessionStart: new Date().toISOString() }));
     break;
   }
   default:
@@ -560,45 +709,9 @@ try {
   });
 } catch (err) { output = (err && err.stdout) || ''; }
 
-// Classify prompt for namespace hint
-var lower = userPrompt.toLowerCase();
-
-var LEARNINGS_HINTS = /\\b(remember|recall|insight|lesson learned|gotcha|post.?mortem)\\b|we (decid|agree|chose|said)/;
-var TEST_HINTS = /\\b(test|spec|coverage|tested|test case|test cases|tests for|spec for)\\b/;
-var EXPLICIT_NS = [
-  { pattern: /\\b(pattern|convention|best practice|style|coding rule)\\b/, ns: 'patterns', label: 'code patterns and conventions' },
-  { pattern: /\\b(code.?map|file structure|project structure|directory)\\b/, ns: 'code-map', label: 'codebase navigation' },
-];
-var PATTERN_HINTS = [/\\b(template|example|similar to|how do we|how should)\\b/];
-var DOMAIN_HINTS = [
-  /\\b(guidance|guide|docs|documentation|rules|how-to)\\b/,
-  /\\b(architecture|design|domain|tenant|migrat|schema|deploy)/,
-  /\\b(rule|requirement|constraint|compliance)\\b/,
-];
-var NAV_PATTERNS = [
-  /\\b(find|where|which file|look up|locate|endpoint|route|url|path)\\b/,
-  /\\b(class|function|method|component|service|entity|module)\\b/,
-];
-
-var nsHint = '';
-if (TEST_HINTS.test(lower)) {
-  nsHint = 'Memory namespace hint: use "tests" for test inventory and coverage lookups.';
-} else if (LEARNINGS_HINTS.test(lower)) {
-  nsHint = 'Memory namespace hint: use "learnings" for user-directed decisions and distilled insights.';
-} else {
-  var found = EXPLICIT_NS.find(function(e) { return e.pattern.test(lower); });
-  if (found) {
-    nsHint = 'Memory namespace hint: use "' + found.ns + '" for ' + found.label + '.';
-  } else if (DOMAIN_HINTS.some(function(p) { return p.test(lower); })) {
-    nsHint = 'Memory namespace hint: search "guidance" and "learnings" for domain rules and project decisions.';
-  } else if (PATTERN_HINTS.some(function(p) { return p.test(lower); })) {
-    nsHint = 'Memory namespace hint: use "patterns" for code patterns and conventions.';
-  } else if (NAV_PATTERNS.some(function(p) { return p.test(lower); })) {
-    nsHint = 'Memory namespace hint: use "code-map" for codebase navigation.';
-  }
-}
-
-var parts = [output.trim(), nsHint].filter(Boolean);
+// #931 — Namespace hint classification moved into gate.cjs (computed by
+// prompt-reminder, stored on workflow-state, emitted once by check-before-agent).
+var parts = [output.trim()].filter(Boolean);
 if (parts.length) process.stdout.write(parts.join('\\n') + '\\n');
 process.exit(0);
 `;