已合并
feat(sift): import v2.6.0 and integrate test runners #11
feat(sift): import v2.6.0 and integrate test runners #11
已合并
hangdu创建于 9月14日
hangdu成员
9月14日

将 Sift v2.6.0 引入 packages/sift,提供评估与审计能力,并接入 Sentry 统一测试入口。

引入内容

  • 导入基线为 Sift v2.6.0(d00bcb97),pyproject.toml 与 __version__ 均为 2.6.0。在该基线上进行 Sentry 集成和内部重构。
  • 评估与审计功能统一位于 packages/sift,使用 sift eval、sift audit 命令。
  • 包内测试入口为 packages/sift/scripts/run-tests.sh,准备测试依赖并运行 pytest。
  • 仓级 scripts/run-tests.sh 支持 insight、cpx、sift、auto、all。auto 根据改动路径选择测试范围,识别 packages/sift 下的变更;共享文件变更或无法判断范围时执行全量测试。

集成调整

  • 整理执行、审计和报告模块的内部实现,复用公共逻辑。报告目录扫描、文件预览和数据组装经过重构,保留原有输出格式及回退行为。
  • 部分内部 Python 调用改用具名配置对象,仓内调用已同步调整;外部直接调用这些接口的代码需要同步适配。CLI 参数和默认值保持一致。
  • 补齐包初始化文件的版权头和模块说明,避免空文件检查失败。
  • 补充 Sift 原创部分的仓库版权声明,保留 Anthropic 衍生部分的 Apache-2.0 许可证及来源说明,并按贡献指南新增第三方组件清单与通知。登记范围为本次引入的 Sift 内容及其直接依赖。

验证

  • 统一入口 ./scripts/run-tests.sh sift:1804 项通过,10 项跳过。
  • 最后一次报告逻辑修复后,报告模块回归:112 项通过。
  • 报告重构前后对照验证:报告数据、完整 HTML 及 HTTP 响应一致;81 种元数据和提示词回退组合的返回值一致。
  • 8 个包初始化文件仅增加版权注释和模块说明,语法检查通过;本地检查确认 PR 变更文件中没有空文件。

远端流水线结果以最新提交运行结果为准。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 hangdu 的贡献)
Hhangdu成员
9月14日 创建了 pull request,commit 0cc31e6f
CANN-robot
CANN-robot成员
9月14日 评论:

Hi @hangdu, welcome to submitting your first PR to cannbot-sentry!

PR Merge Steps

1. CLA Signing

If the current PR label includes cann-cla/yes, it means you have signed the CLA and can proceed to the next step. If the label includes cann-cla/no, please sign the CLA first. If you have any questions, please refer to the FAQ.

2. CI Check

Please comment /compile to trigger the CI pipeline check. If the CI run is successful, the PR will be tagged with ci-pipeline-passed and you can proceed to the next step. If the CI run fails, the PR will be tagged with ci-pipeline-failed, please check the CI logs to fix the issues in the PR. If you have any questions, please refer to the FAQ.

3. Code Review

After CI passes, please refer to the PR Approval Progress and proactively @ the committers in the table to review the code. After approval, committers will comment /lgtm and /approve. Once the lgtm and approved labels are successfully added, the PR will be merged automatically.

likedislike
CANN-robotCANN-robot成员
9月14日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
9月14日 评论:

Thanks for your pull-request.
The full list of commands accepted by me can be found at here.
You can get sig-info at here.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.
For more, you also can visit HICANN.


PR Approval Progress

✅ Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
repo-cann/cannbot-sentry ✅ qwren, guanxinghua (2/2) ✅ qwren (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

hangdu, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
Hhangdu成员
9月15日 预合并成功(commit_id: 842f9e7f13dc118a1df7efacb1a9c82f0e89033e)
此处折叠了161条消息 查看更多
qwren成员
9月20日 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
9月20日 添加了label:approved
guanxinghua成员
9月20日 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
9月20日 添加了label:lgtm
CANN-robotCANN-robot成员
9月20日 合入了pull request