Remove redundant config watching

Author: lionel-

apps/lsp/src/config.ts

@@ -172,8 +172,9 @@ export class ConfigurationManager extends Disposable {
   }
 
   public async subscribe() {
-    // Ignore the settings in parameters, the modern usage is to fetch the settings
-    // when we get this notification
+    // Ignore the settings in parameters, the modern usage is to fetch the
+    // settings when we get this notification. This causes the client to send
+    // any updates for settings under the `quarto` section.
     this.connection_.onDidChangeConfiguration((_params) => {
       this._logger.logNotification('didChangeConfiguration');
       this.update();

apps/vscode/src/lsp/client.ts

@@ -147,17 +147,6 @@ export async function activateLsp(
     clientOptions
   );
 
-  // Notify LSP of quarto setting changes
-  context.subscriptions.push(workspace.onDidChangeConfiguration(e => {
-    if (client.state !== State.Running) {
-      return;
-    }
-
-    if (e.affectsConfiguration("quarto")) {
-      client.sendNotification("workspace/didChangeConfiguration", {});
-    }
-  }));
-
   // return once the server is running
   return new Promise<LanguageClient>((resolve, reject) => {