@kenkaiiii/ggcoder 5.19.3 → 5.19.5

package/dist/app-sidecar.js

@@ -55,7 +55,7 @@ import { loadTasksSync, saveTasksSync, pruneDoneTasksSync, getNextPendingTask, m
 import { initLogger, log } from "./core/logger.js";
 import { RADIO_STATIONS, getCurrentStation, getRadioVolume, playRadio, setRadioVolume, stopRadio, } from "./core/radio.js";
 import { enrichProcessPath } from "./core/shell-path.js";
-import { downscaleForPreview, validateVisionImage } from "./utils/image.js";
+import { downscaleForPreview, shrinkToFit, validateVisionImage } from "./utils/image.js";
 import { startServeMode } from "./modes/serve-mode.js";
 import { loadTelegramConfig, saveTelegramConfig, verifyBotToken } from "./core/telegram-config.js";
 import { loadServers, addServer, removeServer, getServer, parseMcpAddCommand, MCPClientManager, McpOAuthStore, } from "./core/mcp/index.js";
@@ -64,6 +64,7 @@ import { loadProgress, peekProgress, updateProgress } from "./core/progress/stor
 import { awardPrompt, awardCommits } from "./core/progress/engine.js";
 import { detectNewCommits, repoKey } from "./core/progress/git-xp.js";
 import { rebuildFromSessions } from "./core/progress/rebuild.js";
+import { captureSidecarError, flushSidecarErrors, wrapSidecarHandler, } from "./core/sidecar-error-reporter.js";
 const ALL_PROVIDERS = [
     // US
     "anthropic",
@@ -169,6 +170,7 @@ async function persistModelSelection(settingsFile, provider, model) {
         await sm.set("defaultModel", model);
     }
     catch (err) {
+        captureSidecarError(err, "app-sidecar.settings.persist-model");
         log("WARN", "app-sidecar", "failed to persist model selection", { err: String(err) });
     }
 }
@@ -185,6 +187,7 @@ async function persistThinkingLevel(settingsFile, level) {
             await sm.set("thinkingLevel", level);
     }
     catch (err) {
+        captureSidecarError(err, "app-sidecar.settings.persist-thinking");
         log("WARN", "app-sidecar", "failed to persist thinking level", { err: String(err) });
     }
 }
