feat: map parent_operation_id from runtime_config#90
feat: map parent_operation_id from runtime_config#90GabrielVasilescu04 wants to merge 1 commit intomainfrom
Conversation
3977a0c to
ad704d4
Compare
There was a problem hiding this comment.
Pull request overview
This PR adds support for mapping the parent_operation_id field from the runtime configuration's internalArguments.ParentOperationId property. This allows the runtime context to capture the parent operation identifier when provided through the configuration file. The version is bumped from 0.8.6 to 0.8.7.
Changes:
- Added
parent_operation_idfield toUiPathRuntimeContextclass - Implemented mapping logic to extract
ParentOperationIdfromruntime.internalArgumentsconfig section with proper validation - Added test coverage to verify the mapping works correctly
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/uipath/runtime/context.py | Added parent_operation_id field to context class and implemented extraction logic from internalArguments config |
| tests/test_context.py | Added test data with ParentOperationId in internalArguments and assertion to verify correct mapping |
| pyproject.toml | Version bump from 0.8.6 to 0.8.7 |
| uv.lock | Updated version reference from 0.8.6 to 0.8.7 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| org_id: str | None = None | ||
| folder_key: str | None = None | ||
| process_key: str | None = None | ||
| parent_operation_id: str | None = None |
There was a problem hiding this comment.
The new field parent_operation_id lacks a description. Other similar fields like conversation_id, exchange_id, and message_id have Field descriptors with descriptions. Consider adding a Field descriptor with a description to maintain consistency with the codebase conventions. For example: parent_operation_id: str | None = Field(None, description="Parent operation identifier")
| parent_operation_id: str | None = None | |
| parent_operation_id: str | None = Field( | |
| None, description="Parent operation identifier for CAS" | |
| ) |
| assert ctx.state_file == "my_state.db" | ||
| assert ctx.logs_file == "my_logs.log" | ||
|
|
||
| # parentOperationId is mapped correctly from internal_arguments |
There was a problem hiding this comment.
Missing space after '#' in the comment. Python style conventions (PEP 8) recommend adding a space after the comment marker for better readability. The comment should read: # parentOperationId is mapped correctly from internal_arguments
Description
Map
parent_operation_idfromruntime_configDevelopment Package