Fixed an issue where empty firebase.json files would break init

src/commands/firestore-operations-describe.spec.ts

@@ -86,9 +86,9 @@ describe("firestore:operations:describe", () => {
 
   it("should throw a FirebaseError if operation name is invalid", async () => {
     const options = { project: "test-project" };
-    await expect(command.runner()("", options)).to.be.rejectedWith(
+    await expect(command.runner()("/databases/blah", options)).to.be.rejectedWith(
       FirebaseError,
-      '"" is not a valid operation name.',
+      '"/databases/blah" is not a valid operation name.',
     );
     await expect(command.runner()("projects/p/databases/d", options)).to.be.rejectedWith(
       FirebaseError,

src/config.spec.ts

@@ -7,6 +7,7 @@
 import { FIREBASE_JSON_PATH as VALID_CONFIG_PATH } from "./test/fixtures/valid-config";
 import { FIXTURE_DIR as SIMPLE_CONFIG_DIR } from "./test/fixtures/config-imports";
 import { FIXTURE_DIR as DUP_TOP_LEVEL_CONFIG_DIR } from "./test/fixtures/dup-top-level";
+import { FIREBASE_JSON_PATH as EMPTY_CONFIG_PATH } from "./test/fixtures/empty-config";
 
 describe("Config", () => {
   describe("#load", () => {
@@ -17,9 +18,16 @@
         configPath: path.relative(cwd, VALID_CONFIG_PATH),
       });
       expect(config).to.not.be.null;
-      if (config) {
-        expect(config.get("database.rules")).to.eq("config/security-rules.json");
-      }
+      expect(config?.get("database.rules")).to.eq("config/security-rules.json");
+    });
+    it("should fall back to {} when the file is empty", () => {
+      const cwd = __dirname;
+      const config = Config.load({
+        cwd,
+        configPath: path.relative(cwd, EMPTY_CONFIG_PATH),
+      });
+      expect(config).to.not.be.null;
+      expect(config?.data).to.deep.eq({});
     });
   });
 
@@ -39,7 +47,7 @@
   describe("#parseFile", () => {
     it("should load a cjson file", () => {
       const config = new Config({}, { cwd: SIMPLE_CONFIG_DIR });
       expect(config.parseFile("hosting", "hosting.json").public).to.equal(".");
     });
 
     it("should error out for an unknown file", () => {
@@ -60,7 +68,7 @@
   describe("#materialize", () => {
     it("should assign unaltered if an object is found", () => {
       const config = new Config({ example: { foo: "bar" } }, {});
       expect(config.materialize("example").foo).to.equal("bar");
     });
 
     it("should prevent top-level key duplication", () => {

src/config.ts

@@ -6,7 +6,6 @@
 import * as clc from "colorette";
 import * as fs from "fs-extra";
 import * as path from "path";
-const cjson = require("cjson");
 
 import { detectProjectRoot } from "./detectProjectRoot";
 import { FirebaseError } from "./error";
@@ -17,7 +16,7 @@
 import { getValidator, getErrorMessage } from "./firebaseConfigValidate";
 import { logger } from "./logger";
 import { loadCJSON } from "./loadCJSON";
 const parseBoltRules = require("./parseBoltRules");
 
 export class Config {
   static DEFAULT_FUNCTIONS_SOURCE = "functions";
@@ -36,19 +35,19 @@
     "apphosting",
   ];
 
   public options: any;
   public projectDir: string;
   public data: any = {};
   public defaults: any = {};
   public notes: any = {};
 
   private _src: any;
 
   /**
    * @param src incoming firebase.json source, parsed by not validated.
    * @param options command-line options.
    */
   constructor(src: any, options: any = {}) {
     this.options = options;
     this.projectDir = this.options.projectDir || detectProjectRoot(this.options);
     this._src = src;
@@ -303,7 +302,10 @@
     if (pd) {
       try {
         const filePath = path.resolve(pd, path.basename(filename));
-        const data = cjson.load(filePath);
+        let data: unknown = {};
+        if (fs.statSync(filePath).size > 0) {
+          data = loadCJSON(filePath);
+        }
 
         // Validate config against JSON Schema. For now we just print these to debug
         // logs but in a future CLI version they could be warnings and/or errors.

src/test/fixtures/empty-config/index.ts

@@ -0,0 +1,8 @@
+import { resolve } from "path";
+
+/**
+ * A directory containing an empty rebase.json.
+ */
+export const FIXTURE_DIR = __dirname;
+
+export const FIREBASE_JSON_PATH = resolve(__dirname, "firebase.json");