@namzu/sdk 2.0.0 → 3.0.0

package/dist/advisory/__tests__/budget-enforcement.test.js

@@ -0,0 +1,159 @@
+import { describe, expect, it, vi } from 'vitest';
+import { assertBudgetEnforceable } from '../budget.js';
+import { AdvisoryContext } from '../context.js';
+import { TriggerEvaluator } from '../evaluator.js';
+import { AdvisoryExecutor } from '../executor.js';
+import { AdvisorRegistry } from '../registry.js';
+/**
+ * `AdvisoryBudget` declared six caps and enforced one. The other five were
+ * read by nothing: a host could set `maxCostPerRun` and watch an advisor
+ * spend without limit, and the only signal that the cap was inert was that
+ * nothing ever happened.
+ *
+ * Two of the six could not be honoured at all — the advisory stack is built
+ * per run, so a per-SESSION cap had no accumulator to count against — and
+ * those were removed rather than left as decoration. What remains is
+ * enforced here.
+ */
+function record(cost) {
+    return {
+        advisorId: 'a',
+        request: { question: 'q' },
+        result: { advice: '' },
+        usage: {
+            promptTokens: 0,
+            completionTokens: 0,
+            totalTokens: 0,
+            cachedTokens: 0,
+            cacheWriteTokens: 0,
+        },
+        cost: { inputCostPer1M: 0, outputCostPer1M: 0, totalCost: cost, cacheDiscount: 0 },
+        durationMs: 1,
+        iteration: 1,
+        timestamp: 0,
+    };
+}
+function contextWith(budget) {
+    return new AdvisoryContext(new AdvisorRegistry([]), new AdvisoryExecutor(), new TriggerEvaluator([]), budget);
+}
+describe('a per-run cost cap actually stops the next call', () => {
+    it('allows a call while the accumulated cost is under the cap', () => {
+        const ctx = contextWith({ maxCostPerRun: 1 });
+        ctx.recordCall(record(0.4));
+        expect(ctx.checkBudget().allowed).toBe(true);
+    });
+    it('refuses once the accumulated cost reaches the cap', () => {
+        const ctx = contextWith({ maxCostPerRun: 1 });
+        ctx.recordCall(record(0.6));
+        ctx.recordCall(record(0.5));
+        const verdict = ctx.checkBudget();
+        expect(verdict.allowed).toBe(false);
+        // The operator gets the number they set and the number they hit;
+        // "budget exhausted" alone leaves them guessing which cap tripped.
+        expect(verdict.reason).toContain('1.1');
+        expect(verdict.reason).toContain('cost');
+    });
+    it('leaves the call cap working alongside it', () => {
+        const ctx = contextWith({ maxCallsPerRun: 1 });
+        ctx.recordCall(record(0));
+        expect(ctx.checkBudget().allowed).toBe(false);
+    });
+});
+describe('a cost cap without pricing is refused, not silently inert', () => {
+    const advisor = (pricing) => ({
+        id: 'a',
+        name: 'A',
+        model: 'm',
+        provider: {},
+        ...(pricing ? { pricing } : {}),
+    });
+    it('throws when a cost cap is set and an advisor carries no pricing', () => {
+        expect(() => assertBudgetEnforceable({ advisors: [advisor()], budget: { maxCostPerRun: 5 } })).toThrow(/pricing/i);
+    });
+    it('accepts the same cap once pricing is supplied', () => {
+        expect(() => assertBudgetEnforceable({
+            advisors: [advisor({ inputCostPer1M: 1, outputCostPer1M: 2 })],
+            budget: { maxCostPerRun: 5 },
+        })).not.toThrow();
+    });
+    it('says nothing about pricing when no cost cap is set', () => {
+        expect(() => assertBudgetEnforceable({ advisors: [advisor()], budget: { maxCallsPerRun: 2 } })).not.toThrow();
+    });
+});
+describe('a per-call token cap reaches the provider', () => {
+    function providerCapturing(seen) {
+        return {
+            chatStream: vi.fn(async function* (params) {
+                seen.maxTokens = params.maxTokens;
+                yield { id: 'r', delta: { content: 'ok' } };
+                yield {
+                    id: 'r',
+                    delta: {},
+                    finishReason: 'stop',
+                    usage: {
+                        promptTokens: 1,
+                        completionTokens: 1,
+                        totalTokens: 2,
+                        cachedTokens: 0,
+                        cacheWriteTokens: 0,
+                    },
+                };
+            }),
+        };
+    }
+    it('clamps an advisor that asks for more than the budget allows', async () => {
+        const seen = {};
+        const exec = new AdvisoryExecutor(undefined, { maxTokensPerCall: 100 });
+        await exec.consult({
+            id: 'a',
+            name: 'A',
+            model: 'm',
+            provider: providerCapturing(seen),
+            maxResponseTokens: 5000,
+        }, { question: 'q' }, { messages: [], iteration: 1 });
+        expect(seen.maxTokens).toBe(100);
+    });
+    it('applies the cap to an advisor that names no ceiling of its own', async () => {
+        const seen = {};
+        const exec = new AdvisoryExecutor(undefined, { maxTokensPerCall: 100 });
+        await exec.consult({ id: 'a', name: 'A', model: 'm', provider: providerCapturing(seen) }, { question: 'q' }, { messages: [], iteration: 1 });
+        expect(seen.maxTokens).toBe(100);
+    });
+    it('leaves a smaller advisor ceiling alone', async () => {
+        const seen = {};
+        const exec = new AdvisoryExecutor(undefined, { maxTokensPerCall: 100 });
+        await exec.consult({ id: 'a', name: 'A', model: 'm', provider: providerCapturing(seen), maxResponseTokens: 40 }, { question: 'q' }, { messages: [], iteration: 1 });
+        expect(seen.maxTokens).toBe(40);
+    });
+});
+describe('cost is real when the advisor carries pricing', () => {
+    it('computes from the advisor pricing rather than reporting zero', async () => {
+        const provider = {
+            chatStream: vi.fn(async function* () {
+                yield { id: 'r', delta: { content: 'ok' } };
+                yield {
+                    id: 'r',
+                    delta: {},
+                    finishReason: 'stop',
+                    usage: {
+                        promptTokens: 1_000_000,
+                        completionTokens: 1_000_000,
+                        totalTokens: 2_000_000,
+                        cachedTokens: 0,
+                        cacheWriteTokens: 0,
+                    },
+                };
+            }),
+        };
+        const exec = new AdvisoryExecutor();
+        const out = await exec.consult({
+            id: 'a',
+            name: 'A',
+            model: 'm',
+            provider,
+            pricing: { inputCostPer1M: 3, outputCostPer1M: 15 },
+        }, { question: 'q' }, { messages: [], iteration: 1 });
+        expect(out.cost.totalCost).toBe(18);
+    });
+});
+//# sourceMappingURL=budget-enforcement.test.js.map

package/dist/advisory/__tests__/budget-enforcement.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"budget-enforcement.test.js","sourceRoot":"","sources":["../../../src/advisory/__tests__/budget-enforcement.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAA;AAIjD,OAAO,EAAE,uBAAuB,EAAE,MAAM,cAAc,CAAA;AACtD,OAAO,EAAE,eAAe,EAAE,MAAM,eAAe,CAAA;AAC/C,OAAO,EAAE,gBAAgB,EAAE,MAAM,iBAAiB,CAAA;AAClD,OAAO,EAAE,gBAAgB,EAAE,MAAM,gBAAgB,CAAA;AACjD,OAAO,EAAE,eAAe,EAAE,MAAM,gBAAgB,CAAA;AAEhD;;;;;;;;;;GAUG;AAEH,SAAS,MAAM,CAAC,IAAY;IAC3B,OAAO;QACN,SAAS,EAAE,GAAG;QACd,OAAO,EAAE,EAAE,QAAQ,EAAE,GAAG,EAAE;QAC1B,MAAM,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE;QACtB,KAAK,EAAE;YACN,YAAY,EAAE,CAAC;YACf,gBAAgB,EAAE,CAAC;YACnB,WAAW,EAAE,CAAC;YACd,YAAY,EAAE,CAAC;YACf,gBAAgB,EAAE,CAAC;SACnB;QACD,IAAI,EAAE,EAAE,cAAc,EAAE,CAAC,EAAE,eAAe,EAAE,CAAC,EAAE,SAAS,EAAE,IAAI,EAAE,aAAa,EAAE,CAAC,EAAE;QAClF,UAAU,EAAE,CAAC;QACb,SAAS,EAAE,CAAC;QACZ,SAAS,EAAE,CAAC;KACZ,CAAA;AACF,CAAC;AAED,SAAS,WAAW,CAAC,MAA+D;IACnF,OAAO,IAAI,eAAe,CACzB,IAAI,eAAe,CAAC,EAAE,CAAC,EACvB,IAAI,gBAAgB,EAAE,EACtB,IAAI,gBAAgB,CAAC,EAAE,CAAC,EACxB,MAAM,CACN,CAAA;AACF,CAAC;AAED,QAAQ,CAAC,iDAAiD,EAAE,GAAG,EAAE;IAChE,EAAE,CAAC,2DAA2D,EAAE,GAAG,EAAE;QACpE,MAAM,GAAG,GAAG,WAAW,CAAC,EAAE,aAAa,EAAE,CAAC,EAAE,CAAC,CAAA;QAC7C,GAAG,CAAC,UAAU,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAA;QAC3B,MAAM,CAAC,GAAG,CAAC,WAAW,EAAE,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;IAC7C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,mDAAmD,EAAE,GAAG,EAAE;QAC5D,MAAM,GAAG,GAAG,WAAW,CAAC,EAAE,aAAa,EAAE,CAAC,EAAE,CAAC,CAAA;QAC7C,GAAG,CAAC,UAAU,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAA;QAC3B,GAAG,CAAC,UAAU,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAA;QAE3B,MAAM,OAAO,GAAG,GAAG,CAAC,WAAW,EAAE,CAAA;QACjC,MAAM,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QACnC,iEAAiE;QACjE,mEAAmE;QACnE,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,KAAK,CAAC,CAAA;QACvC,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,MAAM,CAAC,CAAA;IACzC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,0CAA0C,EAAE,GAAG,EAAE;QACnD,MAAM,GAAG,GAAG,WAAW,CAAC,EAAE,cAAc,EAAE,CAAC,EAAE,CAAC,CAAA;QAC9C,GAAG,CAAC,UAAU,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAA;QACzB,MAAM,CAAC,GAAG,CAAC,WAAW,EAAE,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;IAC9C,CAAC,CAAC,CAAA;AACH,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,2DAA2D,EAAE,GAAG,EAAE;IAC1E,MAAM,OAAO,GAAG,CAAC,OAA6D,EAAE,EAAE,CACjF,CAAC;QACA,EAAE,EAAE,GAAG;QACP,IAAI,EAAE,GAAG;QACT,KAAK,EAAE,GAAG;QACV,QAAQ,EAAE,EAAmC;QAC7C,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,OAAO,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;KAC/B,CAAsB,CAAA;IAExB,EAAE,CAAC,iEAAiE,EAAE,GAAG,EAAE;QAC1E,MAAM,CAAC,GAAG,EAAE,CACX,uBAAuB,CAAC,EAAE,QAAQ,EAAE,CAAC,OAAO,EAAE,CAAC,EAAE,MAAM,EAAE,EAAE,aAAa,EAAE,CAAC,EAAE,EAAE,CAAC,CAChF,CAAC,OAAO,CAAC,UAAU,CAAC,CAAA;IACtB,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,+CAA+C,EAAE,GAAG,EAAE;QACxD,MAAM,CAAC,GAAG,EAAE,CACX,uBAAuB,CAAC;YACvB,QAAQ,EAAE,CAAC,OAAO,CAAC,EAAE,cAAc,EAAE,CAAC,EAAE,eAAe,EAAE,CAAC,EAAE,CAAC,CAAC;YAC9D,MAAM,EAAE,EAAE,aAAa,EAAE,CAAC,EAAE;SAC5B,CAAC,CACF,CAAC,GAAG,CAAC,OAAO,EAAE,CAAA;IAChB,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,oDAAoD,EAAE,GAAG,EAAE;QAC7D,MAAM,CAAC,GAAG,EAAE,CACX,uBAAuB,CAAC,EAAE,QAAQ,EAAE,CAAC,OAAO,EAAE,CAAC,EAAE,MAAM,EAAE,EAAE,cAAc,EAAE,CAAC,EAAE,EAAE,CAAC,CACjF,CAAC,GAAG,CAAC,OAAO,EAAE,CAAA;IAChB,CAAC,CAAC,CAAA;AACH,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,2CAA2C,EAAE,GAAG,EAAE;IAC1D,SAAS,iBAAiB,CAAC,IAA4B;QACtD,OAAO;YACN,UAAU,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,SAAS,CAAC,EAAE,MAElC;gBACA,IAAI,CAAC,SAAS,GAAG,MAAM,CAAC,SAAS,CAAA;gBACjC,MAAM,EAAE,EAAE,EAAE,GAAG,EAAE,KAAK,EAAE,EAAE,OAAO,EAAE,IAAI,EAAE,EAAiB,CAAA;gBAC1D,MAAM;oBACL,EAAE,EAAE,GAAG;oBACP,KAAK,EAAE,EAAE;oBACT,YAAY,EAAE,MAAM;oBACpB,KAAK,EAAE;wBACN,YAAY,EAAE,CAAC;wBACf,gBAAgB,EAAE,CAAC;wBACnB,WAAW,EAAE,CAAC;wBACd,YAAY,EAAE,CAAC;wBACf,gBAAgB,EAAE,CAAC;qBACnB;iBACc,CAAA;YACjB,CAAC,CAAC;SAC0C,CAAA;IAC9C,CAAC;IAED,EAAE,CAAC,6DAA6D,EAAE,KAAK,IAAI,EAAE;QAC5E,MAAM,IAAI,GAA2B,EAAE,CAAA;QACvC,MAAM,IAAI,GAAG,IAAI,gBAAgB,CAAC,SAAS,EAAE,EAAE,gBAAgB,EAAE,GAAG,EAAE,CAAC,CAAA;QACvE,MAAM,IAAI,CAAC,OAAO,CACjB;YACC,EAAE,EAAE,GAAG;YACP,IAAI,EAAE,GAAG;YACT,KAAK,EAAE,GAAG;YACV,QAAQ,EAAE,iBAAiB,CAAC,IAAI,CAAC;YACjC,iBAAiB,EAAE,IAAI;SACvB,EACD,EAAE,QAAQ,EAAE,GAAG,EAAE,EACjB,EAAE,QAAQ,EAAE,EAAE,EAAE,SAAS,EAAE,CAAC,EAAE,CAC9B,CAAA;QACD,MAAM,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAA;IACjC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,gEAAgE,EAAE,KAAK,IAAI,EAAE;QAC/E,MAAM,IAAI,GAA2B,EAAE,CAAA;QACvC,MAAM,IAAI,GAAG,IAAI,gBAAgB,CAAC,SAAS,EAAE,EAAE,gBAAgB,EAAE,GAAG,EAAE,CAAC,CAAA;QACvE,MAAM,IAAI,CAAC,OAAO,CACjB,EAAE,EAAE,EAAE,GAAG,EAAE,IAAI,EAAE,GAAG,EAAE,KAAK,EAAE,GAAG,EAAE,QAAQ,EAAE,iBAAiB,CAAC,IAAI,CAAC,EAAE,EACrE,EAAE,QAAQ,EAAE,GAAG,EAAE,EACjB,EAAE,QAAQ,EAAE,EAAE,EAAE,SAAS,EAAE,CAAC,EAAE,CAC9B,CAAA;QACD,MAAM,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAA;IACjC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,wCAAwC,EAAE,KAAK,IAAI,EAAE;QACvD,MAAM,IAAI,GAA2B,EAAE,CAAA;QACvC,MAAM,IAAI,GAAG,IAAI,gBAAgB,CAAC,SAAS,EAAE,EAAE,gBAAgB,EAAE,GAAG,EAAE,CAAC,CAAA;QACvE,MAAM,IAAI,CAAC,OAAO,CACjB,EAAE,EAAE,EAAE,GAAG,EAAE,IAAI,EAAE,GAAG,EAAE,KAAK,EAAE,GAAG,EAAE,QAAQ,EAAE,iBAAiB,CAAC,IAAI,CAAC,EAAE,iBAAiB,EAAE,EAAE,EAAE,EAC5F,EAAE,QAAQ,EAAE,GAAG,EAAE,EACjB,EAAE,QAAQ,EAAE,EAAE,EAAE,SAAS,EAAE,CAAC,EAAE,CAC9B,CAAA;QACD,MAAM,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IAChC,CAAC,CAAC,CAAA;AACH,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,+CAA+C,EAAE,GAAG,EAAE;IAC9D,EAAE,CAAC,8DAA8D,EAAE,KAAK,IAAI,EAAE;QAC7E,MAAM,QAAQ,GAAG;YAChB,UAAU,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,SAAS,CAAC;gBAChC,MAAM,EAAE,EAAE,EAAE,GAAG,EAAE,KAAK,EAAE,EAAE,OAAO,EAAE,IAAI,EAAE,EAAiB,CAAA;gBAC1D,MAAM;oBACL,EAAE,EAAE,GAAG;oBACP,KAAK,EAAE,EAAE;oBACT,YAAY,EAAE,MAAM;oBACpB,KAAK,EAAE;wBACN,YAAY,EAAE,SAAS;wBACvB,gBAAgB,EAAE,SAAS;wBAC3B,WAAW,EAAE,SAAS;wBACtB,YAAY,EAAE,CAAC;wBACf,gBAAgB,EAAE,CAAC;qBACnB;iBACc,CAAA;YACjB,CAAC,CAAC;SAC0C,CAAA;QAE7C,MAAM,IAAI,GAAG,IAAI,gBAAgB,EAAE,CAAA;QACnC,MAAM,GAAG,GAAG,MAAM,IAAI,CAAC,OAAO,CAC7B;YACC,EAAE,EAAE,GAAG;YACP,IAAI,EAAE,GAAG;YACT,KAAK,EAAE,GAAG;YACV,QAAQ;YACR,OAAO,EAAE,EAAE,cAAc,EAAE,CAAC,EAAE,eAAe,EAAE,EAAE,EAAE;SACnD,EACD,EAAE,QAAQ,EAAE,GAAG,EAAE,EACjB,EAAE,QAAQ,EAAE,EAAE,EAAE,SAAS,EAAE,CAAC,EAAE,CAC9B,CAAA;QACD,MAAM,CAAC,GAAG,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;IACpC,CAAC,CAAC,CAAA;AACH,CAAC,CAAC,CAAA"}

package/dist/advisory/__tests__/structured-response.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=structured-response.test.d.ts.map

package/dist/advisory/__tests__/structured-response.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"structured-response.test.d.ts","sourceRoot":"","sources":["../../../src/advisory/__tests__/structured-response.test.ts"],"names":[],"mappings":""}

package/dist/advisory/__tests__/structured-response.test.js

@@ -0,0 +1,99 @@
+import { describe, expect, it, vi } from 'vitest';
+import { AdvisoryExecutor } from '../executor.js';
+import { ADVISORY_RESPONSE_CONTRACT, parseAdvisoryResponse } from '../parse.js';
+/**
+ * `AdvisoryResult.warnings` and `.decisions` had two consumers each — the
+ * advisory phase folds decisions into working state and renders warnings
+ * back to the executing agent, and the advisory tool renders both — and no
+ * producer at all. The parser returned `{ advice }` and a comment saying
+ * structure came later. Every consumer branch was dead in a real run, and
+ * their tests passed because they built the result by hand.
+ */
+describe('an advisor answer is read back into its parts', () => {
+    it('lifts warnings and decisions out of the prose', () => {
+        const parsed = parseAdvisoryResponse([
+            'Proceed, but slowly.',
+            '<warnings>',
+            '- the store is not transactional',
+            '- retries are not idempotent',
+            '</warnings>',
+            '<decisions>',
+            '- keep the existing schema',
+            '</decisions>',
+        ].join('\n'));
+        expect(parsed.warnings).toEqual([
+            'the store is not transactional',
+            'retries are not idempotent',
+        ]);
+        expect(parsed.decisions).toEqual(['keep the existing schema']);
+    });
+    it('leaves the blocks out of the advice, so nothing is said twice', () => {
+        const parsed = parseAdvisoryResponse('Do it.\n<warnings>\n- careful\n</warnings>');
+        expect(parsed.advice).toBe('Do it.');
+        expect(parsed.advice).not.toContain('careful');
+    });
+    it('accepts whichever bullet the model reached for', () => {
+        const parsed = parseAdvisoryResponse('<decisions>\n1. first\n* second\n• third\n</decisions>');
+        expect(parsed.decisions).toEqual(['first', 'second', 'third']);
+    });
+    it('omits the fields entirely rather than reporting empty ones', () => {
+        const parsed = parseAdvisoryResponse('Just advice.\n<warnings>\n\n</warnings>');
+        expect(parsed.advice).toBe('Just advice.');
+        expect(parsed.warnings).toBeUndefined();
+        expect(parsed.decisions).toBeUndefined();
+    });
+    it('treats an unclosed block as prose instead of guessing where it ends', () => {
+        const parsed = parseAdvisoryResponse('Careful.\n<warnings>\n- half a thought');
+        expect(parsed.warnings).toBeUndefined();
+        expect(parsed.advice).toContain('half a thought');
+    });
+    it('reads a plain answer as pure advice', () => {
+        expect(parseAdvisoryResponse('  Ship it.  ')).toEqual({ advice: 'Ship it.' });
+    });
+});
+describe('every advisor is told the convention its answer is read with', () => {
+    function capturingProvider(seen) {
+        return {
+            chatStream: vi.fn(async function* (params) {
+                seen.system = params.messages.find((m) => m.role === 'system')?.content ?? undefined;
+                yield { id: 'r', delta: { content: 'ok' } };
+                yield {
+                    id: 'r',
+                    delta: {},
+                    finishReason: 'stop',
+                    usage: {
+                        promptTokens: 0,
+                        completionTokens: 0,
+                        totalTokens: 0,
+                        cachedTokens: 0,
+                        cacheWriteTokens: 0,
+                    },
+                };
+            }),
+        };
+    }
+    async function systemPromptFor(advisor) {
+        const seen = {};
+        await new AdvisoryExecutor().consult({ id: 'a', name: 'A', model: 'm', provider: capturingProvider(seen), ...advisor }, { question: 'q' }, { messages: [], iteration: 1 });
+        return seen.system ?? '';
+    }
+    it('appends the contract to the default prompt', async () => {
+        expect(await systemPromptFor({})).toContain(ADVISORY_RESPONSE_CONTRACT);
+    });
+    // The failure that hides: a host writes its own prompt, the advisor is
+    // never told the convention, and its warnings are silently unparseable.
+    it('appends it to a host-written prompt too, without discarding that prompt', async () => {
+        const prompt = await systemPromptFor({ systemPrompt: 'You are terse.' });
+        expect(prompt).toContain('You are terse.');
+        expect(prompt).toContain(ADVISORY_RESPONSE_CONTRACT);
+    });
+    it('appends it to a persona-assembled prompt too', async () => {
+        const prompt = await systemPromptFor({
+            persona: {
+                identity: { role: 'reviewer', description: 'Reviews plans.' },
+            },
+        });
+        expect(prompt).toContain(ADVISORY_RESPONSE_CONTRACT);
+    });
+});
+//# sourceMappingURL=structured-response.test.js.map

package/dist/advisory/__tests__/structured-response.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"structured-response.test.js","sourceRoot":"","sources":["../../../src/advisory/__tests__/structured-response.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAA;AAIjD,OAAO,EAAE,gBAAgB,EAAE,MAAM,gBAAgB,CAAA;AACjD,OAAO,EAAE,0BAA0B,EAAE,qBAAqB,EAAE,MAAM,aAAa,CAAA;AAE/E;;;;;;;GAOG;AAEH,QAAQ,CAAC,+CAA+C,EAAE,GAAG,EAAE;IAC9D,EAAE,CAAC,+CAA+C,EAAE,GAAG,EAAE;QACxD,MAAM,MAAM,GAAG,qBAAqB,CACnC;YACC,sBAAsB;YACtB,YAAY;YACZ,kCAAkC;YAClC,8BAA8B;YAC9B,aAAa;YACb,aAAa;YACb,4BAA4B;YAC5B,cAAc;SACd,CAAC,IAAI,CAAC,IAAI,CAAC,CACZ,CAAA;QAED,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,OAAO,CAAC;YAC/B,gCAAgC;YAChC,4BAA4B;SAC5B,CAAC,CAAA;QACF,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,OAAO,CAAC,CAAC,0BAA0B,CAAC,CAAC,CAAA;IAC/D,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,+DAA+D,EAAE,GAAG,EAAE;QACxE,MAAM,MAAM,GAAG,qBAAqB,CAAC,4CAA4C,CAAC,CAAA;QAElF,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;QACpC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,SAAS,CAAC,CAAA;IAC/C,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,gDAAgD,EAAE,GAAG,EAAE;QACzD,MAAM,MAAM,GAAG,qBAAqB,CAAC,wDAAwD,CAAC,CAAA;QAE9F,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,OAAO,CAAC,CAAC,OAAO,EAAE,QAAQ,EAAE,OAAO,CAAC,CAAC,CAAA;IAC/D,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,4DAA4D,EAAE,GAAG,EAAE;QACrE,MAAM,MAAM,GAAG,qBAAqB,CAAC,yCAAyC,CAAC,CAAA;QAE/E,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,IAAI,CAAC,cAAc,CAAC,CAAA;QAC1C,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,aAAa,EAAE,CAAA;QACvC,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,aAAa,EAAE,CAAA;IACzC,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,qEAAqE,EAAE,GAAG,EAAE;QAC9E,MAAM,MAAM,GAAG,qBAAqB,CAAC,wCAAwC,CAAC,CAAA;QAE9E,MAAM,CAAC,MAAM,CAAC,QAAQ,CAAC,CAAC,aAAa,EAAE,CAAA;QACvC,MAAM,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,gBAAgB,CAAC,CAAA;IAClD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,qCAAqC,EAAE,GAAG,EAAE;QAC9C,MAAM,CAAC,qBAAqB,CAAC,cAAc,CAAC,CAAC,CAAC,OAAO,CAAC,EAAE,MAAM,EAAE,UAAU,EAAE,CAAC,CAAA;IAC9E,CAAC,CAAC,CAAA;AACH,CAAC,CAAC,CAAA;AAEF,QAAQ,CAAC,8DAA8D,EAAE,GAAG,EAAE;IAC7E,SAAS,iBAAiB,CAAC,IAAyB;QACnD,OAAO;YACN,UAAU,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,SAAS,CAAC,EAAE,MAElC;gBACA,IAAI,CAAC,MAAM,GAAG,MAAM,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,QAAQ,CAAC,EAAE,OAAO,IAAI,SAAS,CAAA;gBACpF,MAAM,EAAE,EAAE,EAAE,GAAG,EAAE,KAAK,EAAE,EAAE,OAAO,EAAE,IAAI,EAAE,EAAiB,CAAA;gBAC1D,MAAM;oBACL,EAAE,EAAE,GAAG;oBACP,KAAK,EAAE,EAAE;oBACT,YAAY,EAAE,MAAM;oBACpB,KAAK,EAAE;wBACN,YAAY,EAAE,CAAC;wBACf,gBAAgB,EAAE,CAAC;wBACnB,WAAW,EAAE,CAAC;wBACd,YAAY,EAAE,CAAC;wBACf,gBAAgB,EAAE,CAAC;qBACnB;iBACc,CAAA;YACjB,CAAC,CAAC;SAC0C,CAAA;IAC9C,CAAC;IAED,KAAK,UAAU,eAAe,CAAC,OAAmC;QACjE,MAAM,IAAI,GAAwB,EAAE,CAAA;QACpC,MAAM,IAAI,gBAAgB,EAAE,CAAC,OAAO,CACnC,EAAE,EAAE,EAAE,GAAG,EAAE,IAAI,EAAE,GAAG,EAAE,KAAK,EAAE,GAAG,EAAE,QAAQ,EAAE,iBAAiB,CAAC,IAAI,CAAC,EAAE,GAAG,OAAO,EAAE,EACjF,EAAE,QAAQ,EAAE,GAAG,EAAE,EACjB,EAAE,QAAQ,EAAE,EAAE,EAAE,SAAS,EAAE,CAAC,EAAE,CAC9B,CAAA;QACD,OAAO,IAAI,CAAC,MAAM,IAAI,EAAE,CAAA;IACzB,CAAC;IAED,EAAE,CAAC,4CAA4C,EAAE,KAAK,IAAI,EAAE;QAC3D,MAAM,CAAC,MAAM,eAAe,CAAC,EAAE,CAAC,CAAC,CAAC,SAAS,CAAC,0BAA0B,CAAC,CAAA;IACxE,CAAC,CAAC,CAAA;IAEF,uEAAuE;IACvE,wEAAwE;IACxE,EAAE,CAAC,yEAAyE,EAAE,KAAK,IAAI,EAAE;QACxF,MAAM,MAAM,GAAG,MAAM,eAAe,CAAC,EAAE,YAAY,EAAE,gBAAgB,EAAE,CAAC,CAAA;QAExE,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,gBAAgB,CAAC,CAAA;QAC1C,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,0BAA0B,CAAC,CAAA;IACrD,CAAC,CAAC,CAAA;IAEF,EAAE,CAAC,8CAA8C,EAAE,KAAK,IAAI,EAAE;QAC7D,MAAM,MAAM,GAAG,MAAM,eAAe,CAAC;YACpC,OAAO,EAAE;gBACR,QAAQ,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE,WAAW,EAAE,gBAAgB,EAAE;aAC7B;SACjC,CAAC,CAAA;QAEF,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,0BAA0B,CAAC,CAAA;IACrD,CAAC,CAAC,CAAA;AACH,CAAC,CAAC,CAAA"}

