@dosu/cli 0.28.0 → 0.28.1

package/bin/dosu.js

@@ -4577,7 +4577,7 @@ var init_dist4 = __esm(() => {
 function getVersionString() {
   return `v${VERSION}`;
 }
-var VERSION = "0.28.0", INSTALL_CHANNEL = "npm";
+var VERSION = "0.28.1", INSTALL_CHANNEL = "npm";
 
 // src/debug/logger.ts
 import {
@@ -15664,17 +15664,32 @@ function requireConfig10() {
 function isNotFound(err) {
   return isTRPCClientError(err) && err.data?.code === "NOT_FOUND";
 }
-async function resolveChange(client, id) {
+function isInvalidId(err) {
+  return isTRPCClientError(err) && err.data?.code === "UNPROCESSABLE_CONTENT";
+}
+var DRAFT_MESSAGE_ID_PREFIX = "draft_message:";
+function isDraftId(id) {
+  return id.startsWith(DRAFT_MESSAGE_ID_PREFIX);
+}
+function bareMessageId(id) {
+  return id.slice(DRAFT_MESSAGE_ID_PREFIX.length);
+}
+function truncateId(id) {
+  return (isDraftId(id) ? bareMessageId(id) : id).slice(0, 8);
+}
+async function requireChange(client, id) {
   try {
     return await client.review.getChange.query({ id });
   } catch (err) {
-    if (isNotFound(err))
-      return null;
+    if (isNotFound(err) || isInvalidId(err)) {
+      console.error(import_picocolors20.default.red(`No review item found for '${id}'. Run 'dosu review list' to see pending items.`));
+      process.exit(1);
+    }
     throw err;
   }
 }
 async function requireDraft(client, id) {
-  const draft = await client.messages.getMessage.query(id);
+  const draft = await client.messages.getMessage.query(bareMessageId(id));
   if (!draft) {
     console.error(import_picocolors20.default.red(`No review item found for '${id}'. Run 'dosu review list' to see pending items.`));
     process.exit(1);
@@ -15781,8 +15796,7 @@ function reviewCommand() {
   cmd.command("diff").description("Show a pending review item (doc-change diff or draft reply body)").argument("<id>", "Review item ID (from `dosu review list`)").option("--json", "Output as JSON").action(async (id, opts) => {
     const cfg = requireConfig10();
     const client = createTypedClient(cfg);
-    const change = await resolveChange(client, id);
-    if (!change) {
+    if (isDraftId(id)) {
       const draft = await requireDraft(client, id);
       if (opts.json) {
         printResult({ id, kind: "draft_message", title: draft.title, body: draft.body }, opts);
@@ -15794,6 +15808,7 @@ function reviewCommand() {
       console.log(draft.body ?? "");
       return;
     }
+    const change = await requireChange(client, id);
     if (opts.json) {
       printResult(change, opts);
       return;
@@ -15824,14 +15839,16 @@ function reviewCommand() {
       console.error(import_picocolors20.default.red("Nothing to edit. Pass --title and/or --body/--body-file."));
       process.exit(1);
     }
-    const change = await resolveChange(client, id);
-    if (!change) {
+    if (isDraftId(id)) {
       if (opts.title !== undefined) {
         console.error(import_picocolors20.default.red("Draft replies support --body only (no --title)."));
         process.exit(1);
       }
       await requireDraft(client, id);
-      await client.messages.saveDraft.mutate({ messageId: id, body });
+      await client.messages.saveDraft.mutate({
+        messageId: bareMessageId(id),
+        body
+      });
     } else {
       try {
         await client.page.updateReview.mutate({
@@ -15851,7 +15868,7 @@ function reviewCommand() {
       printResult({ success: true, id }, opts);
       return;
     }
-    console.log(import_picocolors20.default.green(`Review edited: ${id.slice(0, 8)}`));
+    console.log(import_picocolors20.default.green(`Review edited: ${truncateId(id)}`));
   });
   cmd.command("context").description("Get review context for a thread").argument("<thread-id>", "Thread ID").option("--json", "Output as JSON").action(async (threadId, opts) => {
     const cfg = requireConfig10();
@@ -15882,7 +15899,7 @@ function reviewCommand() {
     cmd.command(name).description(`${verb} a review item (doc change or draft reply; shows a preview, requires --confirm)`).argument("<id>", "Review item ID (from `dosu review list`)").option("--confirm", "Apply without the interactive prompt").option("--json", "Output as JSON").action(async (id, opts) => {
       const cfg = requireConfig10();
       const client = createTypedClient(cfg);
-      const change = await resolveChange(client, id);
+      const change = isDraftId(id) ? null : await requireChange(client, id);
       const draft = change ? null : await requireDraft(client, id);
       const noun = change ? "change" : "draft reply";
       if (!opts.json) {
@@ -15920,26 +15937,26 @@ function reviewCommand() {
       if (change) {
         await client.page.updatePublicationStatus.mutate({ page_version_id: id, action });
       } else if (action === "accept") {
-        await client.messages.publishMessage.mutate({ postId: id });
+        await client.messages.publishMessage.mutate({ postId: bareMessageId(id) });
       } else {
-        await client.messages.deleteMessage.mutate(id);
+        await client.messages.deleteMessage.mutate(bareMessageId(id));
       }
       if (opts.json) {
         printResult({ success: true, id, action }, opts);
         return;
       }
-      console.log(import_picocolors20.default.green(`Review ${name}: ${id.slice(0, 8)}`));
+      console.log(import_picocolors20.default.green(`Review ${name}: ${truncateId(id)}`));
     });
   }
   cmd.command("revert").description("Revert a doc change to pending review (not supported for draft replies)").argument("<id>", "Review item ID (from `dosu review list`)").option("--json", "Output as JSON").action(async (id, opts) => {
     const cfg = requireConfig10();
     const client = createTypedClient(cfg);
-    const change = await resolveChange(client, id);
-    if (!change) {
+    if (isDraftId(id)) {
       await requireDraft(client, id);
       console.error(import_picocolors20.default.red("Revert is not supported for draft replies. A rejected draft is regenerated on the next agent run."));
       process.exit(1);
     }
+    await requireChange(client, id);
     await client.page.updatePublicationStatus.mutate({
       page_version_id: id,
       action: "revert_to_pending"
@@ -15948,7 +15965,7 @@ function reviewCommand() {
       printResult({ success: true, id, action: "revert_to_pending" }, opts);
       return;
     }
-    console.log(import_picocolors20.default.green(`Review revert: ${id.slice(0, 8)}`));
+    console.log(import_picocolors20.default.green(`Review revert: ${truncateId(id)}`));
   });
   return cmd;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dosu/cli",
-  "version": "0.28.0",
+  "version": "0.28.1",
   "type": "module",
   "description": "Dosu CLI - Manage MCP servers for AI tools",
   "license": "MIT",