marketing-shibata50/multica
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(agent): only emit agent_error for auth issues, not runtime 400 errors

Browse source 
Previously all agent errors (including 400 invalid_request_error) were
emitted as agent_error events, triggering the UI error banner and
interrupting the chat flow. Now only auth-related errors (401, no API key)
emit agent_error so the "Configure" banner appears. Runtime errors like
400 are still shown as chat messages but don't block the agent.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Jiang Bohan 2026-02-09 15:51:32 +08:00
parent 15f6604a16
commit 7294e76929
1 changed files with 13 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

16
src/agent/async-agent.ts
View file

@ -73,15 +73,25 @@ export class AsyncAgent {
if (result.error) {
console.error(`[AsyncAgent] Agent run error: ${result.error}`);
this.channel.send({ id: uuidv7(), content: `[error] ${result.error}` });
this.agent.emitError(result.error);
// Only emit agent_error for HTTP 401 from the LLM provider so the
// UI shows the "Configure" banner. All other errors (400, tool errors,
// etc.) should flow back to the agent for self-recovery.
if (/\b401\b/.test(result.error)) {
this.agent.emitError(result.error);
}
}
})
.catch((err) => {
const message = err instanceof Error ? err.message : String(err);
console.error(`[AsyncAgent] Agent run exception: ${message}`);
this.channel.send({ id: uuidv7(), content: `[error] ${message}` });
// Also emit through subscriber mechanism so IPC listeners receive the error
this.agent.emitError(message);
// Only emit agent_error for HTTP 401 from the LLM provider so the
// UI shows the "Configure" banner. All other errors (400, tool errors,
// etc.) should flow back to the agent for self-recovery.
const errMsg = err instanceof Error ? err.message : String(err);
if (/\b401\b/.test(errMsg)) {
this.agent.emitError(message);
}
})
.finally(() => {
this.pendingWrites = Math.max(0, this.pendingWrites - 1);