pi-jar 0.3.2 → 0.3.3

package/README.md

@@ -302,6 +302,7 @@ Providers cache a prompt as an ordered prefix: tools, then the system prompt, th
   { "action": "append", "todos": [{ "content": "Handle newly requested edge case", "status": "pending" }] }
   ```
   Use `parent` with `append` to add a genuinely new subtask to active work; `start`, `done`, `open`, `edit`, `list`, `add` and `delete` remain available.
+- **Todo gate** — a turn cannot end while `jar_todo` has pending or in-progress leaf tasks: pi-jar injects a hidden reminder and continues (at most 2 reminders per user message; skipped in plan/goal mode, aborted turns and subagents). Disable with `PI_JAR_TODO_GATE=off`.
 - **`jar_ask`** — structured questions: numbered options with descriptions, single or multi-select, *Type your own answer* (multi-line, paste-friendly) and *Chat about this* to discuss before choosing.
 - **`/diff`** — review what the agent (and its editing subagents) changed since the first edit to each file: files with `+/−` counts on the left, a full-path header and OMP-inspired split-first **Before/After** diff on the right (unified below 70 pane columns). Changed lines get a colored rail and soft tinted background, replaced lines highlight changed words with a stronger tint, code is syntax-highlighted for known languages, and collapsed regions are labeled `┄ N unchanged lines ┄`. `/` filters files; `t` switches unified and aligned side-by-side views (unified below 70 columns), `←`/`→` (or `[`/`]`) change the context from 0 lines to the whole file, `w` wraps long lines, and `n`/`p`, `Home`/`End` navigate. Large changes offer summary or explicit windowed review (`v`), with clearly labeled coarse fallback and existing safety limits—not arbitrary-file streaming. See [diff review](docs/DIFF-REVIEW.md). `a` accept (keep, stop tracking), `r` then `y` revert (restore the original, or remove a file it created), `A`/`R` for all. The footer shows `± N files · /diff` while anything is unreviewed; each file counts once no matter who edited it. Only `edit`/`write` changes inside the project are tracked; shell-made changes are not.
 - **Context ownership** — pi-jar does not prune prior reasoning/tool results, impose a soft context limit, or abort/compact/resume work at task boundaries. The context meter and cache diagnostics remain passive. Pi's native compaction and explicit `/compact` (including the user-selected plan approval action) remain available. Retired `contextDiet` and `contextBudget` settings are ignored and dropped on the next settings/profile save; loading does not rewrite user files.

package/extensions/index.ts

@@ -16,6 +16,7 @@ import { renderFooterLayout, type ActivityState, type FooterActivity, type Foote
 import { openJarHistory } from "../src/history-ui.ts";
 import { GOAL_ENTRY, GoalStore } from "../src/goals.ts";
 import { GoalLoop } from "../src/goal-loop.ts";
+import { registerTodoGate } from "../src/todo-gate.ts";
 import { FOOTER_FIELDS } from "../src/footer-settings.ts";
 import { defaultVisualSettings, loadVisualSettings, saveVisualSettings, type JarVisualSettings } from "../src/settings.ts";
 import { ProfileStore, PROFILE_FILE, PROFILE_ENTRY, conversationStarted, pinnedProfileId } from "../src/profiles.ts";
@@ -358,6 +359,10 @@ export default function piJar(pi: ExtensionAPI): void {
     completed: () => composer.flash("complete", 4000)
   });
   if (!delegatedChild) goalLoop.register();
+  if (!delegatedChild) registerTodoGate(pi, {
+    todos: () => todos, suspended: () => planMode.isEnabled() || !!goals?.isActive(),
+    enabled: () => process.env.PI_JAR_TODO_GATE !== "off"
+  });
   planMode.setOnEnter((ctx) => goalLoop.pause(ctx, "plan mode is on"));
   const suggestions = new SuggestionState();
   composer.attachSuggestions(suggestions);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-jar",
-  "version": "0.3.2",
+  "version": "0.3.3",
   "description": "A warm, role-aware Pi terminal workspace with switchable profiles, animated UI, plan and goal workflows, tasks, change review, model roles, background shells and subagents.",
   "type": "module",
   "license": "MIT",

package/src/todo-gate.ts

@@ -0,0 +1,45 @@
+import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
+import type { Todo, TodoStore } from "./tasks.ts";
+
+export const TODO_GATE_TYPE = "pi-jar.todo-gate";
+/** Automatic reminders per user message; keeps a stuck model from looping forever. */
+export const TODO_GATE_MAX_NUDGES = 2;
+
+export interface TodoGateOptions {
+  todos: () => TodoStore | undefined;
+  /** True while plan mode or goal mode owns turn continuation. */
+  suspended: () => boolean;
+  enabled?: () => boolean;
+}
+
+/** Leaf tasks that are still open (parents are derived from their subtasks). */
+export function openLeaves(items: readonly Todo[]): Todo[] {
+  const parents = new Set(items.map((item) => item.parentId).filter(Boolean));
+  return items.filter((item) => !item.done && !parents.has(item.id));
+}
+
+export function todoGateMessage(open: readonly Todo[]): string {
+  const lines = open.slice(0, 10).map((item) => `${item.status === "in_progress" ? "[~]" : "[ ]"} ${item.title} (${item.id})`);
+  return [
+    "[PI-JAR TODO CHECK] jar_todo still has unfinished tasks:",
+    ...lines,
+    open.length > lines.length ? `… and ${open.length - lines.length} more` : "",
+    "Before ending the turn, sync jar_todo: mark verified work completed, or continue with the open task. If a task is obsolete or blocked, remove it or say why."
+  ].filter(Boolean).join("\n");
+}
+
+/** Refuses to end a turn while jar_todo has pending or in-progress tasks (bounded reminders). */
+export function registerTodoGate(pi: ExtensionAPI, options: TodoGateOptions): void {
+  let nudges = 0;
+  pi.on("input", (event) => { if (event.source === "interactive") nudges = 0; });
+  pi.on("session_start", () => { nudges = 0; });
+  pi.on("agent_before_settle", (event) => {
+    if (event.outcome !== "completed" || event.continue || event.context?.canContinue === false) return;
+    if (options.enabled && !options.enabled()) return;
+    if (options.suspended() || nudges >= TODO_GATE_MAX_NUDGES) return;
+    const open = openLeaves(options.todos()?.all() ?? []);
+    if (!open.length) return;
+    nudges++;
+    return { entries: [{ type: "custom_message" as const, customType: TODO_GATE_TYPE, display: false, content: todoGateMessage(open) }], continue: true };
+  });
+}