@mcp-abap-adt/llm-agent-libs 16.2.0 → 17.0.0

package/dist/coordinator/dag/llm-dag-planner.js

@@ -0,0 +1,152 @@
+import { ClarifySignal, NeedInfoSignal } from '@mcp-abap-adt/llm-agent';
+import { DirectLlmSubAgent } from '../../subagent/direct-llm-subagent.js';
+import { renderAncestorContext } from './render-ancestor-context.js';
+/**
+ * Attach LLM usage to a thrown Error so the coordinator's runRole catch can
+ * still bill the spend (HIGH/MEDIUM finding: parse-/shape-error paths went
+ * through `throw new Error(...)` and the captured `res.usage` was lost — yet
+ * the LLM call WAS made and tokens WERE spent). Plain Errors with a `usage`
+ * property work in JS without subclassing; runRole reads it via a type cast.
+ */
+function withUsage(err, usage) {
+    if (usage)
+        err.usage = usage;
+    return err;
+}
+// Static planner instructions. The agent catalog and user prompt are NOT here —
+// they are dynamic and go into the per-call `task` (see plan()).
+export const PLANNER_SYSTEM = `You are a planner. Decompose the user request into a DAG of tasks.
+Each node: {"id","goal","agent"(optional worker name),"dependsOn"(optional ids),"needsInput"(optional bool)}.
+Use "dependsOn" to express order/data-flow; independent nodes run in parallel.
+
+DECOMPOSITION COST. Each node spawns a fresh worker pipeline. Workers
+DO NOT share fetched data, tools, or context across nodes — every node
+pays the full classify + RAG + tool-loop overhead again, and any
+source / configuration / table data already fetched by a previous
+node will be re-fetched. Over-decomposition is the most common cause
+of large token bills. Decompose ONLY when:
+- nodes target DIFFERENT objects (e.g. compare program A vs program B),
+- nodes can TRULY run in parallel for wall-clock speedup, or
+- a later node depends on a fact ONLY discoverable by an earlier node.
+
+For analysing a SINGLE object along multiple dimensions (e.g. "review
+program X for security, performance, clean-core, maintainability") use
+ONE node — the worker covers every dimension in one tool-loop.
+
+Emit a plan-level "objective". Respond with ONLY one of:
+{"objective":"...","nodes":[{"id":"n1","goal":"...","agent":"<worker name or omit>","dependsOn":[],"needsInput":false}]}
+{"needInfo":"<query>"}  — if you need a reality fact before planning (e.g. which table exists)
+{"clarify":"<question>"}  — if you need a human decision before planning (e.g. overwrite ok?)`;
+/**
+ * Role adapter: owns a constrained `DirectLlmSubAgent` and turns its string
+ * output into a typed `DagPlan`. (Slice 2: planner now flows through the one
+ * ISubAgent path instead of calling ILlm directly.)
+ */
+export class LlmDagPlanner {
+    name = 'llm-dag';
+    /** Best-effort model identifier from the underlying ILlm (for logger
+     *  attribution). May be undefined for ILlm impls that do not expose one. */
+    model;
+    agent;
+    constructor(llm) {
+        this.model = llm.model;
+        this.agent = new DirectLlmSubAgent('planner', llm, {
+            systemPrompt: PLANNER_SYSTEM,
+            contextPolicy: 'optional',
+        });
+    }
+    async plan(input) {
+        const catalog = input.agents
+            .map((a) => `- ${a.name}: ${a.description ?? '(no description)'}`)
+            .join('\n');
+        const contextPrefix = [];
+        if (input.ancestorContext) {
+            const rendered = renderAncestorContext(input.ancestorContext);
+            if (rendered)
+                contextPrefix.push(rendered);
+        }
+        if (input.reviewerFeedback) {
+            contextPrefix.push(`Reviewer feedback: ${input.reviewerFeedback}`);
+        }
+        const prefix = contextPrefix.length > 0 ? `${contextPrefix.join('\n\n')}\n\n` : '';
+        const task = `${prefix}Available workers:\n${catalog || '(none)'}\n\n${input.prompt}`;
+        const res = await this.agent.run({
+            task,
+            sessionId: input.sessionId,
+            signal: input.signal,
+        });
+        const content = res.output;
+        const match = content.match(/\{[\s\S]*\}/);
+        if (!match)
+            throw withUsage(new Error(`Planner output did not contain a JSON object: ${content.slice(0, 200)}`), res.usage);
+        // Field values come straight from untrusted JSON, so they are typed as
+        // `unknown` and validated below before being narrowed to PlanNode.
+        let parsed;
+        try {
+            parsed = JSON.parse(match[0]);
+        }
+        catch {
+            throw withUsage(new Error(`Planner output contained malformed JSON: ${match[0].slice(0, 200)}`), res.usage);
+        }
+        if (typeof parsed.needInfo === 'string' && parsed.needInfo.trim()) {
+            // Forward LLM usage on the signal path so the coordinator can attribute
+            // planner spend even when the role short-circuits with a signal (HIGH
+            // finding: signal paths previously discarded the captured `res.usage`).
+            throw new NeedInfoSignal(parsed.needInfo, res.usage);
+        }
+        if (typeof parsed.clarify === 'string' && parsed.clarify.trim()) {
+            throw new ClarifySignal(parsed.clarify, res.usage);
+        }
+        if (!Array.isArray(parsed.nodes) || parsed.nodes.length === 0) {
+            throw withUsage(new Error(`Planner returned no nodes: ${match[0].slice(0, 200)}`), res.usage);
+        }
+        if (parsed.objective !== undefined &&
+            typeof parsed.objective !== 'string') {
+            throw withUsage(new Error(`Planner objective must be a string: ${JSON.stringify(parsed.objective)}`), res.usage);
+        }
+        if (parsed.rationale !== undefined &&
+            typeof parsed.rationale !== 'string') {
+            throw withUsage(new Error(`Planner rationale must be a string: ${JSON.stringify(parsed.rationale)}`), res.usage);
+        }
+        const nodes = parsed.nodes.map((n, i) => {
+            if (typeof n.goal !== 'string' || n.goal.trim() === '') {
+                throw withUsage(new Error(`Planner node is missing a goal: ${JSON.stringify(n)}`), res.usage);
+            }
+            if (n.id !== undefined && typeof n.id !== 'string') {
+                throw withUsage(new Error(`Planner node has a non-string id: ${JSON.stringify(n)}`), res.usage);
+            }
+            if (n.agent !== undefined && typeof n.agent !== 'string') {
+                throw withUsage(new Error(`Planner node has a non-string agent: ${JSON.stringify(n)}`), res.usage);
+            }
+            if (n.dependsOn !== undefined &&
+                (!Array.isArray(n.dependsOn) ||
+                    n.dependsOn.some((d) => typeof d !== 'string'))) {
+                throw withUsage(new Error(`Planner node dependsOn must be an array of strings: ${JSON.stringify(n)}`), res.usage);
+            }
+            if (n.needsInput !== undefined && typeof n.needsInput !== 'boolean') {
+                throw withUsage(new Error(`Planner node needsInput must be a boolean: ${JSON.stringify(n)}`), res.usage);
+            }
+            return {
+                id: n.id ?? `n${i + 1}`,
+                goal: n.goal,
+                agent: n.agent,
+                dependsOn: n.dependsOn,
+                needsInput: n.needsInput,
+            };
+        });
+        return {
+            plan: {
+                nodes,
+                objective: parsed.objective,
+                rationale: parsed.rationale,
+                createdAt: Date.now(),
+            },
+            // Forward the underlying ILlm.chat usage on the WRAPPER (not on the
+            // plan itself) so the coordinator can attribute planner-LLM spend to
+            // the session/request logger without polluting the plan domain type
+            // (which the reviewer serializes via JSON.stringify into its prompt).
+            usage: res.usage,
+        };
+    }
+}
+//# sourceMappingURL=llm-dag-planner.js.map

