@adhd/agent-mcp 2.0.1 → 2.1.0

package/src/clients/tool-naming.js

@@ -1,65 +0,0 @@
-/**
- * Tool-name encoding shared across the registry and the orchestrator.
- *
- * Tools are advertised to the model as `<server>__<tool>`. Many OpenAI-compatible
- * and local models (e.g. LM Studio / qwen) restrict function names to
- * `[A-Za-z0-9_]` and silently rewrite other characters — notably `-` → `_` — so
- * an advertised `agent-mcp__agent` comes back from the model as
- * `agent_mcp__agent`. A literal lookup then fails ("No MCP server config found
- * for server: 'agent_mcp'"), which breaks recursion (the `agent-mcp` key) and any
- * MCP server whose name contains a hyphen. Anthropic preserves the name, so this
- * only bites the OpenAI-compatible providers — but the round-trip must be robust
- * for ALL of them.
- *
- * The fix: index both the advertised name and its normalized form, and normalize
- * the model's returned name the same way before resolving.
- */
-export const TOOL_NAME_SEPARATOR = "__";
-/** Normalize a tool name to the charset models restrict function names to. */
-export function normalizeToolName(name) {
-    return name.replace(/[^A-Za-z0-9_]/g, "_");
-}
-/** Split a name on the first separator into { server, tool } (no resolution). */
-function splitToolName(name) {
-    const i = name.indexOf(TOOL_NAME_SEPARATOR);
-    return i === -1
-        ? { server: name, tool: name }
-        : { server: name.slice(0, i), tool: name.slice(i + TOOL_NAME_SEPARATOR.length) };
-}
-/**
- * Resolve a tool name AS RETURNED BY A MODEL into { server, tool }.
- *
- * - A qualified `<server>__<tool>` name splits on the separator (prior behavior).
- * - A **bare** `<tool>` name (no separator) is resolved against the tool names the
- *   model was actually advertised: if exactly one advertised tool has that
- *   tool-part it is used; if several do, an actionable error lists the qualified
- *   candidates; if none do, it falls back to a literal split so downstream
- *   "unknown tool / no server config" handling still applies.
- *
- * Background (BACKLOG DEBT-004): capable models frequently emit the bare tool
- * name (`agent` / `task`) instead of the advertised `agent-mcp__agent`, which used
- * to hard-fail the whole task with "Invalid tool name (missing server prefix)".
- * Since the separator is unambiguous in advertised names, a bare name that maps to
- * exactly one advertised tool can be resolved deterministically.
- *
- * @param rawName    the tool name the model returned
- * @param advertised the advertised tool names the model was given (`<server>__<tool>`)
- */
-export function resolveToolCallName(rawName, advertised = []) {
-    if (rawName.includes(TOOL_NAME_SEPARATOR))
-        return splitToolName(rawName);
-    const norm = normalizeToolName(rawName);
-    const candidates = Array.from(new Set(advertised)).filter((a) => {
-        const { tool } = splitToolName(a);
-        return tool === rawName || normalizeToolName(tool) === norm;
-    });
-    if (candidates.length === 1)
-        return splitToolName(candidates[0]);
-    if (candidates.length > 1) {
-        throw new Error(`Ambiguous tool name '${rawName}': multiple servers expose it — qualify with a server prefix, one of: ${candidates.join(", ")}`);
-    }
-    // No advertised match — preserve prior split (server = tool = rawName); the
-    // registry then surfaces a normal "no server config" / unknown-tool error.
-    return splitToolName(rawName);
-}
-//# sourceMappingURL=tool-naming.js.map

package/src/clients/tool-naming.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"tool-naming.js","sourceRoot":"","sources":["../../../../../../packages/ai/agent-mcp/src/clients/tool-naming.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AACH,MAAM,CAAC,MAAM,mBAAmB,GAAG,IAAI,CAAC;AAExC,8EAA8E;AAC9E,MAAM,UAAU,iBAAiB,CAAC,IAAY;IAC1C,OAAO,IAAI,CAAC,OAAO,CAAC,gBAAgB,EAAE,GAAG,CAAC,CAAC;AAC/C,CAAC;AAOD,iFAAiF;AACjF,SAAS,aAAa,CAAC,IAAY;IAC/B,MAAM,CAAC,GAAG,IAAI,CAAC,OAAO,CAAC,mBAAmB,CAAC,CAAC;IAC5C,OAAO,CAAC,KAAK,CAAC,CAAC;QACX,CAAC,CAAC,EAAE,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,EAAE;QAC9B,CAAC,CAAC,EAAE,MAAM,EAAE,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,IAAI,CAAC,KAAK,CAAC,CAAC,GAAG,mBAAmB,CAAC,MAAM,CAAC,EAAE,CAAC;AACzF,CAAC;AAED;;;;;;;;;;;;;;;;;;GAkBG;AACH,MAAM,UAAU,mBAAmB,CAAC,OAAe,EAAE,aAAgC,EAAE;IACnF,IAAI,OAAO,CAAC,QAAQ,CAAC,mBAAmB,CAAC;QAAE,OAAO,aAAa,CAAC,OAAO,CAAC,CAAC;IAEzE,MAAM,IAAI,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAC;IACxC,MAAM,UAAU,GAAG,KAAK,CAAC,IAAI,CAAC,IAAI,GAAG,CAAC,UAAU,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE;QAC5D,MAAM,EAAE,IAAI,EAAE,GAAG,aAAa,CAAC,CAAC,CAAC,CAAC;QAClC,OAAO,IAAI,KAAK,OAAO,IAAI,iBAAiB,CAAC,IAAI,CAAC,KAAK,IAAI,CAAC;IAChE,CAAC,CAAC,CAAC;IAEH,IAAI,UAAU,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,aAAa,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,CAAC;IACjE,IAAI,UAAU,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACxB,MAAM,IAAI,KAAK,CACX,wBAAwB,OAAO,yEAAyE,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAClI,CAAC;IACN,CAAC;IACD,4EAA4E;IAC5E,2EAA2E;IAC3E,OAAO,aAAa,CAAC,OAAO,CAAC,CAAC;AAClC,CAAC"}

