@kenkaiiii/ggcoder 5.19.4 → 5.19.6

package/dist/app-sidecar.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, shouldCaptureToolFailure, shouldCaptureUsagePollingError, 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) });
     }
 }
@@ -452,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);
     });
@@ -590,6 +597,9 @@ async function main() {
         }
         catch (error) {
             const message = error instanceof Error ? error.message : String(error);
+            if (shouldCaptureUsagePollingError(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);
@@ -649,7 +659,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.
@@ -693,6 +703,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 });
                 }
@@ -729,6 +740,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),
                 });
@@ -744,7 +756,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.
@@ -889,6 +901,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),
             });
@@ -920,6 +933,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),
         });
@@ -1010,6 +1024,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,
@@ -1067,6 +1086,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),
                     });
@@ -1185,6 +1205,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),
             });
@@ -1239,6 +1260,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 && shouldCaptureToolFailure(name, d.result)) {
+            // Expected model-correctable validation failures stay in the local log and
+            // conversation. Unexpected failures are reported without private result data.
+            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),
@@ -1579,6 +1605,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),
                     });
@@ -2058,7 +2085,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/")) {
@@ -2071,14 +2101,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/")) {
@@ -2091,7 +2127,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?"))) {
@@ -2204,6 +2243,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) });
                 }
             });
@@ -2214,6 +2254,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),
                 });
@@ -2247,7 +2288,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.
@@ -2258,7 +2302,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")) {
@@ -2266,6 +2313,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),
                 });
@@ -2833,6 +2881,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 });
                 }
@@ -3158,6 +3207,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",
@@ -3185,6 +3235,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;
@@ -3239,6 +3290,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) });
                 }
             });
@@ -3335,6 +3387,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),
@@ -3476,6 +3529,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) });
                 }
             })();
@@ -3502,6 +3556,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),
                 });
@@ -3558,6 +3613,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),
                     });
@@ -3647,6 +3703,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),
@@ -3693,7 +3750,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);