@monoes/monomindcli 2.16.2 → 2.16.3

package/.claude/helpers/router.cjs

@@ -1,161 +1,16 @@
 'use strict';
 /**
- * Keyword-based task router for hook-handler.cjs
- * Returns: { agent, agentSlug, confidence, reason, semanticRouting, specificAgents, skillMatches, extrasMatches }
+ * Skill keyword matcher for the prompt hook.
+ *
+ * handlers/route-handler.cjs calls matchSkills() for its skill hint. Agent
+ * picks come from .monomind/registry.json via the pick index, not from this
+ * file: the keyword agent table (routeTask) that used to live here named
+ * slugs that are not installed agents, and nothing at runtime called it.
  *
  * Exports:
- *   routeTask(prompt)           → routing result object
- *   routeTaskSemantic(prompt)   → alias for routeTask
- *   matchSkills(prompt, topN)   → array of { skill, invoke, score }
- *   matchExtras(prompt, topN)   → array of { slug, name, category, score }
- *   buildCategoryList()         → array of { name, count, examples }
- *   getAgentsInCategory(cat)    → array of { slug, name }
- *   AGENT_CAPABILITIES          → { [slug]: string[] }
- *   TASK_PATTERNS               → { [agentSlug]: RegExp }
+ *   matchSkills(prompt, topN)   → array of { skill, invoke, description, score }
  */
 
