@jwilger/pi-development-system 0.36.1 → 0.37.0

package/package.json

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

package/src/context/turn-verifier.ts

@@ -29,14 +29,19 @@ export function assistantParts(message: AgentMessage): Parts | undefined {
   return { text, callsTools: message.content.some((c) => c.type === "toolCall") };
 }
 
-/** True when the last non-empty line of the message ends with a question mark. */
+const LIST_ITEM = /^(?:\d+[.)]|[-*+])\s/;
+
+/**
+ * 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)").
+ */
 export function asksUser(text: string): boolean {
   const last = text
     .split("\n")
     .map((l) => l.trim())
-    .filter((l) => l !== "")
+    .filter((l) => l !== "" && !LIST_ITEM.test(l))
     .at(-1);
-  return last !== undefined && /\?[\s)"'*_`]*$/.test(last);
+  return last !== undefined && /\?(?:\s*\([^()]{0,40}\))?[\s)"'*_`]*$/.test(last);
 }
 
 const claimMessage = (): string =>

package/src/core/test-runner.ts

@@ -86,9 +86,10 @@ function reportedExit(result: ResultLike): number {
 
 /** The last few non-empty output lines, redacted and bounded: what the runner said at the end. */
 export function summarizeOutput(output: string): string {
-  const lines = output
+  // Redact first: a multi-line secret (PEM key) must be recognised before the tail is cut off its BEGIN line.
+  const lines = redactSecrets(output)
     .split("\n")
     .map((l) => l.trim())
     .filter((l) => l !== "" && !/^Command exited with code \d+$/.test(l));
-  return redactSecrets(lines.slice(-SUMMARY_LINES).join(" | ")).slice(0, SUMMARY_MAX);
+  return lines.slice(-SUMMARY_LINES).join(" | ").slice(0, SUMMARY_MAX);
 }