@esso0428/pi-subagents 0.17.16 → 0.17.17

package/dist/skill-loader.js

@@ -1,94 +0,0 @@
-/**
- * skill-loader.ts — Preload named skills.
- *
- * Roots, in precedence order:
- *   - <cwd>/.pi/skills           (project, Pi's standard)
- *   - <cwd>/.agents/skills       (project, cross-tool Agent Skills spec — https://agentskills.io)
- *   - getAgentDir()/skills       (user, default ~/.pi/agent/skills — Pi's standard)
- *   - ~/.agents/skills           (user, cross-tool Agent Skills spec)
- *   - ~/.pi/skills               (legacy global, pre-Pi)
- *
- * Layout per root:
- *   - <root>/<name>.md            (flat file at the top level)
- *   - <root>/.../<name>/SKILL.md  (directory skill, may be nested — Pi's standard)
- *
- * Recursion skips dotfile entries and node_modules. A directory that itself contains
- * SKILL.md is a skill — we don't descend into it (Pi: skills don't nest).
- *
- * Symlinks are rejected for security (deviation from Pi, which follows them).
- */
-import { existsSync, readdirSync } from "node:fs";
-import { homedir } from "node:os";
-import { join } from "node:path";
-import { getAgentDir } from "@earendil-works/pi-coding-agent";
-import { isSymlink, isUnsafeName, safeReadFile } from "./memory.js";
-export function preloadSkills(skillNames, cwd) {
-    return skillNames.map((name) => ({ name, content: loadSkillContent(name, cwd) }));
-}
-function loadSkillContent(name, cwd) {
-    if (isUnsafeName(name)) {
-        return `(Skill "${name}" skipped: name contains path traversal characters)`;
-    }
-    const roots = [
-        join(cwd, ".pi", "skills"), // project — Pi standard
-        join(cwd, ".agents", "skills"), // project — Agent Skills spec
-        join(getAgentDir(), "skills"), // user — Pi standard
-        join(homedir(), ".agents", "skills"), // user — Agent Skills spec
-        join(homedir(), ".pi", "skills"), // legacy global, pre-Pi
-    ];
-    for (const root of roots) {
-        const content = findInRoot(root, name);
-        if (content !== undefined)
-            return content;
-    }
-    return `(Skill "${name}" not found in .pi/skills/, .agents/skills/, or global skill locations)`;
-}
-function findInRoot(root, name) {
-    if (isSymlink(root))
-        return undefined; // reject symlinked roots entirely
-    const flat = safeReadFile(join(root, `${name}.md`))?.trim();
-    if (flat !== undefined)
-        return flat;
-    return findSkillDirectory(root, name);
-}
-/** BFS under `root` for a directory named `name` containing `SKILL.md`. Pi-conforming filters. */
-function findSkillDirectory(root, name) {
-    if (!existsSync(root))
-        return undefined;
-    const queue = [root];
-    while (queue.length > 0) {
-        const current = queue.shift();
-        if (current === undefined)
-            continue;
-        let entries;
-        try {
-            entries = readdirSync(current, { withFileTypes: true });
-        }
-        catch {
-            continue;
-        }
-        // Deterministic byte-order traversal — locale-independent.
-        entries.sort((a, b) => (a.name < b.name ? -1 : a.name > b.name ? 1 : 0));
-        for (const entry of entries) {
-            if (!entry.isDirectory())
-                continue;
-            if (entry.name.startsWith(".") || entry.name === "node_modules")
-                continue;
-            // Symlinked dirs already filtered by entry.isDirectory() — Dirent uses lstat semantics.
-            const path = join(current, entry.name);
-            const skillMd = join(path, "SKILL.md");
-            const isSkillDir = existsSync(skillMd);
-            if (isSkillDir) {
-                if (entry.name === name) {
-                    const content = safeReadFile(skillMd)?.trim();
-                    if (content !== undefined)
-                        return content;
-                }
-                continue; // Pi rule: skills don't nest — don't descend into a skill dir
-            }
-            queue.push(path);
-        }
-    }
-    return undefined;
-}
-//# sourceMappingURL=skill-loader.js.map

