Gitlink
/
gitlink-cli
0
0
Fork 49
Code Issues Pull Requests 11 Packages Projects Releases 10 Wiki Activity

精确关联 PR 的 CI 证据并完善上下文解析 #429

Open
Mengz wants to merge 3 commits from Mengz/gitlink-cli:mengz/review-context-evidence-v2 into master
pull from: Mengz/gitlink-cli:mengz/review-context-evidence-v2
merge into: Gitlink:master
Conversation 0 Commits 3 Files Changed 5 +454 -10
Mengz commented 2026-07-20 17:33:58 +08:00
Contributor
Copy Link

本 PR 替代已关闭的 #426，保留 PR 证据包能力，并修复 CI 证据可能误关联的问题。

改动重点：

  • 新增 ci_summary，优先按 PR head commit SHA 精确匹配构建，SHA 无法匹配时才回退到分支。
  • passed、failed、pending、unknown 只统计已匹配构建，并提供 matched、unmatched 和 match_mode，避免把其他分支的失败构建误判为当前 PR 结果。
  • 保留原始 ci_builds 作为可追溯附录，CI 缺失或未关联时明确提示证据不足。
  • 修复 issue_tags/tags 列表包装未被通用解析器识别的问题，避免标签上下文静默为空。
  • 增加 SHA 优先、分支回退、状态组合解析、未匹配构建隔离和标签上下文测试，并补充中文变更说明。

验证结果：

  • go test ./shortcuts/workflow -run TestFetchReviewContext|TestSummarizeReviewCI|TestRenderReviewContextFormats -count=1 通过
  • go build ./... 通过
  • git diff --check 通过

这是对已关闭 #426 的质量增强版，仍保持只读接口、旧参数和默认行为兼容。

本 PR 替代已关闭的 #426，保留 PR 证据包能力，并修复 CI 证据可能误关联的问题。 改动重点： - 新增 `ci_summary`，优先按 PR head commit SHA 精确匹配构建，SHA 无法匹配时才回退到分支。 - `passed`、`failed`、`pending`、`unknown` 只统计已匹配构建，并提供 `matched`、`unmatched` 和 `match_mode`，避免把其他分支的失败构建误判为当前 PR 结果。 - 保留原始 `ci_builds` 作为可追溯附录，CI 缺失或未关联时明确提示证据不足。 - 修复 `issue_tags`/`tags` 列表包装未被通用解析器识别的问题，避免标签上下文静默为空。 - 增加 SHA 优先、分支回退、状态组合解析、未匹配构建隔离和标签上下文测试，并补充中文变更说明。 验证结果： - `go test ./shortcuts/workflow -run TestFetchReviewContext|TestSummarizeReviewCI|TestRenderReviewContextFormats -count=1` 通过 - `go build ./...` 通过 - `git diff --check` 通过 这是对已关闭 #426 的质量增强版，仍保持只读接口、旧参数和默认行为兼容。
Mengz added 2 commits 2026-07-20 17:33:59 +08:00
Mengz added 1 commit 2026-07-22 17:32:29 +08:00
This pull request can be merged automatically.
You are not authorized to merge this pull request.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b Mengz-mengz/review-context-evidence-v2 master
git pull mengz/review-context-evidence-v2

Step 2:

Merge the changes and update on Gitea.
git checkout master
git merge --no-ff Mengz-mengz/review-context-evidence-v2
git push origin master
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Gitlink/gitlink-cli#429
No description provided.
Delete Branch "Mengz/gitlink-cli:mengz/review-context-evidence-v2"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?