@nanobpm/nano-workforce 0.195.1 → 0.196.0

package/pages/home.page.json

@@ -173,6 +173,17 @@
               "keyField": "process_key"
             }
           },
+          {
+            "field": "process_key",
+            "header": "Agent",
+            "template": "Transcript",
+            "width": "7rem",
+            "link": {
+              "kind": "page",
+              "page": "cockpit",
+              "keyField": "process_key"
+            }
+          },
           {
             "field": "incident_message",
             "header": "Incident",

package/pages/lineage.page.json

@@ -122,6 +122,7 @@
             "truncate": true,
             "link": { "kind": "processExplorer", "keyField": "process_key" }
           },
+          { "field": "process_key", "header": "Agent", "template": "Transcript", "width": "7rem", "link": { "kind": "page", "page": "cockpit", "keyField": "process_key" } },
           { "field": "pr_count", "header": "PRs", "width": "5rem" },
           { "field": "updated_at", "header": "Updated", "width": "9rem", "format": "datetime" }
         ],
@@ -145,6 +146,7 @@
               "columns": [
                 { "field": "title", "template": "{{title}}", "header": "PR", "subtitleField": "pr_key", "truncate": true, "linkField": "url" },
                 { "field": "status", "header": "Status", "link": { "kind": "processExplorer", "keyField": "process_key" } },
+                { "field": "process_key", "header": "Agent", "template": "Transcript", "width": "7rem", "link": { "kind": "page", "page": "cockpit", "keyField": "process_key" } },
                 { "field": "current_round", "header": "Round" },
                 { "field": "outcome", "header": "Outcome", "truncate": true },
                 { "field": "updated_at", "header": "Updated", "width": "9rem", "format": "datetime" }

package/pages/overview.page.json

@@ -112,6 +112,7 @@
         "columns": [
           { "field": "title", "template": "{{title}}", "header": "Item", "subtitleField": "pr_key", "truncate": true, "width": "36%", "link": { "kind": "page", "page": "home", "keyField": "pr_key" } },
           { "field": "status", "header": "Status", "link": { "kind": "processExplorer", "keyField": "process_key" } },
+          { "field": "process_key", "header": "Agent", "template": "Transcript", "width": "7rem", "link": { "kind": "page", "page": "cockpit", "keyField": "process_key" } },
           { "field": "incident_message", "header": "Incident", "truncate": true, "badge": { "tone": "danger", "label": "1" } },
           { "field": "current_round", "template": "{{current_round}} · {{active_worker}}", "header": "Round · Agent" },
           { "field": "updated_at", "header": "Updated", "width": "9rem", "format": "datetime" }
@@ -149,6 +150,7 @@
         "columns": [
           { "field": "title", "template": "{{title}}", "header": "Item", "subtitleField": "plan_key", "truncate": true, "width": "36%", "link": { "kind": "page", "page": "epic-detail", "keyField": "plan_key" } },
           { "field": "status", "header": "Status", "link": { "kind": "processExplorer", "keyField": "process_key" } },
+          { "field": "process_key", "header": "Agent", "template": "Transcript", "width": "7rem", "link": { "kind": "page", "page": "cockpit", "keyField": "process_key" } },
           { "field": "delivery_label", "header": "Landing" },
           { "field": "wave_label", "header": "Wave" },
           { "field": "updated_at", "header": "Updated", "width": "9rem", "format": "datetime" }
@@ -186,6 +188,7 @@
         "columns": [
           { "field": "title", "template": "{{title}}", "header": "Item", "subtitleField": "feature_key", "truncate": true, "width": "36%", "link": { "kind": "page", "page": "feature", "keyField": "feature_key" } },
           { "field": "status", "header": "Status", "link": { "kind": "processExplorer", "keyField": "process_key" } },
+          { "field": "process_key", "header": "Agent", "template": "Transcript", "width": "7rem", "link": { "kind": "page", "page": "cockpit", "keyField": "process_key" } },
           { "field": "delivery_label", "header": "Delivery" },
           { "field": "updated_at", "header": "Updated", "width": "9rem", "format": "datetime" }
         ],
@@ -227,6 +230,7 @@
         "columns": [
           { "field": "title", "template": "{{title}}", "header": "Item", "subtitleField": "run_key", "truncate": true, "width": "30%", "link": { "kind": "page", "page": "delivery-graph-detail", "keyField": "run_key" } },
           { "field": "status", "header": "Status", "link": { "kind": "processExplorer", "keyField": "process_key" } },
+          { "field": "process_key", "header": "Agent", "template": "Transcript", "width": "7rem", "link": { "kind": "page", "page": "cockpit", "keyField": "process_key" } },
           {
             "field": "stage",
             "header": "Pipeline",

package/pages/tasks.page.json

@@ -106,6 +106,7 @@
             "header": "Process",
             "link": { "kind": "processExplorer", "keyField": "process_key" }
           },
+          { "field": "process_key", "header": "Agent", "template": "Transcript", "width": "7rem", "link": { "kind": "page", "page": "cockpit", "keyField": "process_key" } },
           { "field": "updated_at", "header": "Updated", "format": "datetime" }
         ],
         "rowKey": "user_task_key",

package/resources/prompts/review-round.md

@@ -71,6 +71,11 @@ Because several agents may run on the same host at once:
    "low confidence" list Copilot folds into the **review body** (`.../reviews`
    `body`). These are NOT in the default inline-comment API set, so a plain
    `.../comments` read misses them; scan the review body for them explicitly.
+   Newer Copilot reviews use an **overview format** (body starts `<!-- ccr-overview-v2 -->`)
+   with no "Suppressed comments" block. There, findings without an inline thread appear
+   **only** in the "What changed in this PR" file table, as clauses in the notes column
+   ending `(<severity>, N vote[s])`. The gate treats **each such clause** as an advisory
+   (even if it duplicates an inline thread), so triage and ack every one (step 5a).
    If `answer` is present, it is a human's instruction: either the decision on the
    escalation you raised last round, or change requests from merge approval of this
    converged PR. Act on it first, on this same PR branch (never open a new PR).
@@ -130,7 +135,11 @@ Because several agents may run on the same host at once:
    new line — a line-based ack would go stale and the gate would escalate to a human
    every round (issue #787). Because the ack is keyed on the prose, a decline you
    made in an earlier round stays acknowledged across the drift and you need **not**
-   re-ack it. The ack thread may sit on any valid diff line. Example:
+   re-ack it. The ack thread may sit on any valid diff line. For an **overview-format
+   table finding**, `<path>` is the file column (backticks and any invisible zero-width
+   characters removed) and `<advisory text>` is the finding clause **verbatim, without**
+   its trailing `(<severity>, N vote[s])` tag — for the row's first finding, include any
+   leading row summary before it (e.g. `nano-ack: a.json :: Adds X; A is wrong`). Example:
 
    ```sh
    # Post the ack thread (pick any changed line in the diff for path/line). Use the PR's real HEAD

package/test/agent-link-column.test.ts

@@ -0,0 +1,66 @@
+// #833 — every grid whose cell links a process instance to the explorer also offers an "Agent" cell
+// linking the SAME process_key to the cockpit (which focuses that process: live terminal, else its
+// latest transcript, plus the engine agent history). Guards the class: a new active-run grid can't
+// ship with a process-explorer link but no way to reach its agent — regardless of which column
+// carries the explorer link (Status, Phase, Stage, Process, …).
+import assert from "node:assert/strict";
+import { readdirSync, readFileSync } from "node:fs";
+import { test } from "node:test";
+
+function isObject(value: unknown): value is Record<string, unknown> {
+  return value !== null && typeof value === "object" && !Array.isArray(value);
+}
+
+// The `link` object of a column, if it has one (no `as`: narrowed through type guards).
+function columnLink(column: unknown): Record<string, unknown> | undefined {
+  if (!isObject(column)) return undefined;
+  return isObject(column.link) ? column.link : undefined;
+}
+
+// The process-explorer link of a column, wherever it lives: a plain column carries it at `link`,
+// but a `pipeline` column carries it on its `locus` (the cell that links the row's process instance —
+// e.g. the Feature runs grid's Pipeline column, pages/feature.page.json). Guarding only `link` would
+// let a pipeline-locus explorer link ship with no Agent column and this test still pass (#834).
+function explorerLink(column: unknown): Record<string, unknown> | undefined {
+  const direct = columnLink(column);
+  if (direct?.kind === "processExplorer") return direct;
+  if (!isObject(column)) return undefined;
+  const locus = isObject(column.locus) ? column.locus : undefined;
+  const locusLink = locus && isObject(locus.link) ? locus.link : undefined;
+  return locusLink?.kind === "processExplorer" ? locusLink : undefined;
+}
+
+// Collect every `columns` array anywhere in the page tree (top-level grids, tab grids, detail
+// children, …) without asserting the parsed JSON's shape.
+function grids(node: unknown, out: unknown[][]): void {
+  if (Array.isArray(node)) {
+    for (const n of node) grids(n, out);
+    return;
+  }
+  if (!isObject(node)) return;
+  if (Array.isArray(node.columns)) out.push(node.columns);
+  for (const v of Object.values(node)) grids(v, out);
+}
+
+test("#833: every process-explorer grid also links its process to the cockpit via an Agent column", () => {
+  let checked = 0;
+  for (const file of readdirSync("pages").filter((f) => f.endsWith(".page.json"))) {
+    const all: unknown[][] = [];
+    grids(JSON.parse(readFileSync(`pages/${file}`, "utf8")), all);
+    for (const cols of all) {
+      // A grid qualifies whenever ANY of its columns links to the process explorer — not just a
+      // Status/Phase header, and not just a plain `link`: a pipeline column's `locus.link` counts too
+      // — so a Stage/Process/Pipeline (or any future) explorer link is guarded too.
+      const explorer = cols.some((c) => explorerLink(c) !== undefined);
+      if (!explorer) continue;
+      checked++;
+      const agent = cols.find((c) => isObject(c) && c.header === "Agent");
+      assert.deepEqual(
+        columnLink(agent),
+        { kind: "page", page: "cockpit", keyField: "process_key" },
+        `${file}: grid missing the Agent → cockpit link`,
+      );
+    }
+  }
+  assert.ok(checked > 0, "found no process-explorer grids to check");
+});