agent-dealer 1.2.7 → 1.2.9

package/bundle/shared/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agent-dealer/shared",
-  "version": "1.2.7",
+  "version": "1.2.9",
   "type": "module",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",

package/dist/action.js

@@ -29,13 +29,16 @@ export async function runActionCommand(args) {
             const id = rest[0];
             const choice = flag(rest, "--choice");
             const resolvedBy = flag(rest, "--by");
+            // NOT-314: operator_verification requires a non-empty note (the result,
+            // waiver reason, or repair note) — the server 400s without one.
+            const note = flag(rest, "--note");
             if (!id || !choice || !resolvedBy) {
                 console.error("resolve requires an action id, --choice, and --by");
                 return 1;
             }
             const result = await apiFetch(`/api/human-actions/${id}/resolve`, {
                 method: "POST",
-                body: { choice, resolvedBy },
+                body: note?.trim() ? { choice, resolvedBy, note: note.trim() } : { choice, resolvedBy },
             });
             console.log(JSON.stringify(result, null, 2));
             return 0;

package/dist/action.test.js

@@ -54,6 +54,48 @@ test("action resolve calls POST /api/human-actions/:id/resolve with resolvedBy a
         stub.restore();
     }
 });
+test("action resolve forwards --note when given (required for operator_verification)", async () => {
+    const stub = stubFetch("/api/human-actions/a1/resolve", "POST", { issueStatus: "needs_human" });
+    try {
+        const code = await runActionCommand([
+            "resolve",
+            "a1",
+            "--choice",
+            "verified",
+            "--by",
+            "alice",
+            "--note",
+            "probe output here",
+        ]);
+        assert.equal(code, 0);
+        const body = stub.assertCalled();
+        assert.deepEqual(body, { choice: "verified", resolvedBy: "alice", note: "probe output here" });
+    }
+    finally {
+        stub.restore();
+    }
+});
+test("action resolve omits a blank --note so noteless resolves send the historical body", async () => {
+    const stub = stubFetch("/api/human-actions/a1/resolve", "POST", { issueStatus: "closed" });
+    try {
+        const code = await runActionCommand([
+            "resolve",
+            "a1",
+            "--choice",
+            "complete",
+            "--by",
+            "alice",
+            "--note",
+            "   ",
+        ]);
+        assert.equal(code, 0);
+        const body = stub.assertCalled();
+        assert.deepEqual(body, { choice: "complete", resolvedBy: "alice" });
+    }
+    finally {
+        stub.restore();
+    }
+});
 test("action resolve returns nonzero on API failure", async () => {
     const stub = stubFetch("/api/human-actions/a1/resolve", "POST", { error: "bad choice" }, 400);
     try {

package/dist/index.js

@@ -29,7 +29,7 @@ function printUsage() {
   agent-dealer queue list
   agent-dealer queue move <issueId> --top|--bottom|--before <id>|--after <id>
   agent-dealer action list
-  agent-dealer action resolve <id> --choice C --by NAME
+  agent-dealer action resolve <id> --choice C --by NAME [--note TEXT]
   agent-dealer --version
 
 Human control plane for agent execution.`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-dealer",
-  "version": "1.2.7",
+  "version": "1.2.9",
   "description": "Human control plane for agent execution — queue, plan approval, audit",
   "license": "MIT",
   "type": "module",