@jwilger/pi-development-system 0.38.2 → 0.40.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jwilger/pi-development-system",
-  "version": "0.38.2",
+  "version": "0.40.0",
   "description": "A pi extension package representing a seasoned approach to software development using a full AI SDLC.",
   "keywords": [
     "pi-package"

package/src/context/compaction.ts

@@ -15,10 +15,15 @@ export function registerCompactionResync(deps: { pi: ExtensionAPI; state: Sessio
   deps.pi.on("session_compact", () => {
     const state = deps.state.get();
     if (state.phase === "idle" && state.openDepartures.length === 0) return;
-    deps.pi.sendMessage({
-      customType: RESYNC_ENTRY_TYPE,
-      content: `[devsys resync] Context was compacted. Current working state:\n\n${renderStateBlock(state)}`,
-      display: true,
-    });
+    // triggerTurn:false: while a run is active pi would otherwise steer the message, and the queued
+    // message would force an extra model turn after automatic compaction.
+    deps.pi.sendMessage(
+      {
+        customType: RESYNC_ENTRY_TYPE,
+        content: `[devsys resync] Context was compacted. Current working state:\n\n${renderStateBlock(state)}`,
+        display: true,
+      },
+      { triggerTurn: false },
+    );
   });
 }

package/src/context/model-advice.ts

@@ -39,7 +39,10 @@ export function registerModelAdvice(deps: ModelAdviceDeps): void {
     });
     if (advice === undefined) return;
     told.add(key);
-    deps.pi.sendMessage({ customType: ADVICE_ENTRY_TYPE, content: advice, display: true });
+    deps.pi.sendMessage(
+      { customType: ADVICE_ENTRY_TYPE, content: advice, display: true },
+      { triggerTurn: false },
+    );
   };
 
   deps.pi.on("session_start", (_event, ctx) => {

package/src/context/turn-verifier.ts

@@ -31,17 +31,27 @@ export function assistantParts(message: AgentMessage): Parts | undefined {
 
 const LIST_ITEM = /^(?:\d+[.)]|[-*+])\s/;
 
+const QUESTION_END = /\?(?:\s*\([^()]{0,40}\))?[\s)"'*_`]*$/;
+/** A lead-in that asks for a choice ("Want me to:", "Which next:"), not a status summary ("Changes:"). */
+const CHOICE_LEAD_IN =
+  /\b(?:want me to|would you like|should i|shall i|which|do you want|let me know|choose|pick)\b[^\n]*:\s*$/i;
+
 /**
- * True when the message ends by asking the user something: the last non-empty line that is not an
- * option-list item ends with a question mark, optionally followed by a short parenthetical ("(y/n)").
+ * True when the message ends by asking the user something: the last non-empty line, or the last line
+ * before an option list, ends with a question mark (optionally followed by a short parenthetical
+ * like "(y/n)"), or a bare option list follows a lead-in that asks for a choice. A plain "Summary:" list is not a question.
  */
 export function asksUser(text: string): boolean {
-  const last = text
+  const lines = text
     .split("\n")
     .map((l) => l.trim())
-    .filter((l) => l !== "" && !LIST_ITEM.test(l))
-    .at(-1);
-  return last !== undefined && /\?(?:\s*\([^()]{0,40}\))?[\s)"'*_`]*$/.test(last);
+    .filter((l) => l !== "");
+  const last = lines.at(-1);
+  if (last === undefined) return false;
+  if (QUESTION_END.test(last)) return true;
+  const lead = lines.filter((l) => !LIST_ITEM.test(l)).at(-1);
+  if (lead === undefined) return false;
+  return QUESTION_END.test(lead) || (LIST_ITEM.test(last) && CHOICE_LEAD_IN.test(lead));
 }
 
 /** True when a scope.expansion departure already covers the slice: the remedy the note asks for exists. */