package/src/clients/types.d.ts

@@ -1,18 +0,0 @@
-import type { ToolDefinition } from "../providers/types.js";
-export interface IMcpClient {
-    /** Returns all tools advertised by this MCP server */
-    listTools(): Promise<ToolDefinition[]>;
-    /**
-     * Call a single tool on this server.
-     * @param toolName  the tool name (without server prefix)
-     * @param args      the arguments to pass
-     * @param signal    optional abort signal — when the task is cancelled (or
-     *                  times out) this fires, so an in-flight tool call is
-     *                  interrupted instead of running to completion. Composed
-     *                  with any client-level timeout.
-     */
-    callTool(toolName: string, args: unknown, signal?: AbortSignal): Promise<unknown>;
-    /** Tear down the client (kill child processes, close connections) */
-    close(): Promise<void>;
-}
-//# sourceMappingURL=types.d.ts.map

package/src/clients/types.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../../../../../../packages/ai/agent-mcp/src/clients/types.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,uBAAuB,CAAC;AAE5D,MAAM,WAAW,UAAU;IACvB,sDAAsD;IACtD,SAAS,IAAI,OAAO,CAAC,cAAc,EAAE,CAAC,CAAC;IAEvC;;;;;;;;OAQG;IACH,QAAQ,CAAC,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,OAAO,EAAE,MAAM,CAAC,EAAE,WAAW,GAAG,OAAO,CAAC,OAAO,CAAC,CAAC;IAElF,qEAAqE;IACrE,KAAK,IAAI,OAAO,CAAC,IAAI,CAAC,CAAC;CAC1B"}

package/src/clients/types.js

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=types.js.map

package/src/clients/types.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"types.js","sourceRoot":"","sources":["../../../../../../packages/ai/agent-mcp/src/clients/types.ts"],"names":[],"mappings":""}

package/src/engine/dag-engine.d.ts

@@ -1,62 +0,0 @@
-import type { BetterSQLite3Database } from "drizzle-orm/better-sqlite3";
-import type { TaskStore } from "../store/task-store.js";
-import type { BackgroundQueue } from "./queue.js";
-/**
- * DagEngine manages task dependency resolution.
- *
- * Two public methods:
- * - `validateNoCycle()` — synchronous BFS cycle check; called BEFORE row insert.
- * - `dispatchReady()` — post-terminal fan-in dispatch; called in the queue runFn
- *   `finally` block so it fires on every terminal event (completed/failed/cancelled).
- *
- * The `dispatchFn` is injected at server startup (built in src/index.ts) to avoid
- * circular imports: DagEngine must NOT import from tools/task.ts.
- */
-export declare class DagEngine {
-    private readonly db;
-    private readonly queue;
-    private readonly taskStore;
-    /**
-     * Injected at server startup to avoid circular imports with tools/task.ts.
-     * Signature: (taskId: string) => Promise<void>
-     * Built in index.ts by closing over the TaskDeps.
-     */
-    private readonly dispatchFn;
-    constructor(db: BetterSQLite3Database<Record<string, never>>, queue: BackgroundQueue, taskStore: TaskStore, 
-    /**
-     * Injected at server startup to avoid circular imports with tools/task.ts.
-     * Signature: (taskId: string) => Promise<void>
-     * Built in index.ts by closing over the TaskDeps.
-     */
-    dispatchFn: (taskId: string) => Promise<void>);
-    /**
-     * BFS cycle detection. Call BEFORE inserting the new task row.
-     *
-     * Walks the existing dependency graph upward from each member of `dependsOn`.
-     * If `newTaskId` appears as an ancestor, adding it would create a cycle.
-     *
-     * Throws ToolError("VALIDATION_ERROR") if a cycle is detected.
-     * Does NOT write any rows — purely a read operation.
-     *
-     * Invariant: [inv:cycle-check-synchronous]
-     */
-    validateNoCycle(newTaskId: string, dependsOn: string[]): void;
-    /**
-     * Called after a task reaches a terminal state (completed/failed/cancelled).
-     *
-     * Scans all `waiting` tasks for those that include `completedTaskId` in their
-     * `depends_on`. For each:
-     *   - If all deps are in a terminal state, evaluate `on_upstream_failure`:
-     *     - "fail" (default): mark the downstream as failed immediately.
-     *     - "skip": dispatch anyway; only include completed upstreams in `inputs`.
-     *
-     * Uses optimistic locking (`AND status='waiting'` on the UPDATE) to prevent
-     * double-enqueue when two concurrent terminal events race for the same fan-in
-     * task.
-     *
-     * Invariants: [inv:waiting-no-queue], [inv:dispatch-on-completion],
-     *             [inv:inputs-populated-at-dispatch]
-     */
-    dispatchReady(completedTaskId: string): Promise<void>;
-}
-//# sourceMappingURL=dag-engine.d.ts.map

package/src/engine/dag-engine.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"dag-engine.d.ts","sourceRoot":"","sources":["../../../../../../packages/ai/agent-mcp/src/engine/dag-engine.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,4BAA4B,CAAC;AAIxE,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,wBAAwB,CAAC;AACxD,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,YAAY,CAAC;AAIlD;;;;;;;;;;GAUG;AACH,qBAAa,SAAS;IAEd,OAAO,CAAC,QAAQ,CAAC,EAAE;IACnB,OAAO,CAAC,QAAQ,CAAC,KAAK;IACtB,OAAO,CAAC,QAAQ,CAAC,SAAS;IAC1B;;;;OAIG;IACH,OAAO,CAAC,QAAQ,CAAC,UAAU;gBARV,EAAE,EAAE,qBAAqB,CAAC,MAAM,CAAC,MAAM,EAAE,KAAK,CAAC,CAAC,EAChD,KAAK,EAAE,eAAe,EACtB,SAAS,EAAE,SAAS;IACrC;;;;OAIG;IACc,UAAU,EAAE,CAAC,MAAM,EAAE,MAAM,KAAK,OAAO,CAAC,IAAI,CAAC;IAGlE;;;;;;;;;;OAUG;IACH,eAAe,CAAC,SAAS,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,EAAE,GAAG,IAAI;IAgC7D;;;;;;;;;;;;;;;OAeG;IACG,aAAa,CAAC,eAAe,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;CAkG9D"}

package/src/engine/dag-engine.js

