@rubytech/create-maxy-code 0.1.221 → 0.1.222

package/payload/platform/plugins/admin/mcp/dist/index.js

@@ -1397,6 +1397,15 @@ eagerTool(server, "logs-read", "Read recent logs. Stream logs (type=agent-stream
     if (!ACCOUNT_ID)
         return refuseNoAccount("logs-read");
     try {
+        // Account-scoped per-session log dir. `<installDir>/data/accounts/
+        // <accountId>/logs/` mirrors the resolution in
+        // `platform/scripts/logs-read.sh`. This dir may not exist on a Pi
+        // where no claude session has yet tee'd stream output — that is NOT
+        // a fatal condition for platform-scoped types like `server` / `vnc`,
+        // which resolve against CONFIG_DIR below. Task 579 — the previous
+        // early-existence check returned "Log directory does not exist" for
+        // every type, including those whose actual target file lived
+        // elsewhere. Each branch now checks its own resolved path.
         const LOG_DIR = resolve(getAccountDir(), "logs");
         // sessionKey is the single identifier on disk.
         // `sessionId` is retained as a legacy alias that maps to the
@@ -1404,7 +1413,12 @@ eagerTool(server, "logs-read", "Read recent logs. Stream logs (type=agent-stream
         const idForResolve = sessionKey ?? sessionId;
         if (idForResolve) {
             if (!existsSync(LOG_DIR)) {
-                return { content: [{ type: "text", text: `Log directory does not exist: ${LOG_DIR}` }] };
+                return {
+                    content: [{
+                            type: "text",
+                            text: `No per-session logs for this account yet (resolved dir does not exist on disk): ${LOG_DIR}. The directory is created on first claude session spawn.`,
+                        }],
+                };
             }
             const prefixMap = {
                 "agent-stream": "claude-agent-stream",
@@ -1443,9 +1457,10 @@ eagerTool(server, "logs-read", "Read recent logs. Stream logs (type=agent-stream
                     }],
             };
         }
-        if (!existsSync(LOG_DIR)) {
-            return { content: [{ type: "text", text: `Log directory does not exist: ${LOG_DIR}` }] };
-        }
+        // Note: no early existence check on LOG_DIR here. Platform-scoped
+        // types (server, vnc, heartbeat) resolve against CONFIG_DIR below
+        // and must run even when the account log dir has not been created
+        // yet. Account-scoped types check LOG_DIR before they reach for it.
         // --- Session-key filtered mode: grep across log files ---
         if (sessionKey) {
             const prefixes = {
@@ -1463,7 +1478,13 @@ eagerTool(server, "logs-read", "Read recent logs. Stream logs (type=agent-stream
             const searchPrefixes = type && type !== "heartbeat" && type !== "server" && type !== "vnc" && prefixes[type]
                 ? { [type]: prefixes[type] }
                 : (type === "server" || type === "vnc") ? {} : prefixes;
-            const allFiles = readdirSync(LOG_DIR).filter(f => f.endsWith(".log"));
+            // LOG_DIR may not exist on a Pi with no per-session logs yet. The
+            // platform-scoped fallback dirs (CONFIG_DIR/logs/) are searched
+            // below; this branch simply contributes nothing rather than
+            // throwing on readdir.
+            const allFiles = existsSync(LOG_DIR)
+                ? readdirSync(LOG_DIR).filter(f => f.endsWith(".log"))
+                : [];
             const sections = [];
             // Search account-scoped prefix-based log files
             for (const [logType, prefix] of Object.entries(searchPrefixes)) {
@@ -1571,13 +1592,24 @@ eagerTool(server, "logs-read", "Read recent logs. Stream logs (type=agent-stream
                 : resolvedType === "public" ? "public-agent-stream-"
                     : resolvedType === "mcp" ? "mcp-"
                         : "claude-agent-stream-";
-        // Find the most recently modified file with the matching prefix
+        // Find the most recently modified file with the matching prefix.
+        // LOG_DIR may not exist yet on a Pi with no per-session logs; return
+        // the resolved path in the error envelope so the operator can
+        // verify on disk in one step rather than re-tracing the resolver.
+        if (!existsSync(LOG_DIR)) {
+            return {
+                content: [{
+                        type: "text",
+                        text: `No per-session logs for this account yet (resolved dir does not exist on disk): ${LOG_DIR}. The directory is created on first claude session spawn.`,
+                    }],
+            };
+        }
         const match = readdirSync(LOG_DIR)
             .filter(f => f.startsWith(prefix) && f.endsWith(".log"))
             .map(f => ({ file: f, mtime: statSync(resolve(LOG_DIR, f)).mtimeMs }))
             .sort((a, b) => b.mtime - a.mtime)[0];
         if (!match) {
-            return { content: [{ type: "text", text: `No log file found for type '${resolvedType}' in ${LOG_DIR}` }] };
+            return { content: [{ type: "text", text: `No log file found for type '${resolvedType}' (resolved dir: ${LOG_DIR}, prefix: ${prefix})` }] };
         }
         const logFile = resolve(LOG_DIR, match.file);
         const result = execFileSync("tail", ["-n", String(lines), logFile], {