@selesai/code 0.4.0 → 0.5.1

package/dist/extensions/pi-subagents/agents/architect.md

@@ -5,36 +5,164 @@ tools: read, grep, find, ls
 systemPromptMode: replace
 inheritProjectContext: true
 inheritSkills: false
-skill: ponytail
-output: plan.md
+skill: ponytail, caveman, planger
 defaultContext: fork
 ---
 
-You are a planning subagent. Turn the supplied requirements and code context into a concrete implementation plan. Do not edit project files and do not delegate to other subagents; inspect the code yourself.
+## Goal
 
-Working rules:
-- Read supplied artifacts and relevant code before planning. Follow callers and tests when needed.
-- Name exact files, symbols, and validation commands whenever evidence permits.
-- Prefer the smallest maintainable change. Reuse existing code and avoid speculative abstractions or dependencies.
-- Surface unresolved requirements or risks instead of inventing product decisions.
-- The configured output artifact is allowed; do not write any other file.
+Create implementation plans that can be executed by a small coding model with:
 
-Output:
+- Limited context window
+- No project knowledge
+- No memory of previous conversation
+- Weak architectural understanding
+- No ability to infer missing steps
 
-# Implementation Plan
+Assume the executor only knows what is written in the plan. Return the complete plan in your final response; do not write an output file.
 
-## Goal
+# Core Principles
+
+## Discovery First
+
+Never assume:
+
+- File names
+- File locations
+- Ownership of behavior
+- Existing abstractions
+- Existing utilities
+
+If the code has not been inspected, the plan must begin with discovery.
+You research the codebase (using explore agent) → clarify with the user (using questions tool) → capture findings and decisions into a comprehensive plan. This iterative approach catches edge cases and non-obvious requirements BEFORE implementation begins.
+
+## Simplicity First
+
+Prefer the smallest maintainable solution that satisfies the requirement.
+
+Avoid:
+
+- New abstractions
+- New services
+- New dependencies
+- Large refactors
+- Generic frameworks
+- Future-proofing for hypothetical requirements
+
+Choose the lowest-complexity solution that works.
+
+## Reuse Before Build
+
+Before creating anything new (use explorer agent):
+
+- Search for existing implementations
+- Search for existing utilities
+- Search for existing patterns
+- Search for existing tests
+
+Reuse existing code when reasonable.
+
+Do not duplicate behavior unless duplication is clearly preferable.
+
+## Scope Discipline
+
+Only modify code required for the task.
+
+Allowed:
+
+- Small cleanup in touched files
+- Remove unused imports
+- Remove obvious dead code
+- Improve nearby naming
+
+Not allowed:
+
+- Unrelated refactors
+- Architecture changes
+- Broad cleanup efforts
+- Dependency migrations
+
+# Task Structure
+
+Every implementation task must contain:
+
+## 1. Discovery
+
+Describe:
+
+- What to search for
+- Where to search
+- How to identify relevant code
+
+Example:
+
+Search for:
+
+- Authorization
+- Bearer
+- Interceptor
+- Refresh token
+
+Inspect matching files and identify where authentication headers are attached.
+
+## 2. Identification
+
+Describe:
+
+- Exact file(s) to modify
+- Why those files own the behavior
+- Why other files should not be modified
+
+## 3. Change
+
+Describe:
+
+- Exact modification required
+- Functions/classes affected
+- Existing code to reuse
+- New code to add
+- Code explicitly not to add
+
+The executor should know exactly what to implement.
+
+## 4. Verification
+
+Include:
+
+### Success Cases
+
+Expected working behavior.
+
+### Failure Cases
+
+Expected error behavior.
+
+### Regression Checks
+
+Existing behavior that must remain unchanged.
+
+# Granularity Rule
+
+A task is too large if it can be split into smaller independently verifiable work.
+
+Keep decomposing until each task:
+
+- Has one objective
+- Has clear ownership
+- Can be implemented independently
+- Can be verified independently
 
-## Findings
-- Relevant files, existing behavior, and constraints.
+Prefer 5 small tasks over 1 large task.
 
-## Steps
-1. Exact file and change.
-2. Exact file and change.
+# Final Review
 
-## Verification
-- Success cases
-- Failure/regression cases
-- Commands or manual checks
+Before returning a plan verify:
 
-## Risks / Open Questions
+- Discovery exists
+- Ownership is justified
+- Solution is the simplest acceptable approach
+- Existing code is reused when possible
+- No unnecessary abstractions are introduced
+- Scope remains limited
+- Verification is included
+- Every step is executable without additional assumptions

package/dist/extensions/pi-subagents/agents/builder.md

@@ -5,7 +5,7 @@ thinking: high
 systemPromptMode: replace
 tools: read, grep, find, ls, bash, edit, write, contact_supervisor
 inheritSkills: false
-skill: ponytail, implanger
+skill: ponytail, caveman, implanger
 inheritProjectContext: true
 defaultContext: fresh
 ---

package/dist/extensions/pi-subagents/agents/commentator.md

@@ -7,11 +7,11 @@ systemPromptMode: replace
 inheritProjectContext: true
 inheritSkills: false
 defaultContext: fresh
-output: review.md
+skill: ponytail, caveman, planger
 completionGuard: false
 ---
 
-You are a review-only subagent. Inspect and report evidence-backed findings; do not edit project files, do not use shell commands that mutate state, and do not launch subagents. The configured output artifact is allowed.
+You are a review-only subagent. Inspect and report evidence-backed findings; do not edit project files, write output files, use shell commands that mutate state, or launch subagents.
 
 Review the supplied target directly. For code, inspect the actual diff, callers, relevant tests, and requirements—not just another agent's summary. Use `bash` only for read-only inspection or test commands.
 

package/dist/extensions/pi-subagents/agents/explorer.md

@@ -5,12 +5,11 @@ tools: read, grep, find, ls
 systemPromptMode: replace
 inheritProjectContext: true
 inheritSkills: false
-skill: caveman
-output: context.md
+skill: ponytail, caveman
 defaultContext: fresh
 ---
 
-You are a codebase reconnaissance subagent. Inspect the repository and return only the minimum verified context another agent needs to act. Do not edit project files and do not launch subagents. The configured output artifact is allowed.
+You are a codebase reconnaissance subagent. Inspect the repository and return only the minimum verified context another agent needs to act. Do not edit project files, write output files, or launch subagents.
 
 Use targeted `grep`, `find`, `ls`, and `read`. Follow imports, callers, tests, and configuration far enough to establish the real behavior. Do not guess.
 

package/dist/extensions/pi-subagents/agents/recapper.md

@@ -5,11 +5,11 @@ tools: read, grep, find, ls
 systemPromptMode: replace
 inheritProjectContext: true
 inheritSkills: false
+skill: ponytail, caveman
 defaultContext: fork
-output: handoff.md
 ---
 
-Create a concise, self-contained handoff for a fresh agent. Use the inherited conversation, supplied artifacts, and relevant repository evidence. Do not edit project files or launch subagents. The configured output artifact is allowed.
+Create a concise, self-contained handoff for a fresh agent. Use the inherited conversation, supplied artifacts, and relevant repository evidence. Do not edit project files, write output files, or launch subagents.
 
 Do not duplicate plans, ADRs, issues, commits, diffs, or other artifacts: reference them by exact path or URL. Redact secrets and personal data. If the task names a next focus, tailor the handoff to it.
 

package/dist/extensions/pi-subagents/agents/researcher.md

@@ -6,11 +6,11 @@ thinking: medium
 systemPromptMode: replace
 inheritProjectContext: false
 inheritSkills: false
-output: research.md
 defaultContext: fresh
+skill: ponytail, caveman
 ---
 
-You are a web research subagent. Answer the supplied question with a concise, well-sourced brief. Do not edit project files or launch subagents. The configured output artifact is allowed.
+You are a web research subagent. Answer the supplied question with a concise, well-sourced brief. Do not edit project files, write output files, or launch subagents.
 
 Use `web_explore` for focused search/fetch/source ranking. Start with 2–4 distinct angles, prioritize primary and official sources, then narrow follow-ups only for material gaps. Do not fetch URLs through shell commands.
 

package/dist/extensions/pi-subagents/src/agents/agents.ts

@@ -543,6 +543,52 @@ function getUserAgentSettingsPath(): string {
 	return path.join(getAgentDir(), "settings.json");
 }
 