@@ -1,158 +0,0 @@
-import { and, eq, inArray } from "drizzle-orm";
-import { tasksTable } from "../db/schema.js";
-import { logger } from "../logger.js";
-import { ToolError } from "../validation/errors.js";
-import { nowIso } from "../utils/timestamps.js";
-/**
- * DagEngine manages task dependency resolution.
- *
- * Two public methods:
- * - `validateNoCycle()` — synchronous BFS cycle check; called BEFORE row insert.
- * - `dispatchReady()` — post-terminal fan-in dispatch; called in the queue runFn
- *   `finally` block so it fires on every terminal event (completed/failed/cancelled).
- *
- * The `dispatchFn` is injected at server startup (built in src/index.ts) to avoid
- * circular imports: DagEngine must NOT import from tools/task.ts.
- */
-export class DagEngine {
-    db;
-    queue;
-    taskStore;
-    dispatchFn;
-    constructor(db, queue, taskStore, 
-    /**
-     * Injected at server startup to avoid circular imports with tools/task.ts.
-     * Signature: (taskId: string) => Promise<void>
-     * Built in index.ts by closing over the TaskDeps.
-     */
-    dispatchFn) {
-        this.db = db;
-        this.queue = queue;
-        this.taskStore = taskStore;
-        this.dispatchFn = dispatchFn;
-    }
-    /**
-     * BFS cycle detection. Call BEFORE inserting the new task row.
-     *
-     * Walks the existing dependency graph upward from each member of `dependsOn`.
-     * If `newTaskId` appears as an ancestor, adding it would create a cycle.
-     *
-     * Throws ToolError("VALIDATION_ERROR") if a cycle is detected.
-     * Does NOT write any rows — purely a read operation.
-     *
-     * Invariant: [inv:cycle-check-synchronous]
-     */
-    validateNoCycle(newTaskId, dependsOn) {
-        const visited = new Set();
-        const queue = [...dependsOn];
-        while (queue.length > 0) {
-            const id = queue.shift();
-            if (id === newTaskId) {
-                throw new ToolError("VALIDATION_ERROR", `Dependency cycle detected: task ${newTaskId} would depend on itself via its dependency chain`);
-            }
-            if (!visited.has(id)) {
-                visited.add(id);
-                const row = this.db
-                    .select({ dependsOn: tasksTable.depends_on })
-                    .from(tasksTable)
-                    .where(eq(tasksTable.id, id))
-                    .get();
-                const upstreamDeps = row?.dependsOn
-                    ? JSON.parse(row.dependsOn)
-                    : [];
-                queue.push(...upstreamDeps);
-            }
-        }
-    }
-    /**
-     * Called after a task reaches a terminal state (completed/failed/cancelled).
-     *
-     * Scans all `waiting` tasks for those that include `completedTaskId` in their
-     * `depends_on`. For each:
-     *   - If all deps are in a terminal state, evaluate `on_upstream_failure`:
-     *     - "fail" (default): mark the downstream as failed immediately.
-     *     - "skip": dispatch anyway; only include completed upstreams in `inputs`.
-     *
-     * Uses optimistic locking (`AND status='waiting'` on the UPDATE) to prevent
-     * double-enqueue when two concurrent terminal events race for the same fan-in
-     * task.
-     *
-     * Invariants: [inv:waiting-no-queue], [inv:dispatch-on-completion],
-     *             [inv:inputs-populated-at-dispatch]
-     */
-    async dispatchReady(completedTaskId) {
-        // Fetch all waiting tasks in a single query
-        const waitingTasks = this.db
-            .select()
-            .from(tasksTable)
-            .where(eq(tasksTable.status, "waiting"))
-            .all();
-        for (const task of waitingTasks) {
-            const deps = task.depends_on
-                ? JSON.parse(task.depends_on)
-                : [];
-            // Skip if this task doesn't depend on the just-completed task
-            if (!deps.includes(completedTaskId))
-                continue;
-            // Nothing to dispatch if there are no deps (should not happen for
-            // waiting tasks, but guard defensively)
-            if (deps.length === 0)
-                continue;
-            // Check all deps for terminal status
-            const depRows = this.db
-                .select({
-                id: tasksTable.id,
-                status: tasksTable.status,
-                result: tasksTable.result,
-            })
-                .from(tasksTable)
-                .where(inArray(tasksTable.id, deps))
-                .all();
-            const terminalStatuses = new Set(["completed", "failed", "cancelled"]);
-            const allTerminal = depRows.every(r => terminalStatuses.has(r.status));
-            if (!allTerminal)
-                continue;
-            const policy = (task.on_upstream_failure ?? "fail");
-            const failedDep = depRows.find(r => r.status === "failed" || r.status === "cancelled");
-            if (failedDep && policy === "fail") {
-                // Mark downstream as failed — no enqueue
-                logger.info({ taskId: task.id, failedDepId: failedDep.id, status: failedDep.status }, "Propagating upstream failure to waiting task");
-                this.taskStore.updateStatus(task.id, "failed", {
-                    error: `Upstream task ${failedDep.id} ${failedDep.status}`,
-                });
-                // Recursively dispatch tasks that depend on this now-failed task
-                await this.dispatchReady(task.id);
-                continue;
-            }
-            // policy === "skip" or all deps completed:
-            // Build inputs from completed upstream results only.
-            // Skip-case: omit failed/cancelled upstreams (no result to inject).
-            const inputs = {};
-            for (const dep of depRows) {
-                if (dep.status === "completed" && dep.result != null) {
-                    inputs[dep.id] = dep.result;
-                }
-            }
-            // Optimistic lock: transition waiting → pending atomically.
-            // If another concurrent dispatchReady call already transitioned this
-            // row, changes === 0 and we skip enqueue to avoid double-dispatch.
-            const updated = this.db
-                .update(tasksTable)
-                .set({
-                status: "pending",
-                inputs: JSON.stringify(inputs),
-                updatedAt: nowIso(),
-            })
-                .where(and(eq(tasksTable.id, task.id), eq(tasksTable.status, "waiting")))
-                .run();
-            if (updated.changes === 0) {
-                // Lost the race — another worker already transitioned this task
-                logger.debug({ taskId: task.id }, "DagEngine: lost optimistic lock, skipping enqueue");
-                continue;
-            }
-            logger.info({ taskId: task.id, deps, inputs: Object.keys(inputs) }, "DagEngine: dispatching ready task");
-            await this.dispatchFn(task.id);
-        }
-    }
-}
-//# sourceMappingURL=dag-engine.js.map

