xiaoju/nerve-workspace
1
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

feat: add pr-code-reviewer workflow #1

Closed
xiaoju wants to merge 0 commits from wf/pr-code-reviewer-modz70v0 into master
pull from: wf/pr-code-reviewer-modz70v0
merge into: xiaoju:master
Conversation 1 Commits - Files Changed -
xiaoju commented 2026-04-25 07:08:55 +00:00
Owner
Copy Link
No description provided.
xiaoju added 1 commit 2026-04-25 07:08:56 +00:00 
Workflow: pr-code-reviewer
User request (summary): Create a code-reviewer workflow that: 1) Takes a PR URL (supporting Gitea git.shazhou.work, GitHub, and Gitee) as input. 2) A fetcher role detects the platform from the URL, authenticates using ava...
Reviewer (summary): npx tsc --noEmit passed and nerve.yaml contains the workflow entry
Staged paths:
- nerve.yaml
- workflows/pr-code-reviewer/index.ts
- workflows/pr-code-reviewer/package.json
- workflows/pr-code-reviewer/pnpm-lock.yaml
- workflows/pr-code-reviewer/tsconfig.json
xiaoju commented 2026-04-25 07:48:30 +00:00
Author
Owner
Copy Link

总览

本 PR 新增 pr-code-reviewer workflow：按 URL 识别 GitHub/Gitea/Gitee，拉取 PR 元数据与 diff，调用 cursor-agent（ask）生成结构化评审 JSON，再回帖。整体职责拆分清晰（fetcher/reviewer/commenter/reporter），meta 字段较完整，与 pr-summarizer 在 diff 截断常量上对齐，方向正确、可合并落地。

做得好的点

  • 多平台分支（GitHub 走 gh，Gitea/Gitee 走 token + fetch）边界清楚；Gitee 用 files 的 patch 拼接属于合理折中（若 API 无统一 .diff）。
  • dryRun/jsonDryRun 与 pr-summarizer 语义对齐：跳过 agent 与发帖，避免误操作。
  • 评论正文与 verdict 分离展示，HTTP 错误映射到 errorCode 便于排障。

主要问题（非阻塞）

  1. reporter 在多条成功路径上缺失（见 warnings），会降低可观测性与自动化下游消费。
  2. JSON 从自然语言输出中抽取的健壮性有限（见 warnings）。
  3. 环境可移植性：硬编码 HOME 回退路径偏部署耦合（见 warnings）。

结论

未发现明显安全级缺陷（token 仅用于请求头、临时文件有 finally 清理）。建议在后续迭代补齐 reporter 覆盖与解析健壮性；当前可作为 v1 带备注合并。

Verdict: comment

## 总览 本 PR 新增 `pr-code-reviewer` workflow：按 URL 识别 GitHub/Gitea/Gitee，拉取 PR 元数据与 diff，调用 `cursor-agent`（ask）生成结构化评审 JSON，再回帖。整体职责拆分清晰（fetcher/reviewer/commenter/reporter），meta 字段较完整，与 `pr-summarizer` 在 diff 截断常量上对齐，**方向正确、可合并落地**。 ## 做得好的点 - 多平台分支（GitHub 走 `gh`，Gitea/Gitee 走 token + `fetch`）边界清楚；Gitee 用 files 的 `patch` 拼接属于合理折中（若 API 无统一 `.diff`）。 - dryRun/jsonDryRun 与 pr-summarizer 语义对齐：跳过 agent 与发帖，避免误操作。 - 评论正文与 verdict 分离展示，HTTP 错误映射到 `errorCode` 便于排障。 ## 主要问题（非阻塞） 1. **reporter 在多条成功路径上缺失**（见 warnings），会降低可观测性与自动化下游消费。 2. **JSON 从自然语言输出中抽取**的健壮性有限（见 warnings）。 3. **环境可移植性**：硬编码 `HOME` 回退路径偏部署耦合（见 warnings）。 ## 结论 未发现明显安全级缺陷（token 仅用于请求头、临时文件有 finally 清理）。建议在后续迭代补齐 reporter 覆盖与解析健壮性；当前可作为 v1 **带备注合并**。 --- **Verdict:** comment
xiaoju closed this pull request 2026-04-28 23:06:12 +00:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
xiaoju
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: xiaoju/nerve-workspace#1
No description provided.
Delete Branch "wf/pr-code-reviewer-modz70v0"

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?