@aexol/spectral 0.7.5 → 0.7.6

package/dist/relay/dispatcher.js

@@ -472,7 +472,7 @@ function makeRelaySubscriber(sessionId, relay) {
  *    an `error` event to subscribers, so we don't double-report.
  */
 export function handleClientMessage(frame, deps) {
-    const { sessionId, message, modelId } = frame;
+    const { sessionId, message, modelId, reasoningEffort } = frame;
     const { manager, relay, subscribers } = deps;
     const logger = deps.logger ?? console;
     // 0. Shutdown gate. Once `gracefulShutdown` flips the flag we refuse
@@ -583,7 +583,7 @@ export function handleClientMessage(frame, deps) {
     //
     //    When `loop: true`, loop state is set before prompting; the normal
     //    model is used — loop replay is handled by session-stream on agent_end.
-    manager.prompt(sessionId, content, modelId, validImages).catch((err) => {
+    manager.prompt(sessionId, content, modelId, validImages, reasoningEffort).catch((err) => {
         logger.error?.(`[dispatcher] manager.prompt failed for ${sessionId}:`, err);
     });
 }

package/dist/server/pi-bridge.js

@@ -842,6 +842,65 @@ export class PiBridge {
             return false;
         }
     }
+    /**
+     * Map a frontend reasoning-effort string to pi's ThinkingLevel.
+     * Frontend sends: xhigh | high | medium | low | minimal | none | undefined
+     * Pi expects:   "high" | "medium" | "low" | "minimal" | "off"
+     *
+     * Mapping:
+     *   xhigh   → high    (pi doesn't have xhigh, default to max)
+     *   high    → high
+     *   medium  → medium
+     *   low     → low
+     *   minimal → minimal
+     *   none    → off
+     *   undefined → no-op (pi keeps whatever it has currently)
+     */
+    mapReasoningEffortToThinkingLevel(effort) {
+        if (effort === undefined)
+            return undefined;
+        switch (effort) {
+            case "xhigh":
+                return "high";
+            case "high":
+            case "medium":
+            case "low":
+            case "minimal":
+                return effort;
+            case "none":
+                return "off";
+            default:
+                return undefined;
+        }
+    }
+    /**
+     * Set the reasoning/thinking effort level for the next prompt.
+     * Pass `undefined` to leave pi's current level unchanged.
+     *
+     * The caller (SessionStreamManager) is responsible for persisting the
+     * value to SQLite; this method only applies it to pi's in-memory session.
+     */
+    setReasoningEffort(effort) {
+        const level = this.mapReasoningEffortToThinkingLevel(effort);
+        if (level === undefined)
+            return;
+        if (!this.session) {
+            // Bridge hasn't started yet — the reasoning effort will be picked
+            // up from storage on the next prompt via session-stream's persistence.
+            return;
+        }
+        try {
+            this.session.setThinkingLevel(level);
+        }
+        catch (err) {
+            const e = err instanceof Error ? err : new Error(String(err));
+            this.opts.onError?.(e);
+            this.opts.emit({
+                type: "error",
+                message: `Failed to set reasoning effort: ${e.message}`,
+            });
+        }
+    }
     /**
      * Forward a user message to pi. Resolves when the full turn ends.
      * The caller is responsible for persisting the user message to SQLite

package/dist/server/session-stream.js

@@ -483,7 +483,7 @@ export class SessionStreamManager {
      *   - When neither envelope nor SQLite have a value, we leave model
      *     selection to pi's own settings file (pre-Phase-3 behaviour).
      */
-    async prompt(sessionId, content, modelId, images) {
+    async prompt(sessionId, content, modelId, images, reasoningEffort) {
         if (this.disposed)
             throw new Error("SessionStreamManager disposed");
         let stream = this.streams.get(sessionId);
@@ -599,6 +599,24 @@ export class SessionStreamManager {
                 console.warn(`[spectral] warn: failed to persist sticky model for ${sessionId}: ${err instanceof Error ? err.message : String(err)}`);
             }
         }
+        // Sticky reasoning-effort resolution. Same order as modelId:
+        //   a) Envelope-supplied value → apply + persist.
+        //   b) Otherwise, look up the persisted value from SQLite.
+        //   c) When neither is present, leave pi's current thinking level unchanged.
+        // We apply BEFORE persisting the user message so the turn runs with the
+        // correct reasoning level.
+        const effectiveReasoningEffort = reasoningEffort ?? this.store.getSessionReasoningEffort(sessionId) ?? undefined;
+        if (effectiveReasoningEffort && stream.bridge.setReasoningEffort) {
+            stream.bridge.setReasoningEffort(effectiveReasoningEffort);
+        }
+        if (reasoningEffort !== undefined) {
+            try {
+                this.store.setSessionReasoningEffort(sessionId, reasoningEffort || null);
+            }
+            catch (err) {
+                console.warn(`[spectral] warn: failed to persist reasoning effort for ${sessionId}: ${err instanceof Error ? err.message : String(err)}`);
+            }
+        }
         // 1. Persist user message first (survives mid-prompt failures).
         let stored;
         try {

package/dist/server/storage.js

@@ -32,9 +32,10 @@ import { dirname, join } from "node:path";
 import { getConfigDir } from "../config.js";
 import { stripJsoncComments } from "../studio-binding.js";
 /**
- * Schema version. Bump + the on-open migration drops & recreates every table.
- * Since this is local-only conversation history pre-1.0, we explicitly do not
- * preserve user data across schema changes.
+ * Schema version — informational, stamped into every new database.
+ *
+ * ALL schema changes are additive (ALTER TABLE ADD COLUMN).  The version is
+ * never used to trigger destructive drops; it is purely for troubleshooting.
  */
 const SCHEMA_VERSION = 4;
 const SCHEMA_SQL = `
@@ -54,7 +55,8 @@ CREATE TABLE IF NOT EXISTS sessions (
   title TEXT NOT NULL,
   created_at INTEGER NOT NULL,
   updated_at INTEGER NOT NULL,
-  model_id TEXT
+  model_id TEXT,
+  reasoning_effort TEXT
 );
 
 CREATE INDEX IF NOT EXISTS idx_sessions_project ON sessions(project_id, updated_at DESC);
@@ -130,7 +132,6 @@ function applyBindingFields(project) {
     };
 }
 /** Tables we own — used by the migration drop step. */
-const KNOWN_TABLES = ["prompt_queue", "project_observations", "session_memory_snapshots", "messages", "sessions", "projects"];
 /** Best-effort parse of the `images_json` column into `ImageAttachment[]`. */
 function parseImagesJson(raw) {
     if (!raw || raw === "")
@@ -188,6 +189,8 @@ export class SessionStore {
     // Phase 3: per-session sticky model.
     stmtGetSessionModel;
     stmtSetSessionModel;
+    stmtGetSessionReasoningEffort;
+    stmtSetSessionReasoningEffort;
     // Fork & Compact: flag + per-session source tracking.
     stmtSetForkCompactSource;
     stmtGetForkCompactSource;
@@ -210,28 +213,17 @@ export class SessionStore {
         this.db = new Database(path);
         this.db.pragma("journal_mode = WAL");
         this.db.pragma("foreign_keys = ON");
-        // ---- migration --------------------------------------------------------
-        // Detect schema version and drop+recreate on mismatch. Pre-1.0: no data
-        // preservation. The first open on a brand-new file reports version 0
-        // and falls through to the create path below (drops are no-ops on
-        // missing tables).
-        const versionRow = this.db.pragma("user_version", { simple: true });
-        if (versionRow !== SCHEMA_VERSION) {
-            // Disable FKs for the duration of the drop so order doesn't matter
-            // (we drop children first regardless, but this keeps it bullet-proof
-            // if KNOWN_TABLES order is ever reshuffled).
-            this.db.pragma("foreign_keys = OFF");
-            const tx = this.db.transaction(() => {
-                for (const t of KNOWN_TABLES) {
-                    this.db.exec(`DROP TABLE IF EXISTS ${t}`);
-                }
-            });
-            tx();
-            this.db.pragma("foreign_keys = ON");
-        }
+        // ---- schema creation -------------------------------------------------
+        // SCHEMA_SQL uses CREATE TABLE IF NOT EXISTS for every table, so it is
+        // always safe to run — on a fresh database all tables are created;
+        // on an existing one every statement is a no-op.  Schema changes that
+        // add columns are handled by the additive migration block below.
+        //
+        // We NO LONGER drop tables when the stored user_version differs from
+        // SCHEMA_VERSION.  The previous destructive behaviour caused permanent
+        // data loss on version bumps.  SCHEMA_VERSION is now informational — it
+        // is stamped into every new database purely for troubleshooting.
         this.db.exec(SCHEMA_SQL);
-        // Stamp the version AFTER schema creation so a crash mid-create is
-        // detected next open and re-runs the (idempotent) create.
         this.db.pragma(`user_version = ${SCHEMA_VERSION}`);
         // ---- additive column migration ---------------------------------------
         // Phase 3 (Available Models): per-session sticky model selection lives in
@@ -248,6 +240,9 @@ export class SessionStore {
         if (!sessionCols.some((c) => c.name === "model_id")) {
             this.db.exec(`ALTER TABLE sessions ADD COLUMN model_id TEXT`);
         }
+        if (!sessionCols.some((c) => c.name === "reasoning_effort")) {
+            this.db.exec(`ALTER TABLE sessions ADD COLUMN reasoning_effort TEXT`);
+        }
         // Additive migration: images_json column for base64 image attachments on
         // user messages. Older rows default to '' (no images).
         const msgCols = this.db
@@ -338,6 +333,8 @@ export class SessionStore {
         this.stmtDeleteSessionMemorySnapshot = this.db.prepare(`DELETE FROM session_memory_snapshots WHERE session_id = ?`);
         this.stmtGetSessionModel = this.db.prepare(`SELECT model_id FROM sessions WHERE id = ?`);
         this.stmtSetSessionModel = this.db.prepare(`UPDATE sessions SET model_id = ? WHERE id = ?`);
+        this.stmtGetSessionReasoningEffort = this.db.prepare(`SELECT reasoning_effort FROM sessions WHERE id = ?`);
+        this.stmtSetSessionReasoningEffort = this.db.prepare(`UPDATE sessions SET reasoning_effort = ? WHERE id = ?`);
         this.stmtSetForkCompactSource = this.db.prepare(`UPDATE sessions SET fork_compact_source_id = ? WHERE id = ?`);
         this.stmtGetForkCompactSource = this.db.prepare(`SELECT fork_compact_source_id FROM sessions WHERE id = ?`);
         this.stmtInsertProjectObs = this.db.prepare(`INSERT OR REPLACE INTO project_observations (id, project_id, session_id, content, relevance, created_at)
@@ -645,6 +642,23 @@ export class SessionStore {
     setSessionModel(sessionId, modelId) {
         this.stmtSetSessionModel.run(modelId, sessionId);
     }
+    /**
+     * Get the persisted reasoningEffort for a session, or null if none
+     * was ever set. Returns null for unknown sessions (consistent with
+     * getSessionModel's contract).
+     */
+    getSessionReasoningEffort(sessionId) {
+        const row = this.stmtGetSessionReasoningEffort.get(sessionId);
+        return row?.reasoning_effort ?? null;
+    }
+    /**
+     * Persist the reasoningEffort for a session. Pass `null` to clear.
+     * Same semantics as setSessionModel — best-effort metadata, does
+     * not bump `updated_at`.
+     */
+    setSessionReasoningEffort(sessionId, effort) {
+        this.stmtSetSessionReasoningEffort.run(effort, sessionId);
+    }
     // ----------------------------------------------------------------------
     // Fork & Compact
     // ----------------------------------------------------------------------

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aexol/spectral",
-  "version": "0.7.5",
+  "version": "0.7.6",
   "description": "Always-on coding agent for Aexol — branded pi wrapper with relay-based browser access.",
   "type": "module",
   "private": false,