Better error when incorrect project ID is used.

CHANGELOG.md

@@ -1 +1,2 @@
 - Fixed Flutter web apps that might require the --no-tree-shake-icons flag in order to build. (#7724)
+- Fixed an issue where the Extensions emulator would fail silently if started with a non-existant project without the `demo-` prefix. (#7779)

src/emulator/controller.ts

@@ -69,20 +69,20 @@
 
 /**
  * Exports emulator data on clean exit (SIGINT or process end)
  * @param options
  */
 export async function exportOnExit(options: any) {
   const exportOnExitDir = options.exportOnExit;
   if (exportOnExitDir) {
     try {
       utils.logBullet(
         `Automatically exporting data using ${FLAG_EXPORT_ON_EXIT_NAME} "${exportOnExitDir}" ` +
           "please wait for the export to finish...",
       );
       await exportEmulatorData(exportOnExitDir, options, /* initiatedBy= */ "exit");
     } catch (e: any) {
       utils.logWarning(e);
       utils.logWarning(`Automatic export to "${exportOnExitDir}" failed, going to exit now...`);
     }
   }
 }
@@ -352,9 +352,21 @@
   // the Functions emulator needs to start.
   let extensionEmulator: ExtensionsEmulator | undefined = undefined;
   if (shouldStart(options, Emulators.EXTENSIONS)) {
-    const projectNumber = isDemoProject
-      ? Constants.FAKE_PROJECT_NUMBER
-      : await needProjectNumber(options);
+    let projectNumber = Constants.FAKE_PROJECT_NUMBER;
+    if (!isDemoProject) {
+      try {
+        projectNumber = await needProjectNumber(options);
+      } catch (err: any) {
+        EmulatorLogger.forEmulator(Emulators.EXTENSIONS).logLabeled(
+          "ERROR",
+          Emulators.EXTENSIONS,
+          `Unable to look up project number for ${options.project}.\n` +
+            " If this is a real project, ensure that you are logged in and have access to it.\n" +
+            " If this is a fake project, please use a project ID starting with 'demo-' to skip production calls.\n" +
+            " Continuing with a fake project number - secrets and other features that require production access may behave unexpectedly.",
+        );
+      }
+    }
     const aliases = getAliases(options, projectId);
     extensionEmulator = new ExtensionsEmulator({
       options,

src/extensions/extensionsHelper.ts

@@ -36,7 +36,7 @@ import * as refs from "./refs";
 import { EXTENSIONS_SPEC_FILE, readFile, getLocalExtensionSpec } from "./localHelper";
 import { confirm, promptOnce } from "../prompt";
 import { logger } from "../logger";
-import { envOverride } from "../utils";
+import { envOverride, logLabeledError } from "../utils";
 import { getLocalChangelog } from "./change-log";
 import { getProjectNumber } from "../getProjectNumber";
 import { Constants } from "../emulator/constants";
@@ -122,10 +122,21 @@ export async function getFirebaseProjectParams(
   const body = emulatorMode
     ? await getProjectAdminSdkConfigOrCached(projectId)
     : await getFirebaseConfig({ project: projectId });
-  const projectNumber =
-    emulatorMode && Constants.isDemoProject(projectId)
-      ? Constants.FAKE_PROJECT_NUMBER
-      : await getProjectNumber({ projectId });
+
+  let projectNumber = Constants.FAKE_PROJECT_NUMBER;
+  if (!Constants.isDemoProject(projectId)) {
+    try {
+      projectNumber = await getProjectNumber({ projectId });
+    } catch (err: any) {
+      logLabeledError(
+        "extensions",
+        `Unable to look up project number for ${projectId}.\n` +
+          " If this is a real project, ensure that you are logged in and have access to it.\n" +
+          " If this is a fake project, please use a project ID starting with 'demo-' to skip production calls.\n" +
+          " Continuing with a fake project number - secrets and other features that require production access may behave unexpectedly.",
+      );
+    }
+  }
   const databaseURL = body?.databaseURL ?? `https://${projectId}.firebaseio.com`;
   const storageBucket = body?.storageBucket ?? `${projectId}.appspot.com`;
   // This env variable is needed for parameter-less initialization of firebase-admin