pi-onlyne 1.1.2 → 1.2.1

package/src/protocol.test.mjs

@@ -128,22 +128,38 @@ test("heartbeat carries a full legal observation", () => {
     delivery: "none",
     resource: "attached",
     recovery: "none",
-    outcome: "pending",
-    public: "working",
   });
 });
 
 test("every observation is a state tuple the reducer calls legal", () => {
-  // Mirrors onlyne-session's `project`: running works, idle/ready wait, booting creates.
-  const expected = { booting: "created", ready: "idle", running: "working", idle: "idle", gone: "created" };
-  for (const [agent, projected] of Object.entries(expected)) {
+  // The tuple carries no derived view and no task result any more: `Observation`
+  // dropped `public` and `outcome`, so the projection mapping that once had to be
+  // mirrored here is gone, and the key set is the whole contract.
+  const keys = [
+    "version",
+    "generation_live",
+    "isolate_after",
+    "terminate_after",
+    "mismatch_count",
+    "agent",
+    "delivery",
+    "resource",
+    "recovery",
+  ];
+  const agents = { booting: "booting", ready: "ready", running: "running", idle: "idle", gone: "booting" };
+  for (const [agent, state] of Object.entries(agents)) {
     const observed = observationFor(agent, { generation: 1, seq: 1 });
-    assert.equal(observed.public, projected, `agent=${agent}`);
+    assert.deepEqual(Object.keys(observed), keys, `agent=${agent}`);
+    assert.equal(observed.agent, state, `agent=${agent}: gone travels as booting; only the host's grace declares death`);
     assert.notEqual(observed.isolate_after, 0);
     assert.notEqual(observed.terminate_after, 0);
-    assert.equal(observed.outcome, "pending");
+    // The plugin can see neither the intent drain nor its own resource's close,
+    // and it does not pretend otherwise: the two placeholders below are
+    // overwritten by the client's composition, and `resource: attached` is the
+    // attach the host's dispatch path already recorded.
     assert.equal(observed.delivery, "none");
     assert.equal(observed.recovery, "none");
+    assert.equal(observed.resource, "attached");
   }
 });
 
@@ -238,7 +254,12 @@ test("the ledger head is capped at the plan's 200 characters", () => {
 test("an assignment becomes one message naming its origin and payload", { skip: !hasVectors }, () => {
   const assign = ASSIGN().args;
   const text = injectionText({ assign, proseIsNew: true });
-  assert.match(text, /^\[onlyne\] task 11111111-1111-4111-8111-111111111111 from role:planner \(kind task\)/);
+  // The vector's causality names no hop budget, and the header is byte for byte
+  // the line this plugin has always injected.
+  assert.equal(
+    text.split("\n")[0],
+    "[onlyne] task 11111111-1111-4111-8111-111111111111 from role:planner (kind task)",
+  );
   assert.match(text, /\[onlyne\] role prose from the spec:\nRead the incoming task/);
   assert.match(text, /\nbuild it\n?$/);
 
@@ -247,6 +268,35 @@ test("an assignment becomes one message naming its origin and payload", { skip:
   assert.match(repeat, /build it/);
 });
 
+test("the header names the hop and the budget once the family names a budget", () => {
+  const header = (causality) =>
+    injectionText({
+      assign: {
+        task_id: "t9",
+        envelope: {
+          id: "e9",
+          kind: "task",
+          from: { role: { role: "planner" } },
+          causality,
+          body: { text: "pass it on" },
+        },
+      },
+      proseIsNew: false,
+    }).split("\n")[0];
+
+  assert.equal(
+    header({ task: "t9", hop: 2, attempt: 0, family: "t1", hop_budget: 5 }),
+    "[onlyne] task t9 from role:planner (kind task, hop 2, hop budget 5)",
+  );
+  // A family may spend no further hop: zero is a figure the header carries.
+  assert.equal(
+    header({ task: "t9", hop: 0, attempt: 0, family: "t1", hop_budget: 0 }),
+    "[onlyne] task t9 from role:planner (kind task, hop 0, hop budget 0)",
+  );
+  // No budget named: the compat rule holds, hop or no hop.
+  assert.equal(header({ task: "t9", hop: 2, attempt: 0 }), "[onlyne] task t9 from role:planner (kind task)");
+});
+
 test("an empty-bodied assignment still produces an instruction", () => {
   const text = injectionText({
     assign: { task_id: "t9", envelope: { id: "e9", kind: "note", from: { gateway: { gateway: "fg1", channel: "fake", conversation: "c1" } }, body: {} } },