chore: appease analyzer

Author: mcquenji

analysis_options.yaml

@@ -11,6 +11,8 @@ plugins:
       base_class:
         ignorable: true
         enabled: true
+      domain_factory_from_di: true
+      prefer_domain_di_factory: true
 analyzer:
   exclude:
     - "**/*.g.dart"

lib/src/domain/services/analytics_service.dart

@@ -142,4 +142,14 @@ abstract class AnalyticsService extends Service {
   Future<void> groupUser(String groupId, {Map<String, dynamic>? traits});
   @override
   String get group => '${super.group}.AnalyticsService';
+
+  /// Internal constructor for subclasses.
+  AnalyticsService.internal();
+
+  /// Returns the DI-registered implementation of [AnalyticsService].
+  ///
+  /// Shorthand for [Service.get].
+  factory AnalyticsService() {
+    return Service.get<AnalyticsService>();
+  }
 }

lib/src/domain/services/routing_service.dart

@@ -1,6 +1,7 @@
 import 'dart:async';
 
 import 'package:grumpy/grumpy.dart';
+import 'package:meta/meta.dart';
 
 /// A service responsible for managing application routing.
 /// Note: This router is only responsible for marking routes as active and
@@ -12,14 +13,21 @@ abstract class RoutingService<T, Config extends Object> extends Service {
   /// Returns the root route of the application with all its nested routes expanded.
   Route<T, Config> get root;
 
+  /// Required for the factory pattern to work.
+  @internal
+  RoutingService.internal();
+
   /// Navigates to the specified [path] and invokes the [callback] with the built presentation.
   /// If [skipPreview] is true, the preview phase is skipped and [callback] is called only after the final build phase.
   Future<void> navigate(
     String path, {
     bool skipPreview = false,
-    required void Function(T, bool) callback,
+    void Function(T, bool) callback = noopCallback,
   });
 
+  /// Default callback for [navigate].
+  static void noopCallback(dynamic _, bool _) {}
+
   /// Checks if the specified [path] is currently active.
   ///
   /// If [exact] is true (default), checks for an exact match; otherwise, checks for a partial match.
@@ -56,6 +64,16 @@ abstract class RoutingService<T, Config extends Object> extends Service {
   StreamSubscription<ViewChangedEvent<T, Config>> onViewChanged(
     void Function(ViewChangedEvent<T, Config>) callback,
   );
+
+  /// A stream of all view change events.
+  Stream<ViewChangedEvent<T, Config>> get viewStream;
+
+  /// Returns the DI-registered implementation of [RoutingService].
+  ///
+  /// Shorthand for [Service.get].
+  factory RoutingService() {
+    return Service.get<RoutingService<T, Config>>();
+  }
 }
 
 /// An event representing a change in the view rendered by the [RoutingService].

lib/src/domain/services/telemetry_service.dart

@@ -122,4 +122,14 @@ abstract class TelemetryService extends Service {
   void addSpanAttribute(String key, String value);
   @override
   String get group => '${super.group}.TelemetryService';
+
+  /// Internal constructor for subclasses.
+  TelemetryService.internal();
+
+  /// Returns the DI-registered implementation of [TelemetryService].
+  ///
+  /// Shorthand for [Service.get].
+  factory TelemetryService() {
+    return Service.get<TelemetryService>();
+  }
 }

lib/src/infra/services/noop_analytics_service.dart

@@ -2,6 +2,9 @@ import 'package:grumpy/grumpy.dart';
 
 /// A no-operation implementation of [AnalyticsService].
 class NoopAnalyticsService extends AnalyticsService {
+  /// A no-operation implementation of [AnalyticsService].
+  NoopAnalyticsService() : super.internal();
+
   @override
   noSuchMethod(Invocation invocation) {
     log('NoopAnalyticsService: ${invocation.memberName} called.');

lib/src/infra/services/noop_telemetry_service.dart

@@ -4,6 +4,9 @@ import 'package:grumpy/grumpy.dart';
 
 /// A no-operation implementation of [TelemetryService].
 class NoopTelemetryService extends TelemetryService {
+  /// A no-operation implementation of [TelemetryService].
+  NoopTelemetryService() : super.internal();
+
   @override
   noSuchMethod(Invocation invocation) {
     log('NoopTelemetryService: ${invocation.memberName} called.');

lib/src/infra/services/routing_kit_routing_service.dart

@@ -27,7 +27,8 @@ class RoutingKitRoutingService<T, Config extends Object>
       StreamController<ViewChangedEvent<T, Config>>.broadcast();
 
   /// [RoutingService] impementation that uses RoutingKit for route parsing and matching.
-  RoutingKitRoutingService(this.rootModule, {this.caseSensitive = false}) {
+  RoutingKitRoutingService(this.rootModule, {this.caseSensitive = false})
+    : super.internal() {
     initialize();
   }
 
@@ -127,12 +128,10 @@ class RoutingKitRoutingService<T, Config extends Object>
   Future<void> navigate(
     String path, {
     bool skipPreview = false,
-    required void Function(T, bool) callback,
+    void Function(T, bool) callback = RoutingService.noopCallback,
   }) async {
     void handler(T view, bool isPreview) {
-      log(
-        'View changed: isPreview=$isPreview, view=${view.runtimeType} for path: $path',
-      );
+      log('View changed: isPreview=$isPreview, view=$view for path: $path');
 
       _viewChangeController.add((
         view: view,
@@ -291,4 +290,8 @@ class RoutingKitRoutingService<T, Config extends Object>
   StreamSubscription<ViewChangedEvent<T, Config>> onViewChanged(
     void Function(ViewChangedEvent<T, Config>) callback,
   ) => _viewChangeController.stream.listen(callback);
+
+  @override
+  Stream<ViewChangedEvent<T, Config>> get viewStream =>
+      _viewChangeController.stream.asBroadcastStream();
 }

lib/src/utils/mutation_mixins.dart

@@ -59,7 +59,7 @@ mixin MutationMixins<T> on Repo<T>, RepoLifecycleHooksMixin<T>, TelemetryMixin {
 
     final stateVersion = _stateVersion;
 
-    final analytics = Service.get<AnalyticsService>();
+    final analytics = AnalyticsService();
 
     try {
       log('Starting mutation $name');
@@ -117,7 +117,7 @@ mixin MutationMixins<T> on Repo<T>, RepoLifecycleHooksMixin<T>, TelemetryMixin {
     final snapshot = state;
     _applyOptimistics(optimisticPolicy);
     final stateVersion = _stateVersion;
-    final analytics = Service.get<AnalyticsService>();
+    final analytics = AnalyticsService();
 
     try {
       log('Starting action $name');

lib/src/utils/query_mixins.dart

@@ -113,7 +113,7 @@ mixin QueryMixin<T> on Repo<T>, RepoLifecycleHooksMixin<T>, TelemetryMixin {
       );
     }
 
-    final analytics = Service.get<AnalyticsService>();
+    final analytics = AnalyticsService();
     if (analyticsAction != null) {
       await analytics.trackEvent(
         analyticsAction,

lib/src/utils/telemetry_mixin.dart

@@ -12,7 +12,7 @@ mixin TelemetryMixin {
     FutureOr<T> Function() function, {
     Map<String, dynamic>? attributes,
   }) async {
-    final telemetry = Service.get<TelemetryService>();
+    final telemetry = TelemetryService();
 
     return telemetry.runSpan(name, function, attributes: attributes);
   }