BREAKING: Ignore 'functions' directory in a Firebase project without …

…a functions config in firebase.json Don't assume all directory named `functions` are functions directory. Instead, expect developers to explicitly set up a "functions" config in their firebase.json.
firebase · Nov 28, 2023 · db631ad · db631ad
1 parent dbedf74
commit db631ad
Showing 1 changed file with 11 additions and 9 deletions.
diff --git a/src/config.ts b/src/config.ts
@@ -76,15 +76,17 @@ export class Config {
       }
     });
 
-    // Inject default functions config and source if missing.
-    if (this.projectDir && fsutils.dirExistsSync(this.path(Config.DEFAULT_FUNCTIONS_SOURCE))) {
-      if (Array.isArray(this.get("functions"))) {
-        if (!this.get("functions.[0].source")) {
-          this.set("functions.[0].source", Config.DEFAULT_FUNCTIONS_SOURCE);
-        }
-      } else {
-        if (!this.get("functions.source")) {
-          this.set("functions.source", Config.DEFAULT_FUNCTIONS_SOURCE);
+    // Inject default functions source if missing.
+    if (this.get("functions")) {
+      if (this.projectDir && fsutils.dirExistsSync(this.path(Config.DEFAULT_FUNCTIONS_SOURCE))) {
+        if (Array.isArray(this.get("functions"))) {
+          if (!this.get("functions.[0].source")) {
+            this.set("functions.[0].source", Config.DEFAULT_FUNCTIONS_SOURCE);
+          }
+        } else {
+          if (!this.get("functions.source")) {
+            this.set("functions.source", Config.DEFAULT_FUNCTIONS_SOURCE);
+          }
         }
       }
     }