Skip to content

feat: add support for system indexes [ECS-1791]#1605

Merged
gcuip merged 6 commits intomainfrom
feature/ECS-1791-system-indexes-support
Apr 30, 2026
Merged

feat: add support for system indexes [ECS-1791]#1605
gcuip merged 6 commits intomainfrom
feature/ECS-1791-system-indexes-support

Conversation

@gcuip
Copy link
Copy Markdown
Contributor

@gcuip gcuip commented Apr 29, 2026

No description provided.

@gcuip gcuip requested a review from radu-mocanu April 29, 2026 20:27
@github-actions github-actions Bot added test:uipath-langchain Triggers tests in the uipath-langchain-python repository test:uipath-integrations labels Apr 29, 2026
@gcuip gcuip force-pushed the feature/ECS-1791-system-indexes-support branch from 6f9ef08 to f0eaefe Compare April 29, 2026 20:38
@gcuip gcuip requested a review from cotovanu-cristian April 30, 2026 08:49
]

@traced(name="contextgrounding_retrieve_system_indexes", run_type="uipath")
def retrieve_system_indexes(
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

would keep this as a private untraced method.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

it will be useful for customers to see why they can run debug projects and where the index came from, but when running a production / deployed project, they'll get a different index (or an error if they have manually deleted it).

)

@staticmethod
def _odata_name_filter(name: str) -> str:
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

should we maybe make this generic and move it one level up (base class) so any downstream service can benefit from the encoding?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

we can extract it later if it's needed by other services.

@github-actions
Copy link
Copy Markdown

🚨 Heads up: uipath-integrations cross-tests are FAILING 🚨

Your changes may break one or more integrations in uipath-integrations-python:

  • uipath-openai-agents
  • uipath-google-adk
  • uipath-agent-framework
  • uipath-llamaindex
  • uipath-pydantic-ai

⚠️ These checks are NOT enforced by branch protection rules. Please review the failures before merging.

🔍 Inspect the failed run →

@github-actions
Copy link
Copy Markdown

🚨 Heads up: uipath-langchain cross-tests are FAILING 🚨

Your changes may break the uipath-langchain-python integration.

⚠️ These checks are NOT enforced by branch protection rules. Please review the failures before merging.

🔍 Inspect the failed run →

@gcuip gcuip merged commit ee98f4d into main Apr 30, 2026
152 of 154 checks passed
@gcuip gcuip deleted the feature/ECS-1791-system-indexes-support branch April 30, 2026 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants