@wrongstack/core 0.8.2 → 0.8.5

package/dist/defaults/index.d.ts

@@ -1,5 +1,5 @@
 export { D as DefaultLogger, a as DefaultLoggerOptions } from '../logger-bOzkF5LL.js';
-export { A as AbandonedSession, a as AttachmentStoreOptions, C as ConfigLoaderOptions, b as ConfigMigration, c as ConfigMigrationError, d as ConfigSource, D as DEFAULT_CONFIG_MIGRATIONS, e as DefaultAttachmentStore, f as DefaultConfigLoader, g as DefaultConfigStore, h as DefaultMemoryStore, i as DefaultSessionStore, M as MemoryStoreOptions, j as MigrationContext, k as MigrationResult, P as PersistedQueueItem, l as PlanFile, m as PlanItem, n as PlanTemplate, Q as QueueStore, R as RecoveryLock, o as RecoveryLockOptions, S as SessionAnalyzer, p as SessionStoreOptions, T as TodosCheckpointFile, q as addPlanItem, r as attachPlanCheckpoint, s as attachTodosCheckpoint, t as clearPlan, u as deriveTodosFromPlanItem, v as emptyPlan, w as formatPlan, x as formatPlanTemplates, y as getPlanTemplate, z as listPlanTemplates, B as loadPlan, E as loadTodosCheckpoint, F as removePlanItem, G as runConfigMigrations, H as savePlan, I as saveTodosCheckpoint, J as setPlanItemStatus } from '../plan-templates-BnlpEkX8.js';
+export { A as AbandonedSession, a as AttachmentStoreOptions, C as ConfigLoaderOptions, b as ConfigMigration, c as ConfigMigrationError, d as ConfigSource, D as DEFAULT_CONFIG_MIGRATIONS, e as DefaultAttachmentStore, f as DefaultConfigLoader, g as DefaultConfigStore, h as DefaultMemoryStore, i as DefaultSessionStore, M as MemoryStoreOptions, j as MigrationContext, k as MigrationResult, P as PersistedQueueItem, l as PlanFile, m as PlanItem, n as PlanTemplate, Q as QueueStore, R as RecoveryLock, o as RecoveryLockOptions, S as SessionAnalyzer, p as SessionStoreOptions, T as TodosCheckpointFile, q as addPlanItem, r as attachPlanCheckpoint, s as attachTodosCheckpoint, t as clearPlan, u as deriveTodosFromPlanItem, v as emptyPlan, w as formatPlan, x as formatPlanTemplates, y as getPlanTemplate, z as listPlanTemplates, B as loadPlan, E as loadTodosCheckpoint, F as removePlanItem, G as runConfigMigrations, H as savePlan, I as saveTodosCheckpoint, J as setPlanItemStatus } from '../plan-templates-D8zQgZzc.js';
 export { D as DefaultSessionReader } from '../session-reader-DsadjyF9.js';
 export { D as DirectorStateCheckpoint, a as DirectorStateSnapshot, b as DirectorSubagentState, c as DirectorTaskState, l as loadDirectorState } from '../director-state-BmYi3DGA.js';
 export { D as DefaultSecretScrubber, a as DefaultSecretVault, S as SecretVaultOptions, d as decryptConfigSecrets, e as encryptConfigSecrets, m as migratePlaintextSecrets, r as rewriteConfigEncrypted } from '../secret-scrubber-C0n1EqrC.js';
@@ -22,7 +22,7 @@ export { D as DEFAULT_CONTEXT_CONFIG, a as DEFAULT_TOOLS_CONFIG } from '../defau
 import '../logger-DDd5C--Z.js';
 import '../events-DyhxkstG.js';
 import '../memory-CEXuo7sz.js';
-import '../wstack-paths-gCrJ631C.js';
+import '../wstack-paths-Bxik3CsK.js';
 import '../config-Bi4Q0fnz.js';
 import '../models-registry-BcYJDKLm.js';
 import '../secret-vault-DoISxaKO.js';

package/dist/defaults/index.js

@@ -1662,9 +1662,7 @@ var RecoveryLock = class {
       hostname: this.hostname,
       startedAt: (/* @__PURE__ */ new Date()).toISOString()
     };
-    const tmp = `${this.file}.tmp`;
-    await fsp.writeFile(tmp, JSON.stringify(lock), { mode: 384 });
-    await fsp.rename(tmp, this.file);
+    await atomicWrite(this.file, JSON.stringify(lock), { mode: 384 });
   }
   /**
    * Release the lock. Idempotent — silently succeeds if the file is
@@ -4842,12 +4840,18 @@ var AutoCompactionMiddleware = class _AutoCompactionMiddleware {
   failureMode;
   policyProvider;
   /**
-   * Overhead factor applied to the rough message-token estimate to produce a
-   * figure comparable to the real API request size (system prompt + tool defs).
-   * Without this factor, raw message tokens undercount real load by 15-50% in
-   * short conversations, causing premature compaction triggers.
+   * Calibration factor applied to the estimator output. The estimator is now
+   * expected to already include messages + system prompt + tool definitions
+   * (see `estimateRequestTokens.total`), so no overhead boost is needed. Kept
+   * as a constant so a future calibration against real provider tokenization
+   * (BPE vs the chars/3.5 rough estimator) can dial this without touching the
+   * threshold-check math.
+   *
+   * Historical note: was 1.3 when the estimator only counted messages — that
+   * double-counted system+tools once the estimator was upgraded, firing
+   * compaction ~30% earlier than intended (e.g. real 56% load showed as 73%).
    */