@@ -202,19 +205,28 @@ async function prepareAttachments(cwd, attachments) {
         const fileName = `${Date.now().toString(36)}-${safe}`;
         const filePath = path.join(dir, fileName);
         const buf = Buffer.from(a.data, "base64");
-        // Validate image attachments before they become native image content blocks.
-        // A corrupt or unsupported-format image (e.g. a malformed .ico, or a .png
-        // with a bad IDAT) makes the provider reject the ENTIRE turn ("image data
-        // ... not a valid image") before the agent can respond. Downgrade such files
-        // to a plain "file" attachment so the model gets a path note and inspects
-        // them with its tools instead of the request 400ing.
+        // Validate and cap image attachments before they become native image content
+        // blocks. Anthropic applies a 2000 px per-dimension cap once conversation
+        // history contains more than 20 images, so every attachment must be safe for
+        // later turns too. Keep the original on disk, but send the resized bytes.
+        // Corrupt or unsupported images become plain files so they cannot reject the
+        // entire provider request.
         let prepared = { ...a };
         if (a.kind === "image") {
-            const validatedType = await validateVisionImage(buf).catch(() => null);
-            if (validatedType)
-                prepared = { ...a, mediaType: validatedType };
-            else
+            try {
+                const resized = await shrinkToFit(buf, a.mediaType);
+                const validatedType = await validateVisionImage(resized.buffer);
+                prepared = validatedType
+                    ? {
+                        ...a,
+                        mediaType: validatedType,
+                        data: resized.buffer.toString("base64"),
+                    }
+                    : { ...a, kind: "file" };
+            }
+            catch {
                 prepared = { ...a, kind: "file" };
+            }
         }
         try {
             await fs.writeFile(filePath, buf);
@@ -443,7 +455,11 @@ async function runJsonModeIfRequested() {
         maxTurns: maxTurnsRaw ? parseInt(maxTurnsRaw, 10) : undefined,
         allowedTools,
         promptCacheKey: values["prompt-cache-key"],
-    }).catch((err) => {
+    }).catch(async (err) => {
+        captureSidecarError(err, "app-sidecar.json-mode", {
+            provider: String(values.provider ?? "anthropic"),
+        });
+        await flushSidecarErrors();
         process.stderr.write((err instanceof Error ? err.message : String(err)) + "\n");
         process.exit(1);
     });
@@ -581,6 +597,7 @@ async function main() {
         }
         catch (error) {
             const message = error instanceof Error ? error.message : String(error);
+            captureSidecarError(error, "app-sidecar.usage.fetch", { provider });
             log("WARN", "app-sidecar", "subscription usage fetch failed", { provider, message });
             if (error instanceof SubscriptionUsageError && error.status === 429) {
                 usageRateLimitedUntil.set(provider, Date.now() + USAGE_RATE_LIMIT_BACKOFF_MS);
@@ -640,7 +657,7 @@ async function main() {
             return null;
         }
     }
-    const server = http.createServer((req, res) => {
+    const server = http.createServer(wrapSidecarHandler((req, res) => {
         const url = req.url ?? "/";
         const method = req.method ?? "GET";
         // CORS preflight — the webview origin differs from 127.0.0.1.
@@ -684,6 +701,7 @@ async function main() {
                 }
                 catch (err) {
                     const message = err instanceof Error ? err.message : String(err);
+                    captureSidecarError(err, "app-sidecar.session.create");
                     log("ERROR", "app-sidecar", "session create failed", { message });
                     daemonJson(res, 500, { error: message });
                 }
@@ -720,6 +738,7 @@ async function main() {
                 }
                 daemonJson(res, 200, await subscriptionUsage(provider));
             })().catch((error) => {
+                captureSidecarError(error, "app-sidecar.usage.request");
                 log("ERROR", "app-sidecar", "subscription usage request failed", {
                     message: error instanceof Error ? error.message : String(error),
                 });
@@ -735,7 +754,7 @@ async function main() {
             return;
         }
         ctx.handle(req, res, url, method);
-    });
+    }, "app-sidecar.http"));
     server.listen(port, host, () => {
         const addr = server.address();
         // The Rust shell reads this line to learn the daemon port.
@@ -880,6 +899,7 @@ async function createProgressManager(agentDir, broadcastAll) {
             broadcastAll(buildSnapshot(updated), originId);
         }
         catch (err) {
+            captureSidecarError(err, "app-sidecar.progress.award");
             log("DEBUG", "app-sidecar", "progress award failed", {
                 message: err instanceof Error ? err.message : String(err),
             });
@@ -911,6 +931,7 @@ async function createProgressManager(agentDir, broadcastAll) {
         watcher.unref();
     }
     catch (err) {
+        captureSidecarError(err, "app-sidecar.progress.watch");
         log("DEBUG", "app-sidecar", "progress watch unavailable", {
             message: err instanceof Error ? err.message : String(err),
         });
@@ -1001,6 +1022,11 @@ async function createSession(deps, opts) {
         const f = formatError(err);
         const message = f.message ? desktopGuidance(f.message) : undefined;
         const guidance = desktopGuidance(f.guidance);
+        captureSidecarError(err, `app-sidecar.${logLabel.replaceAll(" ", "-")}`, {
+            scope: type,
+            ...(f.provider ? { provider: f.provider } : {}),
+            ...(f.statusCode != null ? { status: String(f.statusCode) } : {}),
+        });
         log("ERROR", "app-sidecar", logLabel, {
             headline: f.headline,
             source: f.source,
@@ -1058,6 +1084,7 @@ async function createSession(deps, opts) {
                 chatAgent = nextAgent;
                 broadcast("chat_agent_change", { chatAgent: nextAgent });
                 await session.persistAppMarker("agent_handoff", { chatAgent: nextAgent }).catch((error) => {
+                    captureSidecarError(error, "app-sidecar.chat-agent.persist-handoff");
                     log("WARN", "app-sidecar", "agent handoff marker persist failed", {
                         message: error instanceof Error ? error.message : String(error),
                     });
@@ -1176,6 +1203,7 @@ async function createSession(deps, opts) {
             .catch(() => false)
             .then(() => syncApprovedPlanProgress(generation))
             .catch((error) => {
+            captureSidecarError(error, "app-sidecar.plan.refresh-progress");
             log("WARN", "app-sidecar", "plan progress refresh failed", {
                 message: error instanceof Error ? error.message : String(error),
             });
@@ -1230,6 +1258,11 @@ async function createSession(deps, opts) {
     session.eventBus.on("tool_call_end", (d) => {
         const name = toolCallNames.get(d.toolCallId) ?? "unknown";
         toolCallNames.delete(d.toolCallId);
+        if (d.isError) {
+            // Tool output can contain private project data. Report the failure and tool
+            // identity without shipping the raw result body to Error Mom.
+            captureSidecarError(new Error(`Tool ${name} failed`), `tool.${name}`, { tool: name });
+        }
         log(d.isError ? "ERROR" : "INFO", "tool", `Tool call ended: ${name}`, {
             id: d.toolCallId,
             durationMs: String(d.durationMs),
@@ -1570,6 +1603,7 @@ async function createSession(deps, opts) {
                 catch (error) {
                     // Keep tracking when prompt cleanup fails; hiding the widget here
                     // would claim completion while the approved-plan contract remained.
+                    captureSidecarError(error, "app-sidecar.plan.cleanup");
                     log("WARN", "app-sidecar", "completed plan cleanup failed", {
                         message: error instanceof Error ? error.message : String(error),
                     });
@@ -2049,7 +2083,10 @@ async function createSession(deps, opts) {
             void memoryStore
                 .snapshot()
                 .then((snapshot) => json(res, 200, snapshot))
-                .catch((error) => json(res, 500, { error: error instanceof Error ? error.message : String(error) }));
+                .catch((error) => {
+                captureSidecarError(error, "app-sidecar.memory.snapshot");
+                json(res, 500, { error: error instanceof Error ? error.message : String(error) });
+            });
             return;
         }
         if (method === "DELETE" && url.startsWith("/memories/")) {
@@ -2062,14 +2099,20 @@ async function createSession(deps, opts) {
                 .forget(id)
                 .then(() => memoryStore.snapshot())
                 .then((snapshot) => json(res, 200, snapshot))
-                .catch((error) => json(res, 500, { error: error instanceof Error ? error.message : String(error) }));
+                .catch((error) => {
+                captureSidecarError(error, "app-sidecar.memory.forget");
+                json(res, 500, { error: error instanceof Error ? error.message : String(error) });
+            });
             return;
         }
         if (method === "GET" && url === "/jiwa") {
             void jiwaStore
                 .snapshot()
                 .then((snapshot) => json(res, 200, snapshot))
-                .catch((error) => json(res, 500, { error: error instanceof Error ? error.message : String(error) }));
+                .catch((error) => {
+                captureSidecarError(error, "app-sidecar.jiwa.snapshot");
+                json(res, 500, { error: error instanceof Error ? error.message : String(error) });
+            });
             return;
         }
         if (method === "DELETE" && url.startsWith("/jiwa/")) {
@@ -2082,7 +2125,10 @@ async function createSession(deps, opts) {
                 .forget(id)
                 .then(() => jiwaStore.snapshot())
                 .then((snapshot) => json(res, 200, snapshot))
-                .catch((error) => json(res, 500, { error: error instanceof Error ? error.message : String(error) }));
+                .catch((error) => {
+                captureSidecarError(error, "app-sidecar.jiwa.forget");
+                json(res, 500, { error: error instanceof Error ? error.message : String(error) });
+            });
             return;
         }
         if (method === "GET" && (url === "/events" || url.startsWith("/events?"))) {
@@ -2195,6 +2241,7 @@ async function createSession(deps, opts) {
                     json(res, 200, { path: dir });
                 }
                 catch (err) {
+                    captureSidecarError(err, "app-sidecar.project.create");
                     json(res, 500, { error: err instanceof Error ? err.message : String(err) });
                 }
             });
@@ -2205,6 +2252,7 @@ async function createSession(deps, opts) {
             void discoverProjects()
                 .then((projects) => json(res, 200, { projects }))
                 .catch((err) => {
+                captureSidecarError(err, "app-sidecar.projects.discover");
                 log("ERROR", "app-sidecar", "discoverProjects failed", {
                     message: err instanceof Error ? err.message : String(err),
                 });
@@ -2238,7 +2286,10 @@ async function createSession(deps, opts) {
                         .map(({ item }) => item);
                     json(res, 200, { sessions: recent });
                 })
-                    .catch(() => json(res, 200, { sessions: [] }));
+                    .catch((error) => {
+                    captureSidecarError(error, "app-sidecar.sessions.list-chat");
+                    json(res, 200, { sessions: [] });
+                });
                 return;
             }
             // The picker may be served by a sidecar currently running in Chat mode.
@@ -2249,7 +2300,10 @@ async function createSession(deps, opts) {
                 : paths.sessionsDir;
             void listRecentSessions(target, 5, sessionsDir)
                 .then((sessions) => json(res, 200, { sessions }))
-                .catch(() => json(res, 200, { sessions: [] }));
+                .catch((error) => {
+                captureSidecarError(error, "app-sidecar.sessions.list");
+                json(res, 200, { sessions: [] });
+            });
             return;
         }
         if (method === "GET" && url.startsWith("/files")) {
@@ -2257,6 +2311,7 @@ async function createSession(deps, opts) {
             void searchProjectFiles(cwd, q)
                 .then((files) => json(res, 200, { files }))
                 .catch((err) => {
+                captureSidecarError(err, "app-sidecar.files.search");
                 log("ERROR", "app-sidecar", "searchProjectFiles failed", {
                     message: err instanceof Error ? err.message : String(err),
                 });
@@ -2824,6 +2879,7 @@ async function createSession(deps, opts) {
                 }
                 catch (err) {
                     const message = err instanceof Error ? err.message : String(err);
+                    captureSidecarError(err, "app-sidecar.prompt-enhancer");
                     log("ERROR", "app-sidecar", "enhance failed", { message });
                     json(res, 500, { error: message });
                 }
@@ -3149,6 +3205,7 @@ async function createSession(deps, opts) {
                     drained,
                 });
             })().catch((error) => {
+                captureSidecarError(error, "app-sidecar.run.cancel");
                 broadcast("cancel_failed", { error: "cancel_failed", runState: runLifecycle.state });
                 json(res, 500, {
                     error: "cancel_failed",
@@ -3176,6 +3233,7 @@ async function createSession(deps, opts) {
                 json(res, 200, { ok: true });
             })
                 .catch((err) => {
+                captureSidecarError(err, "app-sidecar.session.new");
                 json(res, 500, { error: err instanceof Error ? err.message : String(err) });
             });
             return;
@@ -3230,6 +3288,7 @@ async function createSession(deps, opts) {
                     json(res, 200, { ok: true, planTotal });
                 }
                 catch (err) {
+                    captureSidecarError(err, "app-sidecar.plan.accept");
                     json(res, 500, { error: err instanceof Error ? err.message : String(err) });
                 }
             });
@@ -3326,6 +3385,7 @@ async function createSession(deps, opts) {
                         broadcast("auth_done", { provider });
                     }
                     catch (err) {
+                        captureSidecarError(err, "app-sidecar.auth.oauth", { provider });
                         broadcast("auth_error", {
                             provider,
                             message: err instanceof Error ? err.message : String(err),
@@ -3467,6 +3527,7 @@ async function createSession(deps, opts) {
                 }
                 catch (err) {
                     serveController = null;
+                    captureSidecarError(err, "app-sidecar.serve.start", { provider: st.provider });
                     json(res, 400, { error: err instanceof Error ? err.message : String(err) });
                 }
             })();
@@ -3493,6 +3554,7 @@ async function createSession(deps, opts) {
             void buildMcpRows(targetCwd)
                 .then((servers) => json(res, 200, { servers }))
                 .catch((err) => {
+                captureSidecarError(err, "app-sidecar.mcp.list");
                 log("ERROR", "app-sidecar", "buildMcpRows failed", {
                     message: err instanceof Error ? err.message : String(err),
                 });
@@ -3549,6 +3611,7 @@ async function createSession(deps, opts) {
                     });
                 }
                 catch (err) {
+                    captureSidecarError(err, "app-sidecar.mcp.add", { server: config.name });
                     json(res, 500, {
                         error: err instanceof Error ? err.message : String(err),
                     });
@@ -3638,6 +3701,7 @@ async function createSession(deps, opts) {
                     }
                 }
                 catch (err) {
+                    captureSidecarError(err, "app-sidecar.mcp.login", { server: name });
                     broadcast("mcp_auth_error", {
                         name,
                         message: err instanceof Error ? err.message : String(err),
@@ -3684,7 +3748,9 @@ async function createSession(deps, opts) {
         dispose,
     };
 }
-main().catch((err) => {
+main().catch(async (err) => {
+    captureSidecarError(err, "app-sidecar.main", { severity: "fatal" });
+    await flushSidecarErrors();
     const message = err instanceof Error ? err.message : String(err);
     process.stderr.write(`GG_APP_FATAL ${message}\n`);
     process.exit(1);