Skip to content

Fix/entity overwrite fixes#1548

Open
UIPath-Harshit wants to merge 4 commits intomainfrom
fix/entity-overwrite-fixes
Open

Fix/entity overwrite fixes#1548
UIPath-Harshit wants to merge 4 commits intomainfrom
fix/entity-overwrite-fixes

Conversation

@UIPath-Harshit
Copy link
Copy Markdown
Contributor

@UIPath-Harshit UIPath-Harshit commented Apr 8, 2026

  1. The major driver behind this changes is Studio and Solution bindings override works with names alone, while the DataFabric works completely with ids(entityId, folderId). As a result, the platform takes the heavy lifting to resolve these identifiers as part of the overwrites and langgraph layer will invoke a simple caller to resolve it and get the right identifiers.
  2. Introduced retrieve_by_entity_name as solution overwrites doesnt send those details.

@github-actions github-actions bot added test:uipath-langchain Triggers tests in the uipath-langchain-python repository test:uipath-llamaindex Triggers tests in the uipath-llamaindex-python repository labels Apr 8, 2026
@UIPath-Harshit UIPath-Harshit force-pushed the fix/entity-overwrite-fixes branch 2 times, most recently from 45a5739 to d30d69b Compare April 8, 2026 09:18
@UIPath-Harshit UIPath-Harshit marked this pull request as ready for review April 8, 2026 09:41
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d30d69be21

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Copy link
Copy Markdown

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 extends the SDK’s entity overwrite and resolution behavior to better support Studio/Solution overwrite semantics (name-based) while DataFabric APIs operate on IDs (entityId/folderId), by adding entity-specific overwrite parsing and introducing an entity-set resolution flow in the platform layer.

Changes:

  • Add EntityResourceOverwrite and integrate entity overwrites into query routing-context generation.
  • Introduce entity-set resolution (resolve_entity_set / async) that applies overwrites and returns a configured EntitiesService.
  • Move/align DataFabric entity-set item model (DataFabricEntityItem) and update tests + SDK reference accordingly.

Reviewed changes

Copilot reviewed 15 out of 17 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
packages/uipath/uv.lock Bumps uipath and uipath-platform versions in lockfile.
packages/uipath/pyproject.toml Bumps uipath package version.
packages/uipath/tests/resource_overrides/test_resource_overrides.py Adds coverage asserting entity overwrite parsing.
packages/uipath/tests/resource_overrides/overwrites.json Adds an entity overwrite example using folderId.
packages/uipath/tests/agent/models/test_agent.py Updates DataFabric entity-set field names (folder_key, entity_key).
packages/uipath/src/uipath/agent/models/agent.py Switches agent config to use platform DataFabricEntityItem instead of a local model.
packages/uipath/src/uipath/_resources/SDK_REFERENCE.md Updates entity APIs (query signature, adds resolve/retrieve-by-name) and adds new documented helpers.
packages/uipath-platform/uv.lock Bumps uipath-platform version in lockfile.
packages/uipath-platform/pyproject.toml Bumps uipath-platform package version.
packages/uipath-platform/tests/services/test_entities_service.py Adds/updates tests for routing-context generation, overwrites, and entity-set resolution.
packages/uipath-platform/src/uipath/platform/entities/entities.py Adds DataFabricEntityItem + EntitySetResolution models (with forward refs).
packages/uipath-platform/src/uipath/platform/entities/_entity_resolution.py New entity resolution + routing-strategy implementation (folders map, context overwrites, pre-resolved).
packages/uipath-platform/src/uipath/platform/entities/_entities_service.py Adds routing-strategy plumbing, retrieve-by-name APIs, resolve_entity_set APIs, and changes query routing handling.
packages/uipath-platform/src/uipath/platform/entities/init.py Exposes DataFabricEntityItem and EntitySetResolution in public exports.
packages/uipath-platform/src/uipath/platform/common/_bindings.py Adds EntityResourceOverwrite and extends overwrite parsing to include entity overwrites.
packages/uipath-platform/src/uipath/platform/common/init.py Exposes EntityResourceOverwrite in public exports.
packages/uipath-platform/src/uipath/platform/_uipath.py Wires UiPath.entities to pass folders_service into EntitiesService.

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

@UIPath-Harshit UIPath-Harshit force-pushed the fix/entity-overwrite-fixes branch from 0f7d305 to c181f23 Compare April 8, 2026 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:uipath-langchain Triggers tests in the uipath-langchain-python repository test:uipath-llamaindex Triggers tests in the uipath-llamaindex-python repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants