coderabbitai[bot]
commented
1 month ago Walkthrough
This pull request introduces significant enhancements across multiple components of the Kairon framework. Key changes include the modification of the remove_bot_access function to ensure a single active BotAccess record is retrieved, impacting error handling and control flow. Additionally, new endpoints for content importer logs and document uploads are added, along with a comprehensive suite of tests to validate these functionalities. The changes also expand the event handling capabilities and introduce a new class for managing content import events.
Changes
| File Path | Change Summary |
|---|---|
kairon/shared/account/processor.py |
Modified the remove_bot_access function to retrieve a single active BotAccess record instead of a collection, changing error handling and control flow. |
tests/integration_test/services_test.py |
Restructured integration tests related to bot membership; removed old tests and added new ones with updated user accounts and bot identifiers for clarity. |
kairon/api/app/routers/bot/bot.py |
Added a new GET endpoint /content/logs for retrieving content importer logs, with user authentication and pagination support. |
kairon/api/app/routers/bot/data.py |
Introduced endpoints for document content uploads and downloading error reports, enhancing user interaction with the API. |
kairon/cli/content_importer.py |
Implemented CLI functionality for importing document content, including command handling and argument parsing. |
kairon/events/definitions/content_importer.py |
Added DocContentImporterEvent class for managing document content import events, including validation and lifecycle management. |
kairon/events/definitions/factory.py |
Integrated DocContentImporterEvent into the EventFactory for handling content import events. |
kairon/importer/content_importer.py |
Introduced ContentImporter class for validating and importing document content, including methods for validation and data importation. |
kairon/shared/cognition/processor.py |
Added delete_all_cognition_data_by_collection method for bulk deletion of documents in a specified collection for a bot. |
kairon/shared/constants.py |
Added content_importer event type to the EventClass enumeration for enhanced event handling capabilities. |
kairon/shared/content_importer/content_processor.py |
Introduced ContentImporterLogProcessor class for managing logs related to content import events, including methods for logging and checking event statuses. |
kairon/shared/content_importer/data_objects.py |
Added ContentValidationLogs class for logging content validation events with a structured document model. |
kairon/shared/data/data_objects.py |
Added content_importer_limit_per_day field to BotSettings for managing daily content import limits. |
kairon/shared/data/processor.py |
Enhanced methods for document validation and logging, including validate_schema_and_log and save_and_validate. |
requirements/prod.txt |
Added google-generativeai package as a new dependency. |
system.yaml |
Introduced configuration for content_importer, including daily limits and event definitions. |
tests/unit_test/action/action_test.py |
Updated tests to include content_importer_limit_per_day configuration parameter. |
tests/unit_test/data_processor/content_importer_log_processor_test.py |
Added unit tests for the ContentImporterLogProcessor class, covering various logging functionalities. |
tests/unit_test/data_processor/data_processor_test.py |
Introduced tests for validating and processing document uploads, including various error handling scenarios. |
tests/unit_test/validator/content_importer_test.py |
Created tests for validating and importing content using the ContentImporter class, covering success and error cases. |
Possibly related PRs
- #1196: Involves uploading and downloading bot content, which relates to the main PR's modifications in handling bot access and membership functionalities.
- #1381: Introduces a new class for handling remote actions, which may connect to the main PR's changes in bot access management and user interactions.
- #1392: The addition of the
LLMSecretdata object and API for listing LLM models could relate to the main PR's focus on managing bot access and user roles, enhancing the overall functionality. - #1411: The introduction of scheduling actions aligns with the main PR's changes in managing bot functionalities, potentially impacting how users interact with bots.
- #1463: The handling of access when a collaborator leaves a bot directly relates to the main PR's focus on bot access management and user roles.
- #1519: The upload document content functionality enhances the bot's capabilities, which is relevant to the main PR's changes in managing bot access and user interactions.
🐰 In the meadow of code, where changes abound,
New paths are forged, and solutions are found.
With logs and imports, we hop and we play,
Celebrating our progress, come join the ballet!
So gather 'round, friends, let’s cheer with delight,
For Kairon’s bright future is now shining bright! 🥕✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
fix audit log issues when user leaves bot, deletes a member and when a bot is deleted
Summary by CodeRabbit
New Features
DocContentImporterEventclass for handling content import events.Bug Fixes
Tests
ContentImporterLogProcessorandContentImporterclasses to ensure robustness.