@0xmaxma/claude-gateway 1.6.8 → 1.7.0

package/dist/agent/skill-learning/notifier.js

@@ -0,0 +1,398 @@
+"use strict";
+/**
+ * Skill notifier — tells the operator when the closed loop writes a skill.
+ *
+ * Two independent surfaces (per user request, 2026-08-17):
+ *   1. **Diary** (`SKILLS_LEARNED.md` in the workspace) — ALWAYS appended, an
+ *      immutable audit log of every auto-write. Cheap, offline, never throttled.
+ *   2. **Channel push** (Telegram) — a short "learned a skill" ping, gated by
+ *      `cfg.notify`. Throttled: up to NOTIFY_THROTTLE_MAX immediate pings per
+ *      rolling window, then further writes coalesce into one digest so a burst
+ *      cannot spam the chat.
+ *
+ * Everything here is best-effort and MUST NOT throw into the review path.
+ * The channel transport (`send`) is injected — the manager builds the real
+ * Telegram sender; tests pass a capturing stub. Diary + throttle logic are
+ * transport-agnostic and fully unit-testable with an injected clock.
+ */
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.SkillNotifier = exports.NOTIFY_THROTTLE_MAX = exports.NOTIFY_THROTTLE_WINDOW_MS = exports.DIARY_FILENAME = void 0;
+exports.readTelegramRecipients = readTelegramRecipients;
+exports.pushTelegram = pushTelegram;
+exports.buildTelegramSend = buildTelegramSend;
+exports.readDiscordRecipients = readDiscordRecipients;
+exports.pushDiscordDM = pushDiscordDM;
+exports.buildDiscordSend = buildDiscordSend;
+exports.readLineRecipients = readLineRecipients;
+exports.pushLine = pushLine;
+exports.buildLineSend = buildLineSend;
+exports.buildChannelSend = buildChannelSend;
+const fs = __importStar(require("fs"));
+const path = __importStar(require("path"));
+exports.DIARY_FILENAME = 'SKILLS_LEARNED.md';
+/** Rolling window for the burst throttle. */
+exports.NOTIFY_THROTTLE_WINDOW_MS = 10 * 60000; // 10 minutes
+/** Immediate pings allowed per window before writes coalesce into a digest. */
+exports.NOTIFY_THROTTLE_MAX = 3;
+const TELEGRAM_API_BASE = 'https://api.telegram.org';
+/**
+ * Read the Telegram allowlist for a workspace — the chat_ids authorized to talk
+ * to this agent, which are exactly the right recipients for its notifications
+ * (Telegram DM chat_id == user_id). Best-effort: any error ⇒ no recipients.
+ */
+function readTelegramRecipients(workspaceDir) {
+    return readAllowFrom(workspaceDir, '.telegram-state');
+}
+/**
+ * Throw a descriptive error when an HTTP push failed, so the per-recipient
+ * `catch` in the transport logs a real status (an HTTP 401/429 is NOT a fetch
+ * throw). The response body is consumed either way (avoids leaking the undici
+ * stream). Bodies never contain the bot token (it lives in the URL path, not
+ * the payload), so this is safe to log.
+ */
+async function ensureOk(resp, channel) {
+    if (resp.ok) {
+        await resp.text().catch(() => '');
+        return;
+    }
+    const body = await resp.text().catch(() => '');
+    throw new Error(`${channel} HTTP ${resp.status}${body ? ` ${body.slice(0, 200)}` : ''}`);
+}
+/** POST a Telegram sendMessage. Throws on HTTP failure so the caller can log it. */
+async function pushTelegram(botToken, chatId, text) {
+    const resp = await fetch(`${TELEGRAM_API_BASE}/bot${botToken}/sendMessage`, {
+        method: 'POST',
+        headers: { 'Content-Type': 'application/json' },
+        body: JSON.stringify({ chat_id: chatId, text }),
+        signal: AbortSignal.timeout(PUSH_TIMEOUT_MS),
+    });
+    await ensureOk(resp, 'telegram');
+}
+/**
+ * Build the channel `send` transport for an agent from its Telegram bot token.
+ * Returns `undefined` when no token is configured (diary-only). The returned fn
+ * resolves recipients fresh on each call (allowlist may change at runtime) and
+ * fans out best-effort — one failed recipient never blocks the others.
+ */
+function buildTelegramSend(workspaceDir, botToken, logger) {
+    if (!botToken)
+        return undefined;
+    return async (text) => {
+        const recipients = readTelegramRecipients(workspaceDir);
+        for (const chatId of recipients) {
+            try {
+                await pushTelegram(botToken, chatId, text);
+            }
+            catch (err) {
+                logger?.warn?.(`[skill-learning] telegram notify failed for ${chatId}: ${err.message}`);
+            }
+        }
+    };
+}
+// ---------------------------------------------------------------------------
+// Multi-channel transport registry
+// ---------------------------------------------------------------------------
+//
+// The notifier is channel-agnostic: it calls one `send(text)` that fans out to
+// every channel the agent has configured. Each channel is a small factory that
+// returns a `SendFn` (or `undefined` when the agent has no token for it). To add
+// a channel, implement its `build<Channel>Send` and register it in
+// TRANSPORT_REGISTRY — nothing else (SkillNotifier, the manager) changes.
+//
+// Recipient resolution is a per-channel concern: Telegram/Discord/Line each read
+// their own `.<channel>-state/access.json` `allowFrom` allowlist (the owners
+// authorized to talk to the agent — exactly who should get its notifications).
+const DISCORD_API_BASE = 'https://discord.com/api/v10';
+const DISCORD_MAX_MESSAGE_LENGTH = 2000;
+const LINE_API_BASE = 'https://api.line.me';
+const LINE_MAX_MESSAGE_LENGTH = 5000; // LINE hard-limits a text message to 5000 chars.
+const PUSH_TIMEOUT_MS = 10000;
+/**
+ * Read an `allowFrom` string[] from `<workspace>/<stateDir>/access.json`.
+ * Shared shape across Telegram/Discord/LINE. Best-effort: any error ⇒ [].
+ */
+function readAllowFrom(workspaceDir, stateDir) {
+    try {
+        const accessPath = path.join(workspaceDir, stateDir, 'access.json');
+        const parsed = JSON.parse(fs.readFileSync(accessPath, 'utf-8'));
+        if (!Array.isArray(parsed.allowFrom))
+            return [];
+        return parsed.allowFrom.filter((x) => typeof x === 'string' && x.length > 0);
+    }
+    catch {
+        return [];
+    }
+}
+/** Split text into chunks no longer than `limit`, preferring paragraph/line/space breaks. */
+function chunkText(text, limit) {
+    if (text.length <= limit)
+        return text.length === 0 ? [] : [text];
+    const chunks = [];
+    let rest = text;
+    while (rest.length > limit) {
+        const para = rest.lastIndexOf('\n\n', limit);
+        const nl = rest.lastIndexOf('\n', limit);
+        const sp = rest.lastIndexOf(' ', limit);
+        const cut = para > limit / 2 ? para : nl > limit / 2 ? nl : sp > 0 ? sp : limit;
+        chunks.push(rest.slice(0, cut));
+        rest = rest.slice(cut).replace(/^\n+/, '');
+    }
+    if (rest)
+        chunks.push(rest);
+    return chunks;
+}
+// ---- Discord ---------------------------------------------------------------
+/** Discord recipients: user_ids from `.discord-state/access.json` `allowFrom`. */
+function readDiscordRecipients(workspaceDir) {
+    return readAllowFrom(workspaceDir, '.discord-state');
+}
+/**
+ * Open (or fetch the existing) DM channel with a user and return its channel id.
+ * Discord push targets a channel, not a user — a proactive DM needs this hop
+ * first (unlike Telegram where chat_id == user_id). Returns undefined on failure.
+ */
+async function openDiscordDmChannel(botToken, userId) {
+    const resp = await fetch(`${DISCORD_API_BASE}/users/@me/channels`, {
+        method: 'POST',
+        headers: { 'Content-Type': 'application/json', Authorization: `Bot ${botToken}` },
+        body: JSON.stringify({ recipient_id: userId }),
+        signal: AbortSignal.timeout(PUSH_TIMEOUT_MS),
+    });
+    await ensureOk(resp, 'discord dm-open');
+    const ch = (await resp.json());
+    if (typeof ch.id !== 'string')
+        throw new Error('discord dm-open: response had no channel id');
+    return ch.id;
+}
+/** DM a Discord user (open DM channel, then post chunked). Throws on HTTP failure. */
+async function pushDiscordDM(botToken, userId, text) {
+    const channelId = await openDiscordDmChannel(botToken, userId);
+    for (const content of chunkText(text, DISCORD_MAX_MESSAGE_LENGTH)) {
+        const resp = await fetch(`${DISCORD_API_BASE}/channels/${channelId}/messages`, {
+            method: 'POST',
+            headers: { 'Content-Type': 'application/json', Authorization: `Bot ${botToken}` },
+            body: JSON.stringify({ content }),
+            signal: AbortSignal.timeout(PUSH_TIMEOUT_MS),
+        });
+        await ensureOk(resp, 'discord');
+    }
+}
+/** Build the Discord `send` transport. undefined when no bot token is configured. */
+function buildDiscordSend(workspaceDir, botToken, logger) {
+    if (!botToken)
+        return undefined;
+    return async (text) => {
+        for (const userId of readDiscordRecipients(workspaceDir)) {
+            try {
+                await pushDiscordDM(botToken, userId, text);
+            }
+            catch (err) {
+                logger?.warn?.(`[skill-learning] discord notify failed for ${userId}: ${err.message}`);
+            }
+        }
+    };
+}
+// ---- LINE ------------------------------------------------------------------
+/** LINE recipients: user_ids from `.line-state/access.json` `allowFrom`. */
+function readLineRecipients(workspaceDir) {
+    return readAllowFrom(workspaceDir, '.line-state');
+}
+/** Push a LINE text message to a user. Throws on HTTP failure so the caller can log it. */
+async function pushLine(accessToken, to, text) {
+    const resp = await fetch(`${LINE_API_BASE}/v2/bot/message/push`, {
+        method: 'POST',
+        headers: { 'Content-Type': 'application/json', Authorization: `Bearer ${accessToken}` },
+        body: JSON.stringify({ to, messages: [{ type: 'text', text: text.slice(0, LINE_MAX_MESSAGE_LENGTH) }] }),
+        signal: AbortSignal.timeout(PUSH_TIMEOUT_MS),
+    });
+    await ensureOk(resp, 'line');
+}
+/**
+ * Build the LINE `send` transport. undefined when no channel access token is
+ * configured — so LINE stays dormant until an agent has LINE set up, at which
+ * point it self-activates with no code change ("just open it": the token flows
+ * in from AgentConfig.line.channelAccessToken at boot).
+ */
+function buildLineSend(workspaceDir, accessToken, logger) {
+    if (!accessToken)
+        return undefined;
+    return async (text) => {
+        for (const to of readLineRecipients(workspaceDir)) {
+            try {
+                await pushLine(accessToken, to, text);
+            }
+            catch (err) {
+                logger?.warn?.(`[skill-learning] line notify failed for ${to}: ${err.message}`);
+            }
+        }
+    };
+}
+/**
+ * Registered channel transports. Add a channel by appending one entry — the
+ * notifier and manager need no changes.
+ *
+ * NOT registered: the web/`api` channel is request/response only and has no
+ * proactive push transport (no SSE/WS/queue for agent-initiated messages).
+ * Wiring it needs new delivery infrastructure — tracked as a separate issue.
+ */
+const TRANSPORT_REGISTRY = [
+    (ws, t, log) => buildTelegramSend(ws, t.telegramBotToken, log),
+    (ws, t, log) => buildDiscordSend(ws, t.discordBotToken, log),
+    (ws, t, log) => buildLineSend(ws, t.lineAccessToken, log),
+];
+/**
+ * Build a single fan-out `send` across every configured channel transport.
+ * Returns undefined when the agent has no push channel wired (diary-only).
+ * Fan-out is best-effort and concurrent — one channel failing never blocks the
+ * others, and no rejection escapes into the review path.
+ */
+function buildChannelSend(workspaceDir, tokens, logger) {
+    const sends = TRANSPORT_REGISTRY.map((build) => build(workspaceDir, tokens, logger)).filter((s) => Boolean(s));
+    if (sends.length === 0)
+        return undefined;
+    return async (text) => {
+        await Promise.all(sends.map(async (send) => {
+            try {
+                await send(text);
+            }
+            catch (err) {
+                logger?.warn?.(`[skill-learning] channel notify failed: ${err.message}`);
+            }
+        }));
+    };
+}
+class SkillNotifier {
+    constructor(opts) {
+        /** Timestamps of immediate pings still inside the current window. */
+        this.recent = [];
+        /** Coalesced writes awaiting a digest flush. */
+        this.buffer = [];
+        this.workspaceDir = opts.workspaceDir;
+        this.notify = opts.notify;
+        this.send = opts.send;
+        this.logger = opts.logger;
+    }
+    /**
+     * Record a live skill write. Appends the diary (always) and, if channel push
+     * is on, either pings immediately or buffers for a digest. Never throws.
+     */
+    onSkillWritten(ev) {
+        this.appendDiary(ev);
+        if (!this.notify || !this.send)
+            return;
+        try {
+            this.pushThrottled(ev);
+        }
+        catch (err) {
+            this.logger?.warn?.(`[skill-learning] notify failed: ${err.message}`);
+        }
+    }
+    /** Emit any buffered digest now (shutdown / test seam / timer callback). */
+    flushPending() {
+        if (this.flushTimer) {
+            clearTimeout(this.flushTimer);
+            this.flushTimer = undefined;
+        }
+        const n = this.buffer.length;
+        if (n > 0 && this.send) {
+            const list = this.buffer.join(', ');
+            this.buffer = [];
+            void this.safeSend(`🧠 ${n} more skill${n === 1 ? '' : 's'} learned (auto): ${list}`);
+        }
+        // Do NOT reset `recent` here: pushThrottled already rolls the window forward
+        // by timestamp on every write, so wiping it would let a mid-life flush (or a
+        // shutdown flush) incorrectly re-open the immediate-ping budget.
+    }
+    // ---- internals -------------------------------------------------------------
+    pushThrottled(ev) {
+        const now = ev.now;
+        // Roll the window forward.
+        this.recent = this.recent.filter((t) => now - t < exports.NOTIFY_THROTTLE_WINDOW_MS);
+        const verb = ev.action === 'create' ? 'created' : 'updated';
+        if (this.recent.length < exports.NOTIFY_THROTTLE_MAX) {
+            this.recent.push(now);
+            void this.safeSend(`🧠 Skill ${verb} (auto): ${ev.name} · session ${shortId(ev.sessionId)} · /skill-metrics`);
+        }
+        else {
+            // Over the burst limit — coalesce into a digest.
+            this.buffer.push(`${verb} ${ev.name}`);
+            this.armFlush(now);
+        }
+    }
+    armFlush(now) {
+        if (this.flushTimer)
+            return;
+        const oldest = this.recent[0] ?? now;
+        const delay = Math.max(0, exports.NOTIFY_THROTTLE_WINDOW_MS - (now - oldest));
+        this.flushTimer = setTimeout(() => this.flushPending(), delay);
+        if (typeof this.flushTimer.unref === 'function') {
+            this.flushTimer.unref();
+        }
+    }
+    async safeSend(text) {
+        if (!this.send)
+            return;
+        try {
+            await this.send(text);
+        }
+        catch (err) {
+            this.logger?.warn?.(`[skill-learning] notify send failed: ${err.message}`);
+        }
+    }
+    appendDiary(ev) {
+        try {
+            const file = path.join(this.workspaceDir, exports.DIARY_FILENAME);
+            const iso = new Date(ev.now).toISOString();
+            let prefix = '';
+            if (!fs.existsSync(file)) {
+                prefix =
+                    '# Skills Learned (auto)\n\n' +
+                        'Audit log of skills the self-improvement loop wrote automatically. ' +
+                        'One line per write; newest at the bottom.\n\n';
+            }
+            const line = `- ${iso} — **${ev.action}** \`${ev.name}\` (session ${shortId(ev.sessionId)})\n`;
+            fs.appendFileSync(file, prefix + line, 'utf-8');
+        }
+        catch (err) {
+            this.logger?.warn?.(`[skill-learning] diary append failed: ${err.message}`);
+        }
+    }
+}
+exports.SkillNotifier = SkillNotifier;
+function shortId(sessionId) {
+    return sessionId.length > 8 ? sessionId.slice(0, 8) : sessionId;
+}
+//# sourceMappingURL=notifier.js.map

