[go_router_builder]: Fix The argument type 'String?' can't be assigned to the parameter type 'String' because 'String?' on path and query params

packages/go_router_builder/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 2.8.2
+
+- Fixes an issue when enum params are not required
+
 ## 2.8.1
 
 - Fixes an issue when navigate to router with invalid params 

packages/go_router_builder/lib/src/route_config.dart

@@ -775,7 +775,7 @@ bool $boolConverterHelperName(String value) {
 
 const String _enumConverterHelper = '''
 extension<T extends Enum> on Map<T, String> {
-  T? $enumExtensionHelperName(String value) =>
+  T? $enumExtensionHelperName(String? value) =>
       entries.where((element) => element.value == value).firstOrNull?.key;
 }''';
 

packages/go_router_builder/lib/src/type_helpers.dart

@@ -115,11 +115,13 @@ String _stateValueAccess(ParameterElement element, Set<String> pathParameters) {
   }
 
   late String access;
+  final String suffix =
+      !element.type.isNullableType && !element.hasDefaultValue ? '!' : '';
   if (pathParameters.contains(element.name)) {
-    access =
-        'pathParameters[${escapeDartString(element.name)}]${element.isRequired ? '!' : ''}';
+    access = 'pathParameters[${escapeDartString(element.name)}]$suffix';
   } else {
-    access = 'uri.queryParameters[${escapeDartString(element.name.kebab)}]';
+    access =
+        'uri.queryParameters[${escapeDartString(element.name.kebab)}]$suffix';
   }
 
   return access;

packages/go_router_builder/pubspec.yaml

@@ -2,7 +2,7 @@ name: go_router_builder
 description: >-
   A builder that supports generated strongly-typed route helpers for
   package:go_router
-version: 2.8.1
+version: 2.8.2
 repository: https://github.com/flutter/packages/tree/main/packages/go_router_builder
 issue_tracker: https://github.com/flutter/flutter/issues?q=is%3Aissue+is%3Aopen+label%3A%22p%3A+go_router_builder%22
 

packages/go_router_builder/test_inputs/enum_parameter.dart.expect

@@ -29,6 +29,6 @@ const _$EnumTestEnumMap = {
 };
 
 extension<T extends Enum> on Map<T, String> {
-  T? _$fromName(String value) =>
+  T? _$fromName(String? value) =>
       entries.where((element) => element.value == value).firstOrNull?.key;
 }

packages/go_router_builder/test_inputs/iterable_with_enum.dart.expect

@@ -38,6 +38,6 @@ const _$EnumOnlyUsedInIterableEnumMap = {
 };
 
 extension<T extends Enum> on Map<T, String> {
-  T? _$fromName(String value) =>
+  T? _$fromName(String? value) =>
       entries.where((element) => element.value == value).firstOrNull?.key;
 }

packages/go_router_builder/test_inputs/required_query_parameter.dart.expect

@@ -7,7 +7,7 @@ extension $NonNullableRequiredParamNotInPathExtension
     on NonNullableRequiredParamNotInPath {
   static NonNullableRequiredParamNotInPath _fromState(GoRouterState state) =>
       NonNullableRequiredParamNotInPath(
-        id: int.parse(state.uri.queryParameters['id'])!,
+        id: int.parse(state.uri.queryParameters['id']!)!,
       );
 
   String get location => GoRouteData.$location(