Merge branch 'main' into dependabot/npm_and_yarn/docs/cross-spawn-7.0.6

docs/docs/api/observers.mdx

@@ -90,6 +90,36 @@ class _CounterExampleState extends State<CounterExample> {
 }
 ```
 
+#### Performance optimizations
+Mostly you try to make `Observer` as small as possible. But sometimes you want to exclude
+the widget subtree from being rebuilt and improve the performance.
+
+For that you can use an `Observer.withBuiltChild` constructor. It takes `builder`
+and `child` arguments and internally uses the same technique as in [AnimatedBuilder](https://api.flutter.dev/flutter/widgets/AnimatedBuilder-class.html)
+
+Here is a brief example:
+
+```dart
+final obsColor = Observable(Colors.green);
+
+Observer.withBuiltChild(
+  builder: (context, child) {
+    return Container(
+      padding: const EdgeInsets.all(16),
+      color: obsColor.value, // this widget will be rebuilt when the color value changes
+      child: child,
+    );
+  },
+  child: ListView.builder( // this part will not be rebuilt
+    shrinkWrap: true,
+    itemCount: 1000,
+    itemBuilder: (context, index) {
+      return ListTile(title: Text('Item $index'));
+    },
+  ),
+)
+```
+
 ## ReactionBuilder widget
 
 If you ever ran into a need for running a reaction when a Widget loads, you most

flutter_mobx/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 2.2.2
+
+- fix: removed runtime asserts and nullables from `Observer.withBuiltChild` - [@subzero911](https://github.com/subzero911)
+
 ## 2.2.1+1
 
 - Analyzer fixes

flutter_mobx/lib/flutter_mobx.dart

@@ -32,6 +32,7 @@
 ///      );
 ///
 
+// ignore: unnecessary_library_name
 library flutter_mobx;
 
 export 'package:flutter_mobx/src/multi_reaction_builder.dart';

flutter_mobx/lib/src/observer.dart

@@ -23,31 +23,24 @@ class Observer extends StatelessObserverWidget {
     required this.builder,
     super.name,
     super.warnWhenNoObservables,
-  })  : debugConstructingStackFrame = debugFindConstructingStackFrame(),
-        builderWithChild = null,
-        child = null,
-        assert(builder != null);
+  }) : debugConstructingStackFrame = debugFindConstructingStackFrame();
 
   /// Observer which excludes the child branch from being rebuilt
+  ///
+  /// - [builder] is a builder function with a child widget as a parameter;
+  ///
+  /// - [child] is the widget to pass to the [builder] function.
   // ignore: prefer_const_constructors_in_immutables
   Observer.withBuiltChild({
     super.key,
-    required this.builderWithChild,
-    required this.child,
+    required Widget Function(BuildContext, Widget) builder,
+    required Widget child,
     super.name,
     super.warnWhenNoObservables,
   })  : debugConstructingStackFrame = debugFindConstructingStackFrame(),
-        builder = null,
-        assert(builderWithChild != null && child != null);
-
-  /// regular builder, suitable for most cases
-  final WidgetBuilder? builder;
-
-  /// builder function with child parameter
-  final TransitionBuilder? builderWithChild;
+        builder = ((context) => builder(context, child));
 
-  /// The child widget to pass to the [builderWithChild].
-  final Widget? child;
+  final WidgetBuilder builder;
 
   /// The stack frame pointing to the source that constructed this instance.
   final String? debugConstructingStackFrame;
@@ -60,8 +53,7 @@ class Observer extends StatelessObserverWidget {
           : '');
 
   @override
-  Widget build(BuildContext context) =>
-      builderWithChild?.call(context, child) ?? builder!.call(context);
+  Widget build(BuildContext context) => builder.call(context);
 
   /// Matches constructor stack frames, in both VM and web environments.
   static final _constructorStackFramePattern = RegExp(r'\bnew\b');

flutter_mobx/lib/version.dart

@@ -1,4 +1,4 @@
 // Generated via set_version.dart. !!!DO NOT MODIFY BY HAND!!!
 
 /// The current version as per `pubspec.yaml`.
-const version = '2.2.1+1';
+const version = '2.2.2';

flutter_mobx/pubspec.yaml

@@ -2,15 +2,15 @@ name: flutter_mobx
 description:
   Flutter integration for MobX. It provides a set of Observer widgets that automatically rebuild
   when the tracked observables change.
-version: 2.2.1+1
+version: 2.2.2
 
 repository: https://github.com/mobxjs/mobx.dart
 issue_tracker: https://github.com/mobxjs/mobx.dart/issues
 homepage: https://mobx.netlify.app
 topics:
   - reactive-programming
   - state-management
-  - ui
+  - user-interface
 
 environment:
   sdk: ">=3.0.0 <4.0.0"
@@ -24,7 +24,7 @@ dependencies:
 
 dev_dependencies:
   build_runner: ^2.4.9
-  flutter_lints: ^4.0.0
+  flutter_lints: ^5.0.0
   flutter_test:
     sdk: flutter
   mocktail: ^1.0.3

flutter_mobx/test/flutter_mobx_test.dart

@@ -69,14 +69,14 @@ void main() {
     await tester.pumpWidget(
       MaterialApp(
         home: Observer.withBuiltChild(
-          builderWithChild: (context, child) {
+          builder: (context, child) {
             return Column(
               children: [
                 ElevatedButton(
                     onPressed: () => message.value = 'Clicked',
                     child: Container()),
                 Text(message.value, key: key1),
-                child!,
+                child,
                 Builder(builder: (context) {
                   return Text(message.value, key: key3);
                 }),

mobx_examples/lib/todos/todo_widgets.dart

@@ -121,7 +121,7 @@ class ActionBar extends StatelessWidget {
         },
       ),
       Observer(
-          builder: (_) => ButtonBar(
+          builder: (_) => OverflowBar(
                 children: <Widget>[
                   ElevatedButton(
                     onPressed: list.canRemoveAllCompleted

pubspec.yaml

@@ -4,4 +4,4 @@ environment:
   sdk: ">=3.0.0 <4.0.0"
 
 dev_dependencies:
-  melos: ^5.2.1
+  melos: ^6.2.0