已合并
修改注释 #389
修改注释 #389
已合并
zhaoxi_00622383创建于 7月22日
zhaoxi_00622383
7月22日

What type of PR is this?

/kind

Self-checklist:(请自检,在[ ]内打上x,我们将检视你的完成情况,否则会导致pr无法合入

likedislike
Pull Request已成功合入, 合并人@张雅晴
(感谢 zhaoxi_00622383 的贡献)
atomgit-bot
atomgit-bot
7月22日 评论:

变更摘要

此 PR 主要将日志空间相关配置从原始字节数值改为支持人类可读的 "500 MB" 格式,并通过 pydantic 的 field_validator 统一将 MB 字符串转为字节整数。同时,将 app.pysdk_log_cleaner.py 中分散的 os.getenv() 调用统一收敛到 Settings 类,利用 pydantic 的校验和默认值能力,避免空字符串导致的 ValueError。变更同时涉及 a2a_serviceversatile_adapter 两个子项目。

主要改动

  • 日志配置值支持 MB 格式:在 config.py 中新增 _parse_size_to_bytes() 工具函数,并在 Settings 类上通过 @field_validatorlog_max_total_sizejiuwen_log_max_bytesjiuwen_log_max_total_size(及 versatile_adapter 中的 adapter_log_max_total_size)进行预处理,使环境变量可直接配置 "500 MB" 这样的可读值。
  • .env.example 配置示例同步更新a2a_serviceversatile_adapter.env.example 文件中日志大小默认值从纯数字(如 524288000)改为 "500 MB" 格式,注释也相应从"字节"修正为"MB"。
  • 统一通过 Settings 读取日志配置app.py_cleanup_logs()lifespan(),以及 sdk_log_cleaner.pyCleanableDefaultLogger._replace_handlers_with_compressed()setup_sdk_log_cleaner(),全部从 os.getenv() 改为调用 get_settings() 获取配置,复用 pydantic 校验与默认值逻辑。
  • 新增运行时依赖pyproject.toml 中为 a2a_service 添加了 openjiuwen-runtime-foundation>=0.1.0 依赖,用于 SDK 日志清理等基础能力。
likedislike
不准确?
atomgit-bot
atomgit-bot
7月22日 评论:

代码审查

发现 3 个问题,均为中低优先级建议。

⚠️ 已识别出整体风险,但无法提取行内评论,请参考整体评估。

likedislike
不准确?
openJiuwen-bot成员
7月22日 评论:

欢迎来到 openJiuwen 社区

Hey @JiangCaifu , 感谢你对社区的贡献.

机器人使用手册

有关指令的使用,可以点击 此处 查看详情。开发人员可以在每个PR或Issue下方评论特定指令来触发机器人任务。

likedislike
atomgit-bot
atomgit-bot
7月22日 评论:

AtomGit AI 助手使用指南

在 PR 评论中使用命令即可触发。支持在普通评论和代码行评论中使用。

功能 命令 说明
代码审查 /ai review 检查代码质量、潜在问题、安全风险
PR 摘要 /ai summary 生成 PR 变更内容的结构化摘要
代码解释 /ai explain 解释代码变更的逻辑和意图
自由提问 @atomgit-bot <问题> 询问关于本 PR 的任何问题
帮助 /ai help 显示此帮助信息

免责声明

AI 助手可能存在误判,请结合自身判断。可以对评论点 👍 或 👎 帮助我们改进。

likedislike
不准确?
OopenJiuwen-bot成员
7月22日 添加了label:stat/needs-squash
OopenJiuwen-bot成员
7月22日 添加了label:openJiuwen-cla/yes
openJiuwen-bot成员
7月22日 评论:

CLA 签署成功

JiangCaifu, 感谢您的代码贡献。 所有的代码贡献者都已完成了 CLA 签署。 👍

likedislike
openJiuwen-bot成员
7月22日 评论:
任务名称 结果 日志操作
静态检查 ✅SUCCESS 点此跳转
防投毒检查 ✅SUCCESS 点此跳转
开源合规检查 ✅SUCCESS 点此跳转
UT测试 N/A N/A
ST测试 N/A N/A
build 编译包 N/A N/A
ruff codecheck {{ruff_codecheck_result}} {{ruff_codecheck_link_text}}
likedislike
OopenJiuwen-bot成员
7月22日 添加了label:ci-successful
张雅晴张雅晴成员
7月22日 合入了pull request
OopenJiuwen-bot成员
7月22日 删除了label:ci-successful