@urateam/core 0.1.29 → 0.1.30

package/dist/pipeline/pr-change-summary.d.ts

@@ -0,0 +1,47 @@
+import type { HandoffArtifact } from "../types.js";
+import type { ReviewFeedbackComment } from "../webhook/github-handler.js";
+import type { addPRComment } from "../repo/github.js";
+import type { Logger } from "pino";
+export interface ChangeSummaryInput {
+    handoff: HandoffArtifact;
+    run: {
+        id: string;
+        totalInputTokens: number;
+        totalOutputTokens: number;
+    };
+    triggeringComments: ReviewFeedbackComment[];
+    dashboardBaseUrl: string;
+    /** PR URL — used as the base for fallback comment URL construction. */
+    prUrl?: string;
+}
+/**
+ * Render the markdown body of the per-PR change-summary comment. Pure
+ * function — no I/O, no GitHub calls.
+ */
+export declare function renderChangeSummary(input: ChangeSummaryInput): string;
+/**
+ * Thin dispatcher called from runner.onPipelineComplete. Decides whether to
+ * post the change summary, builds the input, calls renderChangeSummary, and
+ * posts via addPRComment. Best-effort: any failure is logged at warn level
+ * and the function returns (no throw).
+ */
+export interface MaybePostChangeSummaryDeps {
+    run: {
+        id: string;
+        runType?: string | null;
+        prUrl?: string | null;
+        feedbackContext?: string | null;
+        totalInputTokens: number;
+        totalOutputTokens: number;
+    };
+    handoff: HandoffArtifact | null | undefined;
+    prNumber: number | null;
+    owner: string;
+    repo: string;
+    octokit: Parameters<typeof addPRComment>[0];
+    postPRComment: typeof addPRComment;
+    dashboardBaseUrl: string;
+    logger: Pick<Logger, "info" | "warn" | "error">;
+}
+export declare function maybePostChangeSummary(deps: MaybePostChangeSummaryDeps): Promise<void>;
+//# sourceMappingURL=pr-change-summary.d.ts.map

package/dist/pipeline/pr-change-summary.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"pr-change-summary.d.ts","sourceRoot":"","sources":["../../src/pipeline/pr-change-summary.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,aAAa,CAAC;AACnD,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,8BAA8B,CAAC;AAC1E,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,mBAAmB,CAAC;AACtD,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,MAAM,CAAC;AAsCnC,MAAM,WAAW,kBAAkB;IACjC,OAAO,EAAE,eAAe,CAAC;IACzB,GAAG,EAAE;QAAE,EAAE,EAAE,MAAM,CAAC;QAAC,gBAAgB,EAAE,MAAM,CAAC;QAAC,iBAAiB,EAAE,MAAM,CAAA;KAAE,CAAC;IACzE,kBAAkB,EAAE,qBAAqB,EAAE,CAAC;IAC5C,gBAAgB,EAAE,MAAM,CAAC;IACzB,uEAAuE;IACvE,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAED;;;GAGG;AACH,wBAAgB,mBAAmB,CAAC,KAAK,EAAE,kBAAkB,GAAG,MAAM,CA0DrE;AAED;;;;;GAKG;AACH,MAAM,WAAW,0BAA0B;IACzC,GAAG,EAAE;QACH,EAAE,EAAE,MAAM,CAAC;QACX,OAAO,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;QACxB,KAAK,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;QACtB,eAAe,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;QAChC,gBAAgB,EAAE,MAAM,CAAC;QACzB,iBAAiB,EAAE,MAAM,CAAC;KAC3B,CAAC;IACF,OAAO,EAAE,eAAe,GAAG,IAAI,GAAG,SAAS,CAAC;IAC5C,QAAQ,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,UAAU,CAAC,OAAO,YAAY,CAAC,CAAC,CAAC,CAAC,CAAC;IAC5C,aAAa,EAAE,OAAO,YAAY,CAAC;IACnC,gBAAgB,EAAE,MAAM,CAAC;IACzB,MAAM,EAAE,IAAI,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM,GAAG,OAAO,CAAC,CAAC;CACjD;AAED,wBAAsB,sBAAsB,CAC1C,IAAI,EAAE,0BAA0B,GAC/B,OAAO,CAAC,IAAI,CAAC,CAgFf"}