package/dist/agent/skill-learning/notifier.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"notifier.js","sourceRoot":"","sources":["../../../src/agent/skill-learning/notifier.ts"],"names":[],"mappings":";AAAA;;;;;;;;;;;;;;;GAeG;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAuCH,wDAEC;AAmBD,oCAQC;AAQD,8CAgBC;AAiED,sDAEC;AAqBD,sCAWC;AAGD,4CAeC;AAKD,gDAEC;AAGD,4BAQC;AAQD,sCAeC;AA+BD,4CAoBC;AA3SD,uCAAyB;AACzB,2CAA6B;AAEhB,QAAA,cAAc,GAAG,mBAAmB,CAAC;AAElD,6CAA6C;AAChC,QAAA,yBAAyB,GAAG,EAAE,GAAG,KAAM,CAAC,CAAC,aAAa;AACnE,+EAA+E;AAClE,QAAA,mBAAmB,GAAG,CAAC,CAAC;AAErC,MAAM,iBAAiB,GAAG,0BAA0B,CAAC;AAsBrD;;;;GAIG;AACH,SAAgB,sBAAsB,CAAC,YAAoB;IACzD,OAAO,aAAa,CAAC,YAAY,EAAE,iBAAiB,CAAC,CAAC;AACxD,CAAC;AAED;;;;;;GAMG;AACH,KAAK,UAAU,QAAQ,CAAC,IAAc,EAAE,OAAe;IACrD,IAAI,IAAI,CAAC,EAAE,EAAE,CAAC;QACZ,MAAM,IAAI,CAAC,IAAI,EAAE,CAAC,KAAK,CAAC,GAAG,EAAE,CAAC,EAAE,CAAC,CAAC;QAClC,OAAO;IACT,CAAC;IACD,MAAM,IAAI,GAAG,MAAM,IAAI,CAAC,IAAI,EAAE,CAAC,KAAK,CAAC,GAAG,EAAE,CAAC,EAAE,CAAC,CAAC;IAC/C,MAAM,IAAI,KAAK,CAAC,GAAG,OAAO,SAAS,IAAI,CAAC,MAAM,GAAG,IAAI,CAAC,CAAC,CAAC,IAAI,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;AAC3F,CAAC;AAED,oFAAoF;AAC7E,KAAK,UAAU,YAAY,CAAC,QAAgB,EAAE,MAAc,EAAE,IAAY;IAC/E,MAAM,IAAI,GAAG,MAAM,KAAK,CAAC,GAAG,iBAAiB,OAAO,QAAQ,cAAc,EAAE;QAC1E,MAAM,EAAE,MAAM;QACd,OAAO,EAAE,EAAE,cAAc,EAAE,kBAAkB,EAAE;QAC/C,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC,EAAE,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;QAC/C,MAAM,EAAE,WAAW,CAAC,OAAO,CAAC,eAAe,CAAC;KAC7C,CAAC,CAAC;IACH,MAAM,QAAQ,CAAC,IAAI,EAAE,UAAU,CAAC,CAAC;AACnC,CAAC;AAED;;;;;GAKG;AACH,SAAgB,iBAAiB,CAC/B,YAAoB,EACpB,QAA4B,EAC5B,MAAoC;IAEpC,IAAI,CAAC,QAAQ;QAAE,OAAO,SAAS,CAAC;IAChC,OAAO,KAAK,EAAE,IAAY,EAAE,EAAE;QAC5B,MAAM,UAAU,GAAG,sBAAsB,CAAC,YAAY,CAAC,CAAC;QACxD,KAAK,MAAM,MAAM,IAAI,UAAU,EAAE,CAAC;YAChC,IAAI,CAAC;gBACH,MAAM,YAAY,CAAC,QAAQ,EAAE,MAAM,EAAE,IAAI,CAAC,CAAC;YAC7C,CAAC;YAAC,OAAO,GAAG,EAAE,CAAC;gBACb,MAAM,EAAE,IAAI,EAAE,CAAC,+CAA+C,MAAM,KAAM,GAAa,CAAC,OAAO,EAAE,CAAC,CAAC;YACrG,CAAC;QACH,CAAC;IACH,CAAC,CAAC;AACJ,CAAC;AAED,8EAA8E;AAC9E,mCAAmC;AACnC,8EAA8E;AAC9E,EAAE;AACF,+EAA+E;AAC/E,+EAA+E;AAC/E,iFAAiF;AACjF,mEAAmE;AACnE,0EAA0E;AAC1E,EAAE;AACF,iFAAiF;AACjF,6EAA6E;AAC7E,+EAA+E;AAE/E,MAAM,gBAAgB,GAAG,6BAA6B,CAAC;AACvD,MAAM,0BAA0B,GAAG,IAAI,CAAC;AACxC,MAAM,aAAa,GAAG,qBAAqB,CAAC;AAC5C,MAAM,uBAAuB,GAAG,IAAI,CAAC,CAAC,iDAAiD;AACvF,MAAM,eAAe,GAAG,KAAM,CAAC;AAU/B;;;GAGG;AACH,SAAS,aAAa,CAAC,YAAoB,EAAE,QAAgB;IAC3D,IAAI,CAAC;QACH,MAAM,UAAU,GAAG,IAAI,CAAC,IAAI,CAAC,YAAY,EAAE,QAAQ,EAAE,aAAa,CAAC,CAAC;QACpE,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,EAAE,CAAC,YAAY,CAAC,UAAU,EAAE,OAAO,CAAC,CAA4B,CAAC;QAC3F,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,SAAS,CAAC;YAAE,OAAO,EAAE,CAAC;QAChD,OAAO,MAAM,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC,CAAC,EAAe,EAAE,CAAC,OAAO,CAAC,KAAK,QAAQ,IAAI,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;IAC5F,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,EAAE,CAAC;IACZ,CAAC;AACH,CAAC;AAED,6FAA6F;AAC7F,SAAS,SAAS,CAAC,IAAY,EAAE,KAAa;IAC5C,IAAI,IAAI,CAAC,MAAM,IAAI,KAAK;QAAE,OAAO,IAAI,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;IACjE,MAAM,MAAM,GAAa,EAAE,CAAC;IAC5B,IAAI,IAAI,GAAG,IAAI,CAAC;IAChB,OAAO,IAAI,CAAC,MAAM,GAAG,KAAK,EAAE,CAAC;QAC3B,MAAM,IAAI,GAAG,IAAI,CAAC,WAAW,CAAC,MAAM,EAAE,KAAK,CAAC,CAAC;QAC7C,MAAM,EAAE,GAAG,IAAI,CAAC,WAAW,CAAC,IAAI,EAAE,KAAK,CAAC,CAAC;QACzC,MAAM,EAAE,GAAG,IAAI,CAAC,WAAW,CAAC,GAAG,EAAE,KAAK,CAAC,CAAC;QACxC,MAAM,GAAG,GAAG,IAAI,GAAG,KAAK,GAAG,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,GAAG,KAAK,GAAG,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,KAAK,CAAC;QAChF,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC,CAAC;QAChC,IAAI,GAAG,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,OAAO,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC;IAC7C,CAAC;IACD,IAAI,IAAI;QAAE,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC5B,OAAO,MAAM,CAAC;AAChB,CAAC;AAED,+EAA+E;AAE/E,kFAAkF;AAClF,SAAgB,qBAAqB,CAAC,YAAoB;IACxD,OAAO,aAAa,CAAC,YAAY,EAAE,gBAAgB,CAAC,CAAC;AACvD,CAAC;AAED;;;;GAIG;AACH,KAAK,UAAU,oBAAoB,CAAC,QAAgB,EAAE,MAAc;IAClE,MAAM,IAAI,GAAG,MAAM,KAAK,CAAC,GAAG,gBAAgB,qBAAqB,EAAE;QACjE,MAAM,EAAE,MAAM;QACd,OAAO,EAAE,EAAE,cAAc,EAAE,kBAAkB,EAAE,aAAa,EAAE,OAAO,QAAQ,EAAE,EAAE;QACjF,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC,EAAE,YAAY,EAAE,MAAM,EAAE,CAAC;QAC9C,MAAM,EAAE,WAAW,CAAC,OAAO,CAAC,eAAe,CAAC;KAC7C,CAAC,CAAC;IACH,MAAM,QAAQ,CAAC,IAAI,EAAE,iBAAiB,CAAC,CAAC;IACxC,MAAM,EAAE,GAAG,CAAC,MAAM,IAAI,CAAC,IAAI,EAAE,CAAqB,CAAC;IACnD,IAAI,OAAO,EAAE,CAAC,EAAE,KAAK,QAAQ;QAAE,MAAM,IAAI,KAAK,CAAC,6CAA6C,CAAC,CAAC;IAC9F,OAAO,EAAE,CAAC,EAAE,CAAC;AACf,CAAC;AAED,sFAAsF;AAC/E,KAAK,UAAU,aAAa,CAAC,QAAgB,EAAE,MAAc,EAAE,IAAY;IAChF,MAAM,SAAS,GAAG,MAAM,oBAAoB,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC;IAC/D,KAAK,MAAM,OAAO,IAAI,SAAS,CAAC,IAAI,EAAE,0BAA0B,CAAC,EAAE,CAAC;QAClE,MAAM,IAAI,GAAG,MAAM,KAAK,CAAC,GAAG,gBAAgB,aAAa,SAAS,WAAW,EAAE;YAC7E,MAAM,EAAE,MAAM;YACd,OAAO,EAAE,EAAE,cAAc,EAAE,kBAAkB,EAAE,aAAa,EAAE,OAAO,QAAQ,EAAE,EAAE;YACjF,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC,EAAE,OAAO,EAAE,CAAC;YACjC,MAAM,EAAE,WAAW,CAAC,OAAO,CAAC,eAAe,CAAC;SAC7C,CAAC,CAAC;QACH,MAAM,QAAQ,CAAC,IAAI,EAAE,SAAS,CAAC,CAAC;IAClC,CAAC;AACH,CAAC;AAED,qFAAqF;AACrF,SAAgB,gBAAgB,CAC9B,YAAoB,EACpB,QAA4B,EAC5B,MAAoC;IAEpC,IAAI,CAAC,QAAQ;QAAE,OAAO,SAAS,CAAC;IAChC,OAAO,KAAK,EAAE,IAAY,EAAE,EAAE;QAC5B,KAAK,MAAM,MAAM,IAAI,qBAAqB,CAAC,YAAY,CAAC,EAAE,CAAC;YACzD,IAAI,CAAC;gBACH,MAAM,aAAa,CAAC,QAAQ,EAAE,MAAM,EAAE,IAAI,CAAC,CAAC;YAC9C,CAAC;YAAC,OAAO,GAAG,EAAE,CAAC;gBACb,MAAM,EAAE,IAAI,EAAE,CAAC,8CAA8C,MAAM,KAAM,GAAa,CAAC,OAAO,EAAE,CAAC,CAAC;YACpG,CAAC;QACH,CAAC;IACH,CAAC,CAAC;AACJ,CAAC;AAED,+EAA+E;AAE/E,4EAA4E;AAC5E,SAAgB,kBAAkB,CAAC,YAAoB;IACrD,OAAO,aAAa,CAAC,YAAY,EAAE,aAAa,CAAC,CAAC;AACpD,CAAC;AAED,2FAA2F;AACpF,KAAK,UAAU,QAAQ,CAAC,WAAmB,EAAE,EAAU,EAAE,IAAY;IAC1E,MAAM,IAAI,GAAG,MAAM,KAAK,CAAC,GAAG,aAAa,sBAAsB,EAAE;QAC/D,MAAM,EAAE,MAAM;QACd,OAAO,EAAE,EAAE,cAAc,EAAE,kBAAkB,EAAE,aAAa,EAAE,UAAU,WAAW,EAAE,EAAE;QACvF,IAAI,EAAE,IAAI,CAAC,SAAS,CAAC,EAAE,EAAE,EAAE,QAAQ,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,uBAAuB,CAAC,EAAE,CAAC,EAAE,CAAC;QACxG,MAAM,EAAE,WAAW,CAAC,OAAO,CAAC,eAAe,CAAC;KAC7C,CAAC,CAAC;IACH,MAAM,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;AAC/B,CAAC;AAED;;;;;GAKG;AACH,SAAgB,aAAa,CAC3B,YAAoB,EACpB,WAA+B,EAC/B,MAAoC;IAEpC,IAAI,CAAC,WAAW;QAAE,OAAO,SAAS,CAAC;IACnC,OAAO,KAAK,EAAE,IAAY,EAAE,EAAE;QAC5B,KAAK,MAAM,EAAE,IAAI,kBAAkB,CAAC,YAAY,CAAC,EAAE,CAAC;YAClD,IAAI,CAAC;gBACH,MAAM,QAAQ,CAAC,WAAW,EAAE,EAAE,EAAE,IAAI,CAAC,CAAC;YACxC,CAAC;YAAC,OAAO,GAAG,EAAE,CAAC;gBACb,MAAM,EAAE,IAAI,EAAE,CAAC,2CAA2C,EAAE,KAAM,GAAa,CAAC,OAAO,EAAE,CAAC,CAAC;YAC7F,CAAC;QACH,CAAC;IACH,CAAC,CAAC;AACJ,CAAC;AAWD;;;;;;;GAOG;AACH,MAAM,kBAAkB,GAAuB;IAC7C,CAAC,EAAE,EAAE,CAAC,EAAE,GAAG,EAAE,EAAE,CAAC,iBAAiB,CAAC,EAAE,EAAE,CAAC,CAAC,gBAAgB,EAAE,GAAG,CAAC;IAC9D,CAAC,EAAE,EAAE,CAAC,EAAE,GAAG,EAAE,EAAE,CAAC,gBAAgB,CAAC,EAAE,EAAE,CAAC,CAAC,eAAe,EAAE,GAAG,CAAC;IAC5D,CAAC,EAAE,EAAE,CAAC,EAAE,GAAG,EAAE,EAAE,CAAC,aAAa,CAAC,EAAE,EAAE,CAAC,CAAC,eAAe,EAAE,GAAG,CAAC;CAC1D,CAAC;AAEF;;;;;GAKG;AACH,SAAgB,gBAAgB,CAC9B,YAAoB,EACpB,MAAqB,EACrB,MAAoC;IAEpC,MAAM,KAAK,GAAG,kBAAkB,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,YAAY,EAAE,MAAM,EAAE,MAAM,CAAC,CAAC,CAAC,MAAM,CACzF,CAAC,CAAC,EAAe,EAAE,CAAC,OAAO,CAAC,CAAC,CAAC,CAC/B,CAAC;IACF,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,SAAS,CAAC;IACzC,OAAO,KAAK,EAAE,IAAY,EAAE,EAAE;QAC5B,MAAM,OAAO,CAAC,GAAG,CACf,KAAK,CAAC,GAAG,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE;YACvB,IAAI,CAAC;gBACH,MAAM,IAAI,CAAC,IAAI,CAAC,CAAC;YACnB,CAAC;YAAC,OAAO,GAAG,EAAE,CAAC;gBACb,MAAM,EAAE,IAAI,EAAE,CAAC,2CAA4C,GAAa,CAAC,OAAO,EAAE,CAAC,CAAC;YACtF,CAAC;QACH,CAAC,CAAC,CACH,CAAC;IACJ,CAAC,CAAC;AACJ,CAAC;AAED,MAAa,aAAa;IAYxB,YAAY,IAAuB;QANnC,qEAAqE;QAC7D,WAAM,GAAa,EAAE,CAAC;QAC9B,gDAAgD;QACxC,WAAM,GAAa,EAAE,CAAC;QAI5B,IAAI,CAAC,YAAY,GAAG,IAAI,CAAC,YAAY,CAAC;QACtC,IAAI,CAAC,MAAM,GAAG,IAAI,CAAC,MAAM,CAAC;QAC1B,IAAI,CAAC,IAAI,GAAG,IAAI,CAAC,IAAI,CAAC;QACtB,IAAI,CAAC,MAAM,GAAG,IAAI,CAAC,MAAM,CAAC;IAC5B,CAAC;IAED;;;OAGG;IACH,cAAc,CAAC,EAAmB;QAChC,IAAI,CAAC,WAAW,CAAC,EAAE,CAAC,CAAC;QACrB,IAAI,CAAC,IAAI,CAAC,MAAM,IAAI,CAAC,IAAI,CAAC,IAAI;YAAE,OAAO;QACvC,IAAI,CAAC;YACH,IAAI,CAAC,aAAa,CAAC,EAAE,CAAC,CAAC;QACzB,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,IAAI,CAAC,MAAM,EAAE,IAAI,EAAE,CAAC,mCAAoC,GAAa,CAAC,OAAO,EAAE,CAAC,CAAC;QACnF,CAAC;IACH,CAAC;IAED,4EAA4E;IAC5E,YAAY;QACV,IAAI,IAAI,CAAC,UAAU,EAAE,CAAC;YACpB,YAAY,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;YAC9B,IAAI,CAAC,UAAU,GAAG,SAAS,CAAC;QAC9B,CAAC;QACD,MAAM,CAAC,GAAG,IAAI,CAAC,MAAM,CAAC,MAAM,CAAC;QAC7B,IAAI,CAAC,GAAG,CAAC,IAAI,IAAI,CAAC,IAAI,EAAE,CAAC;YACvB,MAAM,IAAI,GAAG,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YACpC,IAAI,CAAC,MAAM,GAAG,EAAE,CAAC;YACjB,KAAK,IAAI,CAAC,QAAQ,CAAC,MAAM,CAAC,cAAc,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,oBAAoB,IAAI,EAAE,CAAC,CAAC;QACxF,CAAC;QACD,6EAA6E;QAC7E,6EAA6E;QAC7E,iEAAiE;IACnE,CAAC;IAED,+EAA+E;IAEvE,aAAa,CAAC,EAAmB;QACvC,MAAM,GAAG,GAAG,EAAE,CAAC,GAAG,CAAC;QACnB,2BAA2B;QAC3B,IAAI,CAAC,MAAM,GAAG,IAAI,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,GAAG,GAAG,CAAC,GAAG,iCAAyB,CAAC,CAAC;QAC7E,MAAM,IAAI,GAAG,EAAE,CAAC,MAAM,KAAK,QAAQ,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC;QAC5D,IAAI,IAAI,CAAC,MAAM,CAAC,MAAM,GAAG,2BAAmB,EAAE,CAAC;YAC7C,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;YACtB,KAAK,IAAI,CAAC,QAAQ,CAAC,YAAY,IAAI,YAAY,EAAE,CAAC,IAAI,cAAc,OAAO,CAAC,EAAE,CAAC,SAAS,CAAC,mBAAmB,CAAC,CAAC;QAChH,CAAC;aAAM,CAAC;YACN,iDAAiD;YACjD,IAAI,CAAC,MAAM,CAAC,IAAI,CAAC,GAAG,IAAI,IAAI,EAAE,CAAC,IAAI,EAAE,CAAC,CAAC;YACvC,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC;QACrB,CAAC;IACH,CAAC;IAEO,QAAQ,CAAC,GAAW;QAC1B,IAAI,IAAI,CAAC,UAAU;YAAE,OAAO;QAC5B,MAAM,MAAM,GAAG,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,GAAG,CAAC;QACrC,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,iCAAyB,GAAG,CAAC,GAAG,GAAG,MAAM,CAAC,CAAC,CAAC;QACtE,IAAI,CAAC,UAAU,GAAG,UAAU,CAAC,GAAG,EAAE,CAAC,IAAI,CAAC,YAAY,EAAE,EAAE,KAAK,CAAC,CAAC;QAC/D,IAAI,OAAQ,IAAI,CAAC,UAA6B,CAAC,KAAK,KAAK,UAAU,EAAE,CAAC;YACnE,IAAI,CAAC,UAA6B,CAAC,KAAK,EAAE,CAAC;QAC9C,CAAC;IACH,CAAC;IAEO,KAAK,CAAC,QAAQ,CAAC,IAAY;QACjC,IAAI,CAAC,IAAI,CAAC,IAAI;YAAE,OAAO;QACvB,IAAI,CAAC;YACH,MAAM,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACxB,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,IAAI,CAAC,MAAM,EAAE,IAAI,EAAE,CAAC,wCAAyC,GAAa,CAAC,OAAO,EAAE,CAAC,CAAC;QACxF,CAAC;IACH,CAAC;IAEO,WAAW,CAAC,EAAmB;QACrC,IAAI,CAAC;YACH,MAAM,IAAI,GAAG,IAAI,CAAC,IAAI,CAAC,IAAI,CAAC,YAAY,EAAE,sBAAc,CAAC,CAAC;YAC1D,MAAM,GAAG,GAAG,IAAI,IAAI,CAAC,EAAE,CAAC,GAAG,CAAC,CAAC,WAAW,EAAE,CAAC;YAC3C,IAAI,MAAM,GAAG,EAAE,CAAC;YAChB,IAAI,CAAC,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,EAAE,CAAC;gBACzB,MAAM;oBACJ,6BAA6B;wBAC7B,qEAAqE;wBACrE,+CAA+C,CAAC;YACpD,CAAC;YACD,MAAM,IAAI,GAAG,KAAK,GAAG,QAAQ,EAAE,CAAC,MAAM,QAAQ,EAAE,CAAC,IAAI,eAAe,OAAO,CAAC,EAAE,CAAC,SAAS,CAAC,KAAK,CAAC;YAC/F,EAAE,CAAC,cAAc,CAAC,IAAI,EAAE,MAAM,GAAG,IAAI,EAAE,OAAO,CAAC,CAAC;QAClD,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,IAAI,CAAC,MAAM,EAAE,IAAI,EAAE,CAAC,yCAA0C,GAAa,CAAC,OAAO,EAAE,CAAC,CAAC;QACzF,CAAC;IACH,CAAC;CACF;AAvGD,sCAuGC;AAED,SAAS,OAAO,CAAC,SAAiB;IAChC,OAAO,SAAS,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC;AAClE,CAAC"}

