pi-note 0.18.0 → 0.18.1

package/README.md

@@ -84,7 +84,7 @@ The agent can still read the task separately with `pinote_get_current`.
 Click previews work while the agent is running, provided no other Pinote operation
 is open. Each click saves a fresh snapshot in the transcript, above any currently
 streaming response. Later output scrolls it upward; it does not expire. Preview
-reads never lock out the agent's task updates. Settings still requires Pi to be idle.
+reads never lock out the agent's task updates. Settings also works during a running turn.
 
 ![Preview above a running response](../docs/images/pi-note-active-preview.png)
 
@@ -358,7 +358,11 @@ Pinote no longer polls GitHub, runs `gh`, watches completed tasks, or prompts on
 PR merges. No GitHub authentication is required. `PINOTE_PR_POLL_SECONDS` is no
 longer used. Other separately installed Pi PR/Git extensions are unaffected.
 
-Tools work without a TUI, but `/pi-note` needs an idle TUI.
+Tools work without a TUI. `/pi-note` works in a TUI while the agent is running:
+you can select/switch tasks, use Continue, preview tasks, respond to suggestions,
+and edit settings without interrupting the turn or submitting input. Completion
+and CLI installation/upgrades still require an idle session. Another pending
+Pinote operation blocks the menu. Open menus do not block agent handoff updates.
 This package does not synchronize databases or paths between machines. Note text
 and fields loaded into Pi are sent to the configured model when used as context;
 avoid secrets. Agent commands do not send desktop notifications.

package/index.ts

