Skip to content
Back to skills

Run both helper scripts:

ASecurity

Look at the pull request review comments and address any issues raised.

  • 15 stars
  • 0 votes
  • 0 copies
  • 1 view
  • Added May 26, 2026
toolsbashnode

Security analysis

A100/100

Pro scans all 4 files and shows the line behind each finding

Scanned May 27, 2026

npx -y skills add mnapoli/skill-address-pr-review --agent claude-code

Installs into .claude/skills of the current project.

Are you the author of Run both helper scripts:?

Add the live security badge to your README. It updates with every re-scan.

Security grade badge for Run both helper scripts:
[![Security: A — Skills Directory](https://www.skillsdirectory.com/api/skills/mnapoli-run-both-helper-scripts/badge)](https://www.skillsdirectory.com/skills/mnapoli-run-both-helper-scripts)

More formats (shields.io, HTML) on the badges page. Keep it an A: scan every change in CI with Pro.

Download with Pro
SKILL.md
---
description: Look at the pull request review comments and address any issues raised.
disable-model-invocation: true
context: fork
allowed-tools: Bash(bash ~/.claude/skills/address-pr-review/*)
---

## Step 1: Fetch unresolved review threads and CI status

Run both helper scripts:

```bash
bash ./get-unresolved-threads.sh
```

```bash
bash ./get-failing-ci.sh
```

The first returns a JSON array of unresolved threads. Each thread has:
- `id` — thread ID (needed to reply)
- `path` — file path
- `line` / `startLine` — line numbers in the diff
- `isOutdated` — whether the diff has changed since the comment
- `comments.nodes[]` — the comment thread (first = original comment, rest = replies), each with `author.login`, `body`, and `url`

The second checks CI status and outputs failure details if any jobs failed (exit code 1 = failures, 0 = all passed).

If there are no unresolved threads and CI is green, stop here.

## Step 2: Address each issue

If there are many items, create one task per item to keep things organized.

For each unresolved review thread:

1. Read the comment thread to understand the issue
2. Determine if a code change is needed, or if no change is needed (e.g. the comment is invalid or the suggestion is not appropriate)
3. If a change is needed, make the necessary code changes

For each CI failure:

1. Read the failure output to understand what broke
2. Fix the code or tests to make CI pass

## Step 3: Commit and push changes

After addressing all review comments and CI failures, commit your changes and push.

## Step 4: Reply to each review thread

After addressing each comment, reply using the helper script:

```bash
bash ./reply-to-thread.sh "<thread_id>" "<body>" [--resolve]
```

- If the code was fixed as mentioned in the thread, reply with simply: "Fixed as suggested." and pass `--resolve` to mark the thread as resolved.
- If no change was applied, briefly explain why and keep the thread unresolved (omit `--resolve`) so the reviewer can reply if they disagree.
- Don't be unnecessarily verbose.

Files in this skill

  • SKILL.md2 KB
  • get-failing-ci.sh1.2 KB
  • get-unresolved-threads.sh988 B
  • reply-to-thread.sh858 B

Attribution

Is this your skill, or is something wrong with this listing? Request removal or report an issue. Author removals are honored within 72 hours.

Comments

Loading comments…

Newsletter

Get the weekly list of safe new skills, and the ones we pulled.