ag-ui-protocol / ag-ui-protocol/ag-ui

AbstractAgent.onError uses hardcoded console.error, bypassing host application loggers

オープン
#2,360 コメント 3 件 リアクション 1 件 担当者 0 名 GitHub で見る
主要言語
Python
スター
15.9k
フォーク
1.4k
平均マージ
1日 17時間
マージ済み PR(30日)
163

説明

## Problem

`AbstractAgent.onError` in `sdks/typescript/packages/client/src/agent/agent.ts` (line ~507) calls `console.error` directly:

```ts
console.error('Agent execution failed:', error)
```

This bypasses any structured logging the host application has configured (e.g. Winston, Pino, or other JSON loggers). When the error object contains a stack trace — as `MastraError` does — Node.js serialises it as a multi-line string, producing unstructured, multi-line log entries in log aggregation systems (e.g. Humio, Datadog, Splunk) that expect newline-delimited JSON.

## Impact

- Log aggregation pipelines that parse NDJSON receive broken, multi-line entries for every agent execution failure.
- The error is invisible to the host application's error tracking / alerting stack unless the host also intercepts `console.error` globally.
- There is no way to suppress or redirect these logs without monkey-patching the global `console`.

## Suggested fix

Expose a configurable `logger` option on `AbstractAgent` (or the `RunAgentInput` / constructor options) and use it inside `onError` instead of `console.error`. Falling back to `console.error` when no logger is provided keeps the current behaviour for users who have not opted in.

```ts
// example API
const agent = new MyAgent({ logger: myStructuredLogger });

// inside AbstractAgent.onError
(this.logger ?? console).error('Agent execution failed:', error)
```

Alternatively, an `onError` callback hook would give host applications the same control.

## Environment

- Package: `@ag-ui/client`
- Affected method: `AbstractAgent.onError`
- Node.js serialises objects passed to `console.error` using `util.inspect`, which expands stack traces across multiple lines.

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。