@@ -169,6 +169,7 @@ export default function (pi: ExtensionAPI) {
   let epoch = 0;
   let refreshSerial = 0;
   let pending: symbol | undefined;
+  let menuPending: symbol | undefined;
   let previewPending = false;
   let setupAbort: AbortController | undefined;
   let activeContext: ExtensionContext | undefined;
@@ -318,7 +319,7 @@ export default function (pi: ExtensionAPI) {
     }
   };
   const preview = async (ctx: ExtensionContext, requestSignal?: AbortSignal): Promise<boolean> => {
-    if (!ctx.hasUI || ctx.mode !== "tui" || pending || previewPending || !validId(selectedId)) return false;
+    if (!ctx.hasUI || ctx.mode !== "tui" || pending || menuPending || previewPending || !validId(selectedId)) return false;
     const generation = epoch;
     const branch = branchEpoch;
     const id = selectedId;
@@ -346,7 +347,7 @@ export default function (pi: ExtensionAPI) {
     }
   };
   const dispatchCompletion = (ctx: ExtensionContext, displayed: Task, selection: string): Promise<boolean> => {
-    if (!ctx.isIdle() || pending || completionDispatch) return Promise.resolve(false);
+    if (!ctx.isIdle() || pending || menuPending || completionDispatch) return Promise.resolve(false);
     const token = `${selection}:${randomBytes(8).toString("hex")}`;
     return new Promise((resolve) => {
       const finish = (accepted: boolean) => {
@@ -367,7 +368,7 @@ export default function (pi: ExtensionAPI) {
     const token = args.trim();
     const request = token ? completionDispatch : undefined;
     if (token && (request?.token !== token || request.selection !== `${runtimeToken}:${epoch}:${branchEpoch}:${selectedId}`)) return;
-    if (!alive || !ctx.hasUI || ctx.mode !== "tui" || !ctx.isIdle() || pending || setupAbort || !validId(selectedId)) {
+    if (!alive || !ctx.hasUI || ctx.mode !== "tui" || !ctx.isIdle() || pending || menuPending || setupAbort || !validId(selectedId)) {
       request?.finish(false);
       if (ctx.hasUI) ctx.ui.notify("Select a task and wait until Pi is idle and the Pinote operation has finished.", "warning");
       return;
@@ -459,6 +460,7 @@ export default function (pi: ExtensionAPI) {
     if (!alive || generation !== epoch) return;
     // Keep the setup lock until its aborted subprocess has actually settled.
     if (!setupAbort) pending = undefined;
+    menuPending = undefined;
     await checkCLI(ctx, true);
     if (!alive || generation !== epoch) return;
     if (suggestion) previewBridge?.setSuggestion((choice) => respondToSuggestion(choice, ctx));
@@ -522,7 +524,7 @@ export default function (pi: ExtensionAPI) {
 
   const installCLI = async (upgrade: boolean, ctx: ExtensionCommandContext) => {
     if (!ctx.hasUI || ctx.mode !== "tui") return;
-    if (pending || !ctx.isIdle()) {
+    if (pending || menuPending || !ctx.isIdle()) {
       ctx.ui.notify("Wait until Pi is idle and the pinote operation has finished.", "warning");
       return;
     }
@@ -563,16 +565,18 @@ export default function (pi: ExtensionAPI) {
         return;
       }
       if (token) { ctx.ui.notify("Usage: /pi-note", "warning"); return; }
-      if (pending || !ctx.isIdle()) {
-        ctx.ui.notify("Wait until Pi is idle and the pinote operation has finished.", "warning");
+      if (pending || menuPending) {
+        ctx.ui.notify("Wait until the pinote operation has finished.", "warning");
         return;
       }
       const operation = Symbol();
-      pending = operation;
+      menuPending = operation;
       const generation = epoch;
       const branch = branchEpoch;
       const currentSession = () => alive && generation === epoch;
-      const canAct = () => currentSession() && ctx.isIdle() && branch === branchEpoch;
+      // Menus, task selection and preference edits do not interrupt the agent.
+      // Completion and CLI setup retain their own idle-only guards.
+      const canAct = () => currentSession() && branch === branchEpoch && menuPending === operation;
       try {
         await checkCLI(ctx);
         if (!canAct()) return;
@@ -633,7 +637,7 @@ export default function (pi: ExtensionAPI) {
             if (!canAct() || edited === undefined) return;
             if (edited === "tasks") { settings = false; continue; }
             if (!options.some((option) => option.result === edited)) return;
-            if (pending === operation) pending = undefined;
+            if (menuPending === operation) menuPending = undefined;
             if (edited === "preview") {
               if (!await preview(ctx)) ctx.ui.notify("The task is no longer available for preview.", "warning");
             } else if (edited === "done") await completeSelected("", ctx, current ?? undefined);
@@ -649,7 +653,7 @@ export default function (pi: ExtensionAPI) {
           if (!action || !canAct()) return;
           if (action === "Settings") { settings = true; continue; }
           if (action === "Done" && current) {
-            if (pending === operation) pending = undefined;
+            if (menuPending === operation) menuPending = undefined;
             await completeSelected("", ctx, current);
             return;
           }
@@ -667,6 +671,11 @@ export default function (pi: ExtensionAPI) {
             id = choice;
           } else return;
           const handoff = handoffPrompt();
+          if (pending) {
+            ctx.ui.notify("Wait until the pinote operation has finished.", "warning");
+            return;
+          }
+          pending = operation;
           refreshSerial++;
           const chosen = await startTask(id, canAct);
           if (!chosen || !canAct()) return;
@@ -679,6 +688,7 @@ export default function (pi: ExtensionAPI) {
         if (currentSession()) ctx.ui.notify(`Pinote: ${clean(error instanceof Error ? error.message : String(error))}`, "error");
       } finally {
         if (pending === operation) pending = undefined;
+        if (menuPending === operation) menuPending = undefined;
         if (currentSession()) await refresh(ctx);
       }
     },
@@ -755,7 +765,7 @@ export default function (pi: ExtensionAPI) {
       clearSuggestion(ctx);
       return true;
     }
-    if (pending || setupAbort) return false;
+    if (pending || menuPending || setupAbort) return false;
     const generation = epoch;
     const branch = branchEpoch;
     const operation = Symbol();
@@ -804,7 +814,7 @@ export default function (pi: ExtensionAPI) {
       const current = await selected(ctx, signal);
       if (!alive || generation !== epoch || branch !== branchEpoch) throw new Error("Pinote operation cancelled: the session changed.");
       if (current || selectedId !== null) throw new Error("A task is already selected; keep it unless the user asks to switch.");
-      if (pending || setupAbort) throw new Error("A pinote operation is already open. Retry after it finishes.");
+      if (pending || menuPending || setupAbort) throw new Error("A pinote operation is already open. Retry after it finishes.");
       const result = (status: string) => ({ content: [{ type: "text" as const, text: JSON.stringify({ status }) }], details: { status } });
       suggestion = { text: params.text, ...(params.tag === undefined ? {} : { tag: params.tag.trim() }) };
       saveSuggestion();
@@ -834,7 +844,7 @@ export default function (pi: ExtensionAPI) {
       const generation = epoch;
       const branch = branchEpoch;
       if (params.select !== true) return toolResult(await writeTask(ctx, signal, argv, undefined, "0.4.0"));
-      if (pending) throw new Error("A pinote operation is already open. Retry after it finishes.");
+      if (pending || menuPending) throw new Error("A pinote operation is already open. Retry after it finishes.");
       const operation = Symbol();
       const previousSelection = selectedId;
       pending = operation;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-note",
-  "version": "0.18.0",
+  "version": "0.18.1",
   "description": "Persistent pinote tasks, Markdown handoffs, and task status for Pi",
   "license": "MIT",
   "type": "module",