feat: don't touch dart.runPubGetOnPubspecChanges when usePubspecOverrides is enabled

src/settings.ts

@@ -1,6 +1,7 @@
 import * as vscode from 'vscode'
 import { error, info } from './logging'
 import { melosWorkspaces } from './melos-workspace'
+import { loadMelosWorkspaceConfig } from './workspace-config'
 
 /**
  * Ensure that Melos workspaces have default VS Code settings.
@@ -9,14 +10,16 @@ export async function registerDefaultMelosWorkspaceSettings(
   context: vscode.ExtensionContext
 ) {
   await Promise.all(
-    melosWorkspaces.workspaceFolders.map(applyDefaultMelosWorkspaceSettings)
+    melosWorkspaces.workspaceFolders.map((folder) =>
+      applyDefaultMelosWorkspaceSettings(context, folder)
+    )
   )
 
   context.subscriptions.push(
     melosWorkspaces.onDidChangeWorkspaceFolders(async (event) => {
       for (const folder of event.added) {
         try {
-          await applyDefaultMelosWorkspaceSettings(folder)
+          await applyDefaultMelosWorkspaceSettings(context, folder)
         } catch (e) {
           error(
             `Failed to apply default Melos workspace settings in '${folder.name}' folder`,
@@ -31,13 +34,27 @@ export async function registerDefaultMelosWorkspaceSettings(
 /**
  * The default VS Code settings to apply to a Melos workspace.
  */
-const melosWorkspaceDefaultSettings = {
-  'dart.runPubGetOnPubspecChanges': false,
+const melosWorkspaceDefaultSettings: {
+  'dart.runPubGetOnPubspecChanges'?: string
+} = {
+  'dart.runPubGetOnPubspecChanges': 'never',
 }
 
 async function applyDefaultMelosWorkspaceSettings(
+  context: vscode.ExtensionContext,
   folder: vscode.WorkspaceFolder
 ) {
+  const defaultSettings = { ...melosWorkspaceDefaultSettings }
+
+  const melosConfig = await loadMelosWorkspaceConfig(context, folder)
+  if (melosConfig?.command?.bootstrap?.usePubspecOverrides === true) {
+    delete defaultSettings['dart.runPubGetOnPubspecChanges']
+  }
+
+  if (Object.keys(defaultSettings).length === 0) {
+    return
+  }
+
   const settings = vscode.workspace.getConfiguration(undefined, folder)
 
   for (const [key, value] of Object.entries(melosWorkspaceDefaultSettings)) {

src/test/default-settings/default-settings.test.ts

@@ -14,7 +14,7 @@ suite('Default settings', () => {
     await openMelosYamlInEditor()
 
     const melosWorkspaceDefaultSettings = {
-      'dart.runPubGetOnPubspecChanges': false,
+      'dart.runPubGetOnPubspecChanges': 'never',
     }
 
     // Check that all default settings have the correct values.

src/workspace-config.ts

@@ -21,12 +21,38 @@ export interface MelosWorkspaceConfig {
    * The YAML document from which the configuration was parsed.
    */
   readonly yamlDoc: Document
+
+  /**
+   * Configuration for Melos commands.
+   */
+  readonly command?: MelosCommandsConfig
+
   /**
    * The configured Melos scripts.
    */
   readonly scripts: readonly MelosScriptConfig[]
 }
 
+/**
+ * Configuration for Melos commands.
+ */
+export interface MelosCommandsConfig {
+  /**
+   * Configuration for the `melos bootstrap` command.
+   */
+  readonly bootstrap: MelosBootstrapConfig
+}
+
+/**
+ * Configuration for the `melos bootstrap` command.
+ */
+export interface MelosBootstrapConfig {
+  /**
+   * Whether Melos should use `pubspec_overrides.yaml` files to override dependencies.
+   */
+  readonly usePubspecOverrides?: boolean
+}
+
 /**
  * Configuration for a Melos script.
  */
@@ -158,6 +184,7 @@ function showInvalidMelosYamlMessage(folder: vscode.WorkspaceFolder) {
 function melosWorkspaceConfigFromYamlDoc(doc: Document): MelosWorkspaceConfig {
   return {
     yamlDoc: doc,
+    command: doc.toJSON()['command'],
     scripts: melosScriptsConfigsFromYaml(doc.get('scripts')),
   }
 }