feat(oxlint)!: remove oxc_language_server binary (#17457)

Boshen · Boshen · commit f7da875f0b4a · 2025-12-29T11:58:25.000Z
Given that all IDEs now use `--lsp`, we can "safely" remove the binary to reduce the published package size by half. This is breaking and forces people to upgrade their oxc IDE extensions. closes #12251 closes #15740
diff --git a/.github/workflows/release_apps.yml b/.github/workflows/release_apps.yml
@@ -119,9 +119,6 @@ jobs:
       - name: Build Rust binary
         run: cross build --release -p oxlint --features allocator --target=${{ matrix.target }}
 
-      - name: Build language server
-        run: cross build --release -p oxc_language_server --bin oxc_language_server --target=${{ matrix.target }}
-
       # The binaries are zipped to fix permission loss https://github.com/actions/upload-artifact#permission-loss
       - name: Archive oxlint .node binary
         uses: ./.github/actions/archive-binary
@@ -137,13 +134,6 @@ jobs:
           binary_name: oxlint-${{ matrix.code-target }}${{ runner.os == 'Windows' && '.exe' || '' }}
           archive_name: rust-oxlint-${{ matrix.code-target }}
 
-      - name: Archive language server binary
-        uses: ./.github/actions/archive-binary
-        with:
-          source_path: target/${{ matrix.target }}/release/oxc_language_server${{ runner.os == 'Windows' && '.exe' || '' }}
-          binary_name: oxc_language_server-${{ matrix.code-target }}${{ runner.os == 'Windows' && '.exe' || '' }}
-          archive_name: oxc_language_server-${{ matrix.code-target }}
-
       - name: Upload Binary
         uses: actions/upload-artifact@b7c566a772e6b6bfb58ed0dc250532a479d7789f # v6.0.0
         with:
@@ -152,8 +142,6 @@ jobs:
           path: |
             oxlint**.zip
             oxlint**.tar.gz
-            oxc_language_server**.zip
-            oxc_language_server**.tar.gz
 
       # For github release
       - name: Upload Binary
diff --git a/npm/oxfmt/scripts/generate-packages.js b/npm/oxfmt/scripts/generate-packages.js
@@ -92,7 +92,6 @@ function copyDistFiles() {
 }
 
 // NOTE: Must update npm/oxfmt/bin/oxfmt
-// and npm/oxfmt/bin/oxc_language_server
 const TARGETS = [
   "win32-x64",
   "win32-arm64",
diff --git a/npm/oxlint/bin/oxc_language_server b/npm/oxlint/bin/oxc_language_server
diff --git a/npm/oxlint/package.json b/npm/oxlint/package.json
@@ -16,14 +16,12 @@
     "url": "https://github.com/sponsors/Boshen"
   },
   "bin": {
-    "oxc_language_server": "bin/oxc_language_server",
     "oxlint": "bin/oxlint"
   },
   "files": [
     "configuration_schema.json",
     "dist",
     "README.md",
-    "bin/oxc_language_server",
     "bin/oxlint"
   ],
   "type": "module",
diff --git a/npm/oxlint/scripts/generate-packages.js b/npm/oxlint/scripts/generate-packages.js
@@ -7,7 +7,6 @@ import { resolve } from "node:path";
 import { fileURLToPath } from "node:url";
 
 const OXLINT_BIN_NAME = "oxlint";
-const OXLS_BIN_NAME = "oxc_language_server";
 /** <REPO ROOT>/npm/oxlint` */
 const OXLINT_ROOT = resolve(fileURLToPath(import.meta.url), "../..");
 /** `<REPO ROOT>/npm` */
@@ -60,9 +59,6 @@ function generateNativePackage(target) {
     os: [platform],
     cpu: [arch],
     ...libc,
-    publishConfig: {
-      executableFiles: ["oxc_language_server"],
-    },
   };
 
   const manifestPath = resolve(packageRoot, "package.json");
@@ -73,16 +69,8 @@ function generateNativePackage(target) {
   const oxlintBinSource = resolve(REPO_ROOT, `${OXLINT_BIN_NAME}.${target}.node`);
   const oxlintBinTarget = resolve(packageRoot, `${OXLINT_BIN_NAME}.${target}.node`);
 
-  const ext = platform === "win32" ? ".exe" : "";
-  const oxlsBinSource = resolve(REPO_ROOT, `${OXLS_BIN_NAME}-${target}${ext}`);
-  const oxlsBinTarget = resolve(packageRoot, `${OXLS_BIN_NAME}${ext}`);
-
   console.log(`Copy linter binary ${oxlintBinSource}`);
   fs.copyFileSync(oxlintBinSource, oxlintBinTarget);
-
-  console.log(`Copy language server binary ${oxlsBinSource}`);
-  fs.copyFileSync(oxlsBinSource, oxlsBinTarget);
-  fs.chmodSync(oxlsBinTarget, 0o755);
 }
 
 function writeManifest() {
@@ -121,7 +109,6 @@ function copyDistFiles() {
   fs.cpSync(OXLINT_DIST_SRC, OXLINT_DIST_DEST, { recursive: true });
 }
 
-// NOTE: Must update npm/oxlint/bin/oxc_language_server
 const TARGETS = [
   "win32-x64",
   "win32-arm64",

Original file line number	Diff line number	Diff line change
`@@ -92,7 +92,6 @@ function copyDistFiles() {`
`92`	`92`	`}`
`93`	`93`
`94`	`94`	`// NOTE: Must update npm/oxfmt/bin/oxfmt`
`95`		`-// and npm/oxfmt/bin/oxc_language_server`
`96`	`95`	`const TARGETS = [`
`97`	`96`	`"win32-x64",`
`98`	`97`	`"win32-arm64",`