Skip to content

Emit AgentConfigUpdate in OTLP session logs#5601

Merged
theomonnom merged 4 commits into
mainfrom
theo/emit-agent-config-update-otlp
May 8, 2026
Merged

Emit AgentConfigUpdate in OTLP session logs#5601
theomonnom merged 4 commits into
mainfrom
theo/emit-agent-config-update-otlp

Conversation

@theomonnom
Copy link
Copy Markdown
Member

No description provided.

@chenghao-mou chenghao-mou requested a review from a team April 29, 2026 19:39
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

@theomonnom theomonnom merged commit 38c126e into main May 8, 2026
23 of 24 checks passed
@theomonnom theomonnom deleted the theo/emit-agent-config-update-otlp branch May 8, 2026 16:25
Copy link
Copy Markdown
Contributor

This is an automated Claude Code Routine created by @toubatbrian. Right now it is in experimentation stage. The automation will start porting this PR into agents-js automatically.

Notes about this port:

  • agents-js does not yet have the AgentConfigUpdate chat item ported (there are existing TODO(parity) markers in agents/src/llm/chat_context.ts and agents/src/voice/agent_activity.ts). To make this PR meaningful in JS, the port will also build the prerequisite: adding the AgentConfigUpdate chat item, wiring it into ChatItem, recording the initial config and updates, and emitting it in the OTLP chat-item proto.
  • The JS PR will tag @toubatbrian and @livekit/agent-devs for review.

Generated by Claude Code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants