@sjawhar/pi-legion-envoy 1.58.0 → 1.59.1

package/dist/envoy.js

@@ -32251,6 +32251,9 @@ class DispatchClient {
   async syncArchitectureSource(project) {
     return this.#json("POST", ["api", "v1", "projects", project, "architecture-source", "sync"]);
   }
+  async getArchitectureSource(project) {
+    return this.#json("GET", ["api", "v1", "projects", project, "architecture-source"]);
+  }
   async resolveAsk(id, input) {
     return this.#json("POST", ["api", "v1", "asks", id, "resolve"], input);
   }
@@ -32746,6 +32749,20 @@ function componentsChange(input, after) {
       return "components -> inherit";
   }
 }
+var architectureComponentsAction = 'Review the `dispatch` skill, "Architecture components", to attach it to the parts it changes or mark it as non-architectural with a reason.';
+async function architectureGuidance(client, project, components) {
+  if (components.mode === "none" || components.ids.length > 0)
+    return;
+  try {
+    await client.getArchitectureSource(project);
+    return `Project ${project} has an architecture model, but this issue is not linked to any of its current components. ${architectureComponentsAction}`;
+  } catch (error) {
+    if (error instanceof DispatchServiceError && error.status === 404 && error.code === "SOURCE_NOT_FOUND") {
+      return;
+    }
+    return `Could not check whether project ${project} has an architecture model: ${messageFor(error)}. ${architectureComponentsAction}`;
+  }
+}
 function isDuplicateCandidate(value) {
   if (typeof value !== "object" || value === null)
     return false;
@@ -33569,9 +33586,13 @@ async function executeDispatchTool(input) {
           ...Array.isArray(labels) ? { labels } : {},
           actor
         });