package/dist/advisory/budget.d.ts

@@ -0,0 +1,15 @@
+import type { AdvisorDefinition, AdvisoryBudget } from '../types/advisory/index.js';
+/**
+ * Refuse a budget the runtime cannot honour.
+ *
+ * A cost cap is enforced against real spend, and real spend needs a price.
+ * Without one every call costs zero, so the cap never trips — the host sets
+ * a limit, sees no error, and learns it was decoration only from the bill.
+ * Refusing at construction is the same trade the runtime makes elsewhere:
+ * a configuration that cannot do what it says is an error, not a default.
+ */
+export declare function assertBudgetEnforceable(config: {
+    readonly advisors: readonly AdvisorDefinition[];
+    readonly budget?: AdvisoryBudget | undefined;
+}): void;
+//# sourceMappingURL=budget.d.ts.map

package/dist/advisory/budget.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"budget.d.ts","sourceRoot":"","sources":["../../src/advisory/budget.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,iBAAiB,EAAE,cAAc,EAAE,MAAM,4BAA4B,CAAA;AAEnF;;;;;;;;GAQG;AACH,wBAAgB,uBAAuB,CAAC,MAAM,EAAE;IAC/C,QAAQ,CAAC,QAAQ,EAAE,SAAS,iBAAiB,EAAE,CAAA;IAC/C,QAAQ,CAAC,MAAM,CAAC,EAAE,cAAc,GAAG,SAAS,CAAA;CAC5C,GAAG,IAAI,CAYP"}

package/dist/advisory/budget.js

@@ -0,0 +1,21 @@
+/**
+ * Refuse a budget the runtime cannot honour.
+ *
+ * A cost cap is enforced against real spend, and real spend needs a price.
+ * Without one every call costs zero, so the cap never trips — the host sets
+ * a limit, sees no error, and learns it was decoration only from the bill.
+ * Refusing at construction is the same trade the runtime makes elsewhere:
+ * a configuration that cannot do what it says is an error, not a default.
+ */
+export function assertBudgetEnforceable(config) {
+    if (config.budget?.maxCostPerRun === undefined)
+        return;
+    const unpriced = config.advisors.filter((a) => a.pricing === undefined).map((a) => a.id);
+    if (unpriced.length === 0)
+        return;
+    throw new Error(`Advisory budget sets maxCostPerRun but ${unpriced.length === 1 ? 'advisor' : 'advisors'} ` +
+        `${unpriced.join(', ')} carr${unpriced.length === 1 ? 'ies' : 'y'} no pricing, so cost ` +
+        'cannot be measured and the cap could never be reached. Give each advisor a `pricing` ' +
+        'entry, or drop the cost cap and bound the run with `maxCallsPerRun`.');
+}
+//# sourceMappingURL=budget.js.map

package/dist/advisory/budget.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"budget.js","sourceRoot":"","sources":["../../src/advisory/budget.ts"],"names":[],"mappings":"AAEA;;;;;;;;GAQG;AACH,MAAM,UAAU,uBAAuB,CAAC,MAGvC;IACA,IAAI,MAAM,CAAC,MAAM,EAAE,aAAa,KAAK,SAAS;QAAE,OAAM;IAEtD,MAAM,QAAQ,GAAG,MAAM,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,OAAO,KAAK,SAAS,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC,CAAA;IACxF,IAAI,QAAQ,CAAC,MAAM,KAAK,CAAC;QAAE,OAAM;IAEjC,MAAM,IAAI,KAAK,CACd,0CAA0C,QAAQ,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,UAAU,GAAG;QAC1F,GAAG,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,QAAQ,QAAQ,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,GAAG,uBAAuB;QACxF,uFAAuF;QACvF,sEAAsE,CACvE,CAAA;AACF,CAAC"}

package/dist/advisory/context.d.ts

@@ -15,6 +15,8 @@ export declare class AdvisoryContext {
         total: number | undefined;
         used: number;
     };