package/src/engine/dag-engine.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"dag-engine.js","sourceRoot":"","sources":["../../../../../../packages/ai/agent-mcp/src/engine/dag-engine.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,GAAG,EAAE,EAAE,EAAE,OAAO,EAAE,MAAM,aAAa,CAAC;AAG/C,OAAO,EAAE,UAAU,EAAE,MAAM,iBAAiB,CAAC;AAC7C,OAAO,EAAE,MAAM,EAAE,MAAM,cAAc,CAAC;AAGtC,OAAO,EAAE,SAAS,EAAE,MAAM,yBAAyB,CAAC;AACpD,OAAO,EAAE,MAAM,EAAE,MAAM,wBAAwB,CAAC;AAEhD;;;;;;;;;;GAUG;AACH,MAAM,OAAO,SAAS;IAEG;IACA;IACA;IAMA;IATrB,YACqB,EAAgD,EAChD,KAAsB,EACtB,SAAoB;IACrC;;;;OAIG;IACc,UAA6C;QAR7C,OAAE,GAAF,EAAE,CAA8C;QAChD,UAAK,GAAL,KAAK,CAAiB;QACtB,cAAS,GAAT,SAAS,CAAW;QAMpB,eAAU,GAAV,UAAU,CAAmC;IAC/D,CAAC;IAEJ;;;;;;;;;;OAUG;IACH,eAAe,CAAC,SAAiB,EAAE,SAAmB;QAClD,MAAM,OAAO,GAAG,IAAI,GAAG,EAAU,CAAC;QAClC,MAAM,KAAK,GAAG,CAAC,GAAG,SAAS,CAAC,CAAC;QAE7B,OAAO,KAAK,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACtB,MAAM,EAAE,GAAG,KAAK,CAAC,KAAK,EAAG,CAAC;YAE1B,IAAI,EAAE,KAAK,SAAS,EAAE,CAAC;gBACnB,MAAM,IAAI,SAAS,CACf,kBAAkB,EAClB,mCAAmC,SAAS,kDAAkD,CACjG,CAAC;YACN,CAAC;YAED,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,EAAE,CAAC;gBACnB,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC;gBAEhB,MAAM,GAAG,GAAG,IAAI,CAAC,EAAE;qBACd,MAAM,CAAC,EAAE,SAAS,EAAE,UAAU,CAAC,UAAU,EAAE,CAAC;qBAC5C,IAAI,CAAC,UAAU,CAAC;qBAChB,KAAK,CAAC,EAAE,CAAC,UAAU,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC;qBAC5B,GAAG,EAAE,CAAC;gBAEX,MAAM,YAAY,GAAa,GAAG,EAAE,SAAS;oBACzC,CAAC,CAAE,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,SAAS,CAAc;oBACzC,CAAC,CAAC,EAAE,CAAC;gBAET,KAAK,CAAC,IAAI,CAAC,GAAG,YAAY,CAAC,CAAC;YAChC,CAAC;QACL,CAAC;IACL,CAAC;IAED;;;;;;;;;;;;;;;OAeG;IACH,KAAK,CAAC,aAAa,CAAC,eAAuB;QACvC,4CAA4C;QAC5C,MAAM,YAAY,GAAG,IAAI,CAAC,EAAE;aACvB,MAAM,EAAE;aACR,IAAI,CAAC,UAAU,CAAC;aAChB,KAAK,CAAC,EAAE,CAAC,UAAU,CAAC,MAAM,EAAE,SAAS,CAAC,CAAC;aACvC,GAAG,EAAE,CAAC;QAEX,KAAK,MAAM,IAAI,IAAI,YAAY,EAAE,CAAC;YAC9B,MAAM,IAAI,GAAa,IAAI,CAAC,UAAU;gBAClC,CAAC,CAAE,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,UAAU,CAAc;gBAC3C,CAAC,CAAC,EAAE,CAAC;YAET,8DAA8D;YAC9D,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,eAAe,CAAC;gBAAE,SAAS;YAE9C,kEAAkE;YAClE,wCAAwC;YACxC,IAAI,IAAI,CAAC,MAAM,KAAK,CAAC;gBAAE,SAAS;YAEhC,qCAAqC;YACrC,MAAM,OAAO,GAAG,IAAI,CAAC,EAAE;iBAClB,MAAM,CAAC;gBACJ,EAAE,EAAE,UAAU,CAAC,EAAE;gBACjB,MAAM,EAAE,UAAU,CAAC,MAAM;gBACzB,MAAM,EAAE,UAAU,CAAC,MAAM;aAC5B,CAAC;iBACD,IAAI,CAAC,UAAU,CAAC;iBAChB,KAAK,CAAC,OAAO,CAAC,UAAU,CAAC,EAAE,EAAE,IAAI,CAAC,CAAC;iBACnC,GAAG,EAAE,CAAC;YAEX,MAAM,gBAAgB,GAAG,IAAI,GAAG,CAAC,CAAC,WAAW,EAAE,QAAQ,EAAE,WAAW,CAAC,CAAC,CAAC;YACvE,MAAM,WAAW,GAAG,OAAO,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,CAAC,gBAAgB,CAAC,GAAG,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC;YACvE,IAAI,CAAC,WAAW;gBAAE,SAAS;YAE3B,MAAM,MAAM,GAAG,CAAC,IAAI,CAAC,mBAAmB,IAAI,MAAM,CAAoB,CAAC;YACvE,MAAM,SAAS,GAAG,OAAO,CAAC,IAAI,CAC1B,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,MAAM,KAAK,QAAQ,IAAI,CAAC,CAAC,MAAM,KAAK,WAAW,CACzD,CAAC;YAEF,IAAI,SAAS,IAAI,MAAM,KAAK,MAAM,EAAE,CAAC;gBACjC,yCAAyC;gBACzC,MAAM,CAAC,IAAI,CACP,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,EAAE,WAAW,EAAE,SAAS,CAAC,EAAE,EAAE,MAAM,EAAE,SAAS,CAAC,MAAM,EAAE,EACxE,8CAA8C,CACjD,CAAC;gBAEF,IAAI,CAAC,SAAS,CAAC,YAAY,CAAC,IAAI,CAAC,EAAE,EAAE,QAAQ,EAAE;oBAC3C,KAAK,EAAE,iBAAiB,SAAS,CAAC,EAAE,IAAI,SAAS,CAAC,MAAM,EAAE;iBAC7D,CAAC,CAAC;gBAEH,iEAAiE;gBACjE,MAAM,IAAI,CAAC,aAAa,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;gBAClC,SAAS;YACb,CAAC;YAED,2CAA2C;YAC3C,qDAAqD;YACrD,oEAAoE;YACpE,MAAM,MAAM,GAA2B,EAAE,CAAC;YAC1C,KAAK,MAAM,GAAG,IAAI,OAAO,EAAE,CAAC;gBACxB,IAAI,GAAG,CAAC,MAAM,KAAK,WAAW,IAAI,GAAG,CAAC,MAAM,IAAI,IAAI,EAAE,CAAC;oBACnD,MAAM,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,GAAG,CAAC,MAAM,CAAC;gBAChC,CAAC;YACL,CAAC;YAED,4DAA4D;YAC5D,qEAAqE;YACrE,mEAAmE;YACnE,MAAM,OAAO,GAAG,IAAI,CAAC,EAAE;iBAClB,MAAM,CAAC,UAAU,CAAC;iBAClB,GAAG,CAAC;gBACD,MAAM,EAAE,SAAS;gBACjB,MAAM,EAAE,IAAI,CAAC,SAAS,CAAC,MAAM,CAAC;gBAC9B,SAAS,EAAE,MAAM,EAAE;aACtB,CAAC;iBACD,KAAK,CACF,GAAG,CACC,EAAE,CAAC,UAAU,CAAC,EAAE,EAAE,IAAI,CAAC,EAAE,CAAC,EAC1B,EAAE,CAAC,UAAU,CAAC,MAAM,EAAE,SAAS,CAAC,CACnC,CACJ;iBACA,GAAG,EAAE,CAAC;YAEX,IAAI,OAAO,CAAC,OAAO,KAAK,CAAC,EAAE,CAAC;gBACxB,gEAAgE;gBAChE,MAAM,CAAC,KAAK,CAAC,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,EAAE,EAAE,mDAAmD,CAAC,CAAC;gBACvF,SAAS;YACb,CAAC;YAED,MAAM,CAAC,IAAI,CACP,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,EACtD,mCAAmC,CACtC,CAAC;YAEF,MAAM,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACnC,CAAC;IACL,CAAC;CACJ"}