package/dist/pipeline/pr-change-summary.js

@@ -0,0 +1,133 @@
+/**
+ * Markdown-special characters escaped before rendering user-controlled
+ * strings (author names, file paths, comment bodies). Conservative — escapes
+ * anything that could affect markdown rendering.
+ */
+// `.` is omitted from the escape set: it is only markdown-special at the
+// start of a line in ordered-list contexts (e.g. `1.`) — not in author
+// names or file-paths rendered inline within link text.
+function escapeMd(s) {
+    return s.replace(/[\\`*_{}\[\]()<>#+\-!|]/g, (ch) => `\\${ch}`);
+}
+/** Collapse all newlines (and runs of whitespace) into single spaces. */
+function singleLine(s) {
+    return s.replace(/\s+/g, " ").trim();
+}
+/**
+ * Build a comment-URL fallback when ReviewFeedbackComment.htmlUrl is missing.
+ * Inline review comments (have filePath) are anchored to the Files Changed
+ * tab via #discussion_rNNN; general PR comments use #issuecomment-NNN.
+ *
+ * The base PR URL must be supplied via input.prUrl. If absent, returns "".
+ */
+function buildCommentUrl(prUrl, comment) {
+    if (comment.htmlUrl)
+        return comment.htmlUrl;
+    if (!prUrl)
+        return "";
+    const anchor = comment.filePath
+        ? `#discussion_r${comment.commentId}`
+        : `#issuecomment-${comment.commentId}`;
+    return `${prUrl}${anchor}`;
+}
+/**
+ * Render the markdown body of the per-PR change-summary comment. Pure
+ * function — no I/O, no GitHub calls.
+ */
+export function renderChangeSummary(input) {
+    const { handoff, run, triggeringComments, dashboardBaseUrl, prUrl } = input;
+    const summary = singleLine(handoff.summary);
+    const ctx = handoff.context;
+    const addressedById = new Map();
+    for (const ac of ctx.addressedComments ?? []) {
+        addressedById.set(ac.commentId, ac.response);
+    }
+    const responseLines = triggeringComments.map((c) => {
+        const url = buildCommentUrl(prUrl, c);
+        const linkText = c.filePath
+            ? `@${escapeMd(c.author)}'s comment on \`${escapeMd(c.filePath)}${c.lineNumber !== undefined ? `:${c.lineNumber}` : ""}\``
+            : `@${escapeMd(c.author)}'s general comment`;
+        const link = url ? `[${linkText}](${url})` : linkText;
+        const response = addressedById.get(c.commentId);
+        // Agent-emitted responses are interpolated raw — they are pipeline-
+        // controlled output (not third-party user input) and may legitimately
+        // contain markdown formatting (e.g. backticks for code references).
+        // The agent prompt instructs ≤12 words per response, limiting injection
+        // surface. If we ever discover prompt-injection paths from PR comments
+        // that survive into `response`, revisit this decision.
+        return response ? `- ${link} — ${response}` : `- ${link}`;
+    });
+    // Show the fallback disclaimer when there are triggering comments but none
+    // with a matched response.
+    const hasAnyResponse = Array.isArray(ctx.addressedComments) &&
+        ctx.addressedComments.length > 0 &&
+        triggeringComments.some((c) => addressedById.has(c.commentId));
+    const fallbackLine = triggeringComments.length > 0 && !hasAnyResponse
+        ? "\n_(per-comment responses unavailable; see diff)_"
+        : "";
+    const inResponseSection = triggeringComments.length > 0
+        ? `\n**In response to:**\n${responseLines.join("\n")}${fallbackLine}\n`
+        : "";
+    // File paths rendered inside backtick code spans — no markdown escaping needed.
+    const filesSection = handoff.filesChanged.length > 0
+        ? `\n**Files changed:**\n${handoff.filesChanged.map((f) => `- \`${f}\``).join("\n")}\n`
+        : "";
+    const runLink = dashboardBaseUrl
+        ? `[${run.id}](${dashboardBaseUrl}/runs/${run.id})`
+        : run.id;
+    const footer = `\n<sub>Run ${runLink} · auto-generated</sub>`;
+    return `## 🤖 Addressed PR feedback\n\n${summary}\n${inResponseSection}${filesSection}${footer}`;
+}
+export async function maybePostChangeSummary(deps) {
+    const { run, handoff, prNumber, owner, repo, octokit, postPRComment, dashboardBaseUrl, logger, } = deps;
+    if (run.runType !== "review-feedback")
+        return;
+    if (!run.prUrl || prNumber === null) {
+        logger.info({ runId: run.id }, "skipped change summary: no PR URL on run");
+        return;
+    }
+    if (!handoff) {
+        logger.info({ runId: run.id }, "skipped change summary: no handoff persisted");
+        return;
+    }
+    let triggeringComments = [];
+    if (run.feedbackContext) {
+        try {
+            const parsed = JSON.parse(run.feedbackContext);
+            if (Array.isArray(parsed))
+                triggeringComments = parsed;
+        }
+        catch (err) {
+            logger.error({
+                runId: run.id,
+                err: err instanceof Error ? err.message : String(err),
+            }, "failed to parse feedback_context — posting degraded change summary");
+        }
+    }
+    let body;
+    try {
+        body = renderChangeSummary({
+            handoff,
+            run: {
+                id: run.id,
+                totalInputTokens: run.totalInputTokens,
+                totalOutputTokens: run.totalOutputTokens,
+            },
+            triggeringComments,
+            dashboardBaseUrl,
+            prUrl: run.prUrl,
+        });
+    }
+    catch (err) {
+        logger.error({ runId: run.id, err: err instanceof Error ? err.message : String(err) }, "failed to render change summary — skipping post");
+        return;
+    }
+    try {
+        await postPRComment(octokit, owner, repo, prNumber, body);
+        logger.info({ runId: run.id, prNumber }, "posted PR change summary for review-feedback run");
+    }
+    catch (err) {
+        logger.warn({ runId: run.id, prNumber, err: err instanceof Error ? err.message : String(err) }, "PR change summary post failed (non-fatal)");
+    }
+}
+//# sourceMappingURL=pr-change-summary.js.map