+    /** Advisory spend so far this run, summed from the recorded calls. */
+    spentCost(): number;
     checkBudget(): {
         allowed: boolean;
         reason?: string;

package/dist/advisory/context.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"context.d.ts","sourceRoot":"","sources":["../../src/advisory/context.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,kBAAkB,EAAE,MAAM,4BAA4B,CAAA;AACpF,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,gBAAgB,CAAA;AACtD,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,eAAe,CAAA;AACrD,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,eAAe,CAAA;AAEpD,qBAAa,eAAe;IAC3B,QAAQ,CAAC,QAAQ,EAAE,eAAe,CAAA;IAClC,QAAQ,CAAC,QAAQ,EAAE,gBAAgB,CAAA;IACnC,QAAQ,CAAC,SAAS,EAAE,gBAAgB,CAAA;IACpC,QAAQ,CAAC,WAAW,EAAE,kBAAkB,EAAE,CAAK;IAE/C,OAAO,CAAC,QAAQ,CAAC,MAAM,CAA4B;gBAGlD,QAAQ,EAAE,eAAe,EACzB,QAAQ,EAAE,gBAAgB,EAC1B,SAAS,EAAE,gBAAgB,EAC3B,MAAM,CAAC,EAAE,cAAc;IAQxB,UAAU,CAAC,MAAM,EAAE,kBAAkB,GAAG,IAAI;IAI5C,eAAe,IAAI;QAAE,SAAS,EAAE,MAAM,GAAG,SAAS,CAAC;QAAC,KAAK,EAAE,MAAM,GAAG,SAAS,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE;IAO7F,WAAW,IAAI;QAAE,OAAO,EAAE,OAAO,CAAC;QAAC,MAAM,CAAC,EAAE,MAAM,CAAA;KAAE;CAUpD"}
+{"version":3,"file":"context.d.ts","sourceRoot":"","sources":["../../src/advisory/context.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAAE,kBAAkB,EAAE,MAAM,4BAA4B,CAAA;AACpF,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,gBAAgB,CAAA;AACtD,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,eAAe,CAAA;AACrD,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,eAAe,CAAA;AAEpD,qBAAa,eAAe;IAC3B,QAAQ,CAAC,QAAQ,EAAE,eAAe,CAAA;IAClC,QAAQ,CAAC,QAAQ,EAAE,gBAAgB,CAAA;IACnC,QAAQ,CAAC,SAAS,EAAE,gBAAgB,CAAA;IACpC,QAAQ,CAAC,WAAW,EAAE,kBAAkB,EAAE,CAAK;IAE/C,OAAO,CAAC,QAAQ,CAAC,MAAM,CAA4B;gBAGlD,QAAQ,EAAE,eAAe,EACzB,QAAQ,EAAE,gBAAgB,EAC1B,SAAS,EAAE,gBAAgB,EAC3B,MAAM,CAAC,EAAE,cAAc;IAQxB,UAAU,CAAC,MAAM,EAAE,kBAAkB,GAAG,IAAI;IAI5C,eAAe,IAAI;QAAE,SAAS,EAAE,MAAM,GAAG,SAAS,CAAC;QAAC,KAAK,EAAE,MAAM,GAAG,SAAS,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE;IAO7F,sEAAsE;IACtE,SAAS,IAAI,MAAM;IAInB,WAAW,IAAI;QAAE,OAAO,EAAE,OAAO,CAAC;QAAC,MAAM,CAAC,EAAE,MAAM,CAAA;KAAE;CAwBpD"}

package/dist/advisory/context.js

@@ -19,6 +19,10 @@ export class AdvisoryContext {
         const remaining = total !== undefined ? total - used : undefined;
         return { remaining, total, used };
     }
+    /** Advisory spend so far this run, summed from the recorded calls. */
+    spentCost() {
+        return this.callHistory.reduce((sum, call) => sum + call.cost.totalCost, 0);
+    }
     checkBudget() {
         const { remaining, total } = this.getBudgetStatus();
         if (remaining !== undefined && remaining <= 0) {
@@ -27,6 +31,18 @@ export class AdvisoryContext {
                 reason: `Advisory budget exhausted: ${total} calls used of ${total} allowed per run`,
             };
         }
+        // Checked before the call, not after: a cap that only reports
+        // overspend once it has happened is a log line, not a budget.
+        const costCap = this.budget?.maxCostPerRun;
+        if (costCap !== undefined) {
+            const spent = this.spentCost();
+            if (spent >= costCap) {
+                return {
+                    allowed: false,
+                    reason: `Advisory budget exhausted: cost ${spent} of ${costCap} allowed per run`,
+                };
+            }
+        }
         return { allowed: true };
     }
 }

package/dist/advisory/context.js.map

@@ -1 +1 @@
-{"version":3,"file":"context.js","sourceRoot":"","sources":["../../src/advisory/context.ts"],"names":[],"mappings":"AAKA,MAAM,OAAO,eAAe;IAClB,QAAQ,CAAiB;IACzB,QAAQ,CAAkB;IAC1B,SAAS,CAAkB;IAC3B,WAAW,GAAyB,EAAE,CAAA;IAE9B,MAAM,CAA4B;IAEnD,YACC,QAAyB,EACzB,QAA0B,EAC1B,SAA2B,EAC3B,MAAuB;QAEvB,IAAI,CAAC,QAAQ,GAAG,QAAQ,CAAA;QACxB,IAAI,CAAC,QAAQ,GAAG,QAAQ,CAAA;QACxB,IAAI,CAAC,SAAS,GAAG,SAAS,CAAA;QAC1B,IAAI,CAAC,MAAM,GAAG,MAAM,CAAA;IACrB,CAAC;IAED,UAAU,CAAC,MAA0B;QACpC,IAAI,CAAC,WAAW,CAAC,IAAI,CAAC,MAAM,CAAC,CAAA;IAC9B,CAAC;IAED,eAAe;QACd,MAAM,IAAI,GAAG,IAAI,CAAC,WAAW,CAAC,MAAM,CAAA;QACpC,MAAM,KAAK,GAAG,IAAI,CAAC,MAAM,EAAE,cAAc,CAAA;QACzC,MAAM,SAAS,GAAG,KAAK,KAAK,SAAS,CAAC,CAAC,CAAC,KAAK,GAAG,IAAI,CAAC,CAAC,CAAC,SAAS,CAAA;QAChE,OAAO,EAAE,SAAS,EAAE,KAAK,EAAE,IAAI,EAAE,CAAA;IAClC,CAAC;IAED,WAAW;QACV,MAAM,EAAE,SAAS,EAAE,KAAK,EAAE,GAAG,IAAI,CAAC,eAAe,EAAE,CAAA;QACnD,IAAI,SAAS,KAAK,SAAS,IAAI,SAAS,IAAI,CAAC,EAAE,CAAC;YAC/C,OAAO;gBACN,OAAO,EAAE,KAAK;gBACd,MAAM,EAAE,8BAA8B,KAAK,kBAAkB,KAAK,kBAAkB;aACpF,CAAA;QACF,CAAC;QACD,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,CAAA;IACzB,CAAC;CACD"}
+{"version":3,"file":"context.js","sourceRoot":"","sources":["../../src/advisory/context.ts"],"names":[],"mappings":"AAKA,MAAM,OAAO,eAAe;IAClB,QAAQ,CAAiB;IACzB,QAAQ,CAAkB;IAC1B,SAAS,CAAkB;IAC3B,WAAW,GAAyB,EAAE,CAAA;IAE9B,MAAM,CAA4B;IAEnD,YACC,QAAyB,EACzB,QAA0B,EAC1B,SAA2B,EAC3B,MAAuB;QAEvB,IAAI,CAAC,QAAQ,GAAG,QAAQ,CAAA;QACxB,IAAI,CAAC,QAAQ,GAAG,QAAQ,CAAA;QACxB,IAAI,CAAC,SAAS,GAAG,SAAS,CAAA;QAC1B,IAAI,CAAC,MAAM,GAAG,MAAM,CAAA;IACrB,CAAC;IAED,UAAU,CAAC,MAA0B;QACpC,IAAI,CAAC,WAAW,CAAC,IAAI,CAAC,MAAM,CAAC,CAAA;IAC9B,CAAC;IAED,eAAe;QACd,MAAM,IAAI,GAAG,IAAI,CAAC,WAAW,CAAC,MAAM,CAAA;QACpC,MAAM,KAAK,GAAG,IAAI,CAAC,MAAM,EAAE,cAAc,CAAA;QACzC,MAAM,SAAS,GAAG,KAAK,KAAK,SAAS,CAAC,CAAC,CAAC,KAAK,GAAG,IAAI,CAAC,CAAC,CAAC,SAAS,CAAA;QAChE,OAAO,EAAE,SAAS,EAAE,KAAK,EAAE,IAAI,EAAE,CAAA;IAClC,CAAC;IAED,sEAAsE;IACtE,SAAS;QACR,OAAO,IAAI,CAAC,WAAW,CAAC,MAAM,CAAC,CAAC,GAAG,EAAE,IAAI,EAAE,EAAE,CAAC,GAAG,GAAG,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE,CAAC,CAAC,CAAA;IAC5E,CAAC;IAED,WAAW;QACV,MAAM,EAAE,SAAS,EAAE,KAAK,EAAE,GAAG,IAAI,CAAC,eAAe,EAAE,CAAA;QACnD,IAAI,SAAS,KAAK,SAAS,IAAI,SAAS,IAAI,CAAC,EAAE,CAAC;YAC/C,OAAO;gBACN,OAAO,EAAE,KAAK;gBACd,MAAM,EAAE,8BAA8B,KAAK,kBAAkB,KAAK,kBAAkB;aACpF,CAAA;QACF,CAAC;QAED,8DAA8D;QAC9D,8DAA8D;QAC9D,MAAM,OAAO,GAAG,IAAI,CAAC,MAAM,EAAE,aAAa,CAAA;QAC1C,IAAI,OAAO,KAAK,SAAS,EAAE,CAAC;YAC3B,MAAM,KAAK,GAAG,IAAI,CAAC,SAAS,EAAE,CAAA;YAC9B,IAAI,KAAK,IAAI,OAAO,EAAE,CAAC;gBACtB,OAAO;oBACN,OAAO,EAAE,KAAK;oBACd,MAAM,EAAE,mCAAmC,KAAK,OAAO,OAAO,kBAAkB;iBAChF,CAAA;YACF,CAAC;QACF,CAAC;QAED,OAAO,EAAE,OAAO,EAAE,IAAI,EAAE,CAAA;IACzB,CAAC;CACD"}

package/dist/advisory/executor.d.ts

@@ -1,4 +1,4 @@
-import type { AdvisorDefinition } from '../types/advisory/config.js';
+import type { AdvisorDefinition, AdvisoryBudget } from '../types/advisory/config.js';
 import type { AdvisoryRequest, AdvisoryResult } from '../types/advisory/result.js';
 import type { CostInfo, TokenUsage } from '../types/common/index.js';
 import { type Message } from '../types/message/index.js';
@@ -18,22 +18,25 @@ export interface AdvisoryExecutionResult {
 }
 export declare class AdvisoryExecutor {
     private readonly logger;
-    constructor(logger?: Logger);
+    private readonly budget;
+    constructor(logger?: Logger, budget?: AdvisoryBudget);
+    /**
+     * The response ceiling for one call: the tighter of what the advisor
+     * asks for and what the budget allows. An advisor that names no ceiling
+     * still gets the budget's, which is the whole point of a per-call cap.
+     */
+    private responseTokenCeiling;
     consult(advisor: AdvisorDefinition, request: AdvisoryRequest, callCtx: AdvisoryCallContext): Promise<AdvisoryExecutionResult>;
     private buildSystemPrompt;
+    private describeAdvisor;
     private buildContext;
     private truncateMessages;
     /**
-     * Parses the raw LLM response into an AdvisoryResult.
-     *
-     * Phase 1: text-only extraction. Structured parsing comes later.
-     */
-    private parseResult;
-    /**
-     * Computes cost from token usage.
+     * Cost for one call, from the advisor's own pricing.
      *
-     * Returns zero-value cost since pricing is provider-specific.
-     * Callers with pricing data can recompute via calculateCost().
+     * Zero when the advisor carries none — which is honest, not a
+     * placeholder: a cost CAP over unpriced advisors is refused at
+     * construction, so nothing enforces against this zero.
      */
     private computeCost;
 }

package/dist/advisory/executor.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"executor.d.ts","sourceRoot":"","sources":["../../src/advisory/executor.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,6BAA6B,CAAA;AACpE,OAAO,KAAK,EAAE,eAAe,EAAE,cAAc,EAAE,MAAM,6BAA6B,CAAA;AAClF,OAAO,KAAK,EAAE,QAAQ,EAAE,UAAU,EAAE,MAAM,0BAA0B,CAAA;AACpE,OAAO,EAAE,KAAK,OAAO,EAA0C,MAAM,2BAA2B,CAAA;AAChG,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,wBAAwB,CAAA;AAC3D,OAAO,EAAE,KAAK,MAAM,EAAiB,MAAM,oBAAoB,CAAA;AAE/D,MAAM,WAAW,mBAAmB;IACnC,QAAQ,CAAC,QAAQ,EAAE,OAAO,EAAE,CAAA;IAC5B,QAAQ,CAAC,mBAAmB,CAAC,EAAE,MAAM,CAAA;IACrC,QAAQ,CAAC,WAAW,CAAC,EAAE,aAAa,EAAE,CAAA;IACtC,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAA;CAC1B;AAED,MAAM,WAAW,uBAAuB;IACvC,QAAQ,CAAC,MAAM,EAAE,cAAc,CAAA;IAC/B,QAAQ,CAAC,KAAK,EAAE,UAAU,CAAA;IAC1B,QAAQ,CAAC,IAAI,EAAE,QAAQ,CAAA;IACvB,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAA;CAC3B;AAED,qBAAa,gBAAgB;IAC5B,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAQ;gBAEnB,MAAM,CAAC,EAAE,MAAM;IAIrB,OAAO,CACZ,OAAO,EAAE,iBAAiB,EAC1B,OAAO,EAAE,eAAe,EACxB,OAAO,EAAE,mBAAmB,GAC1B,OAAO,CAAC,uBAAuB,CAAC;IAkDnC,OAAO,CAAC,iBAAiB;IAoBzB,OAAO,CAAC,YAAY;IA6CpB,OAAO,CAAC,gBAAgB;IAqBxB;;;;OAIG;IACH,OAAO,CAAC,WAAW;IAMnB;;;;;OAKG;IACH,OAAO,CAAC,WAAW;CAQnB"}
+{"version":3,"file":"executor.d.ts","sourceRoot":"","sources":["../../src/advisory/executor.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,iBAAiB,EAAE,cAAc,EAAE,MAAM,6BAA6B,CAAA;AACpF,OAAO,KAAK,EAAE,eAAe,EAAE,cAAc,EAAE,MAAM,6BAA6B,CAAA;AAClF,OAAO,KAAK,EAAE,QAAQ,EAAE,UAAU,EAAE,MAAM,0BAA0B,CAAA;AACpE,OAAO,EAAE,KAAK,OAAO,EAA0C,MAAM,2BAA2B,CAAA;AAChG,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,wBAAwB,CAAA;AAE3D,OAAO,EAAE,KAAK,MAAM,EAAiB,MAAM,oBAAoB,CAAA;AAG/D,MAAM,WAAW,mBAAmB;IACnC,QAAQ,CAAC,QAAQ,EAAE,OAAO,EAAE,CAAA;IAC5B,QAAQ,CAAC,mBAAmB,CAAC,EAAE,MAAM,CAAA;IACrC,QAAQ,CAAC,WAAW,CAAC,EAAE,aAAa,EAAE,CAAA;IACtC,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAA;CAC1B;AAED,MAAM,WAAW,uBAAuB;IACvC,QAAQ,CAAC,MAAM,EAAE,cAAc,CAAA;IAC/B,QAAQ,CAAC,KAAK,EAAE,UAAU,CAAA;IAC1B,QAAQ,CAAC,IAAI,EAAE,QAAQ,CAAA;IACvB,QAAQ,CAAC,UAAU,EAAE,MAAM,CAAA;CAC3B;AAED,qBAAa,gBAAgB;IAC5B,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAQ;IAC/B,OAAO,CAAC,QAAQ,CAAC,MAAM,CAA4B;gBAEvC,MAAM,CAAC,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,cAAc;IAKpD;;;;OAIG;IACH,OAAO,CAAC,oBAAoB;IAOtB,OAAO,CACZ,OAAO,EAAE,iBAAiB,EAC1B,OAAO,EAAE,eAAe,EACxB,OAAO,EAAE,mBAAmB,GAC1B,OAAO,CAAC,uBAAuB,CAAC;IAkDnC,OAAO,CAAC,iBAAiB;IAOzB,OAAO,CAAC,eAAe;IAoBvB,OAAO,CAAC,YAAY;IA6CpB,OAAO,CAAC,gBAAgB;IAqBxB;;;;;;OAMG;IACH,OAAO,CAAC,WAAW;CAMnB"}

package/dist/advisory/executor.js

@@ -2,11 +2,28 @@ import { CHARS_PER_TOKEN } from '../constants/limits.js';
 import { assembleSystemPrompt } from '../persona/assembler.js';
 import { collect } from '../provider/collect.js';
 import { createSystemMessage, createUserMessage } from '../types/message/index.js';
+import { calculateCost } from '../utils/cost.js';
 import { getRootLogger } from '../utils/logger.js';
+import { ADVISORY_RESPONSE_CONTRACT, parseAdvisoryResponse } from './parse.js';
 export class AdvisoryExecutor {
     logger;
-    constructor(logger) {
+    budget;
+    constructor(logger, budget) {
         this.logger = (logger ?? getRootLogger()).child({ component: 'AdvisoryExecutor' });
+        this.budget = budget;
+    }
+    /**
+     * The response ceiling for one call: the tighter of what the advisor
+     * asks for and what the budget allows. An advisor that names no ceiling
+     * still gets the budget's, which is the whole point of a per-call cap.
+     */
+    responseTokenCeiling(advisor) {
+        const cap = this.budget?.maxTokensPerCall;
+        if (cap === undefined)
+            return advisor.maxResponseTokens;
+        if (advisor.maxResponseTokens === undefined)
+            return cap;
+        return Math.min(advisor.maxResponseTokens, cap);
     }
     async consult(advisor, request, callCtx) {
         const startMs = Date.now();
@@ -27,12 +44,12 @@ export class AdvisoryExecutor {
             model: advisor.model,
             messages,
             temperature: advisor.temperature,
-            maxTokens: advisor.maxResponseTokens,
+            maxTokens: this.responseTokenCeiling(advisor),
             toolChoice: 'none',
         }));
         const durationMs = Date.now() - startMs;
-        const result = this.parseResult(response.message.content ?? '');
-        const cost = this.computeCost(response.usage);
+        const result = parseAdvisoryResponse(response.message.content ?? '');
+        const cost = this.computeCost(advisor, response.usage);
         this.logger.info('advisory call completed', {
             advisorId: advisor.id,
             model: advisor.model,
@@ -47,6 +64,12 @@ export class AdvisoryExecutor {
         };
     }
     buildSystemPrompt(advisor) {
+        // The contract is appended to every branch, not folded into the
+        // default: an advisor with its own prompt or a persona is still read
+        // back by the same parser, and used to be the one never told so.
+        return `${this.describeAdvisor(advisor)}\n\n${ADVISORY_RESPONSE_CONTRACT}`;
+    }
+    describeAdvisor(advisor) {
         if (advisor.systemPrompt) {
             return advisor.systemPrompt;
         }
@@ -113,28 +136,17 @@ export class AdvisoryExecutor {
         return result;
     }
     /**
-     * Parses the raw LLM response into an AdvisoryResult.
+     * Cost for one call, from the advisor's own pricing.
      *
-     * Phase 1: text-only extraction. Structured parsing comes later.
+     * Zero when the advisor carries none — which is honest, not a
+     * placeholder: a cost CAP over unpriced advisors is refused at
+     * construction, so nothing enforces against this zero.
      */
-    parseResult(rawContent) {
-        return {
-            advice: rawContent,
-        };
-    }
-    /**
-     * Computes cost from token usage.
-     *
-     * Returns zero-value cost since pricing is provider-specific.
-     * Callers with pricing data can recompute via calculateCost().
-     */
-    computeCost(_usage) {
-        return {
-            inputCostPer1M: 0,
-            outputCostPer1M: 0,
-            totalCost: 0,
-            cacheDiscount: 0,
-        };
+    computeCost(advisor, usage) {
+        if (!advisor.pricing) {
+            return { inputCostPer1M: 0, outputCostPer1M: 0, totalCost: 0, cacheDiscount: 0 };
+        }
+        return calculateCost(usage, advisor.pricing);
     }
 }
 //# sourceMappingURL=executor.js.map

package/dist/advisory/executor.js.map

@@ -1 +1 @@
-{"version":3,"file":"executor.js","sourceRoot":"","sources":["../../src/advisory/executor.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,eAAe,EAAE,MAAM,wBAAwB,CAAA;AACxD,OAAO,EAAE,oBAAoB,EAAE,MAAM,yBAAyB,CAAA;AAC9D,OAAO,EAAE,OAAO,EAAE,MAAM,wBAAwB,CAAA;AAIhD,OAAO,EAAgB,mBAAmB,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAA;AAEhG,OAAO,EAAe,aAAa,EAAE,MAAM,oBAAoB,CAAA;AAgB/D,MAAM,OAAO,gBAAgB;IACX,MAAM,CAAQ;IAE/B,YAAY,MAAe;QAC1B,IAAI,CAAC,MAAM,GAAG,CAAC,MAAM,IAAI,aAAa,EAAE,CAAC,CAAC,KAAK,CAAC,EAAE,SAAS,EAAE,kBAAkB,EAAE,CAAC,CAAA;IACnF,CAAC;IAED,KAAK,CAAC,OAAO,CACZ,OAA0B,EAC1B,OAAwB,EACxB,OAA4B;QAE5B,MAAM,OAAO,GAAG,IAAI,CAAC,GAAG,EAAE,CAAA;QAE1B,MAAM,YAAY,GAAG,IAAI,CAAC,iBAAiB,CAAC,OAAO,CAAC,CAAA;QACpD,MAAM,eAAe,GAAG,IAAI,CAAC,YAAY,CAAC,OAAO,EAAE,OAAO,EAAE,OAAO,CAAC,CAAA;QAEpE,MAAM,QAAQ,GAAc;YAC3B,mBAAmB,CAAC,YAAY,CAAC;YACjC,GAAG,eAAe;YAClB,iBAAiB,CAAC,OAAO,CAAC,QAAQ,CAAC;SACnC,CAAA;QAED,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,wBAAwB,EAAE;YAC3C,SAAS,EAAE,OAAO,CAAC,EAAE;YACrB,KAAK,EAAE,OAAO,CAAC,KAAK;YACpB,YAAY,EAAE,QAAQ,CAAC,MAAM;YAC7B,OAAO,EAAE,OAAO,CAAC,OAAO;SACxB,CAAC,CAAA;QAEF,MAAM,QAAQ,GAAG,MAAM,OAAO,CAC7B,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAC;YAC3B,KAAK,EAAE,OAAO,CAAC,KAAK;YACpB,QAAQ;YACR,WAAW,EAAE,OAAO,CAAC,WAAW;YAChC,SAAS,EAAE,OAAO,CAAC,iBAAiB;YACpC,UAAU,EAAE,MAAM;SAClB,CAAC,CACF,CAAA;QAED,MAAM,UAAU,GAAG,IAAI,CAAC,GAAG,EAAE,GAAG,OAAO,CAAA;QAEvC,MAAM,MAAM,GAAG,IAAI,CAAC,WAAW,CAAC,QAAQ,CAAC,OAAO,CAAC,OAAO,IAAI,EAAE,CAAC,CAAA;QAE/D,MAAM,IAAI,GAAG,IAAI,CAAC,WAAW,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAA;QAE7C,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,yBAAyB,EAAE;YAC3C,SAAS,EAAE,OAAO,CAAC,EAAE;YACrB,KAAK,EAAE,OAAO,CAAC,KAAK;YACpB,UAAU;YACV,WAAW,EAAE,QAAQ,CAAC,KAAK,CAAC,WAAW;SACvC,CAAC,CAAA;QAEF,OAAO;YACN,MAAM;YACN,KAAK,EAAE,QAAQ,CAAC,KAAK;YACrB,IAAI;YACJ,UAAU;SACV,CAAA;IACF,CAAC;IAEO,iBAAiB,CAAC,OAA0B;QACnD,IAAI,OAAO,CAAC,YAAY,EAAE,CAAC;YAC1B,OAAO,OAAO,CAAC,YAAY,CAAA;QAC5B,CAAC;QAED,IAAI,OAAO,CAAC,OAAO,EAAE,CAAC;YACrB,OAAO,oBAAoB,CAAC,OAAO,CAAC,OAAO,CAAC,CAAA;QAC7C,CAAC;QAED,OAAO;YACN,WAAW,OAAO,CAAC,IAAI,sBAAsB;YAC7C,OAAO,CAAC,OAAO,IAAI,OAAO,CAAC,OAAO,CAAC,MAAM,GAAG,CAAC;gBAC5C,CAAC,CAAC,8BAA8B,OAAO,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG;gBAC7D,CAAC,CAAC,SAAS;YACZ,6EAA6E;SAC7E;aACC,MAAM,CAAC,OAAO,CAAC;aACf,IAAI,CAAC,MAAM,CAAC,CAAA;IACf,CAAC;IAEO,YAAY,CACnB,OAA0B,EAC1B,OAAwB,EACxB,OAA4B;QAE5B,IAAI,OAAO,CAAC,cAAc,KAAK,KAAK,EAAE,CAAC;YACtC,OAAO,EAAE,CAAA;QACV,CAAC;QAED,MAAM,YAAY,GAAa,EAAE,CAAA;QAEjC,IAAI,OAAO,CAAC,mBAAmB,EAAE,CAAC;YACjC,YAAY,CAAC,IAAI,CAAC,qBAAqB,OAAO,CAAC,mBAAmB,EAAE,CAAC,CAAA;QACtE,CAAC;QAED,IAAI,OAAO,CAAC,WAAW,IAAI,OAAO,CAAC,WAAW,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC3D,MAAM,SAAS,GAAG,OAAO,CAAC,WAAW,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE;gBAClD,MAAM,WAAW,GAAG,IAAI,CAAC,QAAQ,CAAC,WAAW,EAAE,IAAI,EAAE,CAAA;gBACrD,OAAO,WAAW,CAAC,CAAC,CAAC,KAAK,IAAI,CAAC,QAAQ,CAAC,IAAI,KAAK,WAAW,EAAE,CAAC,CAAC,CAAC,KAAK,IAAI,CAAC,QAAQ,CAAC,IAAI,EAAE,CAAA;YAC3F,CAAC,CAAC,CAAA;YACF,YAAY,CAAC,IAAI,CAChB;gBACC,yBAAyB;gBACzB,oJAAoJ;gBACpJ,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC;aACpB,CAAC,IAAI,CAAC,IAAI,CAAC,CACZ,CAAA;QACF,CAAC;QAED,MAAM,iBAAiB,GAAG,IAAI,CAAC,gBAAgB,CAAC,OAAO,CAAC,QAAQ,EAAE,OAAO,CAAC,gBAAgB,CAAC,CAAA;QAE3F,IAAI,iBAAiB,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAClC,MAAM,mBAAmB,GAAG,iBAAiB;iBAC3C,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,IAAI,CAAC,CAAC,IAAI,MAAM,CAAC,CAAC,OAAO,IAAI,cAAc,EAAE,CAAC;iBACzD,IAAI,CAAC,IAAI,CAAC,CAAA;YACZ,YAAY,CAAC,IAAI,CAAC,4BAA4B,mBAAmB,EAAE,CAAC,CAAA;QACrE,CAAC;QAED,IAAI,YAAY,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YAC/B,OAAO,EAAE,CAAA;QACV,CAAC;QAED,OAAO,CAAC,iBAAiB,CAAC,YAAY,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,CAAA;IACtD,CAAC;IAEO,gBAAgB,CAAC,QAAmB,EAAE,SAA6B;QAC1E,IAAI,CAAC,SAAS,EAAE,CAAC;YAChB,OAAO,QAAQ,CAAA;QAChB,CAAC;QAED,MAAM,UAAU,GAAG,SAAS,GAAG,eAAe,CAAA;QAC9C,IAAI,UAAU,GAAG,CAAC,CAAA;QAClB,MAAM,MAAM,GAAc,EAAE,CAAA;QAE5B,qEAAqE;QACrE,KAAK,IAAI,CAAC,GAAG,QAAQ,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC,IAAI,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC;YAC/C,MAAM,GAAG,GAAG,QAAQ,CAAC,CAAC,CAAY,CAAA;YAClC,MAAM,QAAQ,GAAG,CAAC,GAAG,CAAC,OAAO,IAAI,EAAE,CAAC,CAAC,MAAM,CAAA;YAC3C,IAAI,UAAU,GAAG,QAAQ,GAAG,UAAU;gBAAE,MAAK;YAC7C,UAAU,IAAI,QAAQ,CAAA;YACtB,MAAM,CAAC,OAAO,CAAC,GAAG,CAAC,CAAA;QACpB,CAAC;QAED,OAAO,MAAM,CAAA;IACd,CAAC;IAED;;;;OAIG;IACK,WAAW,CAAC,UAAkB;QACrC,OAAO;YACN,MAAM,EAAE,UAAU;SAClB,CAAA;IACF,CAAC;IAED;;;;;OAKG;IACK,WAAW,CAAC,MAAkB;QACrC,OAAO;YACN,cAAc,EAAE,CAAC;YACjB,eAAe,EAAE,CAAC;YAClB,SAAS,EAAE,CAAC;YACZ,aAAa,EAAE,CAAC;SAChB,CAAA;IACF,CAAC;CACD"}
+{"version":3,"file":"executor.js","sourceRoot":"","sources":["../../src/advisory/executor.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,eAAe,EAAE,MAAM,wBAAwB,CAAA;AACxD,OAAO,EAAE,oBAAoB,EAAE,MAAM,yBAAyB,CAAA;AAC9D,OAAO,EAAE,OAAO,EAAE,MAAM,wBAAwB,CAAA;AAKhD,OAAO,EAAgB,mBAAmB,EAAE,iBAAiB,EAAE,MAAM,2BAA2B,CAAA;AAEhG,OAAO,EAAE,aAAa,EAAE,MAAM,kBAAkB,CAAA;AAChD,OAAO,EAAe,aAAa,EAAE,MAAM,oBAAoB,CAAA;AAC/D,OAAO,EAAE,0BAA0B,EAAE,qBAAqB,EAAE,MAAM,YAAY,CAAA;AAgB9E,MAAM,OAAO,gBAAgB;IACX,MAAM,CAAQ;IACd,MAAM,CAA4B;IAEnD,YAAY,MAAe,EAAE,MAAuB;QACnD,IAAI,CAAC,MAAM,GAAG,CAAC,MAAM,IAAI,aAAa,EAAE,CAAC,CAAC,KAAK,CAAC,EAAE,SAAS,EAAE,kBAAkB,EAAE,CAAC,CAAA;QAClF,IAAI,CAAC,MAAM,GAAG,MAAM,CAAA;IACrB,CAAC;IAED;;;;OAIG;IACK,oBAAoB,CAAC,OAA0B;QACtD,MAAM,GAAG,GAAG,IAAI,CAAC,MAAM,EAAE,gBAAgB,CAAA;QACzC,IAAI,GAAG,KAAK,SAAS;YAAE,OAAO,OAAO,CAAC,iBAAiB,CAAA;QACvD,IAAI,OAAO,CAAC,iBAAiB,KAAK,SAAS;YAAE,OAAO,GAAG,CAAA;QACvD,OAAO,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,iBAAiB,EAAE,GAAG,CAAC,CAAA;IAChD,CAAC;IAED,KAAK,CAAC,OAAO,CACZ,OAA0B,EAC1B,OAAwB,EACxB,OAA4B;QAE5B,MAAM,OAAO,GAAG,IAAI,CAAC,GAAG,EAAE,CAAA;QAE1B,MAAM,YAAY,GAAG,IAAI,CAAC,iBAAiB,CAAC,OAAO,CAAC,CAAA;QACpD,MAAM,eAAe,GAAG,IAAI,CAAC,YAAY,CAAC,OAAO,EAAE,OAAO,EAAE,OAAO,CAAC,CAAA;QAEpE,MAAM,QAAQ,GAAc;YAC3B,mBAAmB,CAAC,YAAY,CAAC;YACjC,GAAG,eAAe;YAClB,iBAAiB,CAAC,OAAO,CAAC,QAAQ,CAAC;SACnC,CAAA;QAED,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,wBAAwB,EAAE;YAC3C,SAAS,EAAE,OAAO,CAAC,EAAE;YACrB,KAAK,EAAE,OAAO,CAAC,KAAK;YACpB,YAAY,EAAE,QAAQ,CAAC,MAAM;YAC7B,OAAO,EAAE,OAAO,CAAC,OAAO;SACxB,CAAC,CAAA;QAEF,MAAM,QAAQ,GAAG,MAAM,OAAO,CAC7B,OAAO,CAAC,QAAQ,CAAC,UAAU,CAAC;YAC3B,KAAK,EAAE,OAAO,CAAC,KAAK;YACpB,QAAQ;YACR,WAAW,EAAE,OAAO,CAAC,WAAW;YAChC,SAAS,EAAE,IAAI,CAAC,oBAAoB,CAAC,OAAO,CAAC;YAC7C,UAAU,EAAE,MAAM;SAClB,CAAC,CACF,CAAA;QAED,MAAM,UAAU,GAAG,IAAI,CAAC,GAAG,EAAE,GAAG,OAAO,CAAA;QAEvC,MAAM,MAAM,GAAG,qBAAqB,CAAC,QAAQ,CAAC,OAAO,CAAC,OAAO,IAAI,EAAE,CAAC,CAAA;QAEpE,MAAM,IAAI,GAAG,IAAI,CAAC,WAAW,CAAC,OAAO,EAAE,QAAQ,CAAC,KAAK,CAAC,CAAA;QAEtD,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,yBAAyB,EAAE;YAC3C,SAAS,EAAE,OAAO,CAAC,EAAE;YACrB,KAAK,EAAE,OAAO,CAAC,KAAK;YACpB,UAAU;YACV,WAAW,EAAE,QAAQ,CAAC,KAAK,CAAC,WAAW;SACvC,CAAC,CAAA;QAEF,OAAO;YACN,MAAM;YACN,KAAK,EAAE,QAAQ,CAAC,KAAK;YACrB,IAAI;YACJ,UAAU;SACV,CAAA;IACF,CAAC;IAEO,iBAAiB,CAAC,OAA0B;QACnD,gEAAgE;QAChE,qEAAqE;QACrE,iEAAiE;QACjE,OAAO,GAAG,IAAI,CAAC,eAAe,CAAC,OAAO,CAAC,OAAO,0BAA0B,EAAE,CAAA;IAC3E,CAAC;IAEO,eAAe,CAAC,OAA0B;QACjD,IAAI,OAAO,CAAC,YAAY,EAAE,CAAC;YAC1B,OAAO,OAAO,CAAC,YAAY,CAAA;QAC5B,CAAC;QAED,IAAI,OAAO,CAAC,OAAO,EAAE,CAAC;YACrB,OAAO,oBAAoB,CAAC,OAAO,CAAC,OAAO,CAAC,CAAA;QAC7C,CAAC;QAED,OAAO;YACN,WAAW,OAAO,CAAC,IAAI,sBAAsB;YAC7C,OAAO,CAAC,OAAO,IAAI,OAAO,CAAC,OAAO,CAAC,MAAM,GAAG,CAAC;gBAC5C,CAAC,CAAC,8BAA8B,OAAO,CAAC,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG;gBAC7D,CAAC,CAAC,SAAS;YACZ,6EAA6E;SAC7E;aACC,MAAM,CAAC,OAAO,CAAC;aACf,IAAI,CAAC,MAAM,CAAC,CAAA;IACf,CAAC;IAEO,YAAY,CACnB,OAA0B,EAC1B,OAAwB,EACxB,OAA4B;QAE5B,IAAI,OAAO,CAAC,cAAc,KAAK,KAAK,EAAE,CAAC;YACtC,OAAO,EAAE,CAAA;QACV,CAAC;QAED,MAAM,YAAY,GAAa,EAAE,CAAA;QAEjC,IAAI,OAAO,CAAC,mBAAmB,EAAE,CAAC;YACjC,YAAY,CAAC,IAAI,CAAC,qBAAqB,OAAO,CAAC,mBAAmB,EAAE,CAAC,CAAA;QACtE,CAAC;QAED,IAAI,OAAO,CAAC,WAAW,IAAI,OAAO,CAAC,WAAW,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC3D,MAAM,SAAS,GAAG,OAAO,CAAC,WAAW,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE;gBAClD,MAAM,WAAW,GAAG,IAAI,CAAC,QAAQ,CAAC,WAAW,EAAE,IAAI,EAAE,CAAA;gBACrD,OAAO,WAAW,CAAC,CAAC,CAAC,KAAK,IAAI,CAAC,QAAQ,CAAC,IAAI,KAAK,WAAW,EAAE,CAAC,CAAC,CAAC,KAAK,IAAI,CAAC,QAAQ,CAAC,IAAI,EAAE,CAAA;YAC3F,CAAC,CAAC,CAAA;YACF,YAAY,CAAC,IAAI,CAChB;gBACC,yBAAyB;gBACzB,oJAAoJ;gBACpJ,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC;aACpB,CAAC,IAAI,CAAC,IAAI,CAAC,CACZ,CAAA;QACF,CAAC;QAED,MAAM,iBAAiB,GAAG,IAAI,CAAC,gBAAgB,CAAC,OAAO,CAAC,QAAQ,EAAE,OAAO,CAAC,gBAAgB,CAAC,CAAA;QAE3F,IAAI,iBAAiB,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAClC,MAAM,mBAAmB,GAAG,iBAAiB;iBAC3C,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,IAAI,CAAC,CAAC,IAAI,MAAM,CAAC,CAAC,OAAO,IAAI,cAAc,EAAE,CAAC;iBACzD,IAAI,CAAC,IAAI,CAAC,CAAA;YACZ,YAAY,CAAC,IAAI,CAAC,4BAA4B,mBAAmB,EAAE,CAAC,CAAA;QACrE,CAAC;QAED,IAAI,YAAY,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YAC/B,OAAO,EAAE,CAAA;QACV,CAAC;QAED,OAAO,CAAC,iBAAiB,CAAC,YAAY,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,CAAA;IACtD,CAAC;IAEO,gBAAgB,CAAC,QAAmB,EAAE,SAA6B;QAC1E,IAAI,CAAC,SAAS,EAAE,CAAC;YAChB,OAAO,QAAQ,CAAA;QAChB,CAAC;QAED,MAAM,UAAU,GAAG,SAAS,GAAG,eAAe,CAAA;QAC9C,IAAI,UAAU,GAAG,CAAC,CAAA;QAClB,MAAM,MAAM,GAAc,EAAE,CAAA;QAE5B,qEAAqE;QACrE,KAAK,IAAI,CAAC,GAAG,QAAQ,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC,IAAI,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC;YAC/C,MAAM,GAAG,GAAG,QAAQ,CAAC,CAAC,CAAY,CAAA;YAClC,MAAM,QAAQ,GAAG,CAAC,GAAG,CAAC,OAAO,IAAI,EAAE,CAAC,CAAC,MAAM,CAAA;YAC3C,IAAI,UAAU,GAAG,QAAQ,GAAG,UAAU;gBAAE,MAAK;YAC7C,UAAU,IAAI,QAAQ,CAAA;YACtB,MAAM,CAAC,OAAO,CAAC,GAAG,CAAC,CAAA;QACpB,CAAC;QAED,OAAO,MAAM,CAAA;IACd,CAAC;IAED;;;;;;OAMG;IACK,WAAW,CAAC,OAA0B,EAAE,KAAiB;QAChE,IAAI,CAAC,OAAO,CAAC,OAAO,EAAE,CAAC;YACtB,OAAO,EAAE,cAAc,EAAE,CAAC,EAAE,eAAe,EAAE,CAAC,EAAE,SAAS,EAAE,CAAC,EAAE,aAAa,EAAE,CAAC,EAAE,CAAA;QACjF,CAAC;QACD,OAAO,aAAa,CAAC,KAAK,EAAE,OAAO,CAAC,OAAO,CAAC,CAAA;IAC7C,CAAC;CACD"}

package/dist/advisory/executor.test.d.ts

@@ -6,17 +6,19 @@
  *     - Builds a context message block (see buildContext tests).
  *     - Concatenates `[system, ...context, user(question)]` and calls
  *       `advisor.provider.chat(...)` with `toolChoice: 'none'`.
- *     - Parses the response via a passthrough `{ advice: rawContent }`
- *       shape (no structured parsing yet — advisor.ts line 166-168).
- *     - Returns `{result, usage, cost, durationMs}`. `cost` is a
- *       zero-value `CostInfo` — pricing is provider-specific and not
- *       applied here.
+ *     - Parses the response with `parseAdvisoryResponse`, lifting any
+ *       `<warnings>` / `<decisions>` blocks out of the prose.
+ *     - Returns `{result, usage, cost, durationMs}`. `cost` is computed
+ *       from `advisor.pricing` when present, and zero-valued when the
+ *       advisor carries none.
  *
  *   - `buildSystemPrompt` priority:
- *     1. `advisor.systemPrompt` (verbatim).
+ *     1. `advisor.systemPrompt`.
  *     2. `advisor.persona` (via `assembleSystemPrompt`).
  *     3. Fallback: "You are <name>, an advisory agent." + optional
  *        domains line + "Provide concise, actionable advice..."
+ *     Every branch is followed by the response contract, so an advisor
+ *     with its own prompt is still told how its answer is read back.
  *
  *   - `buildContext`:
  *     - Returns [] when `request.includeContext === false`.

package/dist/advisory/executor.test.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"executor.test.d.ts","sourceRoot":"","sources":["../../src/advisory/executor.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAiCG"}
+{"version":3,"file":"executor.test.d.ts","sourceRoot":"","sources":["../../src/advisory/executor.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAmCG"}