Remove Biome (#973)

We switched to TypeScript Language Server so we don't need it anymore.

Author: sonnyp

build-aux/modules/biome.json

@@ -39,7 +39,6 @@
   ],
   "modules": [
     "modules/blueprint-compiler.json",
-    "modules/biome.json",
     "modules/gst-plugin-gtk4.json",
     "modules/vte.json",
     "modules/libshumate.json",

build-aux/re.sonny.Workbench.Devel.json

@@ -39,7 +39,6 @@
   ],
   "modules": [
     "modules/blueprint-compiler.json",
-    "modules/biome.json",
     "modules/gst-plugin-gtk4.json",
     "modules/vte.json",
     "modules/libshumate.json",

build-aux/re.sonny.Workbench.json

@@ -55,7 +55,7 @@
     <release version="46.2" date="2024-07-xx">
       <description translatable="no">
         <ul>
-          <li>Update Biome to 1.8.3</li>
+          <li>Replace Biome with TypeScript Language Server</li>
           <li>Update gst-plugin-gtk4 to 1.12.7</li>
         </ul>
       </description>

data/app.metainfo.xml

@@ -64,23 +64,7 @@ application.connect("open", (_self, files, hint) => {
   load().catch(console.error);
 });
 
-let proc_biome;
-
 application.connect("startup", () => {
-  // biome lsp-proxy starts a background server
-  // it does not get stopped and leaves a process hanging
-  // so manage it manually instead
-  // See https://github.com/workbenchdev/Workbench/issues/828
-  const subprocess_launcher = Gio.SubprocessLauncher.new(
-    Gio.SubprocessFlags.STDERR_SILENCE,
-  );
-  proc_biome = subprocess_launcher.spawnv([
-    "biome",
-    "__run_server",
-    "--config-path",
-    pkg.pkgdatadir,
-  ]);
-
   Library({
     application,
   });
@@ -90,10 +74,6 @@ application.connect("startup", () => {
   restoreSessions().catch(console.error);
 });
 
-application.connect("shutdown", () => {
-  proc_biome?.force_exit();
-});
-
 application.connect("activate", () => {
   if (application.is_remote) {
     bootstrap().catch(console.error);

src/application.js

@@ -25,7 +25,7 @@ export class CssDocument extends Document {
       },
     });
 
-    // Biome doesn't support diff - it just returns one edit
+    // GTKCssLanguageServer doesn't support diff - it just returns one edit
     // we don't want to loose the cursor position so we use this
     const state = this.code_view.saveState();
     applyTextEdits(text_edits, this.buffer);

src/langs/css/CssDocument.js

@@ -26,10 +26,6 @@ export class JavaScriptDocument extends Document {
       },
     });
 
-    // Biome doesn't support diff - it just returns one edit
-    // we don't want to loose the cursor position so we use this
-    const state = this.code_view.saveState();
     applyTextEdits(text_edits, this.buffer);
-    await this.code_view.restoreState(state);
   }
 }

src/langs/javascript/JavaScriptDocument.js

@@ -47,7 +47,6 @@ configure_file(
 )
 
 install_data('langs/vala/workbench.vala', install_dir: pkgdatadir)
-install_data('langs/javascript/biome.json', install_dir: pkgdatadir)
 install_data('project-readme.md', install_dir: pkgdatadir)
 subdir('libworkbench')
 subdir('Previewer')