feat(eval): pass folderKey in simulate-component API payload - #1853
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the uipath package’s eval “simulate-component” mocking flow to include folderKey in the API payload, sourcing it from UiPathConfig.folder_key (which reads UIPATH_FOLDER_KEY), and bumps the uipath package version.
Changes:
- Add
"folderKey": UiPathConfig.folder_keyto the simulate-component request payload inSimulateComponentMocker. - Add tests covering the presence/absence of
folderKeybased onUIPATH_FOLDER_KEY. - Bump
uipathversion to2.13.23(and reflect it inuv.lock).
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| packages/uipath/src/uipath/eval/mocks/_simulate_component_mocker.py | Includes folderKey in the simulate-component payload (sourced from config). |
| packages/uipath/tests/cli/eval/mocks/test_simulate_component.py | Adds tests validating folderKey behavior with/without UIPATH_FOLDER_KEY. |
| packages/uipath/pyproject.toml | Bumps package version to 2.13.23. |
| packages/uipath/uv.lock | Updates locked uipath version to 2.13.23. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
30e4c86 to
4df198c
Compare
github-actions
Bot
added
test:uipath-langchain
- Add test for _get_span_context with valid OTel span (lines 150-152) - Add test verifying log output includes traceId, parentSpanId, folderKey - Achieves 100% coverage on _simulate_component_mocker.py Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Fix mypy error: Module "opentelemetry.trace" does not explicitly export attribute "context_api". Import context_api from opentelemetry.context directly. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
383296d to
d852962
Compare
🚨 Heads up:
|
|
Summary
folderKeyto the simulate-component API payload inSimulateComponentMocker, sourced fromUiPathConfig.folder_key(readsUIPATH_FOLDER_KEYenv var)UiPathConfigimport to the top of the fileuipathpackage version to2.13.23Test plan
folderKeyis included in payload whenUIPATH_FOLDER_KEYenv var is setfolderKeyisNonewhen env var is not set🤖 Generated with Claude Code