@sjawhar/pi-legion-envoy 5.0.1 → 5.1.0

package/dist/envoy.js

@@ -34851,6 +34851,7 @@ function legionRoleClaimBridge() {
     return bridge;
   const createdBridge = {
     instances: [],
+    managedSessions: new Set,
     regained: undefined
   };
   store[LEGION_ROLE_CLAIM_BRIDGE] = createdBridge;
@@ -34858,6 +34859,7 @@ function legionRoleClaimBridge() {
 }
 async function claimEnvoyRole(sessionID, role, context) {
   const bridge = legionRoleClaimBridge();
+  bridge.managedSessions.add(sessionID);
   const instance = bridge.instances.findLast((candidate) => candidate.sessionID() === sessionID) ?? bridge.instances.at(-1);
   if (instance === undefined)
     throw new Error("Envoy has no bound instance for a role claim");
@@ -34975,6 +34977,21 @@ var NATS_RETRY_INTERVAL_MS = 15000;
 var CAPABILITIES_WITHOUT_BTW = DELIVERY_CAPABILITIES.filter((capability) => capability !== "btw");
 var ROLE_CLAIM_ENTRY = "envoy-role-claim";
 var OPEN_ASKS_TIMEOUT_MS = 3000;
+var LEGION_MANAGED_ENTRY = "legion-managed-session";
+var ASK_REMINDER_MESSAGE = "dispatch-ask-reminder";
+var OPEN_ASKS_REMINDER = "You have no unanswered asks in Dispatch. If you are waiting for human input, open an ask. Otherwise ignore this reminder and continue with any remaining work. Do not reply just to acknowledge this reminder.";
+var ASK_OPENING_TOOLS = ["dispatch_ask", "dispatch_request_approval"];
+function isLegionManagedEntry(entry) {
+  if (typeof entry !== "object" || entry === null)
+    return false;
+  if (!("type" in entry) || entry.type !== "custom")
+    return false;
+  if (!("customType" in entry) || entry.customType !== LEGION_MANAGED_ENTRY)
+    return false;
+  if (!("data" in entry) || typeof entry.data !== "object" || entry.data === null)
+    return false;
+  return "session_id" in entry.data && typeof entry.data.session_id === "string";
+}
 function isRoleClaimEntry(entry) {
   if (typeof entry !== "object" || entry === null)
     return false;
@@ -35030,26 +35047,54 @@ function envoyExtension(pi) {
   let claimedRoleTopic;
   let activeSessionContext;
   const inbox = [];
+  let askAwareness = {
+    session_id: "",
+    period: 0,
+    baseline_as_of: null,
+    fired: false,
+    saw_ask: false
+  };
+  let awarenessGeneration = 0;
+  let askCheckInFlight = false;
+  let legionManagedTranscript = false;
+  const legionManaged = (id) => legionManagedTranscript || legionRoleClaimBridge().managedSessions.has(id);
   const availabilityWarningSessionIDs = new Set;
   const warnAskAvailability = (context, error48) => {
     const sessionID2 = context.sessionManager.getSessionId();
     if (availabilityWarningSessionIDs.has(sessionID2))
       return;
     availabilityWarningSessionIDs.add(sessionID2);
-    context.ui.notify(`envoy: Dispatch open-ask check unavailable (${messageFor(error48)}); summary unavailable`, "warning");
+    context.ui.notify(`envoy: Dispatch open-ask check unavailable (${messageFor(error48)}); the stop-time ask reminder is off until it recovers`, "warning");
   };
-  const queryOpenAsks = async (requestedSessionID) => {
+  const queryOpenAsks = async (requestedSessionID, since) => {
     const config2 = activeDispatchConfig();
     if (config2 === null)
       return null;
-    const snapshot = await new DispatchClient(config2.url, config2.token, fetch, AbortSignal.timeout(OPEN_ASKS_TIMEOUT_MS)).openAsks(requestedSessionID);
+    const snapshot = await new DispatchClient(config2.url, config2.token, fetch, AbortSignal.timeout(OPEN_ASKS_TIMEOUT_MS)).openAsks(requestedSessionID, since);
     availabilityWarningSessionIDs.delete(requestedSessionID);
     return { snapshot, url: config2.url };
   };
+  const markLegionManagedSession = (targetSessionID) => {
+    legionRoleClaimBridge().managedSessions.add(targetSessionID);
+    if (legionManagedTranscript)
+      return;
+    legionManagedTranscript = true;
+    pi.appendEntry(LEGION_MANAGED_ENTRY, { session_id: targetSessionID });
+  };
   const restoreLocalSessionState = (context) => {
     sessionDirectory = context.cwd;
     sessionID = context.sessionManager.getSessionId();
     activeSessionContext = context;
+    const branch = context.sessionManager.getBranch?.() ?? [];
+    askAwareness = {
+      session_id: sessionID,
+      period: 0,
+      baseline_as_of: null,
+      fired: false,
+      saw_ask: false
+    };
+    legionManagedTranscript = branch.some(isLegionManagedEntry);
+    awarenessGeneration++;
   };
   pi.on("resources_discover", async () => ({ skillPaths: [SKILLS_DIRECTORY] }));
   registerEnvoyMessageRenderer(pi);
@@ -35399,6 +35444,7 @@ function envoyExtension(pi) {
     if (context === undefined || context.sessionManager.getSessionId() !== targetSessionID) {
       throw new Error(`Envoy has no active session for Legion role claim: ${targetSessionID}`);
     }
+    markLegionManagedSession(targetSessionID);
     if (sessionID !== targetSessionID)
       await establishSession(context);
     await registerSession();
@@ -35562,33 +35608,61 @@ function envoyExtension(pi) {
     }
   }
   registerEnvoyWhoamiCommand(pi, () => sessionID);
-  pi.on("before_agent_start", async (_event, context) => {
+  pi.on("before_agent_start", async (event, context) => {
     const id = context.sessionManager.getSessionId();
-    if (id === "")
+    if (id === "" || event.prompt.trim() === "" || !context.hasUI || legionManaged(id)) {
+      return;
+    }
+    if (await isSubagent(context))
       return;
     try {
       const open = await queryOpenAsks(id);
-      if (open === null)
-        return;
-      return {
-        message: {
-          customType: "dispatch-open-asks",
-          content: `Dispatch authored-ask summary:
-${formatOpenAsksSummary(open.snapshot, open.url)}`,
-          display: false,
-          attribution: "agent"
-        }
-      };
+      if (open !== null)
+        armAskAwareness(id, event.prompt, open.snapshot.as_of);
     } catch (error48) {
       warnAskAvailability(context, error48);
-      return {
-        message: {
-          customType: "dispatch-open-asks",
-          content: `Dispatch authored-ask summary unavailable: ${messageFor(error48)}`,
-          display: false,
-          attribution: "agent"
-        }
-      };
+    }
+    return;
+  });
+  function armAskAwareness(id, prompt, asOf) {
+    if (prompt.trim() === "")
+      return;
+    awarenessGeneration++;
+    askAwareness = {
+      session_id: id,
+      period: (askAwareness.session_id === id ? askAwareness.period : 0) + 1,
+      baseline_as_of: asOf,
+      fired: false,
+      saw_ask: false
+    };
+  }
+  pi.on("agent_end", async (event, context) => {
+    const id = context.sessionManager.getSessionId();
+    const lastReply = event.messages?.findLast((message) => message.role === "assistant");
+    if (event.willContinue === true || lastReply?.stopReason !== "stop" || shuttingDown || !context.hasUI || id === "" || id !== sessionID || legionManaged(id) || askAwareness.session_id !== id || askAwareness.period === 0 || askAwareness.fired || askAwareness.saw_ask || askAwareness.baseline_as_of === null || askCheckInFlight) {
+      return;
+    }
+    const period = askAwareness.period;
+    const generation = awarenessGeneration;
+    askCheckInFlight = true;
+    try {
+      let open;
+      try {
+        open = await queryOpenAsks(id, askAwareness.baseline_as_of);
+      } catch (error48) {
+        if (generation === awarenessGeneration)
+          warnAskAvailability(context, error48);
+        return;
+      }
+      if (open === null || shuttingDown || generation !== awarenessGeneration || sessionID !== id || context.sessionManager.getSessionId() !== id || legionManaged(id) || askAwareness.session_id !== id || askAwareness.period !== period || askAwareness.fired || askAwareness.saw_ask) {
+        return;
+      }
+      if (open.snapshot.count > 0 || open.snapshot.opened_since)
+        return;
+      askAwareness = { ...askAwareness, fired: true };
+      pi.sendMessage({ customType: ASK_REMINDER_MESSAGE, content: OPEN_ASKS_REMINDER, display: false }, { deliverAs: "steer", triggerTurn: true });
+    } finally {
+      askCheckInFlight = false;
     }
   });
   const announceFollow = createFollowAnnouncer((text) => {
@@ -35597,6 +35671,9 @@ ${formatOpenAsksSummary(open.snapshot, open.url)}`,
   pi.on("tool_result", async (event) => {
     if (event.isError)
       return;
+    if (ASK_OPENING_TOOLS.includes(event.toolName) && askAwareness.session_id === sessionID && askAwareness.period > 0 && !askAwareness.saw_ask) {
+      askAwareness = { ...askAwareness, saw_ask: true };
+    }
     announceFollow(event.details);
   });
   async function execute(operation, rawParameters) {

package/dist/legion.js

@@ -16142,7 +16142,7 @@ import { logger } from "@oh-my-pi/pi-utils";
 // package.json
 var package_default = {
   name: "@sjawhar/pi-legion-envoy",
-  version: "5.0.1",
+  version: "5.1.0",
   type: "module",
   omp: {
     extensions: [
@@ -31220,6 +31220,7 @@ function legionRoleClaimBridge() {
     return bridge;
   const createdBridge = {
     instances: [],
+    managedSessions: new Set,
     regained: undefined
   };
   store[LEGION_ROLE_CLAIM_BRIDGE] = createdBridge;
@@ -31227,6 +31228,7 @@ function legionRoleClaimBridge() {
 }
 async function claimEnvoyRole(sessionID, role, context) {
   const bridge = legionRoleClaimBridge();
+  bridge.managedSessions.add(sessionID);
   const instance = bridge.instances.findLast((candidate) => candidate.sessionID() === sessionID) ?? bridge.instances.at(-1);
   if (instance === undefined)
     throw new Error("Envoy has no bound instance for a role claim");

package/dist/skills/legion-architect/SKILL.md

@@ -327,6 +327,11 @@ everything else in the tree, or use `dispatch_ask` for a human question; workers
 Sami directly with `dispatch_ask` the same way. Do not create a wait loop for any wake
 source.
 
+Never yield while waiting on a human. A human is waiting on you only where an open ask sits
+in their inbox, so open it — `dispatch_ask`, or `dispatch_request_approval` for the spec
+gate — before you stop. Otherwise proceed: proceeding is the default, and a stop that waits
+on nobody stalls the tree until someone notices.
+
 ## Architecture components
 
 Bootstrap on a root whose project has an architecture source: in the root's first PR (the implement worker pushes it), write `.dispatch/architecture/<id>.md` files for the planned components — front matter `title`, `parent`, `depends_on`, `external`; no `paths` yet. The importer reads the source branch's head, so the sync and the attach below work only once that PR has merged to the source branch: until then leave the root on `inherit` (the attach would answer `400 COMPONENTS_INPUT`, the component does not exist yet). After the merge, `dispatch_architecture_sync({ project })` and attach the root with `dispatch_issue_update({ issue, components: { mode: "explicit", ids: [...] } })`. Children inherit the root's attachment; give a child its own `components` only when it changes a narrower set, and `{ mode: "none", reason }` when it is not architectural work. Attach before decomposing, and require every implementer to change the component file beside the code it describes in the same review.

package/dist/skills/legion-worker/SKILL.md

@@ -590,3 +590,9 @@ When blocked on lifecycle, scope, or cross-phase matters, `envoy_publish` the ow
 architect a concise message: issue, phase, verified observation, what you tried, and the
 decision required. Reach for `dispatch_ask` yourself only for a standalone human question
 outside that coordination.
+
+Never yield while blocked on a decision someone else owns. Before you stop, make the block
+visible where its owner will see it: a lifecycle, scope, or cross-phase decision goes to the
+owning architect as above, and a standalone human question goes in `dispatch_ask`. Otherwise
+proceed: proceeding is the default, and a phase that stops silently holds its issue until
+someone notices.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sjawhar/pi-legion-envoy",
-  "version": "5.0.1",
+  "version": "5.1.0",
   "type": "module",
   "omp": {
     "extensions": [