-// ── Dev patterns (slug-correct) ─────────────────────────────────────────────
-// TASK_PATTERNS: { keywords: agentSlug } — exported for introspection
-// Values are agent slugs; keys describe what keywords trigger that agent.
-
-var TASK_PATTERNS = {
-  'test|spec|coverage|vitest|jest|mocha|e2e':                  'tester',
-  'review|audit|code quality|lint|refactor|cleanup':           'reviewer',
-  'architect|system design|ADR|bounded context|architecture':  'architect',
-  'security|vulnerability|CVE|injection|XSS|CSRF|OWASP':       'security-engineer',
-  'deploy|CI/CD|docker|kubernetes|infra|devops|helm|terraform': 'devops',
-  'document|readme|docs|api reference|jsdoc':                  'technical-writer',
-  'research|investigate|explore|analyze|survey|compare':       'researcher',
-  'plan|roadmap|prioritize|breakdown|estimate':                 'planner',
-  'mobile|ios|android|react native|flutter':                   'mobile-dev',
-  'ml|machine learning|neural network|model training':          'ai-engineer',
-  'api|rest|graphql|endpoint|http|websocket|grpc|optimize':    'backend-dev',
-  'ui|frontend|react|vue|component|css|layout|style':          'frontend-dev',
-  'bug|fix|error|feature|implement|build|create|develop':      'coder',
-};
-
-// Internal routing patterns (regex per slug for routeTask)
-var _ROUTING_PATTERNS = {
-  'tester':      /\b(test|tests|spec|coverage|vitest|jest|mocha|e2e)\b/i,
-  'reviewer':    /\b(review|audit|code quality|lint|smell|refactor|clean up|cleanup)\b/i,
-  'architect':   /\b(architect|system design|ADR|domain|bounded context|microservice|architecture)\b/i,
-  'security-engineer': /\b(security|vulnerability|CVE|injection|XSS|CSRF|OWASP)\b/i,
-  'devops':      /\b(deploy|CI\/CD|docker|kubernetes|infra|devops|helm|terraform)\b/i,
-  'technical-writer': /\b(document|readme|docs|api reference|jsdoc|write up)\b/i,
-  'researcher':  /\b(research|investigate|explore|analyze|survey|compare)\b/i,
-  'planner':     /\b(plan|roadmap|prioritize|breakdown|estimate)\b/i,
-  'mobile-dev':  /\b(mobile|ios|android|react native|flutter)\b/i,
-  'ai-engineer': /\b(ml|machine learning|neural network|model training|inference)\b/i,
-  'backend-dev': /\b(api|rest|graphql|endpoint|http|websocket|grpc|optimize)\b/i,
-  'frontend-dev': /\b(ui|frontend|react|vue|component|css|layout|style)\b/i,
-  'coder':       /\b(bug|fix|error|exception|crash|broken|fail|regression|feature|implement|add|build|create|develop|new|memory|vector|embedding|hook|swarm|agent|mcp|cli|routing|monomind)\b/i,
-};
-
-var TASK_CONFIDENCES = {
-  'tester':      0.85,
-  'reviewer':    0.82,
-  'architect':   0.85,
-  'security-engineer': 0.90,
-  'devops':      0.85,
-  'technical-writer': 0.82,
-  'researcher':  0.78,
-  'planner':     0.80,
-  'mobile-dev':  0.88,
-  'ai-engineer': 0.85,
-  'backend-dev': 0.80,
-  'frontend-dev': 0.80,
-  'coder':       0.80,
-};
-
-var TASK_AGENTS = {
-  'tester':      'Tester',
-  'reviewer':    'Reviewer',
-  'architect':   'Architect',
-  'security-engineer': 'Security Engineer',
-  'devops':      'DevOps',
-  'technical-writer': 'Technical Writer',
-  'researcher':  'Researcher',
-  'planner':     'Planner',
-  'mobile-dev':  'Mobile Developer',
-  'ai-engineer': 'AI Engineer',
-  'backend-dev': 'Backend Developer',
-  'frontend-dev': 'Frontend Developer',
-  'coder':       'Coder',
-};
-
-// Priority order: higher-priority slugs checked first
-var DEV_PRIORITY = [
-  'tester', 'reviewer', 'architect', 'security-engineer', 'devops',
-  'mobile-dev', 'ai-engineer', 'frontend-dev', 'backend-dev',
-  'researcher', 'planner', 'technical-writer', 'coder',
-];
-
-// ── AGENT_CAPABILITIES ────────────────────────────────────────────────────────
-
-var AGENT_CAPABILITIES = {
-  'coder':         ['implement', 'fix', 'build', 'develop', 'create'],
-  'tester':        ['test', 'spec', 'coverage', 'vitest', 'jest', 'e2e'],
-  'reviewer':      ['review', 'audit', 'refactor', 'code quality', 'lint'],
-  'researcher':    ['research', 'investigate', 'analyze', 'survey', 'compare'],
-  'architect':     ['design', 'architecture', 'ADR', 'domain', 'microservice'],
-  'planner':       ['plan', 'roadmap', 'strategy', 'prioritize', 'estimate'],
-  'security-engineer': ['security', 'vulnerability', 'CVE', 'XSS', 'CSRF'],
-  'backend-dev':   ['api', 'rest', 'graphql', 'endpoint', 'http', 'grpc'],
-  'frontend-dev':  ['ui', 'frontend', 'react', 'css', 'component', 'layout'],
-  'devops':        ['deploy', 'docker', 'kubernetes', 'CI/CD', 'terraform'],
-  'mobile-dev':    ['mobile', 'ios', 'android', 'react native', 'flutter'],
-  'ai-engineer':   ['ml', 'machine learning', 'model', 'neural', 'inference'],
-  'technical-writer': ['docs', 'readme', 'document', 'jsdoc', 'api reference'],
-};
-
-// ── Non-dev domain agent registry ─────────────────────────────────────────────
-
-var DOMAIN_AGENTS = [
-  // Marketing
-  { slug: 'content-strategist', name: 'Content Strategist', category: 'marketing',
-    keywords: /\b(content|brand|blogging|copywriting|content strategy)\b/i },
-  { slug: 'seo-specialist', name: 'SEO Specialist', category: 'marketing',
-    keywords: /\b(seo|search engine|keyword research|backlink|organic traffic)\b/i },
-  { slug: 'social-media-manager', name: 'Social Media Manager', category: 'marketing',
-    keywords: /\b(social media|instagram|tiktok|twitter|linkedin|facebook|campaign)\b/i },
-  { slug: 'marketing-analyst', name: 'Marketing Analyst', category: 'marketing',
-    keywords: /\b(marketing|advertising|analytics|conversion|funnel|cpm|cpa)\b/i },
-
-  // Sales
-  { slug: 'sales-strategist', name: 'Sales Strategist', category: 'sales',
-    keywords: /\b(sales|crm|lead generation|prospect|quota|sales revenue)\b/i },
-  { slug: 'account-manager', name: 'Account Manager', category: 'sales',
-    keywords: /\b(account management|client relationship|upsell|renewal|b2b)\b/i },
-
-  // Academic
-  { slug: 'academic-researcher', name: 'Academic Researcher', category: 'academic',
-    keywords: /\b(anthropolog|ethnograph|kinship|cultural ritual|qualitative study|thesis|dissertation|peer review|academic)\b/i },
-  { slug: 'data-scientist', name: 'Data Scientist', category: 'academic',
-    keywords: /\b(statistical analysis|regression|hypothesis|p-value|dataset|R studio)\b/i },
-
-  // Game development
-  { slug: 'game-developer', name: 'Game Developer', category: 'game-development',
-    keywords: /\b(unity|unreal|godot|game engine|shader|sprite|tilemap|game jam)\b/i },
-  { slug: 'game-designer', name: 'Game Designer', category: 'game-development',
-    keywords: /\b(game design|game mechanic|level design|player experience|narrative design)\b/i },
-
-  // Legal / Finance
-  { slug: 'legal-advisor', name: 'Legal Advisor', category: 'legal',
-    keywords: /\b(legal|contract|compliance|regulation|gdpr|liability|intellectual property)\b/i },
-  { slug: 'financial-analyst', name: 'Financial Analyst', category: 'finance',
-    keywords: /\b(finance|investment|portfolio|valuation|balance sheet|ROI|P&L)\b/i },
-
-  // HR / Operations
-  { slug: 'hr-specialist', name: 'HR Specialist', category: 'hr',
-    keywords: /\b(hiring|recruitment|onboarding|performance review|employee|hr|human resources)\b/i },
-];
-
-// Categories that are opt-in (only returned when keywords match)
-var OPT_IN_CATEGORIES = new Set(['academic', 'game-development', 'legal', 'finance', 'hr']);
-
-// Marketing is also opt-in — keywords must match
-var MARKETING_OPTIN = /\b(marketing|seo|social media|advertising|campaign|content strategy|tiktok|instagram|brand)\b/i;
-
 // ── Skills registry ────────────────────────────────────────────────────────────
 // Loaded from .claude/helpers/skill-registry.json, which is generated from the
 // live .claude/commands + .claude/skills trees by build-skill-registry.cjs.
