Miscellaneous fixes

- Fix sdk disposing not working properly.
- Fix embedded canvases crashing with local storage.
- Fix and improve CStatus constructors.

Author: BirjuVachhani

lib/src/codelessly.dart

@@ -171,6 +171,7 @@ class Codelessly {
   /// if [completeDispose] is true, the SDK's internal stream controllers are
   /// also disposed instead of reset.
   void dispose({bool completeDispose = false}) {
+    log('[Codelessly] Disposing SDK completeDispose: $completeDispose');
     if (completeDispose) {
       _statusStreamController.close();
     } else {

lib/src/codelessly_config.dart

@@ -115,40 +115,30 @@ class CodelesslyConfig with EquatableMixin {
       ];
 }
 
-/// SDK initialization state enums.
-enum CodelesslyStatus {
-  /// The SDK has not been initialized.
-  empty,
-
-  /// The SDK has loaded settings and is ready to be initialized.
-  configured,
+sealed class CStatus {
+  const CStatus();
 
-  /// The SDK is initializing.
-  loading,
+  factory CStatus.empty() => CEmpty();
 
-  /// The SDK is loaded and is ready to use.
-  loaded,
+  factory CStatus.configured() => CConfigured();
 
-  /// The SDK has an error..
-  error,
-}
+  factory CStatus.loading(String step) => CLoading(step);
 
-sealed class CStatus {
-  const CStatus();
+  factory CStatus.loaded() => CLoaded();
 
-  factory CStatus.empty() => const CEmpty();
-  factory CStatus.configured() => const CConfigured();
-  factory CStatus.loading(String step) => CLoading(step);
-  factory CStatus.loaded() => const CLoaded();
-  factory CStatus.error() => const CError();
+  factory CStatus.error() => CError();
 }
 
 class CEmpty extends CStatus {
-  const CEmpty();
+  const CEmpty._();
+
+  factory CEmpty() => const CEmpty._();
 }
 
 class CConfigured extends CStatus {
-  const CConfigured();
+  const CConfigured._();
+
+  factory CConfigured() => const CConfigured._();
 }
 
 class CLoading extends CStatus {
@@ -158,9 +148,13 @@ class CLoading extends CStatus {
 }
 
 class CLoaded extends CStatus {
-  const CLoaded();
+  const CLoaded._();
+
+  factory CLoaded() => const CLoaded._();
 }
 
 class CError extends CStatus {
-  const CError();
+  const CError._();
+
+  factory CError() => const CError._();
 }

lib/src/data/data_manager.dart

@@ -116,7 +116,7 @@ class DataManager {
 
     if (_publishModel != null) {
       log('[DataManager] Publish model is cached locally. Emitting.');
-      emitPublishModel();
+      await emitPublishModel();
 
       loadFontsFromPublishModel();
     }
@@ -224,7 +224,7 @@ class DataManager {
       log('[DataManager] Publish model is still null during init. Waiting for the first publish model from the server.');
       final model = await firstPublishEvent;
       _publishModel = model;
-      emitPublishModel();
+      await emitPublishModel();
       savePublishModel();
 
       log('[DataManager] Publish model during init is now available. Proceeding with init!');
@@ -239,10 +239,6 @@ class DataManager {
       }
     }
 
-    // The PublishModel is guaranteed to be available here. The init exits
-    // otherwise.
-    await onPublishModelLoaded(_publishModel!);
-
     // If a [layoutID] was specified, then that layout must be prioritized and
     // downloaded first if it is not already cached.
     //
@@ -416,7 +412,8 @@ class DataManager {
   }
 
   /// Emits the current [_publishModel] to the [_publishModelStreamController].
-  void emitPublishModel() {
+  Future<void> emitPublishModel() async {
+    await onPublishModelLoaded(_publishModel!);
     log('[DataManager] Emitting publish model to stream. has model: ${_publishModel != null}');
     _publishModelStreamController.add(_publishModel);
   }
@@ -444,6 +441,7 @@ class DataManager {
 
   /// Disposes the [DataManager] instance.
   void dispose() {
+    log('[DataManager] Disposing dataManager...');
     _publishModelStreamController.close();
     _publishModelDocumentListener?.cancel();
     initialized = false;
@@ -666,7 +664,7 @@ class DataManager {
     }
 
     savePublishModel();
-    emitPublishModel();
+    await emitPublishModel();
   }
 
   /// Compares the current [localModel] with a newly fetched [serverModel] and
@@ -979,7 +977,7 @@ class DataManager {
       log('[DataManager] \tLayout [$layoutID] has no variables.');
     }
 
-    emitPublishModel();
+    await emitPublishModel();
     savePublishModel();
 
     log('[DataManager] \tLayoutModel [$layoutID] ready, time for fonts. Get Set...');
@@ -1031,9 +1029,8 @@ class DataManager {
 
     if (model != null) {
       log('[DataManager] Successfully downloaded complete publish bundle. Emitting it.');
-      await onPublishModelLoaded(model);
       _publishModel = model;
-      emitPublishModel();
+      await emitPublishModel();
       savePublishModel();
       return true;
     }

lib/src/ui/codelessly_loading_screen.dart

@@ -1,9 +1,14 @@
 import 'package:flutter/cupertino.dart';
 import 'package:flutter/material.dart';
 
-class CodelesslyLoadingScreen extends StatelessWidget {
+class CodelesslyLoadingScreen extends StatefulWidget {
   const CodelesslyLoadingScreen({super.key});
 
   @override
-  Widget build(BuildContext context) => const SizedBox.shrink();
+  State<CodelesslyLoadingScreen> createState() => _CodelesslyLoadingScreenState();
+}
+
+class _CodelesslyLoadingScreenState extends State<CodelesslyLoadingScreen> {
+  @override
+  Widget build(BuildContext context) => SizedBox.shrink();
 }

lib/src/ui/codelessly_widget_controller.dart

@@ -175,7 +175,7 @@ class CodelesslyWidgetController extends ChangeNotifier {
       // instance, the user explicitly wants more control over the SDK, so we
       // do nothing and let the user handle it.
       if (isGlobalInstance) {
-        if (status == const CEmpty()) {
+        if (status == CEmpty()) {
           effectiveCodelessly.configure(
             config: config,
             authManager: authManager,
@@ -185,7 +185,7 @@ class CodelesslyWidgetController extends ChangeNotifier {
           );
         }
         status = effectiveCodelessly.status;
-        if (status == const CConfigured()) {
+        if (status == CConfigured()) {
           effectiveCodelessly.initialize();
         }
       }

lib/src/ui/layout_builder.dart

@@ -44,7 +44,7 @@ class _CodelesslyLayoutRetrieverState extends State<CodelesslyLayoutRetriever> {
   @override
   void dispose() {
     controller.dispose();
-    codelessly.dispose();
+    codelessly.dispose(completeDispose: true);
     super.dispose();
   }