fix bug and fix unit tests

Author: mathu97

packages/@apphosting/adapter-nextjs/src/overrides.spec.ts

@@ -130,7 +130,7 @@ describe("route overrides", () => {
       fs.readFileSync(routesManifestPath, "utf-8"),
     ) as RoutesManifest;
 
-    assert.strictEqual(updatedManifest.headers.length, 1);
+    assert.strictEqual(updatedManifest.headers.length, 2);
 
     const expectedManifest: RoutesManifest = {
       version: 3,

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

@@ -186,23 +186,22 @@ export async function addRouteOverrides(
   // Add the middleware header to all routes for which middleware is enabled
   const middlewareManifest = loadMiddlewareManifest(appPath, distDir);
   const rootMiddleware = middlewareManifest.middleware["/"];
-  if (!rootMiddleware?.matchers) {
-    console.log("No middleware found for root path, skipping route overrides");
-    return;
-  }
-
-  rootMiddleware.matchers.forEach((matcher) => {
-    routeManifest.headers.push({
-      source: matcher.regexp,
-      headers: [
-        {
-          key: "x-fah-middleware",
-          value: "true",
-        },
-      ],
-      regex: matcher.regexp,
+  if (rootMiddleware?.matchers) {
+    console.log("Middleware detected, adding middleware headers to matching routes");
+
+    rootMiddleware.matchers.forEach((matcher) => {
+      routeManifest.headers.push({
+        source: matcher.regexp,
+        headers: [
+          {
+            key: "x-fah-middleware",
+            value: "true",
+          },
+        ],
+        regex: matcher.regexp,
+      });
     });
-  });
+  }
 
   await writeRouteManifest(appPath, distDir, routeManifest);
 }