@ferris1225/pi-subagents 0.28.0 → 0.31.0

package/agents/explore.md

@@ -20,6 +20,7 @@ You are an explore agent: a fast, read-only reconnaissance specialist. You inves
 2. Read KEY SECTIONS, not whole files. After 1-2 greps, read the top match instead of running more greps.
 3. Identify the types, interfaces, and key function signatures involved; note how files depend on each other.
 4. Record exact paths and line ranges so the caller can jump straight in.
+5. If the brief asks you to inspect images (screenshots, mockups, designs), `read` them — the model receives them as attachments when it supports vision.
 
 ## Thoroughness (infer from the task, default medium)
 - Quick: targeted lookups, key files only.

package/agents/reviewer.md

@@ -21,6 +21,7 @@ Match the type to the task brief; the hunt checklist below applies to every type
 ### 1. Code diffs (default)
 1. Run `git diff` and `git status` to see the recent changes. If a specific file set was given, read those files.
 2. Read the modified files in full where needed; judge the change in the context of the surrounding code.
+3. If the brief includes images (UI screenshots, design mockups), `read` them with the read tool and judge the change against them.
 
 ### 2. Plans
 Validate a proposed plan for feasibility and completeness: missing steps, hidden risks, alignment with the existing architecture, and whether the scope is appropriately bounded.

package/agents/worker.md

@@ -13,7 +13,7 @@ You are a worker agent with full capabilities, operating in an isolated context
 Work in phases. Do not skip planning or verification.
 
 ### Phase 1 — Context
-Read the brief fully. If it references files, read them before editing. If critical context is clearly missing, state what an `explore` should retrieve rather than guessing.
+Read the brief fully. If it references files, read them before editing. If it references images (screenshots, mockups, designs), `read` them too — the model receives them as attachments when it supports vision. If critical context is clearly missing, state what an `explore` should retrieve rather than guessing.
 
 ### Phase 2 — Plan
 Inspect existing code and conventions first. Form the smallest coherent root-cause change that satisfies the brief. For a large task, write a short internal plan (files to touch, order, risks) before editing. Do not refactor unrelated code or create docs unless the brief asks.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ferris1225/pi-subagents",
-  "version": "0.28.0",
+  "version": "0.31.0",
   "description": "Focused sub-agent delegation for pi: explore / worker / reviewer agents in isolated context, with proactive dispatch injection and per-agent model selection.",
   "type": "module",
   "license": "MIT",

package/src/completion.ts

@@ -151,3 +151,31 @@ export function completionTriggersTurn(result: SingleResult, notifyOnReviewPass:
 		reviewVerdict(getResultOutput(result)) === "pass"
 	);
 }
+
+/** Minimal shape of an active run, for the "others still running" footer. Kept
+ * decoupled from the monitor's RunView so this stays a pure, easily tested
+ * formatter; the caller maps its live runs into this shape. */
+export interface ActiveRunFoot {
+	id: number;
+	agent: string;
+	/** Optional content label (task-derived) shown next to the agent name. */
+	label?: string;
+}
+
+/**
+ * Footer appended to a completion message when OTHER runs are still active, so
+ * the main agent does not declare the overall task done prematurely. A result
+ * arriving for one run does not mean sibling runs are finished; naming them
+ * gives the main agent concrete, in-context awareness to keep waiting.
+ *
+ * Returns "" when nothing is active (the common, single-run case stays quiet).
+ */
+export function formatActiveRunsFooter(runs: readonly ActiveRunFoot[], maxListed = 4): string {
+	if (runs.length === 0) return "";
+	const listed = runs.slice(0, maxListed);
+	const items = listed
+		.map((run) => `#${run.id} ${run.agent}${run.label ? `·${run.label}` : ""}`)
+		.join(", ");
+	const more = runs.length > listed.length ? `, +${runs.length - listed.length} more` : "";
+	return `\n\n⚠ ${runs.length} other run${runs.length === 1 ? "" : "s"} still active: ${items}${more}. Do not conclude the overall task yet — wait for their results (they wake you automatically) or check subagent_status.`;
+}

package/src/config.ts

@@ -108,6 +108,18 @@ export interface SubagentsConfig {
 	 * model. 0 disables the idle watchdog. Default: 90.
 	 */
 	idleTimeoutSec: number;
+	/**
+	 * Vision-capable model for tasks flagged `vision: true` (viewing screenshots,
+	 * mockups, designs). Unset means such tasks fall back to the main session's
+	 * current model.
+	 */
+	visionModel?: string;
+	/**
+	 * One-time feature announcements already shown to the user (e.g. "vision
+	 * model" after an update that introduced it). Persisted so the notice never
+	 * nags again.
+	 */
+	announcedFeatures: string[];
 }
 
 export const DEFAULT_CONFIG: SubagentsConfig = {
@@ -122,6 +134,7 @@ export const DEFAULT_CONFIG: SubagentsConfig = {
 	maxConcurrency: DEFAULT_MAX_CONCURRENCY,
 	maxFixRounds: DEFAULT_MAX_FIX_ROUNDS,
 	idleTimeoutSec: DEFAULT_IDLE_TIMEOUT_SEC,
+	announcedFeatures: [],
 };
 
 export function getConfigPath(agentDir: string = getAgentDir()): string {
@@ -168,6 +181,7 @@ export function normalizeConfig(raw: unknown): SubagentsConfig {
 		maxConcurrency: DEFAULT_CONFIG.maxConcurrency,
 		maxFixRounds: DEFAULT_CONFIG.maxFixRounds,
 		idleTimeoutSec: DEFAULT_CONFIG.idleTimeoutSec,
+		announcedFeatures: [],
 	};
 
 	if (Array.isArray(raw.enabledAgents)) {
@@ -240,6 +254,16 @@ export function normalizeConfig(raw: unknown): SubagentsConfig {
 		config.idleTimeoutSec = Math.max(0, Math.min(IDLE_TIMEOUT_SEC_LIMIT, Math.round(raw.idleTimeoutSec)));
 	}
 
+	if (isModelReference(raw.visionModel)) {
+		config.visionModel = (raw.visionModel as string).trim();
+	}
+
+	if (Array.isArray(raw.announcedFeatures)) {
+		config.announcedFeatures = raw.announcedFeatures.filter(
+			(feature): feature is string => typeof feature === "string" && feature.trim().length > 0,
+		);
+	}
+
 	return config;
 }
 
@@ -249,6 +273,7 @@ function defaultConfig(): SubagentsConfig {
 		enabledAgents: [...DEFAULT_CONFIG.enabledAgents],
 		agentModels: {},
 		agentThinkingLevels: {},
+		announcedFeatures: [],
 	};
 }