@vitos-pizza/vitos-pizza 0.4.0 → 0.4.1

package/CHANGELOG.md

@@ -4,6 +4,24 @@ All notable changes to vitos-pizza are documented here.
 
 ## [Unreleased]
 
+## [0.4.1] - 2026-07-12
+
+### Added
+
+- **Multi-select questions** — `question` tool `selectType: "multi"` (Space toggle, Enter submit)
+- **Multi-question tabs** — `questions: [...]` for tabbed prompts (Tab/←→ switch; per-tab single or multi)
+
+### Changed
+
+- **Subagent hypa-only reads** — scout / planner / worker drop builtin `read`/`grep`/`find`/`ls`; exploration uses `hypa_*` only (`bash`/`edit`/`write` unchanged)
+- Plan mode footer: **Worth considering** (gaps/adjacent capabilities) replaces **Next** (no mode-switch verbs)
+- Subagent question forwarding: subprocess children use **file channel only** (skip in-process RPC)
+
+### Fixed
+
+- **Subagent question false cancel** — child no-UI RPC no longer replies `cancelled`; parent TUI receives forwarded prompts
+- **Resize recovery** — after Windows/Cursor reports `columns`/`rows` as 0 (or the border TUI is not ready yet), retry invalidate+full redraw for up to 1s so the UI recovers when size settles without another resize event
+
 ## [0.4.0] - 2026-07-11
 
 ### Added
@@ -50,7 +68,7 @@ All notable changes to vitos-pizza are documented here.
 
 - **`@vitos-pizza/todoist`** — in-memory task list with TUI widget, `/todo` command, and LLM tools (`todo_add` / `todo_update` / `todo_complete` / `todo_delete`)
 - Terminal **resize recovery** in ui-enhancements — debounce + full redraw after resize (fixes blank/corrupt TUI on Windows)
-- Plan-mode **Next** footer guidance and optional scout (skip recon when context is enough; parallel scouts for independent areas)
+- Plan-mode **Worth considering** footer (gaps/adjacent capabilities; not next-steps) and optional scout (skip recon when context is enough; parallel scouts for independent areas)
 
 ### Changed
 
@@ -70,15 +88,13 @@ All notable changes to vitos-pizza are documented here.
 - Pi distribution layout: repo root as single install entry
 - `scripts/sync-pi-manifest.mjs` module assembler
 - Original logo at `assets/logo.svg`
+- Built-in modules: agent-mode, question, session-title, permission-system, subagents, keybindings, ui-enhancements, websearch
+- Agent modes: `agent` / `plan` / `execute` with `/mode` and cycle shortcuts
+- Structured `question` tool with subagent parent-session forwarding
+- Session auto-title from first meaningful user message
 
-### Removed
-
-- Nested `packages/xin-pi` aggregator
-- `@xin-pi/core`, template scaffold, postinstall symlink hack
-
-### Publishing
+### Acknowledgments
 