@@ -217,174 +72,10 @@ function loadSkills() {
   return skills;
 }
 
-// ── Feedback weight system ────────────────────────────────────────────────────
-// Reads routing-feedback.jsonl (written by session-handler at session-end) and
-// computes per-agent success rates. Adjusts routing confidence by up to +/-0.10
-// so agents with proven track records get a slight boost and poor performers get
-// dampened. Cached with a 60-second TTL to avoid repeated disk reads.
-
-var _feedbackWeightsCache = null;
-var _feedbackWeightsCacheTs = 0;
-var FEEDBACK_TTL_MS = 60000;
-
-function loadFeedbackWeights() {
-  var now = Date.now();
-  if (_feedbackWeightsCache && (now - _feedbackWeightsCacheTs) < FEEDBACK_TTL_MS) {
-    return _feedbackWeightsCache;
-  }
-  var weights = {};
-  try {
-    var fs = require('fs');
-    var path = require('path');
-    var cwd = process.env.CLAUDE_PROJECT_DIR || process.cwd();
-    var feedbackPath = path.join(cwd, '.monomind', 'routing-feedback.jsonl');
-    var MAX_SIZE = 512 * 1024;
-    if (!fs.existsSync(feedbackPath)) {
-      _feedbackWeightsCache = weights;
-      _feedbackWeightsCacheTs = now;
-      return weights;
-    }
-    var stat = fs.statSync(feedbackPath);
-    if (stat.size > MAX_SIZE) {
-      _feedbackWeightsCache = weights;
-      _feedbackWeightsCacheTs = now;
-      return weights;
-    }
-    var lines = fs.readFileSync(feedbackPath, 'utf-8').trim().split('\n').filter(Boolean);
-    // Use last 200 entries for weight calculation
-    var recent = lines.slice(-200);
-    var agentStats = {};
-    for (var i = 0; i < recent.length; i++) {
-      try {
-        var entry = JSON.parse(recent[i]);
-        var agent = entry.suggestedAgent;
-        if (!agent) continue;
-        // Skip no-evidence records (sessionSuccess null/missing) — they carry no signal
-        if (typeof entry.intelligenceFeedback !== 'boolean') continue;
-        if (!agentStats[agent]) agentStats[agent] = { total: 0, successes: 0 };
-        agentStats[agent].total++;
-        if (entry.intelligenceFeedback === true) agentStats[agent].successes++;
-      } catch (e) {}
-    }
-    // Compute weight adjustments: agents with >= 5 data points get a weight
-    // successRate > 0.7 → positive boost (up to +0.10)
-    // successRate < 0.4 → negative dampen (down to -0.10)
-    // In between → neutral (0)
-    for (var ag in agentStats) {
-      var s = agentStats[ag];
-      if (s.total < 5) continue;
-      var rate = s.successes / s.total;
-      if (rate > 0.7) {
-        weights[ag] = Math.min(0.10, (rate - 0.7) * 0.33);
-      } else if (rate < 0.4) {
-        weights[ag] = Math.max(-0.10, (rate - 0.4) * 0.25);
-      }
-    }
-  } catch (e) { /* non-fatal */ }
-  _feedbackWeightsCache = weights;
-  _feedbackWeightsCacheTs = now;
-  return weights;
-}
-
-function applyFeedbackWeight(agentSlug, baseConfidence) {
-  var weights = loadFeedbackWeights();
-  var adj = weights[agentSlug] || weights[TASK_AGENTS[agentSlug]] || 0;
-  if (adj === 0) return baseConfidence;
-  return Math.max(0, Math.min(1.0, baseConfidence + adj));
-}
-
 // ── Utilities ─────────────────────────────────────────────────────────────────
 
 var MAX_PROMPT = 2000;
 
