soulbridge 0.0.19 → 0.0.21

package/dist/packages/core/src/index.js

@@ -134,7 +134,9 @@ export class Soul {
         let extractedBy = drafts.length > 0 ? 'rules' : 'none';
         // A model that says "nothing to keep" and a model that answers garbage used
         // to be indistinguishable; keep them apart for measurement.
-        let llmOutcome = 'off';
+        let llmOutcome = this.llm
+            ? 'skipped-rules'
+            : 'off';
         if (drafts.length === 0 && turn.role === 'user' && this.llm) {
             if (turn.content.length <= this.llmMaxChars) {
                 try {
@@ -155,7 +157,7 @@ export class Soul {
                 }
             }
             else {
-                llmOutcome = 'skipped';
+                llmOutcome = 'skipped-long';
             }
         }
         // Deterministic write gates: shape -> semantics -> per-turn budget. Whatever

package/dist/packages/core/src/sqlite.d.ts

@@ -1,3 +1,4 @@
+import { DatabaseSync } from 'node:sqlite';
 import type { SoulStore } from './store.ts';
 export declare const SCHEMA_VERSION = 2;
 export declare const AFFECT_RETENTION_MS: number;
@@ -6,4 +7,13 @@ export declare class DbNewerThanBinaryError extends Error {
     readonly dbVersion: number;
     constructor(dbVersion: number);
 }
+/**
+ * Copy the database aside before a schema upgrade.
+ *
+ * In WAL mode the most recent commits can live in the `-wal` sidecar, so a copy
+ * of the main file alone can be missing data exactly when it matters. Checkpoint
+ * first (TRUNCATE moves the WAL back into the main file and empties it), then
+ * copy. Exported for the upgrade test.
+ */
+export declare function backupBeforeUpgrade(db: DatabaseSync, file: string, from: number): void;
 export declare function openSqlite(file: string): SoulStore;

package/dist/packages/core/src/sqlite.js

@@ -148,12 +148,22 @@ function upgradeToV3(db) {
   )`);
     db.exec('CREATE INDEX IF NOT EXISTS idx_corrections ON corrections(soul_id, user_id, at DESC)');
 }
-function backupBeforeUpgrade(file, from) {
+/**
+ * Copy the database aside before a schema upgrade.
+ *
+ * In WAL mode the most recent commits can live in the `-wal` sidecar, so a copy
+ * of the main file alone can be missing data exactly when it matters. Checkpoint
+ * first (TRUNCATE moves the WAL back into the main file and empties it), then
+ * copy. Exported for the upgrade test.
+ */
+export function backupBeforeUpgrade(db, file, from) {
     if (file === ':memory:' || !existsSync(file))
         return;
     const backup = `${file}.bak-v${from}`;
-    if (!existsSync(backup))
-        copyFileSync(file, backup);
+    if (existsSync(backup))
+        return;
+    db.exec('PRAGMA wal_checkpoint(TRUNCATE);');
+    copyFileSync(file, backup);
 }
 export function openSqlite(file) {
     const db = new DatabaseSync(file);
@@ -166,7 +176,7 @@ export function openSqlite(file) {
         throw new DbNewerThanBinaryError(version);
     }
     if (version < SCHEMA_VERSION) {
-        backupBeforeUpgrade(file, version);
+        backupBeforeUpgrade(db, file, version);
         if (version < 2)
             upgradeToV2(db);
         if (version < 3)

package/dist/packages/core/src/types.d.ts

@@ -133,7 +133,7 @@ export interface ObserveResult {
      * What the optional model did, kept apart from `extractedBy` because a null
      * answer and a broken answer used to look identical in the logs.
      */
-    llmOutcome: 'off' | 'skipped' | 'stored' | 'null' | 'failed';
+    llmOutcome: 'off' | 'skipped-rules' | 'skipped-long' | 'stored' | 'null' | 'failed';
     emotion: Emotion;
     /** Analysis of the user turn. Empty signal for assistant turns or on failure. */
     affect: AffectSignal;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "soulbridge",
-  "version": "0.0.19",
+  "version": "0.0.21",
   "description": "A personality layer for any agent: continuity, forgetting, and growth across tools.",
   "license": "Apache-2.0",
   "type": "module",