package/dist/coordinator/dag/llm-dag-planner.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"llm-dag-planner.js","sourceRoot":"","sources":["../../../src/coordinator/dag/llm-dag-planner.ts"],"names":[],"mappings":"AAQA,OAAO,EAAE,aAAa,EAAE,cAAc,EAAE,MAAM,yBAAyB,CAAC;AACxE,OAAO,EAAE,iBAAiB,EAAE,MAAM,uCAAuC,CAAC;AAC1E,OAAO,EAAE,qBAAqB,EAAE,MAAM,8BAA8B,CAAC;AAErE;;;;;;GAMG;AACH,SAAS,SAAS,CAAC,GAAU,EAAE,KAA2B;IACxD,IAAI,KAAK;QAAG,GAAoC,CAAC,KAAK,GAAG,KAAK,CAAC;IAC/D,OAAO,GAAG,CAAC;AACb,CAAC;AAED,gFAAgF;AAChF,iEAAiE;AACjE,MAAM,CAAC,MAAM,cAAc,GAAG;;;;;;;;;;;;;;;;;;;;;8FAqBgE,CAAC;AAE/F;;;;GAIG;AACH,MAAM,OAAO,aAAa;IACf,IAAI,GAAG,SAAS,CAAC;IAC1B;gFAC4E;IACnE,KAAK,CAAU;IACP,KAAK,CAAoB;IAE1C,YAAY,GAAS;QACnB,IAAI,CAAC,KAAK,GAAG,GAAG,CAAC,KAAK,CAAC;QACvB,IAAI,CAAC,KAAK,GAAG,IAAI,iBAAiB,CAAC,SAAS,EAAE,GAAG,EAAE;YACjD,YAAY,EAAE,cAAc;YAC5B,aAAa,EAAE,UAAU;SAC1B,CAAC,CAAC;IACL,CAAC;IAED,KAAK,CAAC,IAAI,CAAC,KAAmB;QAC5B,MAAM,OAAO,GAAG,KAAK,CAAC,MAAM;aACzB,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,CAAC,IAAI,KAAK,CAAC,CAAC,WAAW,IAAI,kBAAkB,EAAE,CAAC;aACjE,IAAI,CAAC,IAAI,CAAC,CAAC;QACd,MAAM,aAAa,GAAa,EAAE,CAAC;QACnC,IAAI,KAAK,CAAC,eAAe,EAAE,CAAC;YAC1B,MAAM,QAAQ,GAAG,qBAAqB,CAAC,KAAK,CAAC,eAAe,CAAC,CAAC;YAC9D,IAAI,QAAQ;gBAAE,aAAa,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;QAC7C,CAAC;QACD,IAAI,KAAK,CAAC,gBAAgB,EAAE,CAAC;YAC3B,aAAa,CAAC,IAAI,CAAC,sBAAsB,KAAK,CAAC,gBAAgB,EAAE,CAAC,CAAC;QACrE,CAAC;QACD,MAAM,MAAM,GACV,aAAa,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,aAAa,CAAC,IAAI,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC;QACtE,MAAM,IAAI,GAAG,GAAG,MAAM,uBAAuB,OAAO,IAAI,QAAQ,OAAO,KAAK,CAAC,MAAM,EAAE,CAAC;QAEtF,MAAM,GAAG,GAAG,MAAM,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC;YAC/B,IAAI;YACJ,SAAS,EAAE,KAAK,CAAC,SAAS;YAC1B,MAAM,EAAE,KAAK,CAAC,MAAM;SACrB,CAAC,CAAC;QACH,MAAM,OAAO,GAAG,GAAG,CAAC,MAAM,CAAC;QAE3B,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,aAAa,CAAC,CAAC;QAC3C,IAAI,CAAC,KAAK;YACR,MAAM,SAAS,CACb,IAAI,KAAK,CACP,iDAAiD,OAAO,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,EAAE,CACzE,EACD,GAAG,CAAC,KAAK,CACV,CAAC;QACJ,uEAAuE;QACvE,mEAAmE;QACnE,IAAI,MAYH,CAAC;QACF,IAAI,CAAC;YACH,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QAChC,CAAC;QAAC,MAAM,CAAC;YACP,MAAM,SAAS,CACb,IAAI,KAAK,CACP,4CAA4C,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,EAAE,CACrE,EACD,GAAG,CAAC,KAAK,CACV,CAAC;QACJ,CAAC;QACD,IAAI,OAAO,MAAM,CAAC,QAAQ,KAAK,QAAQ,IAAI,MAAM,CAAC,QAAQ,CAAC,IAAI,EAAE,EAAE,CAAC;YAClE,wEAAwE;YACxE,sEAAsE;YACtE,wEAAwE;YACxE,MAAM,IAAI,cAAc,CAAC,MAAM,CAAC,QAAQ,EAAE,GAAG,CAAC,KAAK,CAAC,CAAC;QACvD,CAAC;QACD,IAAI,OAAO,MAAM,CAAC,OAAO,KAAK,QAAQ,IAAI,MAAM,CAAC,OAAO,CAAC,IAAI,EAAE,EAAE,CAAC;YAChE,MAAM,IAAI,aAAa,CAAC,MAAM,CAAC,OAAO,EAAE,GAAG,CAAC,KAAK,CAAC,CAAC;QACrD,CAAC;QACD,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,KAAK,CAAC,IAAI,MAAM,CAAC,KAAK,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YAC9D,MAAM,SAAS,CACb,IAAI,KAAK,CAAC,8BAA8B,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,EAAE,CAAC,EACjE,GAAG,CAAC,KAAK,CACV,CAAC;QACJ,CAAC;QACD,IACE,MAAM,CAAC,SAAS,KAAK,SAAS;YAC9B,OAAO,MAAM,CAAC,SAAS,KAAK,QAAQ,EACpC,CAAC;YACD,MAAM,SAAS,CACb,IAAI,KAAK,CACP,uCAAuC,IAAI,CAAC,SAAS,CAAC,MAAM,CAAC,SAAS,CAAC,EAAE,CAC1E,EACD,GAAG,CAAC,KAAK,CACV,CAAC;QACJ,CAAC;QACD,IACE,MAAM,CAAC,SAAS,KAAK,SAAS;YAC9B,OAAO,MAAM,CAAC,SAAS,KAAK,QAAQ,EACpC,CAAC;YACD,MAAM,SAAS,CACb,IAAI,KAAK,CACP,uCAAuC,IAAI,CAAC,SAAS,CAAC,MAAM,CAAC,SAAS,CAAC,EAAE,CAC1E,EACD,GAAG,CAAC,KAAK,CACV,CAAC;QACJ,CAAC;QACD,MAAM,KAAK,GAAe,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE;YAClD,IAAI,OAAO,CAAC,CAAC,IAAI,KAAK,QAAQ,IAAI,CAAC,CAAC,IAAI,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE,CAAC;gBACvD,MAAM,SAAS,CACb,IAAI,KAAK,CAAC,mCAAmC,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,CAAC,EACjE,GAAG,CAAC,KAAK,CACV,CAAC;YACJ,CAAC;YACD,IAAI,CAAC,CAAC,EAAE,KAAK,SAAS,IAAI,OAAO,CAAC,CAAC,EAAE,KAAK,QAAQ,EAAE,CAAC;gBACnD,MAAM,SAAS,CACb,IAAI,KAAK,CAAC,qCAAqC,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,CAAC,EACnE,GAAG,CAAC,KAAK,CACV,CAAC;YACJ,CAAC;YACD,IAAI,CAAC,CAAC,KAAK,KAAK,SAAS,IAAI,OAAO,CAAC,CAAC,KAAK,KAAK,QAAQ,EAAE,CAAC;gBACzD,MAAM,SAAS,CACb,IAAI,KAAK,CACP,wCAAwC,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,CAC5D,EACD,GAAG,CAAC,KAAK,CACV,CAAC;YACJ,CAAC;YACD,IACE,CAAC,CAAC,SAAS,KAAK,SAAS;gBACzB,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,SAAS,CAAC;oBAC1B,CAAC,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,OAAO,CAAC,KAAK,QAAQ,CAAC,CAAC,EACjD,CAAC;gBACD,MAAM,SAAS,CACb,IAAI,KAAK,CACP,uDAAuD,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,CAC3E,EACD,GAAG,CAAC,KAAK,CACV,CAAC;YACJ,CAAC;YACD,IAAI,CAAC,CAAC,UAAU,KAAK,SAAS,IAAI,OAAO,CAAC,CAAC,UAAU,KAAK,SAAS,EAAE,CAAC;gBACpE,MAAM,SAAS,CACb,IAAI,KAAK,CACP,8CAA8C,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,CAClE,EACD,GAAG,CAAC,KAAK,CACV,CAAC;YACJ,CAAC;YACD,OAAO;gBACL,EAAE,EAAG,CAAC,CAAC,EAAyB,IAAI,IAAI,CAAC,GAAG,CAAC,EAAE;gBAC/C,IAAI,EAAE,CAAC,CAAC,IAAI;gBACZ,KAAK,EAAE,CAAC,CAAC,KAA2B;gBACpC,SAAS,EAAE,CAAC,CAAC,SAAiC;gBAC9C,UAAU,EAAE,CAAC,CAAC,UAAiC;aAChD,CAAC;QACJ,CAAC,CAAC,CAAC;QACH,OAAO;YACL,IAAI,EAAE;gBACJ,KAAK;gBACL,SAAS,EAAE,MAAM,CAAC,SAA+B;gBACjD,SAAS,EAAE,MAAM,CAAC,SAA+B;gBACjD,SAAS,EAAE,IAAI,CAAC,GAAG,EAAE;aACtB;YACD,oEAAoE;YACpE,qEAAqE;YACrE,oEAAoE;YACpE,sEAAsE;YACtE,KAAK,EAAE,GAAG,CAAC,KAAK;SACjB,CAAC;IACJ,CAAC;CACF"}

