@sjawhar/pi-legion-envoy 1.53.2 → 1.53.3

package/dist/envoy.js

@@ -32963,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) {
@@ -32977,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);
@@ -32995,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"));
@@ -33295,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) {
@@ -33726,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(", ")}`],
@@ -33862,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.2",
+  version: "1.53.3",
   type: "module",
   omp: {
     extensions: [

package/package.json

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