grov 0.6.15 → 0.6.16

package/dist/cli/commands/doctor.d.ts

@@ -1 +1,3 @@
-export declare function doctor(agent?: string): Promise<void>;
+export declare function doctor(agent?: string, options?: {
+    repair?: boolean;
+}): Promise<void>;

package/dist/cli/commands/doctor.js

@@ -5,11 +5,16 @@ import { join } from 'path';
 import { request } from 'undici';
 import { readCredentials, getSyncStatus } from '../../core/cloud/credentials.js';
 import { initDatabase } from '../../core/store/database.js';
+import { getOrphanedSessionCount, repairOrphanedSessions } from '../../core/store/sessions.js';
 import { getAllCliAgents, getCliAgentById } from '../agents/registry.js';
 const DB_PATH = join(homedir(), '.grov', 'memory.db');
-export async function doctor(agent) {
+export async function doctor(agent, options) {
     console.log('\nGrov Doctor');
     console.log('===========\n');
+    if (options?.repair) {
+        await runRepair();
+        return;
+    }
     if (!agent) {
         await runGeneralChecks();
         console.log('\n--- Agent Status ---\n');
@@ -60,6 +65,31 @@ async function runGeneralChecks() {
     const dbOk = dbStats.tasks > 0 || dbStats.sessions > 0;
     const dbMsg = `${dbStats.tasks} tasks, ${dbStats.unsynced} unsynced, ${dbStats.sessions} active`;
     printCheck('Local Database', dbOk, dbMsg, 'Empty', 'Use Claude/Codex with proxy, or Cursor with MCP');
+    const orphans = checkOrphanedSessions();
+    printCheck('Session Integrity', orphans === 0, 'No orphaned sessions', `${orphans} orphaned session(s)`, 'grov doctor --repair');
+}
+async function runRepair() {
+    console.log('Database Repair\n');
+    if (!existsSync(DB_PATH)) {
+        console.log('\x1b[31m✗\x1b[0m No database found');
+        return;
+    }
+    initDatabase();
+    const orphans = getOrphanedSessionCount();
+    if (orphans === 0) {
+        console.log('\x1b[32m✓\x1b[0m No orphaned sessions found');
+        return;
+    }
+    console.log(`Found ${orphans} orphaned session(s)`);
+    const repaired = repairOrphanedSessions();
+    console.log(`\x1b[32m✓\x1b[0m Repaired ${repaired} session(s)`);
+}
+function checkOrphanedSessions() {
+    if (!existsSync(DB_PATH)) {
+        return 0;
+    }
+    initDatabase();
+    return getOrphanedSessionCount();
 }
 async function runClaudeChecks() {
     console.log('Claude Code Checks\n');

package/dist/cli/index.js

@@ -166,11 +166,21 @@ program
 program
     .command('doctor [agent]')
     .description('Check grov setup and diagnose issues (optional: claude, codex)')
-    .action(safeAction(async (agent) => {
-    const { doctor } = await import('./commands/doctor.js');
-    const agentName = agent === 'claude' || agent === 'codex' ? agent : undefined;
-    await doctor(agentName);
-}));
+    .option('--repair', 'Repair database integrity issues')
+    .action(async (agent, options) => {
+    try {
+        const { doctor } = await import('./commands/doctor.js');
+        const agentName = agent === 'claude' || agent === 'codex' ? agent : undefined;
+        await doctor(agentName, options);
+    }
+    catch (error) {
+        console.error('Error:', error instanceof Error ? error.message : 'Unknown error');
+        process.exit(1);
+    }
+    finally {
+        closeDatabase();
+    }
+});
 // grov agents - List supported agents
 program
     .command('agents')

package/dist/core/store/convenience.d.ts

@@ -26,6 +26,7 @@ export declare function markCleared(sessionId: string): void;
 export declare function markSessionCompleted(sessionId: string): void;
 /**
  * Cleanup old completed/abandoned sessions and their steps/drift_log
+ * Orders deletion by hierarchy depth (children first) to respect FK constraints
  */
 export declare function cleanupOldCompletedSessions(maxAgeMs?: number): number;
 /**

package/dist/core/store/convenience.js

@@ -52,33 +52,48 @@ export function markSessionCompleted(sessionId) {
 }
 /**
  * Cleanup old completed/abandoned sessions and their steps/drift_log
+ * Orders deletion by hierarchy depth (children first) to respect FK constraints
  */
 export function cleanupOldCompletedSessions(maxAgeMs = 86400000) {
     const database = getDb();
     const cutoff = new Date(Date.now() - maxAgeMs).toISOString();
+    // Get sessions ordered by depth: leaves first, then parents
     const oldSessions = database.prepare(`
-    SELECT session_id FROM session_states
-    WHERE status IN ('completed', 'abandoned')
-      AND completed_at < ?
-      AND session_id NOT IN (
-        SELECT DISTINCT parent_session_id
-        FROM session_states
-        WHERE parent_session_id IS NOT NULL
-          AND status != 'completed'
+    WITH RECURSIVE session_depth AS (
+      SELECT s.session_id, 0 as depth
+      FROM session_states s
+      WHERE s.status IN ('completed', 'abandoned')
+        AND s.completed_at < ?
+        AND NOT EXISTS (
+          SELECT 1 FROM session_states child
+          WHERE child.parent_session_id = s.session_id
+        )
+      UNION ALL
+      SELECT parent.session_id, sd.depth + 1
+      FROM session_states parent
+      INNER JOIN session_depth sd ON sd.session_id IN (
+        SELECT session_id FROM session_states
+        WHERE parent_session_id = parent.session_id
       )
-  `).all(cutoff);
+      WHERE parent.status IN ('completed', 'abandoned')
+        AND parent.completed_at < ?
+    )
+    SELECT DISTINCT session_id, MAX(depth) as depth
+    FROM session_depth
+    GROUP BY session_id
+    ORDER BY depth ASC
+  `).all(cutoff, cutoff);
     if (oldSessions.length === 0) {
         return 0;
     }
-    // Delete in correct order to respect FK constraints
-    for (const session of oldSessions) {
-        // 1. Delete from drift_log (FK to session_states)
-        database.prepare('DELETE FROM drift_log WHERE session_id = ?').run(session.session_id);
-        // 2. Delete from steps (FK to session_states)
-        database.prepare('DELETE FROM steps WHERE session_id = ?').run(session.session_id);
-        // 3. Now safe to delete session_states
-        database.prepare('DELETE FROM session_states WHERE session_id = ?').run(session.session_id);
-    }
+    const deleteTransaction = database.transaction(() => {
+        for (const session of oldSessions) {
+            database.prepare('DELETE FROM drift_log WHERE session_id = ?').run(session.session_id);
+            database.prepare('DELETE FROM steps WHERE session_id = ?').run(session.session_id);
+            database.prepare('DELETE FROM session_states WHERE session_id = ?').run(session.session_id);
+        }
+    });
+    deleteTransaction();
     return oldSessions.length;
 }
 /**

package/dist/core/store/database.js

@@ -377,6 +377,13 @@ export function initDatabase() {
     CREATE INDEX IF NOT EXISTS idx_drift_log_session ON drift_log(session_id);
     CREATE INDEX IF NOT EXISTS idx_drift_log_timestamp ON drift_log(timestamp);
   `);
+    // Nullify orphaned parent references
+    db.prepare(`
+    UPDATE session_states
+    SET parent_session_id = NULL
+    WHERE parent_session_id IS NOT NULL
+      AND parent_session_id NOT IN (SELECT session_id FROM session_states)
+  `).run();
     return db;
 }
 /**

package/dist/core/store/index.d.ts

@@ -1,7 +1,7 @@
 export type { TaskStatus, TriggerReason, SessionStatus, SessionMode, TaskType, StepActionType, DriftType, CorrectionLevel, Task, CreateTaskInput, RecoveryPlan, DriftEvent, SessionState, CreateSessionStateInput, StepRecord, CreateStepInput, DriftLogEntry, CreateDriftLogInput, } from './types.js';
 export { initDatabase, closeDatabase, getDatabasePath } from './database.js';
 export { createTask, getTasksForProject, getTaskCount, getUnsyncedTasks, markTaskSynced, setTaskSyncError, getSyncedTaskCount, cleanupOldSyncedTasks, cleanupFailedSyncTasks } from './tasks.js';
-export { createSessionState, getSessionState, updateSessionState, deleteSessionState, getActiveSessionForUser, getActiveSessionsForStatus, getCompletedSessionForProject, clearStalePendingCorrections } from './sessions.js';
+export { createSessionState, getSessionState, updateSessionState, deleteSessionState, getActiveSessionForUser, getActiveSessionsForStatus, getCompletedSessionForProject, clearStalePendingCorrections, getOrphanedSessionCount, repairOrphanedSessions } from './sessions.js';
 export { createStep, getStepsForSession, getRecentSteps, getValidatedSteps, getKeyDecisions, getEditedFiles, deleteStepsForSession, updateRecentStepsReasoning, updateLastChecked } from './steps.js';
 export { updateSessionDrift, logDriftEvent } from './drift.js';
 export { updateTokenCount, updateSessionMode, markWaitingForRecovery, incrementEscalation, markCleared, markSessionCompleted, cleanupOldCompletedSessions, cleanupStaleActiveSessions } from './convenience.js';

package/dist/core/store/index.js

@@ -4,7 +4,7 @@ export { initDatabase, closeDatabase, getDatabasePath } from './database.js';
 // Re-export task functions
 export { createTask, getTasksForProject, getTaskCount, getUnsyncedTasks, markTaskSynced, setTaskSyncError, getSyncedTaskCount, cleanupOldSyncedTasks, cleanupFailedSyncTasks } from './tasks.js';
 // Re-export session functions
-export { createSessionState, getSessionState, updateSessionState, deleteSessionState, getActiveSessionForUser, getActiveSessionsForStatus, getCompletedSessionForProject, clearStalePendingCorrections } from './sessions.js';
+export { createSessionState, getSessionState, updateSessionState, deleteSessionState, getActiveSessionForUser, getActiveSessionsForStatus, getCompletedSessionForProject, clearStalePendingCorrections, getOrphanedSessionCount, repairOrphanedSessions } from './sessions.js';
 // Re-export step functions
 export { createStep, getStepsForSession, getRecentSteps, getValidatedSteps, getKeyDecisions, getEditedFiles, deleteStepsForSession, updateRecentStepsReasoning, updateLastChecked } from './steps.js';
 // Re-export drift functions

package/dist/core/store/sessions.d.ts

@@ -37,3 +37,11 @@ export declare function getCompletedSessionForProject(projectPath: string): Sess
  * Returns number of sessions cleared
  */
 export declare function clearStalePendingCorrections(): number;
+/**
+ * Detect orphaned parent_session_id references
+ */
+export declare function getOrphanedSessionCount(): number;
+/**
+ * Nullify orphaned parent_session_id references
+ */
+export declare function repairOrphanedSessions(): number;

package/dist/core/store/sessions.js

@@ -255,3 +255,28 @@ export function clearStalePendingCorrections() {
   `).run();
     return result.changes;
 }
+/**
+ * Detect orphaned parent_session_id references
+ */
+export function getOrphanedSessionCount() {
+    const database = getDb();
+    const result = database.prepare(`
+    SELECT COUNT(*) as count FROM session_states
+    WHERE parent_session_id IS NOT NULL
+      AND parent_session_id NOT IN (SELECT session_id FROM session_states)
+  `).get();
+    return result.count;
+}
+/**
+ * Nullify orphaned parent_session_id references
+ */
+export function repairOrphanedSessions() {
+    const database = getDb();
+    const result = database.prepare(`
+    UPDATE session_states
+    SET parent_session_id = NULL
+    WHERE parent_session_id IS NOT NULL
+      AND parent_session_id NOT IN (SELECT session_id FROM session_states)
+  `).run();
+    return result.changes;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "grov",
-  "version": "0.6.15",
+  "version": "0.6.16",
   "description": "Collective AI memory for Coding Agents - captures reasoning from sessions and injects context into future sessions",
   "type": "module",
   "main": "dist/cli/index.js",