fix: move the invalidFunctions check before the no-functions one (#194)

.changeset/modern-crews-tease.md

@@ -0,0 +1,12 @@
+---
+'@cloudflare/next-on-pages': patch
+---
+
+move the invalidFunctions check before the no-functions one
+
+currently if a build contains only invalid functions we'd be presenting a log
+saying that no function was found and simply return the static assets as if
+everything is correct, this is because we check for invalid functions only
+after checking if there are any (valid) ones, this change moves the invalid
+functions check so that is performed first, making sure that the described case
+successfully errors

src/buildApplication/buildApplication.ts

@@ -79,6 +79,11 @@ async function prepareAndBuildWorker(
 		options.experimentalMinify
 	);
 
+	if (invalidFunctions.size > 0) {
+		printInvalidFunctionsErrorMessage(Array.from(invalidFunctions));
+		exit(1);
+	}
+
 	if (functionsMap.size === 0) {
 		cliLog('No functions detected.');
 		return;
@@ -99,11 +104,6 @@ async function prepareAndBuildWorker(
 		exit(1);
 	}
 
-	if (invalidFunctions.size > 0) {
-		printInvalidFunctionsErrorMessage(Array.from(invalidFunctions));
-		exit(1);
-	}
-
 	const staticAssets = await getVercelStaticAssets();
 
 	const processedVercelOutput = processVercelOutput(