@kenkaiiii/ggcoder 5.6.1 → 5.6.3

package/dist/app-sidecar.js

@@ -23,10 +23,9 @@ import { AgentSession } from "./core/agent-session.js";
 import { buildKenSystemPrompt, buildKenAutopilotSystemPrompt } from "./core/ken-prompt.js";
 import { buildKenDigest, buildKenAutopilotContext } from "./core/ken-context.js";
 import { parseAutopilotVerdict } from "./core/autopilot-verdict.js";
-import { isWorkflowCommandText, countAssistantMessages, shouldStartAutopilotCycle, } from "./core/autopilot-gate.js";
+import { isWorkflowCommandText, countAssistantMessages, shouldStartAutopilotCycle, extractTurnToolCalls, isMechanicalOnlyTurn, } from "./core/autopilot-gate.js";
 import { driveAutopilotCycle } from "./core/autopilot-cycle.js";
 import { validateKenModelPref, effectiveKenModel } from "./core/ken-model.js";
-import { collectProjectContext } from "./system-prompt.js";
 import { AuthStorage } from "./core/auth-storage.js";
 import { MOONSHOT_OAUTH_KEY, XIAOMI_CREDITS_KEY } from "@kenkaiiii/gg-core";
 import { loginAnthropic } from "./core/oauth/anthropic.js";
@@ -623,17 +622,18 @@ function lastAssistantText(messages) {
     return "";
 }
 /**
- * Assemble Ken's context digest for one `@Ken` question: project docs (up the
- * tree) + git/env + the build session's compaction summary + recent activity.
- * Prepended to the user's question as Ken's prompt body each turn. Workflow
- * commands + autopilot-injected prompts are passed through so the digest
- * labels them as what they are instead of user-authored asks.
+ * Assemble Ken's context digest for one `@Ken` question: git/env + the build
+ * session's compaction summary + recent activity. Prepended to the user's
+ * question as Ken's prompt body each turn. Project docs (CLAUDE.md/AGENTS.md)
+ * are NOT here — they're folded into Ken's cached system prompt once per
+ * session instead (see ken-prompt.ts), so they hit the provider prompt cache
+ * instead of being re-sent uncached on every question. Workflow commands +
+ * autopilot-injected prompts are passed through so the digest labels them as
+ * what they are instead of user-authored asks.
  */
-async function buildKenContext(buildSession, cwd, gitBranch, question, workflowCommands, injectedPrompts) {
-    const projectContext = await collectProjectContext(cwd).catch(() => []);
+function buildKenContext(buildSession, cwd, gitBranch, question, workflowCommands, injectedPrompts) {
     return buildKenDigest({
         question,
-        projectContext,
         cwd,
         gitBranch,
         messages: buildSession.getMessages(),
@@ -912,11 +912,14 @@ async function createSession(deps, opts) {
             provider: target.provider,
             model: target.model,
             cwd,
-            systemPrompt: buildKenSystemPrompt(),
+            systemPrompt: await buildKenSystemPrompt(cwd),
             allowedTools: KEN_ALLOWED_TOOLS,
             allowedMcpServers: KEN_ALLOWED_MCP_SERVERS,
             transient: true,
             signal: kenAbort.signal,
+            // Ken's bursty, spread-out turns (chat) outlast the default 5-min cache
+            // TTL regardless of the user's global speedProfile pick.
+            forceLongCacheRetention: true,
         });
         await ken.initialize();
         // Bridge Ken's bus to the shared SSE fan-out with ken_-prefixed types so the
@@ -978,11 +981,14 @@ async function createSession(deps, opts) {
             provider: target.provider,
             model: target.model,
             cwd,
-            systemPrompt: buildKenAutopilotSystemPrompt(),
+            systemPrompt: await buildKenAutopilotSystemPrompt(cwd),
             allowedTools: KEN_ALLOWED_TOOLS,
             allowedMcpServers: KEN_ALLOWED_MCP_SERVERS,
             transient: true,
             signal: kenAutoAbort.signal,
+            // Autopilot review rounds routinely span the injected GG Coder run
+            // (often >5 min) regardless of the user's global speedProfile pick.
+            forceLongCacheRetention: true,
         });
         await ken.initialize();
         // Deliberately no bus bridge: the review is silent. Errors surface via the
@@ -1062,9 +1068,7 @@ async function createSession(deps, opts) {
         broadcast("autopilot_review_start", {});
         try {
             const ken = await ensureKenAutoSession();
-            const projectContext = await collectProjectContext(cwd).catch(() => []);
             const digest = buildKenAutopilotContext({
-                projectContext,
                 cwd,
                 gitBranch,
                 messages: session.getMessages(),
@@ -1121,9 +1125,25 @@ async function createSession(deps, opts) {
                 onInjected: (body, round) => {
                     injectedAutopilotPrompts.push(body);
                     broadcast("autopilot_prompted", { round, body });
+                    void session.persistAutopilotMarker("prompted", { body });
                 },
                 runPrompt: (body) => runAgent(body, () => session.prompt(body)),
-                emit: (event) => broadcast(event.type, event.data),
+                emit: (event) => {
+                    broadcast(event.type, event.data);
+                    // Persist the terminal verdict marker so a resumed session renders the
+                    // same Ken bubble the live run showed instead of dropping it or
+                    // falling back to the raw verdict text (e.g. ALL_CLEAR).
+                    if (event.type === "autopilot_done") {
+                        void session.persistAutopilotMarker("done");
+                    }
+                    else if (event.type === "autopilot_human") {
+                        void session.persistAutopilotMarker("human", { reason: event.data.reason });
+                    }
+                    else if (event.type === "autopilot_capped") {
+                        void session.persistAutopilotMarker("capped");
+                    }
+                    // autopilot_ignored renders nothing live, so nothing is persisted either.
+                },
             });
         }
         finally {
@@ -1157,6 +1177,7 @@ async function createSession(deps, opts) {
                 const workflowCommand = next.attachments.length === 0 &&
                     isWorkflowCommandText(next.text, await loadWorkflowCommandSpecs());
                 const assistantsBefore = countAssistantMessages(session.getMessages());
+                const messagesBefore = session.getMessages().length;
                 await runAgent(next.text, async () => {
                     if (next.attachments.length > 0) {
                         await session.promptWithAttachments(next.text, next.attachments);
@@ -1171,6 +1192,11 @@ async function createSession(deps, opts) {
                     planMode: session.getPlanMode(),
                     workflowCommand,
                     assistantMessagesAdded: countAssistantMessages(session.getMessages()) - assistantsBefore,
+                    // Skip the review API call outright for turns that only started a
+                    // background process (dev server/watcher), ran a read-only lookup, or
+                    // committed/pushed — Ken's autopilot contract already IGNOREs these,
+                    // so there's no reason to pay for that verdict.
+                    mechanicalOnly: isMechanicalOnlyTurn(extractTurnToolCalls(session.getMessages(), messagesBefore)),
                 });
                 if (decision.start) {
                     await runAutopilotCycle(next.text);
@@ -1505,9 +1531,37 @@ async function createSession(deps, opts) {
                         history.push({ role: "assistant", text: turn.reply, ken: true });
                     }
                 };
+                // Autopilot verdict markers to interleave, same anchor scheme as Ken
+                // turns — each becomes a single assistant row the webview renders
+                // exactly like the live `autopilot` item (never a raw verdict string).
+                const autopilotByCount = new Map();
+                for (const marker of session.getAutopilotMarkers()) {
+                    const list = autopilotByCount.get(marker.afterMessageCount) ?? [];
+                    list.push(marker);
+                    autopilotByCount.set(marker.afterMessageCount, list);
+                }
+                const flushAutopilot = (count) => {
+                    const markers = autopilotByCount.get(count);
+                    if (!markers)
+                        return;
+                    autopilotByCount.delete(count);
+                    for (const marker of markers) {
+                        history.push({
+                            role: "assistant",
+                            text: "",
+                            autopilot: {
+                                phase: marker.phase,
+                                ...(marker.reason !== undefined ? { reason: marker.reason } : {}),
+                                ...(marker.body !== undefined ? { body: marker.body } : {}),
+                            },
+                        });
+                    }
+                };
                 let nonSystemCount = 0;
-                // Turns recorded before any build message (anchor 0) render at the top.
+                // Turns/markers recorded before any build message (anchor 0) render at
+                // the top.
                 flushKen(0);
+                flushAutopilot(0);
                 for (const msg of messages) {
                     if (msg.role === "system")
                         continue;
@@ -1601,14 +1655,19 @@ async function createSession(deps, opts) {
                             });
                         }
                     }
-                    // Interleave any Ken turns recorded right after this message.
+                    // Interleave any Ken turns / autopilot markers recorded right after
+                    // this message.
                     flushKen(nonSystemCount);
+                    flushAutopilot(nonSystemCount);
                 }
-                // Flush remaining Ken turns whose anchor is at/after the message count
-                // (e.g. asked before any build message, or anchors beyond the current
-                // count after compaction shrank the history) so none are dropped.
+                // Flush remaining Ken turns / autopilot markers whose anchor is at/after
+                // the message count (e.g. recorded before any build message, or anchors
+                // beyond the current count after compaction shrank the history) so none
+                // are dropped.
                 for (const count of [...kenByCount.keys()].sort((a, b) => a - b))
                     flushKen(count);
+                for (const count of [...autopilotByCount.keys()].sort((a, b) => a - b))
+                    flushAutopilot(count);
                 json(res, 200, { history });
             })();
             return;
@@ -1678,6 +1737,7 @@ async function createSession(deps, opts) {
                 // gate reads the post-run value.
                 const workflowCommand = attachments.length === 0 && isWorkflowCommandText(text, await loadWorkflowCommandSpecs());
                 const assistantsBefore = countAssistantMessages(session.getMessages());
+                const messagesBefore = session.getMessages().length;
                 await runAgent(text, async () => {
                     if (attachments.length > 0) {
                         // Persist each attachment under .gg/uploads so files are inspectable
@@ -1708,6 +1768,11 @@ async function createSession(deps, opts) {
                     planMode: session.getPlanMode(),
                     workflowCommand,
                     assistantMessagesAdded: countAssistantMessages(session.getMessages()) - assistantsBefore,
+                    // Skip the review API call outright for turns that only started a
+                    // background process (dev server/watcher), ran a read-only lookup, or
+                    // committed/pushed — Ken's autopilot contract already IGNOREs these,
+                    // so there's no reason to pay for that verdict.
+                    mechanicalOnly: isMechanicalOnlyTurn(extractTurnToolCalls(session.getMessages(), messagesBefore)),
                 });
                 if (decision.start) {
                     await runAutopilotCycle(text);