Run any Skill in Manus with one click

Home
Creators
torrust
torrust-tracker
review-pr

review-pr

Stars513

Forks55

UpdatedMay 12, 2026 at 16:38

Review an existing pull request for the torrust-tracker project. Covers checklist-based PR quality verification, code style standards, test requirements, documentation, and review feedback. Use only when a PR already exists.

Installation

Install with Codex or Claude Copy this prompt, paste it into Codex, Claude, or another assistant, and let it review the skill page and install it for you.

Run Skill in Manus

Source

torrust

torrust/torrust-tracker

View GitHub Repository View Creator Repositories

Download

Run Skill in Manus

Related occupationsSOC

Based on SOC occupation classification

Software Quality Assurance Analysts and TestersComputer and Mathematical Occupations·SOC 15-1253

SKILL.md

readonly

name	review-pr
description	Review an existing pull request for the torrust-tracker project. Covers checklist-based PR quality verification, code style standards, test requirements, documentation, and review feedback. Use only when a PR already exists.
metadata	{"author":"torrust","version":"1.0"}

Reviewing a Pull Request

Use this skill only when a pull request exists (PR number or URL is available).

If there is no PR yet and you need to validate task completion on a local branch, use: .github/skills/dev/task-reviews/review-task/SKILL.md.

Quick Overview Approach

Read the PR title and description for context
Check the diff for scope of change
Identify the affected packages and components
Apply the checklist below

PR Review Checklist

PR Metadata

Title follows Conventional Commits format
Description clearly explains what changes were made and why
Issue is linked (Closes #<number> or Refs #<number>)
Target branch is develop (not main)

Code Quality

Code follows existing patterns in affected packages
No unused imports, variables, or functions
No #[allow(...)] suppressions unless clearly justified with a comment
Errors handled properly (use thiserror for structured errors, avoid .unwrap())
No security vulnerabilities (OWASP Top 10 awareness)

Tests

New functionality has unit tests
Integration tests added if applicable
All existing tests still pass
Test code is clean, readable, and maintainable

Documentation

Public API items have doc comments
AGENTS.md updated if architecture changed
Markdown docs updated if user-facing behavior changed
Spell check: new technical terms added to project-words.txt

Rust-Specific

Imports grouped: std → external → internal
Line length within max_width = 130
GPG-signed commits

Providing Feedback

Categorize comments to help the author prioritize:

Blocker — must fix before merge (correctness, security, breaking changes)
Suggestion — improvement recommended but not blocking
Nit — minor style/readability point

Standards Reference

All code quality standards are defined in the root AGENTS.md. When pointing to a standard, reference the relevant section of AGENTS.md.

More from this repository

process-copilot-suggestions

torrust/torrust-tracker

End-to-end workflow for processing and resolving all Copilot code review suggestions on a pull request in torrust-tracker. Use when asked to handle PR review feedback, process all copilot suggestions, audit and resolve review comments, or manage copilot-generated review threads. Triggers on "process copilot suggestions", "handle all PR feedback", "resolve copilot review", "audit PR suggestions", or "close all copilot comments".

2026-06-30513

run-manual-docker-security-scan

torrust/torrust-tracker

Guide for running a manual Docker security scan for the tracker runtime image and documenting results. Covers build, Trivy scan, CVE triage, per-CVE catalog updates, and scan history updates. Use when asked to run a manual container scan, triage Docker CVEs, or refresh security scan docs.

2026-06-29513

use-rest-api

torrust/torrust-tracker

Use the Torrust Tracker REST API. Covers authentication, all endpoints (stats, metrics, torrents, auth keys, whitelist), and making announce/scrape requests to verify API behaviour. Triggers on "use API", "test API", "call REST API", "query API", "API endpoint", "curl tracker", "tracker client", "announce request", or "verify API".

2026-06-29513

run-linters

torrust/torrust-tracker

Run code quality checks and linters for the torrust-tracker project. Includes Rust clippy, rustfmt, markdown, YAML, TOML, spell checking, and shellcheck. Use when asked to lint code, check formatting, fix code quality issues, or prepare for commit. Triggers on "lint", "run linters", "check code quality", "fix formatting", "run clippy", "run rustfmt", or "pre-commit checks".

2026-06-27513

fix-clippy-warnings

torrust/torrust-tracker

Guide for fixing Rust Clippy warnings in the torrust-tracker project. Covers proper application of clippy suggestions, when to add allowances, and how to document exceptions. Use when asked to fix clippy warnings, improve code quality, or resolve linter issues. Triggers on "fix clippy", "clippy warnings", "rust code quality", or "linting issues".

2026-06-27513

commit-changes

torrust/torrust-tracker

Guide for committing changes in the torrust-tracker project. Covers conventional commit format, pre-commit verification checklist, GPG signing, and commit quality guidelines. Use when committing code, running pre-commit checks, or following project commit standards. Triggers on "commit", "commit changes", "how to commit", "pre-commit", "commit message", "commit format", or "conventional commits".

2026-06-26513