analyzer plugins: Support AnalysisRuleVisitor.afterLibrary

srawlins · Commit Queue · commit 21e5638140a1 · 2025-12-09T13:21:29.000-08:00
Fixes #62010 Change-Id: If0652125d1aefc819c996191869e0b0d8635c774 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/467140 Reviewed-by: Brian Wilkerson <brianwilkerson@google.com> Commit-Queue: Samuel Rawlins <srawlins@google.com>
diff --git a/pkg/analysis_server_plugin/lib/src/plugin_server.dart b/pkg/analysis_server_plugin/lib/src/plugin_server.dart
@@ -467,6 +467,10 @@ class PluginServer {
           AnalysisRuleVisitor(nodeRegistry, shouldPropagateExceptions: true),
         );
       }
+
+      // Now that all lint rules have visited the code in each of the compilation
+      // units, we can accept each lint rule's `afterLibrary` hook.
+      AnalysisRuleVisitor(nodeRegistry).afterLibrary();
     }
 
     // TODO(srawlins): Support `AnalysisRuleVisitor.afterLibrary`. See how it is

Original file line number	Diff line number	Diff line change
`@@ -467,6 +467,10 @@ class PluginServer {`
`467`	`467`	`AnalysisRuleVisitor(nodeRegistry, shouldPropagateExceptions: true),`
`468`	`468`	`);`
`469`	`469`	`}`
	`470`	`+`
	`471`	`+ // Now that all lint rules have visited the code in each of the compilation`
	`472`	+ // units, we can accept each lint rule's `afterLibrary` hook.
	`473`	`+ AnalysisRuleVisitor(nodeRegistry).afterLibrary();`
`470`	`474`	`}`
`471`	`475`
`472`	`476`	// TODO(srawlins): Support `AnalysisRuleVisitor.afterLibrary`. See how it is