Repository navigation
"Preview/Fix All in Workspace" and the "Fix All [in file]" LSP commands/actions do not include Pubspec fixes #64030
Description
If you have:
import 'package:path/path.dart' as path;But path is not listed in the Pubspec dependencies, dart fix will fix it for you. However, the "Fix All in Workspace" command does not. Given it's called "Fix All", it probably should.
Activity
DanTup commentedon Aug 12, 2026
A related question:
In the situation above, if I run the normal "Fix All" command/code action which applies fixes in the current file - should it also fix the Pubspec? The diagnostic is in the current file, but the fix is not.
@bwilkerson my feeling is that it probably should.
BulkFixProcessor.fixPubspec()currently enumerates all files and tries to fix them all, but we could optionally pass in a single file and only consider that one. WDYT?bwilkerson commentedon Aug 13, 2026
I agree, it seems like "Fix All" should fix all of the diagnostics in that file even if it has to make changes in other files in order to do so.
[-]"Preview/Fix All in Workspace" LSP command does not include Pubspec fixes[/-][+]"Preview/Fix All in Workspace" and the "Fix All [in file]" LSP commands/actions do not include Pubspec fixes[/+][analysis_server] Include Pubspec fixes in the LSP "Fix All" command/…
[analysis_server] Include Pubspec fixes in the LSP "Fix All in Worksp…