跳到主要内容
知仓学习社ZHICANG

pr-review-comments

Posts review findings from a JSON file as inline comments on a GitHub Pull Request, attaching each comment to its file and line. Use when you have a…

执行命令严重 0 · 高危 2giuseppe-trisciuoglio/developer-kit

它会碰到什么

扫了多少3 个文本文件,15 KB
它会碰到什么执行命令
命中总数2 处
命中统计严重 0 · 高 2 · 中 0 · 低 0
逐条看命中(2 条严重或高危)
  • scripts/post_pr_comments.py:52exec-spawn
    proc = subprocess.run(
  • scripts/post_pr_comments.py:65exec-spawn
    proc = subprocess.run(

这一栏是扫描器报的事实,不是结论。命中多不等于有毒(安全工具、规则库、示例脚本本来就会包含危险写法),命中少也不等于干净。它和你手上的凭据、文件、网络有什么关系,需要你自己看。

技能内容

PR Review Comments

Publish a JSON array of review findings as inline comments on a GitHub Pull Request,

each anchored to its file and line. Uses the GitHub API through the authenticated

gh CLI, so no token handling is needed.

Prerequisites

  • gh CLI installed and authenticated (gh auth status). The script auto-detects the

repo with gh repo view; pass --repo OWNER/REPO to override.

  • The PR number to comment on.
  • A JSON file: an array of objects. Required keys per object: file, line.

Message comes from summary and/or failure_scenario (combined into the body), or an

explicit body. See [references/json-schema.md](references/json-schema.md) for the full

schema and a sample.

Key constraint: only diff lines are commentable

GitHub only accepts an inline comment if the target line is part of the PR's diff.

line is the line number in the new file (use side: "LEFT" for removed lines).

The script fetches the PR diff, validates every finding against the actual hunks, and

skips any whose line is outside the diff — reporting them at the end so nothing is

lost silently. There is no way to attach a line comment to an unchanged, undiffed line.

Workflow

  1. Confirm the JSON path and the PR number. If the repo isn't obvious, run gh repo view.
  2. Dry-run first to see what will be posted and what gets skipped:
   scripts/post_pr_comments.py --pr <N> --json <path> --dry-run
  1. Review the "Postable" / "Skipped" counts with the user. If lines were skipped because

the diff moved, the line numbers in the JSON may be stale — reconcile before posting.

  1. Post for real, choosing the mode (see below):
   # Grouped (default): one PR review bundling all comments
   scripts/post_pr_comments.py --pr <N> --json <path> --event COMMENT

   # Individual: one separate inline comment per finding
   scripts/post_pr_comments.py --pr <N> --json <path> --mode individual
  1. Report back the created review/comment URLs and the list of any skipped findings.

Choosing the mode

| Mode | Endpoint | Use when |

|------|----------|----------|

| grouped (default) | POST /pulls/{n}/reviews | Publishing a set of findings as one review. One notification; can set --event APPROVE \| REQUEST_CHANGES \| COMMENT. |

| individual | POST /pulls/{n}/comments | Adding standalone comments incrementally, or when each finding should be its own thread/notification. |

Default to grouped with --event COMMENT unless the user wants a verdict or separate threads.

Options reference

--pr N              PR number (required)
--json PATH         JSON array of findings (required)
--repo OWNER/REPO   Override auto-detected repo
--mode grouped|individual   Default: grouped
--event COMMENT|APPROVE|REQUEST_CHANGES   Grouped-mode verdict (default COMMENT)
--review-body TEXT  Top-level summary body for the grouped review
--commit SHA        Commit to anchor to (default: PR head SHA)
--dry-run           Validate and print payloads without posting

Notes

  • Multi-line range comments: include start_line (and optional start_side) in the JSON

object alongside line; the script passes them through.

  • Always --dry-run before a real post on an unfamiliar PR — stale line numbers are the

most common failure and the dry-run surfaces them as "skipped" without side effects.

  • The script is the reliable path; don't hand-roll gh api calls for this — it handles

diff validation, repo/commit detection, and body assembly consistently.

想直接用这个技能?

本站把开放许可(MIT / Apache 等)的技能按仓库打包整理到网盘,点一下转存到你自己的网盘,不用一个个从 GitHub 拉。许可未声明的技能只给原始仓库链接,不打包。