Restart Typespec Language Server when it's not started before or settings changed

.chronus/changes/vscode-restart-lsp-2024-9-30-20-29-32.md

@@ -0,0 +1,8 @@
+---
+changeKind: fix
+packages:
+  - typespec-vscode
+---
+
+1. TypeSpec Language Server would be restarted with new settings when setting "typespec.tsp-server.path" is changed
+2. Typespec Language Server can be restarted properly when the server wasn't running before

packages/typespec-vscode/package.json

@@ -31,7 +31,8 @@
   },
   "activationEvents": [
     "onLanguage:typespec",
-    "onCommand:typespec.restartServer"
+    "onCommand:typespec.restartServer",
+    "workspaceContains:**/tspconfig.yaml"
   ],
   "icon": "./icons/logo.png",
   "contributes": {
@@ -163,7 +164,7 @@
     "deploy": "vsce publish",
     "open-in-browser": "vscode-test-web --extensionDevelopmentPath=. .",
     "test:e2e": "pnpm test:web",
-    "test:web": "vscode-test-web --extensionDevelopmentPath=. --headless --extensionTestsPath=dist/test/suite.js ./test/data"
+    "test:web": "vscode-test-web --extensionDevelopmentPath=. --headless --extensionTestsPath=dist/test/web/suite.js ./test/web/data"
   },
   "devDependencies": {
     "@rollup/plugin-commonjs": "~28.0.0",

packages/typespec-vscode/rollup.config.ts

@@ -1,8 +1,13 @@
 import commonjs from "@rollup/plugin-commonjs";
 import resolve from "@rollup/plugin-node-resolve";
 import typescript from "@rollup/plugin-typescript";
+import path from "path";
 
 import { defineConfig } from "rollup";
+import { fileURLToPath } from "url";
+
+const curFile = fileURLToPath(import.meta.url);
+const curDir = path.dirname(curFile);
 
 const plugins = [(resolve as any)({ preferBuiltins: true }), (commonjs as any)()];
 const baseConfig = defineConfig({
@@ -53,17 +58,24 @@ export default defineConfig([
   },
   {
     ...baseConfig,
-    input: "test/suite.ts",
+    input: "test/web/suite.ts",
     output: {
-      file: "dist/test/suite.js", // VSCode web will add extra .js if you use .cjs
+      file: "dist/test/web/suite.js", // VSCode web will add extra .js if you use .cjs
       format: "commonjs",
       sourcemap: true,
       inlineDynamicImports: true,
     },
-    plugins: [...plugins, ts("dist/test")],
+    plugins: [...plugins, ts("dist/test/web")],
   },
 ]);
 
 function ts(outDir: string) {
-  return (typescript as any)({ tsconfig: "./tsconfig.build.json", outDir });
+  return (typescript as any)({
+    compilerOptions: {
+      // set sourceRoot to absolute path, otherwise the path in the map file generated is incorrect when outDir is given
+      sourceRoot: curDir,
+    },
+    tsconfig: "./tsconfig.build.json",
+    outDir,
+  });
 }

packages/typespec-vscode/src/const.ts

@@ -0,0 +1,3 @@
+export const enum SettingName {
+  TspServerPath = "typespec.tsp-server.path",
+}