@yagni-app/code-staging 1.2.13-staging.2171.1 → 1.2.13-staging.2173.1

package/dist/extension/pipeline/sessionLane.d.ts

@@ -48,6 +48,12 @@ import type { WorktreeExec } from "./worktree.js";
 export declare const SESSION_TIER: ModelTier;
 /** The caller the model proxy attributes the session's own turns to. */
 export declare const SESSION_CALLER_LABEL = "go:session";
+/**
+ * Whether this process is a session-lane turn: the pi child {@link runSessionTurn}
+ * starts, which the runner gives the session's caller label (`YAGNI_CALLER`).
+ * A subagent it spawns gets its own label, so only the turn itself answers yes.
+ */
+export declare function isSessionLaneTurn(env?: NodeJS.ProcessEnv): boolean;
 /** The model's own account of a turn that builds, beside `result.json` in the output folder. */
 export declare const SESSION_REPORT_FILE_NAME = "session-report.json";
 /** What a plan turn may use: everything that reads or runs, nothing that edits. */

package/dist/extension/pipeline/sessionLane.js

@@ -56,6 +56,14 @@ import { snapshotWorkspace } from "./workspace.js";
 export const SESSION_TIER = "peak";
 /** The caller the model proxy attributes the session's own turns to. */
 export const SESSION_CALLER_LABEL = "go:session";
+/**
+ * Whether this process is a session-lane turn: the pi child {@link runSessionTurn}
+ * starts, which the runner gives the session's caller label (`YAGNI_CALLER`).
+ * A subagent it spawns gets its own label, so only the turn itself answers yes.
+ */
+export function isSessionLaneTurn(env = process.env) {
+    return env.YAGNI_CALLER === SESSION_CALLER_LABEL;
+}
 /** The model's own account of a turn that builds, beside `result.json` in the output folder. */
 export const SESSION_REPORT_FILE_NAME = "session-report.json";
 /** What a plan turn may use: everything that reads or runs, nothing that edits. */

package/dist/extension/subagents.d.ts

@@ -69,6 +69,15 @@ export interface SubagentDef {
  * not a second build (on staging 2026-10-28, advanced-tier verifiers ran 34
  * minutes after an 11-minute build). */
 export declare const VERIFICATION_WALL_TIMEOUT_MS: number;
+/**
+ * The tier the built-in verification agent asks for: `peak` in a session-lane
+ * turn (`pipeline/sessionLane.ts`), the one place the proxy serves a verifier
+ * above its advanced ceiling for subagents, and `advanced` everywhere else (the
+ * CLI, desktop, the attempt lane's stage graph), so a request never asks for a
+ * tier it would not be served. A repo's own `verification` agent keeps the tier
+ * it declares.
+ */
+export declare function verificationTier(env?: NodeJS.ProcessEnv): ModelTier;
 /** Claude Code model names → YAGNI tiers; YAGNI tiers pass through. */
 export declare function mapModelTier(model: string | undefined): ModelTier;
 /**
@@ -80,7 +89,7 @@ export declare function parseAgentMarkdown(content: string, fallbackName: string
 export interface DiscoverDeps {
     cwd: string;
     homeDir?: string;
-    /** Env seam for the plugin agent dirs (defaults to process.env). */
+    /** Env seam for the plugin agent dirs and the verification agent's tier (defaults to process.env). */
     env?: NodeJS.ProcessEnv;
 }
 /**
@@ -112,7 +121,7 @@ export interface MakeSubagentToolDeps {
     runStageImpl?: typeof runStage;
     discover?: (deps: DiscoverDeps) => SubagentDef[];
     homeDir?: string;
-    /** Env seam for the telemetry sessionId read (defaults to process.env). */
+    /** Env seam for discovery and the telemetry sessionId read (defaults to process.env). */
     env?: NodeJS.ProcessEnv;
     /** Live ultra-mode probe (/ultra): widens the per-call fan-out ceiling. */
     isUltra?: () => boolean;

package/dist/extension/subagents.js

@@ -20,13 +20,14 @@
 import * as fs from "node:fs";
 import { homedir } from "node:os";
 import { delimiter, join } from "node:path";
