clementine-agent 1.0.89 → 1.0.90

package/dist/agent/assistant.js

@@ -1788,12 +1788,18 @@ You have a cost budget per message — not a hard turn limit. Work until the tas
         const supportsThinking = !resolvedModel.includes('haiku');
         const needsThinking = !isHeartbeat && (isPlanStep || isUnleashed || !isCron);
         const computedThinking = thinking ?? (supportsThinking && needsThinking ? { type: 'adaptive' } : undefined);
-        // Haiku rejects user-configurable task budgets with a 400 ("This model
-        // does not support user-configurable task budgets"). Only pass
-        // taskBudget to models that accept it — otherwise every Haiku cron
-        // run dies on arrival and (historically) got mis-classified as a
-        // permanent "budget exceeded" failure.
-        const supportsTaskBudget = !resolvedModel.includes('haiku');
+        // ── taskBudget: don't pass to the SDK ─────────────────────────
+        // The Anthropic API now rejects `taskBudget` for both Haiku AND Sonnet
+        // ("This model does not support user-configurable task budgets" — 400).
+        // We previously gated by !haiku, but that left Sonnet crons (e.g.,
+        // ross-the-sdr:reply-detection) failing on every run. Cost is
+        // informational on a Claude subscription anyway — `maxTurns` and the
+        // wall-clock cap (`maxHours` for unleashed) are the actual brakes.
+        //
+        // computedTaskBudget is still computed below for any future telemetry
+        // path that wants to log "soft target" values, but it is intentionally
+        // never passed into sdkOptions.
+        const supportsTaskBudget = false;
         // 1M context beta: enable for Sonnet when toggled and context-heavy work benefits
         const isSonnet = resolvedModel.includes('sonnet');
         const computedBetas = ENABLE_1M_CONTEXT && isSonnet

package/dist/gateway/cron-scheduler.js

@@ -457,11 +457,27 @@ export class CronScheduler {
         // phase updates for deep-mode runs get routed back to the originating
         // session instead of fanning out to every registered channel.
         const isDeepMode = (jobName) => jobName.startsWith('deep-');
-        // Wire up push notifications for unleashed task completions
+        // Wire up push notifications for unleashed task completions.
+        //
+        // This callback is only meant for AD-HOC unleashed tasks (chat-triggered
+        // "deep mode" follow-ups that didn't go through a registered job). Three
+        // other paths already own their own delivery and would otherwise produce
+        // double-dispatches:
+        //
+        //   1. deep-mode (`deep-*`)  → router handles delivery via _deliverDeepResult
+        //   2. background tasks (`bg:*`) → processBackgroundTasks dispatches result
+        //   3. registered cron jobs   → cron-runner success path dispatches at line ~1115
+        //
+        // Each path is gated below; without the guards Sasha's morning brief was
+        // landing twice in her Discord channel.
         this.gateway.setUnleashedCompleteCallback((jobName, result) => {
             this.completedJobs.set(jobName, Date.now());
             if (isDeepMode(jobName))
-                return; // router handles delivery via _deliverDeepResult
+                return; // (1) deep-mode router
+            if (jobName.startsWith('bg:'))
+                return; // (2) background-task dispatcher
+            if (this.jobs.some(j => j.name === jobName))
+                return; // (3) registered cron job
             if (result && result !== '__NOTHING__') {
                 const slug = jobName.includes(':') ? jobName.split(':')[0] : undefined;
                 // Strip system metadata for clean conversational delivery

package/package.json

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