fix

Author: bajrangCoder

src/cm/colorView.ts

@@ -213,6 +213,8 @@ class ColorViewPlugin {
 	scheduleDecorations(view: EditorView): void {
 		this.pendingView = view;
 		if (this.raf) return;
+		// Color chips are decorative, so batch rapid viewport/doc changes into
+		// one animation frame instead of rebuilding on every intermediate update.
 		this.raf = requestAnimationFrame(() => {
 			this.raf = 0;
 			const pendingView = this.pendingView;

src/cm/indentGuides.ts

@@ -367,6 +367,8 @@ function createIndentGuidesPlugin(
 			scheduleBuild(view: EditorView): void {
 				this.pendingView = view;
 				if (this.raf) return;
+				// Guide rebuilding is cosmetic and can be expensive on large
+				// viewports, so we intentionally collapse bursts into one frame.
 				this.raf = requestAnimationFrame(() => {
 					this.raf = 0;
 					const pendingView = this.pendingView;

src/cm/lsp/diagnostics.ts

@@ -8,7 +8,7 @@ import {
 	StateEffect,
 	StateField,
 } from "@codemirror/state";
-import type { EditorView } from "@codemirror/view";
+import { type EditorView, ViewPlugin } from "@codemirror/view";
 import type {
 	LSPClientWithWorkspace,
 	LSPPluginAPI,
@@ -18,7 +18,8 @@ import type {
 } from "./types";
 
 const setPublishedDiagnostics = StateEffect.define<LspDiagnostic[]>();
-let diagnosticsEventTimer = 0;
+let diagnosticsEventTimer: ReturnType<typeof setTimeout> | null = null;
+let diagnosticsViewCount = 0;
 
 export const LSP_DIAGNOSTICS_EVENT = "acode:lsp-diagnostics-updated";
 
@@ -67,6 +68,12 @@ function emitDiagnosticsUpdated(): void {
 	document.dispatchEvent(event);
 }
 
+function clearScheduledDiagnosticsUpdated(): void {
+	if (diagnosticsEventTimer == null) return;
+	clearTimeout(diagnosticsEventTimer);
+	diagnosticsEventTimer = null;
+}
+
 const lspPublishedDiagnostics = StateField.define<LspDiagnostic[]>({
 	create(): LspDiagnostic[] {
 		return [];
@@ -164,13 +171,30 @@ function sameDiagnostics(
 }
 
 function scheduleDiagnosticsUpdated(): void {
-	if (diagnosticsEventTimer) return;
-	diagnosticsEventTimer = window.setTimeout(() => {
-		diagnosticsEventTimer = 0;
-		emitDiagnosticsUpdated();
+	if (diagnosticsEventTimer != null) return;
+	diagnosticsEventTimer = setTimeout(() => {
+		diagnosticsEventTimer = null;
+		if (diagnosticsViewCount > 0) {
+			emitDiagnosticsUpdated();
+		}
 	}, 32);
 }
 
+const diagnosticsLifecyclePlugin = ViewPlugin.fromClass(
+	class {
+		constructor() {
+			diagnosticsViewCount++;
+		}
+
+		destroy(): void {
+			diagnosticsViewCount = Math.max(0, diagnosticsViewCount - 1);
+			if (!diagnosticsViewCount) {
+				clearScheduledDiagnosticsUpdated();
+			}
+		}
+	},
+);
+
 function mapDiagnostics(
 	plugin: LSPPluginAPI,
 	state: EditorState,
@@ -262,6 +286,7 @@ export function lspDiagnosticsUiExtension(includeGutter = true): Extension[] {
 		? () => []
 		: undefined;
 	const extensions: Extension[] = [
+		diagnosticsLifecyclePlugin,
 		lspPublishedDiagnostics,
 		linter(lspLinterSource, {
 			needsRefresh(update) {

src/cm/lsp/documentHighlights.ts

@@ -105,6 +105,8 @@ function buildDecos(
 		: [view.viewport];
 	const decos: Range<Decoration>[] = [];
 	let rangeIndex = 0;
+	// process() keeps highlights sorted so visible range culling can advance
+	// with a single cursor instead of rescanning visibleRanges for every mark.
 	for (const h of highlights) {
 		if (h.from < 0 || h.to > docLen || h.from >= h.to) continue;
 		while (

src/cm/rainbowBrackets.ts

@@ -40,6 +40,8 @@ const rainbowBracketsPlugin = ViewPlugin.fromClass(
 		scheduleBuild(view: EditorView): void {
 			this.pendingView = view;
 			if (this.raf) return;
+			// Rainbow bracket colors are purely visual, so batch rebuilds to the
+			// next frame instead of recomputing on every transient update.
 			this.raf = requestAnimationFrame(() => {
 				this.raf = 0;
 				const pendingView = this.pendingView;