Project ID selection

src/firebase_studio/migrate.spec.ts

@@ -22,12 +22,16 @@
   });
 
   describe("extractMetadata", () => {
+    let readFileStub: sinon.SinonStub;
     beforeEach(() => {
-      sandbox.stub(fs, "readFile").callsFake(async (p: any) => {
+      readFileStub = sandbox.stub(fs, "readFile").callsFake(async (p: any) => {
         const pStr = p.toString();
-        if (pStr.endsWith("metadata.json")) {
+        if (pStr.endsWith("metadata.json") || pStr.endsWith("studio.json")) {
           return JSON.stringify({ projectId: "original-project" });
         }
+        if (pStr.endsWith(".firebaserc")) {
+          throw Object.assign(new Error("File not found"), { code: "ENOENT" });
+        }
         if (pStr.endsWith("blueprint.md")) {
           return "# **App Name**: Test App";
         }
@@ -40,7 +44,25 @@
       expect(result.projectId).to.equal("override-project");
     });
 
-    it("should fallback to metadata.json if no override is provided", async () => {
+    it("should use firebaserc project if override is not provided", async () => {
+      readFileStub.callsFake(async (p: any) => {
+        const pStr = p.toString();
+        if (pStr.endsWith("metadata.json") || pStr.endsWith("studio.json")) {
+          return JSON.stringify({ projectId: "original-project" });
+        }
+        if (pStr.endsWith(".firebaserc")) {
+          return JSON.stringify({ projects: { default: "firebaserc-project" } });
+        }
+        if (pStr.endsWith("blueprint.md")) {
+          return "# **App Name**: Test App";
+        }
+        return "";
+      });
+      const result = await extractMetadata(testRoot);
+      expect(result.projectId).to.equal("firebaserc-project");
+    });
+
+    it("should fallback to metadata.json/studio.json if no override or firebaserc is provided", async () => {
       const result = await extractMetadata(testRoot);
       expect(result.projectId).to.equal("original-project");
     });

src/firebase_studio/migrate.ts

@@ -179,16 +179,29 @@ export async function extractMetadata(
 
   logger.debug(`overrideProjectId ${overrideProjectId}`);
   logger.debug(`metadata.projectId ${metadata.projectId}`);
-  let projectId = overrideProjectId || metadata.projectId;
-  if (!projectId) {
+  let projectId: string | undefined;
+  let projectIdSource = "";
+
+  if (overrideProjectId) {
+    projectId = overrideProjectId;
+    projectIdSource = "--project flag";
+  } else {
     // try to get project ID from .firebaserc
     try {
       const firebasercContent = await fs.readFile(path.join(rootPath, ".firebaserc"), "utf8");
       const firebaserc = JSON.parse(firebasercContent) as { projects?: { default?: string } };
-      projectId = firebaserc.projects?.default;
+      if (firebaserc.projects?.default) {
+        projectId = firebaserc.projects.default;
+        projectIdSource = ".firebaserc";
+      }
     } catch (err: unknown) {
       logger.debug(`Could not read .firebaserc at ${rootPath}: ${err}`);
     }
+
+    if (!projectId && metadata.projectId) {
+      projectId = metadata.projectId;
+      projectIdSource = "studio.json";
+    }
   }
 
   if (projectId) {
@@ -197,7 +210,7 @@ export async function extractMetadata(
         exit: 1,
       });
     }
-    logger.info(`✅ Using Firebase Project: ${projectId}`);
+    logger.info(`✅ Using Firebase Project: ${projectId} (source: ${projectIdSource})`);
   } else {
     logger.debug(
       `❌ Failed to determine the Firebase Project ID. You can set a project later by setting the '--project' flag.`,