package/dist/skill-loader.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"skill-loader.js","sourceRoot":"","sources":["../src/skill-loader.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;GAkBG;AAGH,OAAO,EAAE,UAAU,EAAE,WAAW,EAAE,MAAM,SAAS,CAAC;AAClD,OAAO,EAAE,OAAO,EAAE,MAAM,SAAS,CAAC;AAClC,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AACjC,OAAO,EAAE,WAAW,EAAE,MAAM,iCAAiC,CAAC;AAC9D,OAAO,EAAE,SAAS,EAAE,YAAY,EAAE,YAAY,EAAE,MAAM,aAAa,CAAC;AAOpE,MAAM,UAAU,aAAa,CAAC,UAAoB,EAAE,GAAW;IAC7D,OAAO,UAAU,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,OAAO,EAAE,gBAAgB,CAAC,IAAI,EAAE,GAAG,CAAC,EAAE,CAAC,CAAC,CAAC;AACpF,CAAC;AAED,SAAS,gBAAgB,CAAC,IAAY,EAAE,GAAW;IACjD,IAAI,YAAY,CAAC,IAAI,CAAC,EAAE,CAAC;QACvB,OAAO,WAAW,IAAI,qDAAqD,CAAC;IAC9E,CAAC;IACD,MAAM,KAAK,GAAG;QACZ,IAAI,CAAC,GAAG,EAAE,KAAK,EAAE,QAAQ,CAAC,EAAE,wBAAwB;QACpD,IAAI,CAAC,GAAG,EAAE,SAAS,EAAE,QAAQ,CAAC,EAAE,8BAA8B;QAC9D,IAAI,CAAC,WAAW,EAAE,EAAE,QAAQ,CAAC,EAAE,qBAAqB;QACpD,IAAI,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,QAAQ,CAAC,EAAE,2BAA2B;QACjE,IAAI,CAAC,OAAO,EAAE,EAAE,KAAK,EAAE,QAAQ,CAAC,EAAE,wBAAwB;KAC3D,CAAC;IACF,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,MAAM,OAAO,GAAG,UAAU,CAAC,IAAI,EAAE,IAAI,CAAC,CAAC;QACvC,IAAI,OAAO,KAAK,SAAS;YAAE,OAAO,OAAO,CAAC;IAC5C,CAAC;IACD,OAAO,WAAW,IAAI,yEAAyE,CAAC;AAClG,CAAC;AAED,SAAS,UAAU,CAAC,IAAY,EAAE,IAAY;IAC5C,IAAI,SAAS,CAAC,IAAI,CAAC;QAAE,OAAO,SAAS,CAAC,CAAC,kCAAkC;IACzE,MAAM,IAAI,GAAG,YAAY,CAAC,IAAI,CAAC,IAAI,EAAE,GAAG,IAAI,KAAK,CAAC,CAAC,EAAE,IAAI,EAAE,CAAC;IAC5D,IAAI,IAAI,KAAK,SAAS;QAAE,OAAO,IAAI,CAAC;IACpC,OAAO,kBAAkB,CAAC,IAAI,EAAE,IAAI,CAAC,CAAC;AACxC,CAAC;AAED,kGAAkG;AAClG,SAAS,kBAAkB,CAAC,IAAY,EAAE,IAAY;IACpD,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC;QAAE,OAAO,SAAS,CAAC;IACxC,MAAM,KAAK,GAAa,CAAC,IAAI,CAAC,CAAC;IAE/B,OAAO,KAAK,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACxB,MAAM,OAAO,GAAG,KAAK,CAAC,KAAK,EAAE,CAAC;QAC9B,IAAI,OAAO,KAAK,SAAS;YAAE,SAAS;QAEpC,IAAI,OAAyB,CAAC;QAC9B,IAAI,CAAC;YACH,OAAO,GAAG,WAAW,CAAC,OAAO,EAAE,EAAE,aAAa,EAAE,IAAI,EAAE,CAAC,CAAC;QAC1D,CAAC;QAAC,MAAM,CAAC;YACP,SAAS;QACX,CAAC;QAED,2DAA2D;QAC3D,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,GAAG,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,GAAG,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;QAEzE,KAAK,MAAM,KAAK,IAAI,OAAO,EAAE,CAAC;YAC5B,IAAI,CAAC,KAAK,CAAC,WAAW,EAAE;gBAAE,SAAS;YACnC,IAAI,KAAK,CAAC,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,KAAK,CAAC,IAAI,KAAK,cAAc;gBAAE,SAAS;YAE1E,wFAAwF;YACxF,MAAM,IAAI,GAAG,IAAI,CAAC,OAAO,EAAE,KAAK,CAAC,IAAI,CAAC,CAAC;YACvC,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,EAAE,UAAU,CAAC,CAAC;YACvC,MAAM,UAAU,GAAG,UAAU,CAAC,OAAO,CAAC,CAAC;YAEvC,IAAI,UAAU,EAAE,CAAC;gBACf,IAAI,KAAK,CAAC,IAAI,KAAK,IAAI,EAAE,CAAC;oBACxB,MAAM,OAAO,GAAG,YAAY,CAAC,OAAO,CAAC,EAAE,IAAI,EAAE,CAAC;oBAC9C,IAAI,OAAO,KAAK,SAAS;wBAAE,OAAO,OAAO,CAAC;gBAC5C,CAAC;gBACD,SAAS,CAAC,8DAA8D;YAC1E,CAAC;YAED,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACnB,CAAC;IACH,CAAC;IACD,OAAO,SAAS,CAAC;AACnB,CAAC"}

