@vgai/sdk 0.5.20 → 0.5.21

package/package.json

@@ -2,7 +2,7 @@
   "name": "@vgai/sdk",
   "author": "Volter AI, Inc.",
   "license": "Apache-2.0",
-  "version": "0.5.20",
+  "version": "0.5.21",
   "type": "module",
   "repository": {
     "type": "git",
@@ -30,7 +30,7 @@
   },
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.30.0",
-    "@vgai/engine": "0.5.20",
+    "@vgai/engine": "0.5.21",
     "playwright": "^1.58.2",
     "zod": "^4.3.6"
   }

package/src/editor/transport.ts

@@ -179,6 +179,15 @@ export interface EditorSessionInfo {
   pid: number | null;
   /** Exact explicitly targeted editor origin/base URL, including protocol and host. */
   url?: string;
+  /**
+   * Why this server cannot DESCRIBE the `project` it is serving — its
+   * manifest's own parse/validation failure, naming the failing key. `null`
+   * when the project reads fine, and `undefined` from a probe that did not ask
+   * (or a server too old to say). See {@link servedProjectPath}: a session in
+   * this state is still THIS project's session, and callers must report the
+   * reason rather than treat it as no session at all.
+   */
+  manifestError?: string | null;
 }
 
 export interface EditorCommandResult {
@@ -414,6 +423,28 @@ function readRegisteredSessions(): RegistrySessionEntry[] {
   }
 }
 
+/**
+ * WHICH PROJECT a `/__editor/project` body says its server is serving.
+ *
+ * `serving` is that server's own statement of "I AM serving this project, I
+ * just cannot describe it" (its manifest is unparseable or fails strict
+ * validation) — added to the route precisely because a bare `{ project: null }`
+ * is indistinguishable from "no project open". Reading only `project.path`
+ * collapses the two, and the cost is that every project-matched command loses
+ * a live session the moment a save breaks its manifest, reporting it as
+ * belonging to no project rather than as this project's degraded session.
+ */
+function servedProjectPath(body: unknown): { path: string | null; manifestError: string | null } {
+  const b = body as {
+    project?: { path?: string } | null;
+    serving?: { path?: string; error?: string } | null;
+  };
+  return {
+    path: b.project?.path ?? b.serving?.path ?? null,
+    manifestError: b.project ? null : (b.serving?.error ?? null),
+  };
+}
+
 async function fetchJson(url: string, timeoutMs: number): Promise<unknown | undefined> {
   try {
     const res = await fetch(url, { signal: AbortSignal.timeout(timeoutMs) });
@@ -463,9 +494,15 @@ export class HttpEditorTransport implements EditorTransport {
     const base = editorUrl.replace(/\/+$/, '');
     const body = await fetchJson(`${base}/__editor/project`, timeoutMs);
     if (body === undefined) return undefined;
-    const project = (body as { project?: { path?: string } | null }).project?.path ?? null;
+    const served = servedProjectPath(body);
     const port = url.port ? Number(url.port) : url.protocol === 'https:' ? 443 : 80;
-    return { port, project, pid: null, url: base };
+    return {
+      port,
+      project: served.path,
+      pid: null,
+      url: base,
+      manifestError: served.manifestError,
+    };
   }
 
   async listSessions(timeoutMs: number): Promise<EditorSessionInfo[]> {
@@ -478,11 +515,12 @@ export class HttpEditorTransport implements EditorTransport {
           perProbeTimeout,
         );
         if (body === undefined) return undefined;
-        const project = (body as { project?: { path?: string } | null }).project?.path ?? null;
+        const served = servedProjectPath(body);
         const info: EditorSessionInfo = {
           port: s.port,
-          project,
+          project: served.path,
           pid: s.pid,
+          manifestError: served.manifestError,
         };
         return info;
       }),

package/src/play/transport.ts

@@ -426,15 +426,33 @@ function readRegisteredSessions(): RegistrySessionEntry[] {
   try {
     const raw: unknown = JSON.parse(readFileSync(registryFile, 'utf8'));
     return Array.isArray(raw)
-      ? raw
-          .filter(isRegistrySessionEntry)
-          .filter((s) => pidAlive(s.pid))
+      ? raw.filter(isRegistrySessionEntry).filter((s) => pidAlive(s.pid))
       : [];
   } catch {
     return [];
   }
 }
 
+/**
+ * WHICH PROJECT a `/__editor/project` body says its server is serving — the
+ * play lane's copy of `editor/transport.ts`'s `servedProjectPath` (deliberate
+ * light duplicate of the read half, same as `readRegisteredSessions` above).
+ *
+ * `serving` is that server's own statement of "I AM serving this project, I
+ * just cannot describe it" (its manifest is unparseable or fails strict
+ * validation) — added to the route precisely because a bare `{ project: null }`
+ * is indistinguishable from "no project open". Reading only `project.path`
+ * collapses the two, and the cost is that every project-matched command loses
+ * a live session the moment a save breaks its manifest.
+ */
+function servedProjectPath(body: unknown): string | null {
+  const b = body as {
+    project?: { path?: string } | null;
+    serving?: { path?: string } | null;
+  };
+  return b.project?.path ?? b.serving?.path ?? null;
+}
+
 async function fetchJson(url: string, timeoutMs: number): Promise<unknown | undefined> {
   try {
     const res = await fetch(url, { signal: AbortSignal.timeout(timeoutMs) });
@@ -548,7 +566,7 @@ export class HttpPlayTransport implements PlayTransport {
     const base = editorUrl.replace(/\/+$/, '');
     const body = await fetchJson(`${base}/__editor/project`, timeoutMs);
     if (body === undefined) return undefined;
-    const project = (body as { project?: { path?: string } | null }).project?.path ?? null;
+    const project = servedProjectPath(body);
     const port = url.port ? Number(url.port) : url.protocol === 'https:' ? 443 : 80;
     return { port, project, pid: null, url: base };
   }
@@ -563,7 +581,7 @@ export class HttpPlayTransport implements PlayTransport {
           perProbeTimeout,
         );
         if (body === undefined) return undefined;
-        const project = (body as { project?: { path?: string } | null }).project?.path ?? null;
+        const project = servedProjectPath(body);
         const info: PlaySessionInfo = {
           port: s.port,
           project,