Skip to content

fix(tracing): use agentId as span reference_id#1628

Open
ionmincu wants to merge 1 commit into
mainfrom
fix/traces-reference-id-from-agent-id
Open

fix(tracing): use agentId as span reference_id#1628
ionmincu wants to merge 1 commit into
mainfrom
fix/traces-reference-id-from-agent-id

Conversation

@ionmincu
Copy link
Copy Markdown
Contributor

@ionmincu ionmincu commented May 13, 2026

Summary

Tracing

  • Set UiPathSpan.reference_id from the agentId attribute (sourced from the PROJECT_KEY env var) instead of the unset referenceId attribute, so coded-agent spans carry a meaningful reference back to their agent.
  • Bump uipath-platform to 0.1.49.

Integration tests workflow

Make Dependabot PRs actually able to run integration tests, without exposing the full secret blast radius to arbitrary fork PRs:

  • Add pull_request_target trigger alongside the existing pull_request, gated by actor so it fires only for dependabot[bot]. Regular PRs continue to use pull_request exactly as before. No double-runs.
  • Set actions/checkout to ref: ${{ github.event.pull_request.head.sha }} so the PR's code is tested in both event contexts (default for pull_request_target is base).
  • Restrict the Dependabot matrix to the alpha environment only (skip cloud and staging) to minimize credential exposure for dependency-bump PRs.
  • Omit App Insights / telemetry secrets for Dependabot runs.
  • Add a Check secrets availability step that emits a ::warning:: annotation when any of CLIENT_ID/CLIENT_SECRET/BASE_URL is empty, so the previously-misleading downstream UIPATH_ACCESS_TOKEN is not set failure now has clear context (useful for non-Dependabot fork PRs).

Test plan

  • CI green on this PR (non-Dependabot path)
  • Verify emitted spans include ReferenceId matching the project/agent id
  • Confirm next Dependabot PR runs integration tests against alpha only and passes

@github-actions github-actions Bot added test:uipath-langchain Triggers tests in the uipath-langchain-python repository test:uipath-integrations labels May 13, 2026
@ionmincu ionmincu force-pushed the fix/traces-reference-id-from-agent-id branch 2 times, most recently from 9a9ca76 to 1a85a01 Compare May 13, 2026 16:57
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@ionmincu ionmincu force-pushed the fix/traces-reference-id-from-agent-id branch from 1a85a01 to f2731c7 Compare May 13, 2026 16:59
@sonarqubecloud
Copy link
Copy Markdown

@github-actions
Copy link
Copy Markdown

🚨 Heads up: uipath-langchain cross-tests are FAILING 🚨

Your changes may break the uipath-langchain-python integration.

⚠️ These checks are NOT enforced by branch protection rules. Please review the failures before merging.

🔍 Inspect the failed run →

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.

1 participant