@nickmeriano/task 0.7.1 → 0.9.0

package/src/claim.test.ts

@@ -6,8 +6,8 @@
  * 1. Two concurrent `task claim <same-id>`: exactly one exits 0, the other
  *    exits 1 — git's atomic ref creation is the lock.
  * 2. Validation failures exit 2 before anything touches git.
- * 3. `task list --claimable` is the queue: position order, minus blocked,
- *    needs-human, and already-claimed tickets.
+ * 3. `task list --claimable` is the queue: position order, minus blocked
+ *    and already-claimed tickets — open asks never exclude a ticket.
  * 4. `task claim --release` deletes the branch both sides and the ticket is
  *    claimable again.
  */
@@ -15,7 +15,7 @@
 import assert from "node:assert/strict"
 import { test } from "node:test"
 import { execFile, spawnSync } from "node:child_process"
-import { mkdtempSync, rmSync, writeFileSync } from "node:fs"
+import { mkdirSync, mkdtempSync, rmSync, writeFileSync } from "node:fs"
 import { readFileSync } from "node:fs"
 import { tmpdir } from "node:os"
 import { join } from "node:path"
@@ -61,20 +61,22 @@ function cli(cwd: string, ...args: string[]): Promise<CliResult> {
 interface Fixture {
   bare: string
   clone: () => string
+  /** Keys of the seeded tickets, by role. */
+  keys: { one: string; two: string; blocked: string; person: string; parked: string; jumper: string }
 }
 
 /**
  * A bare origin holding a board with the claimable shapes:
- *   1 "Ready one"    todo
- *   2 "Ready two"    todo
- *   3 "Blocked"      todo, blocked by 1
- *   4 "For a person" todo, needs-human
- *   5 "Not groomed"  backlog
- *   6 "Jumped queue" todo, moved to the top of the column after creation —
- *                    claimable order must be [6, 1, 2], proving position
- *                    order beats number order.
+ *   one     "Ready one"    todo
+ *   two     "Ready two"    todo
+ *   blocked "Blocked"      todo, blocked by `one`
+ *   person  "For a person" todo, carrying an open ask
+ *   parked  "Still parked" backlog
+ *   jumper  "Jumped queue" todo, moved to the top of the column after
+ *           creation — claimable order must be [jumper, one, two], proving
+ *           position order beats creation order.
  */
-function fixture(claimPrefix?: string): Fixture {
+function fixture(branchPrefix?: string): Fixture {
   const home = tempDir()
   const bare = join(home, "origin.git")
   sh(home, "git", "init", "--quiet", "--bare", "-b", "main", bare)
@@ -84,20 +86,29 @@ function fixture(claimPrefix?: string): Fixture {
   sh(seed, "git", "config", "user.name", "Test")
   sh(seed, "git", "config", "user.email", "test@example.com")
   const store = initProject(seed, { name: "claim board", prefix: "CLM" })
-  if (claimPrefix !== undefined) {
+  if (branchPrefix !== undefined) {
     const configPath = join(seed, ".task", "config.json")
     const config = JSON.parse(readFileSync(configPath, "utf8")) as Record<string, unknown>
-    config.claimPrefix = claimPrefix
+    config.claims = { branchPrefix }
     writeFileSync(configPath, `${JSON.stringify(config, null, 2)}\n`)
   }
-  store.create({ title: "Ready one" })
-  store.create({ title: "Ready two" })
-  store.create({ title: "Blocked", blockedBy: [1] })
-  store.create({ title: "For a person", needsHuman: true })
-  store.create({ title: "Not groomed", status: "backlog" })
+  const one = store.create({ title: "Ready one", status: "todo" })
+  const two = store.create({ title: "Ready two", status: "todo" })
+  const blocked = store.create({ title: "Blocked", status: "todo", blockedBy: [one.key] })
+  const person = store.create({ title: "For a person", status: "todo" })
+  store.addAsk(person.key, "Approve the vendor contract", "claude")
+  const parked = store.create({ title: "Still parked", status: "backlog" })
   const jumper = store.create({ title: "Jumped queue", status: "backlog" })
   // A status move without an explicit slot lands on top of the new column.
-  store.update(jumper.number, { status: "todo" })
+  store.update(jumper.key, { status: "todo" })
+  const keys = {
+    one: one.key,
+    two: two.key,
+    blocked: blocked.key,
+    person: person.key,
+    parked: parked.key,
+    jumper: jumper.key,
+  }
   sh(seed, "git", "add", "-A")
   sh(seed, "git", "commit", "--quiet", "-m", "board")
   sh(seed, "git", "push", "--quiet", "-u", "origin", "main")
@@ -110,15 +121,15 @@ function fixture(claimPrefix?: string): Fixture {
     sh(dir, "git", "config", "user.email", "test@example.com")
     return dir
   }
-  return { bare, clone }
+  return { bare, clone, keys }
 }
 
 test("two concurrent claims of one ticket: exactly one wins", async () => {
-  const { bare, clone } = fixture()
+  const { bare, clone, keys } = fixture()
   const a = clone()
   const b = clone()
 
-  const [ra, rb] = await Promise.all([cli(a, "claim", "1"), cli(b, "claim", "1")])
+  const [ra, rb] = await Promise.all([cli(a, "claim", keys.one), cli(b, "claim", keys.one)])
   assert.deepEqual([ra.code, rb.code].sort(), [0, 1], `a: ${ra.stderr}\nb: ${rb.stderr}`)
 
   const winner = ra.code === 0 ? a : b
@@ -126,117 +137,336 @@ test("two concurrent claims of one ticket: exactly one wins", async () => {
   assert.match(loser.stderr, /already claimed|rejected/)
 
   // The winner is left on the claim branch, flip committed and pushed.
-  assert.equal(sh(winner, "git", "rev-parse", "--abbrev-ref", "HEAD"), "task/claim/clm-1")
+  const branch = `task/claim/clm-${keys.one}`
+  assert.equal(sh(winner, "git", "rev-parse", "--abbrev-ref", "HEAD"), branch)
   assert.match(
-    readFileSync(join(winner, ".task", "tickets", "1", "ticket.md"), "utf8"),
+    readFileSync(join(winner, ".task", "tickets", keys.one, "ticket.md"), "utf8"),
     /status: in_progress/,
   )
-  assert.match(
-    sh(winner, "git", "ls-remote", "--heads", "origin", "task/claim/clm-1"),
-    /refs\/heads\/task\/claim\/clm-1/,
-  )
+  assert.ok(sh(winner, "git", "ls-remote", "--heads", "origin", branch).includes(`refs/heads/${branch}`))
   // Origin's default branch never saw the flip — the claim rides the branch.
   assert.equal(sh(winner, "git", "diff", "--stat", "origin/main..HEAD", "--", "README.md"), "")
-  assert.match(sh(winner, "git", "show", "origin/main:.task/tickets/1/ticket.md"), /status: todo/)
+  assert.match(
+    sh(winner, "git", "show", `origin/main:.task/tickets/${keys.one}/ticket.md`),
+    /status: todo/,
+  )
   void bare
 })
 
 test("unclaimable tickets exit 2, before anything touches origin", async () => {
-  const { clone } = fixture()
+  const { clone, keys } = fixture()
   const dir = clone()
 
-  assert.equal((await cli(dir, "claim", "5")).code, 2) // backlog
-  assert.equal((await cli(dir, "claim", "3")).code, 2) // blocked
-  assert.equal((await cli(dir, "claim", "4")).code, 2) // needs a human
-  assert.equal((await cli(dir, "claim", "99")).code, 2) // missing
-  assert.match((await cli(dir, "claim", "3")).stderr, /blocked by CLM-1/)
+  assert.equal((await cli(dir, "claim", keys.parked)).code, 2) // backlog
+  assert.equal((await cli(dir, "claim", keys.blocked)).code, 2) // blocked
+  assert.equal((await cli(dir, "claim", "zz9zz")).code, 2) // missing
+  assert.match(
+    (await cli(dir, "claim", keys.blocked)).stderr,
+    new RegExp(`blocked by CLM-${keys.one}`),
+  )
 
-  // A dirty tree refuses too — claiming switches branches.
-  writeFileSync(join(dir, "scratch.txt"), "wip")
-  const dirty = await cli(dir, "claim", "2")
+  // A dirty tree refuses too — claiming switches branches. Only *tracked*
+  // modifications count: untracked files ride a branch switch untouched
+  // (release deliberately leaves one — the reason comment).
+  const gitignore = join(dir, ".task", ".gitignore")
+  writeFileSync(gitignore, `${readFileSync(gitignore, "utf8")}\nwip\n`)
+  const dirty = await cli(dir, "claim", keys.two)
   assert.equal(dirty.code, 2)
   assert.match(dirty.stderr, /working tree is dirty/)
-  rmSync(join(dir, "scratch.txt"))
-  assert.equal((await cli(dir, "claim", "2")).code, 0)
+  sh(dir, "git", "checkout", "--", ".task/.gitignore")
+  writeFileSync(join(dir, "scratch.txt"), "untracked is fine")
+  assert.equal((await cli(dir, "claim", keys.two)).code, 0)
 
   // Nothing above created a stray claim branch for the failed ids.
   const remote = sh(dir, "git", "ls-remote", "--heads", "origin", "task/claim/*")
   assert.deepEqual(
     remote.split("\n").map((l) => l.split("refs/heads/")[1]),
-    ["task/claim/clm-2"],
+    [`task/claim/clm-${keys.two}`],
   )
 })
 
 test("list --claimable is the queue: position order, minus everything unclaimable", async () => {
-  const { clone } = fixture()
+  const { clone, keys } = fixture()
   const a = clone()
-  assert.equal((await cli(a, "claim", "6")).code, 0)
+  assert.equal((await cli(a, "claim", keys.jumper)).code, 0)
 
   const b = clone()
   const listed = await cli(b, "list", "--claimable", "--json")
   assert.equal(listed.code, 0)
-  const { tasks } = JSON.parse(listed.stdout) as { tasks: { number: number }[] }
-  // 6 claimed, 3 blocked, 4 needs-human, 5 not todo — position order keeps
-  // 1 before 2 (creation order within the column).
-  assert.deepEqual(tasks.map((t) => t.number), [1, 2])
+  const { tasks } = JSON.parse(listed.stdout) as { tasks: { key: string }[] }
+  // jumper claimed, blocked blocked, parked not todo — position order keeps
+  // one before two (creation order within the column). person carries an open
+  // ask and is claimable anyway: asks gate done, not claim.
+  assert.deepEqual(tasks.map((t) => t.key), [keys.one, keys.two, keys.person])
 
   assert.equal((await cli(b, "list", "--claimable", "--status", "todo")).code, 1)
 })
 
 test("release deletes the branch everywhere and reopens the claim", async () => {
-  const { clone } = fixture()
+  const { clone, keys } = fixture()
   const dir = clone()
-  assert.equal((await cli(dir, "claim", "2")).code, 0)
-  assert.equal((await cli(dir, "claim", "2")).code, 1) // locally claimed too
+  assert.equal((await cli(dir, "claim", keys.two)).code, 0)
+  assert.equal((await cli(dir, "claim", keys.two)).code, 1) // locally claimed too
 
-  const released = await cli(dir, "claim", "--release", "2")
+  const released = await cli(dir, "claim", "--release", keys.two, "--comment", "trying another shape")
   assert.equal(released.code, 0, released.stderr)
-  assert.match(released.stdout, /released CLM-2/)
+  assert.ok(released.stdout.includes(`released CLM-${keys.two}`), released.stdout)
 
   // Back on the default branch, branch gone on both sides, ticket todo again.
   assert.equal(sh(dir, "git", "rev-parse", "--abbrev-ref", "HEAD"), "main")
-  assert.equal(sh(dir, "git", "ls-remote", "--heads", "origin", "task/claim/clm-2"), "")
+  assert.equal(sh(dir, "git", "ls-remote", "--heads", "origin", `task/claim/clm-${keys.two}`), "")
   assert.match(
-    readFileSync(join(dir, ".task", "tickets", "2", "ticket.md"), "utf8"),
+    readFileSync(join(dir, ".task", "tickets", keys.two, "ticket.md"), "utf8"),
     /status: todo/,
   )
-  assert.equal((await cli(dir, "claim", "2")).code, 0)
+  assert.equal((await cli(dir, "claim", keys.two)).code, 0)
 
   // Releasing an unclaimed ticket is a quiet no-op, not an error.
-  const noop = await cli(dir, "claim", "--release", "1")
+  const noop = await cli(dir, "claim", "--release", keys.one, "--comment", "noop")
   assert.equal(noop.code, 0)
   assert.match(noop.stdout, /wasn't claimed/)
 })
 
-test("a configured claimPrefix moves the whole namespace", async () => {
-  const { clone } = fixture("claude/task/")
+test("a configured claims.branchPrefix moves the whole namespace", async () => {
+  const { clone, keys } = fixture("claude/task/")
   const dir = clone()
 
-  assert.equal((await cli(dir, "claim", "1")).code, 0)
-  assert.equal(sh(dir, "git", "rev-parse", "--abbrev-ref", "HEAD"), "claude/task/clm-1")
-  assert.match(
-    sh(dir, "git", "ls-remote", "--heads", "origin", "claude/task/*"),
-    /refs\/heads\/claude\/task\/clm-1/,
+  assert.equal((await cli(dir, "claim", keys.one)).code, 0)
+  assert.equal(sh(dir, "git", "rev-parse", "--abbrev-ref", "HEAD"), `claude/task/clm-${keys.one}`)
+  assert.ok(
+    sh(dir, "git", "ls-remote", "--heads", "origin", "claude/task/*").includes(
+      `refs/heads/claude/task/clm-${keys.one}`,
+    ),
   )
 
   // Every side of the feature reads the same key: the queue looks for claims
   // under the configured namespace, and release deletes there too.
   const other = clone()
   const listed = await cli(other, "list", "--claimable", "--json")
-  const { tasks } = JSON.parse(listed.stdout) as { tasks: { number: number }[] }
-  assert.deepEqual(tasks.map((t) => t.number), [6, 2])
-  assert.equal((await cli(other, "claim", "1")).code, 1)
-  assert.equal((await cli(dir, "claim", "--release", "1")).code, 0)
+  const { tasks } = JSON.parse(listed.stdout) as { tasks: { key: string }[] }
+  assert.deepEqual(tasks.map((t) => t.key), [keys.jumper, keys.two, keys.person])
+  assert.equal((await cli(other, "claim", keys.one)).code, 1)
+  assert.equal(
+    (await cli(dir, "claim", "--release", keys.one, "--comment", "handing it back")).code,
+    0,
+  )
   assert.equal(sh(dir, "git", "ls-remote", "--heads", "origin", "claude/task/*"), "")
 
   // A missing trailing slash is forgiven; a malformed value fails loudly.
   const bad = clone()
   const configPath = join(bad, ".task", "config.json")
   const config = JSON.parse(readFileSync(configPath, "utf8")) as Record<string, unknown>
-  config.claimPrefix = "spaces are not refs/"
+  config.claims = { branchPrefix: "spaces are not refs/" }
   writeFileSync(configPath, `${JSON.stringify(config, null, 2)}\n`)
   sh(bad, "git", "commit", "--quiet", "-am", "bad prefix")
-  const invalid = await cli(bad, "claim", "2")
+  const invalid = await cli(bad, "claim", keys.two)
   assert.equal(invalid.code, 2)
-  assert.match(invalid.stderr, /invalid claimPrefix/)
+  assert.match(invalid.stderr, /invalid claims\.branchPrefix/)
+
+  // The retired pre-0.8 `claimPrefix` spelling is not honored (TAS-41): a
+  // board carrying only it refuses by name rather than silently defaulting
+  // the lock convention out from under its workers.
+  const retired = clone()
+  const retiredPath = join(retired, ".task", "config.json")
+  const retiredConfig = JSON.parse(readFileSync(retiredPath, "utf8")) as Record<string, unknown>
+  delete retiredConfig.claims
+  retiredConfig.claimPrefix = "claude/task/"
+  writeFileSync(retiredPath, `${JSON.stringify(retiredConfig, null, 2)}\n`)
+  sh(retired, "git", "commit", "--quiet", "-am", "retired key")
+  const refused = await cli(retired, "claim", keys.two)
+  assert.equal(refused.code, 2)
+  assert.match(refused.stderr, /pre-0\.8 spelling and is no longer read/)
+})
+
+/**
+ * A monorepo with two boards for the multi-board selection paths (TAS-27):
+ * a root board (ROO, one ready ticket) and a nested one under pkg/ (NES,
+ * one ready ticket). The root config carries the new `claims` block and,
+ * optionally, the top-level `boards` list — NES deliberately ahead of ROO,
+ * so priority provably follows list order, not nearness.
+ */
+interface MultiFixture {
+  bare: string
+  clone: () => string
+  rootKey: string
+  nestedKey: string
+}
+
+function multiFixture(options: { boards?: string[]; maxOpenCount?: number } = {}): MultiFixture {
+  const home = tempDir()
+  const bare = join(home, "origin.git")
+  sh(home, "git", "init", "--quiet", "--bare", "-b", "main", bare)
+
+  const seed = join(home, "seed")
+  sh(home, "git", "clone", "--quiet", bare, seed)
+  sh(seed, "git", "config", "user.name", "Test")
+  sh(seed, "git", "config", "user.email", "test@example.com")
+
+  const root = initProject(seed, { name: "root board", prefix: "ROO" })
+  const configPath = join(seed, ".task", "config.json")
+  const config = JSON.parse(readFileSync(configPath, "utf8")) as Record<string, unknown>
+  if (options.boards) config.boards = options.boards
+  config.claims = { branchPrefix: "task/claim/", maxOpenCount: options.maxOpenCount }
+  writeFileSync(configPath, `${JSON.stringify(config, null, 2)}\n`)
+  const rootKey = root.create({ title: "Root ready", status: "todo" }).key
+
+  mkdirSync(join(seed, "pkg"), { recursive: true })
+  const nested = initProject(join(seed, "pkg"), { name: "nested board", prefix: "NES" })
+  const nestedKey = nested.create({ title: "Nested ready", status: "todo" }).key
+
+  sh(seed, "git", "add", "-A")
+  sh(seed, "git", "commit", "--quiet", "-m", "boards")
+  sh(seed, "git", "push", "--quiet", "-u", "origin", "main")
+
+  let clones = 0
+  const clone = (): string => {
+    const dir = join(home, `clone-${++clones}`)
+    sh(home, "git", "clone", "--quiet", bare, dir)
+    sh(dir, "git", "config", "user.name", "Test")
+    sh(dir, "git", "config", "user.email", "test@example.com")
+    return dir
+  }
+  return { bare, clone, rootKey, nestedKey }
+}
+
+test("claim --next drains the queue in position order and exits 1 when empty", async () => {
+  const { clone, keys } = fixture()
+  const dir = clone()
+
+  const order: string[] = []
+  for (;;) {
+    const result = await cli(dir, "claim", "--next", "--json")
+    if (result.code === 1) {
+      assert.equal(JSON.parse(result.stdout).task, null)
+      break
+    }
+    assert.equal(result.code, 0, result.stderr)
+    const { task, branch } = JSON.parse(result.stdout) as { task: { key: string }; branch: string }
+    order.push(task.key)
+    assert.equal(sh(dir, "git", "rev-parse", "--abbrev-ref", "HEAD"), branch)
+  }
+  // The jumper leads (position order), then creation order; blocked and
+  // backlog tickets never surface, while the open ask on `person` doesn't
+  // keep it out of the queue.
+  assert.deepEqual(order, [keys.jumper, keys.one, keys.two, keys.person])
+})
+
+test("two concurrent claim --next calls land on different tickets", async () => {
+  const { clone } = fixture()
+  const a = clone()
+  const b = clone()
+
+  const [ra, rb] = await Promise.all([
+    cli(a, "claim", "--next", "--json"),
+    cli(b, "claim", "--next", "--json"),
+  ])
+  assert.equal(ra.code, 0, ra.stderr)
+  assert.equal(rb.code, 0, rb.stderr)
+  const ids = [ra, rb].map((r) => (JSON.parse(r.stdout) as { task: { id: string } }).task.id)
+  assert.notEqual(ids[0], ids[1], `both claimed ${ids[0]}`)
+})
+
+test("--next walks the root boards list in priority order; --board overrides it", async () => {
+  const { clone, rootKey, nestedKey } = multiFixture({ boards: ["NES", "ROO"] })
+  const a = clone()
+
+  // list --claimable reads the same scope, in the same order.
+  const listed = await cli(a, "list", "--claimable", "--json")
+  assert.equal(listed.code, 0, listed.stderr)
+  const { tasks } = JSON.parse(listed.stdout) as { tasks: { id: string }[] }
+  assert.deepEqual(tasks.map((t) => t.id), [`NES-${nestedKey}`, `ROO-${rootKey}`])
+
+  // The nested board wins despite the root board being nearest.
+  const next = await cli(a, "claim", "--next", "--json")
+  assert.equal(next.code, 0, next.stderr)
+  const claimed = JSON.parse(next.stdout) as { task: { id: string }; branch: string }
+  assert.equal(claimed.task.id, `NES-${nestedKey}`)
+  assert.equal(claimed.branch, `task/claim/nes-${nestedKey}`)
+
+  // An explicit --board list is the whole scope, in the given order.
+  const b = clone()
+  const overridden = await cli(b, "claim", "--next", "--board", "ROO", "--json")
+  assert.equal(overridden.code, 0, overridden.stderr)
+  assert.equal((JSON.parse(overridden.stdout) as { task: { id: string } }).task.id, `ROO-${rootKey}`)
+
+  const unknown = await cli(b, "claim", "--next", "--board", "NOPE")
+  assert.equal(unknown.code, 2)
+  assert.match(unknown.stderr, /no board with prefix NOPE/)
+})
+
+test("without a boards list, --next sees the nearest board only", async () => {
+  const { clone, rootKey } = multiFixture()
+  const dir = clone()
+
+  const next = await cli(dir, "claim", "--next", "--json")
+  assert.equal(next.code, 0, next.stderr)
+  assert.equal((JSON.parse(next.stdout) as { task: { id: string } }).task.id, `ROO-${rootKey}`)
+
+  // ROO's queue is drained and NES never entered the scope.
+  const empty = await cli(dir, "claim", "--next", "--json")
+  assert.equal(empty.code, 1)
+})
+
+test("maxOpenCount caps every claim repo-wide; --force is the escape", async () => {
+  const { clone, rootKey } = multiFixture({ boards: ["NES", "ROO"], maxOpenCount: 1 })
+  const a = clone()
+  assert.equal((await cli(a, "claim", "--next", "--json")).code, 0)
+
+  // At the cap, --next and explicit-id claims both refuse with exit 2 —
+  // the fix is to finish or release a claim, not to pick another ticket.
+  const b = clone()
+  const capped = await cli(b, "claim", "--next")
+  assert.equal(capped.code, 2)
+  assert.match(capped.stderr, /maxOpenCount cap \(1\)/)
+  const explicit = await cli(b, "claim", `ROO-${rootKey}`)
+  assert.equal(explicit.code, 2)
+  assert.match(explicit.stderr, /maxOpenCount cap \(1\)/)
+
+  const forced = await cli(b, "claim", "--next", "--force", "--json")
+  assert.equal(forced.code, 0, forced.stderr)
+  assert.equal((JSON.parse(forced.stdout) as { task: { id: string } }).task.id, `ROO-${rootKey}`)
+})
+
+test("release requires a reason and leaves it as a comment on the ticket", async () => {
+  const { clone, keys } = fixture()
+  const dir = clone()
+  assert.equal((await cli(dir, "claim", keys.one)).code, 0)
+
+  // No comment, and an empty one, are both refused before anything happens.
+  const missing = await cli(dir, "claim", "--release", keys.one)
+  assert.equal(missing.code, 1)
+  assert.match(missing.stderr, /requires --comment/)
+  const empty = await cli(dir, "claim", "--release", keys.one, "--comment", "")
+  assert.equal(empty.code, 1)
+  assert.match(empty.stderr, /requires --comment/)
+  assert.ok(
+    sh(dir, "git", "ls-remote", "--heads", "origin", `task/claim/clm-${keys.one}`).includes(
+      `clm-${keys.one}`,
+    ),
+  )
+
+  const released = await cli(
+    dir, "claim", "--release", keys.one,
+    "--comment", "the migration kept flaking — needs a fixture, not a retry",
+    "--author", "claude",
+  )
+  assert.equal(released.code, 0, released.stderr)
+
+  // The reason survives the branch: an ordinary attributed comment, sitting
+  // uncommitted in the working tree on the default branch.
+  const shown = await cli(dir, "show", keys.one, "--json")
+  const { comments } = JSON.parse(shown.stdout) as {
+    comments: { author: string; body: string }[]
+  }
+  assert.equal(comments.length, 1)
+  assert.equal(comments[0].author, "claude")
+  assert.match(comments[0].body, /kept flaking/)
+  assert.ok(sh(dir, "git", "status", "--porcelain").includes(`tickets/${keys.one}/comments`))
+
+  // Releasing an unclaimed ticket stays a no-op — no second comment.
+  const noop = await cli(dir, "claim", "--release", keys.one, "--comment", "again")
+  assert.equal(noop.code, 0)
+  assert.match(noop.stdout, /wasn't claimed/)
+  const after = await cli(dir, "show", keys.one, "--json")
+  assert.equal((JSON.parse(after.stdout) as { comments: unknown[] }).comments.length, 1)
 })