Fix development issues after compiler directory move (microsoft#2077)

1. Remove unnecessary exclude in tsconfig.json now that we use src/ include
in compiler. Somehow this seemed to fix a rebuild failure for me and it's
consistent with other projects.

2. Fix tmlanguage.js path in watch-tmlanguage

3. Fix compiler path for F5 on VS Code extension

4. Use consistent compiler path for F5 on VS Classic extension. Note that we
should entrypoints/ for F5 since we don't want to run resolution.

5. Delete unused and broken regen-samples in json-schema. It had wrong path
to compiler but even before that fails with non-existent samples dir
already. Also remove compile-samples npm script that just ran a broken tsp
compile command.

Author: nguerrera

common/changes/@typespec/compiler/fix-f5_2023-06-16-16-19.json

@@ -0,0 +1,10 @@
+{
+  "changes": [
+    {
+      "packageName": "@typespec/compiler",
+      "comment": "",
+      "type": "none"
+    }
+  ],
+  "packageName": "@typespec/compiler"
+}

common/changes/@typespec/json-schema/fix-f5_2023-06-16-16-19.json

@@ -0,0 +1,10 @@
+{
+  "changes": [
+    {
+      "packageName": "@typespec/json-schema",
+      "comment": "",
+      "type": "none"
+    }
+  ],
+  "packageName": "@typespec/json-schema"
+}

common/changes/typespec-vs/fix-f5_2023-06-16-16-24.json

@@ -0,0 +1,10 @@
+{
+  "changes": [
+    {
+      "packageName": "typespec-vs",
+      "comment": "",
+      "type": "none"
+    }
+  ],
+  "packageName": "typespec-vs"
+}

common/changes/typespec-vscode/fix-f5_2023-06-16-16-19.json

@@ -0,0 +1,10 @@
+{
+  "changes": [
+    {
+      "packageName": "typespec-vscode",
+      "comment": "",
+      "type": "none"
+    }
+  ],
+  "packageName": "typespec-vscode"
+}

packages/compiler/scripts/watch-tmlanguage.js

@@ -4,7 +4,7 @@ import { resolve } from "path";
 import { pathToFileURL } from "url";
 
 let count = 0;
-const scriptPath = resolve("dist/server/tmlanguage.js");
+const scriptPath = resolve("dist/src/server/tmlanguage.js");
 
 async function regenerate() {
   const script = await import(`${pathToFileURL(scriptPath)}?q=${count++}`);
@@ -13,4 +13,4 @@ async function regenerate() {
   await copyFile("dist/typespec.tmLanguage", "../typespec-vscode/dist/typespec.tmLanguage");
 }
 
-runWatch("dist/server/tmlanguage.js", regenerate, {});
+runWatch("dist/src/server/tmlanguage.js", regenerate, {});

packages/compiler/tsconfig.json

@@ -6,6 +6,5 @@
     "tsBuildInfoFile": "temp/tsconfig.tsbuildinfo",
     "types": ["node", "mocha"]
   },
-  "include": ["src/**/*.ts", "test/**/*.ts"],
-  "exclude": ["dist", "node_modules", "temp", "tsp-output"]
+  "include": ["src/**/*.ts", "test/**/*.ts"]
 }

packages/json-schema/package.json

@@ -34,8 +34,7 @@
     "test": "mocha",
     "test-official": "c8 mocha --forbid-only --reporter mocha-multi-reporters",
     "lint": "eslint . --ext .ts --max-warnings=0",
-    "lint:fix": "eslint . --fix --ext .ts",
-    "compile-samples": "tsp compile "
+    "lint:fix": "eslint . --fix --ext .ts"
   },
   "files": [
     "lib/*.tsp",

packages/json-schema/scripts/regen-samples.js

@@ -168,7 +168,7 @@ private static string GetDevelopmentTypeSpecServerPath()
             // the source tree.
             var thisDir = Path.GetDirectoryName(Assembly.GetExecutingAssembly().Location);
             var srcDir = File.ReadAllText(Path.Combine(thisDir, "DebugSourceDirectory.txt")).Trim();
-            return Path.GetFullPath(Path.Combine(srcDir, "..", "..", "compiler", "cmd", "tsp-server.js"));
+            return Path.GetFullPath(Path.Combine(srcDir, "..", "..", "compiler", "entrypoints", "server.js"));
         }
 #endif
 

packages/typespec-vs/src/VSExtension.cs

@@ -84,7 +84,7 @@ async function resolveTypeSpecServer(context: ExtensionContext): Promise<Executa
 
   // In development mode (F5 launch from source), resolve to locally built server.js.
   if (process.env.TYPESPEC_DEVELOPMENT_MODE) {
-    const script = context.asAbsolutePath("../compiler/dist/server/server.js");
+    const script = context.asAbsolutePath("../compiler/entrypoints/server.js");
     // we use CLI instead of NODE_OPTIONS environment variable in this case
     // because --nolazy is not supported by NODE_OPTIONS.
     const options = nodeOptions?.split(" ").filter((o) => o) ?? [];