package/dist/coordinator/dag/llm-finalizer.d.ts

@@ -0,0 +1,21 @@
+import type { FinalizerInput, FinalizerResult, IFinalizer, ILlm } from '@mcp-abap-adt/llm-agent';
+export declare const FINALIZER_SYSTEM: string;
+export interface LlmFinalizerOptions {
+    systemPrompt?: string;
+    name?: string;
+    model?: string;
+}
+/**
+ * Synthesizes the final answer via a single LLM call. NO tools are wired:
+ * the LLM cannot escape into another tool-loop. The user message is a
+ * deterministic rendering of prompt + objective + ancestorContext + trace.
+ */
+export declare class LlmFinalizer implements IFinalizer {
+    private readonly llm;
+    readonly name: string;
+    readonly model?: string;
+    private readonly systemPrompt;
+    constructor(llm: ILlm, opts?: LlmFinalizerOptions);
+    finalize(input: FinalizerInput): Promise<FinalizerResult>;
+}
+//# sourceMappingURL=llm-finalizer.d.ts.map

package/dist/coordinator/dag/llm-finalizer.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"llm-finalizer.d.ts","sourceRoot":"","sources":["../../../src/coordinator/dag/llm-finalizer.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,cAAc,EACd,eAAe,EACf,UAAU,EACV,IAAI,EAGL,MAAM,yBAAyB,CAAC;AAEjC,eAAO,MAAM,gBAAgB,QAM6C,CAAC;AAE3E,MAAM,WAAW,mBAAmB;IAClC,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAkCD;;;;GAIG;AACH,qBAAa,YAAa,YAAW,UAAU;IAM3C,OAAO,CAAC,QAAQ,CAAC,GAAG;IALtB,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC;IACtB,QAAQ,CAAC,KAAK,CAAC,EAAE,MAAM,CAAC;IACxB,OAAO,CAAC,QAAQ,CAAC,YAAY,CAAS;gBAGnB,GAAG,EAAE,IAAI,EAC1B,IAAI,GAAE,mBAAwB;IAO1B,QAAQ,CAAC,KAAK,EAAE,cAAc,GAAG,OAAO,CAAC,eAAe,CAAC;CAqBhE"}

package/dist/coordinator/dag/llm-finalizer.js

@@ -0,0 +1,77 @@
+export const FINALIZER_SYSTEM = 'You synthesize the final user-facing answer for a DAG-coordinated task. ' +
+    'You will receive: (1) the user prompt, (2) the plan objective, (3) an ' +
+    'ordered execution trace of completed DAG nodes (each with its goal and ' +
+    'output). Produce the answer using ONLY the trace outputs. Do NOT propose ' +
+    'new data collection. Do NOT include the trace structure in your reply ' +
+    "unless the user asked for it. Address every part of the user's prompt.";
+function renderUserMessage(input) {
+    const lines = [];
+    lines.push('# User prompt', input.prompt, '');
+    lines.push('# Plan objective', input.objective, '');
+    if (input.ancestorContext) {
+        const ac = input.ancestorContext;
+        if (ac.clarifications.length > 0) {
+            lines.push('# Clarifications');
+            for (const c of ac.clarifications) {
+                lines.push(`- Q: ${c.question}`);
+                lines.push(`  A: ${c.answer}`);
+            }
+            lines.push('');
+        }
+        if (ac.oracleObservations.length > 0) {
+            lines.push('# Oracle observations');
+            for (const o of ac.oracleObservations) {
+                lines.push(`- Q: ${o.query}`);
+                lines.push(`  A: ${o.answer}`);
+            }
+            lines.push('');
+        }
+    }
+    lines.push('# Execution trace');
+    for (const t of input.executionTrace) {
+        lines.push(`## Node ${t.nodeId} — ${t.goal}`);
+        lines.push(t.output);
+        lines.push('');
+    }
+    return lines.join('\n');
+}
+/**
+ * Synthesizes the final answer via a single LLM call. NO tools are wired:
+ * the LLM cannot escape into another tool-loop. The user message is a
+ * deterministic rendering of prompt + objective + ancestorContext + trace.
+ */
+export class LlmFinalizer {
+    llm;
+    name;
+    model;
+    systemPrompt;
+    constructor(llm, opts = {}) {
+        this.llm = llm;
+        this.name = opts.name ?? 'llm-finalizer';
+        this.model = opts.model;
+        this.systemPrompt = opts.systemPrompt ?? FINALIZER_SYSTEM;
+    }
+    async finalize(input) {
+        const messages = [
+            { role: 'system', content: this.systemPrompt },
+            { role: 'user', content: renderUserMessage(input) },
+        ];
+        const { signal, sessionId } = input;
+        let buf = '';
+        let usage;
+        const stream = this.llm.streamChat(messages, [], { signal, sessionId });
+        for await (const chunk of stream) {
+            if (chunk.ok === false)
+                throw new Error(chunk.error?.message ?? 'streamChat failed');
+            const content = chunk.value?.content ?? '';
+            if (content) {
+                buf += content;
+                input.onPartial?.({ kind: 'content', delta: content });
+            }
+            if (chunk.value?.usage)
+                usage = chunk.value.usage;
+        }
+        return { output: buf, usage };
+    }
+}
+//# sourceMappingURL=llm-finalizer.js.map

