-
-
Notifications
You must be signed in to change notification settings - Fork 3
v1.0.0 I miss you. #11
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.
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
Conversation
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR introduces a range of changes including migration to a new database ORM framework, updates to built-in plugins and templates, and removal of legacy migration scripts. Key changes include:
- Migrating database models and CRUD operations to use nonebot-plugin-orm.
- Adding new built-in templates (Muika.jinja2) and plugin updates (e.g., plugin_store, get_username).
- Removing legacy migration code and updating configuration handling for model config management.
Reviewed Changes
Copilot reviewed 33 out of 33 changed files in this pull request and generated 1 comment.
Show a summary per file
File | Description |
---|---|
muicebot/llm/providers/gemini.py | Removed ensure_mimetype() call, potentially affecting MIME setup. |
muicebot/llm/providers/dashscope.py | Removed local file path resolution logic for audio and image types. |
muicebot/database/orm_models.py, muicebot/database/crud.py | Added new database models and CRUD operations using nonebot-plugin-orm. |
muicebot/config.py | Refactored get_model_config_manager for lazy initialization. |
muicebot/builtin_templates/Muika.jinja2 | Added AI personality template with detailed role descriptors. |
muicebot/builtin_plugins/plugin_store, get_username, get_current_time, access_control.py | Updated plugin metadata naming conventions and error messages. |
.github/workflows/pre-commit.yml | Updated workflow to use Python 3.12 directly. |
migrations.py | Removed legacy migration script. |
Comments suppressed due to low confidence (2)
muicebot/llm/providers/gemini.py:86
- The removal of ensure_mimetype() may cause missing MIME type information if resource.mimetype is not otherwise set. Please verify that this change is intentional and that MIME types are still correctly populated.
resource.ensure_mimetype()
muicebot/llm/providers/dashscope.py:121
- The removal of the file path resolution logic for audio and image resources might result in local file paths not being correctly handled. Consider ensuring that non-HTTP paths are appropriately resolved.
if not (item.path.startswith("http") or item.path.startswith("file:")):
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
更新检查、白板检查已通过
主要更新:
nonebot-plugin-orm
,添加内置迁移插件,添加了用户表Muika.jinja2
,具体内容及其说明参见文档站.profile <profile_name>
切换目前的(用户)消息历史上下文存档model_config_manager
实例获取方式get_model_config_manager
3.12.x