Skip to content

Navigation Menu

Sign in
Sign up

chore: 完善 master AI 代码审查与多轮问题复核 - #134

Open
dengyh wants to merge 6 commits into
master from
chore/ai-code-review-glm53-20260909
Open

chore: 完善 master AI 代码审查与多轮问题复核 #134
dengyh wants to merge 6 commits into
master from
chore/ai-code-review-glm53-20260909

Conversation

@dengyh

@dengyh dengyh commented Sep 9, 2026
edited
Loading

Copy link
Copy Markdown
Collaborator

完善 master 的 AI 代码审查:模型通过受限源码阅读发现问题,并在后续提交中按稳定问题编号逐项确认"仍存在、已修复、无法确认"。保留该分支专属知识与规则,只更新审查执行器、测试、工作流和使用说明。

审查与回复

  • 每轮最多新增 8 个 P1/P2 问题;上轮全部跟踪项都必须明确复核,包括已修复项是否回归。漏项、重复/伪造编号、无效源码证据拒绝发布;没有再次报告不等于修复。
  • 更新同一条审查汇总,有历史时另发一条链接原汇总的复核回复。同一 head/base 重跑更新同一回复;不修改人工评论,不自动关闭原生行内讨论。
  • 最多继续跟踪 40 项;优先保留未解决问题,容量不足时明确归档最早的已修复项。本轮回复保留全部复核结论;未解决和新问题自身超限则失败,不静默丢弃。
  • 旧版自然语言评论不作为可信状态导入,从新版首次成功记录开始跟踪;历史过期、缺失或切换目标分支会持续说明未完成的历史复核边界。

权限与状态来源

PR 作者和触发者均需仓库 write/maintain/admin 有效权限。可信目标分支脚本读取 head 普通源码快照与完整 PR diff;模型仅开放 Read/Glob/Grep/StructuredOutput,不接收 GitHub token,不执行命令、编辑、联网工具或 MCP,不读取公开讨论正文。源码与历史描述都只作为数据。

独立发布 job 先校验并保存状态 artifact,再发布评论。历史读取核对同仓库、指定工作流、pull_request_target 事件、确切成功 run attempt、PR/目标分支、artifact 摘要与大小;下载重定向不转发 GitHub token。状态保留 90 天,并保留上一轮成功入口用于取消/失败恢复。历史读取增加 actions:read,模型进程不继承该凭据。发布按实际 artifact ID 取结果,支持仅重跑失败的发布 job。

模型与分支生效

模型由仓库 variable AI_REVIEW_MODEL 选择,沿用 CODEBUDDY_API_KEY Secret。Actions、任务及评论标题均为通用 AI 名称。单次 24 turns、900 秒限制保留,这不是服务端每日金额硬预算。

默认分支工作流提供入口,目标分支提供已合入脚本和知识。应先合入默认入口,再补各维护分支;目标仅有旧版执行器时,默认入口保留其无历史发布方式。接入 PR 的测试通过不代表其 head 已用于线上模型审查,不自动合入或部署。

验证

  • 本地 37 项 runner 测试通过,包括三轮状态继承/回复幂等、取消回退、过期/改目标、历史容量、源码证据、下载凭据隔离和真正的 stage CLI 入口。
  • Black/isort/flake8、actionlint、diff 检查通过。
  • 当前提交 0ec1cb0ae015GitHub 无凭据 runner 检查 已通过 37 项测试。
  • 独立样例真实调用模型三轮:首轮发现租户过滤和负数扣款两个问题;第二轮返回"已修复 / 仍存在";第三轮返回"已修复 / 已修复",问题编号不变。原始模型 JSON 经最终执行器重新验证,当前源码证据匹配。
  • 历史持久化和回复的三轮联调使用本地 GitHub API 测试替身;真实 Actions 历史下载与线上评论链路需本 PR 合入后用新事件验证。未把本地样例当作线上验收。

本次为已授权的 CI 维护,不编造关联 TAPD 单据。

@dengyh dengyh changed the title (削除) chore: 接入 GLM-5.3 AI 审查并补充项目知识与约束 (削除ここまで) (追記) minor: add isolated AI code review and project review rules (追記ここまで) Sep 9, 2026
@dengyh dengyh changed the title (削除) minor: add isolated AI code review and project review rules (削除ここまで) (追記) chore: 完善 master AI 代码审查与多轮问题复核 (追記ここまで) Sep 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

AltStyle によって変換されたページ (->オリジナル) /