Pass target model in AugmentRequest

pkgs/_analyzer_macros/lib/macro_implementation.dart

@@ -81,6 +81,17 @@ class AnalyzerRunningMacro implements injected.RunningMacro {
     return AnalyzerRunningMacro._(impl, name, implementation);
   }
 
+  /// Queries for [target] and returns the [Model] representing the result.
+  ///
+  /// TODO: Make this a more limited query which doesn't fetch members.
+  Future<Model> _queryTarget(QualifiedName target) =>
+      Scope.query.run(() async => (await _impl._host.hostService.handle(
+              MacroRequest.queryRequest(
+                  QueryRequest(query: Query(target: target)),
+                  id: nextRequestId)))
+          .asQueryResponse
+          .model);
+
   @override
   Future<AnalyzerMacroExecutionResult> executeDeclarationsPhase(
       macros_api_v1.MacroTarget target,
@@ -91,7 +102,11 @@ class AnalyzerRunningMacro implements injected.RunningMacro {
     return await AnalyzerMacroExecutionResult.dartModelToInjected(
         target,
         await _impl._host.augment(
-            name, AugmentRequest(phase: 2, target: target.qualifiedName)));
+            name,
+            AugmentRequest(
+                phase: 2,
+                target: target.qualifiedName,
+                model: await _queryTarget(target.qualifiedName))));
   }
 
   @override
@@ -104,7 +119,11 @@ class AnalyzerRunningMacro implements injected.RunningMacro {
     return await AnalyzerMacroExecutionResult.dartModelToInjected(
         target,
         await _impl._host.augment(
-            name, AugmentRequest(phase: 3, target: target.qualifiedName)));
+            name,
+            AugmentRequest(
+                phase: 3,
+                target: target.qualifiedName,
+                model: await _queryTarget(target.qualifiedName))));
   }
 
   @override
@@ -116,7 +135,11 @@ class AnalyzerRunningMacro implements injected.RunningMacro {
     return await AnalyzerMacroExecutionResult.dartModelToInjected(
         target,
         await _impl._host.augment(
-            name, AugmentRequest(phase: 1, target: target.qualifiedName)));
+            name,
+            AugmentRequest(
+                phase: 1,
+                target: target.qualifiedName,
+                model: await _queryTarget(target.qualifiedName))));
   }
 }
 

pkgs/_analyzer_macros/lib/query_service.dart

@@ -36,52 +36,56 @@ class AnalyzerQueryService implements QueryService {
       ..addInterfaceElement(clazz);
 
     final interface = Interface(properties: Properties(isClass: true));
-    for (final constructor in clazz.constructors) {
-      interface.members[constructor.name] = Member(
-          requiredPositionalParameters: constructor
-              .requiredPositionalParameters(types.translator, types.context),
-          optionalPositionalParameters: constructor
-              .optionalPositionalParameters(types.translator, types.context),
-          namedParameters:
-              constructor.namedParameters(types.translator, types.context),
-          properties: Properties(
-            isAbstract: constructor.isAbstract,
-            isConstructor: true,
-            isGetter: false,
-            isField: false,
-            isMethod: false,
-            isStatic: false,
-          ));
-    }
-    for (final field in clazz.fields) {
-      interface.members[field.name] = Member(
-          properties: Properties(
-            isAbstract: field.isAbstract,
-            isConstructor: false,
-            isGetter: false,
-            isField: true,
-            isMethod: false,
-            isStatic: field.isStatic,
-          ),
-          returnType: types.addDartType(field.type));
-    }
-    for (final method in clazz.methods) {
-      interface.members[method.name] = Member(
-          requiredPositionalParameters: method.requiredPositionalParameters(
-              types.translator, types.context),
-          optionalPositionalParameters: method.optionalPositionalParameters(
-              types.translator, types.context),
-          namedParameters:
-              method.namedParameters(types.translator, types.context),
-          properties: Properties(
-            isAbstract: method.isAbstract,
-            isConstructor: false,
-            isGetter: false,
-            isField: false,
-            isMethod: true,
-            isStatic: method.isStatic,
-          ),
-          returnType: types.addDartType(method.returnType));
+    try {
+      for (final constructor in clazz.constructors) {
+        interface.members[constructor.name] = Member(
+            requiredPositionalParameters: constructor
+                .requiredPositionalParameters(types.translator, types.context),
+            optionalPositionalParameters: constructor
+                .optionalPositionalParameters(types.translator, types.context),
+            namedParameters:
+                constructor.namedParameters(types.translator, types.context),
+            properties: Properties(
+              isAbstract: constructor.isAbstract,
+              isConstructor: true,
+              isGetter: false,
+              isField: false,
+              isMethod: false,
+              isStatic: false,
+            ));
+      }
+      for (final field in clazz.fields) {
+        interface.members[field.name] = Member(
+            properties: Properties(
+              isAbstract: field.isAbstract,
+              isConstructor: false,
+              isGetter: false,
+              isField: true,
+              isMethod: false,
+              isStatic: field.isStatic,
+            ),
+            returnType: types.addDartType(field.type));
+      }
+      for (final method in clazz.methods) {
+        interface.members[method.name] = Member(
+            requiredPositionalParameters: method.requiredPositionalParameters(
+                types.translator, types.context),
+            optionalPositionalParameters: method.optionalPositionalParameters(
+                types.translator, types.context),
+            namedParameters:
+                method.namedParameters(types.translator, types.context),
+            properties: Properties(
+              isAbstract: method.isAbstract,
+              isConstructor: false,
+              isGetter: false,
+              isField: false,
+              isMethod: true,
+              isStatic: method.isStatic,
+            ),
+            returnType: types.addDartType(method.returnType));
+      }
+    } catch (_) {
+      // TODO: Fails in types phase, implement fine grained queries.
     }
     return Model(types: types.typeHierarchy)
       ..uris[uri] = (Library()..scopes[clazz.name] = interface);

pkgs/_cfe_macros/lib/macro_implementation.dart

@@ -81,6 +81,17 @@ class CfeRunningMacro implements injected.RunningMacro {
     return CfeRunningMacro._(impl, name, implementation);
   }
 
+  /// Queries for [target] and returns the [Model] representing the result.
+  ///
+  /// TODO: Make this a more limited query which doesn't fetch members.
+  Future<Model> _queryTarget(QualifiedName target) =>
+      Scope.query.run(() async => (await _impl._host.hostService.handle(
+              MacroRequest.queryRequest(
+                  QueryRequest(query: Query(target: target)),
+                  id: nextRequestId)))
+          .asQueryResponse
+          .model);
+
   @override
   Future<CfeMacroExecutionResult> executeDeclarationsPhase(
       macros_api_v1.MacroTarget target,
@@ -91,7 +102,11 @@ class CfeRunningMacro implements injected.RunningMacro {
     return await CfeMacroExecutionResult.dartModelToInjected(
         target,
         await _impl._host.augment(
-            name, AugmentRequest(phase: 2, target: target.qualifiedName)));
+            name,
+            AugmentRequest(
+                phase: 2,
+                target: target.qualifiedName,
+                model: await _queryTarget(target.qualifiedName))));
   }
 
   @override
@@ -104,7 +119,11 @@ class CfeRunningMacro implements injected.RunningMacro {
     return await CfeMacroExecutionResult.dartModelToInjected(
         target,
         await _impl._host.augment(
-            name, AugmentRequest(phase: 3, target: target.qualifiedName)));
+            name,
+            AugmentRequest(
+                phase: 3,
+                target: target.qualifiedName,
+                model: await _queryTarget(target.qualifiedName))));
   }
 
   @override
@@ -116,7 +135,11 @@ class CfeRunningMacro implements injected.RunningMacro {
     return await CfeMacroExecutionResult.dartModelToInjected(
         target,
         await _impl._host.augment(
-            name, AugmentRequest(phase: 1, target: target.qualifiedName)));
+            name,
+            AugmentRequest(
+                phase: 1,
+                target: target.qualifiedName,
+                model: await _queryTarget(target.qualifiedName))));
   }
 }
 

pkgs/_cfe_macros/lib/query_service.dart

@@ -57,8 +57,13 @@ class CfeQueryService implements QueryService {
         isStatic: current.isStatic,
       ));
     }
