@devflow-tools/database 0.18.56 → 0.18.58

package/dist/database.js

@@ -1131,6 +1131,58 @@ class DevFlowDatabase {
             }
         }
         catch { }
+        // Retrieval plans are immutable per session, but host supplied expansion
+        // terms can create a second auditable plan for the same task revision.
+        // Older databases keyed this table only by task identity and rejected that
+        // valid replay. Rebuild the table once with plan_hash in the uniqueness
+        // boundary; all existing rows retain their identifiers and receipts.
+        try {
+            const indexes = this.db.prepare('PRAGMA index_list(devflow_retrieval_sessions_v2)').all();
+            const legacyUnique = indexes.find(index => {
+                if (index.unique !== 1)
+                    return false;
+                const columns = this.db.prepare(`PRAGMA index_info(${JSON.stringify(index.name)})`).all();
+                return columns.map(column => column.name).join(',')
+                    === 'project_root,task_session_id,turn_id,task_revision,coordinator_version';
+            });
+            if (legacyUnique) {
+                this.db.exec('ALTER TABLE devflow_retrieval_sessions_v2 RENAME TO devflow_retrieval_sessions_v2_legacy');
+                this.db.exec(`
+          CREATE TABLE devflow_retrieval_sessions_v2 (
+            id TEXT PRIMARY KEY,
+            project_root TEXT NOT NULL,
+            project_id TEXT NOT NULL,
+            host_id TEXT NOT NULL,
+            task_session_id TEXT NOT NULL,
+            turn_id TEXT NOT NULL,
+            request_id TEXT NOT NULL,
+            execution_id TEXT,
+            task_revision INTEGER NOT NULL CHECK(task_revision > 0),
+            facts_hash TEXT NOT NULL,
+            plan_hash TEXT NOT NULL,
+            activation_snapshot_hash TEXT NOT NULL,
+            coordinator_version TEXT NOT NULL,
+            state TEXT NOT NULL CHECK(state IN ('open','sufficient','gap_actionable','gap_unavailable','abstain')),
+            cycle INTEGER NOT NULL DEFAULT 0 CHECK(cycle BETWEEN 0 AND 3),
+            no_gain_count INTEGER NOT NULL DEFAULT 0 CHECK(no_gain_count BETWEEN 0 AND 2),
+            max_cycles INTEGER NOT NULL DEFAULT 3 CHECK(max_cycles = 3),
+            initial_token_budget INTEGER NOT NULL CHECK(initial_token_budget >= 0),
+            remaining_token_budget INTEGER NOT NULL CHECK(remaining_token_budget >= 0),
+            final_receipt_hash TEXT,
+            created_at INTEGER NOT NULL,
+            updated_at INTEGER NOT NULL,
+            expires_at INTEGER NOT NULL,
+            UNIQUE(project_root, task_session_id, turn_id, task_revision, coordinator_version, plan_hash)
+          );
+          INSERT INTO devflow_retrieval_sessions_v2
+            SELECT * FROM devflow_retrieval_sessions_v2_legacy;
+          DROP TABLE devflow_retrieval_sessions_v2_legacy;
+          CREATE INDEX IF NOT EXISTS idx_retrieval_sessions_v2_owner
+            ON devflow_retrieval_sessions_v2(project_root, task_session_id, state, updated_at);
+        `);
+            }
+        }
+        catch { }
         // Migration: add session_id to skill_executions (SQLite compat — ignore if exists)
         try {
             this.db.exec('ALTER TABLE skill_executions ADD COLUMN session_id TEXT REFERENCES sessions(id)');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@devflow-tools/database",
-  "version": "0.18.56",
+  "version": "0.18.58",
   "description": "SQLite persistence primitives shared by DevFlow runtime packages.",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -12,7 +12,7 @@
     "test": "vitest run"
   },
   "dependencies": {
-    "@devflow-tools/sdk": "0.18.56"
+    "@devflow-tools/sdk": "0.18.58"
   },
   "devDependencies": {
     "@types/node": "^22.15.0",