ci(actions): 同步更新日志检查与署名工作流 - #636
Conversation
There was a problem hiding this comment.
Sorry @HarcoChen, you've used your own review budget of 250,000 diff characters for the last 7 days.
You can request another review in 3 days by commenting @sourcery-ai review. Upgrade to get a review now.
审查者指南本 PR 将 CHANGELOG.md 单源化,新增统一的解析、版本同步与一致性检查机制,并以新的 CI 工作流强制每个 PR 更新日志;合并后工作流会根据 PR 及共同作者自动补充条目署名,同时更新构建使用的 Runtime 版本。 PR 更新日志验证与贡献者署名时序图sequenceDiagram
participant PR as Pull Request
participant Check as check-changelog
participant Script as changelog.py
participant Files as PR 文件列表
participant Merge as append-version-contributor
participant GitHub as GitHub API
participant Dev as dev 分支
PR->>Check: opened / synchronize / reopened
Check->>Script: check
Script-->>Check: 格式与生成物一致性结果
Check->>Files: listFiles
Files-->>Check: PR 文件列表
Check-->>PR: CHANGELOG.md 检查通过或失败
Merge->>GitHub: gh pr view --json author,commits
GitHub-->>Merge: PR 作者与共同作者
Merge->>Script: parse_changelog(base 与 head)
Script-->>Merge: PR 新增条目
Merge->>Merge: 为新增条目补充去重署名
Merge->>Script: render_changelog
Script-->>Merge: 更新 CHANGELOG.md
Merge->>Script: sync
Script-->>Merge: 更新版本生成物
Merge->>Dev: 提交并推送署名与生成物
文件级变更
可能关联的 issue
提示与命令与 Sourcery 交互
自定义你的体验访问你的控制面板,以便:
获取帮助Original review guide in EnglishReviewer's Guide本 PR 将 CHANGELOG.md 单源化,新增统一的解析、版本同步与一致性检查机制,并以新的 CI 工作流强制每个 PR 更新日志;合并后工作流会根据 PR 及共同作者自动补充条目署名,同时更新构建使用的 Runtime 版本。 Sequence diagram for PR changelog validation and contributor signingsequenceDiagram
participant PR as Pull Request
participant Check as check-changelog
participant Script as changelog.py
participant Files as PR 文件列表
participant Merge as append-version-contributor
participant GitHub as GitHub API
participant Dev as dev 分支
PR->>Check: opened / synchronize / reopened
Check->>Script: check
Script-->>Check: 格式与生成物一致性结果
Check->>Files: listFiles
Files-->>Check: PR 文件列表
Check-->>PR: CHANGELOG.md 检查通过或失败
Merge->>GitHub: gh pr view --json author,commits
GitHub-->>Merge: PR 作者与共同作者
Merge->>Script: parse_changelog(base 与 head)
Script-->>Merge: PR 新增条目
Merge->>Merge: 为新增条目补充去重署名
Merge->>Script: render_changelog
Script-->>Merge: 更新 CHANGELOG.md
Merge->>Script: sync
Script-->>Merge: 更新版本生成物
Merge->>Dev: 提交并推送署名与生成物
File-Level Changes
Possibly linked issues
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
共同署名这里建议改回只签 PR 作者。 最近 5 个已合并 PR 里有 3 个会中招(#635、#615、#610),合并后条目会挂上 只签 PR 作者的话正好和 dev 上现在那份一致,也省得 main 与 dev 的机器人又不一样。 另外 |
|
为啥不带co-author,Claude倒是可以过滤掉
…________________________________
From: qiyinxi ***@***.***>
Sent: Wednesday, 09 September 2026 18:36:14
To: AUTO-MAS-Project/AUTO-MAS ***@***.***>
Cc: HarcoChen ***@***.***>; Mention ***@***.***>
Subject: Re: [AUTO-MAS-Project/AUTO-MAS] ci(actions): 同步更新日志检查与署名工作流 (PR #636)
[https://avatars.githubusercontent.com/u/52456734?s=20&v=4]qiyinxi left a comment (AUTO-MAS-Project/AUTO-MAS#636)<#636 (comment)>
共同署名这里建议改回只签 PR 作者。gh pr view --json commits 的 authors 会把 Co-authored-by trailer 算进去,而 ***@***.*** 解析出的是真实账号 claude:
$ gh pr view 635 --json commits -q '.commits[].authors[].login'
qiyinxi
claude
最近 5 个已合并 PR 里有 3 个会中招(#635<#635>、#615<https://github.com/AUTO-MAS-Project/AUTO-MAS/pull/615>、#610<https://github.com/AUTO-MAS-Project/AUTO-MAS/pull/610>),合并后条目会挂上 by ***@***.***(#615> 还会额外签上分支里夹带提交的 @AthenaHibou<https://github.com/AthenaHibou>。
只签 PR 作者的话正好和 dev 上现在那份一致,也省得 main 与 dev 的机器人又不一样。
另外 CHANGELOG.md / res/version.json 看起来是从 main 自己的旧 version.json 重新迁的:beta.3 段 94 条,dev 现在是 122 条,还少了 破坏性变更 和 本次亮点 两个分类。main 的五处版本号已经是 v5.5.0-beta.3,直接换成 dev 上那两份 changelog.py check 一样能过,可以省掉下次 dev→main 合并时的冲突。
―
Reply to this email directly, view it on GitHub<#636?email_source=notifications&email_token=AJAENWWYBQWVFM2Y6HNAXS35OEXB5A5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTKNRQGA2DMNZYGM32M4TFMFZW63VHNVSW45DJN5XKKZLWMVXHJLDGN5XXIZLSL5RWY2LDNM#issuecomment-5600467837>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AJAENWROFSIZ3DJBLN6MZQD5OEXB5AVCNFSNUABFKJSXA33TNF2G64TZHM3TKMZWGA3TENBRHNEXG43VMU5TKMZZHAYTCMBVGQ32C5QC>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
|
@qiyinxi review |
|
第四轮( 必修
建议修
提一句
核过没问题的
静态审 + 用本 PR 的 |
Sourcery 摘要
统一更新日志、版本信息校验与贡献者署名流程,确保 PR 变更能够自动登记并保持相关生成物一致。
新功能:
错误修复:
改进:
持续集成:
文档:
Original summary in English
Sourcery 摘要
统一更新日志、版本信息校验和贡献者署名流程,确保 PR 变更及其生成物保持一致。
新功能:
错误修复:
增强功能:
持续集成(CI):
文档:
Original summary in English
Sourcery 摘要
围绕
CHANGELOG.md统一发布说明和版本管理,同时通过 CI 强制执行一致性并标注贡献者。新功能:
CHANGELOG.md的工具,用于同步、验证和查询项目版本信息。错误修复:
增强功能:
CHANGELOG.md设为发布说明和版本信息的唯一手动维护来源,替代旧版version.json验证工作流。CI:
文档:
测试:
杂项:
CHANGELOG.md条目,并确保生成的版本文件保持同步。Original summary in English
Summary by Sourcery
Standardize release-note and version management around CHANGELOG.md while enforcing consistency and contributor attribution through CI.
New Features:
Bug Fixes:
Enhancements:
CI:
Documentation:
Tests:
Chores: