pi-qq 0.1.7 → 0.1.9

package/README.md

@@ -2,13 +2,14 @@
 
 Ask quick side questions about your current [pi](https://pi.dev) session without polluting the main transcript.
 
-`pi-qq` adds `/qq <question>` plus an **alt+q** / **Option+Q** shortcut that toggles `/qq ` in the editor. The answer appears ephemerally in a bottom overlay, using the current session as read-only context, and nothing is added to the main conversation.
+`pi-qq` adds `/qq <question>` plus an **alt+q** / **Option+Q** shortcut that toggles `/qq ` in the editor. Answers appear in a dismissible bottom overlay, can be reopened from in-memory `/qq-history`, and never enter the main conversation.
 
 ## Why try it?
 
 - **A real side channel:** ask `/qq why are we changing this file?` while the main agent keeps working. The answer shows in a bottom overlay and does not enter the main transcript.
 - **Context-aware, intentionally constrained:** `/qq` passes read-only main-session context, treats ambiguous references like “this”, “that”, “we”, and “the plan” as references to the active session, and gives the side call no tools. Previous `/qq` answers are available only through `/qq-history`; they are not fed back into future `/qq` calls.
-- **Fast, low-friction UX:** `/qq` uses recent context by default, automatically switches to broader context for retrospective questions, and supports explicit `--recent` / `--full` modes. Press **alt+q** / **Option+Q** to toggle `/qq `, then use **Esc** to cancel/dismiss or **↑/↓** to scroll longer answers.
+- **Fast, low-friction UX:** press **alt+q** / **Option+Q** to toggle `/qq `, then use **Esc** to cancel/dismiss or **↑/↓** to scroll longer answers.
+- **Smart context modes:** `/qq` uses recent context by default, automatically switches to broader bounded context for retrospective questions, and supports explicit `--recent` / `--full` modes.
 
 ## Demo
 
@@ -46,10 +47,11 @@ After installing, run `/reload` in pi or restart the session.
 
 By default, `/qq` chooses a context mode automatically:
 
-- **Recent mode** for immediate questions like `what did we do in the last turn?`, `what are we doing right now?`, or `why are we changing this?`.
-- **Full mode** for retrospective questions like `summarize this session`, `what did we decide so far?`, or `what did we try earlier?`.
-
-Use `--recent` when you want the fastest answer from the latest messages. Use `--full` when you explicitly want broader session context.
+| Mode | When to use it | Context sent |
+| --- | --- | --- |
+| Auto | Default for `/qq <question>` | Recent context for immediate questions, broader bounded context for retrospective questions |
+| `--recent` | Fastest answers about the latest work | Latest messages only |
+| `--full` | Recaps or questions about earlier decisions | Broader but still bounded session context, not unlimited history |
 
 Use `/qq-history` to reopen recent `/qq` answers from the current session. History is view-only and is not included as context for future `/qq` model calls.
 
@@ -71,8 +73,8 @@ Press **alt+q** / **Option+Q** to toggle `/qq ` at the front of the editor:
 
 - The main transcript is never polluted by `/qq` questions or answers.
 - The side call receives read-only main-session context.
-- Recent mode sends only the latest messages for speed; full mode sends broader bounded context.
-- Large text parts are clipped, thinking blocks are removed, and images are omitted from the side-call context for speed.
+- Recent mode sends only the latest messages for speed; full mode sends broader but still bounded context, not unlimited history.
+- Large text parts are clipped; images, tool calls, and tool results are converted into plain-text background so the side call never uses provider tool protocol.
 - The side call has no tools.
 - Recent `/qq` answers are kept in memory only so `/qq-history` can reopen them after dismissal.
 - `/qq-history` is view-only; it is not used as context for future `/qq` model calls.

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "pi-qq",
-  "version": "0.1.7",
-  "description": "Pi extension. Ask context-aware side questions with /qq or alt+q, then reopen answers with /qq-history.",
+  "version": "0.1.9",
+  "description": "Ask transcript-safe, context-aware side questions in Pi with /qq or alt+q, then reopen recent answers with /qq-history.",
   "keywords": [
     "pi-package",
     "pi-extension",

package/qq.ts

@@ -71,13 +71,15 @@ export const QQ_SYSTEM_PROMPT = readFileSync(
 ).trimEnd();
 
 function getState(): QqState {
-	const globalState = globalThis as unknown as { [k: symbol]: QqState | undefined };
+	const globalState = globalThis as unknown as { [k: symbol]: Partial<QqState> | undefined };
 	let state = globalState[QQ_STATE_KEY];
 	if (!state) {
-		state = { snapshots: new Map(), histories: new Map() };
+		state = {};
 		globalState[QQ_STATE_KEY] = state;
 	}
-	return state;
+	state.snapshots ??= new Map();
+	state.histories ??= new Map();
+	return state as QqState;
 }
 
 function getSessionFile(ctx: ExtensionContext): string {
@@ -143,33 +145,38 @@ function clipText(text: string): string {
 	return `${text.slice(0, MAX_TEXT_CHARS_PER_PART)}\n[…truncated for /qq speed…]`;
 }
 
-function trimContentArray<T extends Array<{ type: string }>>(content: T): T {
-	return content
-		.filter((part) => part.type !== "thinking")
-		.map((part) => {
-			if (part.type === "text" && "text" in part && typeof part.text === "string") {
-				return { ...part, text: clipText(part.text) };
-			}
-			if (part.type === "image") {
-				return { type: "text", text: "[image omitted for /qq speed]" };
-			}
-			return part;
-		}) as T;
+function contentPartsToText(content: Array<{ type: string }>): string {
+	const parts: string[] = [];
+	for (const part of content) {
+		if (part.type === "text" && "text" in part && typeof part.text === "string") {
+			parts.push(clipText(part.text));
+		} else if (part.type === "image") {
+			parts.push("[image omitted for /qq speed]");
+		} else if (part.type === "toolCall" && "name" in part && typeof part.name === "string") {
+			parts.push(`[assistant requested tool: ${part.name}]`);
+		}
+	}
+	return parts.join("\n").trim();
 }
 
-function trimUserContent(content: UserMessage["content"]): UserMessage["content"] {
-	if (typeof content === "string") return clipText(content);
-	return trimContentArray(content);
+function userContentToText(content: UserMessage["content"]): string {
+	return typeof content === "string" ? clipText(content) : contentPartsToText(content);
 }
 
 function trimMessageForContext(message: Message): Message {
 	if (message.role === "assistant") {
-		return { ...message, content: trimContentArray(message.content) };
+		const text = contentPartsToText(message.content) || "[assistant message omitted for /qq speed]";
+		return { ...message, content: [{ type: "text", text }] };
 	}
 	if (message.role === "user") {
-		return { ...message, content: trimUserContent(message.content) };
+		return { ...message, content: [{ type: "text", text: userContentToText(message.content) }] };
 	}
-	return { ...message, content: trimContentArray(message.content) };
+	const text = contentPartsToText(message.content) || "[tool result omitted for /qq speed]";
+	return {
+		role: "user",
+		content: [{ type: "text", text: `[tool result: ${message.toolName}]\n${text}` }],
+		timestamp: message.timestamp,
+	};
 }
 
 function selectRecentContextMessages(messages: Message[]): Message[] {