package/dist/coordinator/dag/llm-finalizer.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"llm-finalizer.js","sourceRoot":"","sources":["../../../src/coordinator/dag/llm-finalizer.ts"],"names":[],"mappings":"AASA,MAAM,CAAC,MAAM,gBAAgB,GAC3B,0EAA0E;IAC1E,wEAAwE;IACxE,yEAAyE;IACzE,2EAA2E;IAC3E,wEAAwE;IACxE,wEAAwE,CAAC;AAQ3E,SAAS,iBAAiB,CAAC,KAAqB;IAC9C,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,KAAK,CAAC,IAAI,CAAC,eAAe,EAAE,KAAK,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC;IAC9C,KAAK,CAAC,IAAI,CAAC,kBAAkB,EAAE,KAAK,CAAC,SAAS,EAAE,EAAE,CAAC,CAAC;IACpD,IAAI,KAAK,CAAC,eAAe,EAAE,CAAC;QAC1B,MAAM,EAAE,GAAG,KAAK,CAAC,eAAe,CAAC;QACjC,IAAI,EAAE,CAAC,cAAc,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACjC,KAAK,CAAC,IAAI,CAAC,kBAAkB,CAAC,CAAC;YAC/B,KAAK,MAAM,CAAC,IAAI,EAAE,CAAC,cAAc,EAAE,CAAC;gBAClC,KAAK,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,QAAQ,EAAE,CAAC,CAAC;gBACjC,KAAK,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC;YACjC,CAAC;YACD,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACjB,CAAC;QACD,IAAI,EAAE,CAAC,kBAAkB,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACrC,KAAK,CAAC,IAAI,CAAC,uBAAuB,CAAC,CAAC;YACpC,KAAK,MAAM,CAAC,IAAI,EAAE,CAAC,kBAAkB,EAAE,CAAC;gBACtC,KAAK,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,KAAK,EAAE,CAAC,CAAC;gBAC9B,KAAK,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC;YACjC,CAAC;YACD,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QACjB,CAAC;IACH,CAAC;IACD,KAAK,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAC;IAChC,KAAK,MAAM,CAAC,IAAI,KAAK,CAAC,cAAc,EAAE,CAAC;QACrC,KAAK,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC,MAAM,MAAM,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC;QAC9C,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC;QACrB,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;IACjB,CAAC;IACD,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC1B,CAAC;AAED;;;;GAIG;AACH,MAAM,OAAO,YAAY;IAMJ;IALV,IAAI,CAAS;IACb,KAAK,CAAU;IACP,YAAY,CAAS;IAEtC,YACmB,GAAS,EAC1B,OAA4B,EAAE;QADb,QAAG,GAAH,GAAG,CAAM;QAG1B,IAAI,CAAC,IAAI,GAAG,IAAI,CAAC,IAAI,IAAI,eAAe,CAAC;QACzC,IAAI,CAAC,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC;QACxB,IAAI,CAAC,YAAY,GAAG,IAAI,CAAC,YAAY,IAAI,gBAAgB,CAAC;IAC5D,CAAC;IAED,KAAK,CAAC,QAAQ,CAAC,KAAqB;QAClC,MAAM,QAAQ,GAAc;YAC1B,EAAE,IAAI,EAAE,QAAQ,EAAE,OAAO,EAAE,IAAI,CAAC,YAAY,EAAE;YAC9C,EAAE,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,iBAAiB,CAAC,KAAK,CAAC,EAAE;SACpD,CAAC;QACF,MAAM,EAAE,MAAM,EAAE,SAAS,EAAE,GAAG,KAAK,CAAC;QACpC,IAAI,GAAG,GAAG,EAAE,CAAC;QACb,IAAI,KAA2B,CAAC;QAChC,MAAM,MAAM,GAAG,IAAI,CAAC,GAAG,CAAC,UAAU,CAAC,QAAQ,EAAE,EAAE,EAAE,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC,CAAC;QACxE,IAAI,KAAK,EAAE,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;YACjC,IAAI,KAAK,CAAC,EAAE,KAAK,KAAK;gBACpB,MAAM,IAAI,KAAK,CAAC,KAAK,CAAC,KAAK,EAAE,OAAO,IAAI,mBAAmB,CAAC,CAAC;YAC/D,MAAM,OAAO,GAAG,KAAK,CAAC,KAAK,EAAE,OAAO,IAAI,EAAE,CAAC;YAC3C,IAAI,OAAO,EAAE,CAAC;gBACZ,GAAG,IAAI,OAAO,CAAC;gBACf,KAAK,CAAC,SAAS,EAAE,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,OAAO,EAAE,CAAC,CAAC;YACzD,CAAC;YACD,IAAI,KAAK,CAAC,KAAK,EAAE,KAAK;gBAAE,KAAK,GAAG,KAAK,CAAC,KAAK,CAAC,KAAK,CAAC;QACpD,CAAC;QACD,OAAO,EAAE,MAAM,EAAE,GAAG,EAAE,KAAK,EAAE,CAAC;IAChC,CAAC;CACF"}

package/dist/coordinator/dag/llm-review-strategy.d.ts

@@ -0,0 +1,17 @@
+import type { ExecutionFailureInput, ExecutionReviewResult, ILlm, IReviewStrategy, ReviewInput, ReviewResult } from '@mcp-abap-adt/llm-agent';
+/**
+ * Role adapter: owns a constrained `DirectLlmSubAgent` critic and turns its
+ * string output into a typed `ReviewVerdict`.
+ */
+export declare class LlmReviewStrategy implements IReviewStrategy {
+    readonly name = "llm-review";
+    /** Best-effort model identifier from the underlying ILlm (for logger
+     *  attribution). May be undefined for ILlm impls that do not expose one. */
+    readonly model?: string;
+    private readonly agent;
+    private readonly executionAgent;
+    constructor(llm: ILlm);
+    review(input: ReviewInput): Promise<ReviewResult>;
+    reviewExecutionFailure(input: ExecutionFailureInput): Promise<ExecutionReviewResult>;
+}
+//# sourceMappingURL=llm-review-strategy.d.ts.map

package/dist/coordinator/dag/llm-review-strategy.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"llm-review-strategy.d.ts","sourceRoot":"","sources":["../../../src/coordinator/dag/llm-review-strategy.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAEV,qBAAqB,EACrB,qBAAqB,EACrB,IAAI,EACJ,eAAe,EAEf,WAAW,EACX,YAAY,EACb,MAAM,yBAAyB,CAAC;AA+BjC;;;GAGG;AACH,qBAAa,iBAAkB,YAAW,eAAe;IACvD,QAAQ,CAAC,IAAI,gBAAgB;IAC7B;gFAC4E;IAC5E,QAAQ,CAAC,KAAK,CAAC,EAAE,MAAM,CAAC;IACxB,OAAO,CAAC,QAAQ,CAAC,KAAK,CAAoB;IAC1C,OAAO,CAAC,QAAQ,CAAC,cAAc,CAAoB;gBAEvC,GAAG,EAAE,IAAI;IAYf,MAAM,CAAC,KAAK,EAAE,WAAW,GAAG,OAAO,CAAC,YAAY,CAAC;IA6EjD,sBAAsB,CAC1B,KAAK,EAAE,qBAAqB,GAC3B,OAAO,CAAC,qBAAqB,CAAC;CAuFlC"}

package/dist/coordinator/dag/llm-review-strategy.js

