Skip to content

Conversation

@RoxyFarhad
Copy link
Collaborator

@RoxyFarhad RoxyFarhad commented Sep 30, 2025

This configures logging for DD in the agent if DD is setup in the agent environment. It also adds request IDs to the logs that are passed from the agentex server as part of the request. This will let us track requests across the different services in debugging.

This is a separate request id to the jsonrpc request id which serves a different purpose.

Complimentary PR to add the request passing through

Testing

  • Added this commit as a dep to an agent locally running against the agentex version that passes request and the same id was logged across both serviers.

@socket-security
Copy link

socket-security bot commented Sep 30, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedagentex-sdk@​0.4.8 ⏵ 0.4.18100 +8100100100100

View full report

@danielmillerp
Copy link
Contributor

So does this give us DD in local?

@RoxyFarhad RoxyFarhad merged commit 8c481e3 into main Oct 1, 2025
9 of 10 checks passed
@RoxyFarhad RoxyFarhad deleted the add-better-logging-for-dd branch October 1, 2025 14:46
@RoxyFarhad
Copy link
Collaborator Author

So does this give us DD in local?

@danielmillerp There is a way in theory to use this locally but I haven't tried to test that out yet. It requires installing something from the DD side to use on the local machine.

@RoxyFarhad RoxyFarhad changed the title Add better logging for dd feat: Add better logging for dd Oct 1, 2025
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.

4 participants