@sjawhar/pi-legion-envoy 1.57.0 → 1.57.1

package/dist/envoy.js

@@ -30052,7 +30052,7 @@ var dispatchToolSpecs = [
     },
     description: "Edit an open question in place. Use it to correct or refine the same decision; retract the " + "old ask and open a new one when the decision itself changes. Previous text remains in the " + "event log. Only the asking session can edit it; answered or resolved asks cannot be edited.",
     arguments: (z) => ({
-      ask: z.string().describe("Ask id to edit."),
+      ask: z.string().describe("Ask id (uuid); an 8+ hex prefix unique among this session's own open asks works too."),
       question: z.string({ max: ASK_QUESTION_MAX }).describe(`Replacement decision question, at most ${ASK_QUESTION_MAX} characters.`).optional(),
       options: z.array(z.object({
         label: z.string().describe("Selectable option label."),
@@ -30078,7 +30078,7 @@ var dispatchToolSpecs = [
     },
     description: "Retract an open question that is moot or resolve one after finding the answer. This closes the question without answering it.",
     arguments: (z) => ({
-      ask: z.string().describe("Ask id to close."),
+      ask: z.string().describe("Ask id (uuid) to close; an 8+ hex prefix unique among this session's own open asks " + "works too."),
       kind: z.enum(["retracted", "resolved"]).describe("Whether the ask is retracted or self-resolved."),
       reason: z.string({ min: 1 }).describe("Why the open ask no longer needs a human answer.")
     })
@@ -30097,7 +30097,7 @@ var dispatchToolSpecs = [
     example: { ask: "01234567-0000-4000-8000-000000000001", action: "follow" },
     description: "Follow or unfollow an ask. Every session that opens or replies to an ask follows it: its answer, " + "edits, resolution, and replies reach that session directly. Unfollow to stop; follow to rejoin or " + "to hear an ask you never wrote to. Whole-issue subscription is separate: envoy_subscribe " + "notifications.dispatch.issue.<KEY>.>",
     arguments: (z) => ({
-      ask: z.string().describe("Full ask id (uuid)."),
+      ask: z.string().describe("Ask id (uuid); an 8+ hex prefix unique among this session's own open asks works too."),
       action: z.enum(["follow", "unfollow"]).describe("follow | unfollow")
     }),
     strict: true
@@ -32863,24 +32863,51 @@ function messageIdOf(value) {
   return id === undefined ? undefined : normalizeUUID(id);
 }
 var idPrefixPattern = /^[0-9a-f][0-9a-f-]{7,}$/i;
-async function resolveIdPrefix(tool, kind, ref, list) {
-  const fullID = normalizeUUID(ref.id);
+function refOwnerName(ref) {
+  return ref.owner.kind === "issue" ? ref.owner.issue : `${ref.owner.project}/${ref.artifact}`;
+}
+async function resolveIdPrefix(tool, kind, id, ownerName, list) {
+  const fullID = normalizeUUID(id);
   if (fullID !== undefined)
     return fullID;
-  const ownerName = ref.owner.kind === "issue" ? ref.owner.issue : `${ref.owner.project}/${ref.artifact}`;
-  if (!idPrefixPattern.test(ref.id)) {
+  if (!idPrefixPattern.test(id)) {
     throw new ToolInputError(tool, [
-      `${kind} id ${ref.id} must be a full uuid or a prefix of at least 8 hex characters`
+      `${kind} id ${id} must be a full uuid or a prefix of at least 8 hex characters`
     ]);
   }
-  const prefix = ref.id.toLowerCase();
+  const prefix = id.toLowerCase();
   const matches = (await list()).filter((item) => item.id.toLowerCase().startsWith(prefix));
   if (matches.length === 1 && matches[0] !== undefined)
     return matches[0].id;
   throw new ToolInputError(tool, [
-    matches.length === 0 ? `${kind} id ${ref.id} matches none of the ${kind}s on ${ownerName}; use the full id` : `${kind} id ${ref.id} matches ${matches.length} ${kind}s on ${ownerName}; use the full id`
+    matches.length === 0 ? `${kind} id ${id} matches none of the ${kind}s on ${ownerName}; use the full id` : `${kind} id ${id} matches ${matches.length} ${kind}s on ${ownerName}; use the full id`
   ]);
 }
+var askIdShapeProblem = "ask ids are uuids (a prefix of at least 8 hex characters works)";
+async function sessionOpenAsks(client, sessionId) {
+  const session = sessionId?.trim();
+  if (!session)
+    throw new Error("host session id is required to read this session's open asks");
+  return (await client.openAsks(session)).asks;
+}
+function openAskHints(asks) {
+  if (asks.length === 0)
+    return "you have no open asks";
+  const hints = asks.slice(0, askHintLimit).map((ask) => `${ask.id.slice(0, 8)} \u2014 ${textHead(ask.question)}`);
+  return `your open asks: ${hints.join("; ")}`;
+}
+async function resolveAskArgument(tool, args, client, sessionId) {
+  const id = askId(args);
+  if (normalizeUUID(id) === undefined && !idPrefixPattern.test(id)) {
+    const asks = await sessionOpenAsks(client, sessionId).catch(() => {
+      return;
+    });
+    throw new ToolInputError(tool, [
+      asks === undefined ? askIdShapeProblem : `${askIdShapeProblem}; ${openAskHints(asks)}`
+    ]);
+  }
+  return resolveIdPrefix(tool, "ask", id, "this session", () => sessionOpenAsks(client, sessionId));
+}
 function messageInReplyTo(args) {
   const inReplyTo = optionalString(args, "in_reply_to");
   return inReplyTo === undefined ? undefined : messageIdOf(inReplyTo);
@@ -32899,7 +32926,8 @@ function argumentProblems(tool, args) {
       break;
     }
     case "dispatch_edit_ask":
-    case "dispatch_resolve_ask": {
+    case "dispatch_resolve_ask":
+    case "dispatch_follow": {
       const ask = optionalString(args, "ask");
       if (ask?.startsWith("dispatch://") && parseDispatchRef(ask)?.kind !== "ask") {
         problems.push(askIdProblem);
@@ -33693,7 +33721,8 @@ async function executeDispatchTool(input) {
     }
     case "dispatch_resolve_ask": {
       const kind = stringArg(args, "kind");
-      const ask = await client.resolveAsk(stringArg(args, "ask"), {
+      const id = await resolveAskArgument(input.tool, args, client, input.sessionId);
+      const ask = await client.resolveAsk(id, {
         kind,
         reason: stringArg(args, "reason"),
         actor
@@ -33712,11 +33741,11 @@ async function executeDispatchTool(input) {
       let document;
       if (ref?.owner.kind === "issue") {
         const issueKey = ref.owner.issue;
-        id = await resolveIdPrefix(input.tool, "comment", ref, () => client.getComments(issueKey));
+        id = await resolveIdPrefix(input.tool, "comment", ref.id, refOwnerName(ref), () => client.getComments(issueKey));
       } else if (ref !== null) {
         const artifact = (await resolveArtifact(client, ref.owner, ref.artifact)).artifact;
         document = artifact;
-        id = await resolveIdPrefix(input.tool, "comment", ref, () => client.getArtifactComments(artifact.id));
+        id = await resolveIdPrefix(input.tool, "comment", ref.id, refOwnerName(ref), () => client.getArtifactComments(artifact.id));
       }
       const comment = await client.resolveComment(id, actor);
       const { label, details } = await commentOwnerResult(client, comment, document);
@@ -33760,7 +33789,8 @@ ${followsAsk(askOwner)}`,
       const options = args.options;
       const multiple = optionalBoolean(args, "multiple");
       const urgency = askUrgency(args);
-      const ask = await client.editAsk(askId(args), {
+      const id = await resolveAskArgument(input.tool, args, client, input.sessionId);
+      const ask = await client.editAsk(id, {
         ...question === undefined ? {} : { question },
         ...Array.isArray(options) ? { options } : {},
         ...multiple === undefined ? {} : { multiple },
@@ -33987,7 +34017,7 @@ ${trailer.join(`
       const sessionId = input.sessionId?.trim();
       if (!sessionId)
         throw new Error("host session id is required for dispatch_follow");
-      const ask = askId(args);
+      const ask = await resolveAskArgument(input.tool, args, client, sessionId);
       const action = stringArg(args, "action");
       if (action === "unfollow") {
         await client.unfollowAsk(ask, sessionId, actor);
@@ -34003,7 +34033,7 @@ ${trailer.join(`
     case "dispatch_read": {
       if (ownerArguments.ref?.kind === "ask") {
         const ref = ownerArguments.ref;
-        const id = await resolveIdPrefix(input.tool, "ask", ref, async () => ref.owner.kind === "issue" ? client.listIssueAsks(ref.owner.issue) : client.getArtifactAsks((await resolveArtifact(client, ref.owner, ref.artifact)).artifact.id, "all"));
+        const id = await resolveIdPrefix(input.tool, "ask", ref.id, refOwnerName(ref), async () => ref.owner.kind === "issue" ? client.listIssueAsks(ref.owner.issue) : client.getArtifactAsks((await resolveArtifact(client, ref.owner, ref.artifact)).artifact.id, "all"));
         const askRead = await client.getAsk(id);
         const askRef = refTarget(ref, "ask", id);
         return {
@@ -34013,7 +34043,7 @@ ${trailer.join(`
       }
       if (ownerArguments.ref?.kind === "comment") {
         const ref = ownerArguments.ref;
-        const id = await resolveIdPrefix(input.tool, "comment", ref, async () => ref.owner.kind === "issue" ? client.getComments(ref.owner.issue) : client.getArtifactComments((await resolveArtifact(client, ref.owner, ref.artifact)).artifact.id));
+        const id = await resolveIdPrefix(input.tool, "comment", ref.id, refOwnerName(ref), async () => ref.owner.kind === "issue" ? client.getComments(ref.owner.issue) : client.getArtifactComments((await resolveArtifact(client, ref.owner, ref.artifact)).artifact.id));
         const comment = await client.getComment(id);
         const commentRef = refTarget(ref, "comment", id);
         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.57.0",
+  version: "1.57.1",
   type: "module",
   omp: {
     extensions: [
@@ -29371,7 +29371,7 @@ var dispatchToolSpecs = [
     },
     description: "Edit an open question in place. Use it to correct or refine the same decision; retract the " + "old ask and open a new one when the decision itself changes. Previous text remains in the " + "event log. Only the asking session can edit it; answered or resolved asks cannot be edited.",
     arguments: (z) => ({
-      ask: z.string().describe("Ask id to edit."),
+      ask: z.string().describe("Ask id (uuid); an 8+ hex prefix unique among this session's own open asks works too."),
       question: z.string({ max: ASK_QUESTION_MAX }).describe(`Replacement decision question, at most ${ASK_QUESTION_MAX} characters.`).optional(),
       options: z.array(z.object({
         label: z.string().describe("Selectable option label."),
@@ -29397,7 +29397,7 @@ var dispatchToolSpecs = [
     },
     description: "Retract an open question that is moot or resolve one after finding the answer. This closes the question without answering it.",
     arguments: (z) => ({
-      ask: z.string().describe("Ask id to close."),
+      ask: z.string().describe("Ask id (uuid) to close; an 8+ hex prefix unique among this session's own open asks " + "works too."),
       kind: z.enum(["retracted", "resolved"]).describe("Whether the ask is retracted or self-resolved."),
       reason: z.string({ min: 1 }).describe("Why the open ask no longer needs a human answer.")
     })
@@ -29416,7 +29416,7 @@ var dispatchToolSpecs = [
     example: { ask: "01234567-0000-4000-8000-000000000001", action: "follow" },
     description: "Follow or unfollow an ask. Every session that opens or replies to an ask follows it: its answer, " + "edits, resolution, and replies reach that session directly. Unfollow to stop; follow to rejoin or " + "to hear an ask you never wrote to. Whole-issue subscription is separate: envoy_subscribe " + "notifications.dispatch.issue.<KEY>.>",
     arguments: (z) => ({
-      ask: z.string().describe("Full ask id (uuid)."),
+      ask: z.string().describe("Ask id (uuid); an 8+ hex prefix unique among this session's own open asks works too."),
       action: z.enum(["follow", "unfollow"]).describe("follow | unfollow")
     }),
     strict: true

package/dist/skills/dispatch/SKILL.md

@@ -708,7 +708,9 @@ the host tells you once per ask. Leave a thread you no longer need, or rejoin on
 dispatch_follow({ ask, action: "follow" | "unfollow" })
 ```
 
-`ask` is the full ask id or a `dispatch://KEY/ask/<id>` reference. A human may also remove you from the ask card; either way you are
+`ask` is the full ask id or a `dispatch://KEY/ask/<id>` reference; `dispatch_follow`, `dispatch_edit_ask`, and
+`dispatch_resolve_ask` also take an 8+ hex prefix that is unique among your own open asks, and refuse anything shorter by naming
+those asks. A human may also remove you from the ask card; either way you are
 told with an `ask.follower_removed` notice, and a human adding you arrives as `ask.follower_added`.
 
 No write subscribes you to an issue or document. Following covers your own asks and the threads you joined; everything else on the

package/package.json

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