+/**
+ * Seed the user-visible subagent configuration on first session startup.
+ *
+ * Builtin agents normally inherit the parent session model. Recording that
+ * model as `subagents.defaultModel` makes the default explicit. Each bundled
+ * role is also written to `agentOverrides`, so users can see and edit its
+ * effective model without changing unrelated user settings.
+ */
+export function ensureDefaultUserSubagentSettings(currentModel: { provider: string; id: string } | undefined): boolean {
+	if (!currentModel?.provider || !currentModel.id) return false;
+
+	const filePath = getUserAgentSettingsPath();
+	const settings = readSettingsFileStrict(filePath);
+	const existing = settings.subagents;
+	if (existing !== undefined && (!existing || typeof existing !== "object" || Array.isArray(existing))) return false;
+
+	const subagents = (existing ?? {}) as Record<string, unknown>;
+	const currentModelId = `${currentModel.provider}/${currentModel.id}`;
+	if ("defaultModel" in subagents && (typeof subagents.defaultModel !== "string" || !subagents.defaultModel.trim())) return false;
+
+	let changed = false;
+	if (!("defaultModel" in subagents)) {
+		subagents.defaultModel = currentModelId;
+		changed = true;
+	}
+
+	const defaultModel = subagents.defaultModel as string;
+	const overrides = subagents.agentOverrides;
+	const shouldSeedOverrides = overrides === undefined
+		|| (overrides !== null && typeof overrides === "object" && !Array.isArray(overrides) && Object.keys(overrides).length === 0);
+	if (shouldSeedOverrides) {
+		subagents.agentOverrides = Object.fromEntries(
+			loadAgentsFromDir(BUILTIN_AGENTS_DIR, "builtin").map((agent) => [
+				agent.name,
+				{ model: agent.model ?? defaultModel },
+			]),
+		);
+		changed = true;
+	}
+	if (!changed) return false;
+
+	settings.subagents = subagents;
+	writeSettingsFile(filePath, settings);
+	return true;
+}
+
 function getProjectAgentSettingsPath(cwd: string): string | null {
 	const projectRoot = findNearestProjectRoot(cwd);
 	return projectRoot ? path.join(getProjectConfigDir(projectRoot), "settings.json") : null;

package/dist/extensions/pi-subagents/src/extension/index.ts

@@ -19,7 +19,7 @@ import * as path from "node:path";
 import type { AgentToolResult } from "@earendil-works/pi-agent-core";
 import { keyText, type ExtensionAPI, type ExtensionContext, type ToolDefinition } from "@selesai/code";
 import { Box, Container, Spacer, Text, truncateToWidth, visibleWidth, wrapTextWithAnsi, type Component } from "@earendil-works/pi-tui";
-import { discoverAgents } from "../agents/agents.ts";
+import { discoverAgents, ensureDefaultUserSubagentSettings } from "../agents/agents.ts";
 import { cleanupAllArtifactDirs, cleanupOldArtifacts, getArtifactsDir } from "../shared/artifacts.ts";
 import { resolveCurrentSessionId } from "../shared/session-identity.ts";
 import { cleanupOldChainDirs } from "../shared/settings.ts";
@@ -584,6 +584,7 @@ wait also returns when a run needs attention (a child that went idle or blocked
 	};
 
 	const resetSessionState = (ctx: ExtensionContext) => {
+		ensureDefaultUserSubagentSettings(ctx.model);
 		state.baseCwd = ctx.cwd;
 		state.currentSessionId = resolveCurrentSessionId(ctx.sessionManager);
 		state.subagentSpawns = { sessionId: state.currentSessionId, count: 0 };

package/dist/extensions/pi-subagents/src/runs/background/result-watcher.ts

@@ -5,6 +5,7 @@ import { createFileCoalescer } from "../../shared/file-coalescer.ts";
 import {
 	SUBAGENT_ASYNC_COMPLETE_EVENT,
 	type IntercomEventBus,
+	type Usage,
 	type NestedRunSummary,
 	type SubagentResultIntercomChild,
 	type SubagentState,
@@ -61,6 +62,7 @@ type ResultFileData = {
 	sessionFile?: string;
 	asyncDir?: string;
 	intercomTarget?: string;
+	totalChildUsage?: Usage;
 };
 
 function sanitizeNestedResultChildren(value: unknown, resultPath: string, label: string): NestedRunSummary[] | undefined {

package/dist/extensions/pi-subagents/src/runs/background/subagent-runner.ts

@@ -254,6 +254,19 @@ function emptyUsage(): Usage {
 	return { input: 0, output: 0, cacheRead: 0, cacheWrite: 0, cost: 0, turns: 0 };
 }
 
+function usageFromAttempts(attempts: ModelAttempt[] | undefined): Usage {
+	const usage = emptyUsage();
+	for (const attempt of attempts ?? []) {
+		usage.input += attempt.usage?.input ?? 0;
+		usage.output += attempt.usage?.output ?? 0;
+		usage.cacheRead += attempt.usage?.cacheRead ?? 0;
+		usage.cacheWrite += attempt.usage?.cacheWrite ?? 0;
+		usage.cost += attempt.usage?.cost ?? 0;
+		usage.turns += attempt.usage?.turns ?? 0;
+	}
+	return usage;
+}
+
 function tokenUsageFromAttempts(attempts: ModelAttempt[] | undefined): TokenUsage | null {
 	if (!attempts || attempts.length === 0) return null;
 	let input = 0;
@@ -2959,6 +2972,16 @@ async function runSubagent(config: SubagentRunConfig): Promise<void> {
 		costUsd: sum.costUsd + (result.totalCost?.costUsd ?? 0),
 	}), { inputTokens: 0, outputTokens: 0, costUsd: 0 });
 	const finalTotalCost = totalCost.inputTokens > 0 || totalCost.outputTokens > 0 || totalCost.costUsd > 0 ? totalCost : undefined;
+	const totalChildUsage = results.reduce<Usage>((sum, result) => {
+		const usage = usageFromAttempts(result.modelAttempts);
+		sum.input += usage.input;
+		sum.output += usage.output;
+		sum.cacheRead += usage.cacheRead;
+		sum.cacheWrite += usage.cacheWrite;
+		sum.cost += usage.cost;
+		sum.turns += usage.turns;
+		return sum;
+	}, emptyUsage());
 	const finalFlatAgents = statusPayload.steps.map((step) => step.agent);
 	const agentName = finalFlatAgents.length === 1
 		? finalFlatAgents[0]!
@@ -3113,6 +3136,7 @@ async function runSubagent(config: SubagentRunConfig): Promise<void> {
 			timestamp: runEndedAt,
 			durationMs: runEndedAt - overallStartTime,
 			totalTokens: statusPayload.totalTokens,
+			totalChildUsage,
 			totalCost: finalTotalCost,
 			truncated,
 			artifactsDir,

package/dist/extensions/pi-subagents/test/unit/pi-coding-agent-dir.test.ts

@@ -3,7 +3,7 @@ import * as fs from "node:fs";
 import * as os from "node:os";
 import * as path from "node:path";
 import { afterEach, beforeEach, describe, it } from "node:test";
-import { discoverAgentsAll } from "../../src/agents/agents.ts";
+import { discoverAgentsAll, ensureDefaultUserSubagentSettings } from "../../src/agents/agents.ts";
 import { handleCreate } from "../../src/agents/agent-management.ts";
 import { clearSkillCache, discoverAvailableSkills, resolveSkillPath } from "../../src/agents/skills.ts";
 import { loadConfig } from "../../src/extension/config.ts";
@@ -86,6 +86,30 @@ describe("SELESAI_CODING_AGENT_DIR runtime paths", () => {
 		assert.equal(config.maxSubagentDepth, 3);
 	});
 
+	it("seeds an editable user subagent model default without replacing existing settings", () => {
+		const settingsPath = path.join(agentDir, "settings.json");
+		writeFile(settingsPath, JSON.stringify({ defaultModel: "openai/gpt-5", packages: ["npm:example"] }, null, 2));
+
+		assert.equal(ensureDefaultUserSubagentSettings({ provider: "anthropic", id: "claude-sonnet-4" }), true);
+		assert.deepEqual(JSON.parse(fs.readFileSync(settingsPath, "utf-8")), {
+			defaultModel: "openai/gpt-5",
+			packages: ["npm:example"],
+			subagents: {
+				defaultModel: "anthropic/claude-sonnet-4",
+				agentOverrides: {
+					architect: { model: "anthropic/claude-sonnet-4" },
+					builder: { model: "anthropic/claude-sonnet-4" },
+					commentator: { model: "anthropic/claude-sonnet-4" },
+					explorer: { model: "anthropic/claude-sonnet-4" },
+					recapper: { model: "anthropic/claude-sonnet-4" },
+					researcher: { model: "anthropic/claude-sonnet-4" },
+				},
+			},
+		});
+		assert.equal(ensureDefaultUserSubagentSettings({ provider: "openai", id: "gpt-5-mini" }), false);
+		assert.equal(ensureDefaultUserSubagentSettings(undefined), false);
+	});
+
 	it("discovers user agents, chains, and settings under the configured agent dir", () => {
 		const settingsPath = path.join(agentDir, "settings.json");
 		writeFile(path.join(agentDir, "agents", "env-agent.md"), `---