cruldra
1bec57a1cb
- 去掉 tea、gh、Gitea 字样：查工单 spx issue view，查 PR spx pr view，发起 PR spx pr create 且描述必须含 Closes #N - 审查和实施的 diff 不再写死 origin/main：审查取 PR 的目标分支，实施取计划里的目标分支或仓库默认分支 - 工单模板同时看 .gitea/ISSUE_TEMPLATE/ 和 .github/ISSUE_TEMPLATE/，禁止合并的说法不带工具名 - 内联兜底提示词同步改；设置里存着旧版的用户激活时按现有机制被问是否换新版 - 新增测试守住自带提示词和兜底不再点名平台
46 lines
1.8 KiB
Markdown
46 lines
1.8 KiB
Markdown
审查这个仓库的 PR #{{prNumber}}。
|
||
|
||
用 `opencli spx pr view --pr {{prNumber}}` 看 PR 元信息（标题、描述、head → base 分支）。看代码差异用 git（当前目录就是 PR 分支的 worktree）：`<base>` 是上面查到的目标分支（加 `--json` 时是 `base.branch`），先 `git fetch origin <base>`，再 `git diff origin/<base>...HEAD`（概览可加 --stat）。
|
||
|
||
## 审查方式
|
||
|
||
直接把上面的 diff 喂给 ECC 的审查子代理，按改动涉及的语言挑：
|
||
|
||
- `ecc:code-reviewer`（必跑）
|
||
- `ecc:silent-failure-hunter`、`ecc:pr-test-analyzer`（必跑）
|
||
- 语言专项：TypeScript 用 `ecc:typescript-reviewer`，Python 用 `ecc:python-reviewer`，React 用 `ecc:react-reviewer`，其余同理
|
||
|
||
汇总时去重、按严重程度排序，只报置信度高的问题。
|
||
|
||
## 提交审查意见
|
||
|
||
把审查意见（markdown 格式）写到 `/tmp/review-{{prNumber}}.md`，调 spx：
|
||
|
||
```
|
||
opencli spx pr review-comment --pr {{prNumber}} --body-file /tmp/review-{{prNumber}}.md
|
||
```
|
||
|
||
审查结束以后，删除所有审查过程中创建的临时 worktree（`git worktree remove --force <路径>`），你当前所在的 worktree 除外。
|
||
|
||
## 审查结论
|
||
|
||
没有必须改的问题才算通过。小建议不算不通过。
|
||
|
||
通过：
|
||
|
||
```
|
||
opencli spx issue state merge --issue {{issueNumber}} --state-json '{"reviewPassed":true}'
|
||
```
|
||
|
||
有必须改的问题：同一条命令，把 `reviewPassed` 写成 `false`。
|
||
|
||
这条合并发生在审查评论发出之后。不要建议合并 PR，不要执行合并。
|
||
|
||
## 严禁
|
||
|
||
- 不要在审查意见里建议"合并 PR"或"merge"
|
||
- 不要执行任何合并 PR 的命令或操作
|
||
- 不要往 PR 的目标分支或其他共享分支 push
|
||
|
||
合并权完全在用户手上,你的工作只是指出问题或确认通过。
|