package/dist/status-note.d.ts

@@ -1,62 +0,0 @@
-/**
- * status-note.ts — Honest framing for an agent result: the parenthetical status
- * note for a non-normal outcome, and the salvaged partial output of a failure.
- *
- * Lives here rather than in an index.ts closure because both entry points need
- * it — the top-level tools and the nested delegation tools, which can't import
- * from index.ts (that is the extension entry, and it already reaches these tools
- * through agent-runner).
- */
-import type { AgentRecord } from "./types.js";
-/**
- * Explicit parenthetical note for a non-normal terminal outcome, so the parent
- * agent can't mistake partial output for a completed result. Empty string for a
- * clean completion (and any unknown/non-terminal status).
- *
- * `stopped` (a human aborted it) is deliberately distinct from `aborted` (the
- * turn limit was hit) — the parent should treat human intervention differently
- * from a budget cutoff.
- */
-export declare function getStatusNote(status: string): string;
-/**
- * Foreground variant of `getStatusNote`. A foreground caller is in a different
- * position from a background one, so it needs different text:
- *
- *   - It already holds the agent's ENTIRE output inline, whereas the background
- *     notification carries a 500-char preview. So only here can we truthfully
- *     say there is nothing more to fetch — which is the whole point, because
- *   - it has no agent id. The id travels in the tool result's renderer
- *     `details`, which is never serialized to the model. A parent that reads
- *     "output may be partial" as "truncated, go retrieve the rest" therefore
- *     has nothing valid to call `get_subagent_result` with, and will invent an
- *     id (#174).
- *
- * Only the lead clause varies between the three, and each variation carries
- * information: `wrapped up` vs `aborted` tells the parent whether the output is
- * a considered final answer or a fragment, and `stopped` shouts because a human
- * intervening outranks everything else in the string. Only `steered` hedges on
- * completion — it was told to wrap up and did, so it may well have finished at
- * the limit; an aborted run blew through its grace turns while still working,
- * and `stopped` can only fire on a running agent, so neither ever delivered a
- * final answer. Identical confidence gets identical wording: phrasing one fact
- * two ways invites a hunt for a distinction that isn't there.
- *
- * Every clause is a statement about state, never an instruction to act, and
- * `get_subagent_result` is never named — naming the tool we steer away from only
- * raises its salience. Two instructions were tried here and cut: "re-spawn with
- * a higher max_turns" (pushes a fresh multi-minute run to save one wasted tool
- * call) and, on `stopped`, "ask before restarting it" (restates the lead, and
- * presumes someone is present to ask — false under `pi -p`, in scheduled jobs,
- * and in any background-driven run). Nothing here can measure whether wording
- * improves parent behavior, so removing a false cue (which cannot induce new
- * behavior) and adding an instruction (which can) are not equally safe bets.
- * Don't add either back without a way to measure it.
- */
-export declare function getForegroundOutcomeNote(status: string): string;
-/**
- * Salvaged partial output of a failed run, as a labeled suffix for the error
- * surfaces (or "" if the run produced nothing). `record.result` is bounded to
- * the run's own turns, so this is never a stale earlier answer (#144).
- */
-export declare function partialOutputSuffix(record: AgentRecord): string;
-//# sourceMappingURL=status-note.d.ts.map

