@henryqw/pi-prompt-creator 2.0.1 → 2.0.6

package/README.md

@@ -2,7 +2,7 @@
 
 Turn repeated requests or corrections in the current conversation into a reviewed global Pi prompt. A tool-free child drafts one candidate, while Main and the user control review and saving.
 
-Nothing is shown or saved until you choose it.
+Automatic analysis only prepares a candidate; the extension never saves it without your approval.
 
 ## Install
 
@@ -29,13 +29,10 @@ Run `/promptor` in the interactive TUI after a repeated request or correction ap
 
 The prompt is created at `~/.pi/agent/prompts/<name>.md`, then Pi reloads its resources. No selection menu or name prompt is required.
 
-| Command | Action |
-| --- | --- |
-| `/promptor` | Start analysis, or show a candidate prepared by automatic analysis. |
-| `/promptor analyze` | Analyze again after a candidate has been reviewed or dismissed. |
-| `/promptor save [name]` | Save Main's newest completed review reply, using the suggested name by default. |
-| `/promptor dismiss` | Forget a candidate prepared by automatic analysis. |
-| `/promptor automatic <on\|off>` | Enable or disable automatic analysis. |
+| 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. |
+| Analysis status widget and candidate message | ui | Show analysis progress, readiness, or failure; a displayed candidate is marked as untrusted. |
 
 ## Flow
 
@@ -53,7 +50,7 @@ Branch changes reset the input counter. Automatic analysis and candidate widgets
 
 Only one analysis can run at a time. A pending candidate blocks another analysis.
 
-Automatic analysis sends the current conversation to the child model configured through `/task-models`.
+Manual and automatic analysis send Pi's current model-visible session context to the child model configured through `/task-models`. Session context edits that omit or replace messages are honored; original text removed from model context is not sent to the child. Request-local `context` and `context_with_system` transformations are not included.
 
 Its payload includes the active compaction or branch summary, user text, successfully completed assistant text, and effective prompt names and descriptions.
 
@@ -65,23 +62,19 @@ New user input does not stop a running child. Branch navigation discards its old
 
 ### Review and save
 
-A manually requested analysis displays its completed candidate immediately. Automatic background analysis keeps its candidate behind the `Prompt ready — /promptor` widget until you run `/promptor` or `/promptor dismiss`, so background work never interrupts the conversation.
+A manually requested analysis displays its completed candidate immediately. Automatic analysis shows a running widget, then keeps a completed candidate behind the `Prompt ready — /promptor` widget until you run `/promptor` or `/promptor dismiss`, so background work never interrupts the conversation.
 
 Displayed candidates are visible messages marked as untrusted.
 
 Refine the candidate with Main. Ask Main to return only the complete Final Prompt Draft before saving it with `/promptor save [name]`.
 
-The save item appears only after you show a candidate and Main then completes a valid Markdown review reply.
-
-Saving uses that latest retained Main reply as the entire file. Replies before the active compaction or branch summary cannot be saved.
+Saving is allowed only after you show a candidate and Main then completes a valid Markdown review reply. Saving uses that latest retained Main reply as the entire file. Replies before the active compaction or branch summary cannot be saved.
 
 An interrupted, failed, empty, or tool-use reply cannot be saved. The extension never falls back to an older reply.
 
 A successful save ends that review. Show another candidate and complete another Main review reply before saving again.
 
-The menu asks for a lowercase kebab-case name. A candidate name appears only as a hint.
-
-Names start with a letter and contain at most 64 ASCII characters. Existing command names and prompt files are rejected.
+`/promptor save` uses the suggested candidate name. Pass `/promptor save <name>` to choose another. Names must be lowercase kebab-case, start with a letter, and contain at most 64 ASCII characters. Existing command names and prompt files are rejected.
 
 A successful save reloads Pi resources. If reload fails, the prompt remains saved and `/reload` can load it.
 
