Filter out node_modules (#346)

* Filter out node_modules

* lint and update test

* add comment

Author: Yuangwang

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@apphosting/adapter-nextjs",
-  "version": "14.0.14",
+  "version": "14.0.15",
   "main": "dist/index.js",
   "description": "Experimental addon to the Firebase CLI to add web framework support",
   "repository": {

packages/@apphosting/adapter-nextjs/package.json

@@ -62,6 +62,10 @@ metadata:
   adapterVersion: ${adapterMetadata.adapterVersion}
   framework: nextjs
   frameworkVersion: ${defaultNextVersion}
+outputFiles:
+  serverApp:
+    include:
+      - .next/standalone
 `,
     };
     validateTestFiles(tmpDir, expectedFiles);

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

@@ -151,7 +151,9 @@ async function moveResources(
   for (const path of pathsToMove) {
     const isbundleYamlDir = join(appDir, path) === dirname(bundleYamlPath);
     const existsInOutputBundle = await exists(join(outputBundleAppDir, path));
-    if (!isbundleYamlDir && !existsInOutputBundle) {
+    // Keep apphosting.yaml files in the root directory still, as later steps expect them to be there
+    const isApphostingYaml = path === "apphosting_preprocessed" || path === "apphosting.yaml";
+    if (!isbundleYamlDir && !existsInOutputBundle && !isApphostingYaml) {
       await move(join(appDir, path), join(outputBundleAppDir, path));
     }
   }
@@ -190,6 +192,11 @@ async function generateBundleYaml(
       framework: "nextjs",
       frameworkVersion: nextVersion,
     },
+    outputFiles: {
+      serverApp: {
+        include: [".next/standalone"], // we're only including the standalone directory to exclude the node_modules folder specifically
+      },
+    },
   };
   await writeFile(opts.bundleYamlPath, yamlStringify(outputBundle));
   return;