Pull Request File Review

Flag unnecessary test artifacts and temporary files in pull requests.

7|1|Updated Oct 19, 2025
One-click install
npx skills add https://github.com/jack-michaud/faire --skill pull-request-file-review
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: Pull Request File Review
Source: https://github.com/jack-michaud/faire/tree/main/jack-software/skills/code-review/pr-file-review
Command: npx skills add https://github.com/jack-michaud/faire --skill pull-request-file-review

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Identifies and flags unnecessary test artifacts and temporary files in pull requests.

Core Features & Use Cases

  • Pattern-based detection of artifacts
  • Context-aware evaluation of file necessity
  • Clear feedback format for removal or approval

Quick Start

Run a PR review pass to flag artifacts and produce a standardized report.

Frequently Asked Questions about Pull Request File Review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I identify and remove unnecessary files from pull requests?

Pull request file review flags unnecessary test artifacts and temporary files like test*.md, *results.md, *output.md, test_*.csv, scratch.py, temp.py, and debug.log that clutter diffs. Pattern-based detection identifies these files across root or unusual locations, then evaluates whether each is necessary for the changeset, producing clear feedback on what to remove.

What kinds of test artifacts and temporary files should I clean from PRs?

Common artifacts include test output files (test*.md, *results.md, *output.md), test data CSVs (test_*.csv, *_test_data.csv), debug scripts (scratch.py, temp.py, debug.py), and log files (output.txt, results.json, debug.log). These files often accumulate during local development and testing but don't belong in production code.

Can I automate cleanup of temporary files during code review?

Yes, automated pull request review can detect and flag temporary files using pattern matching. This context-aware evaluation identifies suspicious files and provides actionable feedback on removal, keeping diffs clean and reducing manual review overhead.

What's the best way to keep production PRs free of test artifacts?

Integrate artifact detection into your pull request workflow to catch unnecessary files before merge. Pattern-based scanning identifies test outputs, temporary data, and debug scripts early, enabling reviewers to request removal and maintain production-ready code quality.

How does pattern-based file detection work in pull request reviews?

Pattern-based detection matches file names and paths against known artifact signatures—test prefixes, debug keywords, output file extensions—then evaluates whether flagged files serve the changeset. This four-step process identifies suspicious files, assesses necessity, flags or approves them, and delivers structured feedback.

Do I need to manually specify which files are artifacts in my PR reviews?

No, the review process uses predefined patterns to automatically detect common test artifacts and temporary files across standard and unusual locations. Context-aware evaluation determines necessity, reducing manual triage and standardizing removal decisions.