@@ -0,0 +1,149 @@
+import { ClarifySignal, NeedInfoSignal } from '@mcp-abap-adt/llm-agent';
+import { DirectLlmSubAgent } from '../../subagent/direct-llm-subagent.js';
+import { renderAncestorContext } from './render-ancestor-context.js';
+/**
+ * Attach LLM usage to a thrown Error so the coordinator's runRole catch can
+ * still bill the spend. Same rationale as the planner-side helper.
+ */
+function withUsage(err, usage) {
+    if (usage)
+        err.usage = usage;
+    return err;
+}
+// Static critic instructions. The user prompt, plan and catalog are dynamic and
+// go into the per-call `task` (see review()).
+const REVIEWER_SYSTEM = `You are a plan reviewer. Given the user request, the available workers, and a proposed DAG plan, decide whether the plan can fulfil the request with those workers.
+Respond with ONLY a JSON object:
+{"pass": true}  — the plan is adequate
+{"pass": false, "feedback": "<what is wrong or what must be clarified>"}  — otherwise
+{"needInfo":"<query>"}  — if you need a reality fact to assess the plan (e.g. does table X exist?)
+{"clarify":"<question>"}  — if you need a human decision before reviewing (e.g. overwrite ok?)`;
+const EXECUTION_REVIEW_SYSTEM = `You are a recovery reviewer. A step of a DAG plan FAILED during execution. You are given the objective, the current plan, the execution trace (what already ran and its output — this reflects the CURRENT system state), the failed step id, and the error.
+Decide recovery and respond with ONLY a JSON object:
+{"action":"abort"}  — if recovery is not possible
+{"action":"revise","plan":{"nodes":[{"id":"...","goal":"...","agent":"<worker or omit>","dependsOn":[],"needsInput":false}],"objective":"..."}}  — a NEW plan for the REMAINING objective.
+{"needInfo":"<query>"}  — if you need a reality fact before deciding recovery
+{"clarify":"<question>"}  — if you need a human decision before deciding recovery
+The revised plan MUST treat the current state as the starting point: do not redo work already done (per the trace); if an artifact already exists, modify it instead of recreating it (idempotent/adaptive).`;
+/**
+ * Role adapter: owns a constrained `DirectLlmSubAgent` critic and turns its
+ * string output into a typed `ReviewVerdict`.
+ */
+export class LlmReviewStrategy {
+    name = 'llm-review';
+    /** Best-effort model identifier from the underlying ILlm (for logger
+     *  attribution). May be undefined for ILlm impls that do not expose one. */
+    model;
+    agent;
+    executionAgent;
+    constructor(llm) {
+        this.model = llm.model;
+        this.agent = new DirectLlmSubAgent('reviewer', llm, {
+            systemPrompt: REVIEWER_SYSTEM,
+            contextPolicy: 'optional',
+        });
+        this.executionAgent = new DirectLlmSubAgent('recovery-reviewer', llm, {
+            systemPrompt: EXECUTION_REVIEW_SYSTEM,
+            contextPolicy: 'optional',
+        });
+    }
+    async review(input) {
+        const catalog = input.agents
+            .map((a) => `- ${a.name}: ${a.description ?? '(no description)'}`)
+            .join('\n');
+        const contextPrefix = input.ancestorContext
+            ? `${renderAncestorContext(input.ancestorContext)}\n\n`
+            : '';
+        const task = `${contextPrefix}User request:\n${input.prompt}\n\nAvailable workers:\n${catalog || '(none)'}\n\nProposed plan (JSON):\n${JSON.stringify(input.plan)}`;
+        const res = await this.agent.run({
+            task,
+            sessionId: input.sessionId,
+            signal: input.signal,
+        });
+        const match = res.output.match(/\{[\s\S]*\}/);
+        if (!match)
+            throw withUsage(new Error(`Reviewer output did not contain a JSON object: ${res.output.slice(0, 200)}`), res.usage);
+        let parsed;
+        try {
+            parsed = JSON.parse(match[0]);
+        }
+        catch {
+            throw withUsage(new Error(`Reviewer output contained malformed JSON: ${match[0].slice(0, 200)}`), res.usage);
+        }
+        if (typeof parsed.needInfo === 'string' && parsed.needInfo.trim()) {
+            // Forward LLM usage on the signal path so the coordinator can attribute
+            // reviewer spend even when the critic short-circuits with a signal.
+            throw new NeedInfoSignal(parsed.needInfo, res.usage);
+        }
+        if (typeof parsed.clarify === 'string' && parsed.clarify.trim()) {
+            throw new ClarifySignal(parsed.clarify, res.usage);
+        }
+        if (typeof parsed.pass !== 'boolean') {
+            throw withUsage(new Error(`Reviewer verdict must have a boolean 'pass': ${match[0].slice(0, 200)}`), res.usage);
+        }
+        if (parsed.pass === false) {
+            if (typeof parsed.feedback !== 'string' ||
+                parsed.feedback.trim() === '') {
+                throw withUsage(new Error(`Reviewer rejection must include a non-empty 'feedback' string: ${match[0].slice(0, 200)}`), res.usage);
+            }
+            return {
+                verdict: { pass: false, feedback: parsed.feedback },
+                usage: res.usage,
+            };
+        }
+        return { verdict: { pass: true }, usage: res.usage };
+    }
+    async reviewExecutionFailure(input) {
+        const catalog = input.agents
+            .map((a) => `- ${a.name}: ${a.description ?? '(no description)'}`)
+            .join('\n');
+        const traceText = input.trace
+            .map((r) => `- ${r.nodeId} [${r.status}]: ${r.output || r.error || ''}`)
+            .join('\n');
+        const contextPrefix = input.ancestorContext
+            ? `${renderAncestorContext(input.ancestorContext)}\n\n`
+            : '';
+        const task = `${contextPrefix}Objective: ${input.objective ?? '(none)'}\n\nAvailable workers:\n${catalog || '(none)'}\n\nCurrent plan (JSON):\n${JSON.stringify(input.plan)}\n\nExecution trace (current state):\n${traceText || '(nothing completed)'}\n\nFailed step: ${input.failedNodeId}\nError: ${input.error}`;
+        const res = await this.executionAgent.run({
+            task,
+            sessionId: input.sessionId,
+            signal: input.signal,
+        });
+        const match = res.output.match(/\{[\s\S]*\}/);
+        if (!match)
+            throw withUsage(new Error(`Recovery reviewer output did not contain a JSON object: ${res.output.slice(0, 200)}`), res.usage);
+        let parsed;
+        try {
+            parsed = JSON.parse(match[0]);
+        }
+        catch {
+            throw withUsage(new Error(`Recovery reviewer output contained malformed JSON: ${match[0].slice(0, 200)}`), res.usage);
+        }
+        if (typeof parsed.needInfo === 'string' && parsed.needInfo.trim()) {
+            throw new NeedInfoSignal(parsed.needInfo, res.usage);
+        }
+        if (typeof parsed.clarify === 'string' && parsed.clarify.trim()) {
+            throw new ClarifySignal(parsed.clarify, res.usage);
+        }
+        if (parsed.action === 'abort') {
+            return { decision: { action: 'abort' }, usage: res.usage };
+        }
+        if (parsed.action !== 'revise') {
+            throw withUsage(new Error(`Recovery reviewer action must be 'abort' | 'revise': ${match[0].slice(0, 200)}`), res.usage);
+        }
+        const plan = parsed.plan;
+        if (!plan ||
+            !Array.isArray(plan.nodes) ||
+            plan.nodes.length === 0 ||
+            plan.nodes.some((n) => typeof n.id !== 'string' ||
+                typeof n.goal !== 'string' ||
+                (n.goal ?? '').trim() === '')) {
+            throw withUsage(new Error(`Recovery reviewer revise plan must have non-empty nodes with string id+goal: ${match[0].slice(0, 200)}`), res.usage);
+        }
+        return {
+            decision: { action: 'revise', revisedPlan: plan },
+            usage: res.usage,
+        };
+    }
+}
+//# sourceMappingURL=llm-review-strategy.js.map