-
-    return Model(types: _buildTypeHierarchy({classBuilder.actualCls}))
+    TypeHierarchy? types;
+    try {
+      types = _buildTypeHierarchy({classBuilder.actualCls});
+    } catch (_) {
+      // TODO: Fails in types phase, implement fine grained queries.
+    }
+    return Model(types: types)
       ..uris[uri] = (Library()
         ..
             // TODO(davidmorgan): return more than just fields.

pkgs/_macro_client/lib/macro_client.dart

@@ -114,27 +114,29 @@ class MacroClient {
                       '${hostRequest.macroAnnotation.asString}')));
         } else {
           final augmentRequest = hostRequest.asAugmentRequest;
-          await Scope.macro
-              .runAsync(() async => _sendResponse(Response.augmentResponse(
-                  switch (augmentRequest.phase) {
-                        1 => macro.description.runsInPhases.contains(1)
-                            ? await executeTypesMacro(
-                                macro, _host, augmentRequest)
-                            : null,
-                        2 => macro.description.runsInPhases.contains(2)
-                            ? await executeDeclarationsMacro(
-                                macro, _host, augmentRequest)
-                            : null,
-                        3 => macro.description.runsInPhases.contains(3)
-                            ? await executeDefinitionsMacro(
-                                macro, _host, augmentRequest)
-                            : null,
-                        _ => throw StateError(
-                            'Unexpected phase ${augmentRequest.phase}, '
-                            'expected 1, 2, or 3.')
-                      } ??
-                      AugmentResponse(),
-                  requestId: hostRequest.id)));
+          await Scope.macro.runAsync(() async {
+            MacroScope.current.addModel(augmentRequest.model);
+            return _sendResponse(Response.augmentResponse(
+                switch (augmentRequest.phase) {
+                      1 => macro.description.runsInPhases.contains(1)
+                          ? await executeTypesMacro(
+                              macro, _host, augmentRequest)
+                          : null,
+                      2 => macro.description.runsInPhases.contains(2)
+                          ? await executeDeclarationsMacro(
+                              macro, _host, augmentRequest)
+                          : null,
+                      3 => macro.description.runsInPhases.contains(3)
+                          ? await executeDefinitionsMacro(
+                              macro, _host, augmentRequest)
+                          : null,
+                      _ => throw StateError(
+                          'Unexpected phase ${augmentRequest.phase}, '
+                          'expected 1, 2, or 3.')
+                    } ??
+                    AugmentResponse(),
+                requestId: hostRequest.id));
+          });
         }
       default:
       // Ignore unknown request.

pkgs/_macro_client/lib/src/execute_macro.dart

@@ -12,13 +12,12 @@ import 'package:macro_service/macro_service.dart';
 Future<AugmentResponse> executeTypesMacro(
     Macro macro, Host host, AugmentRequest request) async {
   final target = request.target;
-  // TODO: https://github.com/dart-lang/macros/issues/100.
-  final queryResult = await host.query(Query(target: target));
-  final properties =
-      queryResult.uris[target.uri]!.scopes[target.name]!.properties;
-  switch ((properties, macro)) {
+  final model = request.model;
+  final interface = model.uris[target.uri]!.scopes[target.name]!;
+
+  switch ((interface.properties, macro)) {
     case (Properties(isClass: true), ClassTypesMacro macro):
-      return await macro.buildTypesForClass(host, request);
+      return await macro.buildTypesForClass(interface, model, host);
     case (_, LibraryTypesMacro()):
     case (_, ConstructorTypesMacro()):
     case (_, MethodTypesMacro()):
@@ -42,14 +41,12 @@ Future<AugmentResponse> executeTypesMacro(
 Future<AugmentResponse> executeDeclarationsMacro(
     Macro macro, Host host, AugmentRequest request) async {
   final target = request.target;
-  // TODO: https://github.com/dart-lang/macros/issues/100.
-  final queryResult = await host.query(Query(target: target));
-  final properties =
-      queryResult.uris[target.uri]!.scopes[target.name]!.properties;
+  final model = request.model;
+  final interface = model.uris[target.uri]!.scopes[target.name]!;
 
-  switch ((properties, macro)) {
+  switch ((interface.properties, macro)) {
     case (Properties(isClass: true), ClassDeclarationsMacro macro):
-      return await macro.buildDeclarationsForClass(host, request);
+      return await macro.buildDeclarationsForClass(interface, model, host);
     case (_, LibraryDeclarationsMacro()):
     case (_, EnumDeclarationsMacro()):
     case (_, ExtensionDeclarationsMacro()):
@@ -73,14 +70,12 @@ Future<AugmentResponse> executeDeclarationsMacro(
 Future<AugmentResponse> executeDefinitionsMacro(
     Macro macro, Host host, AugmentRequest request) async {
   final target = request.target;
-  // TODO: https://github.com/dart-lang/macros/issues/100.
-  final queryResult = await host.query(Query(target: target));
-  final properties =
-      queryResult.uris[target.uri]!.scopes[target.name]!.properties;
+  final model = request.model;
+  final interface = model.uris[target.uri]!.scopes[target.name]!;
 
-  switch ((properties, macro)) {
+  switch ((interface.properties, macro)) {
     case (Properties(isClass: true), ClassDefinitionsMacro macro):
-      return await macro.buildDefinitionsForClass(host, request);
+      return await macro.buildDefinitionsForClass(interface, model, host);
     case (_, LibraryDefinitionsMacro()):
     case (_, EnumDefinitionsMacro()):
     case (_, ExtensionDefinitionsMacro()):