Skip to content

Update PR 1630 branch with current main#1650

Closed
Chibionos wants to merge 6 commits into
UiPath:fix/ae-1482-trim-legacy-trajectoryfrom
Chibionos:update/pr-1630-merge-main
Closed

Update PR 1630 branch with current main#1650
Chibionos wants to merge 6 commits into
UiPath:fix/ae-1482-trim-legacy-trajectoryfrom
Chibionos:update/pr-1630-merge-main

Conversation

@Chibionos
Copy link
Copy Markdown
Contributor

Updates #1630's head branch through a PR-mediated path because direct pushes and gh pr update-branch are blocked.\n\nChanges in this branch:\n- merges current origin/main into fix/ae-1482-trim-legacy-trajectory\n- fixes the LegacyTrajectoryEvaluator test mypy metadata issue\n- bumps packages/uipath to 2.10.66 because 2.10.64 and 2.10.65 are already published\n\nLocal validation:\n- GITHUB_EVENT_NAME=pull_request BASE_SHA=origin/main HEAD_SHA=HEAD python .github/scripts/check_version_uniqueness.py\n- cd packages/uipath && uv run mypy --config-file pyproject.toml .\n- cd packages/uipath && uv run ruff check .\n- cd packages/uipath && uv run ruff format --check .

@Chibionos
Copy link
Copy Markdown
Contributor Author

Closing because merge commits are disabled for this repository; replacing with a linear CI-fix PR into the branch.

@Chibionos Chibionos closed this May 19, 2026
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a643f24f7b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

"""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"
return f"{self.base_url}/api/Traces/spans?traceId={trace_id}&source=CodedAgents"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Require the matching platform package for coded-agent traces

When users upgrade uipath in an environment that already has uipath-platform 0.1.47/0.1.48, packages/uipath/pyproject.toml:10 still considers that dependency satisfied, and python -m pip install --help shows the default upgrade strategy is only-if-needed (dependencies upgrade only when they no longer satisfy requirements). In that common upgrade path this new URL sends source=CodedAgents, but _SpanUtils is still loaded from the old platform package and emits Source=4/old reference mapping, so traces can be classified inconsistently; bump the minimum dependency to the platform version that contains the companion DEFAULT_SOURCE=10 change.

Useful? React with 👍 / 👎.

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