@shopify/oxygen-cli 1.2.0 → 1.3.1

package/README.md

@@ -68,7 +68,7 @@ environment:
   OXYGEN_DEPLOYMENT_TOKEN: ${YOUR_ENV_VARIABLE}
 ```
 
-When run from a supported CI/CD environment, the tool will automatically retrieve metadata from relevant environment variables. Metadata describes the author making the deployment (user), the version (commit Sha) and the deployment URL (workflow URL).
+When run from a supported CI/CD environment, the tool will automatically retrieve metadata from relevant environment variables. Metadata describes the author making the deployment (user), the version (commit Sha) and the deployment URL (workflow URL). This metadata will be saved and used to show details about the Oxygen deployment in the Hydrogen channel within the Shopify Admin.
 
 The following environment variables are relevant, depending on your environment:
 
@@ -78,9 +78,9 @@ The following environment variables are relevant, depending on your environment:
 | Version | BITBUCKET_COMMIT | CIRCLE_SHA1 | GITHUB_SHA | CI_COMMIT_SHA |
 | URL | BITBUCKET_BUILD_URL | CIRCLE_BUILD_URL | `${GITHUB_SERVER_URL}${GITHUB_REPOSITORY}/actions/runs/${GITHUB_RUN_ID}` | CI_PROJECT_URL |
 
-These values can be overruled by the command line flags `metadataUser`, `metadataUrl` and `metadataVersion` respectively.
+These values can be overridden by the command line flags `metadataUser`, `metadataUrl` and `metadataVersion` respectively.
 
-The tag of the environment to deploy to, will be derived from the following variables:
+The environment tag to deploy to will be derived from the following variables:
 
 | Service | Branch |
 | ------- | ------ |
@@ -89,4 +89,4 @@ The tag of the environment to deploy to, will be derived from the following vari
 | Github | GITHUB_REF_NAME |
 | Gitlab | CI_COMMIT_REF_NAME |
 
-The `environmentTag` command line option can be used to override this, and is required when the tool is run from outside a supported CI environment.
+The `environmentTag` command line option can be used to override this. If not provided, or there is no environment with the specified tag, the deployment will be deployed to the default environment for the app (if one exists).

package/dist/commands/oxygen/deploy.js

@@ -67,17 +67,17 @@ class Deploy extends Command {
       default: false
     }),
     metadataUrl: Flags.string({
-      description: "URL that links to the deployment",
+      description: "URL that links to the deployment. Will be saved and displayed in the Shopify admin",
       required: false,
       env: "OXYGEN_METADATA_URL"
     }),
     metadataUser: Flags.string({
-      description: "User that initiated the deployment",
+      description: "User that initiated the deployment. Will be saved and displayed in the Shopify admin",
       required: false,
       env: "OXYGEN_METADATA_USER"
     }),
     metadataVersion: Flags.string({
-      description: "A version identifier for the deployment",
+      description: "A version identifier for the deployment. Will be saved and displayed in the Shopify admin",
       required: false,
       env: "OXYGEN_METADATA_VERSION"
     })

package/dist/deploy/build-cancel.js

@@ -1,7 +1,7 @@
 import { graphqlRequest } from '@shopify/cli-kit/node/api/graphql';
 import { AbortError } from '@shopify/cli-kit/node/error';
 import { outputInfo } from '@shopify/cli-kit/node/output';
-import { Header, errorHandler } from '../utils/utils.js';
+import { Header, stderrLogger, errorHandler } from '../utils/utils.js';
 import { BuildCancelQuery } from './graphql/build-cancel.js';
 
 async function buildCancel(config, buildId, reason) {
@@ -25,7 +25,7 @@ async function buildCancel(config, buildId, reason) {
         `Failed to cancel build: ${response.buildCancel.userErrors[0]?.message}`
       );
     }
-    outputInfo(`Build with id ${buildId} cancelled.`);
+    outputInfo(`Build with id ${buildId} cancelled.`, stderrLogger);
     return response.buildCancel;
   } catch (error) {
     errorHandler(error);

package/dist/deploy/build-initiate.js

@@ -1,7 +1,7 @@
 import { graphqlRequest } from '@shopify/cli-kit/node/api/graphql';
 import { AbortError } from '@shopify/cli-kit/node/error';
 import { outputCompleted } from '@shopify/cli-kit/node/output';
-import { Header, errorHandler } from '../utils/utils.js';
+import { Header, stderrLogger, errorHandler } from '../utils/utils.js';
 import { BuildInitiateQuery } from './graphql/build-initiate.js';
 
 async function buildInitiate(config, environment, labels = []) {
@@ -26,7 +26,8 @@ async function buildInitiate(config, environment, labels = []) {
       );
     }
     outputCompleted(
-      `Build initiated successfully with id ${response.buildInitiate.build.id}.`
+      `Build initiated successfully with id ${response.buildInitiate.build.id}.`,
+      stderrLogger
     );
     return response.buildInitiate;
   } catch (error) {

package/dist/deploy/build-initiate.test.js

@@ -2,7 +2,7 @@ import { AbortError } from '@shopify/cli-kit/node/error';
 import { graphqlRequest } from '@shopify/cli-kit/node/api/graphql';
 import { outputCompleted } from '@shopify/cli-kit/node/output';
 import { vi, describe, test, expect } from 'vitest';
-import { Header } from '../utils/utils.js';
+import { Header, stderrLogger } from '../utils/utils.js';
 import { createTestConfig } from '../utils/test-helper.js';
 import { buildInitiate } from './build-initiate.js';
 
@@ -39,7 +39,8 @@ describe("BuildInitiate", () => {
       }
     });
     expect(outputCompleted).toHaveBeenCalledWith(
-      "Build initiated successfully with id build-1."
+      "Build initiated successfully with id build-1.",
+      stderrLogger
     );
   });
   test("should throw AbortError when build initiation fails due to user errors", async () => {

package/dist/deploy/build-project.js

@@ -4,7 +4,7 @@ async function buildProject(config, assetPath) {
   const assetPathEnvironment = assetPath ? { HYDROGEN_ASSET_BASE_URL: assetPath } : {};
   await new Promise((resolve, reject) => {
     const buildCommand = spawn(config.buildCommand, [], {
-      stdio: "inherit",
+      stdio: ["inherit", "pipe", "inherit"],
       env: {
         // eslint-disable-next-line no-process-env
         ...process.env,
@@ -13,11 +13,7 @@ async function buildProject(config, assetPath) {
       cwd: config.rootPath,
       shell: true
     });
-    if (buildCommand.stdout) {
-      buildCommand.stdout.on("data", (data) => {
-        console.log(`stdout: ${data}`);
-      });
-    }
+    buildCommand.stdout.pipe(process.stderr);
     buildCommand.on("close", (code) => {
       if (code !== 0) {
         reject(code);

package/dist/deploy/build-project.test.js

@@ -10,6 +10,8 @@ vi.mock("child_process", () => {
     return {
       stdout: {
         on: () => {
+        },
+        pipe: () => {
         }
       },
       on: (event, callback) => {
@@ -32,7 +34,7 @@ test("BuildProject builds the project successfully", async () => {
   expect(spawn).toBeCalledWith("npm run build", [], {
     cwd: "rootFolder",
     shell: true,
-    stdio: "inherit",
+    stdio: ["inherit", "pipe", "inherit"],
     env: {
       // eslint-disable-next-line no-process-env
       ...process.env,

package/dist/deploy/deployment-cancel.d.ts

@@ -1,6 +1,10 @@
 import { DeployConfig } from './types.js';
 import { DeploymentCancelResponse } from './graphql/deployment-cancel.js';
 
-declare function deploymentCancel(config: DeployConfig, deploymentId: string, reason: string): Promise<DeploymentCancelResponse>;
+declare enum DeploymentCancelReason {
+    Failed = "FAILED",
+    Cancelled = "CANCELLED"
+}
+declare function deploymentCancel(config: DeployConfig, deploymentId: string, reason: DeploymentCancelReason): Promise<DeploymentCancelResponse>;
 
-export { deploymentCancel };
+export { DeploymentCancelReason, deploymentCancel };

package/dist/deploy/deployment-cancel.js

@@ -1,9 +1,14 @@
 import { graphqlRequest } from '@shopify/cli-kit/node/api/graphql';
 import { AbortError } from '@shopify/cli-kit/node/error';
 import { outputInfo } from '@shopify/cli-kit/node/output';
-import { Header, errorHandler } from '../utils/utils.js';
+import { Header, stderrLogger, errorHandler } from '../utils/utils.js';
 import { DeploymentCancelQuery } from './graphql/deployment-cancel.js';
 
+var DeploymentCancelReason = /* @__PURE__ */ ((DeploymentCancelReason2) => {
+  DeploymentCancelReason2["Failed"] = "FAILED";
+  DeploymentCancelReason2["Cancelled"] = "CANCELLED";
+  return DeploymentCancelReason2;
+})(DeploymentCancelReason || {});
 async function deploymentCancel(config, deploymentId, reason) {
   const variables = {
     deploymentId,
@@ -25,7 +30,7 @@ async function deploymentCancel(config, deploymentId, reason) {
         `Failed to cancel deployment: ${response.deploymentCancel.userErrors[0]?.message}`
       );
     }
-    outputInfo(`Deployment with id ${deploymentId} cancelled.`);
+    outputInfo(`Deployment with id ${deploymentId} cancelled.`, stderrLogger);
     return response.deploymentCancel;
   } catch (error) {
     errorHandler(error);
@@ -33,4 +38,4 @@ async function deploymentCancel(config, deploymentId, reason) {
   }
 }
 
-export { deploymentCancel };
+export { DeploymentCancelReason, deploymentCancel };

package/dist/deploy/deployment-cancel.test.js

@@ -3,7 +3,7 @@ import { graphqlRequest } from '@shopify/cli-kit/node/api/graphql';
 import { vi, describe, test, expect } from 'vitest';
 import { createTestConfig } from '../utils/test-helper.js';
 import { Header } from '../utils/utils.js';
-import { deploymentCancel } from './deployment-cancel.js';
+import { deploymentCancel, DeploymentCancelReason } from './deployment-cancel.js';
 
 vi.mock("@shopify/cli-kit/node/api/graphql");
 vi.mock("@shopify/cli-kit/node/output");
@@ -22,7 +22,7 @@ describe("DeploymentComplete", () => {
     const completeResponse = await deploymentCancel(
       testConfig,
       "deployment-1",
-      "some reason"
+      DeploymentCancelReason.Failed
     );
     expect(completeResponse).toEqual(response.deploymentCancel);
     expect(graphqlRequest).toHaveBeenCalledWith({
@@ -32,7 +32,7 @@ describe("DeploymentComplete", () => {
       token: testConfig.deploymentToken.accessToken,
       variables: {
         deploymentId: "deployment-1",
-        reason: "some reason"
+        reason: DeploymentCancelReason.Failed
       },
       addedHeaders: {
         [Header.OxygenNamespaceHandle]: `${testConfig.deploymentToken.namespace}`
@@ -51,7 +51,11 @@ describe("DeploymentComplete", () => {
     };
     vi.mocked(graphqlRequest).mockResolvedValueOnce(response);
     await expect(
-      deploymentCancel(testConfig, "deployment-1", "some reason")
+      deploymentCancel(
+        testConfig,
+        "deployment-1",
+        DeploymentCancelReason.Failed
+      )
     ).rejects.toThrow(
       new AbortError(
         `Failed to cancel deployment: ${response.deploymentCancel.userErrors[0]?.message}`
@@ -65,7 +69,11 @@ describe("DeploymentComplete", () => {
     vi.mocked(graphqlRequest).mockRejectedValueOnce(error);
     try {
       await expect(
-        deploymentCancel(testConfig, "deployment-1", "some reason")
+        deploymentCancel(
+          testConfig,
+          "deployment-1",
+          DeploymentCancelReason.Failed
+        )
       ).rejects.toThrow(
         new AbortError(
           "You are not authorized to perform this action. Please check your deployment token."

package/dist/deploy/deployment-initiate.js

@@ -1,7 +1,7 @@
 import { graphqlRequest } from '@shopify/cli-kit/node/api/graphql';
 import { AbortError } from '@shopify/cli-kit/node/error';
 import { outputCompleted } from '@shopify/cli-kit/node/output';
-import { Header, errorHandler } from '../utils/utils.js';
+import { Header, stderrLogger, errorHandler } from '../utils/utils.js';
 import { DeploymentInitiateQuery } from './graphql/deployment-initiate.js';
 
 async function deploymentInitiate(config, input) {
@@ -29,7 +29,8 @@ async function deploymentInitiate(config, input) {
       );
     }
     outputCompleted(
-      `Deployment initiated, ${response.deploymentInitiate.deploymentTargets.length} files to upload.`
+      `Deployment initiated, ${response.deploymentInitiate.deploymentTargets.length} files to upload.`,
+      stderrLogger
     );
     return response.deploymentInitiate;
   } catch (error) {

package/dist/deploy/deployment-initiate.test.js

@@ -3,7 +3,7 @@ import { graphqlRequest } from '@shopify/cli-kit/node/api/graphql';
 import { outputCompleted } from '@shopify/cli-kit/node/output';
 import { vi, describe, test, expect } from 'vitest';
 import { createTestConfig } from '../utils/test-helper.js';
-import { Header } from '../utils/utils.js';
+import { Header, stderrLogger } from '../utils/utils.js';
 import { deploymentInitiate } from './deployment-initiate.js';
 
 vi.mock("@shopify/cli-kit/node/api/graphql");
@@ -51,14 +51,16 @@ describe("DeploymentInitiate", () => {
       variables: {
         buildId: "build-1",
         environment: void 0,
-        files: testManifest
+        files: testManifest,
+        isPrivate: true
       },
       addedHeaders: {
         [Header.OxygenNamespaceHandle]: `${testConfig.deploymentToken.namespace}`
       }
     });
     expect(outputCompleted).toHaveBeenCalledWith(
-      `Deployment initiated, ${testManifest.length} files to upload.`
+      `Deployment initiated, ${testManifest.length} files to upload.`,
+      stderrLogger
     );
   });
   test("should initiate a deployment with an environmentName", async () => {
@@ -77,14 +79,16 @@ describe("DeploymentInitiate", () => {
       variables: {
         buildId: void 0,
         environment: { tag: "preview" },
-        files: testManifest
+        files: testManifest,
+        isPrivate: true
       },
       addedHeaders: {
         [Header.OxygenNamespaceHandle]: `${testConfig.deploymentToken.namespace}`
       }
     });
     expect(outputCompleted).toHaveBeenCalledWith(
-      `Deployment initiated, ${testManifest.length} files to upload.`
+      `Deployment initiated, ${testManifest.length} files to upload.`,
+      stderrLogger
     );
   });
   test("should throw AbortError when deployment initiation fails due to user errors", async () => {

package/dist/deploy/graphql/deployment-complete.d.ts

@@ -1,6 +1,6 @@
 import { OxygenError } from '../types.js';
 
-declare const DeploymentCompleteQuery = "\n  mutation DeploymentComplete($deploymentId: ID!) {\n    deploymentComplete(id: $deploymentId) {\n      deployment {\n        id\n        status\n        url\n      }\n      userErrors {\n        message\n      }\n    }\n  }\n";
+declare const DeploymentCompleteQuery = "\n  mutation DeploymentComplete($deploymentId: ID!) {\n    deploymentComplete(id: $deploymentId) {\n      deployment {\n        id\n        url\n      }\n      userErrors {\n        message\n      }\n    }\n  }\n";
 interface DeploymentCompleteQueryData {
     deploymentComplete: DeploymentCompleteResponse;
 }
@@ -10,7 +10,6 @@ interface DeploymentCompleteResponse {
 }
 interface Deployment {
     id: string;
-    status: string;
     url: string;
 }
 

package/dist/deploy/graphql/deployment-complete.js

@@ -3,7 +3,6 @@ const DeploymentCompleteQuery = `
     deploymentComplete(id: $deploymentId) {
       deployment {
         id
-        status
         url
       }
       userErrors {

package/dist/deploy/graphql/deployment-initiate.d.ts

@@ -1,6 +1,6 @@
 import { OxygenError } from '../types.js';
 
-declare const DeploymentInitiateQuery = "\n  mutation DeploymentInitiate($buildId: ID, $environment: EnvironmentSelectorInput, $labels: [String!], $files: [FileInput!]!, $isPrivate: Boolean) {\n    deploymentInitiate(buildId: $buildId, environment: $environment, labels: $labels, files: $files, isPrivate: $isPrivate) {\n      deployment {\n        id\n        status\n      }\n      deploymentTargets {\n        filePath\n        fileSize\n        uploadUrl\n        fileType\n        parameters {\n          name\n          value\n        }\n      }\n      userErrors {\n        message\n      }\n    }\n  }\n";
+declare const DeploymentInitiateQuery = "\n  mutation DeploymentInitiate($buildId: ID, $environment: EnvironmentSelectorInput, $labels: [String!], $files: [FileInput!]!, $isPrivate: Boolean) {\n    deploymentInitiate(buildId: $buildId, environment: $environment, labels: $labels, files: $files, isPrivate: $isPrivate) {\n      deployment {\n        id\n      }\n      deploymentTargets {\n        filePath\n        fileSize\n        uploadUrl\n        fileType\n        parameters {\n          name\n          value\n        }\n      }\n      userErrors {\n        message\n      }\n    }\n  }\n";
 interface DeploymentInitiateQueryData {
     deploymentInitiate: DeploymentInitiateResponse;
 }
@@ -18,7 +18,6 @@ interface DeploymentTargetResponse {
 }
 interface Deployment {
     id: string;
-    status: string;
 }
 interface DeploymentInitiateParameters {
     name: string;

package/dist/deploy/graphql/deployment-initiate.js

@@ -3,7 +3,6 @@ const DeploymentInitiateQuery = `
     deploymentInitiate(buildId: $buildId, environment: $environment, labels: $labels, files: $files, isPrivate: $isPrivate) {
       deployment {
         id
-        status
       }
       deploymentTargets {
         filePath

package/dist/deploy/index.js

@@ -1,11 +1,11 @@
-import { outputSuccess, outputWarn, consoleError } from '@shopify/cli-kit/node/output';
-import { verifyConfig } from '../utils/utils.js';
+import { outputSuccess, outputInfo, outputWarn, consoleError } from '@shopify/cli-kit/node/output';
+import { verifyConfig, stderrLogger } from '../utils/utils.js';
 import { buildInitiate } from './build-initiate.js';
 import { buildCancel } from './build-cancel.js';
 import { getUploadFiles } from './get-upload-files.js';
 import { deploymentInitiate } from './deployment-initiate.js';
 import { deploymentComplete } from './deployment-complete.js';
-import { deploymentCancel } from './deployment-cancel.js';
+import { deploymentCancel, DeploymentCancelReason } from './deployment-cancel.js';
 import { uploadFiles } from './upload-files.js';
 import { buildProject } from './build-project.js';
 import { getMetadata, createLabels, getEnvironmentInput } from './metadata.js';
@@ -43,31 +43,44 @@ async function createDeploy(config) {
     const urlMessage = config.publicDeployment ? "Public" : "Private";
     outputSuccess(
       `Deployment complete.
-${urlMessage} preview URL: ${deploymentCompleteOp.deployment.url}`
+${urlMessage} preview URL: ${deploymentCompleteOp.deployment.url}`,
+      stderrLogger
     );
+    if (metadata.name !== "none") {
+      outputInfo(deploymentCompleteOp.deployment.url);
+    }
   } catch (error) {
     if (!(error instanceof Error)) {
       console.error("Unknown error", error);
       return;
     }
     if (build.id && !buildCompleted) {
-      outputWarn(`Build failed with: ${error.message}, cancelling build.`);
+      outputWarn(
+        `Build failed with: ${error.message}, cancelling build.`,
+        stderrLogger
+      );
       await buildCancel(config, build.id, error.message).catch((err) => {
         if (err instanceof Error) {
-          outputWarn(`Failed to cancel build: ${err.message}`);
+          outputWarn(`Failed to cancel build: ${err.message}`, stderrLogger);
         }
       });
     } else if (deployment?.deployment.id) {
       outputWarn(
-        `Deployment failed with: ${error.message}, cancelling deployment.`
+        `Deployment failed with: ${error.message}, cancelling deployment.`,
+        stderrLogger
       );
-      await deploymentCancel(config, deployment.deployment.id, "failed").catch(
-        (err) => {
-          if (err instanceof Error) {
-            outputWarn(`Failed to cancel deployment: ${err.message}`);
-          }
+      await deploymentCancel(
+        config,
+        deployment.deployment.id,
+        DeploymentCancelReason.Failed
+      ).catch((err) => {
+        if (err instanceof Error) {
+          outputWarn(
+            `Failed to cancel deployment: ${err.message}`,
+            stderrLogger
+          );
         }
-      );
+      });
     }
     consoleError(error.message);
   }

package/dist/deploy/metadata.js

@@ -1,7 +1,7 @@
 import { ciPlatform } from '@shopify/cli-kit/node/context/local';
 import { getLatestGitCommit } from '@shopify/cli-kit/node/git';
 import { outputWarn } from '@shopify/cli-kit/node/output';
-import { maxLabelLength } from '../utils/utils.js';
+import { stderrLogger, maxLabelLength } from '../utils/utils.js';
 
 async function getMetadata(config) {
   const ciInfo = ciPlatform();
@@ -19,11 +19,11 @@ async function getMetadata(config) {
         commitMessage: gitCommit.message
       };
     } catch (error) {
-      outputWarn("No CI metadata loaded from environment");
+      outputWarn("No CI metadata loaded from environment", stderrLogger);
     }
   }
   return {
-    name: ciInfo.isCI ? ciInfo.name : "unknown",
+    name: ciInfo.isCI ? ciInfo.name : "none",
     ...metadata,
     actor: config.metadata.user ?? metadata.actor,
     commitSha: config.metadata.version ?? metadata.commitSha,

package/dist/deploy/metadata.test.js

@@ -64,7 +64,7 @@ describe("getMetadata", () => {
     const metadataResult = await getMetadata(testConfig);
     expect(metadataResult.actor).toBe("gh_author");
     expect(metadataResult.commitSha).toBe("gh_hash");
-    expect(metadataResult.name).toBe("unknown");
+    expect(metadataResult.name).toBe("none");
     expect(metadataResult.url).toBe(void 0);
   });
 });

package/dist/deploy/upload-files.js

@@ -3,14 +3,14 @@ import { createFileReadStream } from '@shopify/cli-kit/node/fs';
 import { outputInfo, outputCompleted } from '@shopify/cli-kit/node/output';
 import { joinPath } from '@shopify/cli-kit/node/path';
 import { mapLimit } from 'async';
-import { deployDefaults } from '../utils/utils.js';
+import { stderrLogger, deployDefaults } from '../utils/utils.js';
 
 async function uploadFiles(config, targets) {
-  outputInfo(`Uploading ${targets.length} files...`);
+  outputInfo(`Uploading ${targets.length} files...`, stderrLogger);
   return mapLimit(targets, 6, async (target) => {
     await uploadFile(config, target);
   }).then(() => {
-    outputCompleted(`Files uploaded successfully`);
+    outputCompleted(`Files uploaded successfully`, stderrLogger);
   });
 }
 async function uploadFile(config, target) {

package/dist/utils/utils.d.ts

@@ -9,6 +9,7 @@ declare enum Header {
     OxygenNamespaceHandle = "X-Oxygen-Namespace-Handle"
 }
 declare function isClientError(error: unknown): error is ClientError;
+declare function stderrLogger(log: string): void;
 declare const maxLabelLength = 90;
 declare function parseToken(inputToken: string): DeploymentToken;
 interface VerifyConfigParams {
@@ -17,4 +18,4 @@ interface VerifyConfigParams {
 }
 declare function verifyConfig({ config, performedBuild, }: VerifyConfigParams): Promise<void>;
 
-export { Header, deployDefaults, errorHandler, getBuildCommandFromLockFile, isClientError, maxLabelLength, parseToken, verifyConfig };
+export { Header, deployDefaults, errorHandler, getBuildCommandFromLockFile, isClientError, maxLabelLength, parseToken, stderrLogger, verifyConfig };

package/dist/utils/utils.js

@@ -43,13 +43,17 @@ function getBuildCommandFromLockFile(config) {
   }
   if (foundLockFiles.length > 1) {
     const lockFilesList = foundLockFiles.map(({ lockFileName }) => lockFileName).join(", ");
-    outputWarn(`Warning: Multiple lock files found: (${lockFilesList}).`);
+    outputWarn(
+      `Warning: Multiple lock files found: (${lockFilesList}).`,
+      stderrLogger
+    );
   }
   if (foundLockFiles.length > 0) {
     const { lockFileName, buildCommand } = foundLockFiles[0];
     const infoMsg = foundLockFiles.length > 1 ? "" : `Found: ${lockFileName}. `;
     outputInfo(
-      `${infoMsg}Assuming "${buildCommand}" as build command. Use the buildCommand flag to override.`
+      `${infoMsg}Assuming "${buildCommand}" as build command. Use the buildCommand flag to override.`,
+      stderrLogger
     );
     return buildCommand;
   }
@@ -62,6 +66,10 @@ var Header = /* @__PURE__ */ ((Header2) => {
 function isClientError(error) {
   return typeof error === "object" && error !== null && "statusCode" in error;
 }
+function stderrLogger(log) {
+  process.stderr.write(`${log}
+`);
+}
 const maxLabelLength = 90;
 function parseToken(inputToken) {
   try {
@@ -91,7 +99,7 @@ async function verifyConfig({
   for (const pathType of Object.keys(checkPaths)) {
     await checkPath(checkPaths[pathType], pathType);
   }
-  const addressRegex = /^(http|https):\/\/(?:[\w-]+\.)*[\w-]+|(?:http|https):\/\/\d{1,3}\.\d{1,3}\.\d{1,3}\.\d{1,3}/;
+  const addressRegex = /^https:\/\/(?:[\w-]+\.)*[\w-]+|^https:\/\/\d{1,3}\.\d{1,3}\.\d{1,3}\.\d{1,3}/;
   if (!addressRegex.test(config.deploymentUrl)) {
     throw new Error(`Invalid deployment service URL: ${config.deploymentUrl}`);
   }
@@ -100,7 +108,8 @@ async function checkPath(path, pathType) {
   if (!await fileExists(path)) {
     if (pathType === "assets") {
       outputWarn(
-        `Use the "workerOnly" flag to perform a worker-only deployment.`
+        `Use the "workerOnly" flag to perform a worker-only deployment.`,
+        stderrLogger
       );
     }
     throw new Error(`Path not found: ${path}`);
@@ -123,4 +132,4 @@ function convertKeysToCamelCase(obj) {
   return obj;
 }
 
-export { Header, deployDefaults, errorHandler, getBuildCommandFromLockFile, isClientError, maxLabelLength, parseToken, verifyConfig };
+export { Header, deployDefaults, errorHandler, getBuildCommandFromLockFile, isClientError, maxLabelLength, parseToken, stderrLogger, verifyConfig };

package/dist/utils/utils.test.js

@@ -29,7 +29,8 @@ describe("getBuildCommandFromLockfile", () => {
     const buildCommand = utils.getBuildCommandFromLockFile(testConfig);
     expect(buildCommand).toBe("pnpm run build");
     expect(outputInfo).toHaveBeenCalledWith(
-      'Found: pnpm-lock.yaml. Assuming "pnpm run build" as build command. Use the buildCommand flag to override.'
+      'Found: pnpm-lock.yaml. Assuming "pnpm run build" as build command. Use the buildCommand flag to override.',
+      utils.stderrLogger
     );
   });
   test("getBuildCommandFromLockfile the first build command and warns when multiple lockfiles are found", async () => {
@@ -37,10 +38,12 @@ describe("getBuildCommandFromLockfile", () => {
     const buildCommand = utils.getBuildCommandFromLockFile(testConfig);
     expect(buildCommand).toBe("pnpm run build");
     expect(outputInfo).toHaveBeenCalledWith(
-      'Assuming "pnpm run build" as build command. Use the buildCommand flag to override.'
+      'Assuming "pnpm run build" as build command. Use the buildCommand flag to override.',
+      utils.stderrLogger
     );
     expect(outputWarn).toHaveBeenCalledWith(
-      "Warning: Multiple lock files found: (pnpm-lock.yaml, yarn.lock)."
+      "Warning: Multiple lock files found: (pnpm-lock.yaml, yarn.lock).",
+      utils.stderrLogger
     );
   });
 });
@@ -94,7 +97,8 @@ describe("verifyConfig", () => {
       )
     );
     expect(outputWarn).toHaveBeenCalledWith(
-      `Use the "workerOnly" flag to perform a worker-only deployment.`
+      `Use the "workerOnly" flag to perform a worker-only deployment.`,
+      utils.stderrLogger
     );
   });
   test("verifyConfig throws when the workerDir cannot be resolved after a build", async () => {

package/oclif.manifest.json

@@ -1,5 +1,5 @@
 {
-  "version": "1.2.0",
+  "version": "1.3.1",
   "commands": {
     "oxygen:deploy": {
       "id": "oxygen:deploy",
@@ -90,21 +90,21 @@
         "metadataUrl": {
           "name": "metadataUrl",
           "type": "option",
-          "description": "URL that links to the deployment",
+          "description": "URL that links to the deployment. Will be saved and displayed in the Shopify admin",
           "required": false,
           "multiple": false
         },
         "metadataUser": {
           "name": "metadataUser",
           "type": "option",
-          "description": "User that initiated the deployment",
+          "description": "User that initiated the deployment. Will be saved and displayed in the Shopify admin",
           "required": false,
           "multiple": false
         },
         "metadataVersion": {
           "name": "metadataVersion",
           "type": "option",
-          "description": "A version identifier for the deployment",
+          "description": "A version identifier for the deployment. Will be saved and displayed in the Shopify admin",
           "required": false,
           "multiple": false
         }

package/package.json

@@ -5,7 +5,7 @@
     "@shopify:registry": "https://registry.npmjs.org"
   },
   "license": "MIT",
-  "version": "1.2.0",
+  "version": "1.3.1",
   "type": "module",
   "scripts": {
     "build": "tsup --clean --config ./tsup.config.ts && oclif manifest",
@@ -13,7 +13,7 @@
     "dev": "tsup --watch --config ./tsup.config.ts",
     "typecheck": "tsc --noEmit",
     "lint": "eslint --ext .js,.ts --max-warnings 0 src",
-    "lint:format": "npm run lint --fix",
+    "lint:format": "prettier --write \"**/*.*\" && npm run lint --fix",
     "generate:manifest": "oclif manifest",
     "prepack": "npm install && npm run build",
     "release": "npm run build && npx changeset publish",
@@ -31,7 +31,7 @@
     "/oclif.manifest.json"
   ],
   "dependencies": {
-    "@oclif/core": "2.1.4",
+    "@oclif/core": "2.8.6",
     "@shopify/cli-kit": "^3.46.0",
     "async": "^3.2.4"
   },
@@ -41,14 +41,14 @@
     "@shopify/prettier-config": "^1.1.2",
     "@types/async": "^3.2.18",
     "@types/node": "^18.15.11",
-    "@types/prettier": "^2.7.2",
-    "eslint": "^8.37.0",
+    "@types/prettier": "^2.7.3",
+    "eslint": "^8.42.0",
     "node-fetch": "^3.3.1",
     "oclif": "^3",
     "tsup": "^6.7.0",
-    "typescript": "^5.0.3",
-    "vite": "^4.2.1",
-    "vitest": "^0.29.8"
+    "typescript": "^5.1.3",
+    "vite": "^4.3.9",
+    "vitest": "^0.32.0"
   },
   "prettier": "@shopify/prettier-config",
   "oclif": {