package/dist/status-note.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"status-note.d.ts","sourceRoot":"","sources":["../src/status-note.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,YAAY,CAAC;AAE9C;;;;;;;;GAQG;AACH,wBAAgB,aAAa,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM,CAWpD;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAiCG;AACH,wBAAgB,wBAAwB,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM,CAW/D;AAED;;;;GAIG;AACH,wBAAgB,mBAAmB,CAAC,MAAM,EAAE,WAAW,GAAG,MAAM,CAG/D"}

package/dist/status-note.js

@@ -1,86 +0,0 @@
-/**
- * status-note.ts — Honest framing for an agent result: the parenthetical status
- * note for a non-normal outcome, and the salvaged partial output of a failure.
- *
- * Lives here rather than in an index.ts closure because both entry points need
- * it — the top-level tools and the nested delegation tools, which can't import
- * from index.ts (that is the extension entry, and it already reaches these tools
- * through agent-runner).
- */
-/**
- * Explicit parenthetical note for a non-normal terminal outcome, so the parent
- * agent can't mistake partial output for a completed result. Empty string for a
- * clean completion (and any unknown/non-terminal status).
- *
- * `stopped` (a human aborted it) is deliberately distinct from `aborted` (the
- * turn limit was hit) — the parent should treat human intervention differently
- * from a budget cutoff.
- */
-export function getStatusNote(status) {
-    switch (status) {
-        case "stopped":
-            return " (STOPPED BY THE USER before completion — output is partial; the task was NOT finished)";
-        case "aborted":
-            return " (aborted — hit the turn limit before completion; output may be incomplete)";
-        case "steered":
-            return " (wrapped up at the turn limit — output may be partial)";
-        default:
-            return "";
-    }
-}
-/**
- * Foreground variant of `getStatusNote`. A foreground caller is in a different
- * position from a background one, so it needs different text:
- *
- *   - It already holds the agent's ENTIRE output inline, whereas the background
- *     notification carries a 500-char preview. So only here can we truthfully
- *     say there is nothing more to fetch — which is the whole point, because
- *   - it has no agent id. The id travels in the tool result's renderer
- *     `details`, which is never serialized to the model. A parent that reads
- *     "output may be partial" as "truncated, go retrieve the rest" therefore
- *     has nothing valid to call `get_subagent_result` with, and will invent an
- *     id (#174).
- *
- * Only the lead clause varies between the three, and each variation carries
- * information: `wrapped up` vs `aborted` tells the parent whether the output is
- * a considered final answer or a fragment, and `stopped` shouts because a human
- * intervening outranks everything else in the string. Only `steered` hedges on
- * completion — it was told to wrap up and did, so it may well have finished at
- * the limit; an aborted run blew through its grace turns while still working,
- * and `stopped` can only fire on a running agent, so neither ever delivered a
- * final answer. Identical confidence gets identical wording: phrasing one fact
- * two ways invites a hunt for a distinction that isn't there.
- *
- * Every clause is a statement about state, never an instruction to act, and
- * `get_subagent_result` is never named — naming the tool we steer away from only
- * raises its salience. Two instructions were tried here and cut: "re-spawn with
- * a higher max_turns" (pushes a fresh multi-minute run to save one wasted tool
- * call) and, on `stopped`, "ask before restarting it" (restates the lead, and
- * presumes someone is present to ask — false under `pi -p`, in scheduled jobs,
- * and in any background-driven run). Nothing here can measure whether wording
- * improves parent behavior, so removing a false cue (which cannot induce new
- * behavior) and adding an instruction (which can) are not equally safe bets.
- * Don't add either back without a way to measure it.
- */
-export function getForegroundOutcomeNote(status) {
-    switch (status) {
-        case "stopped":
-            return " (STOPPED BY THE USER — everything the agent produced is above; the task is unfinished)";
-        case "aborted":
-            return " (aborted at the turn limit — everything the agent produced is above; the task is unfinished)";
-        case "steered":
-            return " (wrapped up at the turn limit — everything the agent produced is above; the task may be unfinished)";
-        default:
-            return "";
-    }
-}
-/**
- * Salvaged partial output of a failed run, as a labeled suffix for the error
- * surfaces (or "" if the run produced nothing). `record.result` is bounded to
- * the run's own turns, so this is never a stale earlier answer (#144).
- */
-export function partialOutputSuffix(record) {
-    const partial = record.result?.trim();
-    return partial ? `\n\nPartial output before the failure:\n${partial}` : "";
-}
-//# sourceMappingURL=status-note.js.map