-  static OVERHEAD_FACTOR = 1.3;
+  static OVERHEAD_FACTOR = 1;
   /**
    * Once a compaction attempt reduces nothing (preserveK protects everything,
    * no oversized tool_results remain to elide), retrying on every iteration
@@ -5057,7 +5061,7 @@ var ToolExecutor = class {
         return { result, tool, durationMs: Date.now() - start };
       }
       if (hasMalformedArguments(use.input)) {
-        const result = this.malformedInputResult(use);
+        const result = this.malformedInputResult(use, extractMalformedRaw(use.input));
         budget = this.decrementBudget(result, budget);
         return { result, tool, durationMs: Date.now() - start };
       }
@@ -5264,11 +5268,18 @@ var ToolExecutor = class {
       is_error: true
     };
   }
-  malformedInputResult(use) {
+  malformedInputResult(use, raw) {
+    let content = `Tool "${use.name}" received arguments that were not a valid JSON object, so they could not be parsed. Re-issue the call with the arguments encoded as a single well-formed JSON object matching the tool's input schema.`;
+    if (raw) {
+      const max = 800;
+      const excerpt = raw.length > max ? `${raw.slice(0, max)}\u2026 (truncated, ${raw.length} chars total)` : raw;
+      content += ` Common cause: a string field (e.g. code in old_string/new_string) contains literal newlines, quotes, or backslashes that must be JSON-escaped, or the payload was cut off mid-stream. The raw arguments received were:
+${excerpt}`;
+    }
     return {
       type: "tool_result",
       tool_use_id: use.id,
-      content: `Tool "${use.name}" received arguments that were not a valid JSON object, so they could not be parsed. Re-issue the call with the arguments encoded as a single well-formed JSON object matching the tool's input schema.`,
+      content,
       is_error: true
     };
   }
@@ -5324,6 +5335,18 @@ function hasMalformedArguments(input) {
   const keys = Object.keys(obj);
   return keys.length === 1 && MALFORMED_ARG_MARKERS.includes(keys[0]);
 }
+function extractMalformedRaw(input) {
+  if (!hasMalformedArguments(input)) return void 0;
+  const obj = input;
+  const value = obj[Object.keys(obj)[0]];
+  if (value === void 0 || value === null) return void 0;
+  if (typeof value === "string") return value;
+  try {
+    return JSON.stringify(value);
+  } catch {
+    return String(value);
+  }
+}
 
 // src/execution/autonomous-runner.ts
 var DoneConditionChecker = class {
@@ -10670,7 +10693,7 @@ function makeSpawnTool(director, roster) {
           catalog: roster
         });
         const dispatchRole = dispatchResult.role;
-        if (roster && roster[dispatchRole]) {
+        if (roster?.[dispatchRole]) {
           cfg = instantiateRosterConfig(dispatchRole, roster[dispatchRole]);
         } else {
           const def = dispatchResult.definition;
@@ -11160,6 +11183,13 @@ var Director = class {
         extendCounts.delete(guardKey);
         return;
       }
+      if (payload.kind === "cost" && this.maxFleetCostUsd < Number.POSITIVE_INFINITY) {
+        const totalCost = this.usage.snapshot().total?.cost ?? 0;
+        if (totalCost >= this.maxFleetCostUsd) {
+          payload.deny();
+          return;
+        }
+      }
       extendCounts.set(guardKey, prior + 1);
       setImmediate(() => {
         const extra = {};
@@ -13130,6 +13160,13 @@ var TaskTracker = class {
   opts;
   graph = null;
   transitions = [];
+  /**
+   * Attach an existing graph (used by PhaseOrchestrator to associate a tracker
+   * with a phase's pre-built task graph without re-creating it).
+   */
+  setGraph(graph) {
+    this.graph = graph;
+  }
   async createGraph(specId, title) {
     this.graph = {
       id: crypto.randomUUID(),
@@ -13202,10 +13239,6 @@ var TaskTracker = class {
     this.graph.updatedAt = now;
     this.persist();
   }
-  /**
-   * Update node fields (title, description, priority, estimateHours, tags).
-   * Does NOT change status. Use updateNodeStatus for status changes.
-   */
   updateNode(id, patch) {
     if (!this.graph) throw new Error("No graph loaded");
     const node = this.graph.nodes.get(id);
@@ -13283,8 +13316,7 @@ var TaskTracker = class {
     }
     return computeTaskProgress(this.graph);
   }
-  getTransitions(taskId) {
-    if (!taskId) return [...this.transitions];
+  getTransitions(_taskId) {
     return [...this.transitions];
   }
   unblockDependents(completedId) {
@@ -13324,18 +13356,12 @@ var TaskTracker = class {
    * Fire-and-forget persistence with attached error handler.
    * Synchronous mutators (addNode/addEdge/updateNodeStatus) use this to
    * avoid forcing an async cascade through every caller; if the store
-   * rejects, the configured `onPersistError` is invoked so failures are
-   * surfaced instead of swallowed by an unhandled promise rejection.
+   * is missing or throwing, the error is surfaced via onPersistError.
    */
   persist() {
     if (!this.graph) return;
     this.opts.store.saveGraph(this.graph).catch((err) => {
-      if (this.opts.onPersistError) this.opts.onPersistError(err);
-      else
-        console.warn(
-          "[task-tracker] saveGraph failed:",
-          err instanceof Error ? err.message : String(err)
-        );
+      this.opts.onPersistError ? this.opts.onPersistError(err) : console.warn("[task-tracker] saveGraph failed:", err instanceof Error ? err.message : String(err));
     });
   }
 };