nnbd - phase 1

.travis.yml

@@ -1,5 +1,5 @@
 language: dart
-dart: stable
+dart: [dev]
 sudo: required
 addons:
   chrome: stable

build.yaml

@@ -9,7 +9,7 @@ builders:
     target: ":built_redux"
     import: "package:built_redux/builder.dart"
     builder_factories: ["builtRedux"]
-    build_extensions: {".dart": [".built_redux.g.part"]}
+    build_extensions: { ".dart": [".built_redux.g.part"] }
     auto_apply: dependents
     build_to: cache
-    applies_builders: ["source_gen|combining_builder"]
+    applies_builders: ["source_gen|combining_builder"]

lib/built_redux_test_utils.dart

@@ -7,12 +7,12 @@ import 'built_redux.dart';
 /// It takes all of the same optional params as expectAsync.
 void expectDispatched<T>(
   ActionDispatcher<T> actionDispatcher, {
-  void verifier(Action<T> action),
-  @deprecated void verfier(Action<T> action),
+  void Function(Action<T> action)? verifier,
+  @deprecated void Function(Action<T> action)? verfier,
   int count = 1,
   int max = 0,
-  String id,
-  String reason,
+  String? id,
+  String? reason,
 }) {
   actionDispatcher.setDispatcher(expectAsync1((Action<dynamic> action) {
     if (verifier != null) verifier(action as Action<T>);

lib/generator.dart

@@ -40,7 +40,7 @@ ActionsClass _actionsClassFromElement(ClassElement element) => ActionsClass(
 Iterable<ComposedActionClass> _composedActionClasses(ClassElement element) =>
     element.fields
         .where((f) => _isReduxActions(f.type.element))
-        .map((f) => ComposedActionClass(f.name, f.type.name));
+        .map((f) => ComposedActionClass(f.name, f.type.element.name));
 
 Iterable<Action> _actionsFromElement(ClassElement element) => element.fields
     .where(_isActionDispatcher)

lib/src/action.dart

@@ -7,12 +7,51 @@ class Action<Payload> {
   /// The actions payload.
   final Payload payload;
 
+  // factory Action(name, payload) => NullableAction(name, payload);
   Action(this.name, this.payload);
 
   @override
   String toString() => 'Action {\n  name: $name,\n  payload: $payload,\n}';
 }
 
+/// [Action] is the object passed to your reducer to signify the state change that needs to take place.
+/// Action [name]s should always be unique. Uniqeness is guarenteed when using ReduxActions.
+// class NonNullableAction<Payload> implements Action<Payload?> {
+//   /// A unique action name.
+//   final String _name;
+
+//   /// The actions payload.
+//   final Payload _payload;
+
+//   String get name => _name;
+
+//   Payload get payload => _payload;
+
+//   NonNullableAction(this._name, this._payload);
+
+//   @override
+//   String toString() => 'Action {\n  name: $name,\n  payload: $payload,\n}';
+// }
+
+// /// [Action] is the object passed to your reducer to signify the state change that needs to take place.
+// /// Action [name]s should always be unique. Uniqeness is guarenteed when using ReduxActions.
+// class NullableAction<Payload> implements Action<Payload> {
+//   /// A unique action name.
+//   final String _name;
+
+//   /// The actions payload.
+//   final Payload? _payload;
+
+//   String get name => _name;
+
+//   Payload? get payload => _payload;
+
+//   NullableAction(this._name, this._payload);
+
+//   @override
+//   String toString() => 'Action {\n  name: $name,\n  payload: $payload,\n}';
+// }
+
 // Dispatches an action to the store
 typedef Dispatcher<P> = void Function(Action<P> action);
 
@@ -26,12 +65,12 @@ typedef Dispatcher<P> = void Function(Action<P> action);
 /// store.actions.increment(3);
 /// ```
 class ActionDispatcher<P> {
-  Dispatcher _dispatcher;
+  late Dispatcher _dispatcher;
   final String _name;
 
   String get name => _name;
 
-  void call([P payload]) => _dispatcher(Action<P>(_name, payload));
+  void call(P payload) => _dispatcher(Action<P>(_name, payload));
 
   ActionDispatcher(this._name);
 

lib/src/store.dart

@@ -19,8 +19,8 @@ class Store<
       _stateController = StreamController.broadcast();
 
   // the current state
-  State _state;
-  Actions _actions;
+  late State _state;
+  late final Actions _actions;
 
   Store(
     Reducer<State, StateBuilder, dynamic> reducer,
@@ -70,8 +70,6 @@ class Store<
   /// [dispose] removes closes both the dispatch and subscription stream
   Future<Null> dispose() async {
     await _stateController.close();
-    _state = null;
-    _actions = null;
   }
 
   /// [replaceState] replaces the state of your store.

lib/src/store_change.dart

@@ -34,7 +34,8 @@ class StoreChangeHandlerBuilder<
     StateBuilder extends Builder<State, StateBuilder>,
     Actions extends ReduxActions> {
   final _map = Map<String, StoreChangeHandler<dynamic, State, StateBuilder>>();
-  StreamSubscription<StoreChange<State, StateBuilder, dynamic>> _subscription;
+  late StreamSubscription<StoreChange<State, StateBuilder, dynamic>>
+      _subscription;
 
   /// Registers [handler] function to the given [actionName]
   void add<Payload>(ActionName<Payload> actionName,

pubspec.yaml

@@ -1,22 +1,21 @@
 name: built_redux
-version: 7.5.11
-description:
-  A state management library written in dart that enforces immutability
+version: 8.0.0-nullsafety.0
+description: A state management library written in dart that enforces immutability
 homepage: https://github.com/davidmarne/built_redux
 dependencies:
-  analyzer: '>=0.33.0 <0.40.0'
+  analyzer: ">=0.40.0 <0.41.0"
   build: ^1.2.2
-  built_collection: ^4.3.0
-  built_value: '>=6.8.2 <8.0.0'
+  built_collection: ^5.0.0-nullsafety.0
+  built_value: ^8.0.0-nullsafety.0
   source_gen: ^0.9.4+6
-  test: ^1.9.1
+  test: ^1.16.0-nullsafety
 
 dev_dependencies:
   build_runner: ^1.7.1
-  build_test: ^0.10.11
-  built_value_generator: ^7.0.0
+  build_test: ^1.2.0
+  built_value_generator: ^8.0.0-nullsafety.0
   build_web_compilers: ^2.7.1
   workiva_analysis_options: ^1.0.0
 
 environment:
-  sdk: '>=2.4.0 <3.0.0'
+  sdk: ">=2.12.0-0 <3.0.0"

test/unit/action_generics_models.dart

@@ -58,7 +58,7 @@ Reducer<ActionGenerics, ActionGenericsBuilder, dynamic>
               ..add<Map<String, List<int>>>(
                   ActionGenericsActionsNames.mapStringToListIntAction,
                   (s, a, b) =>
-                      b.count += a.payload['k'].fold<int>(0, (c, n) => c + n)))
+                      b.count += a.payload['k']??.fold<int>(0, (c, n) => c + n)))
             .build();
 
 NextActionHandler thunkMiddleware(