@loicngr/kobo 1.10.0 → 1.10.1

package/CHANGELOG.md

@@ -4,6 +4,10 @@ All notable changes to Kōbō are documented here. The format is based on
 [Keep a Changelog](https://keepachangelog.com/). Each release is an `## <version>`
 section — the in-app "What's new" dialog reads this file.
 
+## 1.10.1
+
+- fix(auto-loop): recover from interrupted sessions (#18)
+
 ## 1.10.0
 
 - fix(agent): make Claude sessions recover cleanly from a stuck post-result

package/dist/server/index.js

@@ -388,6 +388,9 @@ setMessageHandler(async (type, payload) => {
     }
     if (type === 'workspace:stop' && p?.workspaceId) {
         try {
+            // A deliberate stop must not be treated like a recoverable engine
+            // interruption: disable before the engine emits session:ended(killed).
+            autoLoopService.disable(p.workspaceId, 'user-action');
             stopAgent(p.workspaceId);
         }
         catch (err) {

package/dist/server/services/agent/orchestrator.js

@@ -395,11 +395,12 @@ function reuseOrCreateFreshSession(workspaceId, existingSessionId, model) {
 }
 // ── Event handler ─────────────────────────────────────────────────────────────
 /**
- * Snapshot of the `tasks` done-count at `session:started`, read back and
- * cleared at `session:ended` to compute the per-session delta. Used by
- * `auto-loop-service.onSessionEnded` for stall detection.
+ * Snapshots of task completion and task state at `session:started`, read back
+ * at `session:ended` for auto-loop stall detection. A task moved to
+ * `in_progress` is meaningful progress even if it is not done yet.
  */
 const tasksDoneSnapshot = new Map();
+const taskStateSnapshot = new Map();
 function getDoneTaskCount(workspaceId) {
     try {
         const db = getDb();
@@ -415,9 +416,23 @@ function getDoneTaskCount(workspaceId) {
         return 0;
     }
 }
+function getTaskStateSignature(workspaceId) {
+    try {
+        const db = getDb();
+        const rows = db
+            .prepare('SELECT id, status, updated_at FROM tasks WHERE workspace_id = ? ORDER BY id')
+            .all(workspaceId);
+        return JSON.stringify(rows);
+    }
+    catch (err) {
+        console.warn('[orchestrator] getTaskStateSignature failed, returning empty state:', err);
+        return '';
+    }
+}
 /** Clear the in-memory done-count snapshot for a workspace (called on delete). */
 export function forgetTasksDoneSnapshot(workspaceId) {
     tasksDoneSnapshot.delete(workspaceId);
+    taskStateSnapshot.delete(workspaceId);
 }
 /** Drop the resume-failed flag for a workspace (called on delete). */
 export function forgetResumeFailed(workspaceId) {
@@ -444,6 +459,7 @@ function handleEvent(workspaceId, agentSessionId, ev) {
     // can compute a delta for auto-loop stall detection.
     if (ev.kind === 'session:started') {
         tasksDoneSnapshot.set(workspaceId, getDoneTaskCount(workspaceId));
+        taskStateSnapshot.set(workspaceId, getTaskStateSignature(workspaceId));
     }
     // Legacy fallback: the built-in `ScheduleWakeup` tool (CLI tradition) isn't
     // declared by the SDK, so we intercept the tool:call event and apply the
@@ -531,8 +547,12 @@ function handleEvent(workspaceId, agentSessionId, ev) {
         const isResumeFailed = resumeFailedSet.delete(workspaceId);
         const before = tasksDoneSnapshot.get(workspaceId) ?? getDoneTaskCount(workspaceId);
         const after = getDoneTaskCount(workspaceId);
-        const delta = Math.max(0, after - before);
+        const completedDelta = Math.max(0, after - before);
+        const taskStateBefore = taskStateSnapshot.get(workspaceId) ?? getTaskStateSignature(workspaceId);
+        const taskStateAfter = getTaskStateSignature(workspaceId);
+        const progressDelta = completedDelta > 0 || taskStateBefore !== taskStateAfter ? 1 : 0;
         tasksDoneSnapshot.delete(workspaceId);
+        taskStateSnapshot.delete(workspaceId);
         clearPendingForSession(workspaceId, agentSessionId);
         // Must run BEFORE autoLoopService.onSessionEnded → spawnNextIteration →
         // startAgent, otherwise startAgent throws "Agent already running" because
@@ -546,7 +566,7 @@ function handleEvent(workspaceId, agentSessionId, ev) {
         // disable() clears it, and the cleanup hook needs to know whether this was
         // a mid-loop session (never cleans) or a standalone one.
         const wasAutoLoop = autoLoopService.getStatus(workspaceId).auto_loop;
-        autoLoopService.onSessionEnded(workspaceId, effectiveReason, delta);
+        autoLoopService.onSessionEnded(workspaceId, effectiveReason, progressDelta);
         cleanupScriptService.onSessionEnded(workspaceId, effectiveReason, { wasAutoLoop });
     }
     if (ev.kind === 'session:user-input-requested') {

package/dist/server/services/auto-loop-service.js

@@ -82,7 +82,15 @@ export function disable(workspaceId, reason) {
         return;
     const db = getDb();
     db.prepare('UPDATE workspaces SET auto_loop = 0 WHERE id = ?').run(workspaceId);
-    emitEphemeral(workspaceId, 'autoloop:disabled', { reason });
+    const diagnostic = {
+        reason,
+        tasksPending: countPendingTasks(workspaceId),
+        noProgressStreak: row.no_progress_streak,
+        status: row.status,
+    };
+    console.info(`[auto-loop] disabled workspace '${workspaceId}' reason=${reason}` +
+        ` pending=${diagnostic.tasksPending} streak=${diagnostic.noProgressStreak} status=${diagnostic.status}`);
+    emitEphemeral(workspaceId, 'autoloop:disabled', diagnostic);
     // The loop finished every task — run the project's cleanup script. Other
     // disable reasons (stall / error / user-action) leave tasks unfinished, so
     // they intentionally skip the cleanup.
@@ -93,15 +101,15 @@ export function disable(workspaceId, reason) {
 /**
  * Route a `session:ended` event into the auto-loop state machine.
  *
- * Called by orchestrator.handleEvent. The delta is the number of tasks that
- * transitioned to `done` during this session (computed from a snapshot taken
- * at `session:started`).
+ * Called by orchestrator.handleEvent. The delta records whether task state
+ * progressed during this session, including transitions such as pending to
+ * in_progress, not only completion.
  *
  * When status is `quota` we skip spawning: the orchestrator's handleQuota
  * already scheduled a backoff timer and will call `onQuotaBackoffExpired` once
  * the window closes — that function owns the next spawn in that case.
  */
-export function onSessionEnded(workspaceId, reason, tasksDoneDelta) {
+export function onSessionEnded(workspaceId, reason, taskProgressDelta) {
     const row = getRow(workspaceId);
     if (!row)
         return;
@@ -115,7 +123,7 @@ export function onSessionEnded(workspaceId, reason, tasksDoneDelta) {
     // will resume the deferred turn explicitly.
     if (row.status === 'awaiting-user')
         return;
-    if (reason === 'error' || reason === 'killed') {
+    if (reason === 'error') {
         disable(workspaceId, 'error');
         return;
     }
@@ -126,7 +134,7 @@ export function onSessionEnded(workspaceId, reason, tasksDoneDelta) {
         return;
     const db = getDb();
     let streak;
-    if (tasksDoneDelta > 0) {
+    if (taskProgressDelta > 0) {
         db.prepare('UPDATE workspaces SET no_progress_streak = 0 WHERE id = ?').run(workspaceId);
         streak = 0;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@loicngr/kobo",
-  "version": "1.10.0",
+  "version": "1.10.1",
   "description": "Kōbō — multi-workspace agent manager for Claude Code. Orchestrates isolated git worktrees with dev servers, Notion integration, and MCP tools.",
   "type": "module",
   "license": "GPL-3.0-or-later",