package/dist/agent/skill-learning/reviewer.d.ts

@@ -0,0 +1,61 @@
+/**
+ * Background reviewer — turns a finished transcript into a skill *proposal*.
+ *
+ * Safety-critical properties (planning-62, deviation #2 + #3):
+ *   - print-only: `claude -p`, NO tools, NO `--dangerously-skip-permissions`.
+ *     The transcript is untrusted input (prompt-injection surface); the model
+ *     can only emit a JSON proposal — the gateway (writer.ts), not the model,
+ *     writes any file.
+ *   - async spawn only (mirror `recoveryTriageSpawn`); NEVER `spawnSync`
+ *     (freezes the daemon event loop).
+ *
+ * The spawn is injectable so unit/e2e tests can drive a scripted reviewer with
+ * no real model.
+ */
+import type { ReviewProposal, ResolvedSkillLearningCfg } from './types';
+/** Hard ceiling on a reviewer spawn — it must never wedge. */
+export declare const REVIEWER_TIMEOUT_MS = 120000;
+export interface ReviewerInput {
+    /** The finished transcript to analyze (already trimmed by the caller). */
+    transcript: string;
+    /** Existing skills the agent already has (name + description), for dedup/priority. */
+    existingSkills: Array<{
+        name: string;
+        description: string;
+    }>;
+}
+export interface ReviewerResult {
+    proposal: ReviewProposal;
+    /** Tokens the reviewer spent (input+output), for the net-token ledger. 0 if unknown. */
+    tokensSpent: number;
+    timedOut?: boolean;
+}
+/**
+ * Injectable spawn: given argv + stdin, resolve with the process stdout.
+ *
+ * `failureReason` / `stderrTail` are diagnostics (#353): previously every failure
+ * mode collapsed into a bare `timedOut` and the child's stderr was discarded, so
+ * an `outcome=error` dream run was undebuggable. These optional fields let a caller
+ * log WHY a spawn failed (timeout vs spawn-error vs non-zero exit + the API error on
+ * stderr) without changing the success contract — consumers that only read
+ * `{ stdout, timedOut }` are unaffected.
+ */
+export type ClaudeSpawnFn = (args: string[], stdin: string) => Promise<{
+    stdout: string;
+    timedOut?: boolean;
+    failureReason?: string;
+    stderrTail?: string;
+}>;
+/**
+ * Default async claude spawn — print-only, structured (`--output-format json`
+ * so we also get token usage), cheap model. Mirrors `recoveryTriageSpawn` but
+ * adds PATH hardening from the compactor idiom. NO tools, NO skip-permissions.
+ */
+export declare function makeClaudeSpawn(reviewModel: string): ClaudeSpawnFn;
+/** Validate a parsed object into a safe ReviewProposal. Any deviation ⇒ `none`. */
+export declare function coerceProposal(raw: unknown): ReviewProposal;
+/**
+ * Run the reviewer. Never throws — any failure resolves to a `none` proposal.
+ */
+export declare function runReviewer(input: ReviewerInput, cfg: ResolvedSkillLearningCfg, spawnFn?: ClaudeSpawnFn): Promise<ReviewerResult>;
+//# sourceMappingURL=reviewer.d.ts.map

