clementine-agent 1.0.24 → 1.0.26

package/dist/agent/insight-engine.js

@@ -11,8 +11,10 @@
 import { existsSync, readFileSync, readdirSync } from 'node:fs';
 import path from 'node:path';
 import pino from 'pino';
-import { GOALS_DIR, BASE_DIR } from '../config.js';
+import { GOALS_DIR, BASE_DIR, MEMORY_DB_PATH, VAULT_DIR } from '../config.js';
 import { listAllGoals } from '../tools/shared.js';
+import { computeBrokenJobs } from '../gateway/failure-monitor.js';
+import { MemoryStore } from '../memory/store.js';
 const logger = pino({ name: 'clementine.insight-engine' });
 const BASE_COOLDOWN_MS = 2 * 60 * 60 * 1000; // 2 hours
 const MAX_DAILY_INSIGHTS = 3;
@@ -172,9 +174,7 @@ export function gatherInsightSignals(gateway) {
     //    with a diagnosis is a real, actionable signal the owner should
     //    see proactively rather than stumble across in the dashboard.
     try {
-        // eslint-disable-next-line @typescript-eslint/no-require-imports
-        const fm = require('../gateway/failure-monitor.js');
-        const broken = fm.computeBrokenJobs();
+        const broken = computeBrokenJobs();
         for (const b of broken.slice(0, 3)) {
             const hint = b.diagnosis?.rootCause
                 ? ` — ${b.diagnosis.rootCause.slice(0, 120)}`
@@ -185,14 +185,13 @@ export function gatherInsightSignals(gateway) {
             }
         }
     }
-    catch { /* failure-monitor may not be loadable; fine */ }
+    catch (err) {
+        logger.debug({ err }, 'Failed to pull broken-jobs signals');
+    }
     // 6. Claim tracker — failed claims in the last N hours erode trust.
     //    Surface them so the owner sees "Clementine said she'd do X; she
     //    didn't" instead of silently swallowing the miss.
     try {
-        // eslint-disable-next-line @typescript-eslint/no-require-imports
-        const { MemoryStore } = require('../memory/store.js');
-        const { MEMORY_DB_PATH, VAULT_DIR } = require('../config.js');
         if (existsSync(MEMORY_DB_PATH)) {
             const store = new MemoryStore(MEMORY_DB_PATH, VAULT_DIR);
             store.initialize();
@@ -209,7 +208,9 @@ export function gatherInsightSignals(gateway) {
             store.close();
         }
     }
-    catch { /* non-fatal */ }
+    catch (err) {
+        logger.debug({ err }, 'Failed to pull failed-claims signals');
+    }
     return signals;
 }
 /**

package/dist/gateway/failure-diagnostics.js

@@ -10,7 +10,7 @@
  * so the response to a silent failure is "here's what's wrong and
  * here's what to change" rather than "go investigate."
  */
-import { existsSync, mkdirSync, readFileSync, writeFileSync, } from 'node:fs';
+import { existsSync, mkdirSync, readFileSync, renameSync, writeFileSync, } from 'node:fs';
 import path from 'node:path';
 import pino from 'pino';
 import { AGENTS_DIR, BASE_DIR, CRON_FILE } from '../config.js';
@@ -54,7 +54,6 @@ function saveCache(cache) {
         mkdirSync(path.dirname(CACHE_FILE), { recursive: true });
         const tmp = CACHE_FILE + '.tmp';
         writeFileSync(tmp, JSON.stringify(cache, null, 2));
-        const { renameSync } = require('node:fs');
         renameSync(tmp, CACHE_FILE);
     }
     catch (err) {

package/dist/gateway/failure-monitor.js

@@ -15,10 +15,11 @@
  * Read-only with respect to the cron run logs and advisor events; mutates
  * only its own state file (cron/failure-monitor.json).
  */
-import { appendFileSync, existsSync, mkdirSync, readFileSync, readdirSync, writeFileSync, } from 'node:fs';
+import { appendFileSync, existsSync, mkdirSync, readFileSync, readdirSync, renameSync, writeFileSync, } from 'node:fs';
 import path from 'node:path';
+import Database from 'better-sqlite3';
 import pino from 'pino';
-import { BASE_DIR } from '../config.js';
+import { BASE_DIR, MEMORY_DB_PATH } from '../config.js';
 const logger = pino({ name: 'clementine.failure-monitor' });
 const RUNS_DIR = path.join(BASE_DIR, 'cron', 'runs');
 const ADVISOR_EVENTS_FILE = path.join(BASE_DIR, 'cron', 'advisor-events.jsonl');
@@ -54,7 +55,6 @@ function saveState(state) {
         // writes from corrupting the state if the process is killed mid-write.
         const tmp = STATE_FILE + '.tmp';
         writeFileSync(tmp, JSON.stringify(state, null, 2));
-        const { renameSync } = require('node:fs');
         renameSync(tmp, STATE_FILE);
     }
     catch (err) {
@@ -105,11 +105,8 @@ function isFailure(entry, gradeCache) {
 function loadGradeCache() {
     const cache = new Map();
     try {
-        const { MEMORY_DB_PATH } = require('../config.js');
         if (!existsSync(MEMORY_DB_PATH))
             return cache;
-        // eslint-disable-next-line @typescript-eslint/no-require-imports
-        const Database = require('better-sqlite3');
         const db = new Database(MEMORY_DB_PATH, { readonly: true });
         try {
             const rows = db.prepare(`SELECT job_name, started_at, passed FROM graded_runs
@@ -121,7 +118,9 @@ function loadGradeCache() {
         catch { /* graded_runs may not exist on older DBs */ }
         db.close();
     }
-    catch { /* non-fatal */ }
+    catch (err) {
+        logger.warn({ err }, 'Failed to load grade cache');
+    }
     return cache;
 }
 /**

package/dist/gateway/fix-verification.js

@@ -7,7 +7,7 @@
  * owner with the verdict — succeeded or still failing — so a self-reported
  * "fix" can't go untested again.
  */
-import { existsSync, mkdirSync, readFileSync, writeFileSync, } from 'node:fs';
+import { existsSync, mkdirSync, readFileSync, renameSync, writeFileSync, } from 'node:fs';
 import path from 'node:path';
 import crypto from 'node:crypto';
 import pino from 'pino';
@@ -31,7 +31,6 @@ function saveState(state) {
         mkdirSync(path.dirname(STATE_FILE), { recursive: true });
         const tmp = STATE_FILE + '.tmp';
         writeFileSync(tmp, JSON.stringify(state, null, 2));
-        const { renameSync } = require('node:fs');
         renameSync(tmp, STATE_FILE);
     }
     catch (err) {

package/dist/gateway/router.js

@@ -5,7 +5,7 @@
  * Manages per-user/channel sessions for conversation continuity.
  */
 import path from 'node:path';
-import { existsSync, readFileSync, writeFileSync } from 'node:fs';
+import { appendFileSync, existsSync, readFileSync, writeFileSync } from 'node:fs';
 import pino from 'pino';
 import { PersonalAssistant } from '../agent/assistant.js';
 import { SelfImproveLoop } from '../agent/self-improve.js';
@@ -743,7 +743,15 @@ export class Gateway {
                 // specific agent profile, classify whether the message should go
                 // to a specialist. Direct-to-agent-bot sessions bypass this entirely.
                 // Small-talk and meta queries stay with Clementine by default.
-                const routingResult = !isInternalMsg && !sess?.profile && !text.startsWith('!')
+                //
+                // Also bypass structured workflow messages — button clicks, approvals,
+                // and other system-injected interactions are not free-form chat and
+                // shouldn't be reclassified. They already have an intended flow.
+                const isStructuredWorkflowMsg = text.startsWith('[Button clicked:')
+                    || text.startsWith('[Approval:')
+                    || text.startsWith('[Reaction:')
+                    || text.startsWith('[System:');
+                const routingResult = !isInternalMsg && !sess?.profile && !text.startsWith('!') && !isStructuredWorkflowMsg
                     ? await this._maybeRouteToSpecialist(sessionKey, text, onText)
                     : null;
                 if (routingResult?.delegated) {
@@ -956,12 +964,36 @@ export class Gateway {
                         }
                     }
                     // ── Deep mode detection ─────────────────────────────────────
-                    // Agent proposes background execution for complex tasks
-                    const deepMatch = response?.match(/^\[DEEP_MODE:\s*(.+?)\]\s*/s);
+                    // Agent proposes background execution for complex tasks.
+                    //
+                    // Syntax:
+                    //   [DEEP_MODE: task description]
+                    //   [DEEP_MODE(work_dir=/abs/path): task description]   ← NEW
+                    //
+                    // The optional work_dir= parameter runs the unleashed task in a
+                    // different project directory. Useful when the task belongs to
+                    // a Claude Code project with its own CLAUDE.md / slash commands
+                    // (e.g. the proposal-builder project for audit-queue approvals).
+                    const deepMatch = response?.match(/^\[DEEP_MODE(?:\(([^)]+)\))?:\s*(.+?)\]\s*/s);
                     if (deepMatch) {
-                        const taskDesc = deepMatch[1].trim() || text;
-                        const ack = response.replace(/^\[DEEP_MODE:[^\]]*\]\s*/s, '').trim();
+                        const paramsStr = deepMatch[1] ?? '';
+                        const taskDesc = deepMatch[2].trim() || text;
+                        const ack = response.replace(/^\[DEEP_MODE(?:\([^)]*\))?:[^\]]*\]\s*/s, '').trim();
                         logger.info({ sessionKey, task: taskDesc }, 'Deep mode triggered by agent');
+                        // Parse optional work_dir parameter — strict: must be an absolute
+                        // path and must exist. Anything else falls back to default.
+                        let deepWorkDir;
+                        const wdMatch = paramsStr.match(/work_dir=([^,\s]+)/);
+                        if (wdMatch) {
+                            const candidate = wdMatch[1].trim().replace(/^["']|["']$/g, '');
+                            if (path.isAbsolute(candidate) && existsSync(candidate)) {
+                                deepWorkDir = candidate;
+                                logger.info({ sessionKey, workDir: deepWorkDir }, 'Deep mode using custom work_dir');
+                            }
+                            else {
+                                logger.warn({ sessionKey, candidate }, 'Deep mode work_dir rejected (not absolute or does not exist)');
+                            }
+                        }
                         const currentSess = this.getSession(sessionKey);
                         const jobName = `deep-${Date.now()}`;
                         currentSess.deepTask = { jobName, taskDesc, startedAt: new Date().toISOString() };
@@ -969,7 +1001,7 @@ export class Gateway {
                         this.assistant.runUnleashedTask(jobName, `${taskDesc}\n\nOriginal request: ${text}`, 2, // tier 2 (Bash/Write/Edit enabled)
                         undefined, // default maxTurns (75/phase)
                         undefined, // default model
-                        undefined, // default workDir
+                        deepWorkDir, // honors [DEEP_MODE(work_dir=...)] if provided
                         1).then(async (result) => {
                             logger.info({ sessionKey, jobName, resultLen: result?.length ?? 0 }, 'Deep mode task completed');
                             if (result && result !== '__NOTHING__') {
@@ -1511,8 +1543,6 @@ function logRouteDecision(opts) {
     while (_routeAuditBuffer.length > 200)
         _routeAuditBuffer.shift();
     try {
-        // eslint-disable-next-line @typescript-eslint/no-require-imports
-        const { appendFileSync } = require('node:fs');
         appendFileSync(Gateway.routeAuditLogPath(), JSON.stringify(entry) + '\n');
     }
     catch (err) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "clementine-agent",
-  "version": "1.0.24",
+  "version": "1.0.26",
   "description": "Clementine — Personal AI Assistant (TypeScript)",
   "type": "module",
   "main": "dist/index.js",