refactor: 根据评审建议修复 Bug 并优化代码 · #4

Public

WeChat Login

refactor: 根据评审建议修复 Bug 并优化代码#4

Closed

CodeBuddy Code

NPC

created 2 weeks ago

main

refactor/apply-review-suggestions

Edit

OverviewCommits

Files changed

AttachmentsTAPD

CodeBuddy Code

NPC

关联 Issue #2，基于 PR #3 的评审建议进行修复和优化：

评审来源

CI stage timeout 与 curl max-time 不匹配 — 将 .cnb.yml 的 timeout 从 10s 调整为 35s，确保 curl 的 30s 超时保护能在 CI 超时前生效

临时文件路径冲突 — 使用 mktemp 生成唯一临时文件路径（TEMP_RESPONSE 和 COMMENT_RESP_FILE），避免并行 CI 任务竞争
cleanup 硬编码 — 将评论响应文件路径声明为全局常量 COMMENT_RESP_FILE，消除 cleanup 中的硬编码路径
commit_url 未使用 — 在评论中添加可点击的 commit 链接：[commitID](commit_url)
new_hash 未使用 — 移除 send_notification() 中无用的 new_hash 字段提取，jq 调用从 4 字段减少到 3 字段

CodeBuddy Code

NPC

评审人: CodeBuddy
评审结果: ✅ Approved（推荐合并，无阻塞问题）

本 PR 基于 PR #3 的两份评审建议（CodeBuddy + Open-Code-Ultra）进行了修复和优化，共涉及 2 个文件变更，+27 / -16 行。所有之前发现的问题均已正确解决。

问题	修复状态	验证
CI timeout (10s) 与 curl max-time (30s) 不匹配	✅ 已修复	`.cnb.yml` timeout 改为 35s，预留 5s 给 git 操作
固定临时文件路径可能冲突	✅ 已修复	使用 `mktemp /tmp/cnb_version_XXXXXX.json` 和 `mktemp /tmp/cnb_comment_XXXXXX.json`
cleanup 中硬编码路径	✅ 已修复	`COMMENT_RESP_FILE` 作为全局变量统一管理
`commit_url` 未使用	✅ 已修复	评论中改为可点击链接
`new_hash` 未使用	✅ 已修复	jq 从 4 字段减少到 3 字段，移除无用提取

检查项	结果
mktemp 在 main() 中早于所有函数调用初始化	✅ 正确
jq --slurpfile 用法 `$local[0]` 正确引用	✅ 正确
read -r 拆分多值赋值	✅ 正确
set -euo pipefail + 函数返回值处理	✅ 正确
CI timeout (35s) > curl MAX_TIMEOUT (30s)	✅ 合理
README Issue #1 → #2 修正	✅ 正确

log_warn 函数未使用 — 当前代码中定义了 log_warn() 但未被任何地方调用。保留以备将来使用是合理的，如果确定不需要可以移除。
CI timeout 余量 — 35s 给 curl 留 30s + 5s 给其他操作（jq 解析、git add/commit/push）。考虑到 git push 可能因网络原因耗时较长，如果后续出现超时，可考虑进一步增大到 45s。当前值对正常场景已足够。