Skip to content

fix(tracing): set Traces source to CodedAgents#1626

Open
ionmincu wants to merge 2 commits into
mainfrom
fix/traces-source-coded-agents
Open

fix(tracing): set Traces source to CodedAgents#1626
ionmincu wants to merge 2 commits into
mainfrom
fix/traces-source-coded-agents

Conversation

@ionmincu
Copy link
Copy Markdown
Contributor

Summary

  • Change source=Robots to source=CodedAgents in the /api/Traces/spans URL built by LlmOpsHttpExporter
  • Update related tests
  • Bump uipath package version to 2.10.63

Test plan

  • pytest packages/uipath/tests/tracing/test_otel_exporters.py

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added test:uipath-langchain Triggers tests in the uipath-langchain-python repository test:uipath-integrations labels May 12, 2026
def _build_url(self, span_list: list[Dict[str, Any]]) -> str:
"""Construct the URL for the API request."""
trace_id = str(span_list[0]["TraceId"])
return f"{self.base_url}/api/Traces/spans?traceId={trace_id}&source=Robots"
Copy link
Copy Markdown
Collaborator

@radu-mocanu radu-mocanu May 13, 2026

Choose a reason for hiding this comment

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

are we sure this won t break low code?
we need to double-check that all the spans we emit are dropped in by their runtime

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@saksharthakkar / @JosephMar could validate this change from a tracing perspective

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This also needs to be changed


source exists both inside the model and on the query path (for convenience)

@@ -1,6 +1,6 @@
[project]
name = "uipath"
version = "2.10.62"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

shouldn't we also re-generate the uv.lock?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

probably

Aligns the model-side source value with the query-path change from
the previous commit (source=CodedAgents).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
50.0% Coverage on New Code (required ≥ 90%)

See analysis details on SonarQube Cloud

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

Labels

test:uipath-integrations test:uipath-langchain Triggers tests in the uipath-langchain-python repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants