Skip to content

Commit 2728e52

Browse files
committed
chore(eval): drop redundant UIPATH_FILE_SOURCE_PROJECT_ID env var
UIPATH_PROJECT_ID already carries the file-source semantic (cloud project the worker fetches files from). UIPATH_FILE_SOURCE_PROJECT_ID was an alias that added a third name for the same value without adding meaning. UiPathConfig.project_id now reads UIPATH_PROJECT_ID directly — no fallback chain. UiPathConfig.agent_id still falls back to project_id when UIPATH_AGENT_ID is unset (cloud-project case where the two are equal). Updated config tests; existing reporter/CLI telemetry behaviour unchanged. Refs: AE-1396
1 parent 81bf325 commit 2728e52

3 files changed

Lines changed: 7 additions & 19 deletions

File tree

packages/uipath-platform/src/uipath/platform/common/_config.py

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -57,14 +57,9 @@ def config_file_name(self) -> str:
5757

5858
@property
5959
def project_id(self) -> str | None:
60-
from uipath.platform.common.constants import (
61-
ENV_UIPATH_FILE_SOURCE_PROJECT_ID,
62-
ENV_UIPATH_PROJECT_ID,
63-
)
60+
from uipath.platform.common.constants import ENV_UIPATH_PROJECT_ID
6461

65-
return os.getenv(ENV_UIPATH_FILE_SOURCE_PROJECT_ID) or os.getenv(
66-
ENV_UIPATH_PROJECT_ID, None
67-
)
62+
return os.getenv(ENV_UIPATH_PROJECT_ID, None)
6863

6964
@property
7065
def agent_id(self) -> str | None:

packages/uipath-platform/src/uipath/platform/common/constants.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@
1717
ENV_TELEMETRY_ENABLED = "UIPATH_TELEMETRY_ENABLED"
1818
ENV_TRACING_ENABLED = "UIPATH_TRACING_ENABLED"
1919
ENV_UIPATH_PROJECT_ID = "UIPATH_PROJECT_ID"
20-
ENV_UIPATH_FILE_SOURCE_PROJECT_ID = "UIPATH_FILE_SOURCE_PROJECT_ID"
2120
ENV_UIPATH_AGENT_ID = "UIPATH_AGENT_ID"
2221
ENV_UIPATH_CLOUD_USER_ID = "UIPATH_CLOUD_USER_ID"
2322
ENV_UIPATH_PROJECT_FILES_SOURCE = "UIPATH_PROJECT_FILES_SOURCE"

packages/uipath-platform/tests/common/test_config_env_vars.py

Lines changed: 5 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -7,31 +7,25 @@
77
def _clear_env(monkeypatch):
88
for var in (
99
"UIPATH_PROJECT_ID",
10-
"UIPATH_FILE_SOURCE_PROJECT_ID",
1110
"UIPATH_AGENT_ID",
1211
"UIPATH_CLOUD_USER_ID",
1312
"UIPATH_PROJECT_FILES_SOURCE",
1413
):
1514
monkeypatch.delenv(var, raising=False)
1615

1716

18-
class TestProjectIdFallback:
19-
def test_returns_file_source_when_set(self, monkeypatch):
20-
monkeypatch.setenv("UIPATH_FILE_SOURCE_PROJECT_ID", "file-source-id")
21-
monkeypatch.setenv("UIPATH_PROJECT_ID", "legacy-id")
17+
class TestProjectId:
18+
def test_reads_env_var(self, monkeypatch):
19+
monkeypatch.setenv("UIPATH_PROJECT_ID", "file-source-id")
2220
assert UiPathConfig.project_id == "file-source-id"
2321

24-
def test_falls_back_to_legacy_when_file_source_unset(self, monkeypatch):
25-
monkeypatch.setenv("UIPATH_PROJECT_ID", "legacy-id")
26-
assert UiPathConfig.project_id == "legacy-id"
27-
28-
def test_returns_none_when_neither_set(self):
22+
def test_returns_none_when_unset(self):
2923
assert UiPathConfig.project_id is None
3024

3125

3226
class TestAgentId:
3327
def test_returns_explicit_agent_id_when_set(self, monkeypatch):
34-
monkeypatch.setenv("UIPATH_FILE_SOURCE_PROJECT_ID", "debug-project-guid")
28+
monkeypatch.setenv("UIPATH_PROJECT_ID", "debug-project-guid")
3529
monkeypatch.setenv("UIPATH_AGENT_ID", "real-agent-id")
3630
assert UiPathConfig.agent_id == "real-agent-id"
3731

0 commit comments

Comments
 (0)