-        const adviceLines = renderAdvice(input.tool, created.key, created.advice, {
-          isPrimarySpec: spec !== undefined
-        });
+        const componentGuidance = await architectureGuidance(client, created.project, created.components);
+        const adviceLines = [
+          ...renderAdvice(input.tool, created.key, created.advice, {
+            isPrimarySpec: spec !== undefined
+          }),
+          ...componentGuidance === undefined ? [] : [componentGuidance]
+        ];
         return {
           text: [
             `Created ${created.key}: ${created.title} ${notSubscribed(issueTopic(created.key))}`,
@@ -34611,6 +34632,40 @@ function onEnvoyRoleRegained(listener) {
   legionRoleClaimBridge().regained = listener;
 }
 
+// src/subagent-session.ts
+import fs from "fs";
+import path2 from "path";
+import * as host from "@oh-my-pi/pi-coding-agent";
+var LEGION_BOOTSTRAPPED_SESSION = Symbol.for("legion.pi-envoy.bootstrapped-session");
+var bootstrappedSessionStore = globalThis;
+function recordBootstrappedSession(sessionFile) {
+  bootstrappedSessionStore[LEGION_BOOTSTRAPPED_SESSION] = sessionFile;
+}
+async function isSubagentSession(context) {
+  await context.sessionManager.ensureOnDisk();
+  const sessionFile = context.sessionManager.getSessionFile();
+  if (sessionFile !== undefined && fs.existsSync(sessionFile)) {
+    return fs.existsSync(`${path2.dirname(sessionFile)}.jsonl`);
+  }
+  const bootstrapped = bootstrappedSessionStore[LEGION_BOOTSTRAPPED_SESSION];
+  return bootstrapped !== undefined && bootstrapped !== sessionFile;
+}
+function isRegisteredSubagent(context) {
+  const registry = host.AgentRegistry?.global();
+  if (registry === undefined)
+    return false;
+  const sessionID = context.sessionManager.getSessionId();
+  const sessionFile = context.sessionManager.getSessionFile();
+  return registry.list().some((ref) => ref.kind !== "main" && (ref.session === null ? sessionFile !== undefined && ref.sessionFile === sessionFile : ref.session.sessionManager.getSessionId() === sessionID));
+}
+function subagentSessionCheck() {
+  let subagentSession;
+  return (context) => {
+    subagentSession ??= isSubagentSession(context);
+    return subagentSession;
+  };
+}
+
 // src/tool-result.ts
 function toolSuccess(text, details = {}) {
   return { content: [{ type: "text", text }], details };
@@ -35112,7 +35167,11 @@ function envoyExtension(pi) {
     sessionID: () => activeSessionContext?.sessionManager.getSessionId() ?? sessionID
   };
   bridge.instances.push(claimInstance);
+  const isSubagentTranscript = subagentSessionCheck();
+  const isSubagent = async (context) => await isSubagentTranscript(context) || isRegisteredSubagent(context);
   pi.on("session_start", async (_event, context) => {
+    if (await isSubagent(context))
+      return;
     const previousSessionID = sessionID;
     restoreLocalSessionState(context);
     if (dispatchConfig.error !== null) {
@@ -35148,6 +35207,8 @@ function envoyExtension(pi) {
     }
   });
   const rebind = async (reason, context) => {
+    if (await isSubagent(context))
+      return;
     const previousID = sessionID;
     restoreLocalSessionState(context);
     if (defaults.natsUrls.length === 0)

package/dist/legion.js

@@ -16113,8 +16113,7 @@ var require_mod4 = __commonJS(function(exports) {
 
 // extensions/legion.ts
 import { randomUUID as randomUUID3 } from "crypto";
-import fs from "fs";
-import path3 from "path";
+import path4 from "path";
 
 // ../envoy-client/src/defaults.ts
 var DEFAULT_ENVOY_URL = "http://127.0.0.1:9020";
@@ -16145,7 +16144,7 @@ import { logger } from "@oh-my-pi/pi-utils";
 // package.json
 var package_default = {
   name: "@sjawhar/pi-legion-envoy",
-  version: "1.58.0",
+  version: "1.59.1",
   type: "module",
   omp: {
     extensions: [
@@ -30702,6 +30701,40 @@ function createLegionTool(deps) {
   };
 }
 
+// src/subagent-session.ts
+import fs from "fs";
+import path3 from "path";
+import * as host from "@oh-my-pi/pi-coding-agent";
+var LEGION_BOOTSTRAPPED_SESSION = Symbol.for("legion.pi-envoy.bootstrapped-session");
+var bootstrappedSessionStore = globalThis;
+function recordBootstrappedSession(sessionFile) {
+  bootstrappedSessionStore[LEGION_BOOTSTRAPPED_SESSION] = sessionFile;
+}
+async function isSubagentSession(context) {
+  await context.sessionManager.ensureOnDisk();
+  const sessionFile = context.sessionManager.getSessionFile();
+  if (sessionFile !== undefined && fs.existsSync(sessionFile)) {
+    return fs.existsSync(`${path3.dirname(sessionFile)}.jsonl`);
+  }
+  const bootstrapped = bootstrappedSessionStore[LEGION_BOOTSTRAPPED_SESSION];
+  return bootstrapped !== undefined && bootstrapped !== sessionFile;
+}
+function isRegisteredSubagent(context) {
+  const registry = host.AgentRegistry?.global();
+  if (registry === undefined)
+    return false;
+  const sessionID = context.sessionManager.getSessionId();
+  const sessionFile = context.sessionManager.getSessionFile();
+  return registry.list().some((ref) => ref.kind !== "main" && (ref.session === null ? sessionFile !== undefined && ref.sessionFile === sessionFile : ref.session.sessionManager.getSessionId() === sessionID));
+}
+function subagentSessionCheck() {
+  let subagentSession;
+  return (context) => {
+    subagentSession ??= isSubagentSession(context);
+    return subagentSession;
+  };
+}
+
 // extensions/legion.ts
 var exitProcess = (code) => process.exit(code);
 function setLegionBootstrapExitForTests(hook) {
@@ -30767,29 +30800,11 @@ async function persistedTranscript(context) {
   if (!sessionFile?.endsWith(".jsonl")) {
     throw new Error("Legion session must have a persisted transcript");
   }
-  const agentId = path3.basename(sessionFile, ".jsonl");
+  const agentId = path4.basename(sessionFile, ".jsonl");
   if (!agentId)
     throw new Error("Legion session transcript has no agent id");
   return { sessionFile, agentId };
 }
-var LEGION_BOOTSTRAPPED_SESSION = Symbol.for("legion.pi-envoy.bootstrapped-session");
-var bootstrappedSessionStore = globalThis;
-function recordBootstrappedSession(sessionFile) {
-  bootstrappedSessionStore[LEGION_BOOTSTRAPPED_SESSION] = sessionFile;
-}
-function resetLegionBootstrappedSessionForTests() {
-  delete bootstrappedSessionStore[LEGION_BOOTSTRAPPED_SESSION];
-}
-async function isSubagentSession(context) {
-  await context.sessionManager.ensureOnDisk();
-  const sessionFile = context.sessionManager.getSessionFile();
-  if (sessionFile === undefined)
-    return false;
-  const bootstrapped = bootstrappedSessionStore[LEGION_BOOTSTRAPPED_SESSION];
-  if (bootstrapped !== undefined && bootstrapped !== sessionFile)
-    return true;
-  return fs.existsSync(`${path3.dirname(sessionFile)}.jsonl`);
-}
 function isSingleLegionCommand(command) {
   if (typeof command !== "string")
     return false;
@@ -30937,11 +30952,7 @@ function legionExtension(pi) {
   const controlCodec = import_nats.StringCodec();
   let capability;
   let bootstrap;
-  let subagentSession;
-  const checkSubagentSession = (context) => {
-    subagentSession ??= isSubagentSession(context);
-    return subagentSession;
-  };
+  const checkSubagentSession = subagentSessionCheck();
   let phaseWorkerPane;
   let daemonClient;
   const roleDaemon = () => {
@@ -31305,6 +31316,5 @@ function legionExtension(pi) {
 }
 export {
   legionExtension as default,
-  resetLegionBootstrappedSessionForTests,
   setLegionBootstrapExitForTests
 };

package/dist/skills/dispatch/SKILL.md

@@ -302,7 +302,9 @@ that follow from it:
 the question carries `dispatch://KEY/artifact/<slug>` (or `ref`), never just its filename. Text
 they must read to decide belongs in the spec in the first place — see [Artifacts](#artifacts).
 
-Attach an issue to the architecture components it changes, before decomposing it — children inherit the parent's attachment unless they choose their own, so attaching the root once classifies the whole tree:
+## Architecture components
+
+Attach every architectural issue to the components it changes. Attach the root before decomposing it: children inherit the root's effective attachment unless they deliberately set their own, so one root attachment classifies the whole tree.
 
 ```
 dispatch_issue({ project: "CORE", title: "...", components: { mode: "explicit", ids: ["dispatch-server", "web"] } })
@@ -311,7 +313,9 @@ dispatch_issue_update({ issue: "CORE-13", components: { mode: "none", reason: "h
 dispatch_issue_update({ issue: "CORE-14", components: { mode: "inherit" } })                  // back to the parent chain's attachment
 ```
 
-Component ids are the file names under the repository's `.dispatch/architecture/` (`web.md` → `web`); an id the model lacks, or an `external` component, is refused with `COMPONENTS_INPUT`. A closed issue can be classified without reopening. Two rules: declare and attach before decomposing work, and change code and its architecture description (`.dispatch/architecture/<id>.md`) in the same review — the tree at `GET /api/v1/projects/{key}/architecture` counts every issue whose effective set names a component or anything it contains.
+No native Dispatch tool lists component ids. Read the configured model with `GET /api/v1/projects/{key}/architecture` and use each non-external `components[].id`; when its source repository is your checkout, those ids are the file names under `.dispatch/architecture/` (`web.md` → `web`). An id absent from the model, or an `external` component, is refused with `COMPONENTS_INPUT`.
+
+Use `mode: "none"` with a concrete reason only when the work is genuinely non-architectural (for example hiring, process, or operations work). A closed issue can be classified without reopening. Change code and its architecture description (`.dispatch/architecture/<id>.md`) in the same review.
 
 Import a project's architecture model from its configured source repository now (a human configures the source in Settings):
 ```ts

package/package.json

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