package/dist/pipeline/pr-change-summary.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"pr-change-summary.js","sourceRoot":"","sources":["../../src/pipeline/pr-change-summary.ts"],"names":[],"mappings":"AAKA;;;;GAIG;AACH,yEAAyE;AACzE,uEAAuE;AACvE,wDAAwD;AACxD,SAAS,QAAQ,CAAC,CAAS;IACzB,OAAO,CAAC,CAAC,OAAO,CAAC,0BAA0B,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC;AAClE,CAAC;AAED,yEAAyE;AACzE,SAAS,UAAU,CAAC,CAAS;IAC3B,OAAO,CAAC,CAAC,OAAO,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC,IAAI,EAAE,CAAC;AACvC,CAAC;AAED;;;;;;GAMG;AACH,SAAS,eAAe,CACtB,KAAyB,EACzB,OAA8B;IAE9B,IAAI,OAAO,CAAC,OAAO;QAAE,OAAO,OAAO,CAAC,OAAO,CAAC;IAC5C,IAAI,CAAC,KAAK;QAAE,OAAO,EAAE,CAAC;IACtB,MAAM,MAAM,GAAG,OAAO,CAAC,QAAQ;QAC7B,CAAC,CAAC,gBAAgB,OAAO,CAAC,SAAS,EAAE;QACrC,CAAC,CAAC,iBAAiB,OAAO,CAAC,SAAS,EAAE,CAAC;IACzC,OAAO,GAAG,KAAK,GAAG,MAAM,EAAE,CAAC;AAC7B,CAAC;AAWD;;;GAGG;AACH,MAAM,UAAU,mBAAmB,CAAC,KAAyB;IAC3D,MAAM,EAAE,OAAO,EAAE,GAAG,EAAE,kBAAkB,EAAE,gBAAgB,EAAE,KAAK,EAAE,GAAG,KAAK,CAAC;IAE5E,MAAM,OAAO,GAAG,UAAU,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC;IAE5C,MAAM,GAAG,GAAG,OAAO,CAAC,OAAO,CAAC;IAC5B,MAAM,aAAa,GAAG,IAAI,GAAG,EAAkB,CAAC;IAChD,KAAK,MAAM,EAAE,IAAI,GAAG,CAAC,iBAAiB,IAAI,EAAE,EAAE,CAAC;QAC7C,aAAa,CAAC,GAAG,CAAC,EAAE,CAAC,SAAS,EAAE,EAAE,CAAC,QAAQ,CAAC,CAAC;IAC/C,CAAC;IAED,MAAM,aAAa,GAAG,kBAAkB,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE;QACjD,MAAM,GAAG,GAAG,eAAe,CAAC,KAAK,EAAE,CAAC,CAAC,CAAC;QACtC,MAAM,QAAQ,GAAG,CAAC,CAAC,QAAQ;YACzB,CAAC,CAAC,IAAI,QAAQ,CAAC,CAAC,CAAC,MAAM,CAAC,mBAAmB,QAAQ,CAAC,CAAC,CAAC,QAAQ,CAAC,GAC3D,CAAC,CAAC,UAAU,KAAK,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,UAAU,EAAE,CAAC,CAAC,CAAC,EACpD,IAAI;YACN,CAAC,CAAC,IAAI,QAAQ,CAAC,CAAC,CAAC,MAAM,CAAC,oBAAoB,CAAC;QAC/C,MAAM,IAAI,GAAG,GAAG,CAAC,CAAC,CAAC,IAAI,QAAQ,KAAK,GAAG,GAAG,CAAC,CAAC,CAAC,QAAQ,CAAC;QACtD,MAAM,QAAQ,GAAG,aAAa,CAAC,GAAG,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC;QAChD,oEAAoE;QACpE,sEAAsE;QACtE,oEAAoE;QACpE,wEAAwE;QACxE,uEAAuE;QACvE,uDAAuD;QACvD,OAAO,QAAQ,CAAC,CAAC,CAAC,KAAK,IAAI,MAAM,QAAQ,EAAE,CAAC,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC;IAC5D,CAAC,CAAC,CAAC;IAEH,2EAA2E;IAC3E,2BAA2B;IAC3B,MAAM,cAAc,GAClB,KAAK,CAAC,OAAO,CAAC,GAAG,CAAC,iBAAiB,CAAC;QACpC,GAAG,CAAC,iBAAiB,CAAC,MAAM,GAAG,CAAC;QAChC,kBAAkB,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,aAAa,CAAC,GAAG,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC;IAEjE,MAAM,YAAY,GAChB,kBAAkB,CAAC,MAAM,GAAG,CAAC,IAAI,CAAC,cAAc;QAC9C,CAAC,CAAC,mDAAmD;QACrD,CAAC,CAAC,EAAE,CAAC;IAET,MAAM,iBAAiB,GACrB,kBAAkB,CAAC,MAAM,GAAG,CAAC;QAC3B,CAAC,CAAC,0BAA0B,aAAa,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,YAAY,IAAI;QACvE,CAAC,CAAC,EAAE,CAAC;IAET,gFAAgF;IAChF,MAAM,YAAY,GAChB,OAAO,CAAC,YAAY,CAAC,MAAM,GAAG,CAAC;QAC7B,CAAC,CAAC,yBAAyB,OAAO,CAAC,YAAY,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI;QACvF,CAAC,CAAC,EAAE,CAAC;IAET,MAAM,OAAO,GAAG,gBAAgB;QAC9B,CAAC,CAAC,IAAI,GAAG,CAAC,EAAE,KAAK,gBAAgB,SAAS,GAAG,CAAC,EAAE,GAAG;QACnD,CAAC,CAAC,GAAG,CAAC,EAAE,CAAC;IACX,MAAM,MAAM,GAAG,cAAc,OAAO,yBAAyB,CAAC;IAE9D,OAAO,kCAAkC,OAAO,KAAK,iBAAiB,GAAG,YAAY,GAAG,MAAM,EAAE,CAAC;AACnG,CAAC;AA2BD,MAAM,CAAC,KAAK,UAAU,sBAAsB,CAC1C,IAAgC;IAEhC,MAAM,EACJ,GAAG,EACH,OAAO,EACP,QAAQ,EACR,KAAK,EACL,IAAI,EACJ,OAAO,EACP,aAAa,EACb,gBAAgB,EAChB,MAAM,GACP,GAAG,IAAI,CAAC;IAET,IAAI,GAAG,CAAC,OAAO,KAAK,iBAAiB;QAAE,OAAO;IAE9C,IAAI,CAAC,GAAG,CAAC,KAAK,IAAI,QAAQ,KAAK,IAAI,EAAE,CAAC;QACpC,MAAM,CAAC,IAAI,CACT,EAAE,KAAK,EAAE,GAAG,CAAC,EAAE,EAAE,EACjB,0CAA0C,CAC3C,CAAC;QACF,OAAO;IACT,CAAC;IAED,IAAI,CAAC,OAAO,EAAE,CAAC;QACb,MAAM,CAAC,IAAI,CACT,EAAE,KAAK,EAAE,GAAG,CAAC,EAAE,EAAE,EACjB,8CAA8C,CAC/C,CAAC;QACF,OAAO;IACT,CAAC;IAED,IAAI,kBAAkB,GAA4B,EAAE,CAAC;IACrD,IAAI,GAAG,CAAC,eAAe,EAAE,CAAC;QACxB,IAAI,CAAC;YACH,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,eAAe,CAAY,CAAC;YAC1D,IAAI,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC;gBAAE,kBAAkB,GAAG,MAAiC,CAAC;QACpF,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,MAAM,CAAC,KAAK,CACV;gBACE,KAAK,EAAE,GAAG,CAAC,EAAE;gBACb,GAAG,EAAE,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC;aACtD,EACD,oEAAoE,CACrE,CAAC;QACJ,CAAC;IACH,CAAC;IAED,IAAI,IAAY,CAAC;IACjB,IAAI,CAAC;QACH,IAAI,GAAG,mBAAmB,CAAC;YACzB,OAAO;YACP,GAAG,EAAE;gBACH,EAAE,EAAE,GAAG,CAAC,EAAE;gBACV,gBAAgB,EAAE,GAAG,CAAC,gBAAgB;gBACtC,iBAAiB,EAAE,GAAG,CAAC,iBAAiB;aACzC;YACD,kBAAkB;YAClB,gBAAgB;YAChB,KAAK,EAAE,GAAG,CAAC,KAAK;SACjB,CAAC,CAAC;IACL,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,MAAM,CAAC,KAAK,CACV,EAAE,KAAK,EAAE,GAAG,CAAC,EAAE,EAAE,GAAG,EAAE,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,EAAE,EACxE,iDAAiD,CAClD,CAAC;QACF,OAAO;IACT,CAAC;IAED,IAAI,CAAC;QACH,MAAM,aAAa,CAAC,OAAO,EAAE,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,IAAI,CAAC,CAAC;QAC1D,MAAM,CAAC,IAAI,CACT,EAAE,KAAK,EAAE,GAAG,CAAC,EAAE,EAAE,QAAQ,EAAE,EAC3B,kDAAkD,CACnD,CAAC;IACJ,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,MAAM,CAAC,IAAI,CACT,EAAE,KAAK,EAAE,GAAG,CAAC,EAAE,EAAE,QAAQ,EAAE,GAAG,EAAE,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,EAAE,EAClF,2CAA2C,CAC5C,CAAC;IACJ,CAAC;AACH,CAAC"}

