Restart Typespec Language Server when it's not started before or sett…

…ings changed (#4912)

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
3. Code refactor in vscode extension.

related issues:
#2996, #4765

---------

Co-authored-by: Timothee Guerin <[email protected]>

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

@@ -0,0 +1,7 @@
+---
+changeKind: fix
+packages:
+  - typespec-vscode
+---
+
+Fix the issue when Typespec Language Server can't be restarted when the server wasn't running before

.chronus/changes/vscode-restart-lsp-2024-9-31-11-4-2.md

@@ -0,0 +1,7 @@
+---
+changeKind: feature
+packages:
+  - typespec-vscode
+---
+
+TypeSpec Language Server would be restarted with new settings when setting "typespec.tsp-server.path" is changed

packages/typespec-vscode/package.json

@@ -164,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,11 @@
 import commonjs from "@rollup/plugin-commonjs";
 import resolve from "@rollup/plugin-node-resolve";
 import typescript from "@rollup/plugin-typescript";
+import { dirname } from "path";
 
 import { defineConfig } from "rollup";
+import { fileURLToPath } from "url";
+const projDir = dirname(fileURLToPath(import.meta.url));
 
 const plugins = [(resolve as any)({ preferBuiltins: true }), (commonjs as any)()];
 const baseConfig = defineConfig({
@@ -53,17 +56,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: projDir,
+    },
+    tsconfig: "./tsconfig.build.json",
+    outDir,
+  });
 }

packages/typespec-vscode/src/const.ts

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