Merge pull request #436 from QueryaHub/issue/424-syntax-highlight-threshold

perf(editor): honor syntax highlight isolate threshold (#424)

Author: ZhuchkaTriplesix

lib/core/editor/syntax_highlight_isolate.dart

@@ -118,9 +118,14 @@ TextStyle? _styleFromSegment(HighlightSegment s, TextStyle? base) {
   );
 }
 
-/// Runs [syntaxHighlightInIsolate] off the UI thread.
+/// Runs highlighting on a worker isolate when [job.code] is large enough;
+/// otherwise highlights synchronously on the calling isolate (avoids
+/// `compute` overhead for small editors).
 Future<List<HighlightSegment>> highlightOffMainThread(
   SyntaxHighlightJob job,
 ) {
+  if (job.code.length < kSyntaxHighlightIsolateThreshold) {
+    return Future<List<HighlightSegment>>.value(syntaxHighlightInIsolate(job));
+  }
   return compute(syntaxHighlightInIsolate, job);
 }

test/core/editor/syntax_highlight_isolate_test.dart

@@ -33,4 +33,27 @@ void main() {
     expect(segments, isNotEmpty);
     expect(segments.map((s) => s.text).join(), code);
   });
+
+  test('small SQL buffer highlights inline below isolate threshold', () async {
+    const code = 'SELECT 1;';
+    expect(code.length, lessThan(kSyntaxHighlightIsolateThreshold));
+
+    final config = buildDefaultEditorHighlighterConfig(
+      QueryaTheme.darkDefault.editor,
+    );
+    final segments = await highlightOffMainThread(
+      SyntaxHighlightJob(
+        code: code,
+        language: 'sql',
+        themeConfigJson: config,
+        grammarJson: SyntaxHighlightService.grammarJsonFor(
+          QueryaCodeLanguage.sql,
+        ),
+        wrapperArgb: QueryaTheme.darkDefault.editor.foreground.toARGB32(),
+      ),
+    );
+
+    expect(segments, isNotEmpty);
+    expect(segments.map((s) => s.text).join(), code);
+  });
 }