-import { parseFrontmatter } from "@earendil-works/pi-coding-agent";
+import { parseFrontmatter, } from "@earendil-works/pi-coding-agent";
 import { Type } from "typebox";
 import { sanitizeCallerSegment } from "./config.js";
 import { logEvent } from "./errorSink.js";
 import { withResilience } from "./pipeline/resilience.js";
 import { runStage } from "./pipeline/runner.js";
-import { DEFAULT_RESILIENCE_POLICY } from "./pipeline/types.js";
+import { isSessionLaneTurn } from "./pipeline/sessionLane.js";
+import { DEFAULT_RESILIENCE_POLICY, } from "./pipeline/types.js";
 import { applyChildEvent, finalizeTask, formatWorkingMessage, newTaskProgress, progressSummaryText, renderSubagentCall, renderSubagentResult, } from "./subagentRender.js";
 /**
  * pi's tool contract signals an error by THROWING, and a normally-returned
@@ -216,26 +217,38 @@ nothing, say exactly what you tried to break and how.`;
  * not a second build (on staging 2026-10-28, advanced-tier verifiers ran 34
  * minutes after an 11-minute build). */
 export const VERIFICATION_WALL_TIMEOUT_MS = 15 * 60_000;
-/** The diamond's reduce layer: refute-first review of completed work on the
- * peak tier (judgment is the whole job, so the premium is worth paying; the
- * server exempts this caller from its advanced ceiling for subagents). One by
- * default, on a fifteen-minute clock. It has no edit/write tools; bash is
- * included for `git diff`-style inspection and is restricted to read-only USE
- * by the persona, the same prompt-level stance as /go's reviewer — not a
- * technical guarantee. */
-const VERIFICATION_AGENT = {
-    name: "verification",
-    description: "Adversarial verification of completed work: tries to refute a change, " +
-        "plan, or claim and reports concrete failure scenarios. Does not edit " +
-        "files. Send one; send two or three with different lenses (correctness, " +
-        "edge cases, codebase fit) only for a risky change such as a migration, " +
-        "auth or money, and treat agreement as confirmation.",
-    model: "peak",
-    tools: ["read", "grep", "find", "ls", "bash", "ask_yagni"],
-    body: VERIFICATION_BODY,
-    source: "builtin",
-    wallTimeoutMs: VERIFICATION_WALL_TIMEOUT_MS,
-};
+/**
+ * The tier the built-in verification agent asks for: `peak` in a session-lane
+ * turn (`pipeline/sessionLane.ts`), the one place the proxy serves a verifier
+ * above its advanced ceiling for subagents, and `advanced` everywhere else (the
+ * CLI, desktop, the attempt lane's stage graph), so a request never asks for a
+ * tier it would not be served. A repo's own `verification` agent keeps the tier
+ * it declares.
+ */
+export function verificationTier(env = process.env) {
+    return isSessionLaneTurn(env) ? "peak" : "advanced";
+}
+/** The diamond's reduce layer: refute-first review of completed work, on
+ * {@link verificationTier} (judgment is the whole job, so the premium is worth
+ * paying where the lane serves it). One by default, on a fifteen-minute clock.
+ * It has no edit/write tools; bash is included for `git diff`-style inspection
+ * and is restricted to read-only USE by the persona, the same prompt-level
+ * stance as /go's reviewer, not a technical guarantee. */
+function verificationAgent(env) {
+    return {
+        name: "verification",
+        description: "Adversarial verification of completed work: tries to refute a change, " +
+            "plan, or claim and reports concrete failure scenarios. Does not edit " +
+            "files. Send one; send two or three with different lenses (correctness, " +
+            "edge cases, codebase fit) only for a risky change such as a migration, " +
+            "auth or money, and treat agreement as confirmation.",
+        model: verificationTier(env),
+        tools: ["read", "grep", "find", "ls", "bash", "ask_yagni"],
+        body: VERIFICATION_BODY,
+        source: "builtin",
+        wallTimeoutMs: VERIFICATION_WALL_TIMEOUT_MS,
+    };
+}
 // Concrete tiers a subagent can actually run on. `balanced` is deliberately NOT
 // a member here even though it is a member of `ModelTier`: a subagent needs
 // ONE model for its whole run, and balanced is a session-level routing policy,
