[_js_interop_checks] Update min SDK version to 3.7.0-0 and format

Uses the latest formatter changes.

PS2 fixes some string wrapping to adapt to the new changes.

PS3 removes an unnecessary null assertions with the latest SDK.

Change-Id: I33387c89395a52a2bc5bc261f29a20a7c9b5fd09
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/400722
Reviewed-by: Mayank Patke <[email protected]>
Commit-Queue: Mayank Patke <[email protected]>
Auto-Submit: Srujan Gaddam <[email protected]>

Author: srujzs

pkg/_js_interop_checks/lib/js_interop_checks.dart

@@ -118,8 +118,11 @@ final _jsInterop = Uri.parse('dart:js_interop');
 ///
 /// If [interopLibraries] is null, we check `package:js`,
 /// `dart:_js_annotations`, and `dart:js_interop`.
-bool _isInteropAnnotation(Expression value, String annotationClassName,
-    {Set<Uri>? interopLibraries}) {
+bool _isInteropAnnotation(
+  Expression value,
+  String annotationClassName, {
+  Set<Uri>? interopLibraries,
+}) {
   interopLibraries ??= {_packageJs, _jsAnnotations, _jsInterop};
   var c = annotationClass(value);
   if (c == null || c.name != annotationClassName) return false;
@@ -130,9 +133,11 @@ bool _isInteropAnnotation(Expression value, String annotationClassName,
 bool _isJSInteropAnnotation(Expression value) =>
     _isInteropAnnotation(value, 'JS');
 
-bool _isPackageJSAnnotation(Expression value) =>
-    _isInteropAnnotation(value, 'JS',
-        interopLibraries: {_packageJs, _jsAnnotations});
+bool _isPackageJSAnnotation(Expression value) => _isInteropAnnotation(
+  value,
+  'JS',
+  interopLibraries: {_packageJs, _jsAnnotations},
+);
 
 bool _isDartJSInteropAnnotation(Expression value) =>
     _isInteropAnnotation(value, 'JS', interopLibraries: {_jsInterop});
@@ -211,15 +216,19 @@ List<String> stringAnnotationValues(Expression node) {
         var value = constant.fieldValues.values.elementAt(0);
         if (value is StringConstant) values.addAll(value.value.split(','));
       } else if (argLength > 1) {
-        throw ArgumentError('Method expects annotation with at most one '
-            'positional argument: $node.');
+        throw ArgumentError(
+          'Method expects annotation with at most one positional argument: '
+          '$node.',
+        );
       }
     }
   } else if (node is ConstructorInvocation) {
     var argLength = node.arguments.positional.length;
     if (argLength > 1 || node.arguments.named.isNotEmpty) {
-      throw ArgumentError('Method expects annotation with at most one '
-          'positional argument: $node.');
+      throw ArgumentError(
+        'Method expects annotation with at most one positional argument: '
+        '$node.',
+      );
     } else if (argLength == 1) {
       var value = node.arguments.positional[0];
       if (value is StringLiteral) {
@@ -267,17 +276,21 @@ Library? _findJsInteropLibrary(Component component, Uri interopUri) {
 /// `calculateTransitiveImportsOfDartFfiIfUsed` in
 /// pkg/vm/lib/transformations/ffi/common.dart.
 Set<Library> calculateTransitiveImportsOfJsInteropIfUsed(
-    Component component, Uri interopUri) {
+  Component component,
+  Uri interopUri,
+) {
   // Check for the presence of [jsInteropLibrary] as a dependency of any of the
   // libraries in [component]. We use this to bypass the expensive
   // [calculateTransitiveDependenciesOf] call for cases where js interop is
   // not used, otherwise we could just use the index of the library instead.
   Library? jsInteropLibrary = _findJsInteropLibrary(component, interopUri);
   if (jsInteropLibrary == null) return const <Library>{};
 
-  kernel_graph.LibraryGraph graph =
-      kernel_graph.LibraryGraph(component.libraries);
-  Set<Library> result =
-      kernel_graph.calculateTransitiveDependenciesOf(graph, {jsInteropLibrary});
+  kernel_graph.LibraryGraph graph = kernel_graph.LibraryGraph(
+    component.libraries,
+  );
+  Set<Library> result = kernel_graph.calculateTransitiveDependenciesOf(graph, {
+    jsInteropLibrary,
+  });
   return result;
 }

pkg/_js_interop_checks/lib/src/js_interop.dart

@@ -15,11 +15,7 @@ import 'package:_js_interop_checks/js_interop_checks.dart'
 import 'package:_js_interop_checks/src/js_interop.dart' as js_interop;
 import 'package:kernel/ast.dart';
 
-enum ExportStatus {
-  exportError,
-  exportable,
-  nonExportable,
-}
+enum ExportStatus { exportError, exportable, nonExportable }
 
 class ExportChecker {
   final JsInteropDiagnosticReporter _diagnosticReporter;
@@ -85,7 +81,7 @@ class ExportChecker {
     var demangledCls = cls.isMixinApplication ? cls.mixin : cls;
     for (var member in [
       ...demangledCls.procedures.where((proc) => proc.exportable),
-      ...demangledCls.fields.where((field) => field.exportable)
+      ...demangledCls.fields.where((field) => field.exportable),
     ]) {
       var memberName = member.name.text;
       if (member is Procedure && member.isSetter) {
@@ -122,23 +118,25 @@ class ExportChecker {
 
     if (classHasJSExport && js_interop.getJSExportName(cls).isNotEmpty) {
       _diagnosticReporter.report(
-          templateJsInteropExportDartInterfaceHasNonEmptyJSExportValue
-              .withArguments(cls.name),
-          cls.fileOffset,
-          cls.name.length,
-          cls.location?.file);
+        templateJsInteropExportDartInterfaceHasNonEmptyJSExportValue
+            .withArguments(cls.name),
+        cls.fileOffset,
+        cls.name.length,
+        cls.location?.file,
+      );
       exportStatus[cls.reference] = ExportStatus.exportError;
     }
 
     _collectOverrides(cls);
 
     var allExportableMembers = _overrideMap[cls.reference]!.values.where(
-        (member) =>
-            // Only members that qualify are those that are exportable, and
-            // either their class has the annotation or they have it themselves.
-            member.exportable &&
-            (js_interop.hasJSExportAnnotation(member) ||
-                js_interop.hasJSExportAnnotation(member.enclosingClass!)));
+      (member) =>
+          // Only members that qualify are those that are exportable, and
+          // either their class has the annotation or they have it themselves.
+          member.exportable &&
+          (js_interop.hasJSExportAnnotation(member) ||
+              js_interop.hasJSExportAnnotation(member.enclosingClass!)),
+    );
     var exports = <String, Set<Member>>{};
 
     // Store the exportable members.
@@ -183,20 +181,24 @@ class ExportChecker {
       var sortedExistingMembers =
           existingMembers.map((member) => member.toString()).toList()..sort();
       _diagnosticReporter.report(
-          templateJsInteropExportMemberCollision.withArguments(
-              exportName, sortedExistingMembers.join(', ')),
-          cls.fileOffset,
-          cls.name.length,
-          cls.location?.file);
+        templateJsInteropExportMemberCollision.withArguments(
+          exportName,
+          sortedExistingMembers.join(', '),
+        ),
+        cls.fileOffset,
+        cls.name.length,
+        cls.location?.file,
+      );
       exportStatus[cls.reference] = ExportStatus.exportError;
     }
 
     if (exports.isEmpty) {
       _diagnosticReporter.report(
-          templateJsInteropExportNoExportableMembers.withArguments(cls.name),
-          cls.fileOffset,
-          cls.name.length,
-          cls.location?.file);
+        templateJsInteropExportNoExportableMembers.withArguments(cls.name),
+        cls.fileOffset,
+        cls.name.length,
+        cls.location?.file,
+      );
       exportStatus[cls.reference] = ExportStatus.exportError;
     }
 
@@ -212,11 +214,13 @@ class ExportChecker {
     if (memberHasJSExportAnnotation) {
       if (!member.exportable) {
         _diagnosticReporter.report(
-            templateJsInteropExportDisallowedMember
-                .withArguments(member.name.text),
-            member.fileOffset,
-            member.name.text.length,
-            member.location?.file);
+          templateJsInteropExportDisallowedMember.withArguments(
+            member.name.text,
+          ),
+          member.fileOffset,
+          member.name.text.length,
+          member.location?.file,
+        );
         if (cls != null) {
           exportStatus[cls.reference] = ExportStatus.exportError;
         }