@inetafrica/open-claudia 2.6.7 → 2.6.8

package/CHANGELOG.md

@@ -1,5 +1,8 @@
 # Changelog
 
+## v2.6.8
+- **Tasks: a plan can't be completed while subtasks are still open.** `task done` (and the loopback task-update path) now refuses to flip a parent to completed if any child is not yet completed — it returns a blocked result listing the open children, and the CLI prints a clean "Can't complete <id>: N subtask(s) still open" message instead of silently corrupting the tree. Standalone tasks and completing the last open child (which removes the whole plan) are unchanged. Prevents the inflated-count drift where parents showed done over still-open children.
+
 ## v2.6.6
 - **Recall: recent-turns window as judge-gated context.** v2.6.5 fixed quotes, but a bare follow-up like "Push" with no quote still recalled nothing — recall only ever saw the current message. The last ~6 user/assistant turns on this channel (read from the project transcript, current turn excluded) now feed recall the same way the quote does: keyword-matched as origin "context", kept only when the haiku judge confirms relevance, dropped on fail-open. A stale thread can never force-inject packs on keywords alone, but terse follow-ups finally inherit the topic of the conversation.
 

package/bin/task.js

@@ -13,6 +13,7 @@ Per-channel todo list with optional plan/subtask hierarchy.
   open-claudia task list [--status pending|in_progress|completed]
   open-claudia task start <id>
   open-claudia task done <id>            # completes AND removes; last subtask done removes the plan
+                                         # a plan won't complete while any subtask is still open
   open-claudia task remove <id>          # removing a plan removes its subtasks
   open-claudia task clear-completed      # prune any leftover completed entries
 
@@ -106,6 +107,12 @@ async function runUpdate(args, status) {
   if (!id) { console.error(`Usage: task ${status === "in_progress" ? "start" : "done"} <id>`); process.exit(2); }
   try {
     const res = await postJson("task-update", { id, status });
+    if (res.blocked) {
+      const kids = res.openChildren || [];
+      const names = kids.map((c) => `  - ${c.content} (${c.id})`).join("\n");
+      console.error(`Can't complete ${id}: ${kids.length} subtask${kids.length === 1 ? "" : "s"} still open. Finish or remove ${kids.length === 1 ? "it" : "them"} first:\n${names}`);
+      process.exit(1);
+    }
     if (!res.ok) { console.error(`Not found: ${id}`); process.exit(1); }
     if (status === "completed") {
       const n = res.removedCount || 0;

package/core/loopback.js

@@ -226,7 +226,9 @@ async function handleJson(req, res, url, kind) {
     if (!payload.id) return reply(res, 400, { error: "missing id" });
     if (payload.status === "completed") {
       const result = tasksStore.complete(adapterId, channelId, payload.id);
-      return reply(res, result ? 200 : 404, { ok: !!result, task: result ? result.task : null, removedCount: result ? result.removedCount : 0 });
+      if (!result) return reply(res, 404, { ok: false, task: null, removedCount: 0 });
+      if (result.blocked) return reply(res, 200, { ok: false, blocked: true, task: result.task, openChildren: result.openChildren });
+      return reply(res, 200, { ok: true, task: result.task, removedCount: result.removedCount });
     }
     const patch = {};
     if (payload.status) patch.status = payload.status;

package/core/tasks.js

@@ -120,7 +120,20 @@ function prune(adapter, channelId) {
 
 // Mark a task completed, then prune. Reports how many entries left the
 // list so callers can tell the agent what disappeared.
+//
+// A plan can't be completed while any of its subtasks are still open:
+// flipping the parent to [x] over [ ] children leaves a contradictory
+// row that lingers (prune only drops a parent once all kids are done)
+// and inflates the visible list. Refuse instead, naming the open kids
+// so the caller finishes or removes them first.
 function complete(adapter, channelId, id) {
+  const list = load(adapter, channelId);
+  const target = list.find((t) => t.id === id);
+  if (!target) return null;
+  if (!target.parentId) {
+    const openChildren = list.filter((c) => c.parentId === id && c.status !== "completed");
+    if (openChildren.length > 0) return { blocked: true, task: target, openChildren };
+  }
   const t = update(adapter, channelId, id, { status: "completed" });
   if (!t) return null;
   const before = load(adapter, channelId).length;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@inetafrica/open-claudia",
-  "version": "2.6.7",
+  "version": "2.6.8",
   "description": "Your always-on AI coding assistant — Claude Code, Cursor Agent, and OpenAI Codex via Telegram or Kazee Chat",
   "main": "bot.js",
   "bin": {