Merge pull request #20 from JunbaeJs/enabled

fix: working enabled, disabled

package.json

@@ -2,7 +2,7 @@
   "name": "junbae-mode",
   "displayName": "Junbae Mode",
   "icon": "images/junbae-logo.png",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "Junbae is caique parrot!",
   "main": "./dist/extension.js",
   "publisher": "JunbaeJs",

src/core/main.ts

@@ -6,7 +6,7 @@ const configurationSection = ['enabled'] as const;
 type ConfigurationSection = (typeof configurationSection)[number];
 export class JunbaeMode {
   // configurations
-  enabled: boolean = vscode.workspace.getConfiguration('junbae-mode').get('enabled') ?? true;
+  enabled: boolean = true;
 
   modes: Mode[] = [];
 
@@ -40,6 +40,24 @@ export class JunbaeMode {
   }
 
   onDidChangeTextDocument = (event: vscode.TextDocumentChangeEvent) => {
-    this.modes.forEach((mode) => mode.onDidChangeTextDocument(event));
+    if (this.enabled) {
+      this.modes.forEach((mode) => mode.onDidChangeTextDocument(event));
+    }
   };
+
+  /**
+   * Reset when configuration changed
+   */
+  onDidChangeConfiguration = () => {
+    this.reset();
+  };
+
+  private reset() {
+    this.dispose();
+    this.enabled = vscode.workspace.getConfiguration('junbae-mode').get('enabled')!;
+  }
+
+  private dispose() {
+    this.modes.forEach((mode) => mode.dispose());
+  }
 }

src/core/modes/mode.interface.ts

@@ -2,4 +2,6 @@ import * as vscode from 'vscode';
 
 export interface Mode {
   onDidChangeTextDocument(event: vscode.TextDocumentChangeEvent): void;
+
+  dispose(): void;
 }

src/extension.ts

@@ -15,6 +15,7 @@ export function activate(context: vscode.ExtensionContext) {
   const enable = vscode.commands.registerCommand('junbae-mode.enable', () => mode.setEnabled(true));
   const disable = vscode.commands.registerCommand('junbae-mode.disable', () => mode.setEnabled(false));
   vscode.workspace.onDidChangeTextDocument(mode.onDidChangeTextDocument);
+  vscode.workspace.onDidChangeConfiguration(mode.onDidChangeConfiguration);
 
   context.subscriptions.push(enable);
   context.subscriptions.push(disable);