pi-qq 0.1.8 → 0.1.10

package/README.md

@@ -1,9 +1,33 @@
 # pi-qq
 
+[![npm version](https://img.shields.io/npm/v/pi-qq.svg)](https://www.npmjs.com/package/pi-qq)
+[![npm downloads](https://img.shields.io/npm/dm/pi-qq.svg)](https://www.npmjs.com/package/pi-qq)
+[![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](./LICENSE)
+
 Ask quick side questions about your current [pi](https://pi.dev) session without polluting the main transcript.
 
+```text
+You: refactor this auth flow…
+agent: [making changes]
+You (alt+q): is there a reason we're not using the existing AuthClient?
+↳ overlay: Yes — AuthClient does X, but this path needs Y because…
+You: [keeps editing, transcript untouched]
+```
+
 `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.
 
+## Try these first
+
+```text
+/qq is this safe to merge?
+/qq why are we doing it this way and not X?
+/qq summarize what's happened so far
+/qq what's the risk in this plan?
+/qq what files have we touched this session?
+/qq --recent did the last tool call succeed?
+/qq --full what decisions have we made so far?
+```
+
 ## 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.
@@ -74,7 +98,7 @@ 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 but still bounded context, not unlimited history.
-- Large text parts are clipped, thinking blocks are removed, and images are omitted from the side-call context for speed.
+- 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,10 +1,11 @@
 {
   "name": "pi-qq",
-  "version": "0.1.8",
+  "version": "0.1.10",
   "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",
+    "pi-qq",
     "quick-question",
     "side-question",
     "context-aware",

package/qq.ts

@@ -145,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[] {