shop-cli 0.1.1 → 0.1.3

package/dist/cli/approvalRequired.d.ts

@@ -0,0 +1,30 @@
+import type { OutputFormat } from './output';
+type ApprovalRequiredAction = Record<string, unknown> & {
+    type?: unknown;
+    status?: unknown;
+    id?: unknown;
+    executed?: unknown;
+    contentType?: unknown;
+    body?: unknown;
+};
+export type ApprovalRequiredInfo = {
+    code: 'APPROVAL_REQUIRED' | 'ACTION_REQUIRED';
+    action: ApprovalRequiredAction;
+};
+/**
+ * Generic GraphQL proxy contract for deferred execution.
+ *
+ * When a proxy requires human approval before executing an operation, it may return a
+ * standard GraphQL response with an `errors[]` entry that includes:
+ * - `error.extensions.code === "APPROVAL_REQUIRED"` (also accepts "ACTION_REQUIRED")
+ * - `error.extensions.action` describing the approval workflow
+ *
+ * In this case, the operation has NOT executed yet. shop-cli surfaces a purpose-built
+ * error payload and exits with code 3.
+ */
+export declare const findApprovalRequired: (errors: unknown[] | undefined | null) => ApprovalRequiredInfo | undefined;
+export declare const maybeThrowApprovalRequired: ({ format, errors, }: {
+    format: OutputFormat;
+    errors: unknown[] | undefined | null;
+}) => void;
+export {};

package/dist/cli/approvalRequired.js

@@ -0,0 +1,87 @@
+"use strict";
+var __defProp = Object.defineProperty;
+var __getOwnPropDesc = Object.getOwnPropertyDescriptor;
+var __getOwnPropNames = Object.getOwnPropertyNames;
+var __hasOwnProp = Object.prototype.hasOwnProperty;
+var __export = (target, all) => {
+  for (var name in all)
+    __defProp(target, name, { get: all[name], enumerable: true });
+};
+var __copyProps = (to, from, except, desc) => {
+  if (from && typeof from === "object" || typeof from === "function") {
+    for (let key of __getOwnPropNames(from))
+      if (!__hasOwnProp.call(to, key) && key !== except)
+        __defProp(to, key, { get: () => from[key], enumerable: !(desc = __getOwnPropDesc(from, key)) || desc.enumerable });
+  }
+  return to;
+};
+var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
+var approvalRequired_exports = {};
+__export(approvalRequired_exports, {
+  findApprovalRequired: () => findApprovalRequired,
+  maybeThrowApprovalRequired: () => maybeThrowApprovalRequired
+});
+module.exports = __toCommonJS(approvalRequired_exports);
+var import_output = require("./output");
+var import_errors = require("./errors");
+const isPlainObject = (value) => typeof value === "object" && value !== null && !Array.isArray(value);
+const findApprovalRequired = (errors) => {
+  if (!Array.isArray(errors)) return void 0;
+  for (const err of errors) {
+    if (!isPlainObject(err)) continue;
+    const extensions = err.extensions;
+    if (!isPlainObject(extensions)) continue;
+    const code = extensions.code;
+    if (code !== "APPROVAL_REQUIRED" && code !== "ACTION_REQUIRED") continue;
+    const action = extensions.action;
+    if (!isPlainObject(action)) continue;
+    if (action.type !== "approval") continue;
+    if (action.status !== "pending") continue;
+    if (action.executed !== false) continue;
+    if (typeof action.id !== "string" || !action.id.trim()) continue;
+    if (typeof action.body !== "string" || !action.body.trim()) continue;
+    return { code, action };
+  }
+  return void 0;
+};
+const writeHumanApprovalMessage = ({ action }) => {
+  const id = String(action.id);
+  const body = typeof action.body === "string" ? action.body.trim() : "";
+  const lines = [
+    "Approval required: this operation was NOT executed yet.",
+    `Approval id: ${id}`,
+    "Execution will happen after approval is granted (out-of-band)."
+  ];
+  if (body) {
+    lines.push("", body);
+  }
+  process.stderr.write(lines.join("\n").trimEnd() + "\n");
+};
+const maybeThrowApprovalRequired = ({
+  format,
+  errors
+}) => {
+  const approval = findApprovalRequired(errors);
+  if (!approval) return;
+  if (format === "json" || format === "jsonl") {
+    (0, import_output.printJsonError)(
+      {
+        ok: false,
+        error: {
+          code: approval.code,
+          action: approval.action
+        }
+      },
+      format === "json"
+    );
+  } else {
+    writeHumanApprovalMessage(approval);
+  }
+  throw new import_errors.CliError("", 3, { silent: true });
+};
+// Annotate the CommonJS export names for ESM import in node:
+0 && (module.exports = {
+  findApprovalRequired,
+  maybeThrowApprovalRequired
+});
+//# sourceMappingURL=approvalRequired.js.map

package/dist/cli/errors.d.ts

@@ -1,4 +1,7 @@
 export declare class CliError extends Error {
     exitCode: number;
-    constructor(message: string, exitCode?: number);
+    silent: boolean;
+    constructor(message: string, exitCode?: number, { silent }?: {
+        silent?: boolean;
+    });
 }

package/dist/cli/errors.js

@@ -23,9 +23,11 @@ __export(errors_exports, {
 module.exports = __toCommonJS(errors_exports);
 class CliError extends Error {
   exitCode;
-  constructor(message, exitCode = 1) {
+  silent;
+  constructor(message, exitCode = 1, { silent = false } = {}) {
     super(message);
     this.exitCode = exitCode;
+    this.silent = silent;
   }
 }
 // Annotate the CommonJS export names for ESM import in node:

package/dist/cli/router.js

@@ -29,6 +29,7 @@ var import_admin_2026_04 = require("../generated/admin-2026-04");
 var import_admin_2026_042 = require("../generated/admin-2026-04");
 var import_errors = require("./errors");
 var import_output = require("./output");
+var import_approvalRequired = require("./approvalRequired");
 var import_introspection = require("./introspection");
 var import_format = require("./introspection/format");
 var import_resources = require("./introspection/resources");
@@ -442,6 +443,7 @@ const runQuery = async (ctx, request) => {
     return await ctx.client.query(request);
   } catch (err) {
     if (err instanceof import_admin_2026_042.GenqlError) {
+      (0, import_approvalRequired.maybeThrowApprovalRequired)({ format: ctx.format, errors: err.errors });
       if (ctx.warnMissingAccessToken && hasNotAuthorizedError(err)) {
         console.error("SHOPIFY_ACCESS_TOKEN not set");
       }
@@ -461,6 +463,7 @@ const runMutation = async (ctx, request) => {
     return await ctx.client.mutation(request);
   } catch (err) {
     if (err instanceof import_admin_2026_042.GenqlError) {
+      (0, import_approvalRequired.maybeThrowApprovalRequired)({ format: ctx.format, errors: err.errors });
       if (ctx.warnMissingAccessToken && hasNotAuthorizedError(err)) {
         console.error("SHOPIFY_ACCESS_TOKEN not set");
       }

package/dist/cli/verbs/collections.js

@@ -447,7 +447,7 @@ const runCollections = async ({
         const pos = Number(item.slice(idx + 1).trim());
         if (!productId) throw new import_errors.CliError("--move productId cannot be empty", 2);
         if (!Number.isFinite(pos) || pos < 0) throw new import_errors.CliError("--move newPosition must be a non-negative number", 2);
-        parsedMoves.push({ id: (0, import_gid.coerceGid)(productId, "Product"), newPosition: Math.floor(pos) });
+        parsedMoves.push({ id: (0, import_gid.coerceGid)(productId, "Product"), newPosition: String(Math.floor(pos)) });
       }
       moves = parsedMoves;
     }
@@ -460,7 +460,7 @@ const runCollections = async ({
       if (typeof mid !== "string" || !mid.trim()) throw new import_errors.CliError(`moves[${i}].id is required`, 2);
       const pos = Number(newPosition);
       if (!Number.isFinite(pos) || pos < 0) throw new import_errors.CliError(`moves[${i}].newPosition must be a non-negative number`, 2);
-      return { id: mid.startsWith("gid://") ? mid : (0, import_gid.coerceGid)(mid, "Product"), newPosition: Math.floor(pos) };
+      return { id: mid.startsWith("gid://") ? mid : (0, import_gid.coerceGid)(mid, "Product"), newPosition: String(Math.floor(pos)) };
     });
     const result = await (0, import_router.runMutation)(ctx, {
       collectionReorderProducts: {

package/dist/cli/verbs/graphql.js

@@ -26,6 +26,7 @@ var import_node_fs = require("node:fs");
 var import_errors = require("../errors");
 var import_output = require("../output");
 var import_router = require("../router");
+var import_approvalRequired = require("../approvalRequired");
 var import_adminClient = require("../../adminClient");
 var import_graphqlValidator = require("../../graphqlValidator");
 const printHelp = () => {
@@ -272,6 +273,7 @@ const runGraphQL = async ({
     throw new import_errors.CliError(`GraphQL request failed: ${err.message}`, 1);
   }
   if (response.errors && response.errors.length > 0) {
+    (0, import_approvalRequired.maybeThrowApprovalRequired)({ format: ctx.format, errors: response.errors });
     (0, import_output.printJsonError)({ errors: response.errors, data: response.data }, ctx.format !== "raw");
     throw new import_errors.CliError("GraphQL request returned errors", 1);
   }

package/dist/cli/verbs/products.js

@@ -2262,13 +2262,13 @@ Missing <verb> for "products ${verb}"`, 2);
       const raw = args.move;
       const parsedMoves = [];
       for (const item of raw) {
-        const parts = item.split(":");
-        if (parts.length !== 2) throw new import_errors.CliError("--move must be <mediaId>:<newPosition>", 2);
-        const mediaId = parts[0].trim();
-        const pos = Number(parts[1].trim());
+        const idx = item.lastIndexOf(":");
+        if (idx <= 0 || idx === item.length - 1) throw new import_errors.CliError("--move must be <mediaId>:<newPosition>", 2);
+        const mediaId = item.slice(0, idx).trim();
+        const pos = Number(item.slice(idx + 1).trim());
         if (!mediaId) throw new import_errors.CliError("--move mediaId cannot be empty", 2);
         if (!Number.isFinite(pos) || pos < 0) throw new import_errors.CliError("--move newPosition must be a non-negative number", 2);
-        parsedMoves.push({ id: normalizeMediaId(mediaId), newPosition: Math.floor(pos) });
+        parsedMoves.push({ id: normalizeMediaId(mediaId), newPosition: String(Math.floor(pos)) });
       }
       moves = parsedMoves;
     }
@@ -2285,7 +2285,7 @@ Missing <verb> for "products ${verb}"`, 2);
       if (!Number.isFinite(pos) || pos < 0) {
         throw new import_errors.CliError(`moves[${i}].newPosition must be a non-negative number`, 2);
       }
-      return { id: normalizeMediaId(id2), newPosition: Math.floor(pos) };
+      return { id: normalizeMediaId(id2), newPosition: String(Math.floor(pos)) };
     });
     const result = await (0, import_router.runMutation)(ctx, {
       productReorderMedia: {

package/dist/cli/verbs/reverse-deliveries.js

@@ -60,7 +60,12 @@ const getReverseDeliverySelection = (view) => {
   return reverseDeliverySummarySelection;
 };
 const parseLineItem = (value) => {
-  const [id, qtyRaw] = value.split(":");
+  const idx = value.lastIndexOf(":");
+  if (idx <= 0 || idx === value.length - 1) {
+    throw new import_errors.CliError("--line-item must be <reverseFulfillmentOrderLineItemId>:<quantity>", 2);
+  }
+  const id = value.slice(0, idx).trim();
+  const qtyRaw = value.slice(idx + 1).trim();
   if (!id || !qtyRaw) throw new import_errors.CliError("--line-item must be <reverseFulfillmentOrderLineItemId>:<quantity>", 2);
   const quantity = Number(qtyRaw);
   if (!Number.isFinite(quantity) || !Number.isInteger(quantity) || quantity <= 0) {

package/dist/cli/verbs/reverse-fulfillment-orders.js

@@ -61,24 +61,49 @@ const getReverseFulfillmentOrderSelection = (view) => {
   return reverseFulfillmentOrderSummarySelection;
 };
 const parseDisposition = (value) => {
-  const parts = value.split(":");
-  if (parts.length < 3) {
-    throw new import_errors.CliError("--disposition must be <reverseFulfillmentOrderLineItemId>:<quantity>:<dispositionType>[:<locationId>]", 2);
+  const raw = value.trim();
+  const usage = () => new import_errors.CliError("--disposition must be <reverseFulfillmentOrderLineItemId>:<quantity>:<dispositionType>[:<locationId>]", 2);
+  const splitOnLastColonOrThrow = (s) => {
+    const idx = s.lastIndexOf(":");
+    if (idx <= 0 || idx === s.length - 1) throw usage();
+    return [s.slice(0, idx), s.slice(idx + 1)];
+  };
+  const parseCore = (core) => {
+    const [rest, dispositionTypeRaw] = splitOnLastColonOrThrow(core);
+    const [idRaw, quantityRaw] = splitOnLastColonOrThrow(rest);
+    const reverseFulfillmentOrderLineItemId = idRaw.trim();
+    const dispositionType = dispositionTypeRaw.trim();
+    const quantity = Number(quantityRaw.trim());
+    if (!reverseFulfillmentOrderLineItemId || !dispositionType) throw usage();
+    if (!Number.isFinite(quantity) || !Number.isInteger(quantity) || quantity <= 0) {
+      throw new import_errors.CliError("--disposition quantity must be a positive integer", 2);
+    }
+    return { reverseFulfillmentOrderLineItemId, quantity, dispositionType };
+  };
+  let firstErr;
+  try {
+    return parseCore(raw);
+  } catch (err) {
+    firstErr = err;
   }
-  const reverseFulfillmentOrderLineItemId = parts[0];
-  const quantityRaw = parts[1];
-  const dispositionType = parts[2];
-  const locationId = parts[3];
-  const quantity = Number(quantityRaw);
-  if (!Number.isFinite(quantity) || !Number.isInteger(quantity) || quantity <= 0) {
-    throw new import_errors.CliError("--disposition quantity must be a positive integer", 2);
+  const gidIdx = raw.lastIndexOf("gid://");
+  if (gidIdx > 0) {
+    const before = raw.slice(0, gidIdx);
+    const colonIdx = before.lastIndexOf(":");
+    if (colonIdx >= 0 && before.slice(colonIdx + 1).trim() === "") {
+      const core = raw.slice(0, colonIdx).trim();
+      const locationId = raw.slice(gidIdx).trim();
+      if (locationId) return { ...parseCore(core), locationId };
+    }
+  }
+  try {
+    const [core, locationIdRaw] = splitOnLastColonOrThrow(raw);
+    const locationId = locationIdRaw.trim();
+    if (!locationId) throw usage();
+    return { ...parseCore(core.trim()), locationId };
+  } catch {
+    throw firstErr;
   }
-  return {
-    reverseFulfillmentOrderLineItemId,
-    quantity,
-    dispositionType,
-    ...locationId ? { locationId } : {}
-  };
 };
 const runReverseFulfillmentOrders = async ({
   ctx,

package/dist/cli.js

@@ -224,12 +224,12 @@ Missing <verb> for "${resource}"`, 2);
 See help:
   ${command} ${resource} ${verb} --help`;
     }
-    throw new import_errors.CliError(message, err.exitCode);
+    throw new import_errors.CliError(message, err.exitCode, { silent: err.silent });
   }
 };
 main().catch((err) => {
   if (err instanceof import_errors.CliError) {
-    console.error(err.message);
+    if (!err.silent) console.error(err.message);
     process.exit(err.exitCode);
   }
   console.error(err);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "shop-cli",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "description": "An agent-friendly command-line interface for managing Shopify stores",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -63,4 +63,4 @@
     "typescript": "^5.9.3",
     "vitest": "^4.0.18"
   }
-}
+}