Performance improvements

Author: theletterf

package.json

@@ -2,7 +2,7 @@
   "name": "elastic-docs-v3-utilities",
   "displayName": "Elastic Docs Utilities",
   "description": "Utilities for Elastic Docs authoring",
-  "version": "0.11.12",
+  "version": "0.11.13",
   "publisher": "Elastic",
   "repository": {
     "type": "git",
@@ -16,8 +16,7 @@
   ],
   "icon": "images/icon.png",
   "activationEvents": [
-    "onLanguage:markdown",
-    "onStartupFinished"
+    "onLanguage:markdown"
   ],
   "main": "./out/extension.js",
   "browser": "./out/extension-web.js",

src/extension.ts

@@ -247,6 +247,17 @@ export function activate(context: vscode.ExtensionContext): void {
 function applyColorCustomizations(): void {
     const config = vscode.workspace.getConfiguration('editor');
     const currentCustomizations = config.get('tokenColorCustomizations') as Record<string, unknown> || {};
+    const existingRules = (currentCustomizations.textMateRules as Array<{ scope?: string }>) || [];
+
+    // Check if our rules are already applied by looking for a known elastic scope
+    const elasticRulesAlreadyApplied = existingRules.some(
+        rule => rule.scope && rule.scope.includes('.elastic')
+    );
+
+    if (elasticRulesAlreadyApplied) {
+        // Rules already exist, skip the expensive settings write
+        return;
+    }
 
     // Define our custom color rules
     const elasticRules = [
@@ -327,11 +338,9 @@ function applyColorCustomizations(): void {
         }
     ];
 
-    // Merge with existing rules
-    const existingRules = (currentCustomizations.textMateRules as unknown[]) || [];
+    // Merge with existing rules and apply (only on first activation)
     const newRules = [...existingRules, ...elasticRules];
 
-    // Apply the customizations
     config.update('tokenColorCustomizations', {
         ...currentCustomizations,
         textMateRules: newRules