Skip to content

fix: remove auto search type #1533

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
CalebMartinUiPath wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix: remove auto search type #1533

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion packages/uipath-platform/pyproject.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
[project]
name = "uipath-platform"
version = "0.1.16"
version = "0.1.17"
description = "HTTP client library for programmatic access to UiPath Platform"
readme = { file = "README.md", content-type = "text/markdown" }
requires-python = ">=3.11"
Expand Down
10 changes: 5 additions & 5 deletions packages/uipath-platform/src/uipath/platform/context_grounding/_context_grounding_service.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1456,7 +1456,7 @@ def unified_search(
self,
name: str,
query: str,
search_mode: SearchMode = SearchMode.AUTO,
search_mode: SearchMode = SearchMode.SEMANTIC,
number_of_results: int = 10,
threshold: float = 0.0,
scope: Optional[UnifiedSearchScope] = None,
Expand All @@ -1471,7 +1471,7 @@ def unified_search(
Args:
name (str): The name of the context index to search in.
query (str): The search query in natural language.
search_mode (SearchMode): The search mode to use. Defaults to AUTO.
search_mode (SearchMode): The search mode to use. Defaults to SEMANTIC.
number_of_results (int): Maximum number of results to return. Defaults to 10.
threshold (float): Minimum similarity threshold. Defaults to 0.0.
scope (Optional[UnifiedSearchScope]): Optional search scope (folder, extension).
Expand Down Expand Up @@ -1511,7 +1511,7 @@ async def unified_search_async(
self,
name: str,
query: str,
search_mode: SearchMode = SearchMode.AUTO,
search_mode: SearchMode = SearchMode.SEMANTIC,
number_of_results: int = 10,
threshold: float = 0.0,
scope: Optional[UnifiedSearchScope] = None,
Expand All @@ -1526,7 +1526,7 @@ async def unified_search_async(
Args:
name (str): The name of the context index to search in.
query (str): The search query in natural language.
search_mode (SearchMode): The search mode to use. Defaults to AUTO.
search_mode (SearchMode): The search mode to use. Defaults to SEMANTIC.
number_of_results (int): Maximum number of results to return. Defaults to 10.
threshold (float): Minimum similarity threshold. Defaults to 0.0.
scope (Optional[UnifiedSearchScope]): Optional search scope (folder, extension).
Expand Down Expand Up @@ -2162,7 +2162,7 @@ def _unified_search_spec(
self,
index_id: str,
query: str,
search_mode: SearchMode = SearchMode.AUTO,
search_mode: SearchMode = SearchMode.SEMANTIC,
number_of_results: int = 10,
threshold: float = 0.0,
scope: Optional[UnifiedSearchScope] = None,
Expand Down
1 change: 0 additions & 1 deletion packages/uipath-platform/src/uipath/platform/context_grounding/context_grounding.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -239,7 +239,6 @@ class ContextGroundingSearchResultItem(BaseModel):
class SearchMode(str, Enum):
"""Enum representing possible unified search modes."""

AUTO = "Auto"
Copy link
Copy Markdown
Contributor

@cfauchere cfauchere Mar 31, 2026

Choose a reason for hiding this comment

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

that's a breaking change

Copy link
Copy Markdown
Contributor

@cfauchere cfauchere Mar 31, 2026

Choose a reason for hiding this comment

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

if the only value is SEMANTIC then why do we need the enum at all?

Copy link
Copy Markdown
Contributor Author

@CalebMartinUiPath CalebMartinUiPath Mar 31, 2026

Choose a reason for hiding this comment

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

preview feature should have never made it into the sdk. It will break once we deprecate structured query anyway

SEMANTIC = "Semantic"


Expand Down
2 changes: 1 addition & 1 deletion packages/uipath-platform/tests/services/test_context_grounding_service.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3111,7 +3111,7 @@ async def test_unified_search_async(
response = await service.unified_search_async(
name="test-index",
query="test query",
search_mode=SearchMode.AUTO,
search_mode=SearchMode.SEMANTIC,
)

assert isinstance(response, UnifiedQueryResult)
Expand Down
2 changes: 1 addition & 1 deletion packages/uipath-platform/uv.lock
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading