fix(EWM-511): rename dataFabric -> createData

Author: levitckii-daniil

docs/type_safe_navigation.md

@@ -87,6 +87,7 @@ abstract class CompassRoute<T extends CompassRouteDataQuery>
 ```
 
 Key features:
+
 - Handles query parameter serialization and deserialization
 - Provides type-safe access to route parameters
 - Preserves parameters during navigation
@@ -103,6 +104,7 @@ abstract class CompassRouteParameterless<T extends CompassRouteData>
 ```
 
 Key features:
+
 - Simplified implementation for routes without parameters
 - Uses a factory method pattern with `dataFabric()` to create route data
 - Type-safe routing without the overhead of parameter handling
@@ -119,6 +121,7 @@ abstract class CompassShellRoute
 ```
 
 Key features:
+
 - Creates a StatefulShellRoute for complex UI patterns like tab navigation
 - Each child route becomes a separate branch in the shell
 - Maintains independent navigation state for each branch
@@ -148,6 +151,7 @@ The Compass router automatically discovers routes and guards through dependency
 4. The router then builds its routing table and middleware chain using these discovered components
 
 Specifically, in `CompassRouter`:
+
 - Routes are discovered via `GetIt.I.getAll<CompassBaseRoute>()`
 - Guards are discovered via `GetIt.I.getAll<CompassGuard>()`
 - Guards are sorted by priority for execution order (higher priority guards execute first)
@@ -161,6 +165,7 @@ This automatic discovery eliminates the need for manual route registration and e
 Routes and guards must be registered using a specific pattern for the DI container to discover them:
 
 **For Routes:**
+
 ```dart
 @named
 @Singleton(as: CompassBaseRoute)
@@ -170,6 +175,7 @@ class MyRoute extends CompassRouteParameterless<MyRouteData> {
 ```
 
 **For Guards:**
+
 ```dart
 @named
 @Singleton(as: CompassGuard)
@@ -179,6 +185,7 @@ class MyGuard extends CompassGuard {
 ```
 
 This pattern ensures that:
+
 1. The concrete implementation is registered with a unique name (`@named`)
 2. The implementation is registered as its base interface type (`CompassBaseRoute` or `CompassGuard`)
 3. The DI container can find all implementations of the base types
@@ -227,6 +234,7 @@ For parameterized routes:
 class ProfileRoute extends CompassRoute<ProfileRouteData> {
   ProfileRoute() : super(
     name: 'profile',
+    path: '/profile',
     isTopLevel: true,
     builder: (context, data, state) => ProfilePage(userId: data.userId),
   );
@@ -249,13 +257,14 @@ For parameterless routes:
 class HomeRoute extends CompassRouteParameterless<HomeRouteData> {
   HomeRoute() : super(
     name: 'home',
+    path: '/home',
     isTopLevel: true,
     isInitial: true,
     builder: (context, data, state) => HomePage(),
   );
 
   @override
-  HomeRouteData dataFabric() {
+  HomeRouteData createData() {
     return HomeRouteData();
   }
 }
@@ -338,6 +347,7 @@ context.compassContinue(ProfileRouteData(userId: '123'));
 ```
 
 Technical implementation:
+
 1. The current path is preserved using the '.' path segment
 2. The new path segments are appended to the relative path
 3. Query parameters from both the original and new location are merged
@@ -401,6 +411,7 @@ class AuthGuard extends CompassGuard {
 ```
 
 Guards are executed in priority order (highest to lowest) when navigation occurs. The predefined priority constants are:
+
 - `priorityHigh = 3` - For critical guards like authentication
 - `priorityMedium = 2` - For feature flag guards and similar
 - `priorityLow = 1` - For analytics and other non-blocking guards
@@ -535,6 +546,7 @@ While Freezed is a powerful tool for creating immutable data classes in Flutter,
    - This mismatch causes the router to fail to find the correct route handler
 
 3. **Example of the Problem**:
+
    ```dart
    // This is what the router expects
    final type = ProfileRouteData;
@@ -599,11 +611,11 @@ While Freezed is a powerful tool for creating immutable data classes in Flutter,
    - Use route verification when redirecting in guards
 
 10. **Router organization and discovery**
-   - The router automatically discovers all routes and guards registered as their base types
-   - Routes must be registered with `@named` and `@Singleton(as: CompassBaseRoute)`
-   - Guards must be registered with `@named` and `@Singleton(as: CompassGuard)`
-   - Only routes with `isTopLevel = true` are registered directly with GoRouter
-   - Only one route should have `isInitial = true` to define the app's entry point
-   - Guards are executed in priority order (highest to lowest)
-   - Routes are identified by both type and path for efficient lookup
 
+- The router automatically discovers all routes and guards registered as their base types
+- Routes must be registered with `@named` and `@Singleton(as: CompassBaseRoute)`
+- Guards must be registered with `@named` and `@Singleton(as: CompassGuard)`
+- Only routes with `isTopLevel = true` are registered directly with GoRouter
+- Only one route should have `isInitial = true` to define the app's entry point
+- Guards are executed in priority order (highest to lowest)
+- Routes are identified by both type and path for efficient lookup

lib/app/router/compass/data_from_query.dart

@@ -77,14 +77,14 @@ mixin CompassRouteDataQueryMixin<T extends CompassRouteDataQuery>
 mixin EmptyRouteDataMixin<T extends CompassRouteData> on CompassBaseGoRoute<T> {
   @override
   T dataFromState(GoRouterState state) {
-    return dataFabric();
+    return createData();
   }
 
   /// Factory method that creates a new instance of the route data.
   ///
   /// Since this mixin is for parameterless routes, this method should
   /// return a default instance of the route data class.
-  T dataFabric();
+  T createData();
 
   @override
   Uri toLocation(T data) {

lib/app/router/compass/go_route.dart

@@ -89,17 +89,17 @@ abstract class CompassBaseGoRoute<T extends CompassRouteData>
   ///
   /// This is used when building the route's widget to provide it with
   /// the correct data from the current navigation state.
-  T dataFromState(GoRouterState state);
+  T routeDataConstructor(GoRouterState state);
 
   @override
   late final GoRoute route = GoRoute(
     path: path,
     name: name,
     builder: builder != null
-        ? (context, state) => builder!(context, dataFromState(state), state)
+        ? (context, state) => builder!(context, routeDataConstructor(state), state)
         : null,
     pageBuilder: pageBuilder != null
-        ? (context, state) => pageBuilder!(context, dataFromState(state), state)
+        ? (context, state) => pageBuilder!(context, routeDataConstructor(state), state)
         : null,
     redirect: redirect,
     routes: routes,

lib/feature/add_seed/add_existing_wallet/route.dart

@@ -17,7 +17,7 @@ class AddExistingWalletRoute
         );
 
   @override
-  AddExistingWalletRouteData dataFabric() {
+  AddExistingWalletRouteData createData() {
     return const AddExistingWalletRouteData();
   }
 }

lib/feature/add_seed/import_wallet/route.dart

@@ -23,7 +23,7 @@ class ImportWalletRoute
         );
 
   @override
-  ImportWalletRouteData dataFabric() {
+  ImportWalletRouteData createData() {
     return const ImportWalletRouteData();
   }
 }

lib/feature/biometry/view/route.dart

@@ -13,7 +13,7 @@ class EnableBiometryRoute
         );
 
   @override
-  EnableBiometryRouteData dataFabric() {
+  EnableBiometryRouteData createData() {
     return const EnableBiometryRouteData();
   }
 }

lib/feature/browser_v2/route.dart

@@ -14,7 +14,7 @@ class BrowserRoute extends CompassRouteParameterless<BrowserRouteData> {
         );
 
   @override
-  BrowserRouteData dataFabric() {
+  BrowserRouteData createData() {
     return const BrowserRouteData();
   }
 }

lib/feature/network/configure_networks/route.dart

@@ -17,7 +17,7 @@ class ConfigureNetworksRoute
         );
 
   @override
-  ConfigureNetworksRouteData dataFabric() {
+  ConfigureNetworksRouteData createData() {
     return const ConfigureNetworksRouteData();
   }
 }

lib/feature/no_internet/route.dart

@@ -14,7 +14,7 @@ class NoInternetRoute extends CompassRouteParameterless<NoInternetRouteData> {
         );
 
   @override
-  NoInternetRouteData dataFabric() {
+  NoInternetRouteData createData() {
     return const NoInternetRouteData();
   }
 }

lib/feature/onboarding/route.dart

@@ -23,7 +23,7 @@ class OnBoardingRoute extends CompassRouteParameterless<OnBoardingRouteData> {
         );
 
   @override
-  OnBoardingRouteData dataFabric() {
+  OnBoardingRouteData createData() {
     return const OnBoardingRouteData();
   }
 }

lib/feature/profile/manage_seeds_accounts/route.dart

@@ -24,7 +24,7 @@ class ManageSeedsAccountsRoute
         );
 
   @override
-  ManageSeedsAccountsRouteData dataFabric() {
+  ManageSeedsAccountsRouteData createData() {
     return const ManageSeedsAccountsRouteData();
   }
 }

lib/feature/profile/route.dart

@@ -21,7 +21,7 @@ class ProfileRoute extends CompassRouteParameterless<ProfileRouteData> {
         );
 
   @override
-  ProfileRouteData dataFabric() {
+  ProfileRouteData createData() {
     return const ProfileRouteData();
   }
 }

lib/feature/splash/route.dart

@@ -16,7 +16,7 @@ class SplashScreenRoute
         );
 
   @override
