Upgrade simple_blocs to latest version

Author: brianegan

mobx/analysis_options.yaml

@@ -9,6 +9,12 @@
 # packages, and plugins designed to encourage good coding practices.
 include: package:flutter_lints/flutter.yaml
 
+analyzer: 
+  language:
+    strict-casts: true
+    strict-inference: true
+    strict-raw-types: true
+
 linter:
   # The lint rules applied to this project can be customized in the
   # section below to disable rules from the `package:flutter_lints/flutter.yaml`

mobx/lib/details_screen.dart

@@ -31,7 +31,7 @@ class DetailsScreen extends StatelessWidget {
         onPressed: () {
           Navigator.push(
             context,
-            MaterialPageRoute(
+            MaterialPageRoute<void>(
               builder: (context) => EditTodoScreen(
                 todo: todo,
                 onEdit: () => Navigator.pop(context),

simple_blocs/.gitignore

@@ -1,9 +1,7 @@
-# Files and directories created by pub
-.packages
-.pub/
-build/
-# Remove the following pattern if you wish to check in your lock file
-pubspec.lock
+# https://dart.dev/guides/libraries/private-files
+# Created by `dart pub`
+.dart_tool/
 
-# Directory created by dartdoc
-doc/api/
+# Avoid committing pubspec.lock for library packages; see
+# https://dart.dev/guides/libraries/private-files#pubspeclock.
+pubspec.lock

simple_blocs/CHANGELOG.md

@@ -1,5 +1,3 @@
-# Changelog
+## 1.0.0
 
-## 0.0.1
-
-- Initial version, created by Stagehand
+- Initial version.

simple_blocs/README.md

@@ -1,22 +1,39 @@
-# blocs
+<!-- 
+This README describes the package. If you publish this package to pub.dev,
+this README's contents appear on the landing page for your package.
 
-A library for Dart developers.
+For information about how to write a good package README, see the guide for
+[writing package pages](https://dart.dev/tools/pub/writing-package-pages). 
 
-Created from templates made available by Stagehand under a BSD-style
-[license](https://github.com/dart-lang/stagehand/blob/master/LICENSE).
+For general information about developing packages, see the Dart guide for
+[creating packages](https://dart.dev/guides/libraries/create-packages)
+and the Flutter guide for
+[developing packages and plugins](https://flutter.dev/to/develop-packages). 
+-->
 
-## Usage
+TODO: Put a short description of the package here that helps potential users
+know whether this package might be useful for them.
+
+## Features
+
+TODO: List what your package can do. Maybe include images, gifs, or videos.
 
-A simple usage example:
+## Getting started
 
-    import 'package:simple_blocs/blocs.dart';
+TODO: List prerequisites and provide or point to information on how to
+start using the package.
+
+## Usage
 
-    main() {
-      var awesome = new Awesome();
-    }
+TODO: Include short and useful examples for package users. Add longer examples
+to `/example` folder. 
 
-## Features and bugs
+```dart
+const like = 'sample';
+```
 
-Please file feature requests and bugs at the [issue tracker][tracker].
+## Additional information
 
-[tracker]: http://example.com/issues/replaceme
+TODO: Tell users more about the package: where to find more information, how to 
+contribute to the package, how to file issues, what response they can expect 
+from the package authors, and more.

simple_blocs/analysis_options.yaml

@@ -1,14 +1,32 @@
+# This file configures the static analysis results for your project (errors,
+# warnings, and lints).
+#
+# This enables the 'recommended' set of lints from `package:lints`.
+# This set helps identify many issues that may lead to problems when running
+# or consuming Dart code, and enforces writing Dart using a single, idiomatic
+# style and format.
+#
+# If you want a smaller set of lints you can change this to specify
+# 'package:lints/core.yaml'. These are just the most critical lints
+# (the recommended set includes the core lints).
+# The core lints are also what is used by pub.dev for scoring packages.
+
+include: package:lints/recommended.yaml
+
+# Uncomment the following section to specify additional rules.
+
 analyzer:
-#   exclude:
-#     - path/to/excluded/files/**
-
-# Lint rules and documentation, see http://dart-lang.github.io/linter/lints
-linter:
-  rules:
-    - cancel_subscriptions
-    - hash_and_equals
-    - iterable_contains_unrelated_type
-    - list_remove_unrelated_type
-    - test_types_in_equals
-    - unrelated_type_equality_checks
-    - valid_regexps
+  language:
+    strict-casts: true
+    strict-inference: true
+    strict-raw-types: true
+
+# linter:
+#   rules:
+#     - camel_case_types
+
+# For more information about the core and recommended set of lints, see
+# https://dart.dev/go/core-lints
+
+# For additional information about configuring this file, see
+# https://dart.dev/guides/language/analysis-options

simple_blocs/lib/src/models/todo.dart

@@ -9,11 +9,10 @@ class Todo {
   final String note;
   final String task;
 
-  Todo(this.task, {this.complete = false, String note = '', String id})
-    : this.note = note ?? '',
-      this.id = id ?? Uuid().generateV4();
+  Todo(this.task, {this.complete = false, this.note = '', String? id})
+    : id = id ?? Uuid().generateV4();
 
-  Todo copyWith({bool complete, String id, String note, String task}) {
+  Todo copyWith({bool? complete, String? id, String? note, String? task}) {
     return Todo(
       task ?? this.task,
       complete: complete ?? this.complete,
@@ -48,9 +47,9 @@ class Todo {
   static Todo fromEntity(TodoEntity entity) {
     return Todo(
       entity.task,
-      complete: entity.complete ?? false,
+      complete: entity.complete,
       note: entity.note,
-      id: entity.id ?? Uuid().generateV4(),
+      id: entity.id,
     );
   }
 }

simple_blocs/lib/src/todos_interactor.dart

@@ -1,5 +1,7 @@
 import 'dart:async';
 
+import 'package:collection/collection.dart';
+import 'package:rxdart/rxdart.dart';
 import 'package:simple_blocs/simple_blocs.dart';
 import 'package:todos_repository_core/todos_repository_core.dart';
 
@@ -21,10 +23,8 @@ class TodosInteractor {
 
   Stream<Todo> todo(String id) {
     return todos
-        .map((todos) {
-          return todos.firstWhere((todo) => todo.id == id, orElse: () => null);
-        })
-        .where((todo) => todo != null);
+        .map<Todo?>((todos) => todos.firstWhereOrNull((todo) => todo.id == id))
+        .whereNotNull();
   }
 
   Stream<bool> get allComplete => todos.map(_allComplete);

simple_blocs/lib/src/todos_list_bloc.dart

@@ -50,7 +50,6 @@ class TodosListBloc {
         case VisibilityFilter.completed:
           return todo.complete;
         case VisibilityFilter.all:
-        default:
           return true;
       }
     }).toList();

simple_blocs/pubspec.yaml

@@ -1,14 +1,18 @@
 name: simple_blocs
 description: The Business Logic Components for a Todo App - Simplfied
+version: 1.0.0
+publish_to: "none"
 
 environment:
-  sdk: '>=2.0.0-dev.28.0 <3.0.0'
+  sdk: ^3.9.0
 
 dependencies:
+  collection: ^1.15.0
+  rxdart: ^0.28.0
   todos_repository_core:
     path: ../todos_repository_core
-  rxdart: ^0.23.1
 
 dev_dependencies:
-  test:
-  mockito:
+  lints: ^6.0.0
+  test: ^1.25.6
+  mockito: ^5.5.0