codeep 2.23.0 → 2.24.0

package/dist/utils/agentChat.js

@@ -259,7 +259,7 @@ export async function summarizeEarlierHistory(history, maxChars = 16000) {
     if (dropped.length === 0)
         return '';
     const key = createHash('sha256')
-        .update(dropped.map(m => `${m.role}:${m.content}`).join(''))
+        .update(dropped.map(m => `${m.role}:${m.content}`).join('\u0000'))
         .digest('hex');
     const cached = earlierSummaryCache.get(key);
     if (cached)
@@ -507,7 +507,12 @@ additionalTools, runtime) {
             if (errorText.includes('tools') || errorText.includes('function') || response.status === 400) {
                 return await agentChatFallback(messages, systemPrompt, onChunk, abortSignal, dynamicTimeout, additionalTools, runtime);
             }
-            throw new Error(`API error: ${response.status} - ${errorText}`);
+            // ApiError, not Error: runAgent's retry loop refuses to retry a 4xx by
+            // checking `err instanceof ApiError && err.status`. A bare Error made
+            // that guard silently inapplicable, so an expired key was retried once
+            // per iteration until the budget ran out and the run reported "Exceeded
+            // maximum of N iterations" — the one explanation unrelated to the cause.
+            throw new ApiError(`API error: ${response.status} - ${errorText}`, response.status);
         }
         if (useStreaming && response.body) {
             if (protocol === 'openai')
@@ -658,7 +663,9 @@ export async function agentChatFallback(messages, systemPrompt, onChunk, abortSi
         });
         if (!response.ok) {
             const error = await response.text();
-            throw new Error(`API error: ${response.status} - ${error}`);
+            // Same reasoning as the native-tools path: the status has to survive, or
+            // the caller cannot tell "your key is wrong" from "the network hiccuped".
+            throw new ApiError(`API error: ${response.status} - ${error}`, response.status);
         }
         let content;
         if (onChunk && response.body) {

package/dist/utils/headlessReview.js

@@ -1,4 +1,4 @@
-import { buildFixPlan, summariseFixPlan } from './reviewFix.js';
+import { buildFixPlan, summariseFixPlan, describeAgentActivity } from './reviewFix.js';
 // Headless `codeep review` — a non-interactive entry point around the
 // deterministic reviewer in codeReview.ts. No API key, no TUI: it scans, prints
 // a report (markdown or JSON), and exits non-zero when issues at/above a chosen
@@ -214,6 +214,18 @@ function defaultDeps() {
  */
 async function runFixPlan(plan, context) {
     try {
+        // Populate the key cache before anything asks for it. `getApiKey` is
+        // synchronous and reads the cache alone — it does not consult the
+        // environment — so without this every request goes out with an empty
+        // bearer token and the provider answers 401. The AI review path next door
+        // has always done this; the fix path never did, which is why a CI run with
+        // a perfectly good key in the environment spent its whole iteration budget
+        // being rejected.
+        const { loadAllApiKeys, isConfigured } = await import('../config/index.js');
+        await loadAllApiKeys();
+        if (!isConfigured()) {
+            return `${summariseFixPlan(plan)} No API key is configured for the current provider, so the fix agent could not start.`;
+        }
         const { runAgent } = await import('./agent.js');
         // No cast here. `as never` on this call once hid the fact that
         // personalityOverride did not exist, which would have run the CI fix with
@@ -230,11 +242,12 @@ async function runFixPlan(plan, context) {
         const edited = new Set(result.actions
             .filter(a => a.type === 'write' || a.type === 'edit')
             .map(a => a.target));
+        const activity = describeAgentActivity(result.actions);
         if (!result.success) {
-            return `${summariseFixPlan(plan)} The run did not finish: ${result.error ?? 'unknown error'}.`;
+            return `${summariseFixPlan(plan)} The run did not finish: ${result.error ?? 'unknown error'}. ${activity}`;
         }
         if (edited.size === 0) {
-            return `${summariseFixPlan(plan)} Nothing was changed — the agent judged the findings not mechanically fixable.`;
+            return `${summariseFixPlan(plan)} Nothing was changed. ${activity}`;
         }
         return `${summariseFixPlan(plan)} Edited ${edited.size} file${edited.size === 1 ? '' : 's'}: ${[...edited].join(', ')}.`;
     }

package/dist/utils/reviewFix.d.ts

@@ -63,3 +63,21 @@ export declare function buildFixPlan(issues: ReviewIssue[], options?: FixPlanOpt
 export declare function formatFixPrompt(issues: ReviewIssue[]): string;
 /** A one-line summary for the pull request body the action opens. */
 export declare function summariseFixPlan(plan: FixPlan): string;
+/**
+ * What the agent actually did, in one line.
+ *
+ * A fix that changes nothing is the hardest outcome to act on, because the
+ * summary that reports it — "the run did not finish", "nothing was changed" —
+ * says what did not happen and never what did. Debugging one such run through
+ * CI cost two releases and forty minutes of guessing at whether the agent
+ * could not find the file, could not write, or was being refused a tool.
+ *
+ * Counting the action log answers that in the message itself. Failures are
+ * called out separately from successes, and one failing detail is quoted,
+ * because a refusal reason is usually the whole explanation.
+ */
+export declare function describeAgentActivity(actions: {
+    type: string;
+    result: string;
+    details?: string;
+}[]): string;

package/dist/utils/reviewFix.js

@@ -139,3 +139,35 @@ export function summariseFixPlan(plan) {
     ].filter(Boolean);
     return `Attempting ${parts.join(' and ')} across ${plan.files.length} file${plan.files.length === 1 ? '' : 's'}.`;
 }
+/**
+ * What the agent actually did, in one line.
+ *
+ * A fix that changes nothing is the hardest outcome to act on, because the
+ * summary that reports it — "the run did not finish", "nothing was changed" —
+ * says what did not happen and never what did. Debugging one such run through
+ * CI cost two releases and forty minutes of guessing at whether the agent
+ * could not find the file, could not write, or was being refused a tool.
+ *
+ * Counting the action log answers that in the message itself. Failures are
+ * called out separately from successes, and one failing detail is quoted,
+ * because a refusal reason is usually the whole explanation.
+ */
+export function describeAgentActivity(actions) {
+    if (actions.length === 0)
+        return 'It made no tool calls at all.';
+    const byType = new Map();
+    for (const action of actions) {
+        const tally = byType.get(action.type) ?? { ok: 0, failed: 0 };
+        if (action.result === 'error')
+            tally.failed++;
+        else
+            tally.ok++;
+        byType.set(action.type, tally);
+    }
+    const parts = [...byType.entries()]
+        .sort((a, b) => (b[1].ok + b[1].failed) - (a[1].ok + a[1].failed))
+        .map(([type, { ok, failed }]) => (failed ? `${ok + failed} ${type} (${failed} failed)` : `${ok} ${type}`));
+    const firstFailure = actions.find(a => a.result === 'error' && a.details);
+    const why = firstFailure ? ` First failure: ${firstFailure.details.slice(0, 200)}` : '';
+    return `It made ${actions.length} tool call${actions.length === 1 ? '' : 's'}: ${parts.join(', ')}.${why}`;
+}

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const VERSION = "2.23.0";
+export declare const VERSION = "2.24.0";

package/dist/version.js

@@ -1,4 +1,4 @@
 // AUTO-GENERATED by scripts/gen-version.js — do not edit by hand.
 // Baked from package.json at build time so the bun-compiled binary reports
 // the right version (it has no package.json on disk to read at runtime).
-export const VERSION = '2.23.0';
+export const VERSION = '2.24.0';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codeep",
-  "version": "2.23.0",
+  "version": "2.24.0",
   "description": "AI-powered coding assistant built for the terminal. Multiple LLM providers, project-aware context, and a seamless development workflow.",
   "type": "module",
   "main": "dist/index.js",