ensure only fields of types annotated with BuiltValue are nested builders

Author: eEQK

goldens/foo/lib/built_value/built_value_test.dart

@@ -54,17 +54,33 @@ void main() {
             b
               ..aPrimitiveFields.anInt = 3
               ..aPrimitiveFields.aString = 'four'
-              ..aString = 'five',
+              ..aString = 'five'
+              ..stringWrapper = StringWraper('six'),
       );
       expect(
         value.toString(),
         'NestedFields(aPrimitiveFields: PrimitiveFields('
-        'anInt: 3, aString: four, aNullableString: null), aString: five)',
+        'anInt: 3, aString: four, aNullableString: null), '
+        'stringWrapper: StringWrapper(aString: six), aString: five)',
       );
     });
   });
 }
 
+class NonMacro {
+  const NonMacro();
+}
+
+@NonMacro()
+class StringWraper {
+  const StringWraper(this.aString);
+
+  final String aString;
+
+  @override
+  String toString() => 'StringWrapper(aString: $aString)';
+}
+
 @BuiltValue()
 class Empty {}
 
@@ -78,5 +94,6 @@ class PrimitiveFields {
 @BuiltValue()
 class NestedFields {
   final PrimitiveFields aPrimitiveFields;
+  final StringWraper stringWrapper;
   final String aString;
 }

pkgs/_test_macros/lib/built_value.dart

@@ -3,6 +3,7 @@
 // BSD-style license that can be found in the LICENSE file.
 
 import 'dart:async';
+import 'dart:io';
 
 import 'package:dart_model/dart_model.dart';
 // ignore: implementation_imports
@@ -183,8 +184,20 @@ class BuiltValueBuilderImplementation implements ClassDeclarationsMacro {
             TypeAnnotationType.namedTypeAnnotation) {
           continue;
         }
-        // TODO(davidmorgan): macro metadata model doesn't actually have the
-        // name yet, just assume any constructor annotation is `BuiltValue`.
+
+        final namedTypeAnnotation =
+            constructorInvocation.type.asNamedTypeAnnotation;
+        if (namedTypeAnnotation.reference.type !=
+            ReferenceType.classReference) {
+          continue;
+        }
+
+        final constructorReference =
+            namedTypeAnnotation.reference.asClassReference;
+        if (constructorReference.name != 'BuiltValue') {
+          continue;
+        }
+
         nestedBuilderTypes.add(qualifiedName.asString);
       }
     }