refactor: rename pr +close to pr +refuse #105

wbtiger · 2026-06-03T23:15:25+08:00

wbtiger commented

2026-06-03 23:15:25 +08:00

Summary

pr +close 这个命令名有误导性 — 它让人以为是"无害地关闭一个 PR"，但实际调用的是 refuse_merge API，会把 PR 标记为"已拒绝"。

这导致了一个实际事故：PR #57 在代码已手动合入 master 后，被错误地用 pr +close 关闭，结果 PR 被标记为"已拒绝"而非"已合并"。

Changes

pr.go: +close → +refuse，描述改为 "Refuse and close a pull request"
pr_test.go: TestPRClose → TestPRRefuse，更新测试用例
skills/gitlink-pr/SKILL.md: 更新 shortcut 表和示例
skills/gitlink-pr/references/gitlink-pr-close.md → gitlink-pr-refuse.md: 重命名并添加注意事项

Risk

Breaking change — 现有脚本如果使用 pr +close 需要改为 pr +refuse。但为了避免更多人被误导，值得做这个改名。

## Summary `pr +close` 这个命令名有误导性 — 它让人以为是"无害地关闭一个 PR"，但实际调用的是 `refuse_merge` API，会把 PR 标记为"已拒绝"。这导致了一个实际事故：PR #57 在代码已手动合入 master 后，被错误地用 `pr +close` 关闭，结果 PR 被标记为"已拒绝"而非"已合并"。 ## Changes - `pr.go`: `+close` → `+refuse`，描述改为 "Refuse and close a pull request" - `pr_test.go`: `TestPRClose` → `TestPRRefuse`，更新测试用例 - `skills/gitlink-pr/SKILL.md`: 更新 shortcut 表和示例 - `skills/gitlink-pr/references/gitlink-pr-close.md` → `gitlink-pr-refuse.md`: 重命名并添加注意事项 ## Risk Breaking change — 现有脚本如果使用 `pr +close` 需要改为 `pr +refuse`。但为了避免更多人被误导，值得做这个改名。

wbtiger force-pushed rename/pr-close-to-refuse from 530ca472c9 to 83fe18f8c0

2026-06-03 23:20:06 +08:00

Compare

wbtiger merged commit 6636175bd8 into master

2026-06-03 23:20:11 +08:00

wbtiger referenced this issue from a commit

2026-06-03 23:20:13 +08:00

Merge pull request 'refactor: rename pr +close to pr +refuse' (#105) from muel/gitlink-cli:rename/pr-close-to-refuse into master

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No project

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#105