@pmelab/gtd 45.1.0 → 45.2.0

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "gtd",
-  "version": "45.1.0",
+  "version": "45.2.0",
   "description": "Drive the gtd loop inside Claude Code: agent turns run as subagents, gtd decides every beat",
   "author": {
     "name": "Philipp Melab"

package/README.md

@@ -371,11 +371,15 @@ the top branch with `git push origin --delete`, since
 ask Claude to hand it off (or run `/gtd throw @dev`) and it opens a draft pull
 request assigned to them, and `/gtd catch <pr>` picks it up exactly where it
 waits, so whoever wrote the requirements can hand the build to someone else.
-Ship, hand-off and throw answer at once and run in the background, with progress
-on the status line; they report by waking Claude, a toast, a transcript line and
-a push notification, and a failed push or `gh` call reports as a failure. While
-a ship or hand-off runs or is queued, `/gtd catch`, the doors,
-`/gtd <requirements>` and bare `/gtd` wait for its report. See
+`/gtd pr` brings a pull request's unresolved review threads into the review:
+each becomes a note in `.gtd/REVIEW.md` ticked "fix as suggested" (untick it for
+"won't fix" plus a reason, or your own answer), the review door fixes them in
+place, and each thread then gets a reply naming the fixing commit (or your
+reason) and is resolved. Ship, hand-off and throw answer at once and run in the
+background, with progress on the status line; they report by waking Claude, a
+toast, a transcript line and a push notification, and a failed push or `gh` call
+reports as a failure. While a ship or hand-off runs or is queued, `/gtd catch`,
+the doors, `/gtd <requirements>` and bare `/gtd` wait for its report. See
 [Inside Claude Code](https://github.com/pmelab/gtd/blob/main/docs/driver.md#inside-claude-code-the-gtd-mod).
 
 ### Then let an agent build your own

package/claude/hooks/pr.ts

@@ -0,0 +1,342 @@
+// `/gtd pr`: a pull request's unresolved review threads become notes in the
+// review gate's file, and once the review door has acted on them each thread
+// gets a reply and is resolved. gtd itself never talks to GitHub: the mod
+// reads and writes the threads, and gtd only sees the notes.
+
+import type { PrFeedback, PrThread } from "../types"
+import { pushBranch } from "./ship"
+import type { ShipIo, Shipped } from "./ship"
+
+export const REVIEW_FILE = ".gtd/REVIEW.md"
+
+export type Pr = {
+  host: string
+  owner: string
+  name: string
+  number: number
+  url: string
+  branch: string
+}
+export type Thread = PrThread
+export type Choice = "fix" | "wontfix" | "answer"
+export type Answer = { thread: string; title: string; choice: Choice; text: string }
+
+const PR_URL = /^https?:\/\/([^/]+)\/([^/]+)\/([^/]+)\/pull\/(\d+)/
+// The thread's node id, tagged on the note's heading.
+const TAG = /\(thread ([A-Za-z0-9_=-]+)\)\s*$/
+const LEAD = "Tick exactly one:"
+const PLACEHOLDER = "_your answer_"
+
+const fail = (text: string): Shipped => ({ ok: false, text })
+
+export async function currentPr(io: ShipIo): Promise<Pr | string> {
+  const branch = (await io.run(["git", "rev-parse", "--abbrev-ref", "HEAD"])).out.trim()
+  const v = await io.run(["gh", "pr", "view", "--json", "number,url"])
+  if (v.code !== 0) return `gh pr view failed: ${v.err.trim()}`
+  try {
+    const { url } = JSON.parse(v.out) as { url: string }
+    const m = PR_URL.exec(url)
+    if (!m) return `gh pr view returned an unexpected url: ${url}`
+    return { host: m[1]!, owner: m[2]!, name: m[3]!, number: Number(m[4]), url, branch }
+  } catch {
+    return "gh pr view returned something unreadable."
+  }
+}
+
+const THREADS = `query($owner: String!, $name: String!, $number: Int!) {
+  repository(owner: $owner, name: $name) {
+    pullRequest(number: $number) {
+      reviewThreads(first: 100) {
+        nodes {
+          id
+          isResolved
+          path
+          line
+          originalLine
+          comments(first: 50) { nodes { author { login } body url } }
+        }
+      }
+    }
+  }
+}`
+
+type ThreadNode = {
+  id: string
+  isResolved: boolean
+  path: string | null
+  line: number | null
+  originalLine: number | null
+  comments: { nodes: { author: { login: string } | null; body: string; url: string }[] }
+}
+
+export async function unresolvedThreads(io: ShipIo, pr: Pr): Promise<Thread[] | string> {
+  const r = await io.run([
+    "gh",
+    "api",
+    "graphql",
+    "--hostname",
+    pr.host,
+    "-f",
+    `query=${THREADS}`,
+    "-f",
+    `owner=${pr.owner}`,
+    "-f",
+    `name=${pr.name}`,
+    "-F",
+    `number=${pr.number}`,
+  ])
+  if (r.code !== 0) return `Reading the review threads failed: ${r.err.trim()}`
+  let nodes: ThreadNode[]
+  try {
+    nodes = JSON.parse(r.out).data.repository.pullRequest.reviewThreads.nodes as ThreadNode[]
+  } catch {
+    return "gh api returned something unreadable for the review threads."
+  }
+  return nodes
+    .filter((t) => !t.isResolved)
+    .map((t) => ({
+      id: t.id,
+      ...(t.path ? { path: t.path } : {}),
+      ...((t.line ?? t.originalLine) ? { line: (t.line ?? t.originalLine)! } : {}),
+      comments: t.comments.nodes.map((c) => ({
+        author: c.author?.login ?? "ghost",
+        body: c.body,
+        url: c.url,
+      })),
+    }))
+}
+
+// Code spans pass as they are; elsewhere only an entity hides a footnote
+// marker or a comment from gtd's raw-text checks.
+const escaped = (line: string) =>
+  line
+    .replace(/^((?:\s*>)*\s*(?:[-*+]|\d+[.)])\s+)\[([ xX])\]/, "$1\\[$2\\]")
+    .split(/(`+[^`]*`+)/)
+    .map((part, i) =>
+      i % 2 === 1 ? part : part.replace(/\[\^/g, "&#91;^").replace(/<!--/g, "&lt;!--"),
+    )
+    .join("")
+
+// The fence open after `line`, given the one open before it. A closing fence
+// carries no info string.
+const fenceAfter = (open: string | undefined, line: string) => {
+  const m = /^(?:\s*>)*\s*(`{3,}|~{3,})(.*)$/.exec(line)
+  if (!m) return open
+  const f = m[1]!
+  if (open === undefined) return f
+  return f[0] === open[0] && f.length >= open.length && !m[2]!.trim() ? undefined : open
+}
+
+// A comment is quoted, and anything that would parse as part of the review's
+// own structure — a task box, a footnote — is escaped. Code in a fence parses
+// as code, so a suggestion is kept verbatim.
+const quoted = (body: string) => {
+  let fence: string | undefined
+  return body
+    .trim()
+    .split(/\r?\n/)
+    .map((l) => {
+      const wasCode = fence !== undefined
+      fence = fenceAfter(fence, l)
+      return wasCode || fence !== undefined ? l : escaped(l)
+    })
+    .map((l) => (l ? `> ${l}` : ">"))
+    .join("\n")
+}
+
+const heading = (t: Thread) =>
+  `## ${t.path ? `${t.path}${t.line ? `:${t.line}` : ""}` : "Pull request review"} (thread ${t.id})`
+
+export function threadNote(t: Thread) {
+  const said = t.comments.map((c) => `**@${c.author}** ([comment](${c.url})):\n\n${quoted(c.body)}`)
+  return [
+    heading(t),
+    ...said,
+    "From the pull request. To decline it, tick won't fix and write your reason after it on that line.",
+    LEAD,
+    "- [x] fix as suggested\n- [ ] won't fix\n- [ ] _your answer_",
+  ].join("\n\n")
+}
+
+// The review with every thread it does not already hold appended, pre-ticked.
+export function withThreads(review: string, threads: readonly Thread[]) {
+  const held = new Set(chunks(review).flatMap((c) => (c.thread ? [c.thread] : [])))
+  const added = threads.filter((t) => !held.has(t.id)).map(threadNote)
+  return added.length === 0 ? review : `${review.trimEnd()}\n\n${added.join("\n\n")}\n`
+}
+
+type Chunk = { title: string; thread?: string; lines: string[] }
+
+// Every imported line is quoted, so a `## ` line is always a chunk's heading.
+function chunks(review: string): Chunk[] {
+  return review
+    .split(/^## /m)
+    .slice(1)
+    .map((part) => {
+      const [first = "", ...lines] = part.split(/\r?\n/)
+      const title = first.trim()
+      const thread = TAG.exec(title)?.[1]
+      return { title, ...(thread ? { thread } : {}), lines }
+    })
+}
+
+// An unindented line right below an item continues it, as markdown reads it.
+const continues = (line: string, afterBlank: boolean) => !afterBlank || /^\s+\S/.test(line)
+
+// The three answer boxes after the lead line, a wrapped item's continuation joined on.
+function boxes(lines: readonly string[]) {
+  const at = lines.findIndex((l) => l.trim() === LEAD)
+  if (at === -1) return undefined
+  const items: { checked: boolean; text: string }[] = []
+  let blank = false
+  for (const line of lines.slice(at + 1)) {
+    const m = /^[-*+] \[([ xX])\]\s*(.*)$/.exec(line)
+    const isText = line.trim() !== ""
+    if (m) items.push({ checked: m[1] !== " ", text: m[2]!.trim() })
+    else if (items.length > 0 && isText && continues(line, blank))
+      items.at(-1)!.text = `${items.at(-1)!.text} ${line.trim()}`.trim()
+    else if (items.length > 0 && isText) break
+    blank = !isText
+  }
+  return items
+}
+
+const reasonOf = (text: string) => text.replace(/^won['’]t fix\b[\s:,;.—–-]*/i, "").trim()
+
+export function answers(review: string): Answer[] {
+  return chunks(review).flatMap((c) => {
+    const b = c.thread ? boxes(c.lines) : undefined
+    if (!c.thread || b?.length !== 3) return []
+    const ticked = b.flatMap((o, i) => (o.checked ? [i] : []))
+    if (ticked.length !== 1) return []
+    const i = ticked[0]!
+    const choice = (["fix", "wontfix", "answer"] as const)[i]!
+    const own = b[2]!.text.toLowerCase() === PLACEHOLDER ? "" : b[2]!.text
+    const text = choice === "wontfix" ? reasonOf(b[1]!.text) : choice === "answer" ? own : ""
+    return [{ thread: c.thread, title: c.title, choice, text }]
+  })
+}
+
+// Thread notes declined without the reason their reply needs.
+export function missingReasons(review: string) {
+  return answers(review)
+    .filter((a) => a.choice === "wontfix" && a.text === "")
+    .map((a) => `${REVIEW_FILE}: "${a.title}": write the reason after "won't fix" on that line.`)
+}
+
+// The first commit since `since` whose review answers any of `ids`: the gate
+// that decided them. Undefined while none has landed.
+export async function decided(io: ShipIo, since: string, ids: readonly string[]) {
+  // A reset or rebase since the import leaves nothing to find the gate in.
+  if ((await io.run(["git", "merge-base", "--is-ancestor", since, "HEAD"])).code !== 0)
+    return "gone"
+  const log = await io.run([
+    "git",
+    "log",
+    "--reverse",
+    "--first-parent",
+    "--format=%H",
+    `${since}..HEAD`,
+  ])
+  if (log.code !== 0) return "gone"
+  for (const commit of log.out.split("\n").filter(Boolean)) {
+    const shown = await io.run(["git", "show", `${commit}:${REVIEW_FILE}`])
+    if (shown.code !== 0) continue
+    const got = answers(shown.out).filter((a) => ids.includes(a.thread))
+    if (got.length > 0) return { gate: commit, answers: got }
+  }
+  return undefined
+}
+
+export type PrRest = { isReviewGate: boolean; isDone: boolean; branch: string }
+
+// What a rest does with `/gtd pr`'s threads. Threads of another branch's pull
+// request never reach this one's review; a process that ends before any review
+// gate drops them.
+export function prAction(pr: PrFeedback | undefined, at: PrRest) {
+  if (!pr) return "none"
+  if (at.branch !== pr.branch) return "forget"
+  if (pr.importedAt === undefined) return at.isReviewGate ? "import" : at.isDone ? "forget" : "none"
+  return at.isReviewGate || at.isDone ? "reply" : "none"
+}
+
+// The fixes the gate asked for are the first turn after it, so its first code
+// commit is the fix; later ones belong to other notes or rounds.
+export function replyText(a: Answer, fix: string | undefined) {
+  if (a.choice === "wontfix") return a.text ? `Won't fix: ${a.text}` : "Won't fix."
+  if (a.choice === "answer") return a.text
+  return fix ? `Fixed in ${fix}.` : "Fixed as suggested; the fix changed no code."
+}
+
+const REPLY = `mutation($thread: ID!, $body: String!) {
+  addPullRequestReviewThreadReply(input: { pullRequestReviewThreadId: $thread, body: $body }) { comment { id } }
+}`
+const RESOLVE = `mutation($thread: ID!) {
+  resolveReviewThread(input: { threadId: $thread }) { thread { isResolved } }
+}`
+
+// Pushes the branch so the commits a reply names exist on GitHub, then replies
+// to each answered thread and resolves it. The code commits after `gate` are
+// the fixes; a thread that fails is named and left open.
+export async function reply(
+  io: ShipIo,
+  host: string,
+  gate: string,
+  decidedAnswers: readonly Answer[],
+): Promise<Shipped & { isPushed: boolean }> {
+  const branch = (await io.run(["git", "rev-parse", "--abbrev-ref", "HEAD"])).out.trim()
+  const pushed = await pushBranch(io, branch)
+  if (pushed) return { ...fail(pushed), isPushed: false }
+  const log = await io.run([
+    "git",
+    "log",
+    "--reverse",
+    "--first-parent",
+    "--format=%H",
+    `${gate}..HEAD`,
+    "--",
+    ".",
+    ":(exclude).gtd",
+  ])
+  const fix = log.out.split("\n").find(Boolean)
+  const failed: string[] = []
+  for (const a of decidedAnswers) {
+    const sent = await io.run([
+      "gh",
+      "api",
+      "graphql",
+      "--hostname",
+      host,
+      "-f",
+      `query=${REPLY}`,
+      "-f",
+      `thread=${a.thread}`,
+      "-f",
+      `body=${replyText(a, fix)}`,
+    ])
+    const resolved =
+      sent.code === 0 &&
+      (
+        await io.run([
+          "gh",
+          "api",
+          "graphql",
+          "--hostname",
+          host,
+          "-f",
+          `query=${RESOLVE}`,
+          "-f",
+          `thread=${a.thread}`,
+        ])
+      ).code === 0
+    if (!resolved) failed.push(a.title)
+  }
+  const done = decidedAnswers.length - failed.length
+  const text = `Replied to and resolved ${done} review thread(s).`
+  return failed.length === 0
+    ? { ok: true, text, isPushed: true }
+    : {
+        ...fail(`${text}\nThese failed and stay open:\n${failed.map((t) => `- ${t}`).join("\n")}`),
+        isPushed: true,
+      }
+}

package/claude/hooks/register.tsx

@@ -10,6 +10,15 @@ import type { Io, Opening, Turn, TurnEnd } from "./drive"
 import { doorFor, doors, enter } from "./doors"
 import { subagentModel } from "./models"
 import { pastedCommand } from "./paste"
+import {
+  currentPr,
+  decided,
+  missingReasons,
+  reply,
+  REVIEW_FILE,
+  unresolvedThreads,
+  withThreads,
+} from "./pr"
 import { personaSpec, resumable } from "./persona"
 import { JUDGE_SYSTEM, judgePrompt, judgeQuestions, judgeRunEnv, toVerdicts } from "./judge"
 import type { Judgment } from "./judge"
@@ -277,8 +286,8 @@ async function start($: $, opening: Opening = {}) {
   moveOn($)
   stopRequested = false
   await findRoot($)
-  // Requests not yet handed to a rest outlive a restart.
-  await update($, run, (r) => ({ isRunning: true, beat: 0, notes: r.notes }))
+  // Requests not yet handed to a rest, and review threads not yet replied to, outlive a restart.
+  await update($, run, (r) => ({ isRunning: true, beat: 0, notes: r.notes, pr: r.pr }))
   void drive(io($), opening)
     .catch((err: unknown): Stop => ({ kind: "error", text: String(err) }))
     .then(async (stop) => {
@@ -290,6 +299,8 @@ async function start($: $, opening: Opening = {}) {
       const rest = restId(at.head, stop.state)
       await update($, run, (r) => ({ ...r, isRunning: false, stop, rest, isRepeat }))
       $.ui.status(undefined)
+      // Before the wake, so nothing the model starts there races the push.
+      await prRest($, stop)
       $.ui.toast(headline(stop))
       await push($, stop)
       await wake($)
@@ -502,6 +513,125 @@ async function enterNow($: $, door: string, args: string[]) {
   return entered.text
 }
 
+const isReviewGate = (stop: Stop | undefined) =>
+  stop?.kind === "gate" &&
+  !stop.isJudge &&
+  stop.file === REVIEW_FILE &&
+  /(^|\.)review\.await-review$/.test(stop.state ?? "")
+
+// The pull request's unresolved threads go into the review gate's file: now,
+// when gtd waits there, or through the review door at its first gate.
+async function prNow($: $) {
+  const r = await read($, run)
+  if (r.isRunning) return "Stop the loop first: /gtd stop."
+  if (r.pending) return "A continue is already waiting for this turn to end."
+  if (r.pr?.importedAt !== undefined) return retryReplies($, r.stop)
+  if (r.pr && !isReviewGate(r.stop)) {
+    return "Review threads from an earlier /gtd pr wait for the review's first review gate."
+  }
+  const io = shipIo($)
+  const pr = await currentPr(io)
+  if (typeof pr === "string") return pr
+  const threads = await unresolvedThreads(io, pr)
+  if (typeof threads === "string") return threads
+  if (threads.length === 0) return `${pr.url} has no unresolved review threads.`
+  await update($, run, (x) => ({ ...x, pr: { threads, host: pr.host, branch: pr.branch } }))
+  if (isReviewGate(r.stop)) {
+    await importThreads($)
+    return `Added ${threads.length} review thread(s) of ${pr.url} to ${REVIEW_FILE}, ticked fix as suggested.`
+  }
+  const entered = await enter(io, "review", [])
+  if (!entered.ok) {
+    await update($, run, (x) => ({ ...x, pr: undefined }))
+    return `${entered.text}\nRun /gtd pr at a review gate, or once nothing is underway.`
+  }
+  await start($)
+  return `Started a review of ${pr.url}; its ${threads.length} review thread(s) join the review.`
+}
+
+// Replies whose push failed are retried here, at a rest that sends them.
+async function retryReplies($: $, stop: Stop | undefined) {
+  // prRest, not a bare reply: threads of another branch must be dropped, never answered.
+  if (stop) {
+    await prRest($, stop)
+    if ((await read($, run)).pr === undefined) return "Done with the earlier review threads."
+  }
+  return "Review threads from an earlier /gtd pr still wait for their replies, sent at the next review gate or when the process ends; /gtd pr retries them there."
+}
+
+async function importThreads($: $) {
+  const pr = (await read($, run)).pr
+  if (!pr || pr.importedAt !== undefined) return
+  const path = `${root}/${REVIEW_FILE}`
+  const review = (await $.fs.exists(path)) ? await $.fs.read(path) : ""
+  await $.fs.write(path, withThreads(review, pr.threads))
+  const importedAt = await head($)
+  await update($, run, (x) => ({ ...x, pr: { ...pr, importedAt } }))
+}
+
+const forgetPr = ($: $) => update($, run, (x) => ({ ...x, pr: undefined }))
+
+// A failing import or reply is reported, never allowed to swallow the rest.
+async function guardedPr($: $, work: () => Promise<unknown>) {
+  try {
+    await work()
+  } catch (err) {
+    $.ui.log(`gtd pr: ${String(err)}`)
+    $.ui.toast("gtd pr: handling the review threads failed")
+  }
+}
+
+async function prRest($: $, stop: Stop) {
+  const pr = (await read($, run)).pr
+  if (!pr) return
+  const branch = (
+    await $.process.run(["git", "rev-parse", "--abbrev-ref", "HEAD"], { cwd: root })
+  ).stdout.trim()
+  const action = prAction(pr, {
+    isReviewGate: isReviewGate(stop),
+    isDone: stop.kind === "done",
+    branch,
+  })
+  if (action === "forget") await forgetPr($)
+  if (action === "import") await guardedPr($, () => importThreads($))
+  if (action === "reply") await guardedPr($, () => replyThreads($, stop))
+}
+
+// Once a gate answered the imported threads and the process rests at a review
+// gate again or is done, each thread is replied to and resolved. Only a failed
+// push keeps them for the next rest.
+async function replyThreads($: $, stop: Stop) {
+  const pr = (await read($, run)).pr
+  if (!pr?.importedAt) return
+  const io = shipIo($)
+  const found = await decided(
+    io,
+    pr.importedAt,
+    pr.threads.map((t) => t.id),
+  )
+  if (found === "gone") {
+    $.ui.log("gtd pr: the history moved since the review threads were added; they get no replies.")
+    await forgetPr($)
+    return
+  }
+  if (!found) {
+    if (stop.kind === "done") await forgetPr($)
+    return
+  }
+  // Nothing has acted on the answers yet, unless the process already ended.
+  if (stop.kind !== "done" && found.gate === (await head($))) return
+  $.ui.status("replying to the review threads…")
+  let replied
+  try {
+    replied = await reply(io, pr.host, found.gate, found.answers)
+  } finally {
+    $.ui.status(undefined)
+  }
+  for (const line of replied.text.split("\n")) $.ui.log(line)
+  $.ui.toast(replied.text.split("\n")[0]!)
+  if (replied.isPushed) await forgetPr($)
+}
+
 // Picks the process up where it was thrown: a rest that waits on a person is
 // shown to the catcher first, never landed unseen.
 async function catchNow($: $, ref: string) {
@@ -743,8 +873,15 @@ async function proceed(
   return undefined
 }
 
-// The rest's own `gtd validate` script, run.
+// The rest's own `gtd validate` script, run; a declined review thread needs
+// the reason its reply carries.
 async function validateRest($: $) {
+  const r = await read($, run)
+  if (r.pr?.importedAt !== undefined && isReviewGate(r.stop)) {
+    const path = `${root}/${REVIEW_FILE}`
+    const missing = (await $.fs.exists(path)) ? missingReasons(await $.fs.read(path)) : []
+    if (missing.length > 0) return { ok: false, out: missing.join("\n") }
+  }
   const script = await gtd($, ["validate"])
   if (script.trim().startsWith("nothing to validate")) return { ok: true, out: "" }
   const v = await $.process.run(["sh", "-c", script], { cwd: root, timeoutMs: TEN_MINUTES })
@@ -872,10 +1009,16 @@ async function command($: $, arg: string): Promise<string | undefined> {
     case "":
       return resume($)
     default:
-      return doorFor(await doors(shipIo($)), verb, rest) ? enterNow($, verb, rest) : begin($, arg)
+      return routed($, verb, rest, arg)
   }
 }
 
+// A door wins over the bare `pr` verb; anything else is requirements.
+async function routed($: $, verb: string, rest: string[], arg: string) {
+  if (doorFor(await doors(shipIo($)), verb, rest)) return enterNow($, verb, rest)
+  return verb === "pr" && rest.length === 0 ? prNow($) : begin($, arg)
+}
+
 async function resume($: $) {
   const b = await docs($).next()
   if (isTrue(b.idle)) {
@@ -1020,7 +1163,7 @@ export const register: Register = (on) => {
       name: "gtd",
       description: "Drive gtd until it rests on you; pass requirements to start a new process",
       argumentHint:
-        "[requirements | <door> [args] | stop | status | ui | ship [-n] | throw [@user] | catch <pr|branch>]",
+        "[requirements | <door> [args] | stop | status | ui | ship [-n] | pr | throw [@user] | catch <pr|branch>]",
       immediate: true,
     })
     return next(e)

package/claude/types/index.d.ts

@@ -10,6 +10,20 @@ export type Stop = {
   mode?: string
 }
 
+export type PrThread = {
+  id: string
+  path?: string
+  line?: number
+  comments: { author: string; body: string; url: string }[]
+}
+
+export type PrFeedback = {
+  threads: PrThread[]
+  host: string
+  branch: string
+  importedAt?: string
+}
+
 export type Run = {
   isRunning: boolean
   beat: number
@@ -33,6 +47,10 @@ export type Run = {
   inflight?: { agentId: string; memory: string }
   // a script (beat or landing) is running, so a reload must not run it again
   isScripting?: boolean
+  // `/gtd pr`'s review threads, for the pull request of `branch`;
+  // `importedAt` is HEAD when they were written into the review gate's file,
+  // and the replies wait on a gate after it
+  pr?: PrFeedback
 }
 
 declare module "claude-code" {

package/dist/chunks/{reserved-ulcSkND6.mjs → reserved-FWrjy7zo.mjs}

@@ -1,5 +1,5 @@
 import { accessFor, agent, answered, changesSince, check, codeThreads, defineWorkflow, displayPath, env, head, human, judge, mark, numeric, quote, read, refuse, scope, skillsFor, start, vars } from "../flows.mjs";
-import { d as ANSWER_FIX, f as ANSWER_LEAD, p as ANSWER_WONT_FIX, v as FREE_TEXT_PLACEHOLDER } from "./steering-D4LUQ-av.mjs";
+import { d as ANSWER_FIX, f as ANSWER_LEAD, p as ANSWER_WONT_FIX, v as FREE_TEXT_PLACEHOLDER } from "./steering-D7hWeMgE.mjs";
 import { escalation as escalation$1, gate as gate$1, healthy as healthy$1, lensScopes, lensesOf, removedAssertions } from "../workflow-kit.mjs";
 import { c as String$, g as standardSchemaV1, h as optionalWith, l as Struct, n as Boolean$, s as Number$ } from "./Schema-C6fLlQzx.mjs";
 //#region src/workflows/shared/registry.ts

package/dist/chunks/{review-Da2ndnB1.mjs → review-Dhl4vvDe.mjs}

@@ -1,7 +1,7 @@
 import { answered, changesSince, head, judge, numeric, read, removeScript, requireRevert, restoreScript, run, showScript, vars, writeScript } from "../flows.mjs";
 import { removedAssertions, review } from "../workflow-kit.mjs";
-import { At as conflictBody, B as composedReview, C as awaitReview, D as fixNits, E as fixCheck, Ft as parseStandards, I as buildReviewTriageMessage, It as renderStandards, J as finisherSystem, Lt as standardKey, Mt as conflictTitle, Nt as isConflictTitle, P as agentWithSkills, Pt as oneLine, S as answerReviewQuestions, T as collecting, b as REVIEW, f as guarded, ft as reviewEditNotesCapture, g as QUALITY, gt as reviewNotesCapture, j as reviewMissing, jt as conflictFor, k as fixRisks, kt as STANDARDS, mt as reviewFeedbackFixPrompt, p as healthy, pt as reviewEditsCapture, w as coder, y as REQUIREMENTS } from "./reserved-ulcSkND6.mjs";
-import { i as isOptionalObject, n as closeReview, o as tail, s as NO_FINDINGS } from "./tail-BW030xz_.mjs";
+import { At as conflictBody, B as composedReview, C as awaitReview, D as fixNits, E as fixCheck, Ft as parseStandards, I as buildReviewTriageMessage, It as renderStandards, J as finisherSystem, Lt as standardKey, Mt as conflictTitle, Nt as isConflictTitle, P as agentWithSkills, Pt as oneLine, S as answerReviewQuestions, T as collecting, b as REVIEW, f as guarded, ft as reviewEditNotesCapture, g as QUALITY, gt as reviewNotesCapture, j as reviewMissing, jt as conflictFor, k as fixRisks, kt as STANDARDS, mt as reviewFeedbackFixPrompt, p as healthy, pt as reviewEditsCapture, w as coder, y as REQUIREMENTS } from "./reserved-FWrjy7zo.mjs";
+import { i as isOptionalObject, n as closeReview, o as tail, s as NO_FINDINGS } from "./tail-B-Es6Hwj.mjs";
 //#region src/workflows/shared/learn.ts
 const SUMMARY_MAX = 200;
 const BANNER = /^_(?:The fixer left this unresolved\.|A fix turn removed this while renaming another finding — check whether it was fixed\.|No detail given\.)_\s*/;

package/dist/chunks/{steering-D4LUQ-av.mjs → steering-D7hWeMgE.mjs}

@@ -15901,7 +15901,7 @@ const reviewDescriptor = {
 	openThreads: openThreadsOf
 };
 const pointerLabel = (f) => `${f.path}${f.line === void 0 ? "" : `#${f.line}${f.rangeEnd === void 0 ? "" : `-${f.rangeEnd}`}`}`;
-const chunkNote = (chunk, prior) => chunk.description === (prior?.description ?? "") ? [] : [{
+const chunkNote = (chunk, prior) => chunk.description === (prior?.description ?? "") || prior === void 0 && chunk.answers !== void 0 ? [] : [{
 	line: chunk.headingLine,
 	note: {
 		kind: "chunk",

package/dist/chunks/{tail-BW030xz_.mjs → tail-B-Es6Hwj.mjs}

@@ -1,6 +1,6 @@
 import { answered, head, judge, mark, numeric, read, refuse, removeScript, run, scope, sectionBodies, showScript, vars, writeScript } from "../flows.mjs";
 import { fixFindings, lensEntriesOf, lensOf, qualityLap as qualityLap$1, reviewToHandOff } from "../workflow-kit.mjs";
-import { C as awaitReview, E as fixCheck, M as reviewQuality, N as reviewing, O as fixQuality, X as handOffAwaitReviewMessage, at as qualityTriageMessage, b as REVIEW, f as guarded, g as QUALITY, it as qualityFindingsFile, j as reviewMissing, k as fixRisks, p as healthy, u as escalation } from "./reserved-ulcSkND6.mjs";
+import { C as awaitReview, E as fixCheck, M as reviewQuality, N as reviewing, O as fixQuality, X as handOffAwaitReviewMessage, at as qualityTriageMessage, b as REVIEW, f as guarded, g as QUALITY, it as qualityFindingsFile, j as reviewMissing, k as fixRisks, p as healthy, u as escalation } from "./reserved-FWrjy7zo.mjs";
 //#region src/workflows/shared/lenses.ts
 const NO_FINDINGS = {
 	notes: [],

package/dist/steering.mjs

@@ -1,2 +1,2 @@
-import { C as parseFootnotes, D as headingSectionBodies, E as getParseCount, O as headingSections, S as openThreadFindings, T as parseThreadsWithFindings, _ as unansweredNotes, a as unansweredQuestions, b as FOOTNOTE_ACTION_TITLE, c as stripCodeThreads, d as ANSWER_FIX, f as ANSWER_LEAD, g as reviewRisks, h as reviewNotes, i as steeringFormatOrFreeForm, k as formatFinding, l as freeFormFormat, m as reviewDecisions, n as openThreads, o as viewOf, p as ANSWER_WONT_FIX, r as steeringFormatFor, s as parseCodeThreads, t as BUILT_IN_MODE_NAMES, u as applySteeringEdits, v as FREE_TEXT_PLACEHOLDER, w as parseThreads, x as THREAD_REPLY_ACTION_TITLE, y as isAnswered } from "./chunks/steering-D4LUQ-av.mjs";
+import { C as parseFootnotes, D as headingSectionBodies, E as getParseCount, O as headingSections, S as openThreadFindings, T as parseThreadsWithFindings, _ as unansweredNotes, a as unansweredQuestions, b as FOOTNOTE_ACTION_TITLE, c as stripCodeThreads, d as ANSWER_FIX, f as ANSWER_LEAD, g as reviewRisks, h as reviewNotes, i as steeringFormatOrFreeForm, k as formatFinding, l as freeFormFormat, m as reviewDecisions, n as openThreads, o as viewOf, p as ANSWER_WONT_FIX, r as steeringFormatFor, s as parseCodeThreads, t as BUILT_IN_MODE_NAMES, u as applySteeringEdits, v as FREE_TEXT_PLACEHOLDER, w as parseThreads, x as THREAD_REPLY_ACTION_TITLE, y as isAnswered } from "./chunks/steering-D7hWeMgE.mjs";
 export { ANSWER_FIX, ANSWER_LEAD, ANSWER_WONT_FIX, BUILT_IN_MODE_NAMES, FOOTNOTE_ACTION_TITLE, FREE_TEXT_PLACEHOLDER, THREAD_REPLY_ACTION_TITLE, applySteeringEdits, formatFinding, freeFormFormat, getParseCount, headingSectionBodies, headingSections, isAnswered, openThreadFindings, openThreads, parseCodeThreads, parseFootnotes, parseThreads, parseThreadsWithFindings, reviewDecisions, reviewNotes, reviewRisks, steeringFormatFor, steeringFormatOrFreeForm, stripCodeThreads, unansweredNotes, unansweredQuestions, viewOf };

package/dist/testkit.mjs

@@ -1,6 +1,6 @@
 import { answered, applyScript, checkScript, configProblems, configure, installContext, isFactoryWorkflow, moveScript, numeric, protocolMismatch, removeScript, restoreScript, revertScript, showScript, t as flows_exports, writeScript } from "./flows.mjs";
 import { A as beatKindOf, B as ACCESS_REFUSAL, C as renderLandPlain, D as renderBeatJson, E as beatDocument, O as statusOf, S as renderLandJson, T as renderBeatPlain, U as renderFormat, _ as selectPath, a as contextLimitFault, g as resolveSession, h as globMatches, j as demandOf, l as parseContextWindow, m as slicesDocument, o as contextShapeFault, p as renderSlicesJson, v as judgeJson, x as noopText, y as landFields } from "./chunks/wire-BJuZWR3C.mjs";
-import { D as headingSectionBodies, O as headingSections, T as parseThreadsWithFindings, a as unansweredQuestions, k as formatFinding, n as openThreads, r as steeringFormatFor, s as parseCodeThreads, t as BUILT_IN_MODE_NAMES } from "./chunks/steering-D4LUQ-av.mjs";
+import { D as headingSectionBodies, O as headingSections, T as parseThreadsWithFindings, a as unansweredQuestions, k as formatFinding, n as openThreads, r as steeringFormatFor, s as parseCodeThreads, t as BUILT_IN_MODE_NAMES } from "./chunks/steering-D7hWeMgE.mjs";
 import { $ as logWarning, $i as BaseProto, $n as map$5, $r as make$19, $t as provide$1, A as catchIf, Ai as add$1, An as scopeMake, Ar as flatMap$3, At as suspend$5, B as flatMap$4, Bi as getOrElse, Bn as get$3, Br as sync$1, Bt as unsafeMakeLatch, C as all$1, Ci as toArray, Cn as isInterrupted, Cr as exitFail, Ct as sandbox, D as async, Di as GenericTag, Dn as squash, Dr as exitZipRight, Dt as scopedWith$1, E as asVoid$1, Ei as unsafeHead, En as reduceWithContext, Er as exitVoid, Et as scoped$4, F as ensuring$2, Fi as fromIterable, Fn as join$3, Fr as map$3, Ft as tryMap, G as forkIn, Gi as match$4, Gn as TypeId$10, Gr as whenEffect, Gt as zipLeft, H as fnUntraced, Hi as isNone, Hn as modify$1, Hr as uninterruptible, Ht as when, I as exit, Ii as splitAt, In as sequential, Ir as onInterrupt, It as tryPromise, J as ignore, Ji as isLeft, Jn as all, Jr as DeferredTypeId$1, Jt as spaced$1, K as forkScoped, Ki as none, Kn as Class, Kr as withFiberRuntime, Kt as zipRight$3, L as fail$5, Li as filter$1, Ln as bind, Lr as scopeClose, Lt as try_, M as die$2, Mi as getOption, Mn as _await$2, Mr as interruptAsFiber, Mt as tap, N as dieMessage, Ni as omit, Nn as inheritAll$1, Nr as interruptFiber, Nt as tapErrorCause, O as catchAll$1, Oi as Reference, On as ensuring$3, Or as fiberId, Ot as serviceOption, P as either, Pi as empty$5, Pn as interruptAllAs$1, Pr as interruptible$1, Pt as timeout, Q as isEffect, Qi as pipeArguments, Qn as isFailure, Qr as ids, Qt as mergeAll$2, R as failCause$4, Ri as flatMap$5, Rn as bindTo, Rr as succeed$4, Rt as uninterruptible$1, S as addFinalizer$1, Si as takeRight, Sn as isDieType, Sr as exitAs, St as runtime, T as as, Ti as unsafeGet, Tn as map$6, Tr as exitSucceed, Tt as scopeWith, U as forEach$2, Ui as isSome, Un as set$1, Ur as uninterruptibleMask$1, Ut as withFiberRuntime$1, V as flatten$4, Vi as getOrUndefined, Vn as make$16, Vr as tap$1, Vt as unsafeMakeSemaphore$1, W as forkDaemon, Wi as map$8, Wn as update$1, Wr as void_$2, Wt as zip$1, X as interruptible, Xi as match$2, Xn as fail$4, Xr as combine, Xt as tagged, Y as interrupt$4, Yi as left, Yn as die, Yr as empty$6, Yt as TaggedError$1, Z as intoDeferred, Zi as right, Zn as failCause$3, Zr as combineAll, Zt as fromEffect$5, _ as decodeSync, _i as of, _n as fail$3, _r as deferredIsDone, _t as raceWith, a as Literal, aa as constFalse, ai as size$2, an as updateContext$1, ar as zipRight$2, at as matchCause, b as acquireRelease, bi as reverse, bn as interrupt$3, br as deferredUnsafeDone, bt as runPromise, c as String$, ca as constVoid, ci as append$1, cn as Scope, cr as asyncInterrupt, ct as negate, d as Union, da as pipe, di as empty$4, dn as close, dr as deferredAwait, dt as orElse, ea as NodeInspectSymbol, ei as none$1, en as provideMerge$1, er as mapBoth, et as makeSemaphore, f as between, fi as filter, fn as extend, fr as deferredComplete, ft as orElseSucceed, gi as map$7, gn as die$1, gr as deferredInterruptWith, gt as race, hi as isEmpty$2, hn as NoSuchElementException, hr as deferredInterrupt, ht as provideService, i as Defect, ia as hasProperty, ii as has, in as unsafeRunPromiseExit, ir as zip$2, it as match$5, j as catchTag, ji as get$5, jn as scopeTag, jr as interrupt$2, jt as sync$2, k as catchAllCause, ki as Tag, kn as forEachConcurrentDiscard, kr as fiberRefLocally, kt as succeed$6, l as Struct, la as dual, li as appendAll, ln as addFinalizer, lr as catchAll, lt as onError, m as optional$1, mi as isChunk, mn as make$18, mr as deferredFailCause, mt as provide$2, n as Boolean$, na as toJSON, ni as make$17, nn as succeed$3, nr as succeed$5, nt as mapBoth$1, o as Never, oa as constTrue, oi as decode, on as forkIn$1, or as as$1, ot as matchCauseEffect, p as decodeUnknown, pi as fromIterable$1, pn as fork, pr as deferredFail, pt as promise, q as gen, qi as some, qn as spanTag, qr as zipRight$1, qt as zipWith, r as Cause, ra as equals, ri as set$2, rn as unsafeFork, rr as void_$3, rt as mapError, s as Number$, sa as constUndefined, si as isFinite, sn as unsafeMakeSemaphore, sr as asVoid, st as matchEffect, t as Array$, ta as format, ti as get$4, tn as scoped$3, tr as match$3, tt as map$4, u as TaggedError, ua as identity, ui as drop, un as addFinalizerExit, ur as currentContext$1, ut as orDie, v as encodeSync, vi as prepend, vn as failureOrCause, vr as deferredMake, vt as repeat, w as annotateLogs, wi as unsafeFromArray, wn as isInterruptedOnly, wr as exitFailCause, wt as scope, x as acquireUseRelease$2, xi as take$2, xn as interruptors, xr as deferredUnsafeMake, xt as runPromiseExit$1, y as _void, yi as reduce, yn as flipCauseOption, yr as deferredSucceed, yt as retry, z as fiberIdWith, zi as fromNullable, zn as sleep, zr as suspend$4, zt as uninterruptibleMask } from "./chunks/Schema-C6fLlQzx.mjs";
 import { createRequire } from "node:module";
 import * as Crypto from "node:crypto";

package/dist/workflow-kit.mjs

@@ -1,5 +1,5 @@
 import { changes, changesSince, hasThreadFor, head, mark, read, refuse, requireReplies, requireThreadsClosed, scope } from "./flows.mjs";
-import { _ as unansweredNotes, c as stripCodeThreads, g as reviewRisks, h as reviewNotes, m as reviewDecisions, w as parseThreads } from "./chunks/steering-D4LUQ-av.mjs";
+import { _ as unansweredNotes, c as stripCodeThreads, g as reviewRisks, h as reviewNotes, m as reviewDecisions, w as parseThreads } from "./chunks/steering-D7hWeMgE.mjs";
 //#region src/workflow-kit/health.ts
 /** Hand a red suite that repeated attempts could not fix to a person, until `limit` rounds are spent. */
 const escalation = async (count, steps) => {

package/dist/workflows/cleanup.mjs

@@ -1,6 +1,6 @@
 import { agent, changes, glob, read, refuse, scope, start, vars } from "../flows.mjs";
-import { Et as envDefaults, Ht as requireFlag, Tt as defaults, Vt as registryEntry, i as skills, n as summary, o as flagShipped, r as access, t as steering, w as coder, zt as flagReferences } from "../chunks/reserved-ulcSkND6.mjs";
-import { a as refusePendingHandOff, r as handOffTail } from "../chunks/tail-BW030xz_.mjs";
+import { Et as envDefaults, Ht as requireFlag, Tt as defaults, Vt as registryEntry, i as skills, n as summary, o as flagShipped, r as access, t as steering, w as coder, zt as flagReferences } from "../chunks/reserved-FWrjy7zo.mjs";
+import { a as refusePendingHandOff, r as handOffTail } from "../chunks/tail-B-Es6Hwj.mjs";
 //#region src/workflows/cleanup/cleanup.ts
 const removingPrompt = (name, registry) => `Clean up the release flag "${name}", which is default-on: delete its entry in ${registry}, and remove every \`GTD_FLAG_${name.toUpperCase()}\` check from the shipped sources. Keep the flag-on path; delete code only the off path reached, including tests that pin the off path.`;
 const cleanupFlow = async () => {

package/dist/workflows/feature.mjs

@@ -1,8 +1,8 @@
 import { agent, answered, applyScript, changes, changesSince, check, codeChanges, env, hasThreadFor, head, human, judge, judgeDiffShape, kept, mark, matches, quote, read, refuse, removeScript, requireAnswers, requireProgress, requireReplies, requireThreadsClosed, revertScript, run, scope, slice, start, vars, wrote } from "../flows.mjs";
 import { lensEntriesOf } from "../workflow-kit.mjs";
-import { $ as implementFixPrompt, A as planner, Ct as unwindFailedMessage, D as fixNits, Dt as testFileGlobs, E as fixCheck, Et as envDefaults, F as buildReviewCollectingPrompt, G as designGateAnswerMessage, H as coverageQuestion, K as designSystem, L as builderSystem, P as agentWithSkills, Q as implementBuildingPrompt, R as clearCriteriaMessage, S as answerReviewQuestions, St as uncoveredMessage, T as collecting, Tt as defaults, U as coveringPrompt, V as coverageMessage, W as criterionLine, Z as idleMessage, _ as REFACTOR, _t as reviewerSystem, a as flagModeOn, bt as slicingPrompt, ct as refactorDroppedRefusal, d as gate, dt as refactorRules, et as naiveApplyFailure, g as QUALITY, h as HANDOFF, i as skills, lt as refactorFixPrompt, m as FEEDBACK, n as summary, nt as normaliseCriteriaScript, ot as qualityUnresolvedMessage, p as healthy, q as designTriagePrompt, r as access, rt as oversizedMessage, s as shipped, st as refactorBuildingPrompt, t as steering, tt as naivePassPrompt, ut as refactorNotePrompt, v as REFACTOR_NOTE, vt as seamByHandMessage, w as coder, wt as unwindFailure, x as SLICES, xt as startGateBlockedMessage, y as REQUIREMENTS, yt as seamSteerMessage, z as clearCriteriaQuestion } from "../chunks/reserved-ulcSkND6.mjs";
-import { a as refusePendingHandOff, o as tail } from "../chunks/tail-BW030xz_.mjs";
-import { a as reUnwind, i as isCode$1, r as humanReview, t as buildTail } from "../chunks/review-Da2ndnB1.mjs";
+import { $ as implementFixPrompt, A as planner, Ct as unwindFailedMessage, D as fixNits, Dt as testFileGlobs, E as fixCheck, Et as envDefaults, F as buildReviewCollectingPrompt, G as designGateAnswerMessage, H as coverageQuestion, K as designSystem, L as builderSystem, P as agentWithSkills, Q as implementBuildingPrompt, R as clearCriteriaMessage, S as answerReviewQuestions, St as uncoveredMessage, T as collecting, Tt as defaults, U as coveringPrompt, V as coverageMessage, W as criterionLine, Z as idleMessage, _ as REFACTOR, _t as reviewerSystem, a as flagModeOn, bt as slicingPrompt, ct as refactorDroppedRefusal, d as gate, dt as refactorRules, et as naiveApplyFailure, g as QUALITY, h as HANDOFF, i as skills, lt as refactorFixPrompt, m as FEEDBACK, n as summary, nt as normaliseCriteriaScript, ot as qualityUnresolvedMessage, p as healthy, q as designTriagePrompt, r as access, rt as oversizedMessage, s as shipped, st as refactorBuildingPrompt, t as steering, tt as naivePassPrompt, ut as refactorNotePrompt, v as REFACTOR_NOTE, vt as seamByHandMessage, w as coder, wt as unwindFailure, x as SLICES, xt as startGateBlockedMessage, y as REQUIREMENTS, yt as seamSteerMessage, z as clearCriteriaQuestion } from "../chunks/reserved-FWrjy7zo.mjs";
+import { a as refusePendingHandOff, o as tail } from "../chunks/tail-B-Es6Hwj.mjs";
+import { a as reUnwind, i as isCode$1, r as humanReview, t as buildTail } from "../chunks/review-Dhl4vvDe.mjs";
 import { posix } from "node:path";
 //#region src/workflows/shared/fences.ts
 const markFenced = (text) => {

package/dist/workflows/fix.mjs

@@ -1,7 +1,7 @@
 import { read, refuse, removeScript, run, scope, start } from "../flows.mjs";
-import { g as reviewRisks, h as reviewNotes } from "../chunks/steering-D4LUQ-av.mjs";
-import { Et as envDefaults, J as finisherSystem, P as agentWithSkills, Tt as defaults, Y as fixReviewNotesPrompt, b as REVIEW, i as skills, l as baseline, n as summary, r as access, s as shipped, t as steering, w as coder } from "../chunks/reserved-ulcSkND6.mjs";
-import { r as handOffTail, t as REVIEW_ORIG } from "../chunks/tail-BW030xz_.mjs";
+import { g as reviewRisks, h as reviewNotes } from "../chunks/steering-D7hWeMgE.mjs";
+import { Et as envDefaults, J as finisherSystem, P as agentWithSkills, Tt as defaults, Y as fixReviewNotesPrompt, b as REVIEW, i as skills, l as baseline, n as summary, r as access, s as shipped, t as steering, w as coder } from "../chunks/reserved-FWrjy7zo.mjs";
+import { r as handOffTail, t as REVIEW_ORIG } from "../chunks/tail-B-Es6Hwj.mjs";
 //#region src/workflows/fix/fix.ts
 /**
 * Fix every note a human left in a handed-off `.gtd/REVIEW.md` and every

package/dist/workflows/review.mjs

@@ -1,7 +1,7 @@
 import { read, refuse, start } from "../flows.mjs";
-import { Et as envDefaults, Tt as defaults, d as gate, ht as reviewGateBlockedMessage, i as skills, n as summary, r as access, s as shipped, t as steering, y as REQUIREMENTS } from "../chunks/reserved-ulcSkND6.mjs";
-import { a as refusePendingHandOff } from "../chunks/tail-BW030xz_.mjs";
-import { a as reUnwind, n as fixFeedback, t as buildTail } from "../chunks/review-Da2ndnB1.mjs";
+import { Et as envDefaults, Tt as defaults, d as gate, ht as reviewGateBlockedMessage, i as skills, n as summary, r as access, s as shipped, t as steering, y as REQUIREMENTS } from "../chunks/reserved-FWrjy7zo.mjs";
+import { a as refusePendingHandOff } from "../chunks/tail-B-Es6Hwj.mjs";
+import { a as reUnwind, n as fixFeedback, t as buildTail } from "../chunks/review-Dhl4vvDe.mjs";
 //#region src/workflows/review/review.ts
 /**
 * The feature's review of everything since `reviewBase` — a green gate, the

package/dist/workflows/rollout.mjs

@@ -1,5 +1,5 @@
 import { agent, changesSince, glob, head, mark, read, refuse, removeScript, run, scope, vars } from "../flows.mjs";
-import { Bt as flipProblems, Et as envDefaults, Ht as requireFlag, Rt as flagMentions, Tt as defaults, d as gate, i as skills, m as FEEDBACK, n as summary, o as flagShipped, r as access, t as steering, w as coder } from "../chunks/reserved-ulcSkND6.mjs";
+import { Bt as flipProblems, Et as envDefaults, Ht as requireFlag, Rt as flagMentions, Tt as defaults, d as gate, i as skills, m as FEEDBACK, n as summary, o as flagShipped, r as access, t as steering, w as coder } from "../chunks/reserved-FWrjy7zo.mjs";
 //#region src/workflows/rollout/rollout.ts
 const flippingPrompt = (name, registry) => `Roll out the release flag "${name}": in ${registry}, set \`defaultOn: true\` on its entry. Change nothing else — no other entry, no other field, no other file, no comments.`;
 const suiteBlockedMessage = () => "The suite is red after the rollout. Fix it by hand — a rollout may only touch the flag's registry entry — then land.";

package/dist/workflows/tests.mjs

@@ -1,7 +1,7 @@
 import { changes, changesSince, glob, read, refuse, removeScript, run, scope, sectionBodies, start, vars } from "../flows.mjs";
-import { E as fixCheck, Et as envDefaults, J as finisherSystem, Ot as testsDefaults, P as agentWithSkills, Tt as defaults, c as testsShipped, d as gate, i as skills, n as summary, p as healthy, r as access, t as steering, w as coder, y as REQUIREMENTS } from "../chunks/reserved-ulcSkND6.mjs";
-import { a as refusePendingHandOff, o as tail } from "../chunks/tail-BW030xz_.mjs";
-import { a as reUnwind, i as isCode, n as fixFeedback, r as humanReview } from "../chunks/review-Da2ndnB1.mjs";
+import { E as fixCheck, Et as envDefaults, J as finisherSystem, Ot as testsDefaults, P as agentWithSkills, Tt as defaults, c as testsShipped, d as gate, i as skills, n as summary, p as healthy, r as access, t as steering, w as coder, y as REQUIREMENTS } from "../chunks/reserved-FWrjy7zo.mjs";
+import { a as refusePendingHandOff, o as tail } from "../chunks/tail-B-Es6Hwj.mjs";
+import { a as reUnwind, i as isCode, n as fixFeedback, r as humanReview } from "../chunks/review-Dhl4vvDe.mjs";
 //#region src/workflows/tests/tests.ts
 const TESTS = ".gtd/TESTS.md";
 const RETIRE = ".gtd/RETIRE.md";

package/dist/workflows.mjs

@@ -16469,7 +16469,7 @@ const reviewDescriptor = {
 	openThreads: openThreadsOf
 };
 const pointerLabel = (f) => `${f.path}${f.line === void 0 ? "" : `#${f.line}${f.rangeEnd === void 0 ? "" : `-${f.rangeEnd}`}`}`;
-const chunkNote = (chunk, prior) => chunk.description === (prior?.description ?? "") ? [] : [{
+const chunkNote = (chunk, prior) => chunk.description === (prior?.description ?? "") || prior === void 0 && chunk.answers !== void 0 ? [] : [{
 	line: chunk.headingLine,
 	note: {
 		kind: "chunk",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pmelab/gtd",
-  "version": "45.1.0",
+  "version": "45.2.0",
   "private": false,
   "description": "Git-aware CLI that emits the next prompt for an autonomous coding agent based on the current repository state",
   "bin": {

package/src/steering/review.test.ts

@@ -1875,6 +1875,12 @@ describe("answer boxes", async () => {
     expect(reviewNotes(open, doc(note("Rounding", [" ", "x", ""])))).toEqual([])
   })
 
+  it("a note added with its own boxes is a decision, never also a note", () => {
+    const added = doc(note("Rounding", [" ", " ", ""]), note("PR thread", ["x", " ", ""]))
+    expect(reviewNotes(open, added)).toEqual([])
+    expect(reviewDecisions(open, added)).toMatchObject([{ anchor: "PR thread", choice: "fix" }])
+  })
+
   it("an unticked note, or one with two ticks, is unanswered", () => {
     expect(unansweredNotes(open)).toEqual([
       { title: "Rounding", line: 5, reason: "tick exactly one box" },

package/src/steering/review.ts

@@ -1177,8 +1177,11 @@ const pointerLabel = (f: ReviewFile): string =>
 
 type Found = { line: number; note: Omit<ReviewNote, "id" | "forced"> }
 
+// A chunk added with its own answer boxes (a driver's imported note) is
+// answered by its tick, never judged as prose as well.
 const chunkNote = (chunk: Changeset, prior: Changeset | undefined): Found[] =>
-  chunk.description === (prior?.description ?? "")
+  chunk.description === (prior?.description ?? "") ||
+  (prior === undefined && chunk.answers !== undefined)
     ? []
     : [
         {

package/src/testkit/steps.ts

@@ -140,6 +140,11 @@ Given(
   },
 )
 
+Given("{string} gets appended:", (world: TestkitWorld, path: string, text: string) => {
+  const content = world.repo().read(path) ?? ""
+  world.repo().write(path, `${content.trimEnd()}\n\n${text}\n`)
+})
+
 Given("the working tree is committed as {string}", (world: TestkitWorld, message: string) => {
   world.repo().commit(message)
 })

package/src/workflows/review/review-door.feature

@@ -156,3 +156,41 @@ Feature: the review door fixes its notes in the same process
       """
     And the working tree is committed as "chore: leftover requirements"
     Then opening door "review" is refused with ".gtd/REQUIREMENTS.md is left over"
+
+  Scenario: a note added with its own boxes is fixed as ticked, never judged
+    Given judge step "build.review.standards.judge" answers:
+      """
+      [{"id":"standard-1","answer":"no","p":0.9}]
+      """
+    And step "build.review.fix.nits.fixing" writes "src/calc.ts" with:
+      """
+      export const add = (a: number, b: number): number => a + b
+      """
+    When gtd opens door "review" with "HEAD~1"
+    And the process is driven
+    And the text "- [ ] won't fix" in ".gtd/REVIEW.md" is replaced with "- [x] won't fix"
+    And ".gtd/REVIEW.md" gets appended:
+      """
+      ## src/calc.ts:1 (thread PRRT_1)
+
+      > Annotate the return type.
+
+      Tick exactly one:
+
+      - [x] fix as suggested
+      - [ ] won't fix
+      - [ ] _your answer_
+      """
+    And gtd lands
+    And the process is driven
+    Then the process rests at "build.review.await-review"
+    And the commit subjects end with:
+      """
+      gtd(human): build.review.await-review → build.review.standards.judge
+      gtd(judge): build.review.standards.judge → build.review.fix.nits.fixing
+      gtd(agent): build.review.fix.nits.fixing → build.health.check
+      gtd(check): build.health.check → build.review.closing
+      gtd(check): build.review.closing → build.review.composing
+      gtd(check): build.review.composing → build.review.await-review
+      """
+    And "src/calc.ts" contains "): number =>"