address comments

Author: Yuangwang

packages/@apphosting/adapter-nextjs/src/bin/build.spec.ts

@@ -20,7 +20,7 @@ describe("build commands", () => {
   });
 
   it("expects all output bundle files to be generated", async () => {
-    const { generateOutputBundle } = await importUtils;
+    const { generateOutputDirectory } = await importUtils;
     const files = {
       ".next/standalone/standalonefile": "",
       ".next/static/staticfile": "",
@@ -31,7 +31,7 @@ describe("build commands", () => {
       }`,
     };
     generateTestFiles(tmpDir, files);
-    await generateOutputBundle(tmpDir, outputBundleOptions, path.join(tmpDir, ".next"));
+    await generateOutputDirectory(tmpDir, outputBundleOptions, path.join(tmpDir, ".next"));
 
     const expectedFiles = {
       ".apphosting/.next/static/staticfile": "",
@@ -50,7 +50,7 @@ staticAssets:
   });
 
   it("expects public directory to be copied over", async () => {
-    const { generateOutputBundle } = await importUtils;
+    const { generateOutputDirectory } = await importUtils;
     const files = {
       ".next/standalone/standalonefile": "",
       ".next/static/staticfile": "",
@@ -62,7 +62,7 @@ staticAssets:
       }`,
     };
     generateTestFiles(tmpDir, files);
-    await generateOutputBundle(tmpDir, outputBundleOptions, path.join(tmpDir, ".next"));
+    await generateOutputDirectory(tmpDir, outputBundleOptions, path.join(tmpDir, ".next"));
 
     const expectedFiles = {
       ".apphosting/.next/static/staticfile": "",
@@ -82,7 +82,7 @@ staticAssets:
   });
 
   it("expects bundle.yaml headers/rewrites/redirects to be generated", async () => {
-    const { generateOutputBundle } = await importUtils;
+    const { generateOutputDirectory } = await importUtils;
     const files = {
       ".next/standalone/standalonefile": "",
       ".next/static/staticfile": "",
@@ -93,7 +93,7 @@ staticAssets:
       }`,
     };
     generateTestFiles(tmpDir, files);
-    await generateOutputBundle(tmpDir, outputBundleOptions, path.join(tmpDir, ".next"));
+    await generateOutputDirectory(tmpDir, outputBundleOptions, path.join(tmpDir, ".next"));
 
     const expectedFiles = {
       ".apphosting/.next/static/staticfile": "",

packages/@apphosting/adapter-nextjs/src/bin/build.ts

@@ -1,5 +1,10 @@
 #! /usr/bin/env node
-import { loadConfig, build, populateOutputBundleOptions, generateOutputBundle } from "../utils.js";
+import {
+  loadConfig,
+  build,
+  populateOutputBundleOptions,
+  generateOutputDirectory,
+} from "../utils.js";
 
 import { join } from "path";
 
@@ -11,4 +16,4 @@ const outputBundleOptions = populateOutputBundleOptions(cwd);
 const { distDir } = await loadConfig(cwd);
 const nextBuildDirectory = join(cwd, distDir);
 
-await generateOutputBundle(cwd, outputBundleOptions, nextBuildDirectory);
+await generateOutputDirectory(cwd, outputBundleOptions, nextBuildDirectory);

packages/@apphosting/adapter-nextjs/src/utils.ts

@@ -52,7 +52,7 @@ export function build(cwd: string): void {
 
 // move the standalone directory, the static directory and the public directory to apphosting output directory
 // as well as generating bundle.yaml
-export async function generateOutputBundle(
+export async function generateOutputDirectory(
   cwd: string,
   outputBundleOptions: OutputBundleOptions,
   nextBuildDirectory: string,