shop-cli 0.1.3 → 0.1.4

package/dist/cli/help/render.js

@@ -216,6 +216,10 @@ const renderTopLevelHelp = (command = (0, import_command.resolveCliCommand)()) =
     "  --selection <graphql>     (selection override; can be @file.gql)",
     "  --quiet                  (IDs only when possible)",
     "",
+    "IDs:",
+    "  By default, many flags accept numeric IDs and will coerce them to gid://shopify/... IDs.",
+    "  --strict-ids             (or env SHOP_CLI_STRICT_IDS=1; require full gid://shopify/... IDs)",
+    "",
     "Debug:",
     "  --dry-run                (print GraphQL op + variables, do not execute)",
     "  --no-fail-on-user-errors (do not exit non-zero on userErrors)",
@@ -234,10 +238,10 @@ const renderTopLevelHelp = (command = (0, import_command.resolveCliCommand)()) =
     `  ${command} products list --first 5 --format table`,
     `  ${command} products get --id gid://shopify/Product/123 --format markdown`,
     `  ${command} products create --set title="Hat" --set status="ACTIVE"`,
-    `  ${command} products add-tags --id 123 --tags "summer,featured"`,
+    `  ${command} products add-tags --id gid://shopify/Product/123 --tags "summer,featured"`,
     `  ${command} publications resolve --publication "Online Store"`,
-    `  ${command} products publish --id 123 --publication "Online Store" --now`,
-    `  ${command} products metafields upsert --id 123 --set namespace=custom --set key=foo --set type=single_line_text_field --set value=bar`
+    `  ${command} products publish --id gid://shopify/Product/123 --publication "Online Store" --now`,
+    `  ${command} products metafields upsert --id gid://shopify/Product/123 --set namespace=custom --set key=foo --set type=single_line_text_field --set value=bar`
   ].join("\n");
 };
 const renderResourceHelp = (resource, command = (0, import_command.resolveCliCommand)()) => {

package/dist/cli/verbs/_shared.js

@@ -55,11 +55,11 @@ const parseIntFlag = (flag, value) => {
 const requireId = (id, type, flag = "--id") => {
   const normalized = typeof id === "string" ? id : typeof id === "number" && Number.isFinite(id) ? String(id) : void 0;
   if (!normalized) throw new import_errors.CliError(`Missing ${flag}`, 2);
-  return (0, import_gid.coerceGid)(normalized, type);
+  return (0, import_gid.coerceGid)(normalized, type, flag);
 };
 const requireGidFlag = (value, flag, type) => {
   if (typeof value !== "string" || !value) throw new import_errors.CliError(`Missing ${flag}`, 2);
-  return (0, import_gid.coerceGid)(value, type);
+  return (0, import_gid.coerceGid)(value, type, flag);
 };
 const requireStringFlag = (value, flag) => {
   if (typeof value !== "string" || !value) throw new import_errors.CliError(`Missing ${flag}`, 2);
@@ -67,7 +67,7 @@ const requireStringFlag = (value, flag) => {
 };
 const requireLocationId = (value, flag = "--location-id") => {
   if (typeof value !== "string" || !value) throw new import_errors.CliError(`Missing ${flag}`, 2);
-  return (0, import_gid.coerceGid)(value, "Location");
+  return (0, import_gid.coerceGid)(value, "Location", flag);
 };
 const parseDateTime = (value, flag) => {
   if (typeof value !== "string" || !value) throw new import_errors.CliError(`Missing ${flag}`, 2);
@@ -87,7 +87,7 @@ const parseIds = (value, type) => {
     parts.push(...v.split(",").map((s) => s.trim()).filter(Boolean));
   }
   if (parts.length === 0) throw new import_errors.CliError("Missing --ids", 2);
-  return parts.map((id) => (0, import_gid.coerceGid)(id, type));
+  return parts.map((id) => (0, import_gid.coerceGid)(id, type, "--ids"));
 };
 const readUtf8 = (path) => (0, import_node_fs.readFileSync)(path, "utf8");
 const parseJson = (value, label) => {

package/dist/cli/verbs/files.js

@@ -22,6 +22,7 @@ __export(files_exports, {
 });
 module.exports = __toCommonJS(files_exports);
 var import_errors = require("../errors");
+var import_gid = require("../gid");
 var import_router = require("../router");
 var import_userErrors = require("../userErrors");
 var import_stagedUploads = require("../workflows/files/stagedUploads");
@@ -123,8 +124,16 @@ const runFiles = async ({
     if (!raw) throw new import_errors.CliError("Missing --id", 2);
     const trimmed = raw.trim();
     if (!trimmed) throw new import_errors.CliError("Missing --id", 2);
-    const query = trimmed.startsWith("gid://") ? `ids:${trimmed}` : /^\d+$/.test(trimmed) ? `id:${trimmed}` : (() => {
-      throw new import_errors.CliError("--id must be a numeric ID or full gid://shopify/... GID", 2);
+    const query = trimmed.startsWith("gid://") ? `ids:${trimmed}` : /^\d+$/.test(trimmed) ? (() => {
+      if ((0, import_gid.isStrictIdsMode)()) {
+        throw new import_errors.CliError(
+          "--id must be a full Shopify GID (gid://shopify/...). Strict IDs mode is enabled (SHOP_CLI_STRICT_IDS or --strict-ids).",
+          2
+        );
+      }
+      return `id:${trimmed}`;
+    })() : (() => {
+      throw new import_errors.CliError("--id must be a full Shopify GID (gid://shopify/...)", 2);
     })();
     const result = await (0, import_router.runQuery)(ctx, {
       files: { __args: { first: 1, query }, nodes: fileSelection }

package/dist/cli/verbs/mobile-platform-applications.js

@@ -67,9 +67,11 @@ const getMobilePlatformApplicationSelection = (view) => {
 const extractUnionId = (node) => node?.on_AndroidApplication?.id ?? node?.on_AppleApplication?.id;
 const normalizeMobileAppId = (raw, platform) => {
   if (typeof raw !== "string" || !raw) throw new import_errors.CliError("Missing --id", 2);
-  if (raw.startsWith("gid://")) return raw;
-  const p = (platform ?? "android").trim().toLowerCase();
+  const p = platform?.trim().toLowerCase();
   const type = p === "apple" || p === "ios" ? "AppleApplication" : "AndroidApplication";
+  if (raw.startsWith("gid://")) {
+    return p ? (0, import_gid.coerceGid)(raw, type, "--id") : raw;
+  }
   return (0, import_gid.coerceGid)(raw, type);
 };
 const runMobilePlatformApplications = async ({

package/dist/cli/verbs/products.js

@@ -228,10 +228,11 @@ const getTopProductMediaIds = async ({
   const nodes = result.product?.media?.nodes ?? [];
   return nodes.map((n) => typeof n?.id === "string" ? n.id : void 0).filter((id) => typeof id === "string" && id.trim() !== "");
 };
-const normalizeMediaId = (value) => {
+const allowedMediaGidTypes = ["MediaImage", "Video", "ExternalVideo", "Model3d"];
+const normalizeMediaId = (value, label = "Media ID") => {
   const raw = value.trim();
-  if (!raw) throw new import_errors.CliError("Media ID cannot be empty", 2);
-  if (raw.startsWith("gid://")) return raw;
+  if (!raw) throw new import_errors.CliError(`${label} cannot be empty`, 2);
+  if (raw.startsWith("gid://")) return (0, import_gid.assertShopifyGidTypeIn)(raw, allowedMediaGidTypes, label);
   throw new import_errors.CliError(
     `Numeric media ID "${raw}" is ambiguous. Use the full GID from "shop products media list" (e.g. gid://shopify/MediaImage/${raw})`,
     2
@@ -2190,7 +2191,7 @@ Missing <verb> for "products ${verb}"`, 2);
     const mediaIds = args["media-id"] ?? [];
     if (mediaIds.length === 0) throw new import_errors.CliError("Missing --media-id (repeatable)", 2);
     const files = mediaIds.map((id) => ({
-      id: normalizeMediaId(id),
+      id: normalizeMediaId(id, "--media-id"),
       referencesToRemove: [productId]
     }));
     const result = await (0, import_router.runMutation)(ctx, {
@@ -2225,7 +2226,7 @@ Missing <verb> for "products ${verb}"`, 2);
     if (!mediaIdRaw) throw new import_errors.CliError("Missing --media-id", 2);
     const alt = args.alt;
     if (alt === void 0) throw new import_errors.CliError("Missing --alt", 2);
-    const files = [{ id: normalizeMediaId(mediaIdRaw), alt }];
+    const files = [{ id: normalizeMediaId(mediaIdRaw, "--media-id"), alt }];
     const result = await (0, import_router.runMutation)(ctx, {
       fileUpdate: {
         __args: { files },
@@ -2268,7 +2269,7 @@ Missing <verb> for "products ${verb}"`, 2);
         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: String(Math.floor(pos)) });
+        parsedMoves.push({ id: normalizeMediaId(mediaId, "--move mediaId"), newPosition: String(Math.floor(pos)) });
       }
       moves = parsedMoves;
     }
@@ -2285,7 +2286,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: String(Math.floor(pos)) };
+      return { id: normalizeMediaId(id2, `moves[${i}].id`), newPosition: String(Math.floor(pos)) };
     });
     const result = await (0, import_router.runMutation)(ctx, {
       productReorderMedia: {

package/dist/cli/verbs/publishables.js

@@ -40,7 +40,7 @@ const runPublishables = async ({
         "  publish-to-current-channel|unpublish-to-current-channel",
         "",
         "Notes:",
-        "  --id must be a full gid://shopify/... ID (numeric IDs cannot be coerced for Publishable)."
+        "  --id must be a full Shopify GID (gid://shopify/...)."
       ].join("\n")
     );
     return;

package/dist/cli.js

@@ -36,6 +36,7 @@ var import_command = require("./cli/command");
 var import_parse_command = require("./cli/parse-command");
 var import_output = require("./cli/output");
 var import_suggest = require("./cli/suggest");
+var import_gid = require("./cli/gid");
 import_dotenv.default.config({ path: (0, import_path.resolve)(process.cwd(), ".env"), quiet: true });
 const helpFlags = /* @__PURE__ */ new Set(["--help", "-h", "--help-full", "--help-all"]);
 const versionFlags = /* @__PURE__ */ new Set(["--version", "-v"]);
@@ -133,6 +134,8 @@ Missing <verb> for "${resource}"`, 2);
   }
   const parsed = (0, import_globalFlags.parseGlobalFlags)(rewrittenRest);
   const dryRun = parsed.dryRun ?? false;
+  const strictIds = parsed.strictIds ?? (0, import_gid.parseEnvBoolean)(process.env.SHOP_CLI_STRICT_IDS);
+  (0, import_gid.setStrictIdsMode)(strictIds);
   const isOfflineCommand = verb === "fields" || isTypesCommand;
   const shopDomain = parsed.shopDomain;
   const graphqlEndpoint = parsed.graphqlEndpoint;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "shop-cli",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "description": "An agent-friendly command-line interface for managing Shopify stores",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",