@@ -106,7 +99,7 @@ A missing file quietly uses both defaults. Startup never creates or rewrites the
 
 Malformed config or unknown keys disable automatic analysis. Pi warns once and leaves the file unchanged.
 
-Only `Automatic On` or `Automatic Off` writes the config. Toggling preserves the configured threshold.
+Only `/promptor automatic on` or `/promptor automatic off` writes the config. Toggling preserves the configured threshold.
 
 ## State and storage
 

package/extensions/prompt-creator.ts

@@ -3,6 +3,7 @@ import { tmpdir } from "node:os";
 import { dirname, join } from "node:path";
 import {
 	BorderedLoader,
+	buildSessionContext,
 	getAgentDir,
 	type ExtensionAPI,
 	type ExtensionCommandContext,
@@ -62,6 +63,7 @@ The object must have exactly these keys. name must start with a lowercase ASCII
 type Config = { automatic: boolean; inputThreshold: number };
 export type PromptCandidate = { name: string; markdown: string };
 type ConversationItem = { role: "summary" | "user" | "assistant"; text: string };
+type ContextMessage = ReturnType<typeof buildSessionContext>["messages"][number];
 type ExistingPrompt = { name: string; description: string };
 type AnalysisPayload = { currentConversation: ConversationItem[]; existingPrompts: ExistingPrompt[] };
 type ActiveRun = { controller: AbortController; branchGeneration: number };
@@ -133,19 +135,19 @@ function messageText(content: unknown): string {
 		.join("\n");
 }
 
-function conversationItem(entry: SessionEntry): ConversationItem | undefined {
-	if (entry.type === "compaction" || entry.type === "branch_summary") {
-		return entry.summary.trim() ? { role: "summary", text: entry.summary } : undefined;
+function conversationItem(message: ContextMessage): ConversationItem | undefined {
+	if (message.role === "compactionSummary" || message.role === "branchSummary") {
+		return message.summary.trim() ? { role: "summary", text: message.summary } : undefined;
 	}
-	if (entry.type !== "message" || (entry.message.role !== "user" && entry.message.role !== "assistant")) return;
-	if (entry.message.role === "assistant" && entry.message.stopReason !== "stop") return;
-	const text = messageText(entry.message.content);
-	return text.trim() ? { role: entry.message.role, text } : undefined;
+	if (message.role !== "user" && message.role !== "assistant") return;
+	if (message.role === "assistant" && message.stopReason !== "stop") return;
+	const text = messageText(message.content);
+	return text.trim() ? { role: message.role, text } : undefined;
 }
 
-function boundedConversation(entries: SessionEntry[], maxChars: number): ConversationItem[] {
-	const items = entries.flatMap((entry, index) => {
-		const item = conversationItem(entry);
+function boundedConversation(messages: ContextMessage[], maxChars: number): ConversationItem[] {
+	const items = messages.flatMap((message, index) => {
+		const item = conversationItem(message);
 		return item ? [{ index, item, chars: JSON.stringify(item).length }] : [];
 	});
 	let summaryIndex = -1;
@@ -178,7 +180,7 @@ function analysisPayload(pi: ExtensionAPI, ctx: ExtensionContext): AnalysisPaylo
 	const payload: AnalysisPayload = { currentConversation: [], existingPrompts: [] };
 	const envelopeChars = JSON.stringify(payload).length;
 	payload.currentConversation = boundedConversation(
-		ctx.sessionManager.buildContextEntries(),
+		buildSessionContext(ctx.sessionManager.getEntries(), ctx.sessionManager.getLeafId()).messages,
 		MAX_PAYLOAD_CHARS - envelopeChars,
 	);
 	let used = JSON.stringify(payload).length;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@henryqw/pi-prompt-creator",
-  "version": "2.0.1",
+  "version": "2.0.6",
   "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": "^18.0.0",
+    "@henryqw/pi-subagent": "^22.0.0",
     "@henryqw/pi-task-models": "^7.0.0"
   },
   "devDependencies": {