Skip to content
Back to skills

Ci Monitoring

ASecurity

Use after creating PR - monitor CI pipeline, resolve failures, address review feedback, and merge when all gates pass

  • 11 stars
  • 0 votes
  • 0 copies
  • 1 view
  • Added February 7, 2026
data-aibashnodetestingcode-reviewgitapisecurity

Works with

  • api
  • mcp

Security analysis

A100/100

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

Scanned February 12, 2026

npx -y skills add troykelly/claude-skills --skill ci-monitoring --agent claude-code

Installs into .claude/skills of the current project.

Are you the author of Ci Monitoring?

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

Security grade badge for Ci Monitoring
[![Security: A — Skills Directory](https://www.skillsdirectory.com/api/skills/troykelly-ci-monitoring/badge)](https://www.skillsdirectory.com/skills/troykelly-ci-monitoring)

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
---
name: ci-monitoring
description: Use after creating PR - monitor CI pipeline, resolve failures, address review feedback, and merge when all gates pass
allowed-tools:
  - Bash
  - Read
  - Edit
  - Grep
  - Glob
  - mcp__github__*
model: opus
---

# CI Monitoring

Monitor CI pipeline, resolve failures, address review feedback, and merge when ready.

**Core principle:** CI failures AND unresolved review feedback are blockers.

**Announce:** "I'm monitoring CI and review feedback, and will resolve any issues before merging."

## The Loop

```
PR Created
     │
     ▼
┌─────────────┐
│ Wait for CI │
└──────┬──────┘
       │
   ┌───┴───┐
   │       │
 Green   Red ──► Fix → Push → Loop back
   │
   ▼
┌──────────────────────┐
│ Unresolved Reviews?  │
└──────────┬───────────┘
       ┌───┴───┐
       │       │
     None    Yes ──► For each: Evaluate → Fix/Pushback → Reply → Resolve → Push → Loop back
       │
       ▼
┌─────────────┐
│ MERGE PR    │
└─────────────┘
```

## Gate: Green CI + No Unresolved Reviews = Merge

When both conditions met, merge immediately:

```bash
gh pr merge [PR_NUMBER] --squash --delete-branch
gh issue edit [ISSUE_NUMBER] --remove-label "status:in-review" --add-label "status:done"
```

Do NOT stop to report or ask. Continue to next issue.

**Exception:** PRs with `do-not-merge` label.

## Checking CI

```bash
gh pr checks [PR_NUMBER]
gh pr checks [PR_NUMBER] --watch
```

## Handling CI Failures

1. Identify: `gh run view [RUN_ID] --log-failed`
2. Reproduce locally
3. Fix and push
4. Wait for CI, repeat if needed

**Detailed patterns:** See [reference/ci-failures.md](reference/ci-failures.md)

## Handling Review Feedback

After CI green, check for unresolved threads:

```bash
gh api graphql -f query='
query($owner: String!, $repo: String!, $pr: Int!) {
  repository(owner: $owner, name: $repo) {
    pullRequest(number: $pr) {
      reviewThreads(first: 100) {
        nodes { id isResolved isOutdated comments(first: 1) { nodes { body author { login } path } } }
      }
    }
  }
}' -f owner="[OWNER]" -f repo="[REPO]" -F pr=[PR_NUMBER]
```

For each unresolved thread (`isResolved: false`):

| Step | Action |
|------|--------|
| 1 | Read and understand feedback |
| 2 | Verify: technically valid for this codebase? |
| 3 | Fix, defer (with issue), or push back with reasoning |
| 4 | Reply in thread: `gh api repos/.../pulls/.../comments/[ID]/replies -f body="..."` |
| 5 | Resolve: `gh api graphql -f query='mutation { resolveReviewThread(...) }'` |
| 6 | Push fixes, loop back to CI |

**Detailed commands:** See [reference/review-feedback.md](reference/review-feedback.md)

### Security Feedback

Security flags from automated reviewers (Codex, CodeRabbit, etc.) should be treated seriously and verified against the codebase.

## Best Practices

**Run locally first.** CI validates, doesn't discover.

```bash
pnpm lint && pnpm typecheck && pnpm test && pnpm build
```

If CI finds bugs you didn't find locally, your local testing was insufficient.

## Checklist

CI:
- [ ] All checks green
- [ ] Failures fixed (if any)

Review feedback:
- [ ] All threads resolved
- [ ] Each: evaluated → fixed/pushback → replied → resolved

Merge:
- [ ] `gh pr merge --squash --delete-branch`
- [ ] Issue marked Done
- [ ] Continue to next issue

## Integration

Called by: `issue-driven-development`, `autonomous-orchestration`

Follows: `pr-creation`

Uses: `receiving-code-review` (principles for evaluating feedback)

Completes: PR lifecycle

Files in this skill

  • SKILL.md3.7 KB
  • reference/ci-failures.md2.1 KB
  • reference/review-feedback.md2.9 KB

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.