Skip to content

Commit 6cdfa57

Browse files
pateljay43claude
andcommitted
fix(context-grounding): address PR review feedback
Remove duplicate decorators on ingest_by_name, use safe .get("value", []) in list_indexes/list_indexes_async, and drop redundant folder_key re-resolution in _list_spec. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent bbfb170 commit 6cdfa57

1 file changed

Lines changed: 4 additions & 7 deletions

File tree

packages/uipath-platform/src/uipath/platform/context_grounding/_context_grounding_service.py

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1723,7 +1723,8 @@ def list_indexes(
17231723
).json()
17241724

17251725
return [
1726-
ContextGroundingIndex.model_validate(item) for item in response["value"]
1726+
ContextGroundingIndex.model_validate(item)
1727+
for item in response.get("value", [])
17271728
]
17281729

17291730
@resource_override(resource_type="index")
@@ -1761,7 +1762,8 @@ async def list_indexes_async(
17611762
).json()
17621763

17631764
return [
1764-
ContextGroundingIndex.model_validate(item) for item in response["value"]
1765+
ContextGroundingIndex.model_validate(item)
1766+
for item in response.get("value", [])
17651767
]
17661768

17671769
@resource_override(resource_type="index")
@@ -1814,8 +1816,6 @@ async def delete_by_name_async(
18141816
index, folder_key=folder_key, folder_path=folder_path
18151817
)
18161818

1817-
@resource_override(resource_type="index")
1818-
@traced(name="contextgrounding_exists", run_type="uipath")
18191819
@resource_override(resource_type="index")
18201820
@traced(name="contextgrounding_ingest_by_name", run_type="uipath")
18211821
def ingest_by_name(
@@ -1903,10 +1903,7 @@ def _retrieve_across_folders_spec(
19031903
def _list_spec(
19041904
self,
19051905
folder_key: Optional[str] = None,
1906-
folder_path: Optional[str] = None,
19071906
) -> RequestSpec:
1908-
folder_key = self._resolve_folder_key(folder_key, folder_path)
1909-
19101907
return RequestSpec(
19111908
method="GET",
19121909
endpoint=Endpoint("/ecs_/v2/indexes"),

0 commit comments

Comments
 (0)