package/dist/status-note.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"status-note.js","sourceRoot":"","sources":["../src/status-note.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAIH;;;;;;;;GAQG;AACH,MAAM,UAAU,aAAa,CAAC,MAAc;IAC1C,QAAQ,MAAM,EAAE,CAAC;QACf,KAAK,SAAS;YACZ,OAAO,yFAAyF,CAAC;QACnG,KAAK,SAAS;YACZ,OAAO,6EAA6E,CAAC;QACvF,KAAK,SAAS;YACZ,OAAO,yDAAyD,CAAC;QACnE;YACE,OAAO,EAAE,CAAC;IACd,CAAC;AACH,CAAC;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAiCG;AACH,MAAM,UAAU,wBAAwB,CAAC,MAAc;IACrD,QAAQ,MAAM,EAAE,CAAC;QACf,KAAK,SAAS;YACZ,OAAO,yFAAyF,CAAC;QACnG,KAAK,SAAS;YACZ,OAAO,+FAA+F,CAAC;QACzG,KAAK,SAAS;YACZ,OAAO,sGAAsG,CAAC;QAChH;YACE,OAAO,EAAE,CAAC;IACd,CAAC;AACH,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,mBAAmB,CAAC,MAAmB;IACrD,MAAM,OAAO,GAAG,MAAM,CAAC,MAAM,EAAE,IAAI,EAAE,CAAC;IACtC,OAAO,OAAO,CAAC,CAAC,CAAC,2CAA2C,OAAO,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;AAC7E,CAAC"}

package/dist/structured-output.d.ts

@@ -1,62 +0,0 @@
-/**
- * structured-output.ts — the synthetic tool behind `agent(prompt, { schema })`.
- *
- * A workflow script that passes a `schema` wants an *object* back, not prose it
- * has to parse. Claude Code does this by giving the child a `StructuredOutput`
- * tool whose input schema is the caller's schema, so the provider fills the
- * fields, and returning the validated payload as the agent's result.
- *
- * We do the same, with one gap named up front: Claude Code *forces* the call,
- * and we cannot. `toolChoice` exists in pi-ai's provider layer but is not
- * plumbed through `AgentSession`, so an extension has no way to require a
- * particular tool. What we have instead is three softer pressures —
- *
- *   1. `constrainedSampling`, so providers that support it hold the payload to
- *      the schema at sampling time;
- *   2. the tool's description, snippet and guideline, which say the answer must
- *      come through this call;
- *   3. validation here, answering a bad payload with `isError` so the model
- *      sees what was wrong and calls again inside the same run.
- *
- * — and, when all three fail, one more prompt from `runAgent`. See
- * {@link structuredRetryPrompt}.
- *
- * The name matches Claude Code's exactly, so a ported prompt that mentions
- * `StructuredOutput` is still telling the truth.
- */
-import { type ToolDefinition } from "@earendil-works/pi-coding-agent";
-import type { CompiledSchema } from "./workflow/json-schema.js";
-/**
- * Deliberately NOT added to `SUBAGENT_TOOL_NAMES`: that list becomes
- * `EXCLUDED_TOOL_NAMES`, which is exactly the denial this tool has to avoid.
- * Nor to `BUILTIN_TOOL_NAMES` — it is ours to inject, never a name a user may
- * ask for in an agent's `tools:` frontmatter.
- */
-export declare const STRUCTURED_OUTPUT_TOOL_NAME = "StructuredOutput";
-/** What the child produced, filled in as the tool is called. */
-export interface StructuredCapture {
-    /** The last payload that validated, canonicalised. Absent until one does. */
-    json?: string;
-    /** Why the most recent attempt was rejected, for the retry prompt. */
-    lastError?: string;
-    /** Whether the tool was called at all — "never tried" reads differently. */
-    called: boolean;
-}
-export declare function createStructuredCapture(): StructuredCapture;
-/**
- * Build the tool for one child.
- *
- * `capture` is the box the caller reads afterwards. It is passed in rather than
- * returned so `runAgent` owns its lifetime and can consult it on every exit
- * path, including the ones where the tool was never reached.
- */
-export declare function createStructuredOutputTool(compiled: CompiledSchema, capture: StructuredCapture): ToolDefinition;
-/**
- * The one extra prompt sent when a run ended with nothing captured.
- *
- * Distinguishes "never called it" from "called it wrongly" — the two need
- * different corrections, and telling a model it got the shape wrong when it
- * never answered at all sends it looking for a mistake it did not make.
- */
-export declare function structuredRetryPrompt(capture: StructuredCapture): string;
-//# sourceMappingURL=structured-output.d.ts.map

package/dist/structured-output.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"structured-output.d.ts","sourceRoot":"","sources":["../src/structured-output.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;GAyBG;AAEH,OAAO,EAAc,KAAK,cAAc,EAAE,MAAM,iCAAiC,CAAC;AAClF,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,2BAA2B,CAAC;AAEhE;;;;;GAKG;AACH,eAAO,MAAM,2BAA2B,qBAAqB,CAAC;AAE9D,gEAAgE;AAChE,MAAM,WAAW,iBAAiB;IAChC,6EAA6E;IAC7E,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,sEAAsE;IACtE,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,4EAA4E;IAC5E,MAAM,EAAE,OAAO,CAAC;CACjB;AAED,wBAAgB,uBAAuB,IAAI,iBAAiB,CAE3D;AAED;;;;;;GAMG;AACH,wBAAgB,0BAA0B,CACxC,QAAQ,EAAE,cAAc,EACxB,OAAO,EAAE,iBAAiB,GACzB,cAAc,CAqDhB;AAED;;;;;;GAMG;AACH,wBAAgB,qBAAqB,CAAC,OAAO,EAAE,iBAAiB,GAAG,MAAM,CAKxE"}

package/dist/structured-output.js

@@ -1,113 +0,0 @@
-/**
- * structured-output.ts — the synthetic tool behind `agent(prompt, { schema })`.
- *
- * A workflow script that passes a `schema` wants an *object* back, not prose it
- * has to parse. Claude Code does this by giving the child a `StructuredOutput`
- * tool whose input schema is the caller's schema, so the provider fills the
- * fields, and returning the validated payload as the agent's result.
- *
- * We do the same, with one gap named up front: Claude Code *forces* the call,
- * and we cannot. `toolChoice` exists in pi-ai's provider layer but is not
- * plumbed through `AgentSession`, so an extension has no way to require a
- * particular tool. What we have instead is three softer pressures —
- *
- *   1. `constrainedSampling`, so providers that support it hold the payload to
- *      the schema at sampling time;
- *   2. the tool's description, snippet and guideline, which say the answer must
- *      come through this call;
- *   3. validation here, answering a bad payload with `isError` so the model
- *      sees what was wrong and calls again inside the same run.
- *
- * — and, when all three fail, one more prompt from `runAgent`. See
- * {@link structuredRetryPrompt}.
- *
- * The name matches Claude Code's exactly, so a ported prompt that mentions
- * `StructuredOutput` is still telling the truth.
- */
-import { defineTool } from "@earendil-works/pi-coding-agent";
-/**
- * Deliberately NOT added to `SUBAGENT_TOOL_NAMES`: that list becomes
- * `EXCLUDED_TOOL_NAMES`, which is exactly the denial this tool has to avoid.
- * Nor to `BUILTIN_TOOL_NAMES` — it is ours to inject, never a name a user may
- * ask for in an agent's `tools:` frontmatter.
- */
-export const STRUCTURED_OUTPUT_TOOL_NAME = "StructuredOutput";
-export function createStructuredCapture() {
-    return { called: false };
-}
-/**
- * Build the tool for one child.
- *
- * `capture` is the box the caller reads afterwards. It is passed in rather than
- * returned so `runAgent` owns its lifetime and can consult it on every exit
- * path, including the ones where the tool was never reached.
- */
-export function createStructuredOutputTool(compiled, capture) {
-    return defineTool({
-        name: STRUCTURED_OUTPUT_TOOL_NAME,
-        label: "Structured Output",
-        description: "Report your final answer. Call this exactly once, with the complete result, and put everything the "
-            + "caller needs inside the arguments — text written outside this call is discarded. If a call is "
-            + "rejected for not matching the schema, fix the reported fields and call it again.",
-        promptSnippet: "Report your final answer as structured data",
-        promptGuidelines: [
-            "Your final answer MUST be reported by calling StructuredOutput. Prose outside that call is discarded.",
-        ],
-        // The caller's schema *is* the tool's input schema, verbatim — that is what
-        // makes the provider fill the fields. pi types this as TypeBox's `TSchema`,
-        // which v1 defines as an open interface, so a plain JSON Schema satisfies
-        // it without a cast at runtime or a conversion at author time.
-        parameters: compiled.schema,
-        // "prefer", not "require": a provider that cannot constrain sampling should
-        // fall through to validation-and-retry rather than fail the call outright.
-        constrainedSampling: { type: "json_schema", strict: "prefer" },
-        // Models occasionally send the whole payload as one JSON string instead of
-        // an object. Recovering that costs nothing and saves a whole retry.
-        prepareArguments: (args) => {
-            if (typeof args !== "string")
-                return args;
-            try {
-                return JSON.parse(args);
-            }
-            catch {
-                return args;
-            }
-        },
-        execute: async (_toolCallId, params) => {
-            capture.called = true;
-            const verdict = compiled.check(params);
-            if (verdict !== true) {
-                capture.lastError = verdict;
-                // `isError` puts the reason in front of the model as a tool result, so
-                // it can correct itself inside this same run. This is where most
-                // mismatches are resolved; the prompt-level retry is the backstop.
-                return {
-                    content: [{
-                            type: "text",
-                            text: `StructuredOutput did not match the required schema:\n${verdict}\nCall it again with a corrected value.`,
-                        }],
-                    isError: true,
-                    details: {},
-                };
-            }
-            // Last valid call wins: a model that calls twice meant the second one.
-            capture.json = JSON.stringify(params);
-            capture.lastError = undefined;
-            return { content: [{ type: "text", text: "Recorded." }], details: {} };
-        },
-    });
-}
-/**
- * The one extra prompt sent when a run ended with nothing captured.
- *
- * Distinguishes "never called it" from "called it wrongly" — the two need
- * different corrections, and telling a model it got the shape wrong when it
- * never answered at all sends it looking for a mistake it did not make.
- */
-export function structuredRetryPrompt(capture) {
-    const reason = capture.called && capture.lastError !== undefined
-        ? `Your last ${STRUCTURED_OUTPUT_TOOL_NAME} call did not match the required schema: ${capture.lastError}`
-        : `You did not call ${STRUCTURED_OUTPUT_TOOL_NAME}, so your answer was not recorded.`;
-    return `${reason}\n\nCall ${STRUCTURED_OUTPUT_TOOL_NAME} now with your complete final answer. Do not reply with prose.`;
-}
-//# sourceMappingURL=structured-output.js.map

package/dist/structured-output.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"structured-output.js","sourceRoot":"","sources":["../src/structured-output.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;GAyBG;AAEH,OAAO,EAAE,UAAU,EAAuB,MAAM,iCAAiC,CAAC;AAGlF;;;;;GAKG;AACH,MAAM,CAAC,MAAM,2BAA2B,GAAG,kBAAkB,CAAC;AAY9D,MAAM,UAAU,uBAAuB;IACrC,OAAO,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC;AAC3B,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,0BAA0B,CACxC,QAAwB,EACxB,OAA0B;IAE1B,OAAO,UAAU,CAAC;QAChB,IAAI,EAAE,2BAA2B;QACjC,KAAK,EAAE,mBAAmB;QAC1B,WAAW,EACT,qGAAqG;cACnG,gGAAgG;cAChG,kFAAkF;QACtF,aAAa,EAAE,6CAA6C;QAC5D,gBAAgB,EAAE;YAChB,uGAAuG;SACxG;QACD,4EAA4E;QAC5E,4EAA4E;QAC5E,0EAA0E;QAC1E,+DAA+D;QAC/D,UAAU,EAAE,QAAQ,CAAC,MAAe;QACpC,4EAA4E;QAC5E,2EAA2E;QAC3E,mBAAmB,EAAE,EAAE,IAAI,EAAE,aAAa,EAAE,MAAM,EAAE,QAAQ,EAAE;QAC9D,2EAA2E;QAC3E,oEAAoE;QACpE,gBAAgB,EAAE,CAAC,IAAa,EAAE,EAAE;YAClC,IAAI,OAAO,IAAI,KAAK,QAAQ;gBAAE,OAAO,IAAa,CAAC;YACnD,IAAI,CAAC;gBACH,OAAO,IAAI,CAAC,KAAK,CAAC,IAAI,CAAU,CAAC;YACnC,CAAC;YAAC,MAAM,CAAC;gBACP,OAAO,IAAa,CAAC;YACvB,CAAC;QACH,CAAC;QACD,OAAO,EAAE,KAAK,EAAE,WAAW,EAAE,MAAM,EAAE,EAAE;YACrC,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC;YACtB,MAAM,OAAO,GAAG,QAAQ,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC;YACvC,IAAI,OAAO,KAAK,IAAI,EAAE,CAAC;gBACrB,OAAO,CAAC,SAAS,GAAG,OAAO,CAAC;gBAC5B,uEAAuE;gBACvE,iEAAiE;gBACjE,mEAAmE;gBACnE,OAAO;oBACL,OAAO,EAAE,CAAC;4BACR,IAAI,EAAE,MAAM;4BACZ,IAAI,EAAE,wDAAwD,OAAO,yCAAyC;yBAC/G,CAAC;oBACF,OAAO,EAAE,IAAI;oBACb,OAAO,EAAE,EAAE;iBACZ,CAAC;YACJ,CAAC;YACD,uEAAuE;YACvE,OAAO,CAAC,IAAI,GAAG,IAAI,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC;YACtC,OAAO,CAAC,SAAS,GAAG,SAAS,CAAC;YAC9B,OAAO,EAAE,OAAO,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,WAAW,EAAE,CAAC,EAAE,OAAO,EAAE,EAAE,EAAE,CAAC;QACzE,CAAC;KACF,CAAmB,CAAC;AACvB,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,qBAAqB,CAAC,OAA0B;IAC9D,MAAM,MAAM,GAAG,OAAO,CAAC,MAAM,IAAI,OAAO,CAAC,SAAS,KAAK,SAAS;QAC9D,CAAC,CAAC,aAAa,2BAA2B,4CAA4C,OAAO,CAAC,SAAS,EAAE;QACzG,CAAC,CAAC,oBAAoB,2BAA2B,oCAAoC,CAAC;IACxF,OAAO,GAAG,MAAM,YAAY,2BAA2B,gEAAgE,CAAC;AAC1H,CAAC"}