The changes introduce a new metadata_filter parameter to the list_docs_snippets_by_owner_query function, allowing for metadata-based filtering of documents.

Sweep Found These Issues

• The use of a mutable default argument metadata_filter: dict[str, Any] = {} can lead to unexpected behavior if the dictionary is modified.

julep/agents-api/agents_api/models/docs/list_docs.py

Line 39 in 8a1053a

metadata_filter: dict[str, Any] = {},


View Diff
• The constructed metadata_filter_str does not handle cases where the metadata_filter dictionary is empty, potentially leading to syntax errors in the query.

julep/agents-api/agents_api/models/docs/list_docs.py

Lines 43 to 48 in 8a1053a

	metadata_filter_str = ", ".join(
	[
	f"metadata->{json.dumps(k)} == {json.dumps(v)}"
	for k, v in metadata_filter.items()
	]
	)


View Diff

The changes allow the metadata_filter parameter to be used for filtering documents and remove the check for the agent's existence before querying documents.

Sweep Found These Issues

• The removal of the check to ensure the agent exists before querying for documents could lead to querying with an invalid agent_id, potentially causing unexpected errors or behavior.

julep/agents-api/agents_api/routers/agents/routers.py

Line 9 in 8a1053a

import pandas as pd


View Diff

The change removes the HTTP 501 Not Implemented error for metadata_filter and updates the function call to support metadata filtering.

Sweep Found These Issues

• The removal of the HTTP 501 Not Implemented error handling for metadata_filter without ensuring that list_docs_snippets_by_owner_query can handle the filter correctly may lead to unexpected behavior or errors if the filter is not properly implemented.

julep/agents-api/agents_api/routers/users/routers.py

Lines 288 to 297 in 8a1053a

	detail="metadata_filter is not a valid JSON",
	)
	if not len(list(ensure_owner_exists_query("user", user_id).iterrows())):
	raise UserNotFoundError("", user_id)
	resp = list_docs_snippets_by_owner_query(
	owner_type="user",
	owner_id=user_id,
	metadata_filter=metadata_filter,


View Diff