package/dist/coordinator/dag/llm-review-strategy.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"llm-review-strategy.js","sourceRoot":"","sources":["../../../src/coordinator/dag/llm-review-strategy.ts"],"names":[],"mappings":"AAUA,OAAO,EAAE,aAAa,EAAE,cAAc,EAAE,MAAM,yBAAyB,CAAC;AACxE,OAAO,EAAE,iBAAiB,EAAE,MAAM,uCAAuC,CAAC;AAC1E,OAAO,EAAE,qBAAqB,EAAE,MAAM,8BAA8B,CAAC;AAErE;;;GAGG;AACH,SAAS,SAAS,CAAC,GAAU,EAAE,KAA2B;IACxD,IAAI,KAAK;QAAG,GAAoC,CAAC,KAAK,GAAG,KAAK,CAAC;IAC/D,OAAO,GAAG,CAAC;AACb,CAAC;AAED,gFAAgF;AAChF,8CAA8C;AAC9C,MAAM,eAAe,GAAG;;;;;+FAKuE,CAAC;AAEhG,MAAM,uBAAuB,GAAG;;;;;;6MAM6K,CAAC;AAE9M;;;GAGG;AACH,MAAM,OAAO,iBAAiB;IACnB,IAAI,GAAG,YAAY,CAAC;IAC7B;gFAC4E;IACnE,KAAK,CAAU;IACP,KAAK,CAAoB;IACzB,cAAc,CAAoB;IAEnD,YAAY,GAAS;QACnB,IAAI,CAAC,KAAK,GAAG,GAAG,CAAC,KAAK,CAAC;QACvB,IAAI,CAAC,KAAK,GAAG,IAAI,iBAAiB,CAAC,UAAU,EAAE,GAAG,EAAE;YAClD,YAAY,EAAE,eAAe;YAC7B,aAAa,EAAE,UAAU;SAC1B,CAAC,CAAC;QACH,IAAI,CAAC,cAAc,GAAG,IAAI,iBAAiB,CAAC,mBAAmB,EAAE,GAAG,EAAE;YACpE,YAAY,EAAE,uBAAuB;YACrC,aAAa,EAAE,UAAU;SAC1B,CAAC,CAAC;IACL,CAAC;IAED,KAAK,CAAC,MAAM,CAAC,KAAkB;QAC7B,MAAM,OAAO,GAAG,KAAK,CAAC,MAAM;aACzB,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,CAAC,IAAI,KAAK,CAAC,CAAC,WAAW,IAAI,kBAAkB,EAAE,CAAC;aACjE,IAAI,CAAC,IAAI,CAAC,CAAC;QACd,MAAM,aAAa,GAAG,KAAK,CAAC,eAAe;YACzC,CAAC,CAAC,GAAG,qBAAqB,CAAC,KAAK,CAAC,eAAe,CAAC,MAAM;YACvD,CAAC,CAAC,EAAE,CAAC;QACP,MAAM,IAAI,GAAG,GAAG,aAAa,kBAAkB,KAAK,CAAC,MAAM,2BACzD,OAAO,IAAI,QACb,8BAA8B,IAAI,CAAC,SAAS,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC;QAE3D,MAAM,GAAG,GAAG,MAAM,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC;YAC/B,IAAI;YACJ,SAAS,EAAE,KAAK,CAAC,SAAS;YAC1B,MAAM,EAAE,KAAK,CAAC,MAAM;SACrB,CAAC,CAAC;QAEH,MAAM,KAAK,GAAG,GAAG,CAAC,MAAM,CAAC,KAAK,CAAC,aAAa,CAAC,CAAC;QAC9C,IAAI,CAAC,KAAK;YACR,MAAM,SAAS,CACb,IAAI,KAAK,CACP,kDAAkD,GAAG,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,EAAE,CAC7E,EACD,GAAG,CAAC,KAAK,CACV,CAAC;QACJ,IAAI,MAKH,CAAC;QACF,IAAI,CAAC;YACH,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QAChC,CAAC;QAAC,MAAM,CAAC;YACP,MAAM,SAAS,CACb,IAAI,KAAK,CACP,6CAA6C,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,EAAE,CACtE,EACD,GAAG,CAAC,KAAK,CACV,CAAC;QACJ,CAAC;QACD,IAAI,OAAO,MAAM,CAAC,QAAQ,KAAK,QAAQ,IAAI,MAAM,CAAC,QAAQ,CAAC,IAAI,EAAE,EAAE,CAAC;YAClE,wEAAwE;YACxE,oEAAoE;YACpE,MAAM,IAAI,cAAc,CAAC,MAAM,CAAC,QAAQ,EAAE,GAAG,CAAC,KAAK,CAAC,CAAC;QACvD,CAAC;QACD,IAAI,OAAO,MAAM,CAAC,OAAO,KAAK,QAAQ,IAAI,MAAM,CAAC,OAAO,CAAC,IAAI,EAAE,EAAE,CAAC;YAChE,MAAM,IAAI,aAAa,CAAC,MAAM,CAAC,OAAO,EAAE,GAAG,CAAC,KAAK,CAAC,CAAC;QACrD,CAAC;QACD,IAAI,OAAO,MAAM,CAAC,IAAI,KAAK,SAAS,EAAE,CAAC;YACrC,MAAM,SAAS,CACb,IAAI,KAAK,CACP,gDAAgD,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,EAAE,CACzE,EACD,GAAG,CAAC,KAAK,CACV,CAAC;QACJ,CAAC;QACD,IAAI,MAAM,CAAC,IAAI,KAAK,KAAK,EAAE,CAAC;YAC1B,IACE,OAAO,MAAM,CAAC,QAAQ,KAAK,QAAQ;gBACnC,MAAM,CAAC,QAAQ,CAAC,IAAI,EAAE,KAAK,EAAE,EAC7B,CAAC;gBACD,MAAM,SAAS,CACb,IAAI,KAAK,CACP,kEAAkE,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,EAAE,CAC3F,EACD,GAAG,CAAC,KAAK,CACV,CAAC;YACJ,CAAC;YACD,OAAO;gBACL,OAAO,EAAE,EAAE,IAAI,EAAE,KAAK,EAAE,QAAQ,EAAE,MAAM,CAAC,QAAQ,EAAE;gBACnD,KAAK,EAAE,GAAG,CAAC,KAAK;aACjB,CAAC;QACJ,CAAC;QACD,OAAO,EAAE,OAAO,EAAE,EAAE,IAAI,EAAE,IAAI,EAAE,EAAE,KAAK,EAAE,GAAG,CAAC,KAAK,EAAE,CAAC;IACvD,CAAC;IAED,KAAK,CAAC,sBAAsB,CAC1B,KAA4B;QAE5B,MAAM,OAAO,GAAG,KAAK,CAAC,MAAM;aACzB,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,CAAC,IAAI,KAAK,CAAC,CAAC,WAAW,IAAI,kBAAkB,EAAE,CAAC;aACjE,IAAI,CAAC,IAAI,CAAC,CAAC;QACd,MAAM,SAAS,GAAG,KAAK,CAAC,KAAK;aAC1B,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,CAAC,MAAM,KAAK,CAAC,CAAC,MAAM,MAAM,CAAC,CAAC,MAAM,IAAI,CAAC,CAAC,KAAK,IAAI,EAAE,EAAE,CAAC;aACvE,IAAI,CAAC,IAAI,CAAC,CAAC;QACd,MAAM,aAAa,GAAG,KAAK,CAAC,eAAe;YACzC,CAAC,CAAC,GAAG,qBAAqB,CAAC,KAAK,CAAC,eAAe,CAAC,MAAM;YACvD,CAAC,CAAC,EAAE,CAAC;QACP,MAAM,IAAI,GAAG,GAAG,aAAa,cAAc,KAAK,CAAC,SAAS,IAAI,QAAQ,2BACpE,OAAO,IAAI,QACb,6BAA6B,IAAI,CAAC,SAAS,CAAC,KAAK,CAAC,IAAI,CAAC,yCACrD,SAAS,IAAI,qBACf,oBAAoB,KAAK,CAAC,YAAY,YAAY,KAAK,CAAC,KAAK,EAAE,CAAC;QAEhE,MAAM,GAAG,GAAG,MAAM,IAAI,CAAC,cAAc,CAAC,GAAG,CAAC;YACxC,IAAI;YACJ,SAAS,EAAE,KAAK,CAAC,SAAS;YAC1B,MAAM,EAAE,KAAK,CAAC,MAAM;SACrB,CAAC,CAAC;QAEH,MAAM,KAAK,GAAG,GAAG,CAAC,MAAM,CAAC,KAAK,CAAC,aAAa,CAAC,CAAC;QAC9C,IAAI,CAAC,KAAK;YACR,MAAM,SAAS,CACb,IAAI,KAAK,CACP,2DAA2D,GAAG,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,EAAE,CACtF,EACD,GAAG,CAAC,KAAK,CACV,CAAC;QACJ,IAAI,MAKH,CAAC;QACF,IAAI,CAAC;YACH,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;QAChC,CAAC;QAAC,MAAM,CAAC;YACP,MAAM,SAAS,CACb,IAAI,KAAK,CACP,sDAAsD,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,EAAE,CAC/E,EACD,GAAG,CAAC,KAAK,CACV,CAAC;QACJ,CAAC;QACD,IAAI,OAAO,MAAM,CAAC,QAAQ,KAAK,QAAQ,IAAI,MAAM,CAAC,QAAQ,CAAC,IAAI,EAAE,EAAE,CAAC;YAClE,MAAM,IAAI,cAAc,CAAC,MAAM,CAAC,QAAQ,EAAE,GAAG,CAAC,KAAK,CAAC,CAAC;QACvD,CAAC;QACD,IAAI,OAAO,MAAM,CAAC,OAAO,KAAK,QAAQ,IAAI,MAAM,CAAC,OAAO,CAAC,IAAI,EAAE,EAAE,CAAC;YAChE,MAAM,IAAI,aAAa,CAAC,MAAM,CAAC,OAAO,EAAE,GAAG,CAAC,KAAK,CAAC,CAAC;QACrD,CAAC;QACD,IAAI,MAAM,CAAC,MAAM,KAAK,OAAO,EAAE,CAAC;YAC9B,OAAO,EAAE,QAAQ,EAAE,EAAE,MAAM,EAAE,OAAO,EAAE,EAAE,KAAK,EAAE,GAAG,CAAC,KAAK,EAAE,CAAC;QAC7D,CAAC;QACD,IAAI,MAAM,CAAC,MAAM,KAAK,QAAQ,EAAE,CAAC;YAC/B,MAAM,SAAS,CACb,IAAI,KAAK,CACP,wDAAwD,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,EAAE,CACjF,EACD,GAAG,CAAC,KAAK,CACV,CAAC;QACJ,CAAC;QACD,MAAM,IAAI,GAAG,MAAM,CAAC,IAAuC,CAAC;QAC5D,IACE,CAAC,IAAI;YACL,CAAC,KAAK,CAAC,OAAO,CAAC,IAAI,CAAC,KAAK,CAAC;YAC1B,IAAI,CAAC,KAAK,CAAC,MAAM,KAAK,CAAC;YACvB,IAAI,CAAC,KAAK,CAAC,IAAI,CACb,CAAC,CAAC,EAAE,EAAE,CACJ,OAAQ,CAAsB,CAAC,EAAE,KAAK,QAAQ;gBAC9C,OAAQ,CAAwB,CAAC,IAAI,KAAK,QAAQ;gBAClD,CAAE,CAAuB,CAAC,IAAI,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,KAAK,EAAE,CACtD,EACD,CAAC;YACD,MAAM,SAAS,CACb,IAAI,KAAK,CACP,gFAAgF,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,EAAE,CACzG,EACD,GAAG,CAAC,KAAK,CACV,CAAC;QACJ,CAAC;QACD,OAAO;YACL,QAAQ,EAAE,EAAE,MAAM,EAAE,QAAQ,EAAE,WAAW,EAAE,IAAe,EAAE;YAC5D,KAAK,EAAE,GAAG,CAAC,KAAK;SACjB,CAAC;IACJ,CAAC;CACF"}

