From 493b79b8881b44b34c2a19ded009308960b16fac Mon Sep 17 00:00:00 2001
From: Justin Baur <19896123+justindbaur@users.noreply.github.com>
Date: Wed, 24 Apr 2024 11:14:53 -0400
Subject: [PATCH] Only Run Migrations in True Background (#8548)

* Only Run Migrations in True Background

* Use `isPrivateMode`

* Use `popupOnlyContext`
---
 apps/browser/src/background/main.background.ts  | 3 ++-
 apps/browser/src/popup/services/init.service.ts | 2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/apps/browser/src/background/main.background.ts b/apps/browser/src/background/main.background.ts
index a64ee2b8a00..6069e9b5e96 100644
--- a/apps/browser/src/background/main.background.ts
+++ b/apps/browser/src/background/main.background.ts
@@ -1095,7 +1095,8 @@ export default class MainBackground {
   async bootstrap() {
     this.containerService.attachToGlobal(self);
 
-    await this.stateService.init({ runMigrations: !this.isPrivateMode });
+    // Only the "true" background should run migrations
+    await this.stateService.init({ runMigrations: !this.popupOnlyContext });
 
     // This is here instead of in in the InitService b/c we don't plan for
     // side effects to run in the Browser InitService.
diff --git a/apps/browser/src/popup/services/init.service.ts b/apps/browser/src/popup/services/init.service.ts
index c9e6d66c2af..ee842565d75 100644
--- a/apps/browser/src/popup/services/init.service.ts
+++ b/apps/browser/src/popup/services/init.service.ts
@@ -22,7 +22,7 @@ export class InitService {
 
   init() {
     return async () => {
-      await this.stateService.init();
+      await this.stateService.init({ runMigrations: false }); // Browser background is responsible for migrations
       await this.i18nService.init();
 
       if (!BrowserPopupUtils.inPopup(window)) {