agentleFS
cobusgreylingcobusgreyling/grok-bot-templates/skills/review-pr-risk/SKILL.md

review-pr-risk

Skill14 starsChanged 3 months agoMITView on GitHub

Paste this into Grok Bot after a successful run: Save the process we just used as a skill called “Review PR Risk”. A useful skill states the six fields from Skills and routines. An open pull request needs a review that starts with what can break, what is untested, and what the description promised but the diff did not do. Every risk cites a path. No invented files. No coverage percentages unless they appear in CI output. Facts and inferences…

1 other project uses a version of this.

Adopted by

Most recently taken up by

The file

We split this skill into the rules it gives an agent

3 rules. About 0 are common, 3 are distinctive, and 0 are very specific to this project.

  • ≈0

    Common · estimate

    Many files like this have a rule of this kind.

  • ≈3

    Distinctive · estimate

    This author's own choices. Worth a look.

  • 0

    Very specific

    Tied to their project's names, files or tools. Probably skip.

  • Common
  • Distinctive
  • Very specific
  • could be automatic: a setting could enforce it
  1. Distinctive:

    Read the description, the diff, and CI — in that order.

    line 23
  2. Distinctive:

    Every risk cites a path. No invented files.

    line 31
  3. Distinctive:

    Never merge, push, comment on GitHub, request reviewers, or change status checks without explicit in-chat approval.

    line 41

≈ “Common” is an estimate: at least 1 in 5 of the 350,495 skill files we read have a rule of the same kind, not the same words. “Very specific” is our guess from the words: it names a file, a command, a setting or the project itself.

Keep what fits and drop the rest. Your version, ready for your agents.
Files
SKILL.md

Review PR Risk

Paste this into Grok Bot after a successful run:

Save the process we just used as a skill called “Review PR Risk”.

A useful skill states the six fields from Skills and routines.

1. When to use it

An open pull request needs a review that starts with what can break, what is untested, and what the description promised but the diff did not do.

2. Required inputs and access

  • GitHub connector or a pasted PR URL
  • Repository access on the shared computer if already cloned
  • CI logs linked from the PR when present

3. Sequence of work

  1. Open the PR. If you cannot, stop and say so.
  2. Read the description, the diff, and CI — in that order.
  3. List breakage risk with file paths.
  4. List untested paths. If CI is missing, write "CI not found" rather than "tests look fine".
  5. Diff the description against the diff (context gap).
  6. Nits last. Verdict is one of request changes, comment, or approve-with-notes.

4. How to validate the result

Every risk cites a path. No invented files. No coverage percentages unless they appear in CI output. Facts and inferences are labeled.

5. What to return

A review pack with Risk, Tests, Context gap, Nits, Verdict, Action log. Not posted as a GitHub comment unless approved.

6. What requires approval

Never merge, push, comment on GitHub, request reviewers, or change status checks without explicit in-chat approval.

Type / in the composer to reference a saved skill. If it does not appear, open Settings → Plugins → Yours and enable it for this Bot.

Download now (ZIP)

Does it work?

Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.

No reviews yet

People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.

  • It worked
  • Partly
  • It didn't
What did you use it for? e.g. Reviewing pull requests in a Rails app

Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.

Your trail

Files you open will collect here, so you can find them again.

Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.

Now do this for your own.

Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.