@shopify/cli-hydrogen 6.1.0 → 7.0.0

package/dist/commands/hydrogen/build.js

@@ -1,22 +1,22 @@
 import { Flags } from '@oclif/core';
 import Command from '@shopify/cli-kit/node/base-command';
-import { outputInfo, outputWarn, outputContent, outputToken } from '@shopify/cli-kit/node/output';
+import { outputInfo, outputContent, outputToken, outputWarn } from '@shopify/cli-kit/node/output';
 import { rmdir, fileSize, glob, readFile, writeFile, fileExists, copyFile } from '@shopify/cli-kit/node/fs';
-import { resolvePath, joinPath, relativePath } from '@shopify/cli-kit/node/path';
+import { resolvePath, relativePath, joinPath } from '@shopify/cli-kit/node/path';
 import { getPackageManager } from '@shopify/cli-kit/node/node-package-manager';
 import colors from '@shopify/cli-kit/node/colors';
 import { getProjectPaths, getRemixConfig, handleRemixImportFail, assertOxygenChecks } from '../../lib/remix-config.js';
-import { commonFlags, deprecated, flagsToCamelObject } from '../../lib/flags.js';
+import { commonFlags, flagsToCamelObject } from '../../lib/flags.js';
 import { checkLockfileStatus } from '../../lib/check-lockfile.js';
 import { findMissingRoutes } from '../../lib/missing-routes.js';
 import { muteRemixLogs, createRemixLogger } from '../../lib/log.js';
 import { codegen } from '../../lib/codegen.js';
-import { hasMetafile, buildBundleAnalysis, getBundleAnalysisSummary } from '../../lib/bundle/analyzer.js';
-import { AbortError } from '@shopify/cli-kit/node/error';
+import { buildBundleAnalysis, getBundleAnalysisSummary } from '../../lib/bundle/analyzer.js';
 import { isCI } from '../../lib/is-ci.js';
+import { prepareDiffDirectory, copyDiffBuild } from '../../lib/template-diff.js';
 
 const LOG_WORKER_BUILT = "\u{1F4E6} Worker built";