package/dist/agent/skill-learning/reviewer.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"reviewer.d.ts","sourceRoot":"","sources":["../../../src/agent/skill-learning/reviewer.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AAGH,OAAO,KAAK,EAAE,cAAc,EAAE,wBAAwB,EAAE,MAAM,SAAS,CAAC;AAExE,8DAA8D;AAC9D,eAAO,MAAM,mBAAmB,SAAU,CAAC;AAE3C,MAAM,WAAW,aAAa;IAC5B,0EAA0E;IAC1E,UAAU,EAAE,MAAM,CAAC;IACnB,sFAAsF;IACtF,cAAc,EAAE,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,WAAW,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;CAC9D;AAED,MAAM,WAAW,cAAc;IAC7B,QAAQ,EAAE,cAAc,CAAC;IACzB,wFAAwF;IACxF,WAAW,EAAE,MAAM,CAAC;IACpB,QAAQ,CAAC,EAAE,OAAO,CAAC;CACpB;AAED;;;;;;;;;GASG;AACH,MAAM,MAAM,aAAa,GAAG,CAC1B,IAAI,EAAE,MAAM,EAAE,EACd,KAAK,EAAE,MAAM,KACV,OAAO,CAAC;IAAE,MAAM,EAAE,MAAM,CAAC;IAAC,QAAQ,CAAC,EAAE,OAAO,CAAC;IAAC,aAAa,CAAC,EAAE,MAAM,CAAC;IAAC,UAAU,CAAC,EAAE,MAAM,CAAA;CAAE,CAAC,CAAC;AAmClG;;;;GAIG;AACH,wBAAgB,eAAe,CAAC,WAAW,EAAE,MAAM,GAAG,aAAa,CAiFlE;AAuCD,mFAAmF;AACnF,wBAAgB,cAAc,CAAC,GAAG,EAAE,OAAO,GAAG,cAAc,CAe3D;AAED;;GAEG;AACH,wBAAsB,WAAW,CAC/B,KAAK,EAAE,aAAa,EACpB,GAAG,EAAE,wBAAwB,EAC7B,OAAO,CAAC,EAAE,aAAa,GACtB,OAAO,CAAC,cAAc,CAAC,CA6BzB"}