dyoshikawa/rulesync/.rulesync/skills/post-review-comments/SKILL.md
post-review-comments
Post line-level review comments and an overall review comment on a PR in English with a natural, concise writing style
- Source repository stars
- 1,263
- Declared platforms
- 0
- Static risk flags
- 0
- Last source update
- 2026-07-28
- Source checked
- 2026-07-28
Decision brief
What it does—and where it fits
Read the PR diff and description to understand:
Not for
- Tasks that require unconfirmed production actions or broad system permissions.
- Environments where the pinned source and install steps cannot be inspected.
Compatibility matrix
Platform support, with evidence labels
| Platform | Status | Evidence | What to check |
|---|---|---|---|
| Codex | Not declared | No explicit evidence | Portability before use |
| Claude Code | Not declared | No explicit evidence | Portability before use |
| Cursor | Not declared | No explicit evidence | Portability before use |
| Gemini CLI | Not declared | No explicit evidence | Portability before use |
Installation
Inspect first. Install second.
The source command is displayed only when detected. A safe inspection prompt is always available so your agent can explain every action before execution.
npx skills add https://github.com/dyoshikawa/rulesync --skill ".rulesync/skills/post-review-comments"Inspect the Agent Skill "post-review-comments" from https://github.com/dyoshikawa/rulesync/blob/310b711fbe8cffc14debb276ade8a384c2b89083/.rulesync/skills/post-review-comments/SKILL.md at commit 310b711fbe8cffc14debb276ade8a384c2b89083. List every install step, command, network request, credential, file read/write, external action, and rollback step. Explain whether it fits my task. Do not install or execute anything until I approve.
Workflow
What the source asks the agent to do
- 01
Step 1: Understand the PR
Read the PR diff and description to understand:
What changed and whyThe scope and impact of the changesRead the PR diff and description to understand: - 02
Step 2: Prepare Line-Level Comments
Identify specific lines in the diff that deserve feedback — bugs, potential issues, style concerns, suggestions for improvement, or questions about intent.
The file pathThe line number (in the new version of the file)Your comment text - 03
Step 3: Prepare an Overall Review Comment
Write a brief overall assessment of the PR. This is a summary-level comment, not a repeat of line-level feedback.
Write a brief overall assessment of the PR. This is a summary-level comment, not a repeat of line-level feedback. - 04
Step 4: Post the Review
Use a single gh api call to submit both line-level comments and the overall comment together as one review:
Use a single gh api call to submit both line-level comments and the overall comment together as one review:Each line comment requires: path, line (line number in the diff's new file), and body.If you have no line-level comments, fall back to:
Permission review
Static risk signals and limitations
No configured static risk pattern was detected
This is not proof of safety. Runtime behavior, indirect dependencies, and hidden external systems are outside the static scan.
Evidence record
Why each signal appears
| Signal | Value | Evidence type | Meaning |
|---|---|---|---|
| Quality score | 81/100 | Computed | Documentation, specificity, maintenance, and trust rules |
| Repository stars | 1,263 | Source | Repository attention, not individual Skill quality |
| Compatibility | 0 platforms | Source | Declared in the catalog source record |
| Usage guide | automated source guide | Editorial | Generated or reviewed according to the visible evidence level |
Pinned source
Provenance and original SKILL.md
- Repository
- dyoshikawa/rulesync
- Skill path
- .rulesync/skills/post-review-comments/SKILL.md
- Commit
- 310b711fbe8cffc14debb276ade8a384c2b89083
- License
- MIT
- Collected
- 2026-07-28
- Default branch
- main
View the original SKILL.md
target_pr = the user's request
If target_pr is not provided, use the PR of the current branch.
Step 1: Understand the PR
Read the PR diff and description to understand:
- What changed and why
- The scope and impact of the changes
Step 2: Prepare Line-Level Comments
Identify specific lines in the diff that deserve feedback — bugs, potential issues, style concerns, suggestions for improvement, or questions about intent.
For each comment, note:
- The file path
- The line number (in the new version of the file)
- Your comment text
Step 3: Prepare an Overall Review Comment
Write a brief overall assessment of the PR. This is a summary-level comment, not a repeat of line-level feedback.
Step 4: Post the Review
Use a single gh api call to submit both line-level comments and the overall comment together as one review:
gh api repos/{owner}/{repo}/pulls/{pr_number}/reviews \
--method POST \
-f event="COMMENT" \
-f body="<overall comment>" \
-f 'comments[]={...}' ...
Each line comment requires: path, line (line number in the diff's new file), and body.
If you have no line-level comments, fall back to:
gh pr review <pr_number> --comment --body "<overall comment>"
Writing Style
Write in English only.
Style guidelines — write like a human teammate, not a bot:
- Use plain paragraphs. Avoid headings (
##), horizontal rules (---), and excessive bullet lists. - Keep it conversational and direct. Say what matters, skip the filler.
- Do not use phrases like "Great work!" or "LGTM!" unless you genuinely mean it and have nothing else to add.
- Be honest. If something looks off, say so clearly but respectfully.
- Short is fine. A two-sentence comment is better than a five-paragraph essay with no substance.
Step 5: Report
Output the PR number, a count of line-level comments posted, and a brief summary of the overall comment.
Alternatives
Compare before choosing
coreyhaines31/marketingskills
ab-testing
When the user wants to plan, design, or implement an A/B test or experiment, or build a growth experimentation program. Also use when the user mentions "A/B test," "split test," "experiment," "test this change," "variant copy," "multivariate test," "hypothesis," "should I test this," "which version is better," "test two versions," "statistical significance," "how long should I run this test," "growth experiments," "experiment velocity," "experiment backlog," "ICE score," "experimentation program
coreyhaines31/marketingskills
churn-prevention
When the user wants to reduce churn, build cancellation flows, set up save offers, recover failed payments, or implement retention strategies. Also use when the user mentions 'churn,' 'cancel flow,' 'offboarding,' 'save offer,' 'dunning,' 'failed payment recovery,' 'win-back,' 'retention,' 'exit survey,' 'pause subscription,' 'involuntary churn,' 'people keep canceling,' 'churn rate is too high,' 'how do I keep users,' or 'customers are leaving.' Use this whenever someone is losing subscribers o
event4u-app/agent-config
design-review
Use when the user says "review the design", "check the UI", or wants a comprehensive UI/UX review. Uses a 7-phase methodology covering interaction, responsiveness, accessibility, and more.
K-Dense-AI/scientific-agent-skills
dask
Distributed computing for larger-than-RAM pandas/NumPy workflows. Use when you need to scale existing pandas/NumPy code beyond memory or across clusters. Best for parallel file processing, distributed ML, integration with existing pandas code. For out-of-core analytics on single machine use vaex; for in-memory speed use polars.