fix(mix_generator): Respect nullability in lerp and update deps (#849)

Author: tilucasoli

packages/mix_generator/lib/src/core/resolvers/lerp_resolver.dart

@@ -48,11 +48,16 @@ class LerpResolver {
   String generateLerpCode(FieldModel field) {
     final strategy = resolveStrategy(field);
     final name = field.name;
+    final isNullable = field.isNullable;
 
     return switch (strategy) {
       .interpolate => 'MixOps.lerp($name, other?.$name, t)',
       .snap => 'MixOps.lerpSnap($name, other?.$name, t)',
-      .delegateToSpec => '$name?.lerp(other?.$name, t)',
+      // Use ?. for nullable fields, . for non-nullable fields
+      .delegateToSpec =>
+        isNullable
+            ? '$name?.lerp(other?.$name, t)'
+            : '$name.lerp(other?.$name, t)',
     };
   }
 }

packages/mix_generator/pubspec.yaml

@@ -9,12 +9,12 @@ environment:
 
 dependencies:
   mix_annotations: ^1.7.0
-  build: ^4.0.4
-  source_gen: ^4.2.0
-  analyzer: ^10.0.1
-  dart_style: ^3.1.4
-  collection: ^1.17.1
-  build_config: ^1.2.0
+  dart_style: ^3.0.0
+  collection: ^1.15.0
+  source_gen: ">=3.0.0 <5.0.0"
+  analyzer: '>=9.0.0 <11.0.0'
+  build: ">=3.0.0 <5.0.0"
+  build_config: ^1.1.0
   logging: ^1.2.0
   path: ^1.8.3
 

packages/mix_generator/test/core/resolvers/lerp_resolver_test.dart

@@ -37,9 +37,14 @@ void main() {
         );
       });
 
-      test('delegate pattern uses direct lerp call', () {
+      test('delegate pattern for nullable field uses ?. operator', () {
         expect('box?.lerp(other?.box, t)', contains('?.lerp('));
       });
+
+      test('delegate pattern for non-nullable field uses . operator', () {
+        expect('box.lerp(other?.box, t)', isNot(contains('?.lerp(')));
+        expect('box.lerp(other?.box, t)', contains('.lerp('));
+      });
     });
   });
 }

pubspec.yaml

@@ -6,4 +6,4 @@ environment:
 
 dependencies:
   melos: ^6.3.3
-  analyzer: ^7.5.4
+  analyzer: '>=9.0.0 <11.0.0'