package/dist/coordinator/dag/noop-review-strategy.d.ts

@@ -0,0 +1,8 @@
+import type { ExecutionReviewResult, IReviewStrategy, ReviewResult } from '@mcp-abap-adt/llm-agent';
+/** Always-pass reviewer; always-abort recovery. Explicit opt-out / test double. */
+export declare class NoopReviewStrategy implements IReviewStrategy {
+    readonly name = "noop-review";
+    review(): Promise<ReviewResult>;
+    reviewExecutionFailure(): Promise<ExecutionReviewResult>;
+}
+//# sourceMappingURL=noop-review-strategy.d.ts.map

package/dist/coordinator/dag/noop-review-strategy.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"noop-review-strategy.d.ts","sourceRoot":"","sources":["../../../src/coordinator/dag/noop-review-strategy.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,qBAAqB,EACrB,eAAe,EACf,YAAY,EACb,MAAM,yBAAyB,CAAC;AAEjC,mFAAmF;AACnF,qBAAa,kBAAmB,YAAW,eAAe;IACxD,QAAQ,CAAC,IAAI,iBAAiB;IACxB,MAAM,IAAI,OAAO,CAAC,YAAY,CAAC;IAG/B,sBAAsB,IAAI,OAAO,CAAC,qBAAqB,CAAC;CAG/D"}

package/dist/coordinator/dag/noop-review-strategy.js

@@ -0,0 +1,11 @@
+/** Always-pass reviewer; always-abort recovery. Explicit opt-out / test double. */
+export class NoopReviewStrategy {
+    name = 'noop-review';
+    async review() {
+        return { verdict: { pass: true } };
+    }
+    async reviewExecutionFailure() {
+        return { decision: { action: 'abort' } };
+    }
+}
+//# sourceMappingURL=noop-review-strategy.js.map

package/dist/coordinator/dag/noop-review-strategy.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"noop-review-strategy.js","sourceRoot":"","sources":["../../../src/coordinator/dag/noop-review-strategy.ts"],"names":[],"mappings":"AAMA,mFAAmF;AACnF,MAAM,OAAO,kBAAkB;IACpB,IAAI,GAAG,aAAa,CAAC;IAC9B,KAAK,CAAC,MAAM;QACV,OAAO,EAAE,OAAO,EAAE,EAAE,IAAI,EAAE,IAAI,EAAE,EAAE,CAAC;IACrC,CAAC;IACD,KAAK,CAAC,sBAAsB;QAC1B,OAAO,EAAE,QAAQ,EAAE,EAAE,MAAM,EAAE,OAAO,EAAE,EAAE,CAAC;IAC3C,CAAC;CACF"}

package/dist/coordinator/dag/passthrough-finalizer.d.ts

@@ -0,0 +1,11 @@
+import type { FinalizerInput, FinalizerResult, IFinalizer } from '@mcp-abap-adt/llm-agent';
+/**
+ * Default finalizer. Returns the interpreter's already-joined output
+ * verbatim — exactly what the DAG coordinator yielded before IFinalizer
+ * was introduced. No LLM call; no usage attributed.
+ */
+export declare class PassthroughFinalizer implements IFinalizer {
+    readonly name = "passthrough";
+    finalize(input: FinalizerInput): Promise<FinalizerResult>;
+}
+//# sourceMappingURL=passthrough-finalizer.d.ts.map

package/dist/coordinator/dag/passthrough-finalizer.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"passthrough-finalizer.d.ts","sourceRoot":"","sources":["../../../src/coordinator/dag/passthrough-finalizer.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,cAAc,EACd,eAAe,EACf,UAAU,EACX,MAAM,yBAAyB,CAAC;AAEjC;;;;GAIG;AACH,qBAAa,oBAAqB,YAAW,UAAU;IACrD,QAAQ,CAAC,IAAI,iBAAiB;IAExB,QAAQ,CAAC,KAAK,EAAE,cAAc,GAAG,OAAO,CAAC,eAAe,CAAC;CAIhE"}

package/dist/coordinator/dag/passthrough-finalizer.js

@@ -0,0 +1,13 @@
+/**
+ * Default finalizer. Returns the interpreter's already-joined output
+ * verbatim — exactly what the DAG coordinator yielded before IFinalizer
+ * was introduced. No LLM call; no usage attributed.
+ */
+export class PassthroughFinalizer {
+    name = 'passthrough';
+    async finalize(input) {
+        input.onPartial?.({ kind: 'content', delta: input.interpreterOutput });
+        return { output: input.interpreterOutput };
+    }
+}
+//# sourceMappingURL=passthrough-finalizer.js.map

package/dist/coordinator/dag/passthrough-finalizer.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"passthrough-finalizer.js","sourceRoot":"","sources":["../../../src/coordinator/dag/passthrough-finalizer.ts"],"names":[],"mappings":"AAMA;;;;GAIG;AACH,MAAM,OAAO,oBAAoB;IACtB,IAAI,GAAG,aAAa,CAAC;IAE9B,KAAK,CAAC,QAAQ,CAAC,KAAqB;QAClC,KAAK,CAAC,SAAS,EAAE,CAAC,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,KAAK,CAAC,iBAAiB,EAAE,CAAC,CAAC;QACvE,OAAO,EAAE,MAAM,EAAE,KAAK,CAAC,iBAAiB,EAAE,CAAC;IAC7C,CAAC;CACF"}

package/dist/coordinator/dag/render-ancestor-context.d.ts

@@ -0,0 +1,8 @@
+import type { ContextPath } from '@mcp-abap-adt/llm-agent';
+/**
+ * Render an ancestor intent path (objective + clarification dialogue + oracle
+ * observations) into plain text, prepended to a role's task. Shared by the
+ * planner and reviewer role adapters.
+ */
+export declare function renderAncestorContext(ac: ContextPath): string;
+//# sourceMappingURL=render-ancestor-context.d.ts.map

package/dist/coordinator/dag/render-ancestor-context.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"render-ancestor-context.d.ts","sourceRoot":"","sources":["../../../src/coordinator/dag/render-ancestor-context.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,yBAAyB,CAAC;AAE3D;;;;GAIG;AACH,wBAAgB,qBAAqB,CAAC,EAAE,EAAE,WAAW,GAAG,MAAM,CAoB7D"}

