@skyramp/mcp 0.2.8-rc.1 → 0.2.9

package/build/workspace/workspace.d.ts

@@ -99,22 +99,22 @@ export declare const workspaceConfigSchema: z.ZodObject<{
     }>>;
     metadata: z.ZodOptional<z.ZodObject<{
         schemaVersion: z.ZodString;
-        mcpVersion: z.ZodString;
+        mcpVersion: z.ZodOptional<z.ZodString>;
         executorVersion: z.ZodString;
-        createdAt: z.ZodString;
-        updatedAt: z.ZodString;
+        createdAt: z.ZodOptional<z.ZodString>;
+        updatedAt: z.ZodOptional<z.ZodString>;
     }, "strict", z.ZodTypeAny, {
-        createdAt: string;
         schemaVersion: string;
-        mcpVersion: string;
         executorVersion: string;
-        updatedAt: string;
+        createdAt?: string | undefined;
+        mcpVersion?: string | undefined;
+        updatedAt?: string | undefined;
     }, {
-        createdAt: string;
         schemaVersion: string;
-        mcpVersion: string;
         executorVersion: string;
-        updatedAt: string;
+        createdAt?: string | undefined;
+        mcpVersion?: string | undefined;
+        updatedAt?: string | undefined;
     }>>;
     services: z.ZodOptional<z.ZodArray<z.ZodObject<{
         serviceName: z.ZodString;
@@ -203,11 +203,11 @@ export declare const workspaceConfigSchema: z.ZodObject<{
     }>, "many">>;
 }, "strict", z.ZodTypeAny, {
     metadata?: {
-        createdAt: string;
         schemaVersion: string;
-        mcpVersion: string;
         executorVersion: string;
-        updatedAt: string;
+        createdAt?: string | undefined;
+        mcpVersion?: string | undefined;
+        updatedAt?: string | undefined;
     } | undefined;
     workspace?: {
         repoName?: string | undefined;
@@ -236,11 +236,11 @@ export declare const workspaceConfigSchema: z.ZodObject<{
     }[] | undefined;
 }, {
     metadata?: {
-        createdAt: string;
         schemaVersion: string;
-        mcpVersion: string;
         executorVersion: string;
-        updatedAt: string;
+        createdAt?: string | undefined;
+        mcpVersion?: string | undefined;
+        updatedAt?: string | undefined;
     } | undefined;
     workspace?: {
         repoName?: string | undefined;
@@ -277,11 +277,11 @@ export type MetadataSection = NonNullable<WorkspaceConfig["metadata"]>;
 /** Validates a workspace configuration object against the Zod schema. */
 export declare function validateWorkspaceConfig(config: unknown): z.SafeParseReturnType<{
     metadata?: {
-        createdAt: string;
         schemaVersion: string;
-        mcpVersion: string;
         executorVersion: string;
-        updatedAt: string;
+        createdAt?: string | undefined;
+        mcpVersion?: string | undefined;
+        updatedAt?: string | undefined;
     } | undefined;
     workspace?: {
         repoName?: string | undefined;
@@ -310,11 +310,11 @@ export declare function validateWorkspaceConfig(config: unknown): z.SafeParseRet
     }[] | undefined;
 }, {
     metadata?: {
-        createdAt: string;
         schemaVersion: string;
-        mcpVersion: string;
         executorVersion: string;
-        updatedAt: string;
+        createdAt?: string | undefined;
+        mcpVersion?: string | undefined;
+        updatedAt?: string | undefined;
     } | undefined;
     workspace?: {
         repoName?: string | undefined;

package/build/workspace/workspace.js

@@ -78,10 +78,14 @@ export const workspaceConfigSchema = z
     metadata: z
         .object({
         schemaVersion: z.string(),
-        mcpVersion: z.string(),
+        // mcpVersion is no longer written by skyramp_init_workspace (SKYR-3818), but
+        // remains an optional, recognized key: testbot reads it from pre-existing
+        // workspaces as a fallback MCP-version pin (testbot.git src/config.ts), and
+        // older files still carry it. Optional so configs without it validate.
+        mcpVersion: z.string().optional(),
         executorVersion: z.string(),
-        createdAt: z.string(),
-        updatedAt: z.string(),
+        createdAt: z.string().optional(),
+        updatedAt: z.string().optional(),
     })
         .strict()
         .optional(),
@@ -121,7 +125,6 @@ export function createDefaultConfig() {
         workspace: {},
         metadata: {
             schemaVersion: "v1",
-            mcpVersion: "",
             executorVersion: "",
             createdAt: now,
             updatedAt: now,

package/build/workspace/workspace.test.js

@@ -7,9 +7,11 @@ import { serviceSchema, WorkspaceConfigManager, validateWorkspaceConfig, createD
 describe("workspace module (ported from @skyramp/skyramp, SKYR-3794)", () => {
     describe("serviceSchema", () => {
         it("accepts an optional repository and rejects a non-string repository", () => {
-            expect(serviceSchema.safeParse({ serviceName: "api", repository: "org/api" }).success).toBe(true);
+            expect(serviceSchema.safeParse({ serviceName: "api", repository: "org/api" })
+                .success).toBe(true);
             expect(serviceSchema.safeParse({ serviceName: "api" }).success).toBe(true);
-            expect(serviceSchema.safeParse({ serviceName: "api", repository: 123 }).success).toBe(false);
+            expect(serviceSchema.safeParse({ serviceName: "api", repository: 123 })
+                .success).toBe(false);
         });
         it("rejects unknown fields (strict)", () => {
             expect(serviceSchema.safeParse({ serviceName: "api", bogus: true }).success).toBe(false);
@@ -41,15 +43,72 @@ describe("workspace module (ported from @skyramp/skyramp, SKYR-3794)", () => {
         it("keeps a same-named service from a different repo distinct, and upserts in place", async () => {
             const mgr = new WorkspaceConfigManager(repoDir);
             await mgr.initialize();
-            await mgr.addService({ serviceName: "svc", testDirectory: "tests/primary" });
-            await mgr.addService({ serviceName: "svc", repository: "letsramp/api-insight", testDirectory: "tests/py" });
-            await mgr.addService({ serviceName: "svc", testDirectory: "tests/primary-v2" }); // upsert primary
+            await mgr.addService({
+                serviceName: "svc",
+                testDirectory: "tests/primary",
+            });
+            await mgr.addService({
+                serviceName: "svc",
+                repository: "letsramp/api-insight",
+                testDirectory: "tests/py",
+            });
+            await mgr.addService({
+                serviceName: "svc",
+                testDirectory: "tests/primary-v2",
+            }); // upsert primary
             const cfg = read();
             expect(cfg.services).toHaveLength(2);
             expect(cfg.services.find((s) => !s.repository).testDirectory).toBe("tests/primary-v2");
-            expect(cfg.services.find((s) => s.repository === "letsramp/api-insight").testDirectory).toBe("tests/py");
+            expect(cfg.services.find((s) => s.repository === "letsramp/api-insight")
+                .testDirectory).toBe("tests/py");
             // repoName auto-detected from the git remote.
             expect(cfg.workspace.repoName).toBe("letsramp/primary");
         });
     });
+    describe("mcpVersion: no longer written, still accepted (SKYR-3818)", () => {
+        let repoDir;
+        beforeEach(() => {
+            repoDir = fs.mkdtempSync(path.join(os.tmpdir(), "ws-mcpver-"));
+            execFileSync("git", ["init", "-q"], { cwd: repoDir });
+            execFileSync("git", ["remote", "add", "origin", "https://github.com/letsramp/primary.git"], { cwd: repoDir });
+        });
+        afterEach(() => {
+            fs.rmSync(repoDir, { recursive: true, force: true });
+        });
+        // Writes a workspace.yml that carries metadata.mcpVersion, emulating a workspace
+        // initialized by an older MCP version (which still stamped the field).
+        const writeLegacyConfig = () => {
+            const dir = path.join(repoDir, WORKSPACE_DIR);
+            fs.mkdirSync(dir, { recursive: true });
+            const now = new Date().toISOString();
+            const legacy = {
+                workspace: { repoName: "letsramp/primary" },
+                metadata: {
+                    schemaVersion: "v1",
+                    mcpVersion: "0.0.64",
+                    executorVersion: "1.2.3",
+                    createdAt: now,
+                    updatedAt: now,
+                },
+                services: [],
+            };
+            fs.writeFileSync(path.join(dir, WORKSPACE_FILENAME), yaml.dump(legacy), "utf8");
+        };
+        const readRaw = () => yaml.load(fs.readFileSync(path.join(repoDir, WORKSPACE_DIR, WORKSPACE_FILENAME), "utf8"), { schema: yaml.JSON_SCHEMA });
+        it("does not write mcpVersion into a freshly initialized workspace", async () => {
+            const mgr = new WorkspaceConfigManager(repoDir);
+            await mgr.initialize();
+            const onDisk = readRaw();
+            expect(onDisk.metadata.mcpVersion).toBeUndefined();
+        });
+        it("preserves mcpVersion from a pre-existing workspace.yml on read (testbot fallback)", async () => {
+            writeLegacyConfig();
+            const mgr = new WorkspaceConfigManager(repoDir);
+            const cfg = await mgr.read();
+            // Kept as an optional recognized key — not stripped — so testbot's
+            // config.ts MCP-version fallback keeps working for existing workspaces.
+            expect(cfg.metadata?.mcpVersion).toBe("0.0.64");
+            expect(cfg.metadata?.executorVersion).toBe("1.2.3");
+        });
+    });
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skyramp/mcp",
-  "version": "0.2.8-rc.1",
+  "version": "0.2.9",
   "main": "build/index.js",
   "exports": {
     ".": "./build/index.js",
@@ -67,7 +67,7 @@
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.24.3",
     "@playwright/test": "^1.55.0",
-    "@skyramp/skyramp": "1.3.29",
+    "@skyramp/skyramp": "1.3.30",
     "dockerode": "^5.0.0",
     "fast-glob": "^3.3.3",
     "js-yaml": "^4.1.1",