@@ -331,9 +344,10 @@ function loadAgentsFromDir(dir, source) {
  */
 export function discoverSubagents(deps) {
     const home = deps.homeDir ?? homedir();
+    const env = deps.env ?? process.env;
     const layers = [
-        [GENERAL_AGENT, SEARCHER_AGENT, IMPLEMENTER_AGENT, VERIFICATION_AGENT],
-        ...pluginAgentDirs(deps.env ?? process.env).map((dir) => loadAgentsFromDir(dir, "plugin")),
+        [GENERAL_AGENT, SEARCHER_AGENT, IMPLEMENTER_AGENT, verificationAgent(env)],
+        ...pluginAgentDirs(env).map((dir) => loadAgentsFromDir(dir, "plugin")),
         loadAgentsFromDir(join(home, ".claude", "agents"), "user-claude"),
         loadAgentsFromDir(join(deps.cwd, ".pi", "agents"), "project-pi"),
         loadAgentsFromDir(join(deps.cwd, ".claude", "agents"), "project-claude"),
@@ -353,9 +367,7 @@ const SOURCE_LABEL = {
     "project-claude": ".claude/agents",
 };
 export function formatAgentList(agents) {
-    return agents
-        .map((a) => `${a.name} (${SOURCE_LABEL[a.source]}): ${a.description}`)
-        .join("\n");
+    return agents.map((a) => `${a.name} (${SOURCE_LABEL[a.source]}): ${a.description}`).join("\n");
 }
 /**
  * A subagent invocation is a synthetic pipeline stage: the task rides in as
@@ -450,7 +462,9 @@ export function buildSubagentStage(def, task, opts = {}) {
             id: "implement",
             agent: def.name,
             model: def.model,
-            tools: grounded ? def.tools ?? DEFAULT_SUBAGENT_TOOLS : stripGroundingTool(def.tools ?? DEFAULT_SUBAGENT_TOOLS),
+            tools: grounded
+                ? (def.tools ?? DEFAULT_SUBAGENT_TOOLS)
+                : stripGroundingTool(def.tools ?? DEFAULT_SUBAGENT_TOOLS),
             taskTemplate: "{ticket}",
         },
         ctx: { ticket: task, grounded },
@@ -463,7 +477,9 @@ const parameters = Type.Object({
     agent: Type.Optional(Type.String({ description: "Agent name (see /agents). Defaults to general." })),
     tasks: Type.Optional(Type.Array(Type.Object({
         task: Type.String(),
-        agent: Type.Optional(Type.String({ description: "Agent name; defaults to the top-level `agent`, then general." })),
+        agent: Type.Optional(Type.String({
+            description: "Agent name; defaults to the top-level `agent`, then general.",
+        })),
     }), {
         description: `Run several independent tasks in parallel (max ${MAX_PARALLEL_SUBAGENTS}; ${MAX_PARALLEL_SUBAGENTS_ULTRA} in ultra mode). Use INSTEAD of task. Entries without their own agent run as the top-level agent.`,
     })),
@@ -504,7 +520,11 @@ export function subagentRunPolicy(wallTimeoutMs = SUBAGENT_WALL_TIMEOUT_MS) {
 export function subagentTelemetry(env) {
     return (rec) => logEvent({
         source: "subagent",
-        level: rec.outcome === "ok" ? "info" : rec.outcome === "timeout" || rec.outcome === "fatal" ? "error" : "warn",
+        level: rec.outcome === "ok"
+            ? "info"
+            : rec.outcome === "timeout" || rec.outcome === "fatal"
+                ? "error"
+                : "warn",
         event: `attempt_${rec.outcome}`,
         sessionId: env.YAGNI_SESSION_ID,
         fields: {
@@ -563,7 +583,7 @@ export function makeSubagentTool(deps = {}) {
     // Fails soft: discovery problems never block the tool.
     let agentListBlock = "";
     try {
-        const discovered = discover({ cwd: process.cwd(), homeDir: deps.homeDir });
+        const discovered = discover({ cwd: process.cwd(), homeDir: deps.homeDir, env: deps.env });
         if (discovered.length > 0) {
             agentListBlock =
                 " Available agents (first sentence only; /agents lists them in full):\n" +
@@ -589,7 +609,8 @@ export function makeSubagentTool(deps = {}) {
             "spell out the task completely. Agents come from this repo's .claude/agents and .pi/agents " +
             "(list them with /agents); omit `agent` for the general-purpose one. The list below was " +
             "captured at session launch — if the session's cwd differs from the launch cwd it may be " +
-            "stale; an unknown name is a hard error, so trust /agents over this list when they differ." + agentListBlock,
+            "stale; an unknown name is a hard error, so trust /agents over this list when they differ." +
+            agentListBlock,
         promptSnippet: "subagent: delegate a self-contained task (or parallel tasks) to a fresh-context agent; returns its report.",
         parameters,
         // Self-framed: the condensed transcript look has no tinted tool boxes.
@@ -619,7 +640,7 @@ export function makeSubagentTool(deps = {}) {
                 return fail(`Error: at most ${maxParallel} parallel tasks per call.`);
             }
             const cwd = ctx?.cwd ?? process.cwd();
-            const agents = discover({ cwd, homeDir: deps.homeDir });
+            const agents = discover({ cwd, homeDir: deps.homeDir, env: deps.env });
             const resolved = [];
             for (const req of requested) {
                 // A tasks[] entry names its own agent first; absent that, it inherits
@@ -647,7 +668,11 @@ export function makeSubagentTool(deps = {}) {
                 onUpdate?.({
                     content: [{ type: "text", text: progressSummaryText(progresses, now) }],
                     details: {
-                        tasks: progresses.map((p) => ({ ...p, actions: [...p.actions], usage: { ...p.usage } })),
+                        tasks: progresses.map((p) => ({
+                            ...p,
+                            actions: [...p.actions],
+                            usage: { ...p.usage },
+                        })),
                     },
                 });
                 const working = formatWorkingMessage(progresses, now);
@@ -678,7 +703,7 @@ export function makeSubagentTool(deps = {}) {
                         // Computed from the tools the runner will actually pass the
                         // child (`--tools`), so a custom definition cannot opt out by
                         // name.
-                        stageWriteCapable: (stage.tools).some((t) => t === "edit" || t === "write"),
+                        stageWriteCapable: stage.tools.some((t) => t === "edit" || t === "write"),
                         // Content-free task discriminator: which entry of THIS call's
                         // task list the attempt belongs to (parallel same-agent
                         // failures stay attributable without leaking task text).
@@ -727,7 +752,10 @@ export function makeSubagentTool(deps = {}) {
                 // The verification agent's contract is a `## Verdict` header; an
                 // output without it is not a verdict, and the driver must not treat
                 // a truncated probe log as "HOLDS".
-                const missingVerdict = o.def.source === "builtin" && o.agent === "verification" && o.result.exitCode === 0 && !output.includes("## Verdict")
+                const missingVerdict = o.def.source === "builtin" &&
+                    o.agent === "verification" &&
+                    o.result.exitCode === 0 &&
+                    !output.includes("## Verdict")
                     ? "\n\n(no verdict — the verification run did not produce its verdict section)"
                     : "";
                 return outcomes.length === 1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yagni-app/code-staging",
-  "version": "1.2.13-staging.2171.1",
+  "version": "1.2.13-staging.2173.1",
   "description": "YAGNI Code: a terminal coding agent that already knows your company. One YAGNI login routes the model and grounds the agent in your team's context.",
   "license": "SEE LICENSE IN LICENSE.md",
   "author": "YAGNI, Inc. <jack@yagni.app> (https://yagni.app)",
@@ -58,5 +58,5 @@
     "turndown": "^7.2.4",
     "typebox": "^1.3.15"
   },
-  "yagniSourceSha": "4c225e34b3364d25830d9d4d88f071ffedf799f3"
+  "yagniSourceSha": "d6206a240e12cff30f8e2e5e8975adfe67e52e83"
 }