toolroll 0.9.94 → 0.9.96

package/dist/workspace-ui.d.ts

@@ -106,7 +106,7 @@ export type NextAction = {
     label: string;
     /** Where the act lives — the page, not a URL, so every surface can
      * build its own link (a task page anchors, a row links out). */
-    kind: "open-result" | "open-review" | "open-task" | "open-run" | "open-pr";
+    kind: "open-result" | "open-review" | "open-task" | "open-findings" | "open-run" | "open-pr";
 };
 export type DisplayStatus = {
     /** The stable `data-work-status` token pages and tests key off. */
@@ -175,6 +175,8 @@ export type WorkFacts = {
         question: string;
         overdue: boolean;
     } | null;
+    /** Findings of this task's family that used their rounds and wait on a person (findings.ts). */
+    heldFindings?: number;
 };
 export type WorkStatus = DisplayStatus & {
     /** Which shortcut views list this row; `all` always does. */

package/dist/workspace-ui.js

@@ -470,7 +470,8 @@ function workStatusWordsOf(facts, resultDisplay) {
     const result = facts.state === "done" ? resultDisplay ?? resultStatusOf(facts.result, facts.publication) : null;
     const running = facts.liveRunId !== null || (dispatch?.condition === "running" && !REVIEW_TOKENS.has(dispatch.code));
     const question = (facts.state === "queued" || facts.state === "running" || facts.state === "done") ? facts.openDecision ?? null : null;
-    const needs = (dispatch !== null && !REVIEW_TOKENS.has(dispatch.code) && needsPerson(dispatch)) || question !== null;
+    const held = (facts.heldFindings ?? 0) > 0 && facts.state !== "cancelled";
+    const needs = (dispatch !== null && !REVIEW_TOKENS.has(dispatch.code) && needsPerson(dispatch)) || question !== null || held;
     const views = ["all"];
     if (needs)
         views.push("needs-you");
@@ -480,6 +481,10 @@ function workStatusWordsOf(facts, resultDisplay) {
         views.push("queued");
     if (facts.state === "done")
         views.push("completed");
+    const findingWords = (behind) => ({
+        token: "finding-held", label: "Needs you", detail: NEEDS.finding.sentence({}), tone: "attention", action: { label: NEEDS.finding.action.label, kind: "open-findings" }, views, rank: 0,
+        ...(behind === null ? {} : { diagnostics: [behind] }),
+    });
     const dispatchWords = (tone, rank, action) => ({
         token: dispatch?.code ?? "unknown",
         label: dispatch?.summary ?? "Status unknown",
@@ -505,6 +510,8 @@ function workStatusWordsOf(facts, resultDisplay) {
                 tone: "attention", action: { label: "Answer question", kind: "open-task" }, views, rank: 0,
                 diagnostics: [{ token: result.token, label: result.label, detail: result.detail, tone: result.tone }] };
         }
+        if (held && result.tone !== "problem")
+            return findingWords({ token: result.token, label: result.label, detail: result.detail, tone: result.tone });
         return { ...result, views, rank: needs ? 0 : 3 };
     }
     if (facts.state === "cancelled") {
@@ -519,6 +526,8 @@ function workStatusWordsOf(facts, resultDisplay) {
             }),
         };
     }
+    if (held && facts.state !== "failed")
+        return findingWords(dispatch === null ? null : { token: dispatch.code, label: dispatch.summary, detail: dispatch.detail, tone: running ? "live" : "attention" });
     if (running) {
         return dispatchWords("live", 1, { label: "Watch the build", kind: facts.liveRunId === null ? "open-task" : "open-run" });
     }

package/dist/worktree.js

@@ -22,6 +22,7 @@
  * synced folder like OneDrive or Dropbox, where a sync client racing an agent
  * over `.git` internals corrupts both.
  */
+import { INCIDENT_QUIET_MS, recentGitIncident } from "./git-incident.js";
 import { createHash, randomBytes } from "node:crypto";
 import { existsSync, lstatSync, readFileSync, writeFileSync, rmSync, realpathSync, mkdirSync, readdirSync, renameSync } from "node:fs";
 import { hostname } from "node:os";
@@ -33,7 +34,7 @@ import { HANDOFF_PREFIX, MAILBOX_SUFFIX, looksLikeProtocolFile, readMailbox } fr
 import { parseHandoff } from "./decision.js";
 import { diskBytes } from "./storage.js";
 import { markNeverIndex } from "./never-index.js";
-import { checkoutGit, teardownGit } from "./teardown-git.js";
+import { checkoutGit, gitIncidentRecord, teardownGit } from "./teardown-git.js";
 /**
  * The note a lease leaves in the checkout, naming the process holding it.
  *
@@ -853,6 +854,9 @@ export class WorktreePool {
      * or remote. Reads only, through `git` when given (teardown's hardened git).
      */
     async inspect(path, git = this.teardown) {
+        // A checkout that had a git incident lately is left alone (git-incident.ts): nothing re-runs git there each pass.
+        if (quietAfterIncident(path))
+            return "unreadable";
         const dirty = await this.isDirty(path, true, git);
         if (dirty !== false)
             return dirty === null ? "unreadable" : "has changes";
@@ -870,6 +874,8 @@ export class WorktreePool {
      * `heads` commit is on a branch, and no ref lives only in this worktree. Reads only; anything unreadable keeps it.
      */
     async proveRecreatable(row, heads) {
+        if (quietAfterIncident(row.path))
+            return "unreadable";
         if (row.branch === "unknown" || !insidePool(this.options.root, row.path) || heads.some(one => !/^[0-9a-f]{40,64}$/.test(one)))
             return "unreadable";
         const git = teardownGit(this.runner, { repo: row.repo });
@@ -1120,3 +1126,19 @@ function firstLine(text) {
     const [line = ""] = text.trim().split("\n");
     return line;
 }
+/** Whether a checkout had a git incident within INCIDENT_QUIET_MS (by its real path, as incidents record it). */
+function quietAfterIncident(path) {
+    let real;
+    try {
+        real = realpathSync(path);
+    }
+    catch {
+        return false;
+    }
+    try {
+        return recentGitIncident(gitIncidentRecord(), real, new Date(Date.now() - INCIDENT_QUIET_MS));
+    }
+    catch {
+        return false;
+    }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "toolroll",
-  "version": "0.9.94",
+  "version": "0.9.96",
   "description": "A control plane for unattended coding agents — queue tasks, walk away, come back to pull requests. Agents build in leased worktrees, never touch your default branch, and interrupt you only for decisions that need a human.",
   "directories": {
     "doc": "docs"
@@ -133,6 +133,7 @@
   },
   "dependencies": {
     "@anthropic-ai/sandbox-runtime": "0.0.79",
+    "@json-render/core": "0.22.0",
     "@slack/socket-mode": "3.1.1",
     "@xterm/addon-serialize": "0.14.0",
     "@xterm/headless": "6.0.0",