@nanobpm/nano-workforce 0.187.3 → 0.187.4

package/CHANGELOG.md

@@ -1,3 +1,9 @@
+## [0.187.4](https://github.com/nanobpm/nano-workforce/compare/v0.187.3...v0.187.4) (2026-09-14)
+
+### Bug Fixes
+
+* raise default review-wait timeout to PT30M ([#792](https://github.com/nanobpm/nano-workforce/issues/792)) ([042573d](https://github.com/nanobpm/nano-workforce/commit/042573dc5e691dc97efebc12da6a63f2cf398c2a)), closes [#783](https://github.com/nanobpm/nano-workforce/issues/783) [#791](https://github.com/nanobpm/nano-workforce/issues/791)
+
 ## [0.187.3](https://github.com/nanobpm/nano-workforce/compare/v0.187.2...v0.187.3) (2026-09-14)
 
 ### Bug Fixes

package/README.md

@@ -360,7 +360,7 @@ agent at that URL to author, compile, and submit a graph unaided. See
 | `NANO_PR_MERGE_METHOD` | `squash` | merge method: `squash`, `merge`, or `rebase` |
 | `NANO_PR_MERGE_ADMIN` | `0` | pass `--admin` to override failing non-required checks (use with care) |
 | `NANO_PR_MAX_CI_FIX_ROUNDS` | `3` | max `senior:fix-ci` attempts to green a `blocked` PR before escalating; `0` disables (escalate immediately), clamped 0–20 |
-| `NANO_PR_REVIEW_WAIT_TIMEOUT` | `PT20M` | ISO-8601 duration the loop waits for a fresh review before escalating a stalled review (timer arm of the `wait-review` gateway) |
+| `NANO_PR_REVIEW_WAIT_TIMEOUT` | `PT30M` | ISO-8601 duration the loop waits for a fresh review before escalating a stalled review (timer arm of the `wait-review` gateway) |
 | `NANO_PR_REVIEW_NUDGE_MINUTES` | `5` | cooldown between the poller's automatic reviewer re-request nudges for one waiting PR (clamped 1–1440) |
 | `NANO_WORKFORCE_BASE_URL` | `http://localhost:3000` | externally-reachable base URL for the capability hooks (`/app/api/hooks/*`). Must resolve from **wherever the agent runs** — set it to the app's LAN address (or console-proxy URL) for a remote fleet. See [Fleet networking](#fleet-networking-remote-workers) |
 | `NANO_AGENTIC_SECRET` | — | enables **secure mode** for the agentic visibility channel (`/agentic`): every peer must present the **same** `NANO_AGENTIC_SECRET` value (set the identical env var on the server and every worker box — Tab A → Slot A). Unset = on-by-default **LOCAL mode** — the well-known token is honoured from **any origin** (open on the trusted LAN, matching the engine's posture); exposure is governed by the server bind address, not a shared secret. Also accepts `NANO_PR_WEBHOOK_SECRET` |

package/SPEC.md

@@ -133,7 +133,7 @@ Notes:
   canonical `readiness-ready` wait-gate message (ADR 0001 §2; correlated by the
   poller when a fresh review lands)
   against a `=reviewWaitTimeout` timer (seeded at submit from
-  `NANO_PR_REVIEW_WAIT_TIMEOUT`, default `PT20M`). Whichever fires first
+  `NANO_PR_REVIEW_WAIT_TIMEOUT`, default `PT30M`). Whichever fires first
   withdraws the other — the message arm advances `round`, the timer arm escalates
   a **stalled review** (`blocked`) so a human decides rather than the instance
   hanging forever. Because `persist-round` already recorded this `round` as
@@ -544,7 +544,7 @@ queries skip (`merging`), so a slow pass can't double-signal.
 | `NANO_PR_AUTO_MERGE` | 1 | run the merge stage after convergence (`0` = review-only; per-submit `convergeOnly: true` override) |
 | `NANO_PR_MERGE_METHOD` | squash | `squash` \| `merge` \| `rebase` |
 | `NANO_PR_MERGE_ADMIN` | 0 | pass `--admin` on merge |
-| `NANO_PR_REVIEW_WAIT_TIMEOUT` | PT20M | ISO-8601 wait before a stalled review escalates (timer arm of the `wait-review` event-based gateway); malformed → default |
+| `NANO_PR_REVIEW_WAIT_TIMEOUT` | PT30M | ISO-8601 wait before a stalled review escalates (timer arm of the `wait-review` event-based gateway); malformed → default |
 | `NANO_PR_REVIEW_NUDGE_MINUTES` | 5 | cooldown between poller Copilot re-request nudges per PR (clamped 1–1440) |
 
 ## 13. Planning fan-out (`plan-fanout.bpmn`) — issue #14

package/app/contracts.ts

@@ -145,6 +145,7 @@ export const ENV_CONTRACTS = {
     name: "NANO_PR_REVIEW_WAIT_TIMEOUT",
     owner: "app/service.ts",
     semantics: "How long to wait for a review before nudging/escalating (FEEL/ISO-8601 duration).",
+    default: "PT30M",
   },
   NANO_PR_REVIEW_NUDGE_MINUTES: {
     category: "env",

package/app/persist-escalation.test.ts

@@ -190,7 +190,7 @@ test("persist-escalation heals from the prKey when repo/prNumber are absent", as
 // never a dead wait with a null question (the #333 defect).
 test("a control-flow arm with a concrete question opens an escalation gw-escalated can park", async () => {
   const { app, inserts } = fakeApp();
-  const question = "No review arrived within the review-wait timeout (PT20M). A human must decide how to proceed.";
+  const question = "No review arrived within the review-wait timeout (PT30M). A human must decide how to proceed.";
   const job = { variables: { prKey: "o/r#5", round: 2, status: "blocked", question, recordRound: false } };
   const out = await handler(job as any, app as any);
   assertEquals((out as any).escalated, true, "a real control-flow escalation reports escalated:true");

package/app/reviewWait.test.ts

@@ -13,6 +13,13 @@ import {
   reviewWaitTimeout,
 } from "./reviewWait.ts";
 
+test("DEFAULT_REVIEW_WAIT_TIMEOUT: pins the user-visible default to PT30M", () => {
+  // A direct regression guard on the literal default — the fallback-based assertions below compare
+  // against DEFAULT_REVIEW_WAIT_TIMEOUT itself, so they would still pass if it were accidentally
+  // reverted to PT20M. This anchors the intended value so that change is caught.
+  assertEquals(DEFAULT_REVIEW_WAIT_TIMEOUT, "PT30M");
+});
+
 test("reviewWaitTimeout: blank / absent / malformed → default", () => {
   assertEquals(reviewWaitTimeout(undefined), DEFAULT_REVIEW_WAIT_TIMEOUT);
   assertEquals(reviewWaitTimeout(""), DEFAULT_REVIEW_WAIT_TIMEOUT);

package/app/reviewWait.ts

@@ -12,7 +12,7 @@
 
 /** Default review-wait timeout (ISO-8601 duration): how long the loop waits for a fresh review
  * before the timer arm of the event-based gateway fires and it escalates to a human. */
-export const DEFAULT_REVIEW_WAIT_TIMEOUT = "PT20M";
+export const DEFAULT_REVIEW_WAIT_TIMEOUT = "PT30M";
 
 // A pragmatic ISO-8601 duration matcher: requires a leading `P`, at least one component, and a
 // `T` before any time components (with at least one time component after it). Good enough to

package/docs/agent-guide.md

@@ -470,7 +470,7 @@ Work through this order:
 3. **Check for an open escalation** (§3) — the process may simply be waiting for a
    human answer. Answer it.
 4. **A review that never arrives** escalates on its own after
-   `NANO_PR_REVIEW_WAIT_TIMEOUT` (default `PT20M`); the poller also re-nudges the
+   `NANO_PR_REVIEW_WAIT_TIMEOUT` (default `PT30M`); the poller also re-nudges the
    reviewer periodically. If the reviewer bot is not provisioned on the repo, no
    review will ever land — that is a repo-config problem, not an app bug.
 5. **Cancel + resubmit** as a last resort. Cancel through the **app-owned** door —

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nanobpm/nano-workforce",
-  "version": "0.187.3",
+  "version": "0.187.4",
   "description": "Nano Workforce — an Agent Graph Orchestration application for Agentic SDLC: durable BPMN processes that coordinate a graph of AI agents across the software delivery lifecycle.",
   "type": "module",
   "main": "main.ts",