@moor-sh/mcp 0.12.0 → 0.14.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@moor-sh/mcp",
-  "version": "0.12.0",
+  "version": "0.14.0",
   "description": "MCP server for moor - lets AI agents (Claude Code, Cursor, etc.) manage your moor projects via the moor HTTP API.",
   "license": "MIT",
   "repository": {

package/src/index.ts

@@ -90,6 +90,17 @@ type Project = {
   domain: string | null;
   docker_image: string | null;
   github_url: string | null;
+  // #71: live_* fields are written by the API's status reconciler.
+  // status above is moor's RECORDED state (only changes on explicit
+  // start/stop/build/cancel). live_status reflects Docker's view at
+  // last successful inspect. Differences mean moor missed an external
+  // change (or the reconciler hasn't run yet). live_error non-null
+  // means the most recent inspect failed; the live_status / exit_code
+  // shown is the last successful snapshot.
+  live_status?: "running" | "stopped" | "error" | "missing" | null;
+  live_exit_code?: number | null;
+  live_checked_at?: string | null;
+  live_error?: string | null;
 };
 
 async function resolveProject(name: string): Promise<Project> {
@@ -284,7 +295,8 @@ server.registerTool(
   "moor_status",
   {
     title: "List Projects",
-    description: "List all projects managed by Moor with their status, source, and domain.",
+    description:
+      "List all projects managed by Moor. `status` is moor's recorded state (only changes on explicit start/stop/build/cancel). `live_status` is Docker's view at last successful inspect; differences (e.g. recorded='running' live='error') mean moor missed an external change like a host docker stop, crash, or OOM kill. `live_error` non-null means the most recent inspect failed and the live_* values are the last successful snapshot, not necessarily current.",
   },
   async () => {
     const res = await apiGet("/api/projects");
@@ -293,6 +305,10 @@ server.registerTool(
     const summary = projects.map((p) => ({
       name: p.name,
       status: p.status,
+      live_status: p.live_status ?? null,
+      live_exit_code: p.live_exit_code ?? null,
+      live_checked_at: p.live_checked_at ?? null,
+      live_error: p.live_error ?? null,
       source: p.docker_image || p.github_url || null,
       domain: p.domain,
     }));
@@ -304,7 +320,8 @@ server.registerTool(
   "moor_logs",
   {
     title: "Get Container Logs",
-    description: "Get recent logs from a project's container.",
+    description:
+      "Get recent logs from a project's container. Annotates output with state: ok (container running), exited (container is stopped but Docker still has logs), no_container (project never started), or missing (container_id is set but Docker doesn't have it). Throws only on docker_error (Docker daemon 5xx / unreachable) so an operator can distinguish infrastructure failure from app silence — pre-#74 the tool returned empty logs for all of these.",
     inputSchema: z.object({
       project: z.string().describe("Project name or ID"),
       lines: z.number().optional().default(100).describe("Number of log lines to retrieve"),
@@ -313,11 +330,43 @@ server.registerTool(
   async ({ project, lines }) => {
     const p = await resolveProject(project);
     const res = await apiGet(`/api/projects/${p.id}/logs?tail=${lines}`);
-    if (!res.ok) throw new Error(`Failed: ${res.status}`);
-    const data = (await res.json()) as { logs: string };
-    return {
-      content: [{ type: "text", text: data.logs || "(no logs)" }],
-    };
+    // 502 = API surfaced a Docker daemon failure. Throw so the agent
+    // gets a tool error, not silent empty logs.
+    if (res.status === 502) {
+      const data = (await res.json().catch(() => ({}))) as { error?: string };
+      throw new Error(`Docker error: ${data.error ?? "unknown"}`);
+    }
+    if (!res.ok) throw new Error(`Failed: ${res.status} ${await res.text()}`);
+    const data = (await res.json()) as { logs: string; state?: string };
+    switch (data.state) {
+      case "no_container":
+        return {
+          content: [{ type: "text", text: "(project hasn't been started yet — no container)" }],
+        };
+      case "missing":
+        return {
+          content: [
+            {
+              type: "text",
+              text: "(container_id was recorded but Docker doesn't have it; moor may need to recreate the project)",
+            },
+          ],
+        };
+      case "exited":
+        return {
+          content: [
+            {
+              type: "text",
+              text: `${data.logs || "(no logs captured)"}\n\n(container is exited; logs above are from before)`,
+            },
+          ],
+        };
+      default:
+        // "ok" or undefined (older API) — render raw.
+        return {
+          content: [{ type: "text", text: data.logs || "(no logs)" }],
+        };
+    }
   },
 );