@dreb/coding-agent 2.4.2 → 2.4.4

package/dist/core/agent-session.js

@@ -392,10 +392,62 @@ export class AgentSession {
         // _processAgentEvent, slow earlier queued events can delay agent_end processing
         // and waitForRetry() can miss the in-flight retry.
         this._createRetryPromiseForAgentEnd(event);
-        this._agentEventQueue = this._agentEventQueue.then(() => this._processAgentEvent(event), () => this._processAgentEvent(event));
-        // Keep queue alive if an event handler fails
-        this._agentEventQueue.catch(() => { });
+        this._agentEventQueue = this._agentEventQueue.then(() => this._processAgentEvent(event), () => {
+            // Prior event failed — already warned by the .catch() below.
+            // Swallow the old rejection and continue with the current event.
+            return this._processAgentEvent(event);
+        });
+        // Prevent unhandled rejection and warn once per error.
+        // This fires for the CURRENT event's failure; the next event's rejection
+        // handler above silently continues without re-warning.
+        this._agentEventQueue.catch((err) => {
+            this.warnInSession(`Event queue error: ${err instanceof Error ? err.message : String(err)}`);
+        });
     };
+    /**
+     * Collect all resource diagnostics from a loader and surface them as a session warning.
+     * Used after both initial load (sdk.ts) and user-initiated reload.
+     */
+    warnResourceDiagnostics(resourceLoader) {
+        const diagnostics = [
+            ...resourceLoader.getSkills().diagnostics,
+            ...resourceLoader.getPrompts().diagnostics,
+            ...resourceLoader.getThemes().diagnostics,
+            ...resourceLoader.getContextDiagnostics(),
+        ];
+        const extErrors = resourceLoader.getExtensions().errors;
+        if (diagnostics.length === 0 && extErrors.length === 0)
+            return;
+        const lines = [
+            ...diagnostics.map((d) => `- [${d.type}] ${d.message}${d.path ? ` (${d.path})` : ""}`),
+            ...extErrors.map((e) => `- [error] Extension: ${e.path}: ${e.error}`),
+        ];
+        this.warnInSession(`Resource loading issues:\n${lines.join("\n")}`);
+    }
+    /**
+     * Surface a warning in the session so both the human and the AI agent can see it.
+     * During streaming: steers the warning as a user message into the conversation.
+     * Between turns: queues for delivery with the next user prompt.
+     */
+    warnInSession(message, options) {
+        const suffix = options?.informational
+            ? " Note this for context but do not interrupt the current task to discuss it."
+            : " Inform the user about this issue and ask how they would like to proceed.";
+        const warningContent = `[System Warning] ${message}${suffix}`;
+        const warningMessage = {
+            role: "custom",
+            customType: "system_warning",
+            content: warningContent,
+            display: true,
+            timestamp: Date.now(),
+        };
+        if (this.isStreaming) {
+            this.agent.steer(warningMessage);
+        }
+        else {
+            this._pendingNextTurnMessages.push(warningMessage);
+        }
+    }
     _createRetryPromiseForAgentEnd(event) {
         if (event.type !== "agent_end" || this._retryPromise) {
             return;
@@ -1011,6 +1063,7 @@ export class AgentSession {
         }
         catch (err) {
             const message = err instanceof Error ? err.message : String(err);
+            this.warnInSession(`Skill expansion failed for "${skillName}": ${message}`);
             if (this._extensionRunner) {
                 this._extensionRunner.emitError({
                     extensionPath: skill.filePath,
@@ -1018,9 +1071,6 @@ export class AgentSession {
                     error: message,
                 });
             }
-            else {
-                console.error(`Skill expansion error for "${skillName}": ${message}`);
-            }
             return text; // Return original on error
         }
     }
@@ -1763,14 +1813,20 @@ export class AgentSession {
                     this.agent.replaceMessages(messages.slice(0, -1));
                 }
                 setTimeout(() => {
-                    this.agent.continue().catch(() => { });
+                    this.agent.continue().catch((err) => {
+                        // Agent failed to continue after auto-compaction — surface to session
+                        this.warnInSession(`Agent failed to continue after compaction: ${err instanceof Error ? err.message : String(err)}`);
+                    });
                 }, 100);
             }
             else if (this.agent.hasQueuedMessages()) {
                 // Auto-compaction can complete while follow-up/steering/custom messages are waiting.
                 // Kick the loop so queued messages are actually delivered.
                 setTimeout(() => {
-                    this.agent.continue().catch(() => { });
+                    this.agent.continue().catch((err) => {
+                        // Agent failed to continue after auto-compaction — surface to session
+                        this.warnInSession(`Agent failed to continue after compaction: ${err instanceof Error ? err.message : String(err)}`);
+                    });
                 }, 100);
             }
         }
@@ -2152,6 +2208,8 @@ export class AgentSession {
             await this._extensionRunner.emit({ type: "session_start" });
             await this.extendResourcesFromExtensions("reload");
         }
+        // After reload completes, surface any resource diagnostics to the session
+        this.warnResourceDiagnostics(this._resourceLoader);
     }
     // =========================================================================
     // Auto-Retry
@@ -2169,7 +2227,7 @@ export class AgentSession {
             return false;
         const err = message.errorMessage;
         // Match: overloaded_error, provider returned error, rate limit, 429, 500, 502, 503, 504, service unavailable, network/connection errors, fetch failed, terminated, retry delay exceeded
-        return /overloaded|provider.?returned.?error|rate.?limit|too many requests|429|500|502|503|504|service.?unavailable|server.?error|internal.?error|network.?error|connection.?error|connection.?refused|other side closed|fetch failed|upstream.?connect|reset before headers|socket hang up|timed? out|timeout|terminated|retry delay/i.test(err);
+        return /overloaded|provider.?returned.?error|rate.?limit|too many requests|429|500|502|503|504|service.?unavailable|server.?error|internal.?error|network.?error|connection.?error|connection.?refused|other side closed|fetch failed|ended without|upstream.?connect|reset before headers|socket hang up|timed? out|timeout|terminated|retry delay/i.test(err);
     }
     /**
      * Handle retryable errors with exponential backoff.
@@ -2236,8 +2294,9 @@ export class AgentSession {
         this._retryAbortController = undefined;
         // Retry via continue() - use setTimeout to break out of event handler chain
         setTimeout(() => {
-            this.agent.continue().catch(() => {
-                // Retry failed - will be caught by next agent_end
+            this.agent.continue().catch((err) => {
+                // Retry failed — surface to session so user knows
+                this.warnInSession(`Agent retry failed: ${err instanceof Error ? err.message : String(err)}`);
             });
         }, 0);
         return true;