@vitos-pizza/vitos-pizza 0.1.0 → 0.2.0

package/node_modules/@vitos-pizza/agent-mode/src/modes.ts

@@ -1,43 +1,43 @@
-import type { PermissionPresetName } from "@vitos-pizza/permission-system/mode-api";
-import {
-	PRESETS_DIR,
-} from "@vitos-pizza/permission-system/mode-api";
-import { detectPresetMode } from "@vitos-pizza/permission-system/detect-preset-mode";
-import type { ExtensionConfig } from "@vitos-pizza/permission-system/types";
-
-export type AgentMode = "agent" | "plan" | "execute";
-
-export const AGENT_MODES: readonly AgentMode[] = [
-	"agent",
-	"plan",
-	"execute",
-] as const;
-
-export const PRESET_BY_MODE: Record<AgentMode, PermissionPresetName> = {
-	agent: "default",
-	plan: "plan",
-	execute: "yolo",
-};
-
-export const MODE_LABELS: Record<AgentMode, string> = {
-	agent: "Agent mode — balanced permissions",
-	plan: "Plan mode — read-only exploration",
-	execute: "Execute mode — minimal gates",
-};
-
-export function isAgentMode(value: string): value is AgentMode {
-	return (AGENT_MODES as readonly string[]).includes(value);
-}
-
-export function cycleAgentMode(current: AgentMode): AgentMode {
-	const index = AGENT_MODES.indexOf(current);
-	return AGENT_MODES[(index + 1) % AGENT_MODES.length];
-}
-
-export function inferAgentMode(config: ExtensionConfig): AgentMode {
-	if (config.agentMode) return config.agentMode;
-	const detected = detectPresetMode(config.permission, PRESETS_DIR);
-	if (detected === "plan") return "plan";
-	if (detected === "yolo") return "execute";
-	return "agent";
-}
+import type { PermissionPresetName } from "@vitos-pizza/permission-system/mode-api";
+import {
+	PRESETS_DIR,
+} from "@vitos-pizza/permission-system/mode-api";
+import { detectPresetMode } from "@vitos-pizza/permission-system/detect-preset-mode";
+import type { ExtensionConfig } from "@vitos-pizza/permission-system/types";
+
+export type AgentMode = "agent" | "plan" | "execute";
+
+export const AGENT_MODES: readonly AgentMode[] = [
+	"agent",
+	"plan",
+	"execute",
+] as const;
+
+export const PRESET_BY_MODE: Record<AgentMode, PermissionPresetName> = {
+	agent: "default",
+	plan: "plan",
+	execute: "yolo",
+};
+
+export const MODE_LABELS: Record<AgentMode, string> = {
+	agent: "Agent mode — balanced permissions",
+	plan: "Plan mode — read-only exploration",
+	execute: "Execute mode — minimal gates",
+};
+
+export function isAgentMode(value: string): value is AgentMode {
+	return (AGENT_MODES as readonly string[]).includes(value);
+}
+
+export function cycleAgentMode(current: AgentMode): AgentMode {
+	const index = AGENT_MODES.indexOf(current);
+	return AGENT_MODES[(index + 1) % AGENT_MODES.length];
+}
+
+export function inferAgentMode(config: ExtensionConfig): AgentMode {
+	if (config.agentMode) return config.agentMode;
+	const detected = detectPresetMode(config.permission, PRESETS_DIR);
+	if (detected === "plan") return "plan";
+	if (detected === "yolo") return "execute";
+	return "agent";
+}

package/node_modules/@vitos-pizza/agent-mode/src/plan-instructions.ts

