wip: Add isReady state property

Author: FrederikBolding

packages/snaps-controllers/src/snaps/SnapController.test.tsx

@@ -10624,6 +10624,8 @@ describe('SnapController', () => {
 
       await waitForStateChange(messenger);
 
+      await waitForStateChange(messenger);
+
       expect(snapController.isRunning(MOCK_SNAP_ID)).toBe(true);
 
       await new Promise((resolve) => setTimeout(resolve, 100));
@@ -10647,6 +10649,7 @@ describe('SnapController', () => {
       );
 
       expect(snapController.state).toStrictEqual({
+        isReady: false,
         snaps: {},
         snapStates: {},
         unencryptedSnapStates: {},
@@ -12997,7 +13000,11 @@ describe('SnapController', () => {
           controller.metadata,
           'includeInDebugSnapshot',
         ),
-      ).toMatchInlineSnapshot(`{}`);
+      ).toMatchInlineSnapshot(`
+        {
+          "isReady": false,
+        }
+      `);
     });
 
     describe('includeInStateLogs', () => {
@@ -13012,6 +13019,7 @@ describe('SnapController', () => {
           ),
         ).toMatchInlineSnapshot(`
           {
+            "isReady": false,
             "snaps": {},
           }
         `);

packages/snaps-controllers/src/snaps/SnapController.ts

@@ -318,6 +318,7 @@ export type SnapControllerState = {
   snaps: StoredSnaps;
   snapStates: Record<SnapId, string | null>;
   unencryptedSnapStates: Record<SnapId, string | null>;
+  isReady: boolean;
 };
 
 export type PersistedSnapControllerState = SnapControllerState & {
@@ -854,6 +855,7 @@ const defaultState: SnapControllerState = {
   snaps: {},
   snapStates: {},
   unencryptedSnapStates: {},
+  isReady: false,
 };
 
 /**
@@ -965,6 +967,12 @@ export class SnapController extends BaseController<
     super({
       messenger,
       metadata: {
+        isReady: {
+          includeInStateLogs: true,
+          includeInDebugSnapshot: true,
+          persist: false,
+          usedInUi: false,
+        },
         snapStates: {
           includeInStateLogs: false,
           persist: true,
@@ -1722,8 +1730,16 @@ export class SnapController extends BaseController<
     await this.#ensureOnboardingComplete();
 
     const flags = this.#getFeatureFlags();
+
+    const isReady = flags.disableSnaps !== true;
+    if (this.state.isReady !== isReady) {
+      this.update((state) => {
+        state.isReady = isReady;
+      });
+    }
+
     assert(
-      flags.disableSnaps !== true,
+      isReady,
       'The Snaps platform requires basic functionality to be used. Enable basic functionality in the settings to use the Snaps platform.',
     );
   }
@@ -2387,6 +2403,7 @@ export class SnapController extends BaseController<
     snapIds.forEach((snapId) => this.#revokeAllSnapPermissions(snapId));
 
     this.update((state) => {
+      state.isReady = false;
       state.snaps = {};
       state.snapStates = {};
       state.unencryptedSnapStates = {};