patchrelay 0.36.5 → 0.36.7

package/dist/run-orchestrator.js

@@ -1,6 +1,4 @@
 import { randomUUID } from "node:crypto";
-import { existsSync, readFileSync } from "node:fs";
-import path from "node:path";
 import { ACTIVE_RUN_STATES, TERMINAL_STATES } from "./factory-state.js";
 import { buildHookEnv, runProjectHook } from "./hook-runner.js";
 import { buildStageReport, countEventMethods, extractTurnId, resolveRunCompletionStatus, summarizeCurrentThread, } from "./run-reporting.js";
@@ -10,9 +8,16 @@ import { resolveAuthoritativeLinearStopState, resolvePreferredCompletedLinearSta
 import { execCommand } from "./utils.js";
 import { getThreadTurns } from "./codex-thread-utils.js";
 import { deriveIssueSessionReactiveIntent } from "./issue-session.js";
+import { loadPatchRelayRepoPrompting } from "./patchrelay-customization.js";
+import { buildRunPrompt as buildPatchRelayRunPrompt, findDisallowedPatchRelayPromptSectionIds, findUnknownPatchRelayPromptSectionIds, mergePromptCustomizationLayers, resolveImplementationDeliveryMode, resolvePromptLayers, } from "./prompting/patchrelay.js";
 const DEFAULT_CI_REPAIR_BUDGET = 3;
 const DEFAULT_QUEUE_REPAIR_BUDGET = 3;
-const DEFAULT_REVIEW_FIX_BUDGET = 6;
+// Requested-changes loops can legitimately take more iterations than CI/queue
+// repair when the reviewer is catching nuanced product or timing bugs across
+// successive heads. Keep a hard ceiling to prevent infinite ping-pong, but make
+// it wide enough that real review cycles can continue after multiple successful
+// head advances.
+const DEFAULT_REVIEW_FIX_BUDGET = 12;
 const DEFAULT_ZOMBIE_RECOVERY_BUDGET = 5;
 const ZOMBIE_RECOVERY_BASE_DELAY_MS = 15_000; // 15s, 30s, 60s, 120s, 240s
 const ISSUE_SESSION_LEASE_MS = 10 * 60_000;
@@ -30,156 +35,6 @@ function sanitizePathSegment(value) {
 function lowerCaseFirst(value) {
     return value ? `${value.slice(0, 1).toLowerCase()}${value.slice(1)}` : value;
 }
-const WORKFLOW_FILES = {
-    implementation: "IMPLEMENTATION_WORKFLOW.md",
-    review_fix: "REVIEW_WORKFLOW.md",
-    branch_upkeep: "REVIEW_WORKFLOW.md",
-    ci_repair: "IMPLEMENTATION_WORKFLOW.md",
-    queue_repair: "IMPLEMENTATION_WORKFLOW.md",
-};
-function readWorkflowFile(repoPath, runType) {
-    const filename = WORKFLOW_FILES[runType];
-    const filePath = path.join(repoPath, filename);
-    if (!existsSync(filePath))
-        return undefined;
-    return readFileSync(filePath, "utf8").trim();
-}
-function collectImplementationInstructionText(issue, context, promptText) {
-    const parts = [];
-    if (issue.title)
-        parts.push(issue.title);
-    if (issue.description)
-        parts.push(issue.description);
-    if (promptText)
-        parts.push(promptText);
-    const stringFields = ["promptContext", "promptBody", "operatorPrompt", "userComment"];
-    for (const field of stringFields) {
-        const value = context?.[field];
-        if (typeof value === "string" && value.trim()) {
-            parts.push(value);
-        }
-    }
-    if (Array.isArray(context?.followUps)) {
-        for (const entry of context.followUps) {
-            if (!entry || typeof entry !== "object")
-                continue;
-            const text = entry.text;
-            if (typeof text === "string" && text.trim()) {
-                parts.push(text);
-            }
-        }
-    }
-    return parts.join("\n").toLowerCase();
-}
-export function resolveImplementationDeliveryMode(issue, context, promptText) {
-    const instructionText = collectImplementationInstructionText(issue, context, promptText);
-    if (!instructionText)
-        return "publish_pr";
-    const hasExplicitNoPr = [
-        /\bdo not open (?:a |any )?pr\b/,
-        /\bdo not open (?:a |any )?pull request\b/,
-        /\bno pr is opened\b/,
-        /\bpatchrelay should not open a pr\b/,
-        /\bwithout opening a pr\b/,
-    ].some((pattern) => pattern.test(instructionText));
-    const forbidsRepoChanges = [
-        /\bdo not make repository changes\b/,
-        /\bdo not make repo changes\b/,
-        /\bno repository changes\b/,
-        /\bno repo changes\b/,
-        /\bdo not modify repo files\b/,
-    ].some((pattern) => pattern.test(instructionText));
-    const planningOnly = [
-        /\bplanning\/specification issue only\b/,
-        /\bplanning[- ]only\b/,
-        /\bspecification[- ]only\b/,
-        /\bplanning issue only\b/,
-    ].some((pattern) => pattern.test(instructionText));
-    if (hasExplicitNoPr || (planningOnly && forbidsRepoChanges)) {
-        return "linear_only";
-    }
-    return "publish_pr";
-}
-function appendPublicationContract(lines, runType, issue, context) {
-    const deliveryMode = runType === "implementation" && issue
-        ? resolveImplementationDeliveryMode(issue, context)
-        : "publish_pr";
-    if (runType === "implementation" && deliveryMode === "linear_only") {
-        lines.push("## Delivery Requirements", "");
-        lines.push("This issue is planning/specification only.", "Do not modify repo files or open a PR for this issue.", "Deliver the result through Linear artifacts such as follow-up issues, documents, and a concise summary.", "Leave the worktree clean before stopping.", "");
-        return;
-    }
-    lines.push("## Publication Requirements", "");
-    if (runType === "implementation") {
-        lines.push("Before finishing, publish the result instead of leaving it only in the worktree.", "If the worktree already contains relevant changes for this issue, verify them and publish them.", "If you changed files for this issue, commit them, push the issue branch, and open or update the PR before stopping.", "Do not stop with only local commits or uncommitted changes.", "");
-        return;
-    }
-    lines.push("Before finishing, publish the result to the existing PR branch.", "If you changed files for this repair, commit them and push the same branch before stopping.", "Do not open a new PR.", "Do not stop with only local commits or uncommitted changes.", "");
-}
-function buildPromptHeader(issue) {
-    return [
-        `Issue: ${issue.issueKey ?? issue.linearIssueId}`,
-        issue.title ? `Title: ${issue.title}` : undefined,
-        issue.branchName ? `Branch: ${issue.branchName}` : undefined,
-        issue.prNumber ? `PR: #${issue.prNumber}` : undefined,
-        "",
-    ].filter(Boolean);
-}
-function appendTaskObjective(lines, issue) {
-    const description = issue.description?.trim();
-    lines.push("## Task Objective", "");
-    lines.push(issue.title || `Complete ${issue.issueKey ?? issue.linearIssueId}.`);
-    if (description) {
-        lines.push("", description);
-    }
-    lines.push("");
-}
-function extractIssueSection(description, heading) {
-    if (!description)
-        return undefined;
-    const escaped = heading.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
-    const pattern = new RegExp(`^## ${escaped}\\s*$([\\s\\S]*?)(?=^##\\s+|$)`, "im");
-    const match = description.match(pattern);
-    const body = match?.[1]?.trim();
-    return body && body.length > 0 ? body : undefined;
-}
-function appendScopeDiscipline(lines, issue) {
-    const description = issue.description?.trim();
-    const scope = extractIssueSection(description, "Scope");
-    const acceptance = extractIssueSection(description, "Acceptance criteria")
-        ?? extractIssueSection(description, "Success criteria");
-    const relevantCode = extractIssueSection(description, "Relevant code");
-    lines.push("## Scope Discipline", "");
-    lines.push("Stay inside the delegated task.", "Finish the issue completely enough to satisfy its stated scope and acceptance criteria, but do not widen it into unrelated product polish or follow-up cleanup.", "Only broaden to adjacent routes, copy, or supporting surfaces when the issue text or repository guidance explicitly says they are the same user flow.", "If you notice a worthwhile broader inconsistency that is not required to make this task correct, mention it in your summary as follow-up context instead of expanding the implementation.", "");
-    if (scope) {
-        lines.push("### In Scope", "", scope, "");
-    }
-    if (acceptance) {
-        lines.push("### Acceptance / Done", "", acceptance, "");
-    }
-    if (relevantCode) {
-        lines.push("### Relevant Code", "", relevantCode, "");
-    }
-    lines.push("### Likely Review Invariants", "", "- Check the surfaces explicitly named in the task before stopping.", "- If repository guidance says certain changed surfaces are one flow, verify that shared flow, but do not treat unrelated surrounding cleanup as part of this task.", "- A review repair should fix the concrete concern on the current head, not silently expand the Linear issue into a broader rewrite.", "");
-}
-function appendLinearContext(lines, context) {
-    const promptContext = typeof context?.promptContext === "string" ? context.promptContext.trim() : "";
-    const latestPrompt = typeof context?.promptBody === "string" ? context.promptBody.trim() : "";
-    const operatorPrompt = typeof context?.operatorPrompt === "string" ? context.operatorPrompt.trim() : "";
-    const userComment = typeof context?.userComment === "string" ? context.userComment.trim() : "";
-    if (promptContext) {
-        lines.push("## Linear Session Context", "", promptContext, "");
-    }
-    if (latestPrompt) {
-        lines.push("## Latest Human Instruction", "", latestPrompt, "");
-    }
-    if (operatorPrompt) {
-        lines.push("## Operator Prompt", "", operatorPrompt, "");
-    }
-    if (userComment) {
-        lines.push("## Human Follow-up Comment", "", userComment, "");
-    }
-}
 function isRequestedChangesRunType(runType) {
     return runType === "review_fix" || runType === "branch_upkeep";
 }
@@ -191,332 +46,6 @@ function resolveRequestedChangesMode(runType, context) {
         ? "branch_upkeep"
         : "address_review_feedback";
 }
-function readReviewFixComments(context) {
-    const raw = context?.reviewComments;
-    if (!Array.isArray(raw)) {
-        return [];
-    }
-    const comments = [];
-    for (const entry of raw) {
-        if (!entry || typeof entry !== "object")
-            continue;
-        const record = entry;
-        const body = typeof record.body === "string" ? record.body.trim() : "";
-        if (!body)
-            continue;
-        comments.push({
-            body,
-            ...(typeof record.path === "string" ? { path: record.path } : {}),
-            ...(typeof record.line === "number" ? { line: record.line } : {}),
-            ...(typeof record.side === "string" ? { side: record.side } : {}),
-            ...(typeof record.startLine === "number" ? { startLine: record.startLine } : {}),
-            ...(typeof record.startSide === "string" ? { startSide: record.startSide } : {}),
-            ...(typeof record.url === "string" ? { url: record.url } : {}),
-            ...(typeof record.authorLogin === "string" ? { authorLogin: record.authorLogin } : {}),
-        });
-    }
-    return comments;
-}
-function appendStructuredReviewContext(lines, context) {
-    const reviewId = typeof context?.reviewId === "number" ? context.reviewId : undefined;
-    const reviewCommitId = typeof context?.reviewCommitId === "string" ? context.reviewCommitId : undefined;
-    const reviewUrl = typeof context?.reviewUrl === "string" ? context.reviewUrl : undefined;
-    const reviewComments = readReviewFixComments(context);
-    if (!reviewId && !reviewCommitId && !reviewUrl && reviewComments.length === 0) {
-        return;
-    }
-    lines.push("## Structured Review Context", "");
-    if (reviewId !== undefined) {
-        lines.push(`Review ID: ${reviewId}`);
-    }
-    if (reviewCommitId) {
-        lines.push(`Reviewed commit: ${reviewCommitId}`);
-    }
-    if (reviewUrl) {
-        lines.push(`Review URL: ${reviewUrl}`);
-    }
-    if (reviewComments.length === 0) {
-        lines.push("No inline review comments were captured for this review.", "");
-        return;
-    }
-    lines.push(`Inline review comments captured: ${reviewComments.length}`, "Resolve each comment below or verify it is already fixed on the current head before you stop.", "A requested-changes turn is only complete if you push a newer PR head or deliberately escalate because you are blocked.", "");
-    for (const comment of reviewComments) {
-        const location = comment.path
-            ? `${comment.path}${comment.line !== undefined ? `:${comment.line}` : ""}${comment.side ? ` (${comment.side})` : ""}`
-            : "general";
-        lines.push(`- ${location}`);
-        lines.push(comment.body);
-        if (comment.url) {
-            lines.push(`  URL: ${comment.url}`);
-        }
-    }
-    lines.push("");
-}
-function collectFollowUpInputs(context) {
-    const followUps = Array.isArray(context?.followUps) ? context.followUps : [];
-    const inputs = [];
-    for (const entry of followUps) {
-        const followUp = entry && typeof entry === "object" ? entry : undefined;
-        const type = typeof followUp?.type === "string" ? followUp.type : "followup";
-        const author = typeof followUp?.author === "string" ? followUp.author : undefined;
-        const text = typeof followUp?.text === "string" ? followUp.text.trim() : "";
-        if (!text)
-            continue;
-        inputs.push({ type, text, ...(author ? { author } : {}) });
-    }
-    return inputs;
-}
-function resolveFollowUpWhy(runType, context) {
-    const wakeReason = typeof context?.wakeReason === "string" ? context.wakeReason : undefined;
-    switch (wakeReason) {
-        case "direct_reply":
-            return "A human reply arrived for the outstanding question from the previous turn.";
-        case "followup_prompt":
-            return "A new Linear agent prompt arrived after the previous turn.";
-        case "followup_comment":
-            return "A human follow-up comment arrived after the previous turn.";
-        case "operator_prompt":
-            return "An operator supplied new guidance for this issue.";
-        case "review_changes_requested":
-            return "GitHub review requested changes on the current PR head.";
-        case "branch_upkeep":
-            return "GitHub still shows the PR branch as needing upkeep after the requested code change was addressed.";
-        case "settled_red_ci":
-            return "Required CI settled red for the current PR head.";
-        case "merge_steward_incident":
-            return "Merge Steward reported an incident on the current PR head.";
-        case "delegated":
-            return runType === "implementation"
-                ? "This is the first implementation turn for the delegated issue."
-                : `This turn continues ${runType.replaceAll("_", " ")} work for the delegated issue.`;
-        default:
-            if (isRequestedChangesRunType(runType)) {
-                return resolveRequestedChangesMode(runType, context) === "branch_upkeep"
-                    ? "This turn continues branch upkeep on the existing PR after requested changes."
-                    : "This turn continues requested-changes work on the existing PR.";
-            }
-            if (runType === "ci_repair")
-                return "This turn continues CI repair work on the existing PR.";
-            if (runType === "queue_repair")
-                return "This turn continues merge-queue repair work on the existing PR.";
-            return "This turn continues implementation on the existing issue session.";
-    }
-}
-function resolveFollowUpAction(runType, context) {
-    if (context?.directReplyMode === true) {
-        return "Apply the latest human answer, continue from the current branch/session context, and only ask another question if you are still blocked.";
-    }
-    if (isRequestedChangesRunType(runType) && resolveRequestedChangesMode(runType, context) === "branch_upkeep") {
-        const baseBranch = typeof context?.baseBranch === "string" ? context.baseBranch : "main";
-        return `Update the existing PR branch onto latest ${baseBranch}, resolve conflicts if needed, rerun narrow verification, and push a newer head on the same branch.`;
-    }
-    switch (runType) {
-        case "review_fix":
-            return "Address the review feedback on the current PR branch, verify the fix, and push a newer head on the same branch.";
-        case "branch_upkeep":
-            return "Repair the existing PR branch after requested changes, rerun narrow verification, and push a newer head on the same branch.";
-        case "ci_repair":
-            return "Fix the failing CI root cause on the current PR branch, verify it locally, and push the same branch.";
-        case "queue_repair":
-            return "Repair the merge-queue incident on the current PR branch, verify the fix, and push the same branch.";
-        case "implementation":
-        default:
-            return "Continue from the latest branch state, incorporate the new input, and publish updates to the existing issue branch if you make changes.";
-    }
-}
-function hasAuthoritativeGitHubFacts(issue, runType, context) {
-    return issue.prNumber !== undefined
-        || issue.prHeadSha !== undefined
-        || runType !== "implementation"
-        || typeof context?.failureHeadSha === "string"
-        || typeof context?.failingHeadSha === "string"
-        || typeof context?.mergeStateStatus === "string"
-        || typeof context?.checkName === "string"
-        || typeof context?.reviewerName === "string";
-}
-function appendAuthoritativeGitHubFacts(lines, issue, runType, context) {
-    if (!hasAuthoritativeGitHubFacts(issue, runType, context)) {
-        return;
-    }
-    const prNumber = issue.prNumber !== undefined ? `#${issue.prNumber}` : undefined;
-    const headSha = typeof context?.failureHeadSha === "string"
-        ? context.failureHeadSha
-        : typeof context?.failingHeadSha === "string"
-            ? context.failingHeadSha
-            : issue.prHeadSha;
-    const mergeStateStatus = typeof context?.mergeStateStatus === "string" ? context.mergeStateStatus : undefined;
-    const baseBranch = typeof context?.baseBranch === "string" ? context.baseBranch : undefined;
-    const checkName = typeof context?.checkName === "string" ? context.checkName : undefined;
-    const jobName = typeof context?.jobName === "string" ? context.jobName : undefined;
-    const stepName = typeof context?.stepName === "string" ? context.stepName : undefined;
-    const reviewerName = typeof context?.reviewerName === "string" ? context.reviewerName : undefined;
-    const reviewBody = typeof context?.reviewBody === "string" ? context.reviewBody.trim() : "";
-    const summary = typeof context?.summary === "string" ? context.summary : undefined;
-    lines.push("## Authoritative GitHub Facts", "");
-    if (prNumber) {
-        lines.push(`- Current PR: ${prNumber}`);
-    }
-    if (headSha) {
-        lines.push(`- Current relevant head SHA: ${headSha}`);
-    }
-    if (issue.prReviewState) {
-        lines.push(`- Current review state: ${issue.prReviewState}`);
-    }
-    if (issue.prCheckStatus) {
-        lines.push(`- Current check status: ${issue.prCheckStatus}`);
-    }
-    if (mergeStateStatus) {
-        lines.push(`- Merge state against ${baseBranch ?? "base"}: ${mergeStateStatus}`);
-    }
-    if (checkName) {
-        lines.push(`- Relevant check: ${checkName}`);
-    }
-    if (jobName && jobName !== checkName) {
-        lines.push(`- Relevant job: ${jobName}`);
-    }
-    if (stepName) {
-        lines.push(`- Relevant step: ${stepName}`);
-    }
-    if (reviewerName) {
-        lines.push(`- Reviewer: ${reviewerName}`);
-    }
-    if (summary) {
-        lines.push(`- Summary: ${summary}`);
-    }
-    if (reviewBody) {
-        lines.push(`- Review body: ${reviewBody}`);
-    }
-    lines.push("");
-}
-function appendFactFreshness(lines, issue, runType, context) {
-    if (!hasAuthoritativeGitHubFacts(issue, runType, context)) {
-        return;
-    }
-    const hasFreshFacts = context?.githubFactsFresh === true || context?.branchUpkeepRequired === true;
-    lines.push("## Fact Freshness", "");
-    if (hasFreshFacts) {
-        lines.push("GitHub facts below were refreshed immediately before this turn was created.");
-    }
-    else {
-        lines.push("GitHub facts below came from the triggering event or last known reconciliation state and may now be stale.");
-        lines.push("Verify the current PR head, review state, and check state in GitHub before making branch-mutating decisions.");
-    }
-    lines.push("");
-}
-function appendFollowUpPromptPrelude(lines, issue, runType, context) {
-    lines.push("## Follow-up Turn", "");
-    lines.push(`Why this turn exists: ${resolveFollowUpWhy(runType, context)}`);
-    lines.push(`Required action now: ${resolveFollowUpAction(runType, context)}`);
-    lines.push("");
-    appendLinearContext(lines, context);
-    const followUps = collectFollowUpInputs(context);
-    if (followUps.length > 0) {
-        lines.push("## What Changed Since The Last Turn", "");
-        for (const followUp of followUps) {
-            lines.push(`- ${followUp.type}${followUp.author ? ` from ${followUp.author}` : ""}: ${followUp.text}`);
-        }
-        lines.push("");
-    }
-    appendFactFreshness(lines, issue, runType, context);
-    appendAuthoritativeGitHubFacts(lines, issue, runType, context);
-}
-function appendRequestedChangesInstructions(lines, runType, context) {
-    if (resolveRequestedChangesMode(runType, context) === "branch_upkeep") {
-        const baseBranch = typeof context?.baseBranch === "string" ? context.baseBranch : "main";
-        lines.push("## Branch Upkeep After Requested Changes", "", "The requested code change may already be present, but the PR branch still needs upkeep before review can continue.", typeof context?.mergeStateStatus === "string" ? `Current merge state: ${String(context.mergeStateStatus)}` : "", "", "Steps:", `1. Update the existing PR branch onto latest ${baseBranch}.`, "2. Resolve conflicts or branch drift without reopening the review-feedback debate unless the merge introduces a new issue.", "3. Run the narrowest verification that proves the branch is healthy again.", "4. Commit and push a newer head on the existing PR branch.", "5. If you cannot produce a new pushed head, stop and surface the exact blocker.", "");
-        return;
-    }
-    lines.push("## Review Changes Requested", "", "A reviewer has requested changes on your PR. Address the feedback and push.", context?.reviewerName ? `Reviewer: ${String(context.reviewerName)}` : "", context?.reviewBody ? `\n## Review comment\n\n${String(context.reviewBody)}` : "", "", "Steps:", "1. Start with the structured review context below. Treat the inline review comments as the primary repair checklist for this turn.", "2. Check the current diff (`git diff origin/main`) — a prior rebase may have already resolved some concerns (e.g., scope-bundling from stale commits).", "3. For each review point: if already resolved on the current head, note why. If not, fix it.", "4. If the structured review context looks incomplete, inspect the latest GitHub review threads directly before deciding you are done.", "5. Run verification, commit, and push a newer head on the existing PR branch.", "6. Do not try to hand the same head back to review. If you cannot produce a new pushed head, stop and surface the blocker clearly.", "7. GitHub review happens after the new head is pushed and CI is green. Do not use `gh pr edit --add-reviewer` as part of this workflow.", "");
-    appendStructuredReviewContext(lines, context);
-}
-export function buildInitialRunPrompt(issue, runType, repoPath, context) {
-    const lines = buildPromptHeader(issue);
-    appendTaskObjective(lines, issue);
-    appendScopeDiscipline(lines, issue);
-    appendLinearContext(lines, context);
-    // Add run-type-specific context for reactive runs
-    switch (runType) {
-        case "ci_repair": {
-            const snapshot = context?.ciSnapshot && typeof context.ciSnapshot === "object"
-                ? context.ciSnapshot
-                : undefined;
-            lines.push("## CI Repair", "", "A full CI iteration has settled failed on your PR. Start from the specific failing check/job/step below on the latest remote PR branch tip, fix that concrete failure first, then push to the same PR branch.", snapshot?.gateCheckName ? `Gate check: ${String(snapshot.gateCheckName)}` : "", snapshot?.gateCheckStatus ? `Gate status: ${String(snapshot.gateCheckStatus)}` : "", snapshot?.settledAt ? `Settled at: ${String(snapshot.settledAt)}` : "", context?.failureHeadSha ? `Failing head SHA: ${String(context.failureHeadSha)}` : "", context?.checkName ? `Failed check: ${String(context.checkName)}` : "", context?.jobName && context?.jobName !== context?.checkName ? `Failed job: ${String(context.jobName)}` : "", context?.stepName ? `Failed step: ${String(context.stepName)}` : "", context?.summary ? `Failure summary: ${String(context.summary)}` : "", Array.isArray(snapshot?.failedChecks) && snapshot.failedChecks.length > 0
-                ? `Other failed checks in the settled snapshot (context only; ignore unless the logs show the same root cause):\n${snapshot.failedChecks.map((entry) => `- ${String(entry.name ?? "unknown")}${entry.summary ? `: ${String(entry.summary)}` : ""}`).join("\n")}`
-                : "", context?.checkUrl ? `Check URL: ${String(context.checkUrl)}` : "", Array.isArray(context?.annotations) && context.annotations.length > 0
-                ? `Annotations:\n${context.annotations.map((entry) => `- ${String(entry)}`).join("\n")}`
-                : "", "", "Fetch the latest remote branch state first. If the branch moved since this failure, restart from the new tip instead of pushing older work.", "Read the latest logs for the named failing check, fix that root cause, and only broaden scope when the logs show direct fallout from the same issue.", "Do not change workflows, dependency installation, or unrelated tests unless the failing logs clearly point there.", "Run focused verification for the named failure, then commit and push.", "Do not open a new PR. Keep working on the existing branch until CI goes green or the situation is clearly stuck.", "Do not change test expectations unless the test is genuinely wrong.", "");
-            break;
-        }
-        case "review_fix":
-        case "branch_upkeep":
-            appendRequestedChangesInstructions(lines, runType, context);
-            break;
-        case "queue_repair":
-            appendQueueRepairContext(lines, context);
-            lines.push("## Merge Queue Failure", "", "The merge queue rejected this PR. Rebase onto latest main and fix conflicts.", context?.failureReason ? `Failure reason: ${String(context.failureReason)}` : "", "", "Fetch and rebase onto latest main, resolve conflicts, run verification, push.", "If the conflict is a semantic contradiction, explain and stop.", "");
-            break;
-    }
-    const workflowBody = readWorkflowFile(repoPath, runType);
-    if (workflowBody) {
-        lines.push(workflowBody);
-    }
-    else if (runType === "implementation") {
-        lines.push("Implement the Linear issue. Read the issue via MCP for details.");
-    }
-    appendPublicationContract(lines, runType, issue, context);
-    return lines.join("\n");
-}
-export function buildFollowUpRunPrompt(issue, runType, repoPath, context) {
-    const lines = buildPromptHeader(issue);
-    appendFollowUpPromptPrelude(lines, issue, runType, context);
-    appendTaskObjective(lines, issue);
-    appendScopeDiscipline(lines, issue);
-    // Add run-type-specific context for reactive runs
-    switch (runType) {
-        case "ci_repair": {
-            const snapshot = context?.ciSnapshot && typeof context.ciSnapshot === "object"
-                ? context.ciSnapshot
-                : undefined;
-            lines.push("## CI Repair", "", "A full CI iteration has settled failed on your PR. Start from the specific failing check/job/step below on the latest remote PR branch tip, fix that concrete failure first, then push to the same PR branch.", snapshot?.gateCheckName ? `Gate check: ${String(snapshot.gateCheckName)}` : "", snapshot?.gateCheckStatus ? `Gate status: ${String(snapshot.gateCheckStatus)}` : "", snapshot?.settledAt ? `Settled at: ${String(snapshot.settledAt)}` : "", context?.failureHeadSha ? `Failing head SHA: ${String(context.failureHeadSha)}` : "", context?.checkName ? `Failed check: ${String(context.checkName)}` : "", context?.jobName && context?.jobName !== context?.checkName ? `Failed job: ${String(context.jobName)}` : "", context?.stepName ? `Failed step: ${String(context.stepName)}` : "", context?.summary ? `Failure summary: ${String(context.summary)}` : "", Array.isArray(snapshot?.failedChecks) && snapshot.failedChecks.length > 0
-                ? `Other failed checks in the settled snapshot (context only; ignore unless the logs show the same root cause):\n${snapshot.failedChecks.map((entry) => `- ${String(entry.name ?? "unknown")}${entry.summary ? `: ${String(entry.summary)}` : ""}`).join("\n")}`
-                : "", context?.checkUrl ? `Check URL: ${String(context.checkUrl)}` : "", Array.isArray(context?.annotations) && context.annotations.length > 0
-                ? `Annotations:\n${context.annotations.map((entry) => `- ${String(entry)}`).join("\n")}`
-                : "", "", "Fetch the latest remote branch state first. If the branch moved since this failure, restart from the new tip instead of pushing older work.", "Read the latest logs for the named failing check, fix that root cause, and only broaden scope when the logs show direct fallout from the same issue.", "Do not change workflows, dependency installation, or unrelated tests unless the failing logs clearly point there.", "Run focused verification for the named failure, then commit and push.", "Do not open a new PR. Keep working on the existing branch until CI goes green or the situation is clearly stuck.", "Do not change test expectations unless the test is genuinely wrong.", "");
-            break;
-        }
-        case "review_fix":
-        case "branch_upkeep":
-            appendRequestedChangesInstructions(lines, runType, context);
-            break;
-        case "queue_repair":
-            appendQueueRepairContext(lines, context);
-            lines.push("## Merge Queue Failure", "", "The merge queue rejected this PR. Rebase onto latest main and fix conflicts.", context?.failureReason ? `Failure reason: ${String(context.failureReason)}` : "", "", "Fetch and rebase onto latest main, resolve conflicts, run verification, push.", "If the conflict is a semantic contradiction, explain and stop.", "");
-            break;
-    }
-    const workflowBody = readWorkflowFile(repoPath, runType);
-    if (workflowBody) {
-        lines.push(workflowBody);
-    }
-    else if (runType === "implementation") {
-        lines.push("Implement the Linear issue. Read the issue via MCP for details.");
-    }
-    appendPublicationContract(lines, runType, issue, context);
-    return lines.join("\n");
-}
-function shouldBuildFollowUpPrompt(runType, context) {
-    if (context?.followUpMode)
-        return true;
-    if (runType !== "implementation")
-        return true;
-    const wakeReason = typeof context?.wakeReason === "string" ? context.wakeReason : undefined;
-    return Boolean(wakeReason && wakeReason !== "delegated");
-}
-export function buildRunPrompt(issue, runType, repoPath, context) {
-    if (shouldBuildFollowUpPrompt(runType, context)) {
-        return buildFollowUpRunPrompt(issue, runType, repoPath, context);
-    }
-    return buildInitialRunPrompt(issue, runType, repoPath, context);
-}
 function shouldCompactThread(issue, threadGeneration, context) {
     const followUpCount = typeof context?.followUpCount === "number" ? context.followUpCount : 0;
     return issue.threadId !== undefined
@@ -693,8 +222,26 @@ export class RunOrchestrator {
                 return;
             }
         }
-        // Build prompt
-        const prompt = buildRunPrompt(issue, runType, project.repoPath, effectiveContext);
+        const repoPrompting = loadPatchRelayRepoPrompting({
+            repoRoot: project.repoPath,
+            logger: this.logger,
+        });
+        const promptLayer = mergePromptCustomizationLayers(resolvePromptLayers(this.config.prompting, runType), resolvePromptLayers(repoPrompting, runType));
+        const unknownPromptSections = findUnknownPatchRelayPromptSectionIds(promptLayer);
+        if (unknownPromptSections.length > 0) {
+            this.logger.warn({ issueKey: issue.issueKey, runType, unknownPromptSections }, "PatchRelay prompt customization references unknown section ids");
+        }
+        const disallowedPromptSections = findDisallowedPatchRelayPromptSectionIds(promptLayer);
+        if (disallowedPromptSections.length > 0) {
+            this.logger.warn({ issueKey: issue.issueKey, runType, disallowedPromptSections }, "PatchRelay prompt customization attempted to replace non-overridable sections");
+        }
+        const prompt = buildPatchRelayRunPrompt({
+            issue,
+            runType,
+            repoPath: project.repoPath,
+            ...(effectiveContext ? { context: effectiveContext } : {}),
+            ...(promptLayer ? { promptLayer } : {}),
+        });
         // Resolve workspace
         const issueRef = sanitizePathSegment(issue.issueKey ?? issue.linearIssueId);
         const slug = issue.title ? slugify(issue.title) : "";
@@ -2372,82 +1919,3 @@ function buildReviewFixBranchUpkeepContext(prNumber, baseBranch, pr, context) {
         baseBranch,
     };
 }
-function appendQueueRepairContext(lines, context) {
-    const incidentTitle = typeof context?.incidentTitle === "string" ? context.incidentTitle.trim() : "";
-    const incidentSummary = typeof context?.incidentSummary === "string" ? context.incidentSummary.trim() : "";
-    const incidentId = typeof context?.incidentId === "string" ? context.incidentId.trim() : "";
-    const incidentUrl = typeof context?.incidentUrl === "string" ? context.incidentUrl.trim() : "";
-    const incidentContext = context?.incidentContext && typeof context.incidentContext === "object"
-        ? context.incidentContext
-        : undefined;
-    const failureClass = typeof incidentContext?.failureClass === "string" ? incidentContext.failureClass : "";
-    const baseSha = typeof incidentContext?.baseSha === "string" ? incidentContext.baseSha : "";
-    const prHeadSha = typeof incidentContext?.prHeadSha === "string" ? incidentContext.prHeadSha : "";
-    const baseBranch = typeof incidentContext?.baseBranch === "string" ? incidentContext.baseBranch : "";
-    const branch = typeof incidentContext?.branch === "string" ? incidentContext.branch : "";
-    const queuePosition = typeof incidentContext?.queuePosition === "number" ? String(incidentContext.queuePosition) : "";
-    const conflictFiles = Array.isArray(incidentContext?.conflictFiles)
-        ? incidentContext.conflictFiles.filter((entry) => typeof entry === "string")
-        : [];
-    const failedChecks = Array.isArray(incidentContext?.failedChecks)
-        ? incidentContext.failedChecks
-            .filter((entry) => Boolean(entry) && typeof entry === "object")
-            .map((entry) => ({
-            name: typeof entry.name === "string" ? entry.name : "unknown",
-            conclusion: typeof entry.conclusion === "string" ? entry.conclusion : "unknown",
-            ...(typeof entry.url === "string" ? { url: entry.url } : {}),
-        }))
-        : [];
-    const retryHistory = Array.isArray(incidentContext?.retryHistory)
-        ? incidentContext.retryHistory
-            .filter((entry) => Boolean(entry) && typeof entry === "object")
-            .map((entry) => ({
-            at: typeof entry.at === "string" ? entry.at : "unknown",
-            baseSha: typeof entry.baseSha === "string" ? entry.baseSha : "unknown",
-            outcome: typeof entry.outcome === "string" ? entry.outcome : "unknown",
-        }))
-        : [];
-    if (!incidentTitle && !incidentSummary && !incidentId && !incidentUrl && !failureClass && !baseSha && !prHeadSha
-        && !queuePosition && conflictFiles.length === 0 && failedChecks.length === 0 && retryHistory.length === 0) {
-        return;
-    }
-    lines.push("## Queue Incident Context", "");
-    if (incidentTitle)
-        lines.push(`Incident: ${incidentTitle}`);
-    if (incidentId)
-        lines.push(`Incident ID: ${incidentId}`);
-    if (incidentUrl)
-        lines.push(`Incident URL: ${incidentUrl}`);
-    if (incidentSummary)
-        lines.push("", incidentSummary, "");
-    if (failureClass)
-        lines.push(`Failure class: ${failureClass}`);
-    if (baseBranch)
-        lines.push(`Base branch: ${baseBranch}`);
-    if (baseSha)
-        lines.push(`Base SHA: ${baseSha}`);
-    if (branch)
-        lines.push(`Queue branch: ${branch}`);
-    if (prHeadSha)
-        lines.push(`Queue branch head SHA: ${prHeadSha}`);
-    if (queuePosition)
-        lines.push(`Queue position at eviction: ${queuePosition}`);
-    if (conflictFiles.length > 0) {
-        lines.push("", "Conflicting files:");
-        for (const file of conflictFiles)
-            lines.push(`- ${file}`);
-    }
-    if (failedChecks.length > 0) {
-        lines.push("", "Failed checks:");
-        for (const check of failedChecks) {
-            lines.push(`- ${check.name} (${check.conclusion})${check.url ? ` ${check.url}` : ""}`);
-        }
-    }
-    if (retryHistory.length > 0) {
-        lines.push("", "Retry history:");
-        for (const retry of retryHistory) {
-            lines.push(`- ${retry.at}: ${retry.outcome} on base ${retry.baseSha}`);
-        }
-    }
-    lines.push("");
-}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "patchrelay",
-  "version": "0.36.5",
+  "version": "0.36.7",
   "license": "MIT",
   "type": "module",
   "repository": {