package/dist/coordinator/dag/render-ancestor-context.js

@@ -0,0 +1,27 @@
+/**
+ * Render an ancestor intent path (objective + clarification dialogue + oracle
+ * observations) into plain text, prepended to a role's task. Shared by the
+ * planner and reviewer role adapters.
+ */
+export function renderAncestorContext(ac) {
+    const lines = [];
+    if (ac.objective) {
+        lines.push(`Ancestor objective: ${ac.objective}`);
+    }
+    if (ac.clarifications.length > 0) {
+        lines.push('Prior clarifications:');
+        for (const c of ac.clarifications) {
+            lines.push(`  Q: ${c.question}`);
+            lines.push(`  A: ${c.answer}`);
+        }
+    }
+    if (ac.oracleObservations.length > 0) {
+        lines.push('Oracle observations:');
+        for (const o of ac.oracleObservations) {
+            lines.push(`  Query: ${o.query}`);
+            lines.push(`  Answer: ${o.answer}`);
+        }
+    }
+    return lines.join('\n');
+}
+//# sourceMappingURL=render-ancestor-context.js.map

package/dist/coordinator/dag/render-ancestor-context.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"render-ancestor-context.js","sourceRoot":"","sources":["../../../src/coordinator/dag/render-ancestor-context.ts"],"names":[],"mappings":"AAEA;;;;GAIG;AACH,MAAM,UAAU,qBAAqB,CAAC,EAAe;IACnD,MAAM,KAAK,GAAa,EAAE,CAAC;IAC3B,IAAI,EAAE,CAAC,SAAS,EAAE,CAAC;QACjB,KAAK,CAAC,IAAI,CAAC,uBAAuB,EAAE,CAAC,SAAS,EAAE,CAAC,CAAC;IACpD,CAAC;IACD,IAAI,EAAE,CAAC,cAAc,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACjC,KAAK,CAAC,IAAI,CAAC,uBAAuB,CAAC,CAAC;QACpC,KAAK,MAAM,CAAC,IAAI,EAAE,CAAC,cAAc,EAAE,CAAC;YAClC,KAAK,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,QAAQ,EAAE,CAAC,CAAC;YACjC,KAAK,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC;QACjC,CAAC;IACH,CAAC;IACD,IAAI,EAAE,CAAC,kBAAkB,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACrC,KAAK,CAAC,IAAI,CAAC,sBAAsB,CAAC,CAAC;QACnC,KAAK,MAAM,CAAC,IAAI,EAAE,CAAC,kBAAkB,EAAE,CAAC;YACtC,KAAK,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC,KAAK,EAAE,CAAC,CAAC;YAClC,KAAK,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC;QACtC,CAAC;IACH,CAAC;IACD,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC1B,CAAC"}

package/dist/coordinator/dag/replan-error-strategy.d.ts

@@ -0,0 +1,15 @@
+import { type ErrorContext, type ErrorReaction, type IErrorStrategy, type IPlanner, type PlanNode } from '@mcp-abap-adt/llm-agent';
+/**
+ * Replans a node ONLY for NeedsDecompositionError (the explicit "decompose me"
+ * signal). Any other error → abort (a transient MCP/LLM failure is not fixed by
+ * decomposition). Stateless: holds the maxReplans ceiling but never counts — the
+ * interpreter owns the per-run counter and passes `remainingReplans`.
+ */
+export declare class ReplanErrorStrategy implements IErrorStrategy {
+    private readonly planner;
+    readonly maxReplans: number;
+    readonly name = "replan";
+    constructor(planner: IPlanner, maxReplans?: number);
+    onNodeFailure(_node: PlanNode, error: unknown, ctx: ErrorContext): Promise<ErrorReaction>;
+}
+//# sourceMappingURL=replan-error-strategy.d.ts.map

package/dist/coordinator/dag/replan-error-strategy.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"replan-error-strategy.d.ts","sourceRoot":"","sources":["../../../src/coordinator/dag/replan-error-strategy.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,KAAK,YAAY,EACjB,KAAK,aAAa,EAClB,KAAK,cAAc,EACnB,KAAK,QAAQ,EAEb,KAAK,QAAQ,EACd,MAAM,yBAAyB,CAAC;AAEjC;;;;;GAKG;AACH,qBAAa,mBAAoB,YAAW,cAAc;IAGtD,OAAO,CAAC,QAAQ,CAAC,OAAO;IACxB,QAAQ,CAAC,UAAU;IAHrB,QAAQ,CAAC,IAAI,YAAY;gBAEN,OAAO,EAAE,QAAQ,EACzB,UAAU,SAAI;IAGnB,aAAa,CACjB,KAAK,EAAE,QAAQ,EACf,KAAK,EAAE,OAAO,EACd,GAAG,EAAE,YAAY,GAChB,OAAO,CAAC,aAAa,CAAC;CAe1B"}

package/dist/coordinator/dag/replan-error-strategy.js

@@ -0,0 +1,32 @@
+import { NeedsDecompositionError, } from '@mcp-abap-adt/llm-agent';
+/**
+ * Replans a node ONLY for NeedsDecompositionError (the explicit "decompose me"
+ * signal). Any other error → abort (a transient MCP/LLM failure is not fixed by
+ * decomposition). Stateless: holds the maxReplans ceiling but never counts — the
+ * interpreter owns the per-run counter and passes `remainingReplans`.
+ */
+export class ReplanErrorStrategy {
+    planner;
+    maxReplans;
+    name = 'replan';
+    constructor(planner, maxReplans = 4) {
+        this.planner = planner;
+        this.maxReplans = maxReplans;
+    }
+    async onNodeFailure(_node, error, ctx) {
+        if (!(error instanceof NeedsDecompositionError)) {
+            return { action: 'abort' };
+        }
+        if (ctx.remainingReplans <= 0) {
+            return { action: 'abort' };
+        }
+        const { plan: subPlan } = await this.planner.plan({
+            prompt: `${ctx.task}\n\nThis task needs decomposition: ${error.reason}`,
+            agents: ctx.agents,
+            sessionId: ctx.sessionId,
+            signal: ctx.signal,
+        });
+        return { action: 'replan', subPlan };
+    }
+}
+//# sourceMappingURL=replan-error-strategy.js.map

package/dist/coordinator/dag/replan-error-strategy.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"replan-error-strategy.js","sourceRoot":"","sources":["../../../src/coordinator/dag/replan-error-strategy.ts"],"names":[],"mappings":"AAAA,OAAO,EAKL,uBAAuB,GAExB,MAAM,yBAAyB,CAAC;AAEjC;;;;;GAKG;AACH,MAAM,OAAO,mBAAmB;IAGX;IACR;IAHF,IAAI,GAAG,QAAQ,CAAC;IACzB,YACmB,OAAiB,EACzB,aAAa,CAAC;QADN,YAAO,GAAP,OAAO,CAAU;QACzB,eAAU,GAAV,UAAU,CAAI;IACtB,CAAC;IAEJ,KAAK,CAAC,aAAa,CACjB,KAAe,EACf,KAAc,EACd,GAAiB;QAEjB,IAAI,CAAC,CAAC,KAAK,YAAY,uBAAuB,CAAC,EAAE,CAAC;YAChD,OAAO,EAAE,MAAM,EAAE,OAAO,EAAE,CAAC;QAC7B,CAAC;QACD,IAAI,GAAG,CAAC,gBAAgB,IAAI,CAAC,EAAE,CAAC;YAC9B,OAAO,EAAE,MAAM,EAAE,OAAO,EAAE,CAAC;QAC7B,CAAC;QACD,MAAM,EAAE,IAAI,EAAE,OAAO,EAAE,GAAG,MAAM,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC;YAChD,MAAM,EAAE,GAAG,GAAG,CAAC,IAAI,sCAAsC,KAAK,CAAC,MAAM,EAAE;YACvE,MAAM,EAAE,GAAG,CAAC,MAAM;YAClB,SAAS,EAAE,GAAG,CAAC,SAAS;YACxB,MAAM,EAAE,GAAG,CAAC,MAAM;SACnB,CAAC,CAAC;QACH,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,OAAO,EAAE,CAAC;IACvC,CAAC;CACF"}