agentleFS
elon-chooelon-choo/fablever/.agents/skills/fable-review/SKILL.md

fable-review

Skill33 starsChanged 5 months agoMITView on GitHub

Adversarially review a change or artifact before it is locked in — try to break it, find the failure paths, rate severity — instead of confirming it looks fine. Use before merging/shipping/finalizing, or when the user says "review this", "poke holes", "what breaks", "find the weaknesses", "red-team it", "what did I miss". Finds and rates defects; does not rewrite. Pulled on demand; not always-on.

7 other projects use a version of this.

Adopted by

Most recently taken up by

The file

We split this skill into the rules it gives an agent

1 rule. 0 are common, 1 is distinctive, and 0 are very specific to this project.

  • 0

    Common

    Most of the 5 edited versions we read keep these.

  • 1

    Distinctive

    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
  1. Distinctive:

    Before merge / ship / finalize / lock-in of a change or artifact.

    line 20

“Common” is measured: kept by at least 6 in 10 of the 5 edited versions of this file we could read. “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

fable-review — try to break it before it ships, don't just nod at it

A review that sets out to confirm an artifact finds little; a review that sets out to break it finds what matters. This skill is the adversarial pass — for code diffs, specs, migration/implementation plans, prompts, and designs — before a decision is locked in. It rates defects by severity; it does not fix them in place (fixing is a separate, explicit step so the reviewer and the fixer stay independent).

Honest scope note: on enumerable, planted defects a single strong model is already near ceiling, so a second independent reviewer added 0 extra recall there (eval/xverify-value/RESULTS.md). The value of an adversarial pass is on judgment/design misses and untested failure paths, not on re-counting bugs a careful first read already caught.

When to use this

  • Before merge / ship / finalize / lock-in of a change or artifact.
  • The user says "review", "poke holes", "what breaks", "find weaknesses", "red-team it", "what did I miss", "attack this".

When NOT to use this

  • You are being asked to implement or fix, not to assess — do that instead.
  • A throwaway draft with no decision riding on it.

Procedure

  1. Assume it is wrong and find out how. For each part, ask: what input, ordering, failure, or adversarial user makes this break or mislead?
  2. Walk the real failure paths: silent failures and fire-and-forget, race conditions, missing timeouts/retries, injection/encoding bypasses, unhandled error branches, boundary and empty/null cases, permission and trust assumptions, claims in a spec that the implementation cannot guarantee.
  3. Cite evidence for each finding — the line, the input, the sequence that triggers it. No hand-waving.
  4. Rate severity Critical / High / Medium / Low, and say what is needed to close each.
  5. Do not rewrite the artifact. Report the findings; let the fix be a separate, deliberate step. If you find nothing real, say so plainly rather than inventing nits.

Expected output

A severity-rated findings list, each with concrete evidence and a close condition — or an explicit "no real defects found; here is what I checked".

Verification

For each finding, could you actually reproduce or point to it? Drop anything you cannot ground — a plausible-but-unverifiable finding is noise. Confirm you reviewed the high-risk surfaces, not just the easy ones.

Failure cases

  • Confirming "looks good" without trying any breaking input.
  • Listing style nits while missing a real failure path.
  • Quietly rewriting the code instead of reporting the defect (collapses reviewer and fixer into one).

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.