-// ── routeTask ─────────────────────────────────────────────────────────────────
-
-function routeTask(prompt) {
-  // Empty / null → confidence 0
-  if (!prompt || typeof prompt !== 'string' || prompt.trim() === '') {
-    return {
-      agent: 'coder',
-      agentSlug: 'coder',
-      confidence: 0,
-      reason: 'Default routing — empty input',
-      semanticRouting: false,
-      specificAgents: [],
-      skillMatches: [],
-      extrasMatches: [],
-    };
-  }
-
-  var safePrompt = prompt.slice(0, MAX_PROMPT);
-
-  // Skills are always checked — they're the primary value of routing now.
-  var skills = matchSkills(safePrompt);
-
-  // Check non-dev domain agents first (only if opt-in keywords match)
-  var extras = matchExtras(safePrompt);
-  if (extras.length > 0) {
-    var topExtra = extras[0];
-    return {
-      agent: topExtra.name,
-      agentSlug: topExtra.slug,
-      confidence: applyFeedbackWeight(topExtra.slug, 0.80),
-      reason: 'Domain: ' + topExtra.category,
-      semanticRouting: false,
-      specificAgents: extras,
-      skillMatches: skills,
-      extrasMatches: extras,
-    };
-  }
-
-  // Check if a skill matched strongly (score >= 2) — if so, skip broad catch-all agents
-  var hasStrongSkill = false;
-  for (var si = 0; si < skills.length; si++) {
-    if (skills[si].score >= 2) { hasStrongSkill = true; break; }
-  }
-
-  // Check dev patterns in priority order
-  for (var i = 0; i < DEV_PRIORITY.length; i++) {
-    var slug = DEV_PRIORITY[i];
-    // Skip the broad "coder" catch-all when a specialized skill matched strongly
-    if (slug === 'coder' && hasStrongSkill) continue;
-    var pattern = _ROUTING_PATTERNS[slug];
-    if (pattern && pattern.test(safePrompt)) {
-      // Count how many distinct keywords from the pattern actually matched
-      // so confidence reflects real match quality, not just a static constant.
-      var caps = AGENT_CAPABILITIES[slug] || [];
-      var promptLower = safePrompt.toLowerCase();
-      var matchedKw = 0;
-      for (var ki = 0; ki < caps.length; ki++) {
-        if (promptLower.indexOf(caps[ki].toLowerCase()) !== -1) matchedKw++;
-      }
-      var baseConf = TASK_CONFIDENCES[slug] || 0.75;
-      var matchBonus = Math.min(0.10, matchedKw * 0.02);
-      var confidence = Math.min(0.98, applyFeedbackWeight(slug, baseConf + matchBonus));
-      return {
-        agent: TASK_AGENTS[slug],
-        agentSlug: slug,
-        confidence: confidence,
-        reason: ('Keyword match: ' + slug + ' (' + matchedKw + ' kw)').slice(0, 80),
-        semanticRouting: false,
-        specificAgents: [{ slug: slug, name: TASK_AGENTS[slug], confidence: confidence }],
-        skillMatches: skills,
-        extrasMatches: [],
-      };
-    }
-  }
-
-  // No match → default
-  return {
-    agent: 'coder',
-    agentSlug: 'coder',
-    confidence: 0.5,
-    reason: 'Default routing — no strong keyword match',
-    semanticRouting: false,
-    specificAgents: [],
-    skillMatches: skills,
-    extrasMatches: [],
-  };
-}
-
 // ── matchSkills ────────────────────────────────────────────────────────────────
 
 /** Conservative suffix stemmer so "animate" matches the keyword "animation"
@@ -501,71 +192,8 @@ function matchSkills(prompt, topN) {
   return scored.slice(0, topN);
 }
 
-// ── matchExtras ────────────────────────────────────────────────────────────────
-
-function matchExtras(prompt, topN) {
-  if (!prompt || typeof prompt !== 'string') return [];
-  topN = topN || 8;
-  var safePrompt = prompt.slice(0, MAX_PROMPT);
-
-  var scored = [];
-  for (var i = 0; i < DOMAIN_AGENTS.length; i++) {
-    var a = DOMAIN_AGENTS[i];
-    if (!a.keywords.test(safePrompt)) continue;
-
-    // Opt-in categories: only include if keywords explicitly match
-    if (OPT_IN_CATEGORIES.has(a.category)) {
-      scored.push({ slug: a.slug, name: a.name, category: a.category, score: 1.0 });
-      continue;
-    }
-
-    // Marketing is opt-in too
-    if (a.category === 'marketing') {
-      if (MARKETING_OPTIN.test(safePrompt)) {
-        scored.push({ slug: a.slug, name: a.name, category: a.category, score: 1.0 });
-      }
-      continue;
-    }
-
-    // Sales: only include when sales keywords match
-    scored.push({ slug: a.slug, name: a.name, category: a.category, score: 1.0 });
-  }
-
-  scored.sort(function(a, b) { return b.score - a.score; });
-  return scored.slice(0, topN);
-}
-
-// ── buildCategoryList ─────────────────────────────────────────────────────────
-
-function buildCategoryList() {
-  var catMap = {};
-  for (var i = 0; i < DOMAIN_AGENTS.length; i++) {
-    var a = DOMAIN_AGENTS[i];
-    if (!catMap[a.category]) catMap[a.category] = { name: a.category, count: 0, examples: [] };
-    catMap[a.category].count++;
-    if (catMap[a.category].examples.length < 3) catMap[a.category].examples.push(a.slug);
-  }
-  return Object.values(catMap);
-}
-
-// ── getAgentsInCategory ────────────────────────────────────────────────────────
-
-function getAgentsInCategory(category) {
-  if (!category || typeof category !== 'string') return [];
-  return DOMAIN_AGENTS
-    .filter(function(a) { return a.category === category; })
-    .map(function(a) { return { slug: a.slug, name: a.name }; });
-}
-
 // ── exports ────────────────────────────────────────────────────────────────────
 
 module.exports = {
-  routeTask: routeTask,
-  routeTaskSemantic: routeTask,
   matchSkills: matchSkills,
-  matchExtras: matchExtras,
-  buildCategoryList: buildCategoryList,
-  getAgentsInCategory: getAgentsInCategory,
-  AGENT_CAPABILITIES: AGENT_CAPABILITIES,
-  TASK_PATTERNS: TASK_PATTERNS,
 };

package/.claude/helpers/utils/monograph-resolve.cjs

@@ -366,12 +366,20 @@ function _suppressSqliteWarning(fn) {
 var COMMIT_SQL = "SELECT value FROM index_meta WHERE key IN ('last_commit_hash','lastCommit') ORDER BY key = 'last_commit_hash' DESC LIMIT 1";
 
 // The commit the graph was built at, read with node:sqlite so it works even
-// when @monoes/monograph or its native addon is what's missing.
-function readIndexedCommit(dbPath) {
+// when @monoes/monograph or its native addon is what's missing. `immutable`
+// (doctor --read-only) opens without SQLite's -shm/-wal files, which even a
+// read-only connection to a WAL database creates or updates; it does not see
+// changes still in the WAL, which a finished build has checkpointed.
+function readIndexedCommit(dbPath, immutable) {
   try {
     return _suppressSqliteWarning(function () {
       var DatabaseSync = require('node:sqlite').DatabaseSync;
-      var db = new DatabaseSync(dbPath, { readOnly: true });
+      var target = dbPath;
+      if (immutable) {
+        target = require('node:url').pathToFileURL(dbPath);
+        target.search = '?immutable=1';
+      }
+      var db = new DatabaseSync(target, { readOnly: true });
       try { var row = db.prepare(COMMIT_SQL).get(); return row && row.value ? String(row.value) : null; }
       finally { db.close(); }
     });
@@ -400,7 +408,7 @@ function diagnose(projectDir, opts) {
   var native = resolved ? checkNative(resolved, projectDir) : null;
   var dbPath = path.join(projectDir, '.monomind', 'monograph.db');
   var dbExists = fs.existsSync(dbPath);
-  var commit = dbExists ? readIndexedCommit(dbPath) : null;
+  var commit = dbExists ? readIndexedCommit(dbPath, opts.readOnly) : null;
   var problem = null;
   var fix = null;
   if (!resolved && !cliBin) {