@sjawhar/pi-legion-envoy 0.23.0 → 0.25.0

package/README.md

@@ -98,7 +98,9 @@ Configure the shared `envoy.json` with:
 
 The user file is `~/.config/opencode/envoy.json`; a
 `<cwd>/.opencode/envoy.json` file shallow-merges over it. `DISPATCH_URL` and
-`DISPATCH_TOKEN` override the file values for one process. Invalid configuration,
+`DISPATCH_TOKEN` override the file values for one process. Omitting
+`dispatch.serverUrl` while `dispatch.enabled` is true targets
+`http://localhost:8766`, the Go server's listen address. Invalid configuration,
 an invalid URL, or an empty token leaves the nine tools unavailable and reports
 the source of the error.
 

package/dist/envoy.js

@@ -29677,6 +29677,8 @@ var CommentEventPayloadSchema = object({
   artifact_name: string2().optional(),
   body: string2().optional(),
   reply_to: string2().nullish(),
+  ask_id: string2().nullish(),
+  ask_question: string2().optional(),
   anchor: object({ quote: string2().optional() }).nullish(),
   suggestion: object({ replace_with: string2().optional() }).nullish()
 });
@@ -29727,14 +29729,15 @@ var dispatchToolSpecs = [
   },
   {
     name: "dispatch_comment",
-    description: "Add review feedback to an issue or document quote. Do not use it for an exact replacement; use " + `dispatch_suggest instead. Body is at most 2,000 characters. ${ISSUE_REFERENCE}`,
+    description: "Add review feedback to an issue or document quote, or reply to a question asked with dispatch_ask. " + "Do not use it for an exact replacement; use " + `dispatch_suggest instead. Body is at most 2,000 characters. ${ISSUE_REFERENCE}`,
     arguments: (z) => ({
       issue: z.string().describe(ISSUE_REFERENCE),
       artifact: z.string().describe("Artifact slug or id required when quote is given.").optional(),
       quote: z.string().describe("Optional exact quoted document text.").optional(),
       occurrence: z.number({ int: true, min: 0 }).describe("Optional zero-based occurrence of quote.").optional(),
       body: z.string({ max: 2000 }).describe("Review comment, at most 2,000 characters."),
-      reply_to: z.string().describe("Optional comment id to reply to.").optional()
+      reply_to: z.string().describe("Optional comment id to reply to.").optional(),
+      reply_to_ask: z.string().describe("Optional ask id to reply to, threading this comment under that question. Mutually " + "exclusive with reply_to.").optional()
     })
   },
   {
@@ -30809,6 +30812,12 @@ function dispatchPayload(event) {
   const parsed = schema.safeParse(event.payload);
   return parsed.success ? parsed.data : event.payload;
 }
+function dispatchAskQuestion(event) {
+  if (event.type !== "comment.created")
+    return;
+  const parsed = CommentEventPayloadSchema.safeParse(event.payload);
+  return parsed.success && parsed.data.ask_question !== "" ? parsed.data.ask_question : undefined;
+}
 function parseDispatchFrame(rawPayload) {
   let value;
   try {
@@ -30858,6 +30867,7 @@ function renderInbound(raw, sessionID, subject) {
   }
   let dispatchEvent;
   let dispatchIssue;
+  let askQuestion;
   const dispatchRendered = envelope.source === "dispatch" && envelope.payload !== undefined;
   if (envelope.source === "dispatch") {
     if (envelope.payload === undefined) {
@@ -30871,6 +30881,7 @@ function renderInbound(raw, sessionID, subject) {
         if (frame.event.actor.kind === "session" && frame.event.actor.id === sessionID) {
           return { skip: true, content: "", envelope };
         }
+        askQuestion = dispatchAskQuestion(frame.event);
         dispatchEvent = {
           issue_key: frame.event.issue_key,
           type: frame.event.type,
@@ -30918,7 +30929,7 @@ ${envelope.payload ?? ""}`;
     ...envelope.expires_at === undefined ? {} : { by: inboundTimestamp(envelope.expires_at) },
     ...envelope.urgency === undefined ? {} : { urgency: envelope.urgency },
     ...envelope.expects_reply === undefined ? {} : { expects_reply: envelope.expects_reply },
-    ...envelope.in_reply_to === undefined ? {} : { re: envelope.in_reply_to },
+    ...envelope.in_reply_to === undefined ? {} : { re: askQuestion ?? envelope.in_reply_to },
     ...envelope.supersedes === undefined ? {} : { supersedes: envelope.supersedes },
     ...reply === undefined ? {} : { reply_with: reply },
     ...role === undefined ? {} : {
@@ -31151,6 +31162,9 @@ class DispatchClient {
   async issue(input) {
     return this.#json("POST", ["api", "v1", "issues"], input);
   }
+  async listIssues(options = {}) {
+    return this.#json("GET", ["api", "v1", "issues"], undefined, options);
+  }
   async getIssue(issue) {
     return this.#json("GET", ["api", "v1", "issues", await this.#resolveIssue(issue)]);
   }
@@ -31294,8 +31308,11 @@ class DispatchClient {
   #url(path, query) {
     const url = new URL(`${path.map((segment) => encodeURIComponent(segment)).join("/")}`, `${this.#baseUrl}/`);
     if (query) {
-      for (const [name, value] of Object.entries(query))
-        url.searchParams.set(name, String(value));
+      for (const [name, value] of Object.entries(query)) {
+        if (typeof value === "string" || typeof value === "number") {
+          url.searchParams.set(name, String(value));
+        }
+      }
     }
     return url.toString();
   }
@@ -31471,8 +31488,12 @@ function childrenSummary(issue) {
   ].join(`
 `);
 }
-function askSummary(ask) {
+function askSummary({ ask, replies }) {
   const answer = ask.answer;
+  const chain = replies.flatMap((reply) => [
+    `${reply.id} \xB7 ${reply.author.kind} ${reply.author.id}`,
+    `Body: ${reply.body}`
+  ]);
   return [
     `Question: ${ask.question}`,
     "Options:",
@@ -31483,7 +31504,9 @@ function askSummary(ask) {
       `- By: ${answer.user}`,
       `- Selected: ${answer.selected.length === 0 ? "none" : answer.selected.join(", ")}`,
       ...answer.text === null ? [] : [`- Text: ${answer.text}`]
-    ]
+    ],
+    "Replies:",
+    ...chain.length === 0 ? ["- none"] : chain
   ].join(`
 `);
 }
@@ -31582,10 +31605,15 @@ async function executeDispatchTool(input) {
       const resolved = artifactReference ? await resolveArtifact(client, issue(), artifactReference) : undefined;
       const anchored = resolved ? anchor(resolved.artifact, args) : undefined;
       const replyTo = optionalString(args, "reply_to");
+      const replyToAsk = optionalString(args, "reply_to_ask");
+      if (replyTo !== undefined && replyToAsk !== undefined) {
+        throw new Error("reply_to and reply_to_ask cannot both be set");
+      }
       const comment = await client.comment(issue(), {
         body: stringArg(args, "body"),
         ...anchored === undefined ? {} : { anchor: anchored },
         ...replyTo === undefined ? {} : { reply_to: replyTo },
+        ...replyToAsk === undefined ? {} : { ask_id: replyToAsk },
         actor
       });
       return {
@@ -31694,10 +31722,13 @@ Open anchored asks/comments: ${marks.join(", ")}`,
     }
     case "dispatch_read": {
       if (issueArguments.ref?.kind === "ask") {
-        const ask = await client.getAsk(issueArguments.ref.id);
+        const askRead = await client.getAsk(issueArguments.ref.id);
         return {
-          text: askSummary(ask),
-          details: { issue: ask.issue_key, topic: dispatchIssueSubject(ask.issue_key, ">") }
+          text: askSummary(askRead),
+          details: {
+            issue: askRead.ask.issue_key,
+            topic: dispatchIssueSubject(askRead.ask.issue_key, ">")
+          }
         };
       }
       if (issueArguments.ref?.kind === "comment") {

package/dist/legion.js

@@ -29675,6 +29675,8 @@ var CommentEventPayloadSchema = object({
   artifact_name: string2().optional(),
   body: string2().optional(),
   reply_to: string2().nullish(),
+  ask_id: string2().nullish(),
+  ask_question: string2().optional(),
   anchor: object({ quote: string2().optional() }).nullish(),
   suggestion: object({ replace_with: string2().optional() }).nullish()
 });
@@ -29725,14 +29727,15 @@ var dispatchToolSpecs = [
   },
   {
     name: "dispatch_comment",
-    description: "Add review feedback to an issue or document quote. Do not use it for an exact replacement; use " + `dispatch_suggest instead. Body is at most 2,000 characters. ${ISSUE_REFERENCE}`,
+    description: "Add review feedback to an issue or document quote, or reply to a question asked with dispatch_ask. " + "Do not use it for an exact replacement; use " + `dispatch_suggest instead. Body is at most 2,000 characters. ${ISSUE_REFERENCE}`,
     arguments: (z) => ({
       issue: z.string().describe(ISSUE_REFERENCE),
       artifact: z.string().describe("Artifact slug or id required when quote is given.").optional(),
       quote: z.string().describe("Optional exact quoted document text.").optional(),
       occurrence: z.number({ int: true, min: 0 }).describe("Optional zero-based occurrence of quote.").optional(),
       body: z.string({ max: 2000 }).describe("Review comment, at most 2,000 characters."),
-      reply_to: z.string().describe("Optional comment id to reply to.").optional()
+      reply_to: z.string().describe("Optional comment id to reply to.").optional(),
+      reply_to_ask: z.string().describe("Optional ask id to reply to, threading this comment under that question. Mutually " + "exclusive with reply_to.").optional()
     })
   },
   {
@@ -31301,6 +31304,12 @@ function dispatchPayload(event) {
   const parsed = schema.safeParse(event.payload);
   return parsed.success ? parsed.data : event.payload;
 }
+function dispatchAskQuestion(event) {
+  if (event.type !== "comment.created")
+    return;
+  const parsed = CommentEventPayloadSchema.safeParse(event.payload);
+  return parsed.success && parsed.data.ask_question !== "" ? parsed.data.ask_question : undefined;
+}
 function parseDispatchFrame(rawPayload) {
   let value;
   try {
@@ -31350,6 +31359,7 @@ function renderInbound(raw, sessionID, subject) {
   }
   let dispatchEvent;
   let dispatchIssue;
+  let askQuestion;
   const dispatchRendered = envelope.source === "dispatch" && envelope.payload !== undefined;
   if (envelope.source === "dispatch") {
     if (envelope.payload === undefined) {
@@ -31363,6 +31373,7 @@ function renderInbound(raw, sessionID, subject) {
         if (frame.event.actor.kind === "session" && frame.event.actor.id === sessionID) {
           return { skip: true, content: "", envelope };
         }
+        askQuestion = dispatchAskQuestion(frame.event);
         dispatchEvent = {
           issue_key: frame.event.issue_key,
           type: frame.event.type,
@@ -31410,7 +31421,7 @@ ${envelope.payload ?? ""}`;
     ...envelope.expires_at === undefined ? {} : { by: inboundTimestamp(envelope.expires_at) },
     ...envelope.urgency === undefined ? {} : { urgency: envelope.urgency },
     ...envelope.expects_reply === undefined ? {} : { expects_reply: envelope.expects_reply },
-    ...envelope.in_reply_to === undefined ? {} : { re: envelope.in_reply_to },
+    ...envelope.in_reply_to === undefined ? {} : { re: askQuestion ?? envelope.in_reply_to },
     ...envelope.supersedes === undefined ? {} : { supersedes: envelope.supersedes },
     ...reply === undefined ? {} : { reply_with: reply },
     ...role === undefined ? {} : {
@@ -31636,6 +31647,9 @@ class DispatchClient {
   async issue(input) {
     return this.#json("POST", ["api", "v1", "issues"], input);
   }
+  async listIssues(options = {}) {
+    return this.#json("GET", ["api", "v1", "issues"], undefined, options);
+  }
   async getIssue(issue) {
     return this.#json("GET", ["api", "v1", "issues", await this.#resolveIssue(issue)]);
   }
@@ -31779,8 +31793,11 @@ class DispatchClient {
   #url(path, query) {
     const url = new URL(`${path.map((segment) => encodeURIComponent(segment)).join("/")}`, `${this.#baseUrl}/`);
     if (query) {
-      for (const [name, value] of Object.entries(query))
-        url.searchParams.set(name, String(value));
+      for (const [name, value] of Object.entries(query)) {
+        if (typeof value === "string" || typeof value === "number") {
+          url.searchParams.set(name, String(value));
+        }
+      }
     }
     return url.toString();
   }
@@ -31956,8 +31973,12 @@ function childrenSummary(issue) {
   ].join(`
 `);
 }
-function askSummary(ask) {
+function askSummary({ ask, replies }) {
   const answer = ask.answer;
+  const chain = replies.flatMap((reply) => [
+    `${reply.id} \xB7 ${reply.author.kind} ${reply.author.id}`,
+    `Body: ${reply.body}`
+  ]);
   return [
     `Question: ${ask.question}`,
     "Options:",
@@ -31968,7 +31989,9 @@ function askSummary(ask) {
       `- By: ${answer.user}`,
       `- Selected: ${answer.selected.length === 0 ? "none" : answer.selected.join(", ")}`,
       ...answer.text === null ? [] : [`- Text: ${answer.text}`]
-    ]
+    ],
+    "Replies:",
+    ...chain.length === 0 ? ["- none"] : chain
   ].join(`
 `);
 }
@@ -32067,10 +32090,15 @@ async function executeDispatchTool(input) {
       const resolved = artifactReference ? await resolveArtifact(client, issue(), artifactReference) : undefined;
       const anchored = resolved ? anchor(resolved.artifact, args) : undefined;
       const replyTo = optionalString(args, "reply_to");
+      const replyToAsk = optionalString(args, "reply_to_ask");
+      if (replyTo !== undefined && replyToAsk !== undefined) {
+        throw new Error("reply_to and reply_to_ask cannot both be set");
+      }
       const comment = await client.comment(issue(), {
         body: stringArg(args, "body"),
         ...anchored === undefined ? {} : { anchor: anchored },
         ...replyTo === undefined ? {} : { reply_to: replyTo },
+        ...replyToAsk === undefined ? {} : { ask_id: replyToAsk },
         actor
       });
       return {
@@ -32179,10 +32207,13 @@ Open anchored asks/comments: ${marks.join(", ")}`,
     }
     case "dispatch_read": {
       if (issueArguments.ref?.kind === "ask") {
-        const ask = await client.getAsk(issueArguments.ref.id);
+        const askRead = await client.getAsk(issueArguments.ref.id);
         return {
-          text: askSummary(ask),
-          details: { issue: ask.issue_key, topic: dispatchIssueSubject(ask.issue_key, ">") }
+          text: askSummary(askRead),
+          details: {
+            issue: askRead.ask.issue_key,
+            topic: dispatchIssueSubject(askRead.ask.issue_key, ">")
+          }
         };
       }
       if (issueArguments.ref?.kind === "comment") {

package/dist/skills/dispatch/SKILL.md

@@ -46,6 +46,12 @@ Anchor a document question with `anchor: { artifact, quote, occurrence? }`; `occ
 zero-based and selects a repeated quote. An anchor whose quote disappears becomes orphaned but
 remains readable against its original document version. An ask stays open until a human answers;
 you cannot withdraw it. Ask once and well. The host steers you when a human answers.
+
+An ask is a thread, not a dead end: a human can reply to it before or after answering, and you
+(the asker) can reply too — e.g. acknowledging a clarifying question, or following up after the
+answer. Use `reply_to_ask` on `dispatch_comment` to reply under your own ask; it is mutually
+exclusive with `reply_to`.
+
 ## The spec is where narrative goes
 
 Read the current document before changing it:
@@ -85,11 +91,12 @@ message.
 Add feedback with:
 
 ```ts
-dispatch_comment({ issue, artifact?, quote?, occurrence?, body, reply_to? })
+dispatch_comment({ issue, artifact?, quote?, occurrence?, body, reply_to?, reply_to_ask? })
 ```
 
 It returns `details` `{ issue, topic, comment }`. `quote` requires `artifact`; omit both for a
-floating issue comment. Use `reply_to` to continue a comment.
+floating issue comment. 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:
 
@@ -149,9 +156,9 @@ dispatch_read({ issue?, ref? })
 ```
 
 With an issue ref, it returns the issue summary, open asks, and recent events with `details`
-`{ issue }`. With an ask ref, it returns that ask's question, options, state, and answer. With a
-comment ref, it returns that comment and its quoted reply chain. Use `dispatch_doc_read` for
-document contents.
+`{ issue }`. With an ask ref, it returns that ask's question, options, state, answer, and its
+reply thread. With a comment ref, it returns that comment and its quoted reply chain. Use
+`dispatch_doc_read` for document contents.
 
 ## References
 

package/package.json

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