@@ -1,44 +1,85 @@
-export const PLAN_MODE_TOOLS = [
-	"read",
-	"grep",
-	"find",
-	"ls",
-	"question",
-	"subagent",
-	"wait",
-	"web_search",
-	"web_read",
-] as const;
-
-export const PLAN_INSTRUCTIONS = `## PLAN MODE
-
-You are in **plan** mode. Do not call write, edit, or bash on the main agent.
-
-**Planning workflow**
-- Default path: \`subagent({ chain: [{ agent: "scout", task: "..." }, { agent: "planner", task: "..." }] })\`
-- Use the \`question\` tool when requirements are unclear
-- Produce an Implementation Plan and wait for user confirmation
-- Do **not** call worker or edit code directly in this mode
-
-**To implement**, the user must switch to execute or agent mode (\`/mode execute\` or \`/mode agent\`).`;
-
-export const PLAN_MODE_MESSAGE = `[PLAN MODE ACTIVE]
-
-You are in plan mode. The main agent must NOT write, edit, or run bash.
-
-For any task that needs codebase exploration or an implementation plan, you MUST delegate to subagents:
-
-1. Call \`subagent\` with a scout → planner chain (do not only use read/grep yourself)
-2. Use \`question\` when requirements are unclear
-3. Return the planner's output and wait for user confirmation
-4. Do NOT call worker or make code changes
-
-Example:
-\`\`\`json
-{
-  "chain": [
-    { "agent": "scout", "task": "Map relevant code for this task" },
-    { "agent": "planner", "task": "Create an implementation plan from {previous}" }
-  ]
-}
-\`\`\``;
+export const PLAN_MODE_TOOLS = [
+	"read",
+	"grep",
+	"find",
+	"ls",
+	"question",
+	"subagent",
+	"wait",
+	"web_search",
+	"web_read",
+] as const;
+
+export const PLAN_INSTRUCTIONS = `## PLAN MODE
+
+You are in **plan** mode. Do not call write, edit, or bash on the main agent.
+
+**Planning workflow**
+- Mutually exclusive design choices (2–5 options) → always use the \`question\` tool; do not ask in free text
+- Produce an Implementation Plan and wait for user confirmation
+- Do **not** call worker or edit code directly in this mode
+
+**When to explore**
+- Scout is **optional**. Skip it when the task is clear, scoped to known files, or the user already provided enough context
+- Call scout only when you need codebase recon (unfamiliar area, many files, unclear entry points)
+- Independent areas → parallel scouts via \`subagent({ tasks: [...] })\` to finish recon faster; then planner
+- For light lookups, the main agent may use read/grep/find/ls directly — no scout required
+
+**Delegation**
+- Enough context already → \`subagent({ agent: "planner", task: "..." })\`
+- Need recon first → \`subagent({ chain: [{ agent: "scout", task: "..." }, { agent: "planner", task: "..." }] })\`
+- Multi-area recon → parallel scouts, then planner with the combined findings
+
+**After the plan (optional)**
+When helpful, add a short **Next** footer — Cursor-style suggested actions. Skip it if the next step is already obvious.
+- 2–3 bullets max; each one line
+- Concrete verbs the user can reply with (e.g. confirm → \`/mode execute\`, tweak scope)
+- No essays, no restating the whole plan
+
+Example:
+\`\`\`
+**Next**
+- Confirm → \`/mode execute\` (or F6) to implement
+- Say what to change if the scope is off
+\`\`\`
+
+**To implement**, the user must switch to execute or agent mode (\`/mode execute\` or \`/mode agent\`).`;
+
+export const PLAN_MODE_MESSAGE = `[PLAN MODE ACTIVE]
+
+You are in plan mode. The main agent must NOT write, edit, or run bash.
+
+1. Mutually exclusive choices → \`question\` tool (not free-text "which do you prefer?")
+2. Scout only if needed; parallel scouts (\`tasks: [...]\`) when exploring independent areas; otherwise go straight to planner
+3. Return the plan and wait; optionally a short **Next** footer (2–3 one-line actions) when helpful
+4. Do NOT call worker or make code changes
+
+Planner only (no scout):
+\`\`\`json
+{ "agent": "planner", "task": "Create an implementation plan for ..." }
+\`\`\`
+
+Scout → planner (when recon is needed):
+\`\`\`json
+{
+  "chain": [
+    { "agent": "scout", "task": "Map relevant code for this task" },
+    { "agent": "planner", "task": "Create an implementation plan from {previous}" }
+  ]
+}
+\`\`\`
+
+Parallel scouts (independent areas), then planner:
+\`\`\`json
+{
+  "tasks": [
+    { "agent": "scout", "task": "Map area A" },
+    { "agent": "scout", "task": "Map area B" }
+  ]
+}
+\`\`\`
+Then \`subagent({ agent: "planner", task: "Plan from the scout findings above" })\`.`;
+
+export const PLAN_MODE_ENDED_MESSAGE = `[PLAN MODE ENDED]
+
+You left plan mode. write, edit, and bash are available again according to the current mode (agent or execute). Do not treat earlier [PLAN MODE ACTIVE] messages as still in effect.`;

package/node_modules/@vitos-pizza/agent-mode/tests/apply-mode.test.ts

