-
Notifications
You must be signed in to change notification settings - Fork 269
Add AgentTrafficLog functionality #3707
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add AgentTrafficLog functionality #3707
Conversation
krngrover94
commented
Dec 7, 2025
- Introduced AgentTrafficLog DTO for storing raw agent traffic data.
- Implemented AgentTrafficLogDao for MongoDB operations, including index creation.
- Enhanced Main class to save agent traffic logs after processing responses.
- Added bulk write capability for agent traffic logs in DataActor and DbActor.
- Created dedicated thread pool for agent traffic log writes to optimize performance.
- Introduced AgentTrafficLog DTO for storing raw agent traffic data. - Implemented AgentTrafficLogDao for MongoDB operations, including index creation. - Enhanced Main class to save agent traffic logs after processing responses. - Added bulk write capability for agent traffic logs in DataActor and DbActor. - Created dedicated thread pool for agent traffic log writes to optimize performance.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤖 AI Security analysis: "Automated checks found no security issues in this change. Multiple files were modified, so tooling may miss context-dependent or subtle vulnerabilities; perform manual review and targeted testing before merging."
| Risk Level | AI Score |
|---|---|
| 🟢 NO RISK | 0.0/100 |
Top 0 security issues / 0 total (Critical: 0, High: 0, Medium: 0, Low: 0)
| Title | Location | Recommendation |
|---|---|---|
| — | — | No issues to display |
…curity/akto into mini-runtime-log-agent-traffic
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤖 AI Security analysis: "Automated checks found no security issues in this change. Multiple files were modified, so tooling may miss context-dependent or subtle vulnerabilities; perform manual review and targeted testing before merging."
| Risk Level | AI Score |
|---|---|
| 🟢 NO RISK | 0.0/100 |
Top 0 security issues / 0 total (Critical: 0, High: 0, Medium: 0, Low: 0)
| Title | Location | Recommendation |
|---|---|---|
| — | — | No issues to display |
| ClassModel<McpReconResult> mcpReconResultClassModel = ClassModel.builder(McpReconResult.class).enableDiscriminator(true).build(); | ||
| ClassModel<McpReconRequest> mcpReconRequestClassModel = ClassModel.builder(McpReconRequest.class).enableDiscriminator(true).build(); | ||
| ClassModel<AgentConversationResult> agentConversationResultClassModel = ClassModel.builder(AgentConversationResult.class).enableDiscriminator(true).build(); | ||
| ClassModel<AgentTrafficLog> agentTrafficLogClassModel = ClassModel.builder(AgentTrafficLog.class).enableDiscriminator(true).build(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The daoinit changes and create indices calls are also needed on cyborg and #3677