@henryqw/pi-prompt-creator 2.0.6 → 2.0.8

package/README.md

@@ -32,6 +32,7 @@ The prompt is created at `~/.pi/agent/prompts/<name>.md`, then Pi reloads its re
 | Surface | Type | Purpose |
 | --- | --- | --- |
 | `/promptor [analyze|dismiss|save [name]|automatic <on\|off>]` | command | For users in the interactive TUI: start or show analysis, dismiss a pending candidate, save a reviewed prompt, or control automatic analysis. |
+| `pi-prompt-creator/draft` | task | For users configuring `/task-models`: choose the child model and thinking level for drafting. |
 | Analysis status widget and candidate message | ui | Show analysis progress, readiness, or failure; a displayed candidate is marked as untrusted. |
 
 ## Flow
@@ -91,7 +92,7 @@ Package-owned: `~/.pi/agent/config/pi-prompt-creator/config.json`
 | Name | Description | Values | Default |
 | --- | --- | --- | --- |
 | `automatic` | Runs automatic analysis after enough user inputs. | Boolean. | `true` |
-| `inputThreshold` | Sets the number of non-empty user inputs before automatic analysis. | Positive integer. | `3` |
+| `inputThreshold` | Sets the number of non-empty user inputs before automatic analysis. | Positive safe integer (`1`–`9007199254740991`). | `3` |
 
 Edit either field, then run `/reload` to apply the change. Omitted fields use their defaults.
 
@@ -121,7 +122,7 @@ Candidate and Final Prompt Draft Markdown must be non-empty and at most 16 KiB.
 
 Candidate and saved Markdown reject NUL and other terminal controls. Ordinary tabs and line feeds are allowed.
 
-No candidate clears the running widget without a notice. Invalid output or child failure shows `Prompt analysis failed — /promptor`.
+No candidate clears the running widget without a notice. Invalid output or child failure shows `Prompt analysis failed — /promptor`. Failed analysis is not retried automatically; run `/promptor` or `/promptor analyze` to start again.
 
 The failure widget clears after the next non-empty user input. Analysis has a two-minute idle timeout and five-minute hard timeout.
 

package/extensions/prompt-creator.ts

@@ -146,34 +146,31 @@ function conversationItem(message: ContextMessage): ConversationItem | undefined
 }
 
 function boundedConversation(messages: ContextMessage[], maxChars: number): ConversationItem[] {
-	const items = messages.flatMap((message, index) => {
+	const items = messages.flatMap((message) => {
 		const item = conversationItem(message);
-		return item ? [{ index, item, chars: JSON.stringify(item).length }] : [];
+		return item ? [item] : [];
 	});
 	let summaryIndex = -1;
 	for (let index = items.length - 1; index >= 0; index--) {
-		if (items[index]!.item.role === "summary") {
+		if (items[index]!.role === "summary") {
 			summaryIndex = index;
 			break;
 		}
 	}
-	const selected = new Set<number>();
-	let used = 0;
-	if (summaryIndex >= 0 && items[summaryIndex]!.chars <= maxChars) {
-		selected.add(items[summaryIndex]!.index);
-		used = items[summaryIndex]!.chars;
-	}
+	const summary = items[summaryIndex];
+	const summaryChars = summary ? JSON.stringify(summary).length : 0;
+	const keepSummary = !!summary && summaryChars <= maxChars;
+	const selected: ConversationItem[] = [];
+	let used = keepSummary ? summaryChars : 0;
 	for (let index = items.length - 1; index > summaryIndex; index--) {
 		const item = items[index]!;
-		const cost = item.chars + (selected.size ? 1 : 0);
+		const cost = JSON.stringify(item).length + (used ? 1 : 0);
 		if (used + cost > maxChars) continue;
-		selected.add(item.index);
+		selected.push(item);
 		used += cost;
 	}
-	return items
-		.slice(summaryIndex < 0 ? 0 : summaryIndex)
-		.filter(({ index }) => selected.has(index))
-		.map(({ item }) => item);
+	selected.reverse();
+	return keepSummary ? [summary, ...selected] : selected;
 }
 
 function analysisPayload(pi: ExtensionAPI, ctx: ExtensionContext): AnalysisPayload {
@@ -378,10 +375,8 @@ export default function promptCreatorExtension(pi: ExtensionAPI, options: Prompt
 		draft: string,
 		requestedName: string | undefined,
 		expectedReview: ReviewBoundary,
-		expectedBranch: number,
 		ctx: ExtensionCommandContext,
 	) => {
-		if (expectedBranch !== branchGeneration || reviewBoundary !== expectedReview) return;
 		const name = requestedName ?? candidateNameHint ?? "";
 		if (!isPromptName(name)) {
 			ctx.ui.notify(`Use lowercase kebab-case starting with a letter, up to ${MAX_NAME_CHARS} characters.`, "warning");
@@ -471,7 +466,6 @@ export default function promptCreatorExtension(pi: ExtensionAPI, options: Prompt
 				ctx.ui.notify("/promptor requires the interactive TUI.", "warning");
 				return;
 			}
-			const commandBranch = branchGeneration;
 			const review = reviewBoundary;
 			const draft = latestAssistantDraft(ctx, review);
 			const [action = "", ...values] = args.trim().split(/\s+/);
@@ -502,7 +496,7 @@ export default function promptCreatorExtension(pi: ExtensionAPI, options: Prompt
 					ctx.ui.notify("No reviewed Main draft is ready to save.", "warning");
 					return;
 				}
-				await saveLatestDraft(draft, values[0], review, commandBranch, ctx);
+				await saveLatestDraft(draft, values[0], review, ctx);
 				return;
 			}
 			if (action === "automatic" && values.length === 1 && (values[0] === "on" || values[0] === "off")) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@henryqw/pi-prompt-creator",
-  "version": "2.0.6",
+  "version": "2.0.8",
   "description": "Turn current Pi conversation signals into user-approved prompt templates.",
   "keywords": [
     "pi-package",
@@ -46,7 +46,7 @@
   },
   "dependencies": {
     "@henryqw/pi-config-store": "^2.0.0",
-    "@henryqw/pi-subagent": "^22.0.0",
+    "@henryqw/pi-subagent": "^22.0.0 || ^23.0.0",
     "@henryqw/pi-task-models": "^7.0.0"
   },
   "devDependencies": {