-  SplashScreenRouteData dataFabric() {
+  SplashScreenRouteData createData() {
     return const SplashScreenRouteData();
   }
 }

lib/feature/update_version/route.dart

@@ -22,7 +22,7 @@ class UpdateVersionRoute
         );
 
   @override
-  UpdateVersionRouteData dataFabric() {
+  UpdateVersionRouteData createData() {
     return const UpdateVersionRouteData();
   }
 }

lib/feature/wallet/new_account/add_external_account/route.dart

@@ -13,7 +13,7 @@ class NewExternalAccountRoute
         );
 
   @override
-  NewExternalAccountRouteData dataFabric() {
+  NewExternalAccountRouteData createData() {
     return const NewExternalAccountRouteData();
   }
 }

lib/feature/wallet/new_account/route.dart

@@ -24,7 +24,7 @@ class AddAccountRoute extends CompassRouteParameterless<AddAccountRouteData> {
         );
 
   @override
-  AddAccountRouteData dataFabric() {
+  AddAccountRouteData createData() {
     return const AddAccountRouteData();
   }
 }

lib/feature/wallet/new_account/select_seed/route.dart

@@ -17,7 +17,7 @@ class SelectSeedRoute extends CompassRouteParameterless<SelectSeedRouteData> {
         );
 
   @override
-  SelectSeedRouteData dataFabric() {
+  SelectSeedRouteData createData() {
     return const SelectSeedRouteData();
   }
 }

lib/feature/wallet/route.dart

@@ -60,7 +60,7 @@ class WalletRoute extends CompassRouteParameterless<WalletRouteData> {
         );
 
   @override
-  WalletRouteData dataFabric() {
+  WalletRouteData createData() {
     return const WalletRouteData();
   }
 }