shop-cli 0.1.2 → 0.1.3

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/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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "shop-cli",
-  "version": "0.1.2",
+  "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"
   }
-}
+}