package/src/engine/hooks.d.ts

@@ -1,24 +0,0 @@
-import type { HookEvent, HookEventMap, HookHandler, IHookRegistry, EnforcementEvent, EnforcementHandler } from "@adhd/agent-mcp-types";
-/**
- * Concrete implementation of IHookRegistry used by the agent-mcp server.
- *
- * NOTE: A copy of this class also lives in @adhd/agent-mcp-types so plugin
- * authors can instantiate it in their own tests without taking a runtime
- * dependency on the full server package. Keep the two in sync until DEBT-009
- * is resolved (extraction to @adhd/agent-mcp-hooks).
- *
- * Observational handlers registered via register() have their throws swallowed
- * and logged — a buggy plugin never kills a task.
- *
- * Enforcement handlers registered via registerEnforcement() propagate throws
- * so the orchestrator can fail the task with BUDGET_EXCEEDED.
- */
-export declare class HookRegistry implements IHookRegistry {
-    private readonly handlers;
-    private readonly enforcementHandlers;
-    register<E extends HookEvent>(event: E, handler: HookHandler<E>): void;
-    emit<E extends HookEvent>(event: E, payload: HookEventMap[E]): Promise<void>;
-    registerEnforcement<E extends EnforcementEvent>(event: E, handler: EnforcementHandler<E>): void;
-    enforce<E extends EnforcementEvent>(event: E, payload: HookEventMap[E]): Promise<void>;
-}
-//# sourceMappingURL=hooks.d.ts.map

package/src/engine/hooks.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"hooks.d.ts","sourceRoot":"","sources":["../../../../../../packages/ai/agent-mcp/src/engine/hooks.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACR,SAAS,EACT,YAAY,EACZ,WAAW,EACX,aAAa,EACb,gBAAgB,EAChB,kBAAkB,EACrB,MAAM,uBAAuB,CAAC;AAE/B;;;;;;;;;;;;;GAaG;AACH,qBAAa,YAAa,YAAW,aAAa;IAC9C,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAA6D;IACtF,OAAO,CAAC,QAAQ,CAAC,mBAAmB,CAAuE;IAE3G,QAAQ,CAAC,CAAC,SAAS,SAAS,EAAE,KAAK,EAAE,CAAC,EAAE,OAAO,EAAE,WAAW,CAAC,CAAC,CAAC,GAAG,IAAI;IAMhE,IAAI,CAAC,CAAC,SAAS,SAAS,EAAE,KAAK,EAAE,CAAC,EAAE,OAAO,EAAE,YAAY,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC;IAYlF,mBAAmB,CAAC,CAAC,SAAS,gBAAgB,EAAE,KAAK,EAAE,CAAC,EAAE,OAAO,EAAE,kBAAkB,CAAC,CAAC,CAAC,GAAG,IAAI;IAMzF,OAAO,CAAC,CAAC,SAAS,gBAAgB,EAAE,KAAK,EAAE,CAAC,EAAE,OAAO,EAAE,YAAY,CAAC,CAAC,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC;CAO/F"}

package/src/engine/hooks.js

@@ -1,50 +0,0 @@
-/**
- * Concrete implementation of IHookRegistry used by the agent-mcp server.
- *
- * NOTE: A copy of this class also lives in @adhd/agent-mcp-types so plugin
- * authors can instantiate it in their own tests without taking a runtime
- * dependency on the full server package. Keep the two in sync until DEBT-009
- * is resolved (extraction to @adhd/agent-mcp-hooks).
- *
- * Observational handlers registered via register() have their throws swallowed
- * and logged — a buggy plugin never kills a task.
- *
- * Enforcement handlers registered via registerEnforcement() propagate throws
- * so the orchestrator can fail the task with BUDGET_EXCEEDED.
- */
-export class HookRegistry {
-    handlers = new Map();
-    enforcementHandlers = new Map();
-    register(event, handler) {
-        const list = this.handlers.get(event) ?? [];
-        list.push(handler);
-        this.handlers.set(event, list);
-    }
-    async emit(event, payload) {
-        const list = this.handlers.get(event);
-        if (!list?.length)
-            return;
-        for (const handler of list) {
-            try {
-                await handler(payload);
-            }
-            catch (err) {
-                console.warn("[HookRegistry] hook handler error (swallowed)", { event, err });
-            }
-        }
-    }
-    registerEnforcement(event, handler) {
-        const list = this.enforcementHandlers.get(event) ?? [];
-        list.push(handler);
-        this.enforcementHandlers.set(event, list);
-    }
-    async enforce(event, payload) {
-        const list = this.enforcementHandlers.get(event);
-        if (!list?.length)
-            return;
-        for (const handler of list) {
-            await handler(payload);
-        }
-    }
-}
-//# sourceMappingURL=hooks.js.map

package/src/engine/hooks.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"hooks.js","sourceRoot":"","sources":["../../../../../../packages/ai/agent-mcp/src/engine/hooks.ts"],"names":[],"mappings":"AASA;;;;;;;;;;;;;GAaG;AACH,MAAM,OAAO,YAAY;IACJ,QAAQ,GAAc,IAAI,GAAG,EAAuC,CAAC;IACrE,mBAAmB,GAAG,IAAI,GAAG,EAA4D,CAAC;IAE3G,QAAQ,CAAsB,KAAQ,EAAE,OAAuB;QAC3D,MAAM,IAAI,GAAG,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC;QAC5C,IAAI,CAAC,IAAI,CAAC,OAAiC,CAAC,CAAC;QAC7C,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,KAAK,EAAE,IAAI,CAAC,CAAC;IACnC,CAAC;IAED,KAAK,CAAC,IAAI,CAAsB,KAAQ,EAAE,OAAwB;QAC9D,MAAM,IAAI,GAAG,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;QACtC,IAAI,CAAC,IAAI,EAAE,MAAM;YAAE,OAAO;QAC1B,KAAK,MAAM,OAAO,IAAI,IAAI,EAAE,CAAC;YACzB,IAAI,CAAC;gBACD,MAAO,OAA0B,CAAC,OAAO,CAAC,CAAC;YAC/C,CAAC;YAAC,OAAO,GAAG,EAAE,CAAC;gBACX,OAAO,CAAC,IAAI,CAAC,+CAA+C,EAAE,EAAE,KAAK,EAAE,GAAG,EAAE,CAAC,CAAC;YAClF,CAAC;QACL,CAAC;IACL,CAAC;IAED,mBAAmB,CAA6B,KAAQ,EAAE,OAA8B;QACpF,MAAM,IAAI,GAAG,IAAI,CAAC,mBAAmB,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC;QACvD,IAAI,CAAC,IAAI,CAAC,OAA+C,CAAC,CAAC;QAC3D,IAAI,CAAC,mBAAmB,CAAC,GAAG,CAAC,KAAK,EAAE,IAAI,CAAC,CAAC;IAC9C,CAAC;IAED,KAAK,CAAC,OAAO,CAA6B,KAAQ,EAAE,OAAwB;QACxE,MAAM,IAAI,GAAG,IAAI,CAAC,mBAAmB,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;QACjD,IAAI,CAAC,IAAI,EAAE,MAAM;YAAE,OAAO;QAC1B,KAAK,MAAM,OAAO,IAAI,IAAI,EAAE,CAAC;YACzB,MAAO,OAAiC,CAAC,OAAO,CAAC,CAAC;QACtD,CAAC;IACL,CAAC;CACJ"}

package/src/engine/orchestrator.d.ts

@@ -1,40 +0,0 @@
-import type { LLMProvider } from "../providers/types.js";
-import type { ExecutionContext, Message } from "../validation/index.js";
-import type { IHookRegistry } from "@adhd/agent-mcp-types";
-import type { McpClientRegistry } from "../clients/registry.js";
-import type { PolicyEngine } from "./policy.js";
-import type { TaskStore } from "../store/task-store.js";
-import type { SessionStore } from "../store/session-store.js";
-/**
- * Resolve a suspended HITL task with the provided user input.
- * Returns `true` if the resolver was found and called; `false` if the
- * process restarted and the in-memory resolver no longer exists.
- */
-export declare function resolveHitl(taskId: string, userInput: string): boolean;
-export interface OrchestratorRunInput {
-    executionContext: ExecutionContext;
-    messages: Message[];
-    registry: McpClientRegistry;
-    provider: LLMProvider;
-    policy: PolicyEngine;
-    taskStore: TaskStore;
-    sessionStore: SessionStore;
-    signal: AbortSignal;
-    taskId: string;
-    hooks?: IHookRegistry;
-    /**
-     * Set to `true` for ephemeral (one-shot) tasks (agent_name mode).
-     * Ephemeral tasks DO persist a tasks row + task_events + task_usage, but
-     * have no sessions row and no messages rows. When `true`,
-     * `request_human_input` is forbidden because HITL cannot be resumed across
-     * a process restart without a durable session context.
-     */
-    isEphemeral?: boolean;
-}
-export interface OrchestratorRunResult {
-    result: string;
-}
-export declare class Orchestrator {
-    run(input: OrchestratorRunInput): Promise<OrchestratorRunResult>;
-}
-//# sourceMappingURL=orchestrator.d.ts.map

package/src/engine/orchestrator.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"orchestrator.d.ts","sourceRoot":"","sources":["../../../../../../packages/ai/agent-mcp/src/engine/orchestrator.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,uBAAuB,CAAC;AACzD,OAAO,KAAK,EAAE,gBAAgB,EAAE,OAAO,EAAE,MAAM,wBAAwB,CAAC;AACxE,OAAO,KAAK,EAAE,aAAa,EAAqB,MAAM,uBAAuB,CAAC;AAM9E,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,wBAAwB,CAAC;AAChE,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,aAAa,CAAC;AAChD,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,wBAAwB,CAAC;AACxD,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,2BAA2B,CAAC;AA4C9D;;;;GAIG;AACH,wBAAgB,WAAW,CAAC,MAAM,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,GAAG,OAAO,CAMtE;AAID,MAAM,WAAW,oBAAoB;IACjC,gBAAgB,EAAE,gBAAgB,CAAC;IACnC,QAAQ,EAAE,OAAO,EAAE,CAAC;IACpB,QAAQ,EAAE,iBAAiB,CAAC;IAC5B,QAAQ,EAAE,WAAW,CAAC;IACtB,MAAM,EAAE,YAAY,CAAC;IACrB,SAAS,EAAE,SAAS,CAAC;IACrB,YAAY,EAAE,YAAY,CAAC;IAC3B,MAAM,EAAE,WAAW,CAAC;IACpB,MAAM,EAAE,MAAM,CAAC;IACf,KAAK,CAAC,EAAE,aAAa,CAAC;IACtB;;;;;;OAMG;IACH,WAAW,CAAC,EAAE,OAAO,CAAC;CACzB;AAmBD,MAAM,WAAW,qBAAqB;IAClC,MAAM,EAAE,MAAM,CAAC;CAClB;AAED,qBAAa,YAAY;IACf,GAAG,CAAC,KAAK,EAAE,oBAAoB,GAAG,OAAO,CAAC,qBAAqB,CAAC;CAmiBzE"}