@@ -1,36 +1,36 @@
-import { mkdtempSync, readFileSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { describe, expect, it } from "vitest";
-import { loadProjectConfig } from "@vitos-pizza/permission-system/mode-api";
-import { applyAgentMode } from "../src/apply-mode.ts";
-
-describe("apply-mode", () => {
-	it("persists agentMode and matching preset", () => {
-		const cwd = mkdtempSync(join(tmpdir(), "agent-mode-"));
-		applyAgentMode(cwd, "plan");
-		const config = loadProjectConfig(cwd);
-		expect(config.agentMode).toBe("plan");
-		expect(config.permission?.write).toBe("deny");
-		expect(config.yoloMode).toBe(false);
-	});
-
-	it("writes execute mode with yolo preset permissions", () => {
-		const cwd = mkdtempSync(join(tmpdir(), "agent-mode-"));
-		applyAgentMode(cwd, "execute");
-		const config = loadProjectConfig(cwd);
-		expect(config.agentMode).toBe("execute");
-		expect(config.permission?.bash).toEqual({
-			"*": "allow",
-			"rm -rf *": "deny",
-		});
-
-		const raw = JSON.parse(
-			readFileSync(
-				join(cwd, ".pi", "extensions", "pi-permission-system", "config.json"),
-				"utf8",
-			),
-		) as { agentMode?: string };
-		expect(raw.agentMode).toBe("execute");
-	});
-});
+import { mkdtempSync, readFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { describe, expect, it } from "vitest";
+import { loadProjectConfig } from "@vitos-pizza/permission-system/mode-api";
+import { applyAgentMode } from "../src/apply-mode.ts";
+
+describe("apply-mode", () => {
+	it("persists agentMode and matching preset", () => {
+		const cwd = mkdtempSync(join(tmpdir(), "agent-mode-"));
+		applyAgentMode(cwd, "plan");
+		const config = loadProjectConfig(cwd);
+		expect(config.agentMode).toBe("plan");
+		expect(config.permission?.write).toBe("deny");
+		expect(config.yoloMode).toBe(false);
+	});
+
+	it("writes execute mode with yolo preset permissions", () => {
+		const cwd = mkdtempSync(join(tmpdir(), "agent-mode-"));
+		applyAgentMode(cwd, "execute");
+		const config = loadProjectConfig(cwd);
+		expect(config.agentMode).toBe("execute");
+		expect(config.permission?.bash).toEqual({
+			"*": "allow",
+			"rm -rf *": "deny",
+		});
+
+		const raw = JSON.parse(
+			readFileSync(
+				join(cwd, ".pi", "extensions", "pi-permission-system", "config.json"),
+				"utf8",
+			),
+		) as { agentMode?: string };
+		expect(raw.agentMode).toBe("execute");
+	});
+});

package/node_modules/@vitos-pizza/agent-mode/tests/mode-events.test.ts

@@ -1,22 +1,22 @@
-import { describe, expect, it } from "vitest";
-import {
-	formatAgentModeStatusText,
-	isAgentModeChangedPayload,
-} from "../src/mode-events.ts";
-
-describe("mode-events", () => {
-	it("formats status text", () => {
-		expect(formatAgentModeStatusText("plan")).toBe("mode: plan");
-	});
-
-	it("validates changed payloads", () => {
-		expect(
-			isAgentModeChangedPayload({
-				mode: "execute",
-				label: "execute",
-				color: "warning",
-			}),
-		).toBe(true);
-		expect(isAgentModeChangedPayload({ mode: "bad" })).toBe(false);
-	});
-});
+import { describe, expect, it } from "vitest";
+import {
+	formatAgentModeStatusText,
+	isAgentModeChangedPayload,
+} from "../src/mode-events.ts";
+
+describe("mode-events", () => {
+	it("formats status text", () => {
+		expect(formatAgentModeStatusText("plan")).toBe("mode: plan");
+	});
+
+	it("validates changed payloads", () => {
+		expect(
+			isAgentModeChangedPayload({
+				mode: "execute",
+				label: "execute",
+				color: "warning",
+			}),
+		).toBe(true);
+		expect(isAgentModeChangedPayload({ mode: "bad" })).toBe(false);
+	});
+});

package/node_modules/@vitos-pizza/agent-mode/tests/modes.test.ts

@@ -1,38 +1,38 @@
-import { describe, expect, it } from "vitest";
-import {
-	AGENT_MODES,
-	cycleAgentMode,
-	inferAgentMode,
-	isAgentMode,
-	PRESET_BY_MODE,
-} from "../src/modes.ts";
-
-describe("modes", () => {
-	it("cycles agent → plan → execute → agent", () => {
-		expect(cycleAgentMode("agent")).toBe("plan");
-		expect(cycleAgentMode("plan")).toBe("execute");
-		expect(cycleAgentMode("execute")).toBe("agent");
-	});
-
-	it("maps modes to permission presets", () => {
-		expect(PRESET_BY_MODE.agent).toBe("default");
-		expect(PRESET_BY_MODE.plan).toBe("plan");
-		expect(PRESET_BY_MODE.execute).toBe("yolo");
-	});
-
-	it("validates agent mode strings", () => {
-		for (const mode of AGENT_MODES) {
-			expect(isAgentMode(mode)).toBe(true);
-		}
-		expect(isAgentMode("careful")).toBe(false);
-	});
-
-	it("prefers persisted agentMode over preset detection", () => {
-		expect(
-			inferAgentMode({
-				agentMode: "agent",
-				permission: { "*": "deny" },
-			}),
-		).toBe("agent");
-	});
-});
+import { describe, expect, it } from "vitest";
+import {
+	AGENT_MODES,
+	cycleAgentMode,
+	inferAgentMode,
+	isAgentMode,
+	PRESET_BY_MODE,
+} from "../src/modes.ts";
+
+describe("modes", () => {
+	it("cycles agent → plan → execute → agent", () => {
+		expect(cycleAgentMode("agent")).toBe("plan");
+		expect(cycleAgentMode("plan")).toBe("execute");
+		expect(cycleAgentMode("execute")).toBe("agent");
+	});
+
+	it("maps modes to permission presets", () => {
+		expect(PRESET_BY_MODE.agent).toBe("default");
+		expect(PRESET_BY_MODE.plan).toBe("plan");
+		expect(PRESET_BY_MODE.execute).toBe("yolo");
+	});
+
+	it("validates agent mode strings", () => {
+		for (const mode of AGENT_MODES) {
+			expect(isAgentMode(mode)).toBe(true);
+		}
+		expect(isAgentMode("careful")).toBe(false);
+	});
+
+	it("prefers persisted agentMode over preset detection", () => {
+		expect(
+			inferAgentMode({
+				agentMode: "agent",
+				permission: { "*": "deny" },
+			}),
+		).toBe("agent");
+	});
+});

package/node_modules/@vitos-pizza/agent-mode/tests/plan-instructions.test.ts

@@ -1,45 +1,68 @@
-import { readFileSync } from "node:fs";
-import { dirname, join } from "node:path";
-import { fileURLToPath } from "node:url";
-import { describe, expect, it } from "vitest";
-import { PermissionEvaluator } from "../../permission-system/src/permission-evaluator.ts";
-import { sanitizeSkillPromptBlocks } from "../../permission-system/src/sanitizers/skill-prompt.ts";
-import { PLAN_MODE_TOOLS } from "../src/plan-instructions.ts";
-
-const presetsDir = join(
-	dirname(fileURLToPath(import.meta.url)),
-	"../../permission-system/presets",
-);
-
-describe("plan-instructions", () => {
-	it("lists subagent tooling for plan mode", () => {
-		expect(PLAN_MODE_TOOLS).toContain("subagent");
-		expect(PLAN_MODE_TOOLS).toContain("question");
-		expect(PLAN_MODE_TOOLS).toContain("web_search");
-		expect(PLAN_MODE_TOOLS).toContain("web_read");
-	});
-
-	it("plan preset keeps subagents skill in the system prompt", () => {
-		const planPreset = JSON.parse(
-			readFileSync(join(presetsDir, "plan.json"), "utf8"),
-		) as { permission: Record<string, unknown> };
-
-		const prompt = [
-			"# System",
-			"",
-			"## Skill: subagents",
-			"Use scout and planner.",
-		].join("\n");
-
-		const evaluator = new PermissionEvaluator();
-		const result = sanitizeSkillPromptBlocks(
-			prompt,
-			planPreset.permission as never,
-			evaluator,
-			{},
-		);
-
-		expect(result.prompt).toContain("## Skill: subagents");
-		expect(result.entries).toHaveLength(1);
-	});
-});
+import { readFileSync } from "node:fs";
+import { dirname, join } from "node:path";
+import { fileURLToPath } from "node:url";
+import { describe, expect, it } from "vitest";
+import { PermissionEvaluator } from "../../permission-system/src/permission-evaluator.ts";
+import { sanitizeSkillPromptBlocks } from "../../permission-system/src/sanitizers/skill-prompt.ts";
+import {
+	PLAN_INSTRUCTIONS,
+	PLAN_MODE_ENDED_MESSAGE,
+	PLAN_MODE_MESSAGE,
+	PLAN_MODE_TOOLS,
+} from "../src/plan-instructions.ts";
+
+const presetsDir = join(
+	dirname(fileURLToPath(import.meta.url)),
+	"../../permission-system/presets",
+);
+
+describe("plan-instructions", () => {
+	it("lists subagent tooling for plan mode", () => {
+		expect(PLAN_MODE_TOOLS).toContain("subagent");
+		expect(PLAN_MODE_TOOLS).toContain("question");
+		expect(PLAN_MODE_TOOLS).toContain("web_search");
+		expect(PLAN_MODE_TOOLS).toContain("web_read");
+	});
+
+	it("allows an optional short Next footer and structured questions", () => {
+		expect(PLAN_INSTRUCTIONS).toContain("**Next**");
+		expect(PLAN_INSTRUCTIONS).toContain("optional");
+		expect(PLAN_INSTRUCTIONS).toContain("2–3 bullets max");
+		expect(PLAN_INSTRUCTIONS).toContain("question");
+		expect(PLAN_INSTRUCTIONS).toContain("parallel scouts");
+		expect(PLAN_MODE_MESSAGE).toContain("optionally a short **Next** footer");
+		expect(PLAN_MODE_MESSAGE).toContain("Parallel scouts");
+		expect(PLAN_MODE_MESSAGE).toContain("question");
+	});
+
+	it("defines a clear plan-ended signal for leaving plan mode", () => {
+		expect(PLAN_MODE_ENDED_MESSAGE).toContain("[PLAN MODE ENDED]");
+		expect(PLAN_MODE_ENDED_MESSAGE).toMatch(/write/i);
+		expect(PLAN_MODE_ENDED_MESSAGE).toMatch(/bash/i);
+		expect(PLAN_MODE_ENDED_MESSAGE).toContain("[PLAN MODE ACTIVE]");
+	});
+
+	it("plan preset keeps subagents skill in the system prompt", () => {
+		const planPreset = JSON.parse(
+			readFileSync(join(presetsDir, "plan.json"), "utf8"),
+		) as { permission: Record<string, unknown> };
+
+		const prompt = [
+			"# System",
+			"",
+			"## Skill: subagents",
+			"Use scout and planner.",
+		].join("\n");
+
+		const evaluator = new PermissionEvaluator();
+		const result = sanitizeSkillPromptBlocks(
+			prompt,
+			planPreset.permission as never,
+			evaluator,
+			{},
+		);
+
+		expect(result.prompt).toContain("## Skill: subagents");
+		expect(result.entries).toHaveLength(1);
+	});
+});

package/node_modules/@vitos-pizza/agent-mode/tsconfig.json

@@ -1,13 +1,13 @@
-{
-	"extends": "../../tsconfig.base.json",
-	"compilerOptions": {
-		"allowImportingTsExtensions": true
-	},
-	"include": [
-		"extensions/**/*.ts",
-		"src/**/*.ts",
-		"tests/**/*.ts",
-		"vitest.config.ts"
-	],
-	"exclude": ["node_modules"]
-}
+{
+	"extends": "../../tsconfig.base.json",
+	"compilerOptions": {
+		"allowImportingTsExtensions": true
+	},
+	"include": [
+		"extensions/**/*.ts",
+		"src/**/*.ts",
+		"tests/**/*.ts",
+		"vitest.config.ts"
+	],
+	"exclude": ["node_modules"]
+}

package/node_modules/@vitos-pizza/agent-mode/vitest.config.ts

@@ -1,8 +1,8 @@
-import { defineConfig } from "vitest/config";
-
-export default defineConfig({
-	test: {
-		environment: "node",
-		include: ["tests/**/*.test.ts"],
-	},
-});
+import { defineConfig } from "vitest/config";
+
+export default defineConfig({
+	test: {
+		environment: "node",
+		include: ["tests/**/*.test.ts"],
+	},
+});