@sjawhar/pi-legion-envoy 0.30.0 → 0.32.0

package/dist/legion.js

@@ -33279,6 +33279,36 @@ var exitProcess = (code) => process.exit(code);
 function setLegionBootstrapExitForTests(hook) {
   exitProcess = hook;
 }
+var READY_RETRY_ATTEMPTS = 3;
+var READY_RETRY_DELAY_MS = 1000;
+function isNetworkOrTimeoutError(error) {
+  if (!(error instanceof Error))
+    return false;
+  if (error.name === "NetworkError" || error.name === "TimeoutError")
+    return true;
+  const code = "code" in error && typeof error.code === "string" ? error.code : undefined;
+  return code === "ConnectionRefused" || code === "ConnectionTimeout" || code === "EAI_AGAIN" || code === "ECONNREFUSED" || code === "ECONNRESET" || code === "EHOSTUNREACH" || code === "ENETUNREACH" || code === "ENOTFOUND" || code === "ETIMEDOUT" || error instanceof TypeError && (error.message === "Failed to fetch" || error.message === "fetch failed");
+}
+var callReadyWithRetry = async (label, call) => {
+  for (let attempt = 1;attempt <= READY_RETRY_ATTEMPTS; attempt++) {
+    try {
+      await call();
+      return;
+    } catch (error) {
+      const retryable = error instanceof LegionDaemonApiError ? error.status >= 500 && error.status < 600 : isNetworkOrTimeoutError(error);
+      if (!retryable)
+        throw error;
+      if (attempt === READY_RETRY_ATTEMPTS) {
+        console.error(`[legion] ${label} failed after ${attempt} attempts: ${messageFor(error)}`);
+        throw error;
+      }
+      console.error(`[legion] ${label} failed (attempt ${attempt}/${READY_RETRY_ATTEMPTS}), retrying: ${messageFor(error)}`);
+      const retryDelay = Promise.withResolvers();
+      setTimeout(retryDelay.resolve, READY_RETRY_DELAY_MS);
+      await retryDelay.promise;
+    }
+  }
+};
 async function persistedTranscript(context) {
   await context.sessionManager.ensureOnDisk();
   const sessionFile = context.sessionManager.getSessionFile();
@@ -33453,15 +33483,19 @@ function legionExtension(pi) {
         };
         await claimEnvoyRole(sessionID, roleToken, context);
         await startControlSubscription(sessionID);
-        await roleDaemon().processReady({
+        await callReadyWithRetry("root process/ready", () => roleDaemon().processReady({
           tree,
           sessionId: sessionID,
           secret: started.secret
-        });
+        }));
         registerArchitectTools();
         await activateLegionTool();
       } catch (error) {
-        console.error(`[legion] root bootstrap failed after process/started registered a role; exiting so the daemon respawns a fresh attempt: ${messageFor(error)}`);
+        if (error instanceof LegionDaemonApiError && (error.status === 401 || error.status === 403)) {
+          console.error(`[legion] root bootstrap authorization failed after process/started registered a role; exiting so the daemon respawns a fresh attempt: ${messageFor(error)}`);
+        } else {
+          console.error(`[legion] root bootstrap failed after process/started registered a role; exiting so the daemon respawns a fresh attempt: ${messageFor(error)}`);
+        }
         exitProcess(1);
       }
     })();
@@ -33518,16 +33552,20 @@ function legionExtension(pi) {
           registerArchitectTools();
           await activateLegionTool();
         }
-        await roleDaemon().workerReady({
+        await callReadyWithRetry("worker/ready", () => roleDaemon().workerReady({
           tree,
           issue,
           role,
           sessionId: sessionID,
           secret: started.secret,
           generation: generation(process.env)
-        });
+        }));
       } catch (error) {
-        console.error(`[legion] worker bootstrap failed after worker/started registered a role; exiting so the daemon respawns a fresh attempt: ${messageFor(error)}`);
+        if (error instanceof LegionDaemonApiError && (error.status === 401 || error.status === 403)) {
+          console.error(`[legion] worker bootstrap authorization failed after worker/started registered a role; exiting so the daemon respawns a fresh attempt: ${messageFor(error)}`);
+        } else {
+          console.error(`[legion] worker bootstrap failed after worker/started registered a role; exiting so the daemon respawns a fresh attempt: ${messageFor(error)}`);
+        }
         exitProcess(1);
       }
     })();

package/dist/skills/dispatch/SKILL.md

@@ -76,8 +76,10 @@ dispatch_ask({
 It returns `details` `{ issue, topic, ask }`. Options are buttons: never enumerate choices in
 prose. Put the recommendation in `question`, and put each selectable choice in `options`.
 Anchor a document question with `anchor: { artifact, quote, occurrence? }`; `occurrence` is
-zero-based and selects a repeated quote. An anchor whose quote disappears becomes orphaned but
-remains readable against its original document version.
+zero-based and selects a repeated quote. The server writes the resulting mark. The HTTP API also
+accepts `{ artifact, mark_id }` from a browser that has already written its mark; Dispatch tools
+use the quote form. An anchor whose quote disappears becomes orphaned but remains readable against
+its original document version.
 
 An ask stays open until a human answers, unless its question no longer needs that answer. Retract a
 moot or superseded question, or self-resolve one after finding the answer:
@@ -141,7 +143,8 @@ dispatch_comment({ issue, artifact?, quote?, occurrence?, body, reply_to?, reply
 ```
 
 It returns `details` `{ issue, topic, comment }`. `quote` requires `artifact`; omit both for a
-floating issue comment. Use `reply_to` to continue a comment thread; use `reply_to_ask` to reply
+floating issue comment. A reply (`reply_to`/`reply_to_ask`) takes no `quote`; it belongs to its
+parent's anchor. Use `reply_to` to continue a comment thread; use `reply_to_ask` to reply
 directly under a question asked with `dispatch_ask`. The two are mutually exclusive.
 
 Propose an exact replacement instead of describing it:
@@ -152,8 +155,12 @@ dispatch_suggest({ issue, artifact, quote, replace_with, body?, occurrence? })
 
 It returns `details` `{ issue, topic, comment }`. A human accepts or rejects a suggestion. On
 `TARGET_AMBIGUOUS`, add zero-based `occurrence`. On `TARGET_NOT_FOUND`, re-read the document
-before retrying. `INVALID_OP` names a malformed edit; `CAP_EXCEEDED` never truncates;
-`ISSUE_CLOSED` rejects a write. `ACTOR_KIND` and `ROUTE_INVALID` reject an invalid actor or route.
+before retrying. `INVALID_ANCHOR` requires exactly one nonempty anchor `quote` or `mark_id`;
+`ANCHOR_MISSING` means a browser mark was not observed in the live tree, and
+`ANCHOR_ORPHANED` means its marked text no longer exists. `INVALID_MARKDOWN` and `DOC_SCHEMA`
+reject Markdown or a live tree outside the Proof schema. `INVALID_OP` names a malformed edit;
+`CAP_EXCEEDED` never truncates; `ISSUE_CLOSED` rejects a write. `ACTOR_KIND` and `ROUTE_INVALID`
+reject an invalid actor or route.
 
 ## Artifacts
 

package/package.json

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