Merge language tools into monorepo

.github/workflows/ci.yml

@@ -154,6 +154,54 @@ jobs:
       - name: Test ${{ matrix.TEST_SUITE.name }}
         run: ${{ matrix.TEST_SUITE.script }}
 
+  test-language-tools:
+    runs-on: ${{ matrix.os }}
+    timeout-minutes: 30
+    needs: build
+    strategy:
+      matrix:
+        os: [ubuntu-latest]
+        node_version: [20.9.0, 22]
+        include:
+          - os: windows-latest
+            node_version: 20.9.0
+          - os: macos-latest
+            node_version: 20.9.0
+      fail-fast: false
+
+    name: 'Test (language-tools): node-${{ matrix.node_version }}, ${{ matrix.os }}'
+    steps:
+      - name: Checkout
+        uses: actions/checkout@08eba0b27e820071cde6df949e0beb9ba4906955 # v4.3.0
+
+      - name: Setup PNPM
+        uses: pnpm/action-setup@41ff72655975bd51cab0327fa583b6e92b6d3061 # v4.2.0
+
+      - name: Setup node@${{ matrix.node_version }}
+        uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0
+        with:
+          node-version: ${{ matrix.node_version }}
+          cache: "pnpm"
+
+      - name: Install dependencies
+        run: pnpm install
+
+      - name: Build the packages
+        run: pnpm build
+
+      - name: Test (Linux)
+        if: runner.os == 'Linux'
+        run: |
+          pnpm run test:language-tools:no-vs
+          cd packages/language-tools/vscode
+          xvfb-run -a pnpm test
+          cd ../ts-plugin
+          xvfb-run -a pnpm test
+
+      - name: Test
+        run: pnpm test:language-tools
+        if: runner.os != 'Linux'
+
   e2e:
     name: "Test (E2E): ${{ matrix.os }} (node@${{ matrix.NODE_VERSION }})"
     runs-on: ${{ matrix.os }}

package.json

@@ -20,9 +20,11 @@
     "format:code:ci": "biome format && prettier -w \"**/*\" --ignore-unknown --cache --check",
     "format:imports": "biome check --formatter-enabled=false --write",
     "format:imports:ci": "biome ci --formatter-enabled=false",
-    "test": "pnpm run test:astro && pnpm run test:integrations",
+    "test": "pnpm run test:astro && pnpm run test:integrations && pnpm run test:language-tools",
     "test:astro": "turbo run test --concurrency=1 --filter=astro --only",
-    "test:integrations": "turbo run test --concurrency=1 --filter=create-astro --filter=\"@astrojs/*\" --only",
+    "test:integrations": "turbo run test --concurrency=1 --filter=create-astro --filter=\"@astrojs/*\" --filter=\"!./packages/language-tools/**/*\" --only",
+    "test:language-tools": "turbo run test --concurrency=1 --filter=\"./packages/language-tools/**/*\" --only",
+    "test:language-tools:no-vs": "turbo run test --filter=\"@astrojs/language-server\" --filter=\"@astrojs/check\" --only",
     "test:citgm": "pnpm -r --filter=astro test",
     "test:match": "cd packages/astro && pnpm run test:match",
     "test:cli": "cd packages/astro && pnpm run test:cli",

packages/astro/package.json

@@ -91,7 +91,7 @@
   ],
   "scripts": {
     "prebuild": "astro-scripts prebuild --to-string \"src/runtime/server/astro-island.ts\" \"src/runtime/client/{idle,load,media,only,visible}.ts\"",
-    "build": "pnpm run prebuild && astro-scripts build \"src/**/*.{ts,js}\" --copy-wasm && tsc && astro-check --root ./components",
+    "build": "pnpm run prebuild && astro-scripts build \"src/**/*.{ts,js}\" --copy-wasm && tsc",
     "build:ci": "pnpm run prebuild && astro-scripts build \"src/**/*.{ts,js}\" --copy-wasm",
     "dev": "astro-scripts dev --copy-wasm --prebuild \"src/runtime/server/astro-island.ts\" --prebuild \"src/runtime/client/{idle,load,media,only,visible}.ts\" \"src/**/*.{ts,js}\"",
     "test": "pnpm run test:unit && pnpm run test:integration && pnpm run test:types",
@@ -173,7 +173,6 @@
     "sharp": "^0.34.0"
   },
   "devDependencies": {
-    "@astrojs/check": "^0.9.4",
     "@playwright/test": "1.51.1",
     "@types/aria-query": "^5.0.4",
     "@types/common-ancestor-path": "^1.0.2",

packages/astro/src/cli/check/index.ts

@@ -10,6 +10,7 @@ export async function check(flags: Flags) {
 		skipAsk: !!flags.yes || !!flags.y,
 		cwd: flags.root,
 	};
+	// @ts-ignore For some unknown reason, in CI TS isn't able to get the type here even though it works locally.
 	const checkPackage = await getPackage<typeof import('@astrojs/check')>(
 		'@astrojs/check',
 		logger,

packages/astro/src/cli/index.ts

@@ -200,7 +200,7 @@ async function runCommand(cmd: string, flags: yargs.Arguments) {
 			if (flags.watch) {
 				return await new Promise(() => {}); // lives forever
 			} else {
-				return process.exit(checkServer ? 1 : 0);
+				return process.exit(typeof checkServer === 'boolean' && checkServer ? 1 : 0);
 			}
 		}
 	}