Skip to content

Version/4.0 #1220

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

Draft
wants to merge 55 commits into
base: master
Choose a base branch
from
Draft

Version/4.0 #1220

wants to merge 55 commits into from

Conversation

RockChinQ
Copy link
Owner

@RockChinQ RockChinQ commented Mar 19, 2025

概述

实现/解决/优化的内容:

检查清单

PR 作者完成

*请在方括号间写x以打勾

  • 阅读仓库贡献指引了吗?
  • 与项目所有者沟通过了吗?
  • 我确定已自行测试所作的更改,确保功能符合预期。

项目所有者完成

  • 相关 issues 链接了吗?
  • 配置项写好了吗?迁移写好了吗?生效了吗?
  • 依赖写到 requirements.txt 和 core/bootutils/deps.py 了吗
  • 文档编写了吗?

TODO

  • 处理各个命令的兼容性
  • 组件扩展发现机制
  • 配置文件跨版本迁移
    • config.yaml 中字段的迁移
  • 加回 MCP 和 百度云审核
  • 独立工具页
  • 插件自动热重载

@RockChinQ RockChinQ added this to the 4.0 milestone Mar 29, 2025
Copy link
Contributor

@Copilot Copilot AI left a 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 implements version 4.0 by integrating new YAML configuration loading for pipeline metadata and extending the application with additional service managers and HTTP API endpoints for pipelines, models, and bots.

  • Introduces YAML config loading in load_config and config modules
  • Updates app/boot initialization to inject new managers (pipeline, model, bot)
  • Adds new API routes and controllers for pipelines, models, bots, and platform adapters

Reviewed Changes

Copilot reviewed 73 out of 73 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pkg/core/stages/load_config.py Loads additional YAML metadata configurations for pipelines
pkg/core/stages/build_app.py Initializes new managers (pipeline, model, bot) within the app
pkg/core/bootutils/deps.py Adds sqlmodel as a new dependency
pkg/core/bootutils/config.py Exposes the YAML config loader to the application
pkg/core/app.py Declares new application variables for pipeline and service management
pkg/command/operators/model.py Updates the attribute reference for token manager output
pkg/api/http/service/* Introduces new services for pipelines, models, and bots
pkg/api/http/controller/groups/* Adds new routing groups for pipelines, models, bots, and platform adapters
pkg/api/http/controller/group.py Modifies error handling to print traceback and return standardized error
Comments suppressed due to low confidence (2)

pkg/command/operators/model.py:57

  • Verify that 'model.token_mgr.name' is the intended property instead of the previous 'provider' attribute. If this change is intentional, consider updating documentation to reflect the new attribute.
content += f"密钥组: {model.token_mgr.name}\n"

pkg/api/http/controller/group.py:79

  • [nitpick] Consider replacing 'traceback.print_exc()' with a proper logging mechanism to avoid unintended console output in production environments.
traceback.print_exc()

@RockChinQ
Copy link
Owner Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant