Skip to content

feat: map parent_operation_id from runtime_config#90

Open
GabrielVasilescu04 wants to merge 1 commit intomainfrom
feature/add-parent-operation-id-on-context
Open

feat: map parent_operation_id from runtime_config#90
GabrielVasilescu04 wants to merge 1 commit intomainfrom
feature/add-parent-operation-id-on-context

Conversation

@GabrielVasilescu04
Copy link
Contributor

@GabrielVasilescu04 GabrielVasilescu04 commented Feb 17, 2026

Description

Map parent_operation_id from runtime_config

Development Package

  • Add this package as a dependency in your pyproject.toml:
[project]
dependencies = [
  # Exact version:
  "uipath-runtime==0.8.7.dev1000900320",

  # Any version from PR
  "uipath-runtime>=0.8.7.dev1000900000,<0.8.7.dev1000910000"
]

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

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

@GabrielVasilescu04 GabrielVasilescu04 force-pushed the feature/add-parent-operation-id-on-context branch from 3977a0c to ad704d4 Compare February 17, 2026 14:13
Copy link

Copilot AI left a comment

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 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_id field to UiPathRuntimeContext class
  • Implemented mapping logic to extract ParentOperationId from runtime.internalArguments config 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
Copy link

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

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

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")

Suggested change
parent_operation_id: str | None = None
parent_operation_id: str | None = Field(
None, description="Parent operation identifier for CAS"
)

Copilot uses AI. Check for mistakes.
assert ctx.state_file == "my_state.db"
assert ctx.logs_file == "my_logs.log"

# parentOperationId is mapped correctly from internal_arguments
Copy link

Copilot AI Feb 17, 2026

Choose a reason for hiding this comment

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

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

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant