feat: remove obsolete parseTriggers fallback

Author: taeold

firebase-vscode/webpack.common.js

@@ -196,15 +196,8 @@ const extensionConfig = {
           to: "./schema",
         },
         // TODO(hlshen): Sanity check if these should be fixed or removed. AFIACT, they exist for functions and hosting deploys, which are not relevant anymore.
-        // Copy uncompiled JS files called at runtime by
-        // firebase-tools/src/parseTriggers.ts
-        // {
-        //   from: "*.js",
-        //   to: "./",
-        //   context: "../src/deploy/functions/runtimes/node",
-        // },
-        // // Copy cross-env-shell.js used to run predeploy scripts
-        // // to ensure they work in Windows
+        // Copy cross-env-shell.js used to run predeploy scripts
+        // to ensure they work in Windows
         // {
         //   from: "../node_modules/cross-env/dist",
         //   to: "./cross-env/dist",

src/deploy/functions/runtimes/node/extractTriggers.js

@@ -77,4 +77,33 @@ describe("NodeDelegate", () => {
       expect(() => delegate.getNodeBinary()).to.throw(FirebaseError);
     });
   });
+
+  describe("discoverBuild", () => {
+    let getFunctionsSDKVersionMock: sinon.SinonStub;
+
+    beforeEach(() => {
+      getFunctionsSDKVersionMock = sinon.stub(versioning, "getFunctionsSDKVersion");
+    });
+
+    afterEach(() => {
+      getFunctionsSDKVersionMock.restore();
+    });
+
+    it("throws error if SDK version is too old", async () => {
+      getFunctionsSDKVersionMock.returns("3.19.0");
+      const delegate = new node.Delegate(
+        PROJECT_ID,
+        PROJECT_DIR,
+        SOURCE_DIR,
+        "nodejs16" as Runtime,
+      );
+      try {
+        await delegate.discoverBuild({}, {});
+        throw new Error("Should have thrown");
+      } catch (err: any) {
+        expect(err).to.be.instanceOf(FirebaseError);
+        expect(err.message).to.include("Please update firebase-functions SDK");
+      }
+    });
+  });
 });

src/deploy/functions/runtimes/node/extractTriggers.spec.js

@@ -23,7 +23,7 @@ import { DelegateContext } from "..";
 import * as supported from "../supported";
 import * as validate from "./validate";
 import * as versioning from "./versioning";
-import * as parseTriggers from "./parseTriggers";
+
 import { fileExistsSync } from "../../../../fsutils";
 
 // The versions of the Firebase Functions SDK that added support for the container contract.
@@ -292,18 +292,12 @@ export class Delegate {
     env: backend.EnvironmentVariables,
   ): Promise<build.Build> {
     if (!semver.valid(this.sdkVersion)) {
-      logger.debug(
-        `Could not parse firebase-functions version '${this.sdkVersion}' into semver. Falling back to parseTriggers.`,
-      );
-      return parseTriggers.discoverBuild(this.projectId, this.sourceDir, this.runtime, config, env);
-    }
-    if (semver.lt(this.sdkVersion, MIN_FUNCTIONS_SDK_VERSION)) {
-      logLabeledWarning(
-        "functions",
+      logger.debug(`Could not parse firebase-functions version '${this.sdkVersion}' into semver.`);
+    } else if (semver.lt(this.sdkVersion, MIN_FUNCTIONS_SDK_VERSION)) {
+      throw new FirebaseError(
         `You are using an old version of firebase-functions SDK (${this.sdkVersion}). ` +
           `Please update firebase-functions SDK to >=${MIN_FUNCTIONS_SDK_VERSION}`,
       );
-      return parseTriggers.discoverBuild(this.projectId, this.sourceDir, this.runtime, config, env);
     }
     // Perform a check for the minimum SDK version that added annotation support for the `Build.extensions` property
     // and log to the user explaining why they need to upgrade their version.