Skip to content

fix(platform): expose SecretValue on assets#1719

Merged
GabrielVasilescu04 merged 1 commit into
mainfrom
fix/expose-secret-asset-value
Jun 16, 2026
Merged

fix(platform): expose SecretValue on assets#1719
GabrielVasilescu04 merged 1 commit into
mainfrom
fix/expose-secret-asset-value

Conversation

@GabrielVasilescu04

@GabrielVasilescu04 GabrielVasilescu04 commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Description

Map SecretValue to the sdk asset models

Development Packages

uipath

[project]
dependencies = [
  # Exact version (copy-paste ready):
  "uipath==2.10.83.dev1017196750",

  # Any version from this PR (uncomment to use a range instead):
  # "uipath>=2.10.83.dev1017190000,<2.10.83.dev1017200000",
]

[[tool.uv.index]]
name = "testpypi"
url = "https://test.pypi.org/simple/"
publish-url = "https://test.pypi.org/legacy/"
explicit = true

[tool.uv.sources]
uipath = { index = "testpypi" }
uipath-platform = { index = "testpypi" }

[tool.uv]
override-dependencies = ["uipath-platform==0.1.65.dev1017196750"]

uipath-platform

[project]
dependencies = [
  # Exact version (copy-paste ready):
  "uipath-platform==0.1.65.dev1017196750",

  # Any version from this PR (uncomment to use a range instead):
  # "uipath-platform>=0.1.65.dev1017190000,<0.1.65.dev1017200000",
]

[[tool.uv.index]]
name = "testpypi"
url = "https://test.pypi.org/simple/"
publish-url = "https://test.pypi.org/legacy/"
explicit = true

[tool.uv.sources]
uipath-platform = { index = "testpypi" }

@GabrielVasilescu04 GabrielVasilescu04 self-assigned this Jun 15, 2026
Copilot AI review requested due to automatic review settings June 15, 2026 14:50
@GabrielVasilescu04 GabrielVasilescu04 added the build:dev Create a dev build from the pr label Jun 15, 2026
@github-actions github-actions Bot added test:uipath-langchain Triggers tests in the uipath-langchain-python repository test:uipath-integrations labels Jun 15, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the uipath-platform SDK asset models and AssetsService credential retrieval helpers to support Secret-type assets by exposing/mapping the SecretValue field end-to-end.

Changes:

  • Add secret_value (SecretValue) to UserAsset and Asset models.
  • Update retrieve_credential / retrieve_credential_async to return secret_value when ValueType == "Secret".
  • Add tests covering Secret-type asset retrieval and credential retrieval behavior; bump package version to 0.1.65.

Reviewed changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
packages/uipath/uv.lock Bumps locked editable uipath-platform version to 0.1.65.
packages/uipath-platform/uv.lock Bumps locked package version to 0.1.65.
packages/uipath-platform/pyproject.toml Bumps uipath-platform project version to 0.1.65.
packages/uipath-platform/src/uipath/platform/orchestrator/assets.py Adds secret_value field mapping (SecretValue) to asset models.
packages/uipath-platform/src/uipath/platform/orchestrator/_assets_service.py Returns secret_value for Secret assets in credential retrieval methods; updates docstrings.
packages/uipath-platform/tests/services/test_assets_service.py Adds test coverage for Secret assets and SecretValue exposure.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Optional[str]: The decrypted value — `credential_password` for Credential assets, `secret_value` for Secret assets.

Raises:
ValueError: If the method is called for a user asset.
Optional[str]: The decrypted value — `credential_password` for Credential assets, `secret_value` for Secret assets.

Raises:
ValueError: If the method is called for a user asset.
@GabrielVasilescu04 GabrielVasilescu04 force-pushed the fix/expose-secret-asset-value branch 5 times, most recently from bf00e9a to d08e1bd Compare June 16, 2026 08:43
@GabrielVasilescu04 GabrielVasilescu04 force-pushed the fix/expose-secret-asset-value branch from d08e1bd to 3fabc3b Compare June 16, 2026 09:14
@sonarqubecloud

Copy link
Copy Markdown

@GabrielVasilescu04 GabrielVasilescu04 merged commit 99b686b into main Jun 16, 2026
164 of 165 checks passed
@GabrielVasilescu04 GabrielVasilescu04 deleted the fix/expose-secret-asset-value branch June 16, 2026 09:49
@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

build:dev Create a dev build from the pr 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.

3 participants