@shopify/cli-kit 2.0.13 → 2.0.14

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # @shopify/cli-kit
 
+## 2.0.14
+
+### Patch Changes
+
+- 87e51326: Report errors coming from child processes as abort errors
+- 87e51326: Output the sub-processes that we execute when the user invokes the CLI with --verbose
+- b10ddafc: Output requests' HTTP headers when the user invokes the CLI with --verbose
+
 ## 2.0.13
 
 ### Patch Changes

package/dist/{index-814fd0de.js → index-cf216492.js}

@@ -2190,7 +2190,7 @@ class ArrayPrompt$3 extends Prompt$5 {
       let len = this.choices.length;
       let num = this.num;
 
-      let handle = (val, res) => {
+      let handle = (val = false, res) => {
         clearTimeout(this.numberTimeout);
         if (val) res = number(num);
         this.num = '';
@@ -2810,7 +2810,7 @@ const utils$r = utils$z;
  * @api public
  */
 
-var placeholder$2 = (prompt, options) => {
+var placeholder$2 = (prompt, options = {}) => {
   prompt.cursorHide();
 
   let { input = '', initial = '', pos, showCursor = true, color } = options;
@@ -4059,7 +4059,7 @@ class Item {
   }
 }
 
-const tokenize = async(options = {}, defaults, fn = token => token) => {
+const tokenize = async(options = {}, defaults = {}, fn = token => token) => {
   let unique = new Set();
   let fields = options.fields || [];
   let input = options.template;
@@ -5610,7 +5610,7 @@ async function pLocate$2(
 	{
 		concurrency = Number.POSITIVE_INFINITY,
 		preserveOrder = true,
-	},
+	} = {},
 ) {
 	const limit = pLimit$3(concurrency);
 
@@ -5677,7 +5677,7 @@ const toPath = urlOrPath => urlOrPath instanceof URL ? fileURLToPath(urlOrPath)
 
 const findUpStop = Symbol('findUpStop');
 
-async function findUpMultiple(name, options) {
+async function findUpMultiple(name, options = {}) {
 	let directory = path$I.resolve(toPath(options.cwd) || '');
 	const {root} = path$I.parse(directory);
 	const stopAt = path$I.resolve(directory, options.stopAt || root);
@@ -6611,7 +6611,7 @@ const invalidStep = (step, options) => {
   return [];
 };
 
-const fillNumbers = (start, end, step = 1, options) => {
+const fillNumbers = (start, end, step = 1, options = {}) => {
   let a = Number(start);
   let b = Number(end);
 
@@ -6663,7 +6663,7 @@ const fillNumbers = (start, end, step = 1, options) => {
   return range;
 };
 
-const fillLetters = (start, end, step = 1, options) => {
+const fillLetters = (start, end, step = 1, options = {}) => {
   if ((!isNumber$3(start) && start.length > 1) || (!isNumber$3(end) && end.length > 1)) {
     return invalidRange(start, end, options);
   }
@@ -12317,7 +12317,7 @@ var path$w = /*#__PURE__*/Object.freeze({
 });
 
 var name = "@shopify/cli-kit";
-var version$4 = "2.0.13";
+var version$4 = "2.0.14";
 var description$1 = "A set of utilities, interfaces, and models that are common across all the platform features";
 var keywords = [
 	"shopify",
@@ -12432,11 +12432,11 @@ var cliKitPackageJson = {
 	devDependencies: devDependencies
 };
 
-var version$3 = "2.0.13";
+var version$3 = "2.0.14";
 
-var version$2 = "2.0.13";
+var version$2 = "2.0.14";
 
-var version$1 = "2.0.13";
+var version$1 = "2.0.14";
 
 const homedir$1 = os$8.homedir();
 const tmpdir$1 = os$8.tmpdir();
@@ -13225,7 +13225,7 @@ function npmRunPath(options = {}) {
 	return [...result, path_].join(path$I.delimiter);
 }
 
-function npmRunPathEnv({env = process$2.env, ...options}) {
+function npmRunPathEnv({env = process$2.env, ...options} = {}) {
 	env = {...env};
 
 	const path = pathKey({env});
@@ -14674,13 +14674,22 @@ const exec$2 = (command, args, options) => {
     cwd: options?.cwd,
     input: options?.stdin
   });
+  debug$5(`
+Running system process:
+  \xB7 Command: ${command} ${args.join(" ")}
+  \xB7 Working directory: ${options?.cwd ?? process.cwd()}
+`);
   if (options?.stderr) {
     commandProcess.stderr?.pipe(options.stderr);
   }
   if (options?.stdout) {
     commandProcess.stdout?.pipe(options.stdout);
   }
-  return commandProcess;
+  return commandProcess.catch((processError) => {
+    const abortError = new Abort(processError.message);
+    abortError.stack = processError.stack;
+    throw abortError;
+  });
 };
 const concurrentExec = async (commands) => {
   await concurrent(commands.map((command) => {
@@ -19029,7 +19038,7 @@ const rmkidsSync = (p, options) => {
 var rimraf_1 = rimraf$1;
 rimraf$1.sync = rimrafSync;
 
-var indentString$3 = (string, count, options) => {
+var indentString$3 = (string, count = 1, options) => {
 	options = {
 		indent: ' ',
 		includeEmptyLines: false,
@@ -19306,7 +19315,9 @@ del$2.exports = async (patterns, {force, dryRun, cwd = process.cwd(), onProgress
 		});
 	}
 
-	const mapper = async (file, fileIndex) => {
+	let deletedCount = 0;
+
+	const mapper = async file => {
 		file = path$a.resolve(cwd, file);
 
 		if (!force) {
@@ -19317,10 +19328,12 @@ del$2.exports = async (patterns, {force, dryRun, cwd = process.cwd(), onProgress
 			await rimrafP(file, rimrafOptions);
 		}
 
+		deletedCount += 1;
+
 		onProgress({
 			totalCount: files.length,
-			deletedCount: fileIndex,
-			percent: fileIndex / files.length
+			deletedCount,
+			percent: deletedCount / files.length
 		});
 
 		return file;
@@ -19328,12 +19341,6 @@ del$2.exports = async (patterns, {force, dryRun, cwd = process.cwd(), onProgress
 
 	const removedFiles = await pMap(files, mapper, options);
 
-	onProgress({
-		totalCount: files.length,
-		deletedCount: files.length,
-		percent: 1
-	});
-
 	removedFiles.sort((a, b) => a.localeCompare(b));
 
 	return removedFiles;
@@ -21094,7 +21101,7 @@ if (typeof Symbol === "function" && typeof Symbol.toStringTag === "symbol") {
 }
 
 class ContentToken {
-  constructor(value, metadata, type) {
+  constructor(value, metadata = {}, type) {
     this.type = type;
     this.value = value;
     this.metadata = metadata;
@@ -21131,17 +21138,25 @@ const token = {
   green: (value) => {
     return new ContentToken(value, {}, 9 /* Green */);
   },
-  command: (dependencyManager, scriptNameAndArgs) => {
-    return new ContentToken(formatPackageManagerCommand(dependencyManager, scriptNameAndArgs), {}, 0 /* Command */);
+  command: (dependencyManager, scriptName, ...scriptArgs) => {
+    return new ContentToken(formatPackageManagerCommand(dependencyManager, scriptName, scriptArgs), {}, 0 /* Command */);
   }
 };
-function formatPackageManagerCommand(dependencyManager, scriptNameAndArgs) {
+function formatPackageManagerCommand(dependencyManager, scriptName, scriptArgs) {
   switch (dependencyManager) {
-    case "yarn":
-      return `yarn ${scriptNameAndArgs}`;
+    case "yarn": {
+      const pieces = ["yarn", scriptName, ...scriptArgs];
+      return pieces.join(" ");
+    }
     case "pnpm":
-    case "npm":
-      return `${dependencyManager} run ${scriptNameAndArgs}`;
+    case "npm": {
+      const pieces = [dependencyManager, "run", scriptName];
+      if (scriptArgs.length > 0) {
+        pieces.push("--");
+        pieces.push(...scriptArgs);
+      }
+      return pieces.join(" ");
+    }
   }
 }
 class TokenizedString {
@@ -21305,7 +21320,7 @@ function stringifyMessage(message2) {
     return message2;
   }
 }
-const message = (content2, level) => {
+const message = (content2, level = "info") => {
   if (shouldOutput(level)) {
     consoleLog(stringifyMessage(content2));
   }
@@ -21315,7 +21330,8 @@ async function concurrent(processes) {
   const concurrentColors = [token.yellow, token.cyan, token.magenta, token.green];
   const prefixColumnSize = Math.max(...processes.map((process2) => process2.prefix.length));
   function linePrefix(prefix, index) {
-    const color = concurrentColors[0];
+    const colorIndex = index < concurrentColors.length ? index : index % concurrentColors.length;
+    const color = concurrentColors[colorIndex];
     return color(`${prefix}${" ".repeat(prefixColumnSize - prefix.length)} ${colors$9.bold("|")} `);
   }
   try {
@@ -21324,7 +21340,7 @@ async function concurrent(processes) {
         write(chunk, _encoding, next) {
           const lines = stripAnsiEraseCursorEscapeCharacters(chunk.toString("ascii")).split(/\n/);
           for (const line of lines) {
-            info(content`${linePrefix(process2.prefix)}${line}`);
+            info(content`${linePrefix(process2.prefix, index)}${line}`);
           }
           next();
         }
@@ -21333,7 +21349,7 @@ async function concurrent(processes) {
         write(chunk, _encoding, next) {
           const lines = stripAnsiEraseCursorEscapeCharacters(chunk.toString("ascii")).split(/\n/);
           for (const line of lines) {
-            message(content`${linePrefix(process2.prefix)}${colors$9.bold("ERROR")} ${line}`, "error");
+            message(content`${linePrefix(process2.prefix, index)}${colors$9.bold("ERROR")} ${line}`, "error");
           }
           next();
         }
@@ -21439,7 +21455,8 @@ var error$j = /*#__PURE__*/Object.freeze({
 	AbortSilent: AbortSilent,
 	Bug: Bug,
 	handler: handler,
-	mapper: mapper$1
+	mapper: mapper$1,
+	AbortSignal: AbortSignal
 });
 
 /******************************************************************************
@@ -35593,7 +35610,7 @@ function isTaskError(result) {
 function getErrorMessage(result) {
   return Buffer.concat([...result.stdOut, ...result.stdErr]);
 }
-function errorDetectionHandler(overwrite, isError = isTaskError, errorMessage = getErrorMessage) {
+function errorDetectionHandler(overwrite = false, isError = isTaskError, errorMessage = getErrorMessage) {
   return (error, result) => {
     if (!overwrite && error || !isError(result)) {
       return error;
@@ -40106,7 +40123,7 @@ try {
 const POOL_SIZE = 65536;
 
 /** @param {(Blob | Uint8Array)[]} parts */
-async function * toIterator (parts, clone) {
+async function * toIterator (parts, clone = true) {
   for (const part of parts) {
     if ('stream' in part) {
       yield * (/** @type {AsyncIterableIterator<Uint8Array>} */ (part.stream()));
@@ -40648,7 +40665,7 @@ class Body$1 {
 			return formData;
 		}
 
-		const {toFormData} = await import('./multipart-parser-2aa3d069.js');
+		const {toFormData} = await import('./multipart-parser-e0ecbeb6.js');
 		return toFormData(this.body, ct);
 	}
 
@@ -41301,7 +41318,7 @@ class Response$3 extends Body$1 {
 	 * @param {number} status An optional status code for the response (e.g., 302.)
 	 * @returns {Response}    A Response object.
 	 */
-	static redirect(url, status) {
+	static redirect(url, status = 302) {
 		if (!isRedirect(status)) {
 			throw new RangeError('Failed to execute "redirect" on "response": Invalid status code');
 		}
@@ -58020,7 +58037,7 @@ var decompressResponse = response => {
 };
 
 class QuickLRU$2 {
-	constructor(options) {
+	constructor(options = {}) {
 		if (!(options.maxSize && options.maxSize > 0)) {
 			throw new TypeError('`maxSize` must be a number greater than 0');
 		}
@@ -65887,7 +65904,7 @@ const compare = compare_1;
 //     - If no C has a prerelease and the LT.semver tuple, return false
 // - Else return true
 
-const subset = (sub, dom, options) => {
+const subset = (sub, dom, options = {}) => {
   if (sub === dom) {
     return true
   }
@@ -84244,7 +84261,7 @@ var mimicFn_1 = mimicFn$1;
 
 const mimicFn = mimicFn_1;
 
-var debounceFn = (inputFunction, options) => {
+var debounceFn = (inputFunction, options = {}) => {
 	if (typeof inputFunction !== 'function') {
 		throw new TypeError(`Expected the first argument to be a function, got \`${typeof inputFunction}\``);
 	}
@@ -84860,6 +84877,18 @@ async function buildHeaders(token) {
   };
   return headers;
 }
+function sanitizedHeadersOutput(headers) {
+  const sanitized = {};
+  const keywords = ["token", "authorization"];
+  Object.keys(headers).forEach((header) => {
+    if (keywords.find((keyword) => header.toLocaleLowerCase().includes(keyword)) === void 0) {
+      sanitized[header] = headers[header];
+    }
+  });
+  return Object.keys(sanitized).map((header) => {
+    return ` - ${header}: ${sanitized[header]}`;
+  }).join("\n");
+}
 
 var dist$1 = {};
 
@@ -170582,7 +170611,7 @@ const printDocASTReducer = {
  * print all items together separated by separator if provided
  */
 
-function join(maybeArray, separator) {
+function join(maybeArray, separator = '') {
   var _maybeArray$filter$jo;
 
   return (_maybeArray$filter$jo =
@@ -171890,17 +171919,28 @@ function HeadersInstanceToPlainObject(headers) {
 
 }(dist$1));
 
+const UnauthorizedAccessError = () => {
+  return new Bug(`You can't use Shopify CLI with development stores if you only have Partner staff member access.
+If you want to use Shopify CLI to work on a development store, then you should be the store owner or create a staff account on the store`, `If you're the store owner, then you need to log in to the store directly using the store URL at least once (for example, using %s.myshopify.com/admin) before you log in using Shopify CLI.
+Logging in to the Shopify admin directly connects the development store with your Shopify login.`);
+};
+const UnknownError = () => {
+  return new Bug(`Unknown error connecting to your store`);
+};
 async function request$1(query, session, variables) {
-  debug$5(`
-Sending Admin GraphQL request:
-${query}
-
-With variables:
-${variables ? JSON.stringify(variables, null, 2) : ""}
-  `);
   const version = await fetchApiVersion(session);
   const url = adminUrl(session.storeFqdn, version);
   const headers = await buildHeaders(session.token);
+  debug$5(`
+  Sending Admin GraphQL request:
+  ${query}
+
+  With variables:
+  ${variables ? JSON.stringify(variables, null, 2) : ""}
+
+  And headers:
+  ${sanitizedHeadersOutput(headers)}
+`);
   return dist$1.request(url, query, variables, headers);
 }
 async function fetchApiVersion(session) {
@@ -171911,7 +171951,9 @@ async function fetchApiVersion(session) {
 Sending Admin GraphQL request to URL ${url} with query:
 ${query}
   `);
-  const data = await dist$1.request(url, query, {}, headers);
+  const data = await dist$1.request(url, query, {}, headers).catch((err) => {
+    throw err.response.status === 403 ? UnauthorizedAccessError() : UnknownError();
+  });
   return data.publicApiVersions.filter((item) => item.supported).map((item) => item.handle).sort().reverse()[0];
 }
 function adminUrl(store, version) {
@@ -171944,6 +171986,9 @@ ${query}
 
 With variables:
 ${variables ? JSON.stringify(variables, null, 2) : ""}
+
+And headers:
+${sanitizedHeadersOutput(headers)}
   `);
   return dist$1.request(url, query, variables, headers);
 }
@@ -172143,6 +172188,25 @@ const ExtensionCreateQuery = dist$1.gql`
   }
 `;
 
+const FindProductVariantQuery = dist$1.gql`
+  query {
+    products(first: 1) {
+      edges {
+        node {
+          id
+          variants(first: 1) {
+            edges {
+              node {
+                id
+              }
+            }
+          }
+        }
+      }
+    }
+  }
+`;
+
 var index = /*#__PURE__*/Object.freeze({
 	__proto__: null,
 	FindOrganizationQuery: FindOrganizationQuery,
@@ -172155,7 +172219,8 @@ var index = /*#__PURE__*/Object.freeze({
 	CreateDeployment: CreateDeployment,
 	AllStoresByOrganizationQuery: AllStoresByOrganizationQuery,
 	ConvertDevToTestStoreQuery: ConvertDevToTestStoreQuery,
-	ExtensionCreateQuery: ExtensionCreateQuery
+	ExtensionCreateQuery: ExtensionCreateQuery,
+	FindProductVariantQuery: FindProductVariantQuery
 });
 
 var api = /*#__PURE__*/Object.freeze({
@@ -172795,4 +172860,4 @@ var plugins = /*#__PURE__*/Object.freeze({
 });
 
 export { npm as A, port as B, cli as C, id as D, temporary as E, FormData$3 as F, dotEnv as G, constants$2 as H, plugins as I, File$1 as a, string as b, github as c, dependency as d, error$j as e, file$1 as f, git as g, os$2 as h, environment as i, session as j, schema$2 as k, toml as l, store as m, api as n, output as o, path$w as p, http$2 as q, archiver as r, system as s, template as t, ui as u, version as v, checksum as w, ruby as x, yaml as y, semver as z };
-//# sourceMappingURL=index-814fd0de.js.map
+//# sourceMappingURL=index-cf216492.js.map