commit 99bbf8d67ece3456fd31407930ad85046cdeb17e
author Keerti Parthasarathy <keertip@google.com> Wed Nov 15 17:08:22 2023 +0000
committer Commit Queue <dart-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Nov 15 17:08:22 2023 +0000
tree a91ec1ff79db10f72ae6642b9ceaf4f6ad0aee49
parent 444ad4daae858ea8ac07d50b4c29a4bc8b401915

Revert "Reland adding a pass to fix pubspec during dart fix runs."

This reverts commit 9186eb1b94ebe93a6e36d16cd926bae9e629bee0.

Reason for revert: https://github.com/flutter/flutter/issues/138485
Flutter roll failure

Original change's description:
> Reland adding a pass to fix pubspec during dart fix runs.
>
> The regressions were fixed with changes to MisssingDependencyValidator in previous CL's.
>
> Change-Id: Id8418811c067535e44fc58c1d2d47afb1c30716f
> Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/335864
> Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
> Commit-Queue: Keerti Parthasarathy <keertip@google.com>

Change-Id: I68cd6640b0620c24e02b9a5f8c0d659481146541
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/336460
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Auto-Submit: Keerti Parthasarathy <keertip@google.com>
Commit-Queue: Brian Wilkerson <brianwilkerson@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>

pkg/dartdev/lib/src/analysis_server.dart

diff --git a/pkg/dartdev/lib/src/analysis_server.dart b/pkg/dartdev/lib/src/analysis_server.dart
index 1648f3f..5aac198 100644
--- a/pkg/dartdev/lib/src/analysis_server.dart
+++ b/pkg/dartdev/lib/src/analysis_server.dart
@@ -199,12 +199,10 @@
   }
 
   Future<EditBulkFixesResult> requestBulkFixes(
-      String filePath, bool inTestMode, List<String> codes,
-      {bool updatePubspec = false}) {
+      String filePath, bool inTestMode, List<String> codes) {
     return _sendCommand('edit.bulkFixes', params: <String, dynamic>{
       'included': [path.canonicalize(filePath)],
       'inTestMode': inTestMode,
-      'updatePubspec': updatePubspec,
       if (codes.isNotEmpty) 'codes': codes,
     }).then((result) {
       return EditBulkFixesResult.fromJson(
@@ -214,18 +212,13 @@
 
   Future<void> shutdown({Duration? timeout}) async {
     // Request shutdown.
-    final Future<void> future =
-        _sendCommand('server.shutdown').then((Map<String, dynamic> value) {
+    final Future<void> future = _sendCommand('server.shutdown').then((Map<String, dynamic> value) {
       _shutdownResponseReceived = true;
       return;
     });
-    await (timeout != null
-            ? future.timeout(timeout, onTimeout: () {
-                log.stderr(
-                    'The analysis server timed out while shutting down.');
-              })
-            : future)
-        .whenComplete(dispose);
+    await (timeout != null ? future.timeout(timeout, onTimeout: () {
+      log.stderr('The analysis server timed out while shutting down.');
+    }) : future).whenComplete(dispose);
   }
 
   /// Send an `analysis.updateContent` request with the given [files].

pkg/dartdev/lib/src/commands/fix.dart

diff --git a/pkg/dartdev/lib/src/commands/fix.dart b/pkg/dartdev/lib/src/commands/fix.dart
index 097e421..e262901 100644
--- a/pkg/dartdev/lib/src/commands/fix.dart
+++ b/pkg/dartdev/lib/src/commands/fix.dart
@@ -151,15 +151,6 @@
         // TODO(brianwilkerson) Be more intelligent about detecting infinite
         //  loops so that we can increase [maxPasses].
       } while (pass < maxPasses && edits.isNotEmpty);
-      // If there are no more dart edits, check if there are any changes
-      // to pubspec
-      if (edits.isEmpty && detailsMap.isNotEmpty) {
-        var fixes = await server.requestBulkFixes(fixPath, inTestMode, [],
-            updatePubspec: true);
-        _mergeDetails(detailsMap, fixes.details);
-        edits = fixes.edits;
-        _applyEdits(server, edits);
-      }
       return _FixRequestResult(details: detailsMap);
     }