Skip to content

minor: 优化 bamboo_pipeline_3.24_lts AI 审查报告与默认配置 - #306

Open
dengyh wants to merge 1 commit into
TencentBlueKing:bamboo_pipeline_3.24_ltsfrom
dengyh:chore/ai-review-report-bamboo-pipeline-3-24-lts-20260915
Open

dengyh wants to merge 1 commit into
TencentBlueKing:bamboo_pipeline_3.24_ltsfrom
dengyh:chore/ai-review-report-bamboo-pipeline-3-24-lts-20260915

Conversation

@dengyh

@dengyh dengyh commented Sep 15, 2026

Copy link
Copy Markdown
Collaborator

将 AI 审查默认模型切换为现有 CodeBuddy 通道的 DeepSeek V4 Pro(deepseek-v4-pro-ioa),并优化 Review / Validate 报告的阅读层次。默认模型直接配置在工作流中,旧的仓库模型变量不再覆盖该配置;任务和评论标题保持通用名称,沿用现有 CODEBUDDY_API_KEY

  • 🔎 Review:结论与数量概览置顶,待处理问题保持可见,已修复项和代码证据可折叠;完整历史复核说明不再截断。正文已有明确标签时按影响、原因、修复分表展示,否则保留完整正文。
  • 🧪 Validate:从真实自测结果生成统计和失败诊断,零用例、全部跳过或中断均不显示为通过;保留失败退出码。
  • 📋 运行状态:说明自测和模型审查的事件分工,以及跳过、超时、空输出、结果无效和发布失败;汇总任务无 checkout、模型凭据或写权限。
  • 🔒 保留作者和事件发起者写权限校验、模型只读工具、可信目标分支知识、结构化输出校验、稳定问题 ID 和多轮回复幂等;不扩大 24 轮 / 900 秒预算。

验证:本地 51 项无凭据测试通过(含原有三轮问题复核、安全与历史状态用例),工作流 actionlint 检查通过。已用此前三轮真实模型输出重放新渲染器,并通过 GitHub Markdown 接口核对折叠区、表格和原样代码证据。本次不重复调用真实模型,不将本地检查当作合入后线上生效证明。

目标分支:bamboo_pipeline_3.24_lts。模型与报告改动在同一 PR 交付,分支专属知识和业务代码保持原样。默认分支的工作流提供事件入口,目标 base 提供审查脚本;合入后需以新 PR 事件验证新配置。此 PR 的模型审查可能仍使用已合入的旧配置。

关联需求:138275259

@codecov

codecov Bot commented Sep 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.38%. Comparing base (b237121) to head (7cbef52).
⚠️ Report is 1 commits behind head on bamboo_pipeline_3.24_lts.

Additional details and impacted files

Impacted file tree graph

@@                     Coverage Diff                      @@
##           bamboo_pipeline_3.24_lts     #306      +/-   ##
============================================================
+ Coverage                     86.35%   86.38%   +0.02%     
============================================================
  Files                            62       62              
  Lines                          4031     4031              
============================================================
+ Hits                           3481     3482       +1     
+ Misses                          550      549       -1     

see 1 file with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3830639...7cbef52. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

Copy link
Copy Markdown

AI 代码审查 · 7cbef528cb04

仅辅助人工审查;未执行测试,也不代表已满足合入或发布条件。

[P2] 大历史 PR 报告可超 58k 上限致整轮发布失败(移除 300 字截断)

问题编号:F-b5e1b8f576cf

.github/scripts/ai_review.py:607

触发条件:PR 经多轮审查累计接近 MAX_HISTORY=40 个跟踪问题(validate_review 第 531 行允许“未解决 followup + 新增 findings ≤ 40”),每项 body 接近 600 字上限并附 300 字 evidence quote(SCHEMA 第 61/72 行)。

调用链:publish 任务 → stage_publication(第 647 行调用 render_review 预检)→ render_followups。本次修改删除了旧版 item["body"][:300] 截断(被删除的第 596-597 行),改为完整渲染 body(第 607 行 render_explanation),并新增证据引用块(第 612-615 行:<pre> 内最多 300 字 quote 加证据链接)。单条 open 项渲染体积从约 0.8-1KB 增至约 1.5-3KB(title≤100 转义可膨胀、path 出现在链接文本与 URL 各一次、body 转义后可膨胀约一倍、表格与 disclosure 包装)。约 30-40 项未解决历史再加最多 8 条 findings(每条 body≤1600)即可超过第 880 行的 58000 字符守卫。

影响:stage_publication 抛 ValueError → publish 任务失败:state.json 不写、历史 artifact 不上传、PR 评论不更新、历史指针不前进;本轮已消耗的模型审查结果整体丢失,无降级路径。旧版同样输入(40 项 × 300 字截断)约 30-40KB 可正常发布,属本 PR 引入的容量回归。长生命周期多问题 PR 正是多轮跟踪的设计场景。

最小修复:超预算时降级而非整体失败——例如按剩余预算逐项截断 body/quote 并注明“已截断”,或减少渲染条数并保留指向 state artifact 的说明;也可按 MAX_HISTORY×每项上限重新核算并收紧 schema 上限。

审查边界

仅静态审查,未执行 validate 自测或任何测试,结论不含运行验证。快照按任务说明过滤了 4 个 engine_admin 静态 JS/CSS 文件(本次 diff 未涉及,按要求列明不可审查)。report 任务 permissions: {} 下向 GITHUB_STEP_SUMMARY 写入是否被 runner 正常上传无法静态确认;经验上通常可用,若失败只影响状态说明页,不影响审查发布本身。CodeBuddy CLI 2.147.0、deepseek-v4-pro-ioa 模型及公司通道可用性、GitHub Actions 运行时行为(needs/outputs 传递、skipped 语义、pull_request_target 默认分支语义)均按文档与源码推演,未实测。长度超限触发点为基于 schema 上限与转义膨胀的字符预算估算,未构造 40 项历史实际运行验证。本 PR 未修改业务源码(bamboo_engine/runtime),业务 CI 与包发布不受本次改动影响。

快照过滤了 4 个大文件、链接或工具配置文件;diff 仍包含其修改。

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