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

feat(pr): 增加行级审查评论快捷命令 #312

Closed
Mengz wants to merge 0 commits from Mengz/gitlink-cli:mengz/pr-inline-review-comment-shortcuts-pr into master
pull from: Mengz/gitlink-cli:mengz/pr-inline-review-comment-shortcuts-pr
merge into: Gitlink:master
Conversation 0 Commits - Files Changed -
Mengz commented 2026-07-01 11:50:49 +08:00
Contributor
Copy Link

变更说明

这次补齐的是 PR 评审里非常常用、但之前 CLI 还缺失的一块：行级审查评论管理。

新增后可以直接通过命令行完成以下动作：

  • pr +review-comments：查看某个 PR 的行级评论列表
  • pr +review-comment：新增行级评论
  • pr +update-review-comment：更新已有行级评论
  • pr +delete-review-comment：删除行级评论

对维护者和自动化流程来说，很多评审意见并不是简单的整段评论，而是需要精确挂到某个文件、某一行或某段 diff 上。此前 CLI 已经支持普通 review，但缺少对行级评论的读取和维护能力，导致自动审查结果、脚本化 review 建议、以及后续纠错流程都很难闭环。这次把完整的增删改查补齐后，PR 审查工作流就更完整了。

包含内容

  • 新增 pr +review-comments
  • 新增 pr +review-comment
  • 新增 pr +update-review-comment
  • 新增 pr +delete-review-comment
  • 补充命令测试与 README/变更说明文档

本地验证

已通过：

  • go test ./shortcuts/pr/...
  • go build ./...
  • go test ./...
  • git diff --check

另外做了命令级冒烟验证：

  • go run . --format json pr +review-comment --owner Gitlink --repo gitlink-cli --id 302 --review-id 1 --path README.md --line-code fake-line-code --note smoke-test --dry-run

返回中包含完整 payload，diff_source=pr_files_api，说明命令可以正确拼装行级评论请求。

## 变更说明 这次补齐的是 PR 评审里非常常用、但之前 CLI 还缺失的一块：行级审查评论管理。 新增后可以直接通过命令行完成以下动作： - `pr +review-comments`：查看某个 PR 的行级评论列表 - `pr +review-comment`：新增行级评论 - `pr +update-review-comment`：更新已有行级评论 - `pr +delete-review-comment`：删除行级评论 对维护者和自动化流程来说，很多评审意见并不是简单的整段评论，而是需要精确挂到某个文件、某一行或某段 diff 上。此前 CLI 已经支持普通 review，但缺少对行级评论的读取和维护能力，导致自动审查结果、脚本化 review 建议、以及后续纠错流程都很难闭环。这次把完整的增删改查补齐后，PR 审查工作流就更完整了。 ## 包含内容 - 新增 `pr +review-comments` - 新增 `pr +review-comment` - 新增 `pr +update-review-comment` - 新增 `pr +delete-review-comment` - 补充命令测试与 README/变更说明文档 ## 本地验证 已通过： - `go test ./shortcuts/pr/...` - `go build ./...` - `go test ./...` - `git diff --check` 另外做了命令级冒烟验证： - `go run . --format json pr +review-comment --owner Gitlink --repo gitlink-cli --id 302 --review-id 1 --path README.md --line-code fake-line-code --note smoke-test --dry-run` 返回中包含完整 payload，`diff_source=pr_files_api`，说明命令可以正确拼装行级评论请求。
Mengz added 1 commit 2026-07-01 11:50:50 +08:00
wbtiger closed this pull request 2026-07-14 22:57:14 +08:00
wbtiger reopened this pull request 2026-07-14 23:02:24 +08:00
wbtiger closed this pull request 2026-07-14 23:05:56 +08:00

Pull request closed

Please reopen this pull request to perform a merge.
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#312
No description provided.
Delete Branch "Mengz/gitlink-cli:mengz/pr-inline-review-comment-shortcuts-pr"

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?