@lchase/kiln 0.13.0 → 0.14.0

package/dist/driver.js

@@ -13,7 +13,7 @@ import { ensureFixTrailers } from "./fix-commit.js";
 import { holdDetail, pausedFrom, recordDailyCap, reviseNotesSinceRun, spendSnapshot, stopReasonFromEvents, withReviseNotes, } from "./rails.js";
 import { appendInvestigationFindings, existingFindingsPath, findingsFilePath, INVESTIGATION_WORKFLOW, readFindingsMarkdown, writeFindings, } from "./findings.js";
 import { defaultHost } from "./host.js";
-import { readJobSpec, resolveJobSpec } from "./jobs.js";
+import { markJobCancelled, readJobSpec, resolveJobSpec } from "./jobs.js";
 import { jobBasename, packageRoot } from "./paths.js";
 import { decide, isLanded, jobState, } from "./reconcile.js";
 import { fileMtimeMs, isPidAlive, readAttemptLedger, recoverRuns, resetAttemptLedger, } from "./reliability.js";
@@ -23,7 +23,7 @@ import { gateAnswerCommand, renderStatus, renderTick, statusView, yakCliPath, sa
 import { AGENT_SANDBOX_IMAGE, injectAgentSandbox, jobIsSandboxed, probeSandboxEnvironment, PROXY_SANDBOX_IMAGE, requiredSandboxImages, resolveSandbox, sandboxIsEnabled, sandboxPreflight, sandboxPreflightFor, } from "./sandbox.js";
 import { JobSchema } from "./schemas.js";
 import { ensureReviewHead, evaluateTriggers, recordOwnedPullRequest, reviewHeadOf, } from "./triggers.js";
-import { leafStepIds, materializeWorkflow, REVIEW_WORKFLOW, workflowInputs, workflowNeedsTestCommand, workflowStepPlan, } from "./workflow.js";
+import { assertWorkflowName, leafStepIds, materializeWorkflow, REVIEW_WORKFLOW, workflowInputs, workflowNeedsTestCommand, workflowStepPlan, } from "./workflow.js";
 const TAG_PREFIX = "kiln:";
 export function jobTag(jobId) {
     return `${TAG_PREFIX}${jobId}`;
@@ -137,14 +137,48 @@ export function answerJob(config, jobId, choice, host = defaultHost()) {
     return output.endsWith("\n") || output.length === 0 ? output : `${output}\n`;
 }
 /**
- * Run a failed or dead job again. The board Retry button and `kiln retry`
- * both call this.
- * A landed job whose only failure is opening the pull request is handed to
- * the same deliver path tick uses. Everything else is a new `yak run`, the
- * same launch tick uses. The attempt ledger is cleared either way, and the
- * current run id is ignored so a dead job is not stuck on the old journal.
- * A dead journal is cancelled first.
+ * Cancel queued jobs that have not started.
+ * A running, blocked, or finished job is left alone. Calling again is a no-op.
+ * The event log records `job.cancelled`. Branches are not deleted.
  */
+export function cancelQueuedJobs(config, options = {}, host = defaultHost()) {
+    return withTickLock(config.kilnDir, () => {
+        if (options.repo && !config.repos[options.repo]) {
+            throw new Error(`Unknown repo ${options.repo}.`);
+        }
+        if (options.workflow)
+            assertWorkflowName(options.workflow);
+        const now = options.now ?? new Date();
+        const world = observe(config, host, now);
+        const cancelled = [];
+        for (const observed of world.observation.jobs) {
+            if (observed.error)
+                continue;
+            if (options.repo && observed.repo !== options.repo)
+                continue;
+            if (options.workflow && observed.workflow !== options.workflow)
+                continue;
+            if (jobState(observed, world.observation) !== "queued")
+                continue;
+            const job = world.jobs.get(observed.jobId);
+            if (!job || job.cancelledAt)
+                continue;
+            markJobCancelled(config, job, now);
+            appendEvent(config.kilnDir, {
+                job_id: job.id,
+                event: "job.cancelled",
+                ts: now.toISOString(),
+                properties: {
+                    repo: job.repo,
+                    workflow: job.workflow,
+                    once: "job.cancelled",
+                },
+            });
+            cancelled.push(job.id);
+        }
+        return { cancelled };
+    });
+}
 export function retryJob(config, jobId, host = defaultHost()) {
     return withTickLock(config.kilnDir, () => {
         const world = observe(config, host);
@@ -366,6 +400,7 @@ function observe(config, host, now = new Date()) {
             ...(launchActivityMs !== undefined ? { launchActivityMs } : {}),
             ...(logPath ? { logPath } : {}),
             ...(entry.job?.createdAt ? { createdAt: entry.job.createdAt } : {}),
+            ...(entry.job?.cancelledAt ? { cancelled: true } : {}),
             ...(logTail ? { logTail } : {}),
             ...(diffSummary ? { diffSummary } : {}),
             ...(stepPlan && stepPlan.length > 0 ? { stepPlan } : {}),

package/dist/events.js

@@ -14,6 +14,7 @@ import { redactDeep } from "./redact.js";
  */
 export const EVENT_NAMES = [
     "job.added",
+    "job.cancelled",
     "job.retried",
     "run.started",
     "phase.started",
@@ -52,6 +53,11 @@ const JobRetried = z.object({
     repo: z.string().min(1),
     once,
 });
+const JobCancelled = z.object({
+    repo: z.string().min(1),
+    workflow: z.string().min(1),
+    once,
+});
 const RunStarted = z.object({
     kiln_version: z.string().min(1),
     yak_version: z.string().min(1),
@@ -163,7 +169,9 @@ const TriggerFailed = z.object({
 });
 const TriggerSkipped = z.object({
     trigger_id: z.string().min(1),
-    reason: z.enum(["paused", "capped"]),
+    reason: z.enum(["paused", "capped", "burst-cap", "queued-cap"]),
+    /** Pull request or issue the skip applies to, when the fire skipped one item. */
+    external_id: z.string().min(1).optional(),
     once,
 });
 const ReviewPosted = z.object({
@@ -196,6 +204,7 @@ const envelope = {
 };
 export const EventSchema = z.discriminatedUnion("event", [
     z.object({ ...envelope, event: z.literal("job.added"), job_id: z.string().min(1), properties: JobAdded }),
+    z.object({ ...envelope, event: z.literal("job.cancelled"), job_id: z.string().min(1), properties: JobCancelled }),
     z.object({ ...envelope, event: z.literal("job.retried"), job_id: z.string().min(1), properties: JobRetried }),
     z.object({ ...envelope, event: z.literal("run.started"), job_id: z.string().min(1), properties: RunStarted }),
     z.object({ ...envelope, event: z.literal("phase.started"), job_id: z.string().min(1), properties: PhaseStarted }),

package/dist/host.js

@@ -15,6 +15,15 @@ export class HostError extends Error {
         this.name = "HostError";
     }
 }
+/** GitHub label `review-requests` requires when the trigger sets none. */
+export const DEFAULT_REVIEW_LABEL = "kiln-review";
+/** `gh pr list --search` query. Opt-in: the pull request must carry `label`. */
+export function reviewSearchQuery(label) {
+    const trimmed = label.trim();
+    if (!trimmed || /[\r\n]/.test(trimmed))
+        throw new Error("Review label must be a single line.");
+    return `review-requested:@me label:${quoteSearch(trimmed)}`;
+}
 export function defaultHost() {
     return { github: realGitHub, git: realGitRemote, feedback: realPullRequestFeedback };
 }
@@ -24,14 +33,27 @@ export function productionHost() {
 }
 /**
  * Open pull requests awaiting the authenticated user's review, per registered repo.
- * A remote that is not GitHub is skipped, so a local test origin does not call gh.
+ * The search is opt-in: a pull request must carry `label`, or `kiln-review` when
+ * that is omitted. A remote that is not GitHub is skipped, so a local test origin does not call gh.
  */
 export function listReviewRequests(repos) {
     const found = [];
     for (const repo of repos) {
         if (!githubRemote(repo.path))
             continue;
-        const result = spawnSync(ghBin(), ["pr", "list", "--search", "review-requested:@me", "--state", "open", "--limit", "50", "--json", "number,title,url,headRefOid"], {
+        const label = repo.label?.trim() || DEFAULT_REVIEW_LABEL;
+        const result = spawnSync(ghBin(), [
+            "pr",
+            "list",
+            "--search",
+            reviewSearchQuery(label),
+            "--state",
+            "open",
+            "--limit",
+            "50",
+            "--json",
+            "number,title,url,headRefOid,labels",
+        ], {
             cwd: repo.path,
             encoding: "utf8",
             timeout: 20_000,
@@ -57,6 +79,8 @@ export function listReviewRequests(repos) {
             const sha = typeof record.headRefOid === "string" ? record.headRefOid.trim().toLowerCase() : "";
             if (!Number.isInteger(number) || number <= 0 || !/^[0-9a-f]{7,64}$/.test(sha))
                 continue;
+            if (!carriesLabel(record.labels, label))
+                continue;
             const title = typeof record.title === "string" && record.title.trim() ? record.title.trim() : `PR ${number}`;
             const url = typeof record.url === "string" ? record.url : "";
             found.push({ repo: repo.name, number, title, url, headSha: sha });
@@ -64,6 +88,25 @@ export function listReviewRequests(repos) {
     }
     return found;
 }
+/** Missing `labels` trusts the search. A present list must include `label`. */
+function carriesLabel(labels, label) {
+    if (labels === undefined)
+        return true;
+    if (!Array.isArray(labels))
+        return false;
+    const want = label.trim().toLowerCase();
+    return labels.some((item) => {
+        if (typeof item === "string")
+            return item.trim().toLowerCase() === want;
+        if (typeof item !== "object" || item === null || !("name" in item))
+            return false;
+        const name = item.name;
+        return typeof name === "string" && name.trim().toLowerCase() === want;
+    });
+}
+function quoteSearch(value) {
+    return `"${value.replaceAll("\\", "\\\\").replaceAll('"', '\\"')}"`;
+}
 function githubRemote(checkout) {
     const result = gitResult(checkout, ["remote", "get-url", "origin"]);
     if (result.status !== 0)