@doist/automations-cli 1.21.3 → 1.22.1

package/dist/index.js

@@ -7,7 +7,7 @@ import {
   collectSkillStatus,
   getInstaller,
   updateAllInstalledSkills
-} from "./chunk-ZWFZ2DD3.js";
+} from "./chunk-NB7HIIJZ.js";
 
 // ../../node_modules/ajv/dist/compile/codegen/code.js
 var require_code = __commonJS({
@@ -6899,7 +6899,7 @@ import { fileURLToPath } from "node:url";
 // package.json
 var package_default = {
   name: "@doist/automations-cli",
-  version: "1.21.3",
+  version: "1.22.1",
   description: "Command-line client for Todoist Automations",
   type: "module",
   bin: {
@@ -6945,7 +6945,7 @@ var package_default = {
   dependencies: {
     "@doist/cli-core": "1.7.0",
     "@napi-rs/keyring": "2.1.1",
-    ai: "7.0.141",
+    ai: "7.0.142",
     commander: "15.0.0",
     oauth4webapi: "3.8.8",
     open: "11.0.4"
@@ -6959,7 +6959,7 @@ var package_default = {
     "semantic-release": "25.0.9",
     "semantic-release-monorepo": "8.0.2",
     typescript: "6.0.3",
-    vitest: "5.0.4"
+    vitest: "5.0.5"
   },
   engines: {
     node: ">=24"
@@ -33105,15 +33105,27 @@ function diffManifests(before, after) {
   const added = [...nextSteps.keys()].filter((id) => !previousSteps.has(id));
   const removed = [...previousSteps.keys()].filter((id) => !nextSteps.has(id));
   const changed = [...nextSteps].filter(([id, step]) => previousSteps.has(id) && !isDeepEqual(previousSteps.get(id), step)).map(([id]) => id);
+  const changedFields = Object.fromEntries(changed.map((id) => [id, differingPaths(previousSteps.get(id), nextSteps.get(id), "")]));
   const sharedBefore = [...previousSteps.keys()].filter((id) => nextSteps.has(id));
   const sharedAfter = [...nextSteps.keys()].filter((id) => previousSteps.has(id));
   const reordered = sharedBefore.some((id, index) => sharedAfter[index] !== id);
   return {
     changedProperties,
-    steps: { added, removed, changed, reordered },
+    steps: { added, removed, changed, changedFields, reordered },
     isEmpty: changedProperties.length === 0 && added.length === 0 && removed.length === 0 && changed.length === 0 && !reordered
   };
 }
+function differingPaths(before, after, path) {
+  if (isDeepEqual(before, after))
+    return [];
+  if (isManifestRecord(before) && isManifestRecord(after)) {
+    return [.../* @__PURE__ */ new Set([...Object.keys(before), ...Object.keys(after)])].sort().flatMap((key) => differingPaths(before[key], after[key], path ? `${path}.${key}` : key));
+  }
+  if (path && Array.isArray(before) && Array.isArray(after) && before.length === after.length) {
+    return before.flatMap((item, index) => differingPaths(item, after[index], `${path}[${index}]`));
+  }
+  return path ? [path] : [];
+}
 function asManifest(value) {
   return isManifestRecord(value) ? value : {};
 }
@@ -33137,7 +33149,7 @@ function sortKeys(value) {
 
 // ../../packages/sdk/dist/esm/drafts/publish-draft.js
 async function publishDraft(draftApi, { draftId, confirmed }) {
-  const { summary, manifest } = await summarizeForPublish(draftApi, draftId);
+  const { summary, detail } = await previewPublish(draftApi, draftId);
   if (!confirmed) {
     return { action: "publish_preview", summary };
   }
@@ -33164,7 +33176,7 @@ async function publishDraft(draftApi, { draftId, confirmed }) {
       verification: { status: "unavailable", reason: getErrorMessage(error62) }
     };
   }
-  const expected = normaliseManifest(manifest);
+  const expected = normaliseManifest(detail.draft.partialManifest);
   const actual = normaliseManifest(automation.manifest);
   return {
     action: "published",
@@ -33173,7 +33185,7 @@ async function publishDraft(draftApi, { draftId, confirmed }) {
     verification: isDeepEqual(expected, actual) ? { status: "matched" } : { status: "mismatched", mismatch: diffManifests(expected, actual) }
   };
 }
-async function summarizeForPublish(draftApi, draftId) {
+async function previewPublish(draftApi, draftId) {
   const [detail, connections] = await Promise.all([
     draftApi.get(draftId),
     draftApi.getConnectionStatus(draftId)
@@ -33207,7 +33219,7 @@ async function summarizeForPublish(draftApi, draftId) {
     schedule: describeSchedule(draft.partialManifest),
     diff
   };
-  return { summary, manifest: draft.partialManifest };
+  return { summary, detail };
 }
 function describeSchedule(manifest) {
   if (!hasTriggerType(manifest))
@@ -34513,6 +34525,9 @@ function parsePositiveInteger(value) {
   return parsed;
 }
 
+// src/lib/api-command.ts
+import { CliError as CliError3 } from "@doist/cli-core";
+
 // src/lib/command-error.ts
 import { formatJson as formatJson4, formatNdjson as formatNdjson4, getErrorMessage as getErrorMessage6 } from "@doist/cli-core";
 
@@ -34646,10 +34661,11 @@ async function runApiCommand(command, operation, presentation, execution = {}) {
     if (exitCode !== void 0) process.exitCode = exitCode;
   } catch (error62) {
     if (abortController.signal.aborted) {
-      writeFailure(mode, "interrupted", {
-        message: "Interrupted.",
-        humanMessage: "Interrupted."
-      });
+      writeFailure(
+        mode,
+        "interrupted",
+        error62 instanceof CliError3 ? renderFailure(error62, { mode, source }) : { message: "Interrupted.", humanMessage: "Interrupted." }
+      );
       process.exitCode = 130;
       return;
     }
@@ -37251,7 +37267,7 @@ function registerConnectorCommand(program) {
 }
 
 // src/lib/manifest-input.ts
-import { CliError as CliError3, getErrorMessage as getErrorMessage7 } from "@doist/cli-core";
+import { CliError as CliError4, getErrorMessage as getErrorMessage7 } from "@doist/cli-core";
 import { Option as Option6 } from "commander";
 import { readFile as readFile2 } from "node:fs/promises";
 import { text } from "node:stream/consumers";
@@ -37265,7 +37281,7 @@ async function readManifestInput(options, input2 = process.stdin) {
   try {
     return JSON.parse(source.content);
   } catch (error62) {
-    throw new CliError3(
+    throw new CliError4(
       "MANIFEST_UNREADABLE",
       `${source.label} is not valid JSON: ${getErrorMessage7(error62)}`
     );
@@ -37276,7 +37292,7 @@ async function readSource(options, input2) {
     try {
       return { label: options.file, content: await readFile2(options.file, "utf8") };
     } catch (error62) {
-      throw new CliError3(
+      throw new CliError4(
         "MANIFEST_UNREADABLE",
         `Could not read ${options.file}: ${getErrorMessage7(error62)}`
       );
@@ -37286,17 +37302,17 @@ async function readSource(options, input2) {
     try {
       return { label: "Standard input", content: await text(input2) };
     } catch (error62) {
-      throw new CliError3(
+      throw new CliError4(
         "MANIFEST_UNREADABLE",
         `Could not read standard input: ${getErrorMessage7(error62)}`
       );
     }
   }
-  throw new CliError3("MANIFEST_UNREADABLE", "Provide the manifest with --file <path> or --stdin.");
+  throw new CliError4("MANIFEST_UNREADABLE", "Provide the manifest with --file <path> or --stdin.");
 }
 
 // src/commands/draft/draft-service.ts
-import { CliError as CliError4, getErrorMessage as getErrorMessage8 } from "@doist/cli-core";
+import { CliError as CliError5, getErrorMessage as getErrorMessage8 } from "@doist/cli-core";
 import { readFile as readFile3 } from "node:fs/promises";
 function createDraft2(draftApi, input2) {
   return withRecoveryHints(draftApi, createDraft(draftApi, input2));
@@ -37329,13 +37345,13 @@ async function withRecoveryHints(draftApi, operation) {
     const draftId = error62.draftId ?? "<draft-id>";
     switch (error62.code) {
       case "manifest_invalid":
-        throw new CliError4("MANIFEST_INVALID", error62.message);
+        throw new CliError5("MANIFEST_INVALID", error62.message);
       case "draft_conflict":
-        throw new CliError4("DRAFT_CONFLICT", error62.message, {
+        throw new CliError5("DRAFT_CONFLICT", error62.message, {
           hints: draftHints(scope, draftId)
         });
       case "draft_manifest_rejected":
-        throw new CliError4("DRAFT_MANIFEST_REJECTED", error62.message, {
+        throw new CliError5("DRAFT_MANIFEST_REJECTED", error62.message, {
           hints: draftHints(scope, draftId)
         });
       case "draft_not_ready":
@@ -37380,7 +37396,12 @@ async function startDraftTest2(draftApi, {
   if (!waitFor) {
     return { action: "test_started", ...identity, status: started.status };
   }
-  const finished = await waitForDraftTest(draftApi, started, { ...waitFor, signal });
+  let finished;
+  try {
+    finished = await waitForDraftTest(draftApi, started, { ...waitFor, signal });
+  } catch (error62) {
+    throw toDraftTestWaitError(error62, identity, signal);
+  }
   return {
     action: "test_finished",
     ...identity,
@@ -37391,9 +37412,25 @@ async function startDraftTest2(draftApi, {
     result: finished.result
   };
 }
+function toDraftTestWaitError(error62, identity, signal) {
+  const run = `test run ${identity.testRunId} of draft ${identity.draftId}`;
+  const hints = [`See how it ends: ${identity.command}`];
+  if (signal.aborted) {
+    return new CliError5(
+      "DRAFT_TEST_WAIT_INTERRUPTED",
+      `Stopped waiting for ${run}. It keeps running.`,
+      { hints }
+    );
+  }
+  return new CliError5(
+    "DRAFT_TEST_WAIT_FAILED",
+    `Checking on ${run} failed: ${getErrorMessage8(error62).replace(/\.?$/, ".")} It keeps running.`,
+    { hints }
+  );
+}
 function draftNotReadyError(message, draftId, scope) {
   const flag = scopeFlag(scope);
-  return new CliError4("DRAFT_NOT_READY", message, {
+  return new CliError5("DRAFT_NOT_READY", message, {
     hints: [
       `See the issues: tda draft view ${draftId} ${flag}`,
       `Bind connections: tda draft connect ${draftId} --auto ${flag}`
@@ -37407,13 +37444,13 @@ function toDraftTestCliError(refusal, draftId, scope) {
       return draftNotReadyError(refusal.message, draftId, scope);
     case "draft_test_payload_required":
     case "draft_test_payload_filtered":
-      return new CliError4(refusal.code.toUpperCase(), refusal.message, {
+      return new CliError5(refusal.code.toUpperCase(), refusal.message, {
         hints: [
           `Test with a real event: tda draft test ${draftId} --trigger <event> --payload-file <path> ${flag}`
         ]
       });
     default:
-      return new CliError4(refusal.code.toUpperCase(), refusal.message);
+      return new CliError5(refusal.code.toUpperCase(), refusal.message);
   }
 }
 async function readTestPayloadFile(path) {
@@ -37421,7 +37458,7 @@ async function readTestPayloadFile(path) {
   try {
     content = await readFile3(path, "utf8");
   } catch (error62) {
-    throw new CliError4(
+    throw new CliError5(
       "PAYLOAD_UNREADABLE",
       `Could not read ${path}: ${getErrorMessage8(error62)}`
     );
@@ -37429,7 +37466,7 @@ async function readTestPayloadFile(path) {
   try {
     return JSON.parse(content);
   } catch (error62) {
-    throw new CliError4(
+    throw new CliError5(
       "PAYLOAD_UNREADABLE",
       `${path} is not valid JSON: ${getErrorMessage8(error62)}`
     );
@@ -37540,7 +37577,13 @@ function describeDiff(diff) {
   }
   if (diff.steps.added.length > 0) parts.push(`steps added ${diff.steps.added.join(", ")}`);
   if (diff.steps.removed.length > 0) parts.push(`steps removed ${diff.steps.removed.join(", ")}`);
-  if (diff.steps.changed.length > 0) parts.push(`steps changed ${diff.steps.changed.join(", ")}`);
+  if (diff.steps.changed.length > 0) {
+    const changed = diff.steps.changed.map((id) => {
+      const fields = diff.steps.changedFields[id] ?? [];
+      return fields.length > 0 ? `${id} (${fields.join(", ")})` : id;
+    });
+    parts.push(`steps changed ${changed.join(", ")}`);
+  }
   if (diff.steps.reordered) parts.push("steps reordered");
   return parts.join("; ");
 }
@@ -37612,7 +37655,7 @@ function formatDraftTest(result) {
 }
 
 // src/commands/draft/validate-command.ts
-import { CliError as CliError5 } from "@doist/cli-core";
+import { CliError as CliError6 } from "@doist/cli-core";
 
 // src/lib/local-command.ts
 async function runLocalCommand(command, operation, presentation, execution = {}) {
@@ -37644,7 +37687,7 @@ function registerDraftValidateCommand(draft) {
 function validateManifestInput(manifest) {
   const validation = validateManifest(manifest);
   if (!validation.isValid) {
-    throw new CliError5(
+    throw new CliError6(
       "MANIFEST_INVALID",
       `Manifest is invalid: ${validation.errors.join("; ")}`
     );
@@ -37993,11 +38036,11 @@ function registerRunCommand(program) {
 }
 
 // src/commands/skill/shared.ts
-import { CliError as CliError6 } from "@doist/cli-core";
+import { CliError as CliError7 } from "@doist/cli-core";
 function requireInstaller(agent) {
   const installer = getInstaller(agent);
   if (!installer) {
-    throw new CliError6("UNKNOWN_AGENT", `Unknown agent: ${agent}.`, {
+    throw new CliError7("UNKNOWN_AGENT", `Unknown agent: ${agent}.`, {
       hints: [`Available agents: ${AGENT_NAMES.join(", ")}.`]
     });
   }
@@ -38075,7 +38118,7 @@ async function uninstallSkill(agent, options, command) {
 }
 
 // src/commands/skill/update.ts
-import { CliError as CliError7 } from "@doist/cli-core";
+import { CliError as CliError8 } from "@doist/cli-core";
 async function updateSkill(agent, options, command) {
   await runLocalCommand(
     command,
@@ -38101,7 +38144,7 @@ async function updateAllSkills(options, command) {
     async () => {
       const result = await updateAllInstalledSkills(options.local === true);
       if (result.errors.length > 0) {
-        throw new CliError7(
+        throw new CliError8(
           "UPDATE_FAILED",
           `Could not update ${result.errors.length} skill file(s).`,
           { hints: result.errors.map((error62) => `${error62.agent}: ${error62.message}`) }