-- GitHub Actions publishes `@vitos-pizza/vitos-pizza` via **npm Trusted Publishing (OIDC)** — no long-lived `NPM_TOKEN`
-- Workflow: `.github/workflows/publish.yml` (triggers on `v*` tags / `workflow_dispatch`)
-- `scripts/prepare-publish.mjs` rewrites `file:` workspace deps for the npm tarball
-- First package version must be published once manually (or as a stub) before Trusted Publisher can be configured on npmjs.com
+- [Pi](https://pi.dev/) / [@earendil-works/pi-coding-agent](https://www.npmjs.com/package/@earendil-works/pi-coding-agent)
+- [@gotgenes/pi-permission-system](https://www.npmjs.com/package/@gotgenes/pi-permission-system)
+- Pi official `question` and subagent examples

package/README.md

@@ -16,7 +16,7 @@ Inspired by [Vito's Pizzeria](https://garfield.fandom.com/wiki/Vito_Cappelletti)
 
 ![vitos-pizza architecture](./assets/architecture.png)
 
-Typical flow: **plan** → optional **scout** → **planner** → confirm → **execute** / **worker**. Shell output is compressed via Hypa bash rewrite; prefer `hypa_*` read tools when exploring.
+Typical flow: **plan** → optional **scout** → **planner** → confirm → **execute** / **worker**. Shell output is compressed via Hypa bash rewrite; scout/planner/worker explore with `hypa_*` only (main session may still use builtin reads).
 
 | Capability | Module | What you get |
 |------------|--------|--------------|

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vitos-pizza/vitos-pizza",
-	"version": "0.4.0",
+	"version": "0.4.1",
 	"description": "Vito's Pizzeria — a Pi distribution (Garfield-approved)",
 	"publishConfig": {
 		"access": "public",
@@ -39,17 +39,17 @@
 	"dependencies": {
 		"@hypabolic/pi-hypa": "^0.1.10",
 		"typebox": "^1.1.38",
-		"@vitos-pizza/agent-mode": "0.4.0",
-		"@vitos-pizza/hypa": "0.4.0",
-		"@vitos-pizza/keybindings": "0.4.0",
-		"@vitos-pizza/permission-system": "0.4.0",
-		"@vitos-pizza/question": "0.4.0",
-		"@vitos-pizza/session-title": "0.4.0",
-		"@vitos-pizza/settings-preset": "0.4.0",
-		"@vitos-pizza/subagents": "0.4.0",
-		"@vitos-pizza/todoist": "0.4.0",
-		"@vitos-pizza/ui-enhancements": "0.4.0",
-		"@vitos-pizza/websearch": "0.4.0"
+		"@vitos-pizza/agent-mode": "0.4.1",
+		"@vitos-pizza/hypa": "0.4.1",
+		"@vitos-pizza/keybindings": "0.4.1",
+		"@vitos-pizza/permission-system": "0.4.1",
+		"@vitos-pizza/question": "0.4.1",
+		"@vitos-pizza/session-title": "0.4.1",
+		"@vitos-pizza/settings-preset": "0.4.1",
+		"@vitos-pizza/subagents": "0.4.1",
+		"@vitos-pizza/todoist": "0.4.1",
+		"@vitos-pizza/ui-enhancements": "0.4.1",
+		"@vitos-pizza/websearch": "0.4.1"
 	},
 	"bundledDependencies": [
 		"@hypabolic/pi-hypa",

package/packages/agent-mode/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vitos-pizza/agent-mode",
-	"version": "0.4.0",
+	"version": "0.4.1",
 	"description": "Centralized agent / plan / execute mode switching for Vito's Pizzeria",
 	"type": "module",
 	"license": "MIT",
@@ -13,8 +13,8 @@
 		"lint:fix": "biome check --write ."
 	},
 	"dependencies": {
-		"@vitos-pizza/keybindings": "0.4.0",
-		"@vitos-pizza/permission-system": "0.4.0"
+		"@vitos-pizza/keybindings": "0.4.1",
+		"@vitos-pizza/permission-system": "0.4.1"
 	},
 	"peerDependencies": {
 		"@earendil-works/pi-coding-agent": "*"

package/packages/agent-mode/src/plan-instructions.ts

@@ -28,26 +28,33 @@ This supersedes conflicting skill or prior guidance about implementing code or c
 
 **Exploration**
 - Scout is optional — only for unfamiliar or multi-file recon. Parallel scouts via \`subagent({ tasks: [...] })\` for independent areas, then planner.
-- Prefer \`hypa_read\` / \`hypa_grep\` / \`hypa_find\` / \`hypa_ls\` when available; fall back to read/grep/find/ls.
+- Scout/planner use \`hypa_read\` / \`hypa_grep\` / \`hypa_find\` / \`hypa_ls\` only. Main session: prefer hypa_*; fall back to read/grep/find/ls if needed.
 - Web tools only for current external facts outside the repo.
 
 **Delegation**
 - Enough context → \`subagent({ agent: "planner", task })\`
 - Need recon → \`subagent({ chain: [scout, planner] })\`
 
-**Optional Next footer** (2–3 one-line reply verbs) when the next step is not obvious:
+**Optional open-questions footer** (2–4 one-line prompts) only when the plan may miss scope or adjacent capabilities the user has not clarified. Soft guidance — not next-steps, not mode-switching:
 
 \`\`\`
-**Next**
-- Confirm → \`/mode execute\` (or Ctrl+. / Alt+M)
-- Say what to change if scope is off
-\`\`\``;
+**Worth considering**
+- <gap or adjacent capability the user may not have decided yet>?
+- <in/out of scope this round>?
+\`\`\`
+
+Rules:
+- Ask about omissions, edge cases, related features, or "do you also need X?"
+- Do NOT list mode-switching or how-to-proceed verbs (e.g. \`/mode execute\`, Ctrl+.)
+- Do NOT restate the plan as todos
+- Prefer \`question\` for mutually exclusive choices; use this footer for soft, optional prompts
+- Skip the footer when scope is already clear`;
 
 export const PLAN_MODE_MESSAGE = `[PLAN MODE ACTIVE]
 
 Read-only planning. No write/edit/bash/worker.
 Clarify with \`question\` when needed. Scout only if recon is required; otherwise planner.
-Prefer hypa_* read tools when available. Return the plan and wait; optionally a short **Next** footer.`;
+Prefer hypa_* read tools when available. Return the plan and wait; optionally a short **Worth considering** footer for gaps/adjacent capabilities.`;
 
 export const PLAN_MODE_ENDED_MESSAGE = `[PLAN MODE ENDED]
 

package/packages/hypa/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vitos-pizza/hypa",
-	"version": "0.4.0",
+	"version": "0.4.1",
 	"description": "Vito's Pizzeria Hypa defaults — additive mode, MCP proxy off, config seed",
 	"type": "module",
 	"license": "MIT",

package/packages/keybindings/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vitos-pizza/keybindings",
-	"version": "0.4.0",
+	"version": "0.4.1",
 	"description": "Centralized keyboard shortcut registry for Vito's Pizzeria Pi distribution",
 	"type": "module",
 	"license": "MIT",

package/packages/permission-system/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vitos-pizza/permission-system",
-	"version": "0.4.0",
+	"version": "0.4.1",
 	"description": "Permission enforcement for Vito's Pizzeria Pi distribution",
 	"type": "module",
 	"license": "MIT",
@@ -18,7 +18,7 @@
 		"lint:fix": "biome check --write ."
 	},
 	"dependencies": {
-		"@vitos-pizza/keybindings": "0.4.0",
+		"@vitos-pizza/keybindings": "0.4.1",
 		"tree-sitter-bash": "^0.25.1",
 		"web-tree-sitter": "^0.25.10",
 		"zod": "^3.25.76"

package/packages/question/extensions/index.ts

@@ -1,15 +1,18 @@
 /**
  * @vitos-pizza/question — structured ask-user question tool.
+ * Supports single-question (legacy) and multi-question tabbed modes.
  */
 
 import type { ExtensionContext } from "@earendil-works/pi-coding-agent";
 import { getAgentDir } from "@earendil-works/pi-coding-agent";
 import { startQuestionFileWatcher } from "../src/forwarding/file-watcher.ts";
 import { startQuestionRpcServer } from "../src/forwarding/forwarder.ts";
+import type { ForwardQuestionPayload } from "../src/forwarding/forwarder.ts";
 import {
 	promptQuestionLocally,
 	registerQuestionTool,
 } from "../src/register-tool.ts";
+import type { MultiQuestionParams, QuestionParams } from "../src/types.ts";
 
 export default function (
 	pi: import("@earendil-works/pi-coding-agent").ExtensionAPI,
@@ -18,24 +21,42 @@ export default function (
 	let stopFileWatcher: (() => void) | null = null;
 	const agentDir = getAgentDir();
 
-	const unsubRpc = startQuestionRpcServer(pi.events, async (payload) => {
-		const ctx = activeCtx;
-		if (!ctx?.hasUI) {
-			return {
-				question: payload.question,
-				options: payload.options.map((o) => o.label),
-				answer: null,
-				cancelled: true,
-				responderSessionId: ctx?.sessionManager.getSessionId?.() ?? "unknown",
-				respondedAt: Date.now(),
-			};
-		}
-		return promptQuestionLocally(
-			ctx.ui,
-			{ question: payload.question, options: payload.options },
-			ctx.sessionManager.getSessionId?.() ?? "unknown",
-		);
-	});
+	const unsubRpc = startQuestionRpcServer(
+		pi.events,
+		async (payload: ForwardQuestionPayload) => {
+			const ctx = activeCtx;
+			// No UI or wrong session — stay silent so callers can fall through
+			// to file-based forwarding (do not fake cancelled).
+			if (!ctx?.hasUI) return null;
+
+			const sessionId =
+				ctx.sessionManager.getSessionId?.() ?? "unknown";
+			if (
+				payload.targetSessionId &&
+				payload.targetSessionId !== sessionId
+			) {
+				return null;
+			}
+
+			if (payload.questions) {
+				return promptQuestionLocally(
+					ctx.ui,
+					{ questions: payload.questions } as MultiQuestionParams,
+					sessionId,
+				);
+			}
+
+			return promptQuestionLocally(
+				ctx.ui,
+				{
+					question: payload.question ?? "",
+					options: payload.options ?? [],
+					selectType: payload.selectType,
+				} as QuestionParams,
+				sessionId,
+			);
+		},
+	);
 
 	registerQuestionTool(pi, {
 		getAgentDir: () => agentDir,
@@ -54,16 +75,28 @@ export default function (
 				responderSessionId: sessionId,
 				showQuestion: async (params) => {
 					if (!activeCtx?.hasUI) {
+						if ("questions" in params) {
+							return {
+								answers: {},
+								cancelled: true,
+								responderSessionId: sessionId,
+								respondedAt: Date.now(),
+							};
+						}
 						return {
-							question: params.question,
-							options: params.options.map((o) => o.label),
+							question: (params as QuestionParams).question,
+							options: (params as QuestionParams).options.map((o) => o.label),
 							answer: null,
 							cancelled: true,
 							responderSessionId: sessionId,
 							respondedAt: Date.now(),
 						};
 					}
-					return promptQuestionLocally(activeCtx.ui, params, sessionId);
+					return promptQuestionLocally(
+						activeCtx.ui,
+						params,
+						sessionId,
+					);
 				},
 			});
 		}

package/packages/question/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vitos-pizza/question",
-	"version": "0.4.0",
+	"version": "0.4.1",
 	"description": "Structured ask-user question tool for Vito's Pizzeria Pi distribution",
 	"type": "module",
 	"license": "MIT",

package/packages/question/src/forwarding/file-watcher.ts

@@ -11,6 +11,7 @@ import { join } from "node:path";
 import type {
 	ForwardedQuestionRequest,
 	ForwardedQuestionResponse,
+	MultiQuestionParams,
 	QuestionParams,
 } from "../types.ts";
 import { getQuestionForwardingRoot } from "./forwarder.ts";
@@ -21,7 +22,9 @@ export function startQuestionFileWatcher(options: {
 	agentDir: string;
 	sessionId: string;
 	responderSessionId: string;
-	showQuestion: (params: QuestionParams) => Promise<ForwardedQuestionResponse>;
+	showQuestion: (
+		params: QuestionParams | MultiQuestionParams,
+	) => Promise<ForwardedQuestionResponse>;
 	pollIntervalMs?: number;
 }): () => void {
 	const forwardingRoot = getQuestionForwardingRoot(
@@ -62,16 +65,43 @@ export function startQuestionFileWatcher(options: {
 				readFileSync(processingPath, "utf8"),
 			) as ForwardedQuestionRequest;
 
-			const response = await options.showQuestion({
-				question: request.question,
-				options: request.options,
-			});
-
-			writeFileSync(
-				join(responsesDir, `${requestId}.json`),
-				JSON.stringify(response),
-				"utf8",
-			);
+			// Determine single vs multi
+			if (request.questions && request.questions.length > 0) {
+				const response = await options.showQuestion({
+					questions: request.questions,
+				});
+				writeFileSync(
+					join(responsesDir, `${requestId}.json`),
+					JSON.stringify(response),
+					"utf8",
+				);
+			} else if (request.question !== undefined && request.options) {
+				const response = await options.showQuestion({
+					question: request.question,
+					options: request.options,
+					selectType: request.selectType,
+				});
+				writeFileSync(
+					join(responsesDir, `${requestId}.json`),
+					JSON.stringify(response),
+					"utf8",
+				);
+			} else {
+				// Unknown format — write failure
+				const failure: ForwardedQuestionResponse = {
+					question: "",
+					options: [],
+					answer: null,
+					cancelled: true,
+					responderSessionId: options.responderSessionId,
+					respondedAt: Date.now(),
+				};
+				writeFileSync(
+					join(responsesDir, `${requestId}.json`),
+					JSON.stringify(failure),
+					"utf8",
+				);
+			}
 		} catch {
 			const failure: ForwardedQuestionResponse = {
 				question: "",

package/packages/question/src/forwarding/forwarder.ts

@@ -7,11 +7,15 @@ import {
 	writeFileSync,
 } from "node:fs";
 import { join } from "node:path";
+import { isSubagentChild } from "../parent-session.ts";
 import type {
 	ForwardedQuestionRequest,
 	ForwardedQuestionResponse,
+	MultiQuestionParams,
 	QuestionOption,
 	QuestionParams,
+	QuestionTab,
+	SelectType,
 } from "../types.ts";
 import { QUESTION_RPC_PROMPT, questionReplyChannel } from "./channels.ts";
 
@@ -20,6 +24,22 @@ interface EventBus {
 	on(channel: string, handler: (payload: unknown) => void): () => void;
 }
 
+/**
+ * Payload sent over RPC / file for a forwarded question prompt.
+ * Supports both single- and multi-question modes.
+ */
+export interface ForwardQuestionPayload {
+	requestId: string;
+	targetSessionId: string;
+	requesterSessionId: string;
+	/** Single‑question mode */
+	question?: string;
+	options?: QuestionOption[];
+	selectType?: SelectType;
+	/** Multi‑question mode */
+	questions?: QuestionTab[];
+}
+
 export function getQuestionForwardingRoot(
 	agentDir: string,
 	sessionId: string,
@@ -43,23 +63,37 @@ export async function forwardQuestionPrompt(options: {
 	agentDir?: string;
 	targetSessionId: string;
 	requesterSessionId: string;
-	params: QuestionParams;
+	params: QuestionParams | MultiQuestionParams;
 	timeoutMs?: number;
 }): Promise<ForwardedQuestionResponse | null> {
 	const requestId = randomUUID();
-	const payload = {
+
+	const isMulti = "questions" in options.params;
+	const payload: ForwardQuestionPayload = {
 		requestId,
 		targetSessionId: options.targetSessionId,
 		requesterSessionId: options.requesterSessionId,
-		question: options.params.question,
-		options: options.params.options,
 	};
 
-	if (options.events) {
+	if (isMulti) {
+		const mp = options.params as MultiQuestionParams;
+		payload.questions = mp.questions;
+	} else {
+		const sp = options.params as QuestionParams;
+		payload.question = sp.question;
+		payload.options = sp.options;
+		payload.selectType = sp.selectType;
+	}
+
+	// Subprocess children have an isolated event bus; their own no-UI RPC
+	// server would poison-reply with cancelled. Skip RPC and use files only.
+	if (options.events && !isSubagentChild()) {
+		// Same-process RPC. If no capable listener replies, fall through to
+		// file-based forwarding quickly instead of waiting the full timeout.
 		const rpcReply = await waitForRpcReply(
 			options.events,
 			payload,
-			options.timeoutMs ?? 600_000,
+			1500, // short RPC-specific timeout
 		);
 		if (rpcReply) return rpcReply;
 	}
@@ -74,7 +108,7 @@ export async function forwardQuestionPrompt(options: {
 				targetSessionId: options.targetSessionId,
 				params: options.params,
 			},
-			options.timeoutMs ?? 600_000,
+			options.timeoutMs ?? 300_000, // 5 min for user response
 		);
 	}
 
@@ -83,13 +117,7 @@ export async function forwardQuestionPrompt(options: {
 
 function waitForRpcReply(
 	events: EventBus,
-	payload: {
-		requestId: string;
-		targetSessionId: string;
-		requesterSessionId: string;
-		question: string;
-		options: QuestionOption[];
-	},
+	payload: ForwardQuestionPayload,
 	timeoutMs: number,
 ): Promise<ForwardedQuestionResponse | null> {
 	return new Promise((resolve) => {
@@ -121,20 +149,31 @@ function waitForFileReply(
 		requestId: string;
 		requesterSessionId: string;
 		targetSessionId: string;
-		params: QuestionParams;
+		params: QuestionParams | MultiQuestionParams;
 	},
 	timeoutMs: number,
 ): Promise<ForwardedQuestionResponse | null> {
 	const forwardingRoot = getQuestionForwardingRoot(agentDir, targetSessionId);
 	const { requestsDir, responsesDir } = getForwardingDirs(forwardingRoot);
+
+	const isMulti = "questions" in payload.params;
 	const request: ForwardedQuestionRequest = {
 		id: payload.requestId,
 		createdAt: Date.now(),
 		requesterSessionId: payload.requesterSessionId,
 		targetSessionId: payload.targetSessionId,
-		question: payload.params.question,
-		options: payload.params.options,
 	};
+
+	if (isMulti) {
+		const mp = payload.params as MultiQuestionParams;
+		request.questions = mp.questions;
+	} else {
+		const sp = payload.params as QuestionParams;
+		request.question = sp.question;
+		request.options = sp.options;
+		request.selectType = sp.selectType;
+	}
+
 	writeFileSync(
 		join(requestsDir, `${request.id}.json`),
 		JSON.stringify(request),
@@ -169,19 +208,14 @@ function waitForFileReply(
 
 export function startQuestionRpcServer(
 	events: EventBus,
-	handler: (payload: {
-		requestId: string;
-		question: string;
-		options: QuestionOption[];
-	}) => Promise<ForwardedQuestionResponse>,
+	handler: (
+		payload: ForwardQuestionPayload,
+	) => Promise<ForwardedQuestionResponse | null | undefined>,
 ): () => void {
 	return events.on(QUESTION_RPC_PROMPT, async (raw) => {
-		const payload = raw as {
-			requestId: string;
-			question: string;
-			options: QuestionOption[];
-		};
+		const payload = raw as ForwardQuestionPayload;
 		const response = await handler(payload);
+		if (response == null) return;
 		events.emit(questionReplyChannel(payload.requestId), response);
 	});
 }