fix issue where tool does not work with firebase-tools v15

Author: amalej

CHANGELOG.md

@@ -1,5 +1,10 @@
 <!-- ADD CHANGES HERE -->
 
+Unreleased
+
+- Fixed issue where Onfire CLI cannot parse commands on firebase-tools v15
+  - This is due to the lazy loading feature preventing all commands from being loaded
+
 v1.3.2
 
 - Added feature to list folders/files in current path to make it easier to use

src/firebase-cmd.ts

@@ -7,6 +7,7 @@ import {
   spawnSync,
 } from "child_process";
 import spawn from "cross-spawn";
+import { writeFileSync } from "fs";
 
 const COMMAND_TIMEOUT = 10000;
 
@@ -251,12 +252,15 @@ export class FirebaseCommands {
             });
 
             child.on("exit", () => {
-              if (childBufferString === "ENOENT") {
-                throw new Error(
-                  `Firebase Tools module not found. Install using 'npm install -g firebase-tools'`
+              this.loadModuleChildProcess?.kill();
+              this.loadModuleChildProcess = null;
+              if (childBufferString.includes("ERROR__")) {
+                const message = childBufferString.replace(
+                  "ERROR__",
+                  "Error loading module: "
                 );
+                throw new Error(message);
               }
-              this.loadModuleChildProcess = null;
               try {
                 res(JSON.parse(childBufferString));
               } catch (_) {
@@ -266,7 +270,6 @@ export class FirebaseCommands {
             });
 
             child.send(`${rootPath}/firebase-tools`);
-            // this.loadModuleChildProcess.kill();
           });
 
           // await new Promise((res, rej) => setTimeout(res, 5000));

src/module-loader.js

@@ -15,12 +15,54 @@ function stringify(obj) {
     return str;
 }
 
+function determineCommandList(map, parentKey = null, depth = 0) {
+    if (depth > 10) return []
+    const commandList = []
+    if (map) {
+        for (let [key, value] of Object.entries(map)) {
+            if (key === "load") continue
+            if (typeof value?.load === "function") {
+                if (!parentKey) {
+                    commandList.push(key)
+                } else {
+                    commandList.push([parentKey, key].join(":"))
+                }
+            }
+
+            if (!parentKey) {
+                commandList.push(...determineCommandList(value, key, depth + 1))
+            } else {
+                commandList.push(...determineCommandList(value, [parentKey, key].join(":"), depth + 1))
+            }
+        }
+    }
+
+    return commandList.map((e) => e.toLowerCase())
+}
+
 process.on('message', function (path) {
     try {
         const client = require(path); // Load the Firebase Tools module
-        const configStr = stringify(client) // Removes circular reference and convert to string
+        let configStr = stringify(client) // Removes circular reference and convert to string
+
+        const clientObj = JSON.parse(configStr)
+        if (clientObj.cli.commands.length === 0) {
+            /**
+             *  Due to firebase-tools v15 introducing lazy laoding of commands
+             *  we would need to run `getCommand([command_name])` for each command
+             *  to register each command
+             */
+            const commands = determineCommandList(client)
+            for (let command of commands) {
+                require(path).getCommand(command)
+            }
+
+            const clientWithLoadedCommands = require(path);
+            configStr = stringify(clientWithLoadedCommands)
+        }
+
         process.send(configStr); // Send to main process
-    } catch (_) {
-        process.send('ENOENT'); // Send to main process
+    } catch (err) {
+        process.send(`ERROR__${err}`); // Send to main process
     }
 });

src/onfire-cli.ts

@@ -1034,23 +1034,25 @@ export class OnFireCLI extends CommandLineInterface {
       } else {
         this.firebaseCommands = cmdConfig;
       }
+      this.attachCustomCommands();
+      this.savedConfig["firebaseCommands"] = this.firebaseCommands;
     } else {
       this.firebaseCommands = this.savedConfig["firebaseCommands"];
       this.firebaseCli
         .getCommandConfig()
         .then((firebaseCommands) => {
           if (firebaseCommands !== null) {
             this.firebaseCommands = firebaseCommands;
-            this.savedConfig["firebaseCommands"] = this.firebaseCommands;
             this.attachCustomCommands();
+            this.savedConfig["firebaseCommands"] = this.firebaseCommands;
+          } else {
+            throw new Error("Could not load firebase-tools commands");
           }
         })
-        .catch((_) => {
-          // No action needed since we can use the cached firebaseCommands
+        .catch((err) => {
+          throw err;
         });
     }
-    this.attachCustomCommands();
-    this.savedConfig["firebaseCommands"] = this.firebaseCommands;
   }
 
   async init() {