-const MAX_WORKER_BUNDLE_SIZE = 10;
+const WORKER_BUILD_SIZE_LIMIT = 5;
 class Build extends Command {
   static description = "Builds a Hydrogen storefront for production.";
   static flags = {
@@ -44,18 +44,24 @@ class Build extends Command {
     }),
     codegen: commonFlags.codegen,
     "codegen-config-path": commonFlags.codegenConfigPath,
-    base: deprecated("--base")(),
-    entry: deprecated("--entry")(),
-    target: deprecated("--target")()
+    diff: commonFlags.diff
   };
   async run() {
     const { flags } = await this.parse(Build);
-    const directory = flags.path ? resolvePath(flags.path) : process.cwd();
+    const originalDirectory = flags.path ? resolvePath(flags.path) : process.cwd();
+    let directory = originalDirectory;
+    if (flags.diff) {
+      directory = await prepareDiffDirectory(originalDirectory, false);
+    }
     await runBuild({
       ...flagsToCamelObject(flags),
       useCodegen: flags.codegen,
       directory
     });
+    if (flags.diff) {
+      await copyDiffBuild(directory, originalDirectory);
+    }
+    process.exit(0);
   }
 }
 async function runBuild({
@@ -104,28 +110,37 @@ async function runBuild({
       fileWatchCache: createFileWatchCache()
     }).catch((thrown) => {
       logThrown(thrown);
-      process.exit(1);
+      if (process.env.SHOPIFY_UNIT_TEST) {
+        throw thrown;
+      } else {
+        process.exit(1);
+      }
     }),
     useCodegen && codegen({ ...remixConfig, configFilePath: codegenConfigPath })
   ]);
   if (process.env.NODE_ENV !== "development") {
     console.timeEnd(LOG_WORKER_BUILT);
+    const bundleAnalysisPath = await buildBundleAnalysis(buildPath);
     const sizeMB = await fileSize(buildPathWorkerFile) / (1024 * 1024);
-    if (await hasMetafile(buildPath)) {
-      await writeBundleAnalysis(
-        buildPath,
-        root,
-        buildPathWorkerFile,
-        sizeMB,
-        bundleStats,
-        remixConfig
+    const formattedSize = colors.yellow(sizeMB.toFixed(2) + " MB");
+    outputInfo(
+      outputContent`   ${colors.dim(
+        relativePath(root, buildPathWorkerFile)
+      )}  ${bundleAnalysisPath ? outputToken.link(formattedSize, bundleAnalysisPath) : formattedSize}\n`
+    );
+    if (bundleStats && bundleAnalysisPath) {
+      outputInfo(
+        outputContent`${await getBundleAnalysisSummary(buildPathWorkerFile) || "\n"}\n    │\n    └─── ${outputToken.link(
+          "Complete analysis: " + bundleAnalysisPath,
+          bundleAnalysisPath
+        )}\n\n`
       );
-    } else {
-      await writeSimpleBuildStatus(
-        root,
-        buildPathWorkerFile,
-        sizeMB,
-        remixConfig
+    }
+    if (sizeMB >= WORKER_BUILD_SIZE_LIMIT) {
+      outputWarn(
+        `\u{1F6A8} Smaller worker bundles are faster to deploy and run.${remixConfig.serverMinify ? "" : "\n   Minify your bundle by adding `serverMinify: true` to remix.config.js."}
+   Learn more about optimizing your worker bundle file: https://h2o.fyi/debugging/bundle-size
+`
       );
     }
   }
@@ -145,9 +160,6 @@ This build is missing ${missingRoutes.length} route${missingRoutes.length > 1 ?
   if (process.env.NODE_ENV !== "development") {
     await cleanClientSourcemaps(buildPathClient);
   }
-  if (!process.env.SHOPIFY_UNIT_TEST && !assetPath) {
-    process.exit(0);
-  }
 }
 async function cleanClientSourcemaps(buildPathClient) {
   const bundleFiles = await glob(joinPath(buildPathClient, "**/*.js"));
@@ -161,56 +173,6 @@ async function cleanClientSourcemaps(buildPathClient) {
     })
   );
 }
-async function writeBundleAnalysis(buildPath, root, buildPathWorkerFile, sizeMB, bundleStats, remixConfig) {
-  const bundleAnalysisPath = await buildBundleAnalysis(buildPath);
-  outputInfo(
-    outputContent`   ${colors.dim(
-      relativePath(root, buildPathWorkerFile)
-    )}  ${outputToken.link(
-      colors.yellow(sizeMB.toFixed(2) + " MB"),
-      bundleAnalysisPath
-    )}\n`
-  );
-  if (bundleStats && sizeMB < MAX_WORKER_BUNDLE_SIZE) {
-    outputInfo(
-      outputContent`${await getBundleAnalysisSummary(buildPathWorkerFile) || "\n"}\n    │\n    └─── ${outputToken.link(
-        "Complete analysis: " + bundleAnalysisPath,
-        bundleAnalysisPath
-      )}\n\n`
-    );
-  }
-  if (sizeMB >= MAX_WORKER_BUNDLE_SIZE) {
-    throw new AbortError(
-      "\u{1F6A8} Worker bundle exceeds 10 MB! Oxygen has a maximum worker bundle size of 10 MB.",
-      outputContent`See the bundle analysis for a breakdown of what is contributing to the bundle size:\n${outputToken.link(
-        bundleAnalysisPath,
-        bundleAnalysisPath
-      )}`
-    );
-  } else if (sizeMB >= 5) {
-    outputWarn(
-      `\u{1F6A8} Worker bundle exceeds 5 MB! This can delay your worker response.${remixConfig.serverMinify ? "" : " Minify your bundle by adding `serverMinify: true` to remix.config.js."}
-`
-    );
-  }
-}
-async function writeSimpleBuildStatus(root, buildPathWorkerFile, sizeMB, remixConfig) {
-  outputInfo(
-    outputContent`   ${colors.dim(
-      relativePath(root, buildPathWorkerFile)
-    )}  ${colors.yellow(sizeMB.toFixed(2) + " MB")}\n`
-  );
-  if (sizeMB >= MAX_WORKER_BUNDLE_SIZE) {
-    throw new AbortError(
-      "\u{1F6A8} Worker bundle exceeds 10 MB! Oxygen has a maximum worker bundle size of 10 MB."
-    );
-  } else if (sizeMB >= 5) {
-    outputWarn(
-      `\u{1F6A8} Worker bundle exceeds 5 MB! This can delay your worker response.${remixConfig.serverMinify ? "" : " Minify your bundle by adding `serverMinify: true` to remix.config.js."}
-`
-    );
-  }
-}
 async function copyPublicFiles(publicPath, buildPathClient) {
   if (!await fileExists(publicPath)) {
     return;

package/dist/commands/hydrogen/codegen.js

@@ -1,6 +1,7 @@
 import path from 'path';
 import Command from '@shopify/cli-kit/node/base-command';
 import { renderSuccess } from '@shopify/cli-kit/node/ui';
+import colors from '@shopify/cli-kit/node/colors';
 import { Flags } from '@oclif/core';
 import { getProjectPaths, getRemixConfig } from '../../lib/remix-config.js';
 import { commonFlags, flagsToCamelObject } from '../../lib/flags.js';
@@ -24,8 +25,6 @@ class Codegen extends Command {
       default: false
     })
   };
-  static aliases = ["codegen-unstable"];
-  static deprecateAliases = true;
   async run() {
     const { flags } = await this.parse(Codegen);
     const directory = flags.path ? path.resolve(flags.path) : process.cwd();
@@ -53,7 +52,13 @@ async function runCodegen({
   if (!watch) {
     renderSuccess({
       headline: "Generated types for GraphQL:",
-      body: generatedFiles.map((file) => `- ${file}`).join("\n")
+      body: {
+        list: {
+          items: Object.entries(generatedFiles).map(
+            ([key, value]) => key + "\n" + value.map((item) => colors.dim(`- ${item}`)).join("\n")
+          )
+        }
+      }
     });
   }
 }

package/dist/commands/hydrogen/deploy.js

@@ -19,12 +19,17 @@ const deploymentLogger = (message, level = "info") => {
   }
 };
 class Deploy extends Command {
+  static description = "Builds and deploys a Hydrogen storefront to Oxygen.";
   static flags = {
     "env-branch": Flags.string({
-      char: "e",
-      description: "Environment branch (tag) for environment to deploy to",
+      description: "Environment branch (tag) for environment to deploy to.",
       required: false
     }),
+    preview: Flags.boolean({
+      description: "Deploys to the Preview environment. Overrides --env-branch and Git metadata.",
+      required: false,
+      default: false
+    }),
     force: Flags.boolean({
       char: "f",
       description: "Forces a deployment to proceed if there are uncommited changes in its Git repository.",
@@ -32,22 +37,21 @@ class Deploy extends Command {
       env: "SHOPIFY_HYDROGEN_FLAG_FORCE",
       required: false
     }),
-    path: commonFlags.path,
-    shop: commonFlags.shop,
-    "public-deployment": Flags.boolean({
-      env: "SHOPIFY_HYDROGEN_FLAG_PUBLIC_DEPLOYMENT",
-      description: "Marks a preview deployment as publicly accessible.",
+    "auth-bypass-token": Flags.boolean({
+      description: "Generate an authentication bypass token, which can be used to perform end-to-end tests against the deployment.",
       required: false,
       default: false
     }),
+    path: commonFlags.path,
+    shop: commonFlags.shop,
     "no-json-output": Flags.boolean({
-      description: "Prevents the command from creating a JSON file containing the deployment URL (in CI environments).",
+      description: "Prevents the command from creating a JSON file containing the deployment URL in CI environments.",
       required: false,
       default: false
     }),
     token: Flags.string({
       char: "t",
-      description: "Oxygen deployment token",
+      description: "Oxygen deployment token. Defaults to the linked storefront's token if available.",
       env: "SHOPIFY_HYDROGEN_DEPLOYMENT_TOKEN",
       required: false
     }),
@@ -57,9 +61,9 @@ class Deploy extends Command {
       env: "SHOPIFY_HYDROGEN_FLAG_METADATA_DESCRIPTION"
     }),
     "metadata-url": Flags.string({
-      description: "URL that links to the deployment. Will be saved and displayed in the Shopify admin",
       required: false,
-      env: "SHOPIFY_HYDROGEN_FLAG_METADATA_URL"
+      env: "SHOPIFY_HYDROGEN_FLAG_METADATA_URL",
+      hidden: true
     }),
     "metadata-user": Flags.string({
       description: "User that initiated the deployment. Will be saved and displayed in the Shopify admin",
@@ -67,12 +71,11 @@ class Deploy extends Command {
       env: "SHOPIFY_HYDROGEN_FLAG_METADATA_USER"
     }),
     "metadata-version": Flags.string({
-      description: "A version identifier for the deployment. Will be saved and displayed in the Shopify admin",
       required: false,
-      env: "SHOPIFY_HYDROGEN_FLAG_METADATA_VERSION"
+      env: "SHOPIFY_HYDROGEN_FLAG_METADATA_VERSION",
+      hidden: true
     })
   };
-  static hidden = true;
   async run() {
     const { flags } = await this.parse(Deploy);
     const deploymentOptions = this.flagsToOxygenDeploymentOptions(flags);
@@ -87,19 +90,39 @@ class Deploy extends Command {
     const camelFlags = flagsToCamelObject(flags);
     return {
       ...camelFlags,
+      defaultEnvironment: flags.preview,
       environmentTag: flags["env-branch"],
       path: flags.path ? resolvePath(flags.path) : process.cwd()
     };
   }
 }
+function createUnexpectedAbortError(message) {
+  return new AbortError(
+    message || "The deployment failed due to an unexpected error.",
+    "Retrying the deployement may succeed.",
+    [
+      [
+        "If the issue persits, please check the",
+        {
+          link: {
+            label: "Shopify status page",
+            url: "https://status.shopify.com/"
+          }
+        },
+        "for any known issues."
+      ]
+    ]
+  );
+}
 async function oxygenDeploy(options) {
   const {
+    authBypassToken: generateAuthBypassToken,
+    defaultEnvironment,
     environmentTag,
     force: forceOnUncommitedChanges,
     noJsonOutput,
     path,
     shop,
-    publicDeployment,
     metadataUrl,
     metadataUser,
     metadataVersion
@@ -168,12 +191,17 @@ async function oxygenDeploy(options) {
     ] : `Could not obtain an Oxygen deployment token, please try again or contact Shopify support.`;
     throw new AbortError(errMessage);
   }
-  if (!isCI && !environmentTag && deploymentData?.environments) {
+  if (!isCI && !defaultEnvironment && !environmentTag && deploymentData?.environments) {
     if (deploymentData.environments.length > 1) {
       const choices = [
-        ...deploymentData.environments.map(({ name, branch: branch2 }) => ({
+        ...deploymentData.environments.map(({ name, branch: branch2, type }) => ({
           label: name,
-          value: branch2
+          // The preview environment will never have an associated branch so
+          // we're using a custom string here to identify it later in our code.
+          // Using a period at the end of the value is an invalid branch name
+          // in Git so we can be sure that this won't conflict with a merchant's
+          // repository.
+          value: type === "PREVIEW" ? "shopify-preview-environment." : branch2
         }))
       ];
       deploymentEnvironmentTag = await renderSelectPrompt({
@@ -194,12 +222,21 @@ async function oxygenDeploy(options) {
       "Using a custom deployment service. Don't do this in production!"
     );
   }
+  let fallbackEnvironmentTag = branch;
+  let isPreview = false;
+  if (deploymentEnvironmentTag === "shopify-preview-environment.") {
+    fallbackEnvironmentTag = void 0;
+    deploymentEnvironmentTag = void 0;
+    isPreview = true;
+  }
   const config = {
     assetsDir: "dist/client",
     bugsnag: true,
     deploymentUrl,
+    defaultEnvironment: defaultEnvironment || isPreview,
     deploymentToken: parseToken(token),
-    environmentTag: environmentTag || deploymentEnvironmentTag || branch,
+    environmentTag: environmentTag || deploymentEnvironmentTag || fallbackEnvironmentTag,
+    generateAuthBypassToken,
     verificationMaxDuration: 180,
     metadata: {
       ...metadataDescription ? { description: metadataDescription } : {},
@@ -207,7 +244,6 @@ async function oxygenDeploy(options) {
       ...metadataUser ? { user: metadataUser } : {},
       ...metadataVersion ? { version: metadataVersion } : {}
     },
-    publicDeployment,
     skipVerification: false,
     rootPath: path,
     skipBuild: false,
@@ -218,10 +254,16 @@ async function oxygenDeploy(options) {
   const uploadPromise = new Promise((resolve) => {
     resolveUpload = resolve;
   });
-  let resolveHealthCheck;
-  const healthCheckPromise = new Promise((resolve) => {
-    resolveHealthCheck = resolve;
+  let resolveRoutableCheck;
+  const routableCheckPromise = new Promise((resolve) => {
+    resolveRoutableCheck = resolve;
   });
+  let resolveDeploymentCompletedVerification;
+  const deploymentCompletedVerificationPromise = new Promise(
+    (resolve) => {
+      resolveDeploymentCompletedVerification = resolve;
+    }
+  );
   let deployError = null;
   let resolveDeploy;
   let rejectDeploy;
@@ -237,11 +279,21 @@ async function oxygenDeploy(options) {
       await runBuild({
         directory: path,
         assetPath,
-        sourcemap: false,
+        sourcemap: true,
         useCodegen: false
       });
     },
-    onVerificationComplete: () => resolveHealthCheck(),
+    onDeploymentCompleted: () => resolveDeploymentCompletedVerification(),
+    onVerificationComplete: () => resolveRoutableCheck(),
+    onDeploymentCompletedVerificationError() {
+      deployError = new AbortError(
+        "Unable to verify the deployment was completed successfully",
+        "Please verify the deployment status in the Shopify Admin and retry deploying if necessary."
+      );
+    },
+    onDeploymentFailed: (details) => {
+      deployError = createUnexpectedAbortError(details.error || details.status);
+    },
     onUploadFilesStart: () => uploadStart(),
     onUploadFilesComplete: () => resolveUpload(),
     onVerificationError: (error) => {
@@ -267,23 +319,43 @@ async function oxygenDeploy(options) {
         task: async () => await uploadPromise
       },
       {
-        title: "Verifying deployment",
-        task: async () => await healthCheckPromise
+        title: "Verifying deployment has been completed",
+        task: async () => await deploymentCompletedVerificationPromise
+      },
+      {
+        title: "Verifying deployment is routable",
+        task: async () => await routableCheckPromise
       }
     ]);
   };
-  await createDeploy({ config, hooks, logger: deploymentLogger }).then(async (url) => {
-    const deploymentType = config.publicDeployment ? "public" : "private";
+  await createDeploy({ config, hooks, logger: deploymentLogger }).then(async (completedDeployment) => {
+    if (!completedDeployment) {
+      rejectDeploy(createUnexpectedAbortError());
+      return;
+    }
+    const nextSteps = [
+      [
+        "Open",
+        { link: { url: completedDeployment.url } },
+        `in your browser to view your deployment.`
+      ]
+    ];
+    if (completedDeployment?.authBypassToken) {
+      nextSteps.push([
+        "Use the",
+        { subdued: completedDeployment.authBypassToken },
+        "token to perform end-to-end tests against the deployment."
+      ]);
+    }
     renderSuccess({
       body: ["Successfully deployed to Oxygen"],
-      nextSteps: [
-        [
-          `Open ${url} in your browser to view your ${deploymentType} deployment`
-        ]
-      ]
+      nextSteps
     });
     if (isCI && !noJsonOutput) {
-      await writeFile("h2_deploy_log.json", JSON.stringify({ url }));
+      await writeFile(
+        "h2_deploy_log.json",
+        JSON.stringify(completedDeployment)
+      );
     }
     resolveDeploy();
   }).catch((error) => {

package/dist/commands/hydrogen/deploy.test.js

@@ -66,11 +66,12 @@ describe("deploy", () => {
   };
   const originalExit = process.exit;
   const deployParams = {
+    authBypassToken: true,
+    defaultEnvironment: false,
     force: false,
     noJsonOutput: false,
     path: "./",
     shop: "snowdevil.myshopify.com",
-    publicDeployment: false,
     metadataUrl: "https://example.com",
     metadataUser: "user",
     metadataVersion: "1.0.0"
@@ -87,15 +88,16 @@ describe("deploy", () => {
   const expectedConfig = {
     assetsDir: "dist/client",
     bugsnag: true,
+    defaultEnvironment: false,
     deploymentUrl: "https://oxygen.shopifyapps.com",
     deploymentToken: mockToken,
+    generateAuthBypassToken: true,
     verificationMaxDuration: 180,
     metadata: {
       url: deployParams.metadataUrl,
       user: deployParams.metadataUser,
       version: deployParams.metadataVersion
     },
-    publicDeployment: deployParams.publicDeployment,
     skipVerification: false,
     rootPath: deployParams.path,
     skipBuild: false,
@@ -104,10 +106,13 @@ describe("deploy", () => {
   };
   const expectedHooks = {
     buildFunction: expect.any(Function),
+    onDeploymentCompleted: expect.any(Function),
+    onDeploymentFailed: expect.any(Function),
+    onDeploymentCompletedVerificationError: expect.any(Function),
     onVerificationComplete: expect.any(Function),
+    onVerificationError: expect.any(Function),
     onUploadFilesStart: expect.any(Function),
     onUploadFilesComplete: expect.any(Function),
-    onVerificationError: expect.any(Function),
     onUploadFilesError: expect.any(Function)
   };
   beforeEach(async () => {
@@ -125,9 +130,10 @@ describe("deploy", () => {
       }
     ]);
     vi.mocked(renderSelectPrompt).mockResolvedValue(FULL_SHOPIFY_CONFIG.shop);
-    vi.mocked(createDeploy).mockResolvedValue(
-      "https://a-lovely-deployment.com"
-    );
+    vi.mocked(createDeploy).mockResolvedValue({
+      authBypassToken: "some-token",
+      url: "https://a-lovely-deployment.com"
+    });
     vi.mocked(getOxygenDeploymentData).mockResolvedValue({
       oxygenDeploymentToken: "some-encoded-token",
       environments: []
@@ -252,19 +258,52 @@ describe("deploy", () => {
     vi.mocked(getOxygenDeploymentData).mockResolvedValue({
       oxygenDeploymentToken: "some-encoded-token",
       environments: [
-        { name: "production", branch: "main" },
-        { name: "preview", branch: "staging" }
+        { name: "Production", branch: "main", type: "PRODUCTION" },
+        { name: "Preview", branch: null, type: "PREVIEW" }
       ]
     });
     await oxygenDeploy(deployParams);
     expect(vi.mocked(renderSelectPrompt)).toHaveBeenCalledWith({
       message: "Select an environment to deploy to",
       choices: [
-        { label: "production", value: "main" },
-        { label: "preview", value: "staging" }
+        { label: "Production", value: "main" },
+        { label: "Preview", value: "shopify-preview-environment." }
       ]
     });
   });
+  describe("when Preview is selected", () => {
+    it("calls createDeploy with defaultEnvironment and an undefined environmentTag", async () => {
+      vi.mocked(getLatestGitCommit).mockResolvedValue({
+        hash: "123",
+        message: "test commit",
+        date: "2021-01-01",
+        author_name: "test author",
+        author_email: "test@author.com",
+        body: "test body",
+        refs: "HEAD -> main"
+      });
+      vi.mocked(getOxygenDeploymentData).mockResolvedValue({
+        oxygenDeploymentToken: "some-encoded-token",
+        environments: [
+          { name: "Production", branch: "main", type: "PRODUCTION" },
+          { name: "Preview", branch: null, type: "PREVIEW" }
+        ]
+      });
+      vi.mocked(renderSelectPrompt).mockResolvedValue(
+        "shopify-preview-environment."
+      );
+      await oxygenDeploy(deployParams);
+      expect(vi.mocked(createDeploy)).toHaveBeenCalledWith({
+        config: {
+          ...expectedConfig,
+          defaultEnvironment: true,
+          environmentTag: void 0
+        },
+        hooks: expectedHooks,
+        logger: deploymentLogger
+      });
+    });
+  });
   it("writes a file with JSON content in CI environments", async () => {
     vi.mocked(ciPlatform).mockReturnValue({
       isCI: true,
@@ -274,12 +313,16 @@ describe("deploy", () => {
     const ciDeployParams = {
       ...deployParams,
       token: "some-token",
-      metadataDescription: "cool new stuff"
+      metadataDescription: "cool new stuff",
+      generateAuthBypassToken: true
     };
     await oxygenDeploy(ciDeployParams);
     expect(vi.mocked(writeFile)).toHaveBeenCalledWith(
       "h2_deploy_log.json",
-      JSON.stringify({ url: "https://a-lovely-deployment.com" })
+      JSON.stringify({
+        authBypassToken: "some-token",
+        url: "https://a-lovely-deployment.com"
+      })
     );
     vi.mocked(writeFile).mockClear();
     ciDeployParams.noJsonOutput = true;
@@ -311,7 +354,7 @@ describe("deploy", () => {
       }
     }
   });
-  it("handles error during deployment verification", async () => {
+  it("handles error during deployment routability verification", async () => {
     const mockRenderFatalError = vi.fn();
     vi.mocked(renderFatalError).mockImplementation(mockRenderFatalError);
     const error = new Error("Cloudflare is down!");
@@ -337,4 +380,31 @@ describe("deploy", () => {
       }
     }
   });
+  it("handles error during deployment completion verification", async () => {
+    const mockRenderFatalError = vi.fn();
+    vi.mocked(renderFatalError).mockImplementation(mockRenderFatalError);
+    vi.mocked(createDeploy).mockImplementation((options) => {
+      options.hooks?.onUploadFilesStart?.();
+      options.hooks?.onUploadFilesComplete?.();
+      options.hooks?.onDeploymentCompletedVerificationStart?.();
+      options.hooks?.onDeploymentFailed?.({
+        status: "oh shit",
+        url: "https://a-lovely-deployment.com"
+      });
+      return new Promise((_resolve, reject) => {
+        reject();
+      });
+    });
+    try {
+      await oxygenDeploy(deployParams);
+      expect(true).toBe(false);
+    } catch (err) {
+      if (err instanceof AbortError) {
+        expect(err.message).toBe("oh shit");
+        expect(err.tryMessage).toBe("Retrying the deployement may succeed.");
+      } else {
+        expect(true).toBe(false);
+      }
+    }
+  });
 });