@sjawhar/pi-legion-envoy 1.53.1 → 1.53.3

package/dist/envoy.js

@@ -29756,13 +29756,20 @@ var CommentEventPayloadSchema = object({
   ask_state: _enum2(["open", "answered", "resolved"]).optional(),
   ask_waiting_on: _enum2(ASK_TURNS).optional(),
   turn: _enum2(ASK_TURNS).nullish(),
-  anchor: object({ block_id: string2().nullable().optional(), quote: string2().optional() }).nullish(),
+  anchor: object({
+    artifact_id: string2().optional(),
+    block_id: string2().nullable().optional(),
+    mark_id: string2().optional(),
+    orphaned: boolean2().optional(),
+    quote: string2().optional(),
+    version: number2().int().optional()
+  }).nullish(),
   suggestion: object({ replace_with: string2().optional() }).nullish(),
   author: object({ kind: string2(), id: string2() }).optional(),
   created_at: string2().optional(),
   mentions: array(CommentMentionSchema).optional(),
   deliveries: array(CommentDeliverySchema).optional()
-});
+}).passthrough();
 var MessageEventPayloadSchema = object({
   id: string2().optional(),
   body: string2().optional(),
@@ -31452,10 +31459,12 @@ var DISPATCH_PAYLOAD_SCHEMAS = {
   "artifact.created": ArtifactCreatedEventPayloadSchema,
   "artifact.version": ArtifactVersionEventPayloadSchema,
   "ask.opened": AskEventPayloadSchema,
+  "ask.anchor_refreshed": AskEventPayloadSchema,
   "ask.edited": AskEditedEventPayloadSchema,
   "ask.answered": AskEventPayloadSchema,
   "ask.resolved": AskEventPayloadSchema,
   "comment.created": CommentEventPayloadSchema,
+  "comment.anchor_refreshed": CommentEventPayloadSchema,
   "comment.resolved": CommentEventPayloadSchema,
   "comment.reopened": CommentEventPayloadSchema,
   "comment.edited": CommentEventPayloadSchema,
@@ -32954,6 +32963,20 @@ async function resolveOwnerArguments(tool, input, cwd, env, exec, serverUrl, pro
   const issue = `${repo}#${legionIssue}`;
   return { args: { ...args, issue }, ref, owner: { kind: "issue", issue } };
 }
+function artifactByReference(artifacts, artifactReference, owner) {
+  const byIdOrSlug = artifacts.find((candidate) => candidate.id === artifactReference) ?? artifacts.find((candidate) => candidate.slug === artifactReference);
+  if (byIdOrSlug !== undefined)
+    return byIdOrSlug;
+  const byName = artifacts.filter((candidate) => candidate.name === artifactReference);
+  if (byName.length > 1) {
+    throw new Error(documentReferenceProblem(artifactReference, byName, owner, true));
+  }
+  const [artifact] = byName;
+  if (artifact === undefined) {
+    throw new Error(documentReferenceProblem(artifactReference, artifacts, owner));
+  }
+  return artifact;
+}
 async function resolveArtifact(client, owner, artifactReference) {
   if (owner.kind === "project") {
     if (artifactReference === undefined) {
@@ -32968,17 +32991,10 @@ async function resolveArtifact(client, owner, artifactReference) {
       if (!(error instanceof DispatchServiceError) || error.status !== 404)
         throw error;
       const artifacts = await client.listProjectArtifacts(owner.project, true);
-      const artifact = artifacts.find((candidate) => candidate.id === artifactReference) ?? artifacts.find((candidate) => candidate.slug === artifactReference);
-      if (artifact)
-        return { owner, artifact };
-      const names = artifacts.filter((candidate) => candidate.name === artifactReference);
-      if (names.length > 1) {
-        throw new Error(documentReferenceProblem(artifactReference, names, "project", true));
-      }
-      if (names[0] === undefined) {
-        throw new Error(documentReferenceProblem(artifactReference, artifacts, "project"));
-      }
-      return { owner, artifact: names[0] };
+      return {
+        owner,
+        artifact: artifactByReference(artifacts, artifactReference, "project")
+      };
     }
   }
   const issue = await client.getIssue(owner.issue);
@@ -32986,14 +33002,7 @@ async function resolveArtifact(client, owner, artifactReference) {
   if (artifactReference === undefined || artifactReference === "spec") {
     artifact = issue.artifacts.find((candidate) => candidate.primary || candidate.id === issue.primary_artifact_id);
   } else {
-    artifact = issue.artifacts.find((candidate) => candidate.id === artifactReference || candidate.slug === artifactReference);
-    if (artifact === undefined) {
-      const names = issue.artifacts.filter((candidate) => candidate.name === artifactReference);
-      if (names.length > 1) {
-        throw new Error(documentReferenceProblem(artifactReference, names, "issue", true));
-      }
-      artifact = names[0];
-    }
+    artifact = artifactByReference(issue.artifacts, artifactReference, "issue");
   }
   if (!artifact) {
     throw new Error(documentReferenceProblem(artifactReference ?? "spec", issue.artifacts, "issue"));
@@ -33141,12 +33150,14 @@ function logSummary(issue, events) {
 function eventHead(event) {
   switch (event.type) {
     case "ask.opened":
+    case "ask.anchor_refreshed":
     case "ask.edited":
     case "ask.resolved":
       return textHead(event.payload.question);
     case "ask.answered":
       return `${textHead(event.payload.question)} -> ${textHead(askAnswerText(event.payload.answer))}`;
     case "comment.created":
+    case "comment.anchor_refreshed":
     case "comment.edited":
     case "comment.resolved":
     case "comment.reopened":
@@ -33284,19 +33295,21 @@ function messageSummary({ message, replies }, graph) {
 `);
 }
 async function openArtifactMarks(client, resolved) {
-  const asks = resolved.owner.kind === "project" ? await client.getArtifactAsks(resolved.artifact.id) : resolved.issue?.open_asks ?? [];
+  const asksPromise = resolved.owner.kind === "project" ? client.getArtifactAsks(resolved.artifact.id) : Promise.resolve(resolved.issue?.open_asks ?? []);
+  const commentsPromise = resolved.owner.kind === "project" ? client.getArtifactComments(resolved.artifact.id) : client.getComments(resolved.issue?.key ?? "", resolved.artifact.id);
+  const commentsResultPromise = commentsPromise.then((value) => ({ status: "fulfilled", value }), (reason) => ({ status: "rejected", reason }));
+  const asks = await asksPromise;
   const marks = asks.filter((ask) => ask.state === "open" && ask.anchor?.artifact_id === resolved.artifact.id).map((ask) => `ask ${ask.id}`);
-  let comments;
-  try {
-    comments = resolved.owner.kind === "project" ? await client.getArtifactComments(resolved.artifact.id) : await client.getComments(resolved.issue?.key ?? "", resolved.artifact.id);
-  } catch (error) {
-    if (error instanceof DispatchServiceError && error.status === 404)
+  const commentsResult = await commentsResultPromise;
+  if (commentsResult.status === "rejected") {
+    if (commentsResult.reason instanceof DispatchServiceError && commentsResult.reason.status === 404) {
       return marks;
-    throw error;
+    }
+    throw commentsResult.reason;
   }
   return [
     ...marks,
-    ...comments.filter((comment) => !comment.resolved && comment.anchor?.artifact_id === resolved.artifact.id).map((comment) => `comment ${comment.id}`)
+    ...commentsResult.value.filter((comment) => !comment.resolved && comment.anchor?.artifact_id === resolved.artifact.id).map((comment) => `comment ${comment.id}`)
   ];
 }
 async function executeDispatchTool(input) {
@@ -33715,8 +33728,14 @@ ${followsAsk(askOwner)}`,
       const artifactReference = optionalString(args, "artifact") ?? (ownerArguments.ref?.kind === "spec" || ownerArguments.ref?.kind === "artifact" ? ownerArguments.ref.id : undefined);
       const resolved = await resolveArtifact(client, documentOwner(), artifactReference);
       const version = optionalNumber(args, "version") ?? ownerArguments.ref?.version;
-      const document = await client.docRead(resolved.artifact.id, version);
-      const marks = await openArtifactMarks(client, resolved);
+      const documentPromise = client.docRead(resolved.artifact.id, version);
+      const marksPromise = openArtifactMarks(client, resolved);
+      const marksResultPromise = marksPromise.then((value) => ({ status: "fulfilled", value }), (reason) => ({ status: "rejected", reason }));
+      const document = await documentPromise;
+      const marksResult = await marksResultPromise;
+      if (marksResult.status === "rejected")
+        throw marksResult.reason;
+      const marks = marksResult.value;
       const approval = approvalLine(resolved.artifact);
       const trailer = [
         ...marks.length === 0 ? [] : [`Open anchored asks/comments: ${marks.join(", ")}`],
@@ -33851,21 +33870,26 @@ ${trailer.join(`
           }
         };
       }
-      const read = await client.read(issue());
+      const issueKey = issue();
       if (ownerArguments.ref?.kind === "log") {
+        const read = await client.read(issueKey);
         return {
           text: logSummary(read.issue, read.events),
           details: { issue: read.issue.key }
         };
       }
       if (ownerArguments.ref?.kind === "children") {
+        const read = await client.read(issueKey);
         return {
           text: childrenSummary(read.issue),
           details: { issue: read.issue.key }
         };
       }
+      const readPromise = client.read(issueKey);
+      const referencesPromise = issueReferencesOrUnavailable(client, issueKey);
+      const read = await readPromise;
       const [references, graph] = await Promise.all([
-        issueReferencesOrUnavailable(client, read.issue.key),
+        referencesPromise,
         graphSections(client, dispatchIssueRef(read.issue.key))
       ]);
       return {

package/dist/legion.js

@@ -16145,7 +16145,7 @@ import { logger } from "@oh-my-pi/pi-utils";
 // package.json
 var package_default = {
   name: "@sjawhar/pi-legion-envoy",
-  version: "1.53.1",
+  version: "1.53.3",
   type: "module",
   omp: {
     extensions: [
@@ -29075,13 +29075,20 @@ var CommentEventPayloadSchema = object({
   ask_state: _enum2(["open", "answered", "resolved"]).optional(),
   ask_waiting_on: _enum2(ASK_TURNS).optional(),
   turn: _enum2(ASK_TURNS).nullish(),
-  anchor: object({ block_id: string2().nullable().optional(), quote: string2().optional() }).nullish(),
+  anchor: object({
+    artifact_id: string2().optional(),
+    block_id: string2().nullable().optional(),
+    mark_id: string2().optional(),
+    orphaned: boolean2().optional(),
+    quote: string2().optional(),
+    version: number2().int().optional()
+  }).nullish(),
   suggestion: object({ replace_with: string2().optional() }).nullish(),
   author: object({ kind: string2(), id: string2() }).optional(),
   created_at: string2().optional(),
   mentions: array(CommentMentionSchema).optional(),
   deliveries: array(CommentDeliverySchema).optional()
-});
+}).passthrough();
 var MessageEventPayloadSchema = object({
   id: string2().optional(),
   body: string2().optional(),

package/package.json

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