package/dist/pipeline/runner.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"runner.d.ts","sourceRoot":"","sources":["../../src/pipeline/runner.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,cAAc,EACd,UAAU,EAEV,QAAQ,EAGR,cAAc,EAId,qBAAqB,EACtB,MAAM,aAAa,CAAC;AACrB,OAAO,KAAK,EAAE,EAAE,EAAS,MAAM,iBAAiB,CAAC;AA8CjD,OAAO,EAML,KAAK,YAAY,EAClB,MAAM,mBAAmB,CAAC;AAC3B,OAAO,EAGL,KAAK,YAAY,EAClB,MAAM,mBAAmB,CAAC;AAE3B,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,8BAA8B,CAAC;AAgC1E;;;;;;;;;;;;;GAaG;AACH,wBAAgB,0BAA0B,CACxC,KAAK,EAAE,MAAM,EACb,QAAQ,EAAE,MAAM,EAChB,QAAQ,EAAE,qBAAqB,EAAE,GAChC,qBAAqB,CAYvB;AAED,MAAM,WAAW,oBAAoB;IACnC,EAAE,EAAE,EAAE,CAAC;IACP,QAAQ,EAAE,QAAQ,CAAC;IACnB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,CAAC,EAAE,YAAY,CAAC;IACtB,MAAM,CAAC,EAAE,YAAY,CAAC;IACtB;;;OAGG;IACH,eAAe,CAAC,EAAE,MAAM,CAAC;CAC1B;AAGD,MAAM,WAAW,WAAW;IAC1B,EAAE,EAAE,MAAM,CAAC;IACX,UAAU,EAAE,MAAM,CAAC;IACnB,KAAK,EAAE,MAAM,CAAC;IACd,WAAW,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;IAChC,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;IACf,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED,qBAAa,cAAc;IACzB,OAAO,CAAC,KAAK,CAAY;IACzB;;2EAEuE;IACvE,OAAO,CAAC,SAAS,CAAY;IAC7B,OAAO,CAAC,EAAE,CAAK;IACf,OAAO,CAAC,QAAQ,CAAW;IAC3B,OAAO,CAAC,UAAU,CAA6B;IAC/C,OAAO,CAAC,iBAAiB,CAAqB;IAC9C,4EAA4E;IAC5E,OAAO,CAAC,kBAAkB,CAA6B;IACvD,OAAO,CAAC,WAAW,CAAS;IAC5B,OAAO,CAAC,YAAY,CAAS;IAC7B,OAAO,CAAC,YAAY,CAAC,CAAe;IACpC,OAAO,CAAC,YAAY,CAAC,CAAe;IACpC,OAAO,CAAC,WAAW,CAAc;IACjC,OAAO,CAAC,eAAe,CAAS;gBAEpB,MAAM,EAAE,oBAAoB;IAalC,KAAK,CACT,KAAK,EAAE,WAAW,EAClB,WAAW,EAAE,MAAM,EACnB,cAAc,EAAE,cAAc,EAC9B,UAAU,EAAE,UAAU,EACtB,cAAc,EAAE,cAAc,EAC9B,YAAY,GAAE,MAAM,GAAG,IAAW,GACjC,OAAO,CAAC,IAAI,CAAC;IA4EV,MAAM,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IA0MtC,KAAK,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAUrC,KAAK,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAY3C,QAAQ,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO;IAIlC,yFAAyF;IACzF,gBAAgB,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO;IAIxC;;;;;;;;OAQG;IACG,aAAa,CAAC,MAAM,EAAE;QAC1B,KAAK,EAAE,WAAW,CAAC;QACnB,WAAW,EAAE,MAAM,CAAC;QACpB,cAAc,EAAE,cAAc,CAAC;QAC/B,UAAU,EAAE,UAAU,CAAC;QACvB,cAAc,EAAE,cAAc,CAAC;QAC/B,MAAM,EAAE,MAAM,CAAC;QACf,KAAK,EAAE,MAAM,CAAC;QACd,QAAQ,CAAC,EAAE,MAAM,CAAC;QAClB,WAAW,CAAC,EAAE,MAAM,CAAC;QACrB,gBAAgB,EAAE,qBAAqB,EAAE,CAAC;QAC1C,eAAe,CAAC,EAAE,OAAO,CAAC;KAC3B,GAAG,OAAO,CAAC,IAAI,CAAC;IAoGjB;;;;OAIG;YACW,iBAAiB;YAajB,eAAe;YAorDf,YAAY;IAiF1B;;;OAGG;YACW,gBAAgB;IAoC9B;;;;;;;;;;;;;;;;;OAiBG;IACG,gBAAgB,IAAI,OAAO,CAAC,IAAI,CAAC;IAyEvC;;;;OAIG;YACW,oBAAoB;IAIlC;;;OAGG;IACG,qBAAqB,CAAC,IAAI,CAAC,EAAE,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC;IAqCvD;;;;;;;;;OASG;YACW,uBAAuB;IA+TrC,OAAO,CAAC,gBAAgB;CAoBzB"}
+{"version":3,"file":"runner.d.ts","sourceRoot":"","sources":["../../src/pipeline/runner.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EACV,cAAc,EACd,UAAU,EAEV,QAAQ,EAGR,cAAc,EAId,qBAAqB,EACtB,MAAM,aAAa,CAAC;AACrB,OAAO,KAAK,EAAE,EAAE,EAAS,MAAM,iBAAiB,CAAC;AA+CjD,OAAO,EAML,KAAK,YAAY,EAClB,MAAM,mBAAmB,CAAC;AAC3B,OAAO,EAGL,KAAK,YAAY,EAClB,MAAM,mBAAmB,CAAC;AAE3B,OAAO,KAAK,EAAE,qBAAqB,EAAE,MAAM,8BAA8B,CAAC;AAgC1E;;;;;;;;;;;;;GAaG;AACH,wBAAgB,0BAA0B,CACxC,KAAK,EAAE,MAAM,EACb,QAAQ,EAAE,MAAM,EAChB,QAAQ,EAAE,qBAAqB,EAAE,GAChC,qBAAqB,CAYvB;AAED,MAAM,WAAW,oBAAoB;IACnC,EAAE,EAAE,EAAE,CAAC;IACP,QAAQ,EAAE,QAAQ,CAAC;IACnB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,MAAM,CAAC,EAAE,YAAY,CAAC;IACtB,MAAM,CAAC,EAAE,YAAY,CAAC;IACtB;;;OAGG;IACH,eAAe,CAAC,EAAE,MAAM,CAAC;CAC1B;AAGD,MAAM,WAAW,WAAW;IAC1B,EAAE,EAAE,MAAM,CAAC;IACX,UAAU,EAAE,MAAM,CAAC;IACnB,KAAK,EAAE,MAAM,CAAC;IACd,WAAW,EAAE,MAAM,CAAC;IACpB,MAAM,EAAE,KAAK,CAAC;QAAE,IAAI,EAAE,MAAM,CAAA;KAAE,CAAC,CAAC;IAChC,QAAQ,EAAE,MAAM,CAAC;IACjB,MAAM,EAAE,MAAM,CAAC;IACf,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB;AAED,qBAAa,cAAc;IACzB,OAAO,CAAC,KAAK,CAAY;IACzB;;2EAEuE;IACvE,OAAO,CAAC,SAAS,CAAY;IAC7B,OAAO,CAAC,EAAE,CAAK;IACf,OAAO,CAAC,QAAQ,CAAW;IAC3B,OAAO,CAAC,UAAU,CAA6B;IAC/C,OAAO,CAAC,iBAAiB,CAAqB;IAC9C,4EAA4E;IAC5E,OAAO,CAAC,kBAAkB,CAA6B;IACvD,OAAO,CAAC,WAAW,CAAS;IAC5B,OAAO,CAAC,YAAY,CAAS;IAC7B,OAAO,CAAC,YAAY,CAAC,CAAe;IACpC,OAAO,CAAC,YAAY,CAAC,CAAe;IACpC,OAAO,CAAC,WAAW,CAAc;IACjC,OAAO,CAAC,eAAe,CAAS;gBAEpB,MAAM,EAAE,oBAAoB;IAalC,KAAK,CACT,KAAK,EAAE,WAAW,EAClB,WAAW,EAAE,MAAM,EACnB,cAAc,EAAE,cAAc,EAC9B,UAAU,EAAE,UAAU,EACtB,cAAc,EAAE,cAAc,EAC9B,YAAY,GAAE,MAAM,GAAG,IAAW,GACjC,OAAO,CAAC,IAAI,CAAC;IA4EV,MAAM,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IA0MtC,KAAK,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAUrC,KAAK,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAY3C,QAAQ,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO;IAIlC,yFAAyF;IACzF,gBAAgB,CAAC,KAAK,EAAE,MAAM,GAAG,OAAO;IAIxC;;;;;;;;OAQG;IACG,aAAa,CAAC,MAAM,EAAE;QAC1B,KAAK,EAAE,WAAW,CAAC;QACnB,WAAW,EAAE,MAAM,CAAC;QACpB,cAAc,EAAE,cAAc,CAAC;QAC/B,UAAU,EAAE,UAAU,CAAC;QACvB,cAAc,EAAE,cAAc,CAAC;QAC/B,MAAM,EAAE,MAAM,CAAC;QACf,KAAK,EAAE,MAAM,CAAC;QACd,QAAQ,CAAC,EAAE,MAAM,CAAC;QAClB,WAAW,CAAC,EAAE,MAAM,CAAC;QACrB,gBAAgB,EAAE,qBAAqB,EAAE,CAAC;QAC1C,eAAe,CAAC,EAAE,OAAO,CAAC;KAC3B,GAAG,OAAO,CAAC,IAAI,CAAC;IAqGjB;;;;OAIG;YACW,iBAAiB;YAajB,eAAe;YA8tDf,YAAY;IAiF1B;;;OAGG;YACW,gBAAgB;IAoC9B;;;;;;;;;;;;;;;;;OAiBG;IACG,gBAAgB,IAAI,OAAO,CAAC,IAAI,CAAC;IAyEvC;;;;OAIG;YACW,oBAAoB;IAIlC;;;OAGG;IACG,qBAAqB,CAAC,IAAI,CAAC,EAAE,IAAI,GAAG,OAAO,CAAC,IAAI,CAAC;IAqCvD;;;;;;;;;OASG;YACW,uBAAuB;IA+TrC,OAAO,CAAC,gBAAgB;CAoBzB"}

package/dist/pipeline/runner.js

@@ -12,6 +12,7 @@ import { postFanoutCommentsToPR } from "../executor/review/post-fanout-comments.
 import { extractHandoff } from "../executor/extract-handoff.js";
 import { DEFAULT_AGENT_CLAUDE_MD } from "../executor/agent-config.js";
 import { generatePRDescription } from "./pr-description.js";
+import { maybePostChangeSummary } from "./pr-change-summary.js";
 import { access, writeFile, appendFile } from "node:fs/promises";
 import { join, resolve } from "node:path";
 import { homedir } from "node:os";
@@ -388,6 +389,7 @@ export class PipelineRunner {
         const run = this.buildPipelineRun(runId, issue, pipelineKey, repoConfig, branch);
         run.prUrl = prUrl;
         run.runType = "review-feedback";
+        run.feedbackContext = JSON.stringify(feedbackComments);
         // Register in activeFeedbackRuns BEFORE enqueue so rate-limit check works
         this.activeFeedbackRuns.set(prUrl, runId);
         this.queue
@@ -1736,6 +1738,43 @@ export class PipelineRunner {
                     runLog.warn({ err: err instanceof Error ? err.message : String(err) }, "BEC-175: PR cost summary post failed (non-fatal)");
                 }
             }
+            // PR change-summary comment for review-feedback runs. Always-on (no env
+            // flag) — a review-feedback run only exists because a human asked for
+            // changes, so silent shipping is a bug.
+            if (run.runType === "review-feedback" &&
+                prUrl &&
+                repoConfig.provider !== "gitlab" &&
+                this.githubConfig) {
+                try {
+                    const summaryPrMatch = prUrl.match(/\/pull\/(\d+)/);
+                    const summaryPrNumber = summaryPrMatch
+                        ? parseInt(summaryPrMatch[1], 10)
+                        : null;
+                    const { owner: csOwner, repo: csRepo } = parseRepoUrl(repoConfig.url);
+                    const csOctokit = await createGitHubClient(this.githubConfig);
+                    await maybePostChangeSummary({
+                        run: {
+                            id: run.id,
+                            runType: run.runType,
+                            prUrl: run.prUrl,
+                            feedbackContext: run.feedbackContext ?? null,
+                            totalInputTokens: run.totalInputTokens,
+                            totalOutputTokens: run.totalOutputTokens,
+                        },
+                        handoff: handoff ?? null,
+                        prNumber: summaryPrNumber,
+                        owner: csOwner,
+                        repo: csRepo,
+                        octokit: csOctokit,
+                        postPRComment: addPRComment,
+                        dashboardBaseUrl: process.env.URATEAM_DASHBOARD_URL ?? "",
+                        logger: runLog,
+                    });
+                }
+                catch (err) {
+                    runLog.warn({ err: err instanceof Error ? err.message : String(err) }, "PR change summary post failed (non-fatal)");
+                }
+            }
         }
         catch (error) {
             // If failPipeline was already called inside the push queue (e.g. failOnAutoCommit