@nanobpm/nano-workforce 0.175.1 → 0.175.2

package/CHANGELOG.md

@@ -1,3 +1,9 @@
+## [0.175.2](https://github.com/nanobpm/nano-workforce/compare/v0.175.1...v0.175.2) (2026-09-02)
+
+### Bug Fixes
+
+* **abandon:** don't let the cancel-check deadlock a shell-less agent ([#678](https://github.com/nanobpm/nano-workforce/issues/678)) ([#709](https://github.com/nanobpm/nano-workforce/issues/709)) ([f5fed93](https://github.com/nanobpm/nano-workforce/commit/f5fed93bdd0e98cbb24f7dff4b0f9a650c235a69)), closes [#76](https://github.com/nanobpm/nano-workforce/issues/76) [#76](https://github.com/nanobpm/nano-workforce/issues/76)
+
 ## [0.175.1](https://github.com/nanobpm/nano-workforce/compare/v0.175.0...v0.175.1) (2026-09-02)
 
 ### Bug Fixes

package/app/abandon.test.ts

@@ -83,6 +83,32 @@ test("renderAbandonBrief embeds the concrete URL and the stop contract", () => {
   assertEquals(brief.includes("curl -fsS"), true);
 });
 
+test("renderAbandonBrief forbids delegating the check to a sub-agent (issue #678)", () => {
+  // A non-Claude agent (e.g. qwen) that hands this shell command to a shell-less
+  // `general-purpose` sub-agent deadlocks: the sub-agent has no shell, produces no
+  // output, and the worker's idle timeout kills the round. The brief must steer the
+  // agent to run the check inline with its OWN shell and never delegate it.
+  const brief = renderAbandonBrief("https://host/app/api/hooks/abandon?token=tok");
+  // Not merely that it *mentions* sub-agents/inline — it must explicitly PROHIBIT delegation, so a
+  // future edit that softened the brief into *encouraging* delegation would fail this test.
+  assertEquals(/do \*\*not\*\* delegate it to a\s+sub-?agent/i.test(brief), true, "explicitly forbids delegating to a sub-agent");
+  assertEquals(/inline/i.test(brief), true, "tells the agent to run it inline");
+});
+
+test("renderAbandonBrief degrades gracefully when the agent has no shell (issue #678)", () => {
+  // The check is ADVISORY over an airtight harness job-fence (issue #76 layer 2). An
+  // agent that cannot run a shell command at all must be told to SKIP and proceed
+  // rather than thrash/hang — the harness job-fence (issue #76 layer 2) still fences
+  // a cancelled run, so a skipped check degrades safely rather than promising full
+  // independent enforcement.
+  const brief = renderAbandonBrief("https://host/app/api/hooks/abandon?token=tok");
+  assertEquals(/skip this check/i.test(brief), true, "offers a skip path");
+  // Assert the actual anti-deadlock contract, not an explanatory word: a shell-less agent must be
+  // told its lack of a shell is NOT a failed check, and must be forbidden from stalling/hanging.
+  assertEquals(/not (a )?failed check/i.test(brief), true, "no shell is not treated as a failed check");
+  assertEquals(/never stall,.*hang/i.test(brief), true, "forbids stalling/hanging when it can't run curl");
+});
+
 test("prKeyForAbandonToken resolves a known token and rejects unknowns", async () => {
   const data = memData();
   await seedPr(data, "o/r#1", "tok", "converging");

package/app/abandon.ts

@@ -148,5 +148,18 @@ down), instead of silently printing an error body with exit 0.
   try to complete the job is EXPECTED after a cancel — do not treat it as an error to retry.
 - **Proceed** only when the command **succeeds** AND reports \`"abandoned": false\` — and re-check
   right before the push, since a cancel can land at any moment. Checking as late as possible keeps
-  the window tiny.`;
+  the window tiny.
+
+**Run this check yourself, inline, with your own shell tool.** Do **not** delegate it to a
+sub-agent, helper, or a separate "run this command" task: a sub-agent may have **no shell**, in which
+case it cannot run \`curl\`, produces no output, and the worker's idle timeout will kill the round.
+One inline \`curl\` in your own turn is all this needs.
+
+**Having no way to run a shell command at all is NOT a failed check — SKIP this check and proceed.**
+The "abort on non-zero exit" rule above is about \`curl\` *running and reporting* a torn-down run; it
+does **not** mean "abort because you have no shell to run \`curl\` with". This check is a best-effort
+optimisation, not a hard gate: the harness that runs your job also fences a cancelled run's
+completion, so a skipped check only widens a tiny race — it does not by itself create an orphaned
+run. Never stall, thrash, hang, or fail the round trying to run \`curl\` through a tool you do not
+have; a skipped check is strictly better than a hung one.`;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nanobpm/nano-workforce",
-  "version": "0.175.1",
+  "version": "0.175.2",
   "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",