admin-api-lib 3.3.0__tar.gz → 3.4.0__tar.gz

{admin_api_lib-3.3.0 → admin_api_lib-3.4.0}/PKG-INFO

@@ -1,6 +1,6 @@
-Metadata-Version: 2.3
+Metadata-Version: 2.4
 Name: admin-api-lib
-Version: 3.3.0
+Version: 3.4.0
 Summary: The admin backend is responsible for the document management. This includes deletion, upload and returning the source document.
 License: Apache-2.0
 Author: STACKIT GmbH & Co. KG
@@ -11,6 +11,7 @@ Requires-Python: >=3.13,<4.0
 Classifier: License :: OSI Approved :: Apache Software License
 Classifier: Programming Language :: Python :: 3
 Classifier: Programming Language :: Python :: 3.13
+Classifier: Programming Language :: Python :: 3.14
 Requires-Dist: boto3 (>=1.38.10,<2.0.0)
 Requires-Dist: dependency-injector (>=4.46.0,<5.0.0)
 Requires-Dist: fastapi (>=0.118.0,<0.119.0)
@@ -20,7 +21,7 @@ Requires-Dist: nltk (>=3.9.2,<4.0.0)
 Requires-Dist: python-dateutil (>=2.9.0.post0,<3.0.0)
 Requires-Dist: python-multipart (>=0.0.20,<0.0.21)
 Requires-Dist: pyyaml (>=6.0.2,<7.0.0)
-Requires-Dist: rag-core-lib (>=3.3.0,<4.0.0)
+Requires-Dist: rag-core-lib (==3.4.0)
 Requires-Dist: redis (>=6.0.0,<7.0.0)
 Requires-Dist: starlette (>=0.47.2,<0.49.0)
 Requires-Dist: tenacity (==9.1.2)

{admin_api_lib-3.3.0 → admin_api_lib-3.4.0}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "poetry.core.masonry.api"
 
 [tool.poetry]
 name = "admin-api-lib"
-version = "v3.3.0"
+version = "v3.4.0"
 description = "The admin backend is responsible for the document management. This includes deletion, upload and returning the source document."
 authors = [
     "STACKIT GmbH & Co. KG <data-ai@stackit.cloud>",
@@ -99,12 +99,12 @@ flake8-wot = "^0.2.0"
 flake8-function-order = "^0.0.5"
 flake8-tidy-imports = "^4.10.0"
 black = "^25.1.0"
-# flake8-logging-format = "^2024.24.12"
-# flake8-docstrings = "^1.7.0"
+flake8-logging-format = "^2024.24.12"
+flake8-docstrings = "^1.7.0"
 pytest-asyncio = "^1.0.0"
 
 [tool.poetry.dependencies]
-rag-core-lib = "^3.3.0"
+rag-core-lib = "3.4.0"
 python = "^3.13"
 uvicorn = "^0.37.0"
 fastapi = "^0.118.0"
@@ -127,7 +127,6 @@ log_cli_level = "DEBUG"
 pythonpath = "src"
 testpaths = "src/tests"
 
-
 [[tool.poetry.source]]
 name = "testpypi"
 url = "https://test.pypi.org/simple/"

{admin_api_lib-3.3.0 → admin_api_lib-3.4.0}/src/admin_api_lib/api_endpoints/file_uploader.py

@@ -8,6 +8,7 @@ from admin_api_lib.api_endpoints.uploader_base import UploaderBase
 
 
 class FileUploader(UploaderBase):
+    """File uploader endpoint of the admin API."""
 
     @abstractmethod
     async def upload_file(
@@ -16,7 +17,7 @@ class FileUploader(UploaderBase):
         file: UploadFile,
     ) -> None:
         """
-        Uploads a source file for content extraction.
+        Upload a source file for content extraction.
 
         Parameters
         ----------

{admin_api_lib-3.3.0 → admin_api_lib-3.4.0}/src/admin_api_lib/api_endpoints/source_uploader.py

@@ -21,7 +21,7 @@ class SourceUploader(UploaderBase):
         timeout: Optional[float],
     ) -> None:
         """
-        Uploads the parameters for source content extraction.
+        Upload the parameters for source content extraction.
 
         Parameters
         ----------

{admin_api_lib-3.3.0 → admin_api_lib-3.4.0}/src/admin_api_lib/api_endpoints/uploader_base.py

@@ -7,9 +7,7 @@ class UploaderBase:
     """Base class for uploader API endpoints."""
 
     def __init__(self):
-        """
-        Initialize the UploaderBase.
-        """
+        """Initialize the UploaderBase."""
         self._background_threads = []
 
     def _prune_background_threads(self) -> list[Thread]:

{admin_api_lib-3.3.0 → admin_api_lib-3.4.0}/src/admin_api_lib/apis/admin_api.py

@@ -149,7 +149,7 @@ async def upload_file(
     request: Request,
 ) -> None:
     """
-    Uploads user selected sources.
+    Upload user selected sources.
 
     Parameters
     ----------
@@ -181,7 +181,7 @@ async def upload_source(
     key_value_pair: List[KeyValuePair] = Body(None, description="The key-value pairs for the source"),
 ) -> None:
     """
-    Uploads user selected sources.
+    Upload user selected sources.
 
     Parameters
     ----------

{admin_api_lib-3.3.0 → admin_api_lib-3.4.0}/src/admin_api_lib/impl/api_endpoints/default_document_reference_retriever.py

@@ -2,7 +2,6 @@
 
 import io
 import logging
-import traceback
 
 from fastapi import HTTPException, Response, status
 
@@ -54,10 +53,9 @@ class DefaultDocumentReferenceRetriever(DocumentReferenceRetriever):
                 self._file_service.download_file(identification, document_buffer)
                 logger.debug("DONE retrieving document with id: %s", identification)
                 document_data = document_buffer.getvalue()
-            except Exception as e:
-                logger.error(
-                    "Error retrieving document with id: %s. Error: %s %s", identification, e, traceback.format_exc()
-                )
+            except Exception:
+                # Log full stack trace without embedding the exception object in the message (G200)
+                logger.exception("Error retrieving document with id: %s.", identification)
                 raise ValueError(f"Document with id '{identification}' not found.")
             finally:
                 document_buffer.close()

{admin_api_lib-3.3.0 → admin_api_lib-3.4.0}/src/admin_api_lib/impl/api_endpoints/default_file_uploader.py

@@ -1,6 +1,7 @@
+"""Module for the default file uploader implementation."""
+
 import logging
 from pathlib import Path
-import traceback
 import urllib
 import tempfile
 import asyncio
@@ -78,7 +79,7 @@ class DefaultFileUploader(FileUploader):
         file: UploadFile,
     ) -> None:
         """
-        Uploads a source file for content extraction.
+        Upload a source file for content extraction.
 
         Parameters
         ----------
@@ -109,7 +110,7 @@ class DefaultFileUploader(FileUploader):
             raise HTTPException(status_code=status.HTTP_400_BAD_REQUEST, detail=str(e))
         except Exception as e:
             self._key_value_store.upsert(source_name, Status.ERROR)
-            logger.error("Error while uploading %s = %s", source_name, str(e))
+            logger.exception("Error while uploading %s", source_name)
             raise HTTPException(status_code=status.HTTP_500_INTERNAL_SERVER_ERROR, detail=str(e))
 
     def _log_task_exception(self, task: asyncio.Task) -> None:
@@ -124,19 +125,16 @@ class DefaultFileUploader(FileUploader):
         if task.done() and not task.cancelled():
             try:
                 task.result()  # This will raise the exception if one occurred
-            except Exception as e:
-                logger.error("Background task failed with exception: %s", str(e))
-                logger.debug("Background task exception traceback: %s", traceback.format_exc())
+            except Exception:
+                logger.exception("Background task failed with exception.")
 
     def _prune_background_tasks(self) -> None:
-        """
-        Remove completed background tasks from the list.
-        """
+        """Remove completed background tasks from the list."""
         self._background_tasks = [task for task in self._background_tasks if not task.done()]
 
     def _check_if_already_in_processing(self, source_name: str) -> None:
         """
-        Checks if the source is already in processing state.
+        Check if the source is already in processing state.
 
         Parameters
         ----------
@@ -196,9 +194,9 @@ class DefaultFileUploader(FileUploader):
             await asyncio.to_thread(self._rag_api.upload_information_piece, rag_information_pieces)
             self._key_value_store.upsert(source_name, Status.READY)
             logger.info("Source uploaded successfully: %s", source_name)
-        except Exception as e:
+        except Exception:
             self._key_value_store.upsert(source_name, Status.ERROR)
-            logger.error("Error while uploading %s = %s", source_name, str(e))
+            logger.exception("Error while uploading %s", source_name)
 
     def _add_file_url(self, file_name: str, base_url: str, chunked_documents: list[Document]):
         document_url = f"{base_url.rstrip('/')}/document_reference/{urllib.parse.quote_plus(file_name)}"
@@ -229,6 +227,6 @@ class DefaultFileUploader(FileUploader):
 
                 self._file_service.upload_file(Path(temp_file_path), filename)
                 return filename
-        except Exception as e:
-            logger.error("Error during document saving: %s %s", e, traceback.format_exc())
+        except Exception:
+            logger.exception("Error during document saving")
             self._key_value_store.upsert(source_name, Status.ERROR)

{admin_api_lib-3.3.0 → admin_api_lib-3.4.0}/src/admin_api_lib/impl/api_endpoints/default_source_uploader.py

@@ -1,3 +1,5 @@
+"""Module for the default source uploader implementation."""
+
 import logging
 import asyncio
 from threading import Thread
@@ -28,6 +30,7 @@ logger = logging.getLogger(__name__)
 
 
 class DefaultSourceUploader(SourceUploader):
+    """The DefaultSourceUploader is responsible for uploading source files for content extraction."""
 
     def __init__(
         self,
@@ -78,7 +81,7 @@ class DefaultSourceUploader(SourceUploader):
         kwargs: list[KeyValuePair],
     ) -> None:
         """
-        Uploads the parameters for source content extraction.
+        Upload the parameters for source content extraction.
 
         Parameters
         ----------
@@ -95,7 +98,6 @@ class DefaultSourceUploader(SourceUploader):
         -------
         None
         """
-
         self._prune_background_threads()
 
         source_name = f"{source_type}:{sanitize_document_name(name)}"
@@ -111,12 +113,12 @@ class DefaultSourceUploader(SourceUploader):
             raise HTTPException(status_code=status.HTTP_400_BAD_REQUEST, detail=str(e))
         except Exception as e:
             self._key_value_store.upsert(source_name, Status.ERROR)
-            logger.error("Error while uploading %s = %s", source_name, str(e))
+            logger.exception("Error while uploading %s", source_name)
             raise HTTPException(status_code=status.HTTP_500_INTERNAL_SERVER_ERROR, detail=str(e))
 
     def _check_if_already_in_processing(self, source_name: str) -> None:
         """
-        Checks if the source is already in processing state.
+        Check if the source is already in processing state.
 
         Parameters
         ----------
@@ -150,7 +152,7 @@ class DefaultSourceUploader(SourceUploader):
             logger.error("Upload of %s timed out after %s seconds", source_name, timeout)
             self._key_value_store.upsert(source_name, Status.ERROR)
         except Exception:
-            logger.error("Error while uploading %s", source_name)
+            logger.exception("Error while uploading %s", source_name)
             self._key_value_store.upsert(source_name, Status.ERROR)
         finally:
             loop.close()
@@ -197,6 +199,6 @@ class DefaultSourceUploader(SourceUploader):
             await asyncio.to_thread(self._rag_api.upload_information_piece, rag_information_pieces)
             self._key_value_store.upsert(source_name, Status.READY)
             logger.info("Source uploaded successfully: %s", source_name)
-        except Exception as e:
+        except Exception:
             self._key_value_store.upsert(source_name, Status.ERROR)
-            logger.error("Error while uploading %s = %s", source_name, str(e))
+            logger.exception("Error while uploading %s", source_name)