@inetafrica/open-claudia 2.6.6 → 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/runner.js

@@ -132,6 +132,7 @@ async function buildClaudeArgs(prompt, opts = {}) {
     }
   }
   args.push("--model", settings.model || DEFAULT_CLAUDE_MODEL);
+  if (opts.maxTurns) args.push("--max-turns", String(opts.maxTurns));
   if (settings.effort) args.push("--effort", settings.effort);
   if (settings.budget) args.push("--max-budget-usd", String(settings.budget));
   if (settings.permissionMode) args.push("--permission-mode", settings.permissionMode);
@@ -276,6 +277,11 @@ function compactSeedPrompt(summary, extras = {}) {
     "Treat the following summary as the prior conversation context. Do not repeat it back unless asked.",
     "Continue from this state in future turns.",
     "",
+    // A seed turn that starts working (e.g. resuming a CI watch from the
+    // brief) can background a long-running command, and `claude -p` won't
+    // exit while a background task runs — that wedges the message queue.
+    "RIGHT NOW: reply with ONE short acknowledgement line only. Do NOT use any tools, run any commands, or start/resume any work in this turn — even if the summary, pending tasks, or recent turns suggest obvious next steps. Work resumes with the next user message.",
+    "",
     "Before telling the user you lack context on something they reference:",
     "1. Check the summary below.",
     "2. Read the full archived brief on disk (path given below, when present) — it is the detailed version of this summary.",
@@ -545,10 +551,6 @@ async function compactActiveSession(cwd, opts = {}) {
     if (opts.notify) await send(opts.message || "Context is getting large, compacting first so this stays fast…");
     const summaryRun = await runClaudeCapture(compactSummaryPrompt(), cwd, { resumeSessionId: oldSessionId, skipAutoCompact: true, label: "compact-summary", timeoutMs: COMPACT_SUMMARY_TIMEOUT });
     summary = summaryRun.text || "No prior context was returned by the summarizer.";
-    state[sessionKey] = null;
-    state.sessionUsage = { turns: 0, inputTokens: 0, outputTokens: 0, cacheReadTokens: 0, cacheCreationTokens: 0, costUsd: 0, lastInputTokens: 0 };
-    state.isFirstMessage = true;
-    saveState();
   } finally {
     state.isCompacting = false;
     saveState();
@@ -559,9 +561,13 @@ async function compactActiveSession(cwd, opts = {}) {
   // Only seed with the condensed version when the full text actually made it to disk.
   const seedSummary = (condensed && briefPath) ? condensed : (condensed ? `${fullBrief}\n\n${condensed}` : fullBrief);
   const repoFacts = collectRepoStateFacts(cwd);
-  const seedRun = await runClaudeCapture(compactSeedPrompt(seedSummary, { briefPath, condensed: !!(condensed && briefPath), repoFacts, recentTurns }), cwd, { fresh: true, skipAutoCompact: true, label: "compact-seed" });
-  const newSessionId = seedRun.sessionId || state[sessionKey];
-  if (newSessionId) state[sessionKey] = newSessionId;
+  // The old session id stays in place until the seed succeeds: if the seed
+  // dies (timeout, /stop, crash) the conversation keeps resuming the
+  // pre-compaction session instead of being orphaned. maxTurns: 1 hard-stops
+  // a seed that ignores the no-work instruction and reaches for tools.
+  const seedRun = await runClaudeCapture(compactSeedPrompt(seedSummary, { briefPath, condensed: !!(condensed && briefPath), repoFacts, recentTurns }), cwd, { fresh: true, skipAutoCompact: true, label: "compact-seed", timeoutMs: COMPACT_SUMMARY_TIMEOUT, maxTurns: 1 });
+  const newSessionId = seedRun.sessionId || null;
+  state[sessionKey] = newSessionId;
   state.isFirstMessage = false;
   state.lastCompactedAt = Date.now();
   state.sessionUsage = { turns: 0, inputTokens: 0, outputTokens: 0, cacheReadTokens: 0, cacheCreationTokens: 0, costUsd: 0, lastInputTokens: 0 };

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.6",
+  "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": {