dsh-continual-evolve 0.15.3 → 0.15.4

package/README.md

@@ -7,7 +7,7 @@ English | [中文](README.zh.md)
 [![CI](https://github.com/ZK-Andy/dsh-continual-evolve/actions/workflows/ci.yml/badge.svg)](https://github.com/ZK-Andy/dsh-continual-evolve/actions/workflows/ci.yml)
 [![License: MIT](https://img.shields.io/badge/license-MIT-blue.svg)](LICENSE)
 [![Node](https://img.shields.io/badge/node-%5E22.19%20%7C%7C%20%3E%3D24-339933)](package.json)
-[![Tests](https://img.shields.io/badge/tests-194%20passing-brightgreen)]()
+[![Tests](https://img.shields.io/badge/tests-199%20passing-brightgreen)]()
 [![Coverage · statements](https://img.shields.io/badge/coverage_statements-97%25-brightgreen)]()
 [![Coverage · branches](https://img.shields.io/badge/coverage_branches-91%25-green)]()
 [![Coverage · functions](https://img.shields.io/badge/coverage_functions-100%25-brightgreen)]()
@@ -71,7 +71,7 @@ Profile patch example:
 
 ```bash
 pnpm install && pnpm build   # deps + clean build -> lib/
-pnpm test                    # vitest (194 tests)
+pnpm test                    # vitest (199 tests)
 pnpm test:coverage           # v8 coverage, CI-enforced thresholds
 pnpm lint                    # oxlint src test client
 pnpm check:pack              # published-artifact consistency (also runs on prepack)

package/README.zh.md

@@ -7,7 +7,7 @@
 [![CI](https://github.com/ZK-Andy/dsh-continual-evolve/actions/workflows/ci.yml/badge.svg)](https://github.com/ZK-Andy/dsh-continual-evolve/actions/workflows/ci.yml)
 [![License: MIT](https://img.shields.io/badge/license-MIT-blue.svg)](LICENSE)
 [![Node](https://img.shields.io/badge/node-%5E22.19%20%7C%7C%20%3E%3D24-339933)](package.json)
-[![Tests](https://img.shields.io/badge/tests-194%20passing-brightgreen)]()
+[![Tests](https://img.shields.io/badge/tests-199%20passing-brightgreen)]()
 [![Coverage · statements](https://img.shields.io/badge/coverage_statements-97%25-brightgreen)]()
 [![Coverage · branches](https://img.shields.io/badge/coverage_branches-91%25-green)]()
 [![Coverage · functions](https://img.shields.io/badge/coverage_functions-100%25-brightgreen)]()
@@ -71,7 +71,7 @@ profile patch 示例：
 
 ```bash
 pnpm install && pnpm build   # 依赖 + 干净构建 -> lib/
-pnpm test                    # vitest（194 例）
+pnpm test                    # vitest（199 例）
 pnpm test:coverage           # v8 覆盖率，CI 强制阈值
 pnpm lint                    # oxlint src test client
 pnpm check:pack              # 发布产物一致性（prepack 也会跑）

package/lib/extraction.d.ts

@@ -34,8 +34,11 @@ export interface RunOptions {
     incrementMaxChars?: number;
 }
 /**
- * Run one extraction pass for a target. Never throws: every terminal path
- * is either a landed application, a ledgered skip, or a ledgered failure.
+ * Run one extraction pass for a target. Never throws unless the ledger
+ * itself is unwritable: every terminal path is either a landed application,
+ * a ledgered skip, or a ledgered failure — anything that escapes the
+ * per-stage handling inside lands a failed `unexpected-error` ledger row
+ * here, so no failure ends as a scheduler warn without an account trace.
  */
 export declare function runExtraction(deps: ExtractionDeps, target: ExtractionTarget, opts: RunOptions): Promise<ExtractionResult>;
 /** The event wiring surface (duck-typed cordis context). */

package/lib/extraction.js

@@ -3,11 +3,57 @@ import { EXTRACTION_SYSTEM_PROMPT, extractionPrompt, manifestLineOf, parseExtrac
 import { cwdOf } from "./memory-section.js";
 /** Head length of a failed run's raw answer kept in the ledger `files` column. */
 const FAILURE_RAW_MAX = 2048;
+/** Head length of an unexpected error's message/stack trace kept in the ledger `files` column. */
+const UNEXPECTED_ERROR_RAW_MAX = 2048;
+/** Head length of an error message embedded in a ledger `skip_reason`. */
+const ERROR_REASON_MAX = 300;
+/** Split an unexpected throw into a ledger-sized reason head and file body. */
+function errorText(error) {
+    const message = error instanceof Error && error.message.length > 0 ? error.message : String(error);
+    const stack = error instanceof Error && typeof error.stack === "string" ? error.stack : message;
+    return { message, detail: `${message}\n${stack}`.slice(0, UNEXPECTED_ERROR_RAW_MAX) };
+}
 /**
- * Run one extraction pass for a target. Never throws: every terminal path
- * is either a landed application, a ledgered skip, or a ledgered failure.
+ * Run one extraction pass for a target. Never throws unless the ledger
+ * itself is unwritable: every terminal path is either a landed application,
+ * a ledgered skip, or a ledgered failure — anything that escapes the
+ * per-stage handling inside lands a failed `unexpected-error` ledger row
+ * here, so no failure ends as a scheduler warn without an account trace.
  */
 export async function runExtraction(deps, target, opts) {
+    const started = Date.now();
+    try {
+        return await runExtractionInner(deps, target, opts);
+    }
+    catch (error) {
+        const { message, detail } = errorText(error);
+        const reason = `unexpected-error: ${message.slice(0, ERROR_REASON_MAX)}`;
+        try {
+            deps.store.logSkipped({
+                sessionId: target.sessionId,
+                workspaceId: target.workspaceId,
+                trigger: opts.trigger,
+                skipReason: reason,
+                status: "failed",
+                model: target.model,
+                durationMs: Date.now() - started,
+                files: detail,
+            });
+        }
+        catch (ledgerError) {
+            // The ledger itself is down — nothing left to write to. Chain
+            // both errors so the scheduler's last-resort row keeps the
+            // triggering failure, not just the ledger fault.
+            throw new Error(`extraction failed (${message}); failure ledger write also failed (${errorText(ledgerError).message})`);
+        }
+        return { status: "failed", skipReason: reason };
+    }
+}
+/**
+ * The pipeline body: one bounded increment of one session, one LLM call, one
+ * gated transactional application, one ledger row, one cursor.
+ */
+async function runExtractionInner(deps, target, opts) {
     const { store } = deps;
     const ledgerBase = {
         sessionId: target.sessionId,
@@ -37,8 +83,14 @@ export async function runExtraction(deps, target, opts) {
         all = Array.isArray(snapshot.events) ? snapshot.events : [];
     }
     catch (error) {
-        store.logSkipped({ ...ledgerBase, skipReason: "surface-read-failed", status: "failed" });
-        return { status: "failed", skipReason: `surface read failed: ${error instanceof Error ? error.message : String(error)}` };
+        const { message, detail } = errorText(error);
+        try {
+            store.logSkipped({ ...ledgerBase, skipReason: "surface-read-failed", status: "failed", files: detail });
+        }
+        catch (ledgerError) {
+            throw new Error(`surface read failed (${message}); failure ledger write also failed (${errorText(ledgerError).message})`);
+        }
+        return { status: "failed", skipReason: `surface read failed: ${message}` };
     }
     const events = eventsAfterCursor(all, cursor === undefined ? undefined : `seq:${cursor}`);
     const slice = evaluateSlice({ events, internalAgent: false });
@@ -72,8 +124,14 @@ export async function runExtraction(deps, target, opts) {
         usage = answer.usage;
     }
     catch (error) {
-        store.logSkipped({ ...ledgerBase, skipReason: "llm-failed", status: "failed", durationMs: Date.now() - started });
-        return { status: "failed", skipReason: `llm call failed: ${error instanceof Error ? error.message : String(error)}` };
+        const { message, detail } = errorText(error);
+        try {
+            store.logSkipped({ ...ledgerBase, skipReason: "llm-failed", status: "failed", durationMs: Date.now() - started, files: detail });
+        }
+        catch (ledgerError) {
+            throw new Error(`llm call failed (${message}); failure ledger write also failed (${errorText(ledgerError).message})`);
+        }
+        return { status: "failed", skipReason: `llm call failed: ${message}` };
     }
     const parsed = parseExtractionAnswer(text);
     if (parsed.decision === "error") {
@@ -213,7 +271,26 @@ export function createExtractionScheduler(host, deps) {
             provider: target.provider,
             model: target.model,
         }, { trigger }).catch((error) => {
-            host.logger("continual-evolve").warn(`extraction run failed unexpectedly: ${error instanceof Error ? error.message : String(error)}`);
+            // Last resort: runExtraction ledgers its own unexpected failures,
+            // so this fires only when even that failed (the ledger write
+            // threw) — retry one ledger row here before the warn, so the
+            // failure still has an account trace.
+            const { message, detail } = errorText(error);
+            try {
+                deps.store.logSkipped({
+                    sessionId: target.sessionId,
+                    workspaceId: cwdOf(target.agent),
+                    trigger,
+                    skipReason: `unexpected-error: ${message.slice(0, ERROR_REASON_MAX)}`,
+                    status: "failed",
+                    model: target.model,
+                    files: detail,
+                });
+            }
+            catch {
+                // The ledger is down — the warn below is the last resort.
+            }
+            host.logger("continual-evolve").warn(`extraction run failed unexpectedly: ${message}`);
         }).finally(() => {
             slot.running = false;
             if (slot.pending !== null) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-continual-evolve",
-  "version": "0.15.3",
+  "version": "0.15.4",
   "description": "ZCode-style workspace memory for DeepSeek Harness: one central SQLite store partitioned by workspace, index injection, and sole-writer gated writes.",
   "license": "MIT",
   "repository": {