clementine-agent 1.18.192 → 1.18.193

package/dist/agent/intent-classifier.d.ts

@@ -76,6 +76,34 @@ export declare function classifyMessageShape(text: string, opts?: {
  */
 export type PlanApprovalSignal = 'approve' | 'revise' | 'cancel' | 'other';
 export declare function detectPlanApproval(message: string): PlanApprovalSignal;
+/**
+ * 1.18.193 — plan-mode opt-in detector.
+ *
+ * Plan-mode used to auto-trigger when `classifyMessageShape` flagged a
+ * message as 'multi-step'. That was too aggressive — Nora's April 28-29
+ * work (38 Bash calls in one chat session) would have been routed through
+ * the planner unnecessarily. Comparison vs friend's 1.18.62 install showed
+ * the auto-route was the main behavior divergence.
+ *
+ * Now plan-mode is opt-in via explicit owner intent:
+ *   - Message starts with `/plan` (case-insensitive)
+ *   - Message contains the `[plan-mode]` token anywhere
+ *
+ * The chat-overflow recovery path (queueBackgroundTaskAfterContextOverflow)
+ * still routes to the planner when the SDK session ACTUALLY overflows —
+ * that's a separate escape hatch, not an auto-trigger.
+ *
+ * Returns `{ requested: true, cleaned }` if the owner asked for plan mode,
+ * where `cleaned` is the message with the trigger token stripped.
+ * Returns `{ requested: false }` otherwise.
+ */
+export type PlanModeRequest = {
+    requested: true;
+    cleaned: string;
+} | {
+    requested: false;
+};
+export declare function detectPlanModeRequest(message: string): PlanModeRequest;
 /**
  * Generate a follow-up suggestion prompt suffix based on completed work.
  *

package/dist/agent/intent-classifier.js

@@ -332,6 +332,16 @@ export function detectPlanApproval(message) {
         return 'revise';
     return 'other';
 }
+const PLAN_MODE_TRIGGER = /^\s*\/plan\b|\[plan-mode\]/i;
+export function detectPlanModeRequest(message) {
+    if (!message || !PLAN_MODE_TRIGGER.test(message))
+        return { requested: false };
+    const cleaned = message
+        .replace(/^\s*\/plan\b\s*/i, '')
+        .replace(/\[plan-mode\]/gi, '')
+        .trim();
+    return { requested: true, cleaned };
+}
 /**
  * Generate a follow-up suggestion prompt suffix based on completed work.
  *

package/dist/gateway/router.js

@@ -493,7 +493,7 @@ export class Gateway {
      */
     async _maybeHandlePlanMode(opts) {
         const sess = this.sessions.get(opts.sessionKey);
-        const { detectPlanApproval } = await import('../agent/intent-classifier.js');
+        const { detectPlanApproval, detectPlanModeRequest } = await import('../agent/intent-classifier.js');
         const { planRequest, savePlan, loadPlan } = await import('../agent/bg-planner.js');
         const { dispatchChain } = await import('../agent/bg-orchestrator.js');
         // ── Path A: approval-pending ────────────────────────────────────
@@ -564,11 +564,31 @@ export class Gateway {
             // pending state. The model will see the user message normally.
             return { handled: false };
         }
-        // ── Path B: multi-step entry ────────────────────────────────────
-        if (opts.shape === 'multi-step' && sess) {
+        // ── Path B: explicit plan-mode entry (1.18.193) ─────────────────
+        //
+        // Plan mode is now OPT-IN, not auto-triggered by message shape.
+        // Default chat behavior matches 1.18.62: the SDK query runs the
+        // work in one continuous Sonnet session, like Nora did on April
+        // 28-29 (38 Bash calls in one session — no decomposition needed).
+        //
+        // The planner-orchestrator remains available for genuinely huge
+        // jobs, but the owner has to opt in:
+        //
+        //   - Message starts with `/plan` (case-insensitive)
+        //   - Message contains `[plan-mode]` token anywhere
+        //
+        // The chat-overflow escape hatch (queueBackgroundTaskAfterContext-
+        // Overflow → planner) still works as a separate path when the SDK
+        // session ACTUALLY overflows. That's the legitimate "this job is
+        // too big for one session" trigger.
+        //
+        // We keep shape classification (it still gates turn-context
+        // density for token savings on 'simple' messages), but shape no
+        // longer routes execution.
+        const planRequestSignal = detectPlanModeRequest(opts.userMessage);
+        if (planRequestSignal.requested && sess) {
+            const cleanedRequest = planRequestSignal.cleaned;
             try {
-                // Stream a "thinking..." update so the user knows planning is
-                // happening rather than seeing 30s of silence.
                 if (opts.onText) {
                     try {
                         opts.onText('🤔 Planning the steps...');
@@ -576,7 +596,7 @@ export class Gateway {
                     catch { /* non-fatal */ }
                 }
                 const plan = await planRequest({
-                    userRequest: opts.userMessage,
+                    userRequest: cleanedRequest || opts.userMessage,
                     originatingSessionKey: opts.sessionKey,
                     ...(opts.activeProject ? { project: opts.activeProject } : {}),
                 });
@@ -592,12 +612,14 @@ export class Gateway {
                 };
             }
             catch (err) {
-                logger.warn({ err, sessionKey: opts.sessionKey }, 'Plan mode: planRequest failed at entry');
+                logger.warn({ err, sessionKey: opts.sessionKey }, 'Plan mode: planRequest failed at explicit entry');
                 // Fall through to normal chat. Better than blocking the owner.
                 return { handled: false };
             }
         }
         // Not a plan-mode case — fall through to normal chat.
+        // This is the path 99% of messages take. Like 1.18.62 — the SDK
+        // query runs the work in one continuous Sonnet session.
         return { handled: false };
     }
     /** Format a plan for owner approval in chat. */

package/package.json

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