next-issue 0.15.0 → 0.16.0

package/README.md

@@ -22,8 +22,10 @@ For each open issue, oldest first:
 1. Claim the issue only when it holds the `status:todo` label and holds no
    `status:in-progress` or `status:in-review` label. A `status:done`,
    `status:needs-human` or `status:blocked` label always stops the claim, a
-   resume too. An issue with saved state under `.next-issue/` resumes instead,
-   whatever its other labels.
+   resume too. An issue that GitHub lists as blocked by an open issue waits,
+   because its blocker closes only when the pull request of the blocker merges.
+   An issue with saved state under `.next-issue/` resumes instead, whatever its
+   other labels and blockers.
 2. Add `status:in-progress` to the issue.
 3. Fetch the remote, fast-forward the local copy of the default branch when
    that is safe, then add a git worktree at `../<repo>-issue-<n>` on the
@@ -264,8 +266,8 @@ does not know all stop the run, so a typo cannot pass without a word.
 An empty `labels.ready` turns the ready requirement off. The harness then
 claims every open issue that no other label holds back.
 
-The log gives the reason for a skipped issue as `stop-label`, `not-ready` or
-`in-flight`.
+The log gives the reason for a skipped issue as `stop-label`, `not-ready`,
+`in-flight` or `blocked`, the last one with the open blockers.
 
 A role set to `null`, or with no entry in `models`, uses the default model of
 the agent.

package/dist/github.mjs

@@ -44,6 +44,19 @@ export async function issueComments(repo, issue) {
     const parsed = JSON.parse(raw);
     return parsed.comments.map((comment) => `@${comment.author.login}: ${comment.body}`);
 }
+export async function openBlockers(repo, issue) {
+    const raw = await must("gh", [
+        "api",
+        "--paginate",
+        `repos/${slug(repo)}/issues/${issue}/dependencies/blocked_by`,
+        "--jq",
+        '.[] | select(.state == "open") | (.repository_url | split("/") | .[-2:] | join("/")) + "#" + (.number | tostring)',
+    ], { cwd: repo.root });
+    return raw
+        .split("\n")
+        .filter((line) => line.length > 0)
+        .map((ref) => (ref.startsWith(`${slug(repo)}#`) ? ref.slice(slug(repo).length) : ref));
+}
 async function labelsOf(repo, kind, number) {
     const raw = await gh(repo, [kind, "view", String(number), "--json", "labels"]);
     const parsed = JSON.parse(raw);

package/dist/pipeline.mjs

@@ -2,7 +2,7 @@ import { CODING, REVIEWING, runAgent } from "./agents.mjs";
 import { managedLabels } from "./config.mjs";
 import { run } from "./exec.mjs";
 import { addWorktree, branchName, commitAll, deleteBranch, diff, discardChanges, hasWorktree, isDirty, push, removeWorktree, revision, worktreePath, } from "./git.mjs";
-import { commentOnPr, createPr, failedCheckLogs, findPr, issueComments, markPrReady, setLabel, waitForChecks, } from "./github.mjs";
+import { commentOnPr, createPr, failedCheckLogs, findPr, issueComments, markPrReady, openBlockers, setLabel, waitForChecks, } from "./github.mjs";
 import { message } from "./observe.mjs";
 import { capDiff, fixPrompt, implementPrompt, parseCommitSubject, parseUnrelated, reviewPrompt, } from "./prompts.mjs";
 import { detectSetupCommand } from "./setup.mjs";
@@ -23,6 +23,7 @@ export const PORTS = {
     isDirty,
     issueComments,
     markPrReady,
+    openBlockers,
     push,
     removeWorktree,
     revision,
@@ -55,7 +56,21 @@ export async function processIssue(context, issue) {
     const skip = skipReason(issue, context.config, saved !== undefined);
     if (skip !== undefined) {
         log.event("issue.skip", { title: issue.title, reason: skip });
-        return { issue: issue.number, outcome: "skipped", ciFixes: 0, reviewRounds: 0, ms: 0 };
+        return skipped(issue);
+    }
+    if (saved === undefined) {
+        let blockers;
+        try {
+            blockers = await context.ports.openBlockers(context.repo, issue.number);
+        }
+        catch (error) {
+            log.event("issue.error", { error: message(error), ms: Date.now() - started }, "quiet");
+            return { ...skipped(issue), outcome: "error", reason: message(error), ms: Date.now() - started };
+        }
+        if (blockers.length > 0) {
+            log.event("issue.skip", { title: issue.title, reason: "blocked", blockers });
+            return skipped(issue);
+        }
     }
     log.event("issue.start", { title: issue.title, createdAt: issue.createdAt }, "quiet");
     const fresh = {
@@ -92,6 +107,9 @@ export async function processIssue(context, issue) {
         };
     }
 }
+function skipped(issue) {
+    return { issue: issue.number, outcome: "skipped", ciFixes: 0, reviewRounds: 0, ms: 0 };
+}
 async function work(context, issue, log, state) {
     const started = Date.now();
     const { repo, config, ports } = context;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "next-issue",
-  "version": "0.15.0",
+  "version": "0.16.0",
   "description": "Autonomous issue harness driven by the Claude Agent SDK",
   "keywords": [
     "github",