[@typespec/spector] Update start server scenario paths (#5309)

With my earlier PRs, I have already moved all the `scenarioPath`
parameters to `scenarioPaths` parameter i.e. made that parameter from a
single value to an array of values.

But, this start script was missed earlier. But, the Go SDK generator
uses this script and need this to be updated. This PR acoomplishes that.

Please review and approve this PR. Thanks

.chronus/changes/UpdateStartServerScenarioPaths-2024-11-9-11-3-38.md

@@ -0,0 +1,7 @@
+---
+changeKind: internal
+packages:
+  - "@typespec/spector"
+---
+
+Update `server start` script parameter - `scenarioPath` to `scenarioPaths`.

.chronus/changes/UpdateStartServerScenarioPaths-2024-11-9-17-21-55.md

@@ -0,0 +1,7 @@
+---
+changeKind: internal
+packages:
+  - "@typespec/spec-coverage-sdk"
+---
+
+Added packageName property to metadata

packages/spec-coverage-sdk/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Change Log - @typespec/spec-coverage-sdk
 
+## 0.1.0-alpha.2
+
+- Added `packageName` to `scenariosMetadata`.
+
 ## 0.1.0-alpha.1
 
 No changes, version bump only.

packages/spec-coverage-sdk/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@typespec/spec-coverage-sdk",
-  "version": "0.1.0-alpha.1",
+  "version": "0.1.0-alpha.2",
   "description": "Spec utility to manage the reported coverage",
   "main": "dist/index.js",
   "type": "module",

packages/spec-coverage-sdk/src/types.ts

@@ -58,6 +58,7 @@ export interface ResolvedCoverageReport extends CoverageReport {
 export interface ScenariosMetadata {
   version: string;
   commit: string;
+  packageName: string;
 }
 
 export interface GeneratorMetadata {

packages/spec-dashboard/src/apis.ts

@@ -7,27 +7,31 @@ import {
 const storageAccountName = "typespec";
 
 export type GeneratorNames =
-  | "python"
-  | "typescript/rlc"
-  | "typescript/modular"
-  | "csharp"
+  | "@typespec/http-client-python"
+  | "@azure-tools/typespec-python"
+  | "@azure-tools/typespec-go"
+  | "@azure-tools/typespec-csharp"
   | "@typespec/http-client-csharp"
-  | "java"
-  | "go"
-  | "cpp"
-  | "rust"
+  | "@azure-tools/typespec-ts-rlc"
+  | "@azure-tools/typespec-ts-modular"
+  | "@azure-tools/typespec-java"
+  | "@typespec/http-client-java"
+  | "@azure-tools/typespec-cpp"
+  | "@azure-tools/typespec-rust"
   | "test";
 const query = new URLSearchParams(window.location.search);
 const generatorNames: GeneratorNames[] = [
-  "python",
-  "typescript/rlc",
-  "typescript/modular",
-  "csharp",
+  "@typespec/http-client-python",
+  "@azure-tools/typespec-python",
+  "@azure-tools/typespec-go",
+  "@azure-tools/typespec-csharp",
   "@typespec/http-client-csharp",
-  "java",
-  "go",
-  "cpp",
-  "rust",
+  "@azure-tools/typespec-ts-rlc",
+  "@azure-tools/typespec-ts-modular",
+  "@azure-tools/typespec-java",
+  "@typespec/http-client-java",
+  "@azure-tools/typespec-cpp",
+  "@azure-tools/typespec-rust",
   ...(query.has("showtest") ? (["test"] as const) : []),
 ];
 

packages/spector/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Change Log - @typespec/spector
 
+## 0.1.0-alpha.4
+
+- Update `server start` script parameter - `scenarioPath` to `scenarioPaths`.
+
 ## 0.1.0-alpha.3
 
 No changes, version bump only.

packages/spector/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@typespec/spector",
-  "version": "0.1.0-alpha.3",
+  "version": "0.1.0-alpha.4",
   "description": "Typespec Core Tool to validate, run mock api, collect coverage.",
   "exports": {
     ".": {

packages/spector/src/actions/serve.ts

@@ -39,14 +39,14 @@ export async function startInBackground(config: ServeConfig) {
     const [nodeExe, entrypoint] = process.argv;
     logger.info(`Starting server in background at port ${config.port}`);
     const scenariosPath = Array.isArray(config.scenariosPath)
-      ? config.scenariosPath.join(" ")
-      : config.scenariosPath;
+      ? config.scenariosPath
+      : [config.scenariosPath];
     const cp = spawn(
       nodeExe,
       [
         entrypoint,
         "serve",
-        scenariosPath,
+        ...scenariosPath,
         "--port",
         config.port.toString(),
         "--coverageFile",

packages/spector/src/cli/cli.ts

@@ -92,13 +92,14 @@ async function main() {
     .command("server", "Server management", (cmd) => {
       cmd
         .command(
-          "start <scenariosPath>",
+          "start <scenariosPaths..>",
           "Start the server in the background.",
           (cmd) => {
             return cmd
-              .positional("scenariosPath", {
-                description: "Path to the scenarios and mock apis",
+              .positional("scenariosPaths", {
+                description: "Path(s) to the scenarios and mock apis",
                 type: "string",
+                array: true,
                 demandOption: true,
               })
               .option("port", {
@@ -115,7 +116,7 @@ async function main() {
           },
           async (args) =>
             startInBackground({
-              scenariosPath: resolve(process.cwd(), args.scenariosPath),
+              scenariosPath: args.scenariosPaths,
               port: args.port,
               coverageFile: args.coverageFile,
             }),

packages/spector/src/coverage/common.ts

@@ -6,5 +6,6 @@ export async function getScenarioMetadata(scenariosPath: string): Promise<Scenar
   return {
     commit: getCommit(scenariosPath),
     version: pkg?.version ?? "?",
+    packageName: pkg?.name ?? "@typespec/http-specs",
   };
 }