agentleFS
fabioc-alohafabioc-aloha/Alex_ACT_Edition/.github/skills/skill-review/SKILL.md

skill-review

Skill0 starsChanged 4 months agoMITView on GitHub

Audits a candidate skill (.github/skills/<name>/SKILL.md) against five gates (spec compliance, content quality, scope fit, safety, currency & coherence). Use when reviewing a new skill draft before commit, evaluating a Mall unit or store skill for adoption, or re-auditing existing skills on a periodic cadence. For instructions, prompts, agents — use the matching per-type review skill.

25 other projects use a version of this.

Adopted by

Most recently taken up by

Used with

Projects with it also use sigma, skill-forge and 1 more.

Other files people use alongside this one.

Files found in the same public projects as a version of this one, more often than they turn up anywhere else. Each project counts once, large collections count for less, and this file's own repository is left out.
  • sanyuan0704Skill

    sigma

    Personalized 1-on-1 AI tutor using Bloom's 2-Sigma mastery learning. Guides users through any topic with Socratic questioning, adaptive pacing, and rich visual output (HTML dashboards, Excalidraw concept maps, generated images). Use when user wants to learn something, study a topic, understand a concept, requests tutoring, says 'teach me', 'I want to learn', 'explain X to me step by step', 'help me understand', or invokes /sigma. Triggers on: learn, study, teach, tutor, understand, master, explain step by step.

    Also in 3 projects that use this

  • WilliamSaysXSkill

    skill-forge

    Automated skill creation workshop with intelligent source detection, smart path management, and end-to-end workflow automation. This skill should be used when users want to create a new skill or convert external resources (GitHub repositories, online documentation, or local directories) into a skill. Automatically fetches, organizes, and packages skills with proactive cleanup management.

    Also in 3 projects that use this

  • GoogleSkill

    grill-me

    A relentless interview to sharpen a plan or design.

    Also in 3 projects that use this

The file

Files
SKILL.md
<!-- intentional divergence from Supervisor: Edition strips Supervisor-only refs (ADR-007, `scripts/brain-qa.cjs` per-script citations, instruction-specific Gate 1 rows since heirs review skills only, and retired-entity examples Supervisor uses in Gate 5 illustrations). Same five gates, same criteria. Audited 2026-05-31. -->

Skill Review

Audit a candidate skill against five gates. This brain accepts only what passes all five.

For other artifact types: instruction-review, prompt-review, agent-review. All four pairs share the same five-gate contract; per-type criteria differ.

When to Use

Three fire contexts:

  1. Author self-audit — dogfood your own draft before committing. Invoked from skill-creator Phase 7.
  2. External candidate adoption — gate before pulling a Mall unit, store skill, or any external artifact into this brain. Decide whether the artifact is fit before adopting.
  3. Periodic re-audit — re-check existing skills during currency-audit or quarterly retraining. Skills that no longer pass the gates revise or retire.

This skill carries the judgment checks a brain-qa validator cannot do (mechanical regex/date validation vs. content-quality judgment). Gate 1 partially overlaps with brain-qa where present; Gates 2–4 are judgment-only.

The Five Gates

A candidate must pass all five to ship. Failure on any gate = decline or revise.

These gates are the canonical source of truth. skill-creator inverts them into authoring phases — if the two ever disagree, this file wins and skill-creator must follow.

Gate 1 — Spec Compliance

CheckPass criterion
Frontmatter present AND minimalYAML block carries exactly the canonical fields. Skills: name + description + lastReviewed. Reject any legacy extras (type, application, applyTo, inheritance, tier, currency, lifecycle, user-invokable, evidence).
name validkebab-case, ≤64 chars, matches folder name
description validThird-person, ≤1024 chars, names what the skill does AND when to use it (avoid slogans like "Clear documentation through visual excellence")
File location matches typeSKILL.md in skills/<name>/
Markdown lints cleanNo broken links, no missing code-fence languages

This gate overlaps with a brain-qa validator where present. If brain-qa passes, Gate 1 is presumptively met; spot-check the judgment items (description third-person/trigger phrases).

Gate 2 — Content Quality

CheckPass criterion
Single responsibilityThe artifact does one thing; if title contains "and" or "+", split it
Behavioral, not encyclopedicTells the agent what to do, not what a topic is
Has falsifiability or visible markersThe reader can tell whether the artifact fired correctly
≤ 500 linesAnthropic skill-spec ceiling; longer = signal of overload, split or trim
No duplicated content from existing artifactsGrep for overlapping descriptions across .github/instructions/ and .github/skills/
No graveyard proseNo "removed/dropped/used-to-have" sections; the file describes the live shape only

Gate 3 — Scope Fit

CheckPass criterion
Target brain matches scopeGeneric across ≥2 projects → this brain. Project-specific → that project's local skills, not here. External-surface delivery → Plugin Mall, not here.
Not framework-levelDoes not modify ACT manifesto, tenets, or claims registry — framework changes go through an ADR, not a skill
Doesn't duplicate Plugin Mall contentIf the value is a marketplace listing, it goes in the Mall, not the brain baseline

Gate 4 — Safety

CheckPass criterion
No destructive defaultsAnything that deletes, force-pushes, or overwrites must require explicit user approval
No hardcoded credentials or PIIRun pii-memory-filter mentally over the diff
No prompt-injection vectorsIf the artifact reads external content (URLs, files), it sanitizes or quotes it
ReversibleA user can disable or remove the artifact without breaking the brain

Gate 5 — Currency & Coherence

The semantic layer of currency-and-coherence judgment. A mechanical validator (brain-qa script) typically catches the obvious cases (broken links, missing allow markers, stale dates, H1/name divergence); this gate covers the subtler ones that require deep reading.

CheckPass criterion
Frontmatter matches bodyDescription's "what + when" claims survive a deep read of the body. No drift between advertised scope and actual content.
No stale entity references (semantic)References to retired entities carry a per-file <!-- brain-qa: allow <Entity> --> marker AND the marker is justified — the reference adds historical or operational value, isn't fossil.
Cross-references resolve and add valueEvery markdown link (the \[label\]\(target\) form) points to a live artifact AND the target adds something the reader needs. Dead links and decorative xrefs both fail.
lastReviewed is honestThe date reflects when the file was actually re-verified, not a rubber-stamp. Body content must be consistent with what was true at that date.
H1 matches advertised scopeThe H1 reflects the skill's actual scope per name + description. "MCP Server Development Guide" vs name "mcp-builder" fails this gate.
Description has "what" AND "when"Third-person; names the operation AND the trigger phrases. No slogans, no missing trigger clauses.
Body free of slogans / marketing prosePlain operational language. No "powerful", "comprehensive", "seamless", "unleash". No graveyard prose ("removed/dropped/used-to-have" sections).
Related section is liveLinked-to artifacts exist AND each adds material value beyond cross-linking for the sake of it.

Decision Matrix

Gates passedAction
All 5Accept — land the change
4 of 5Revise — name the failing gate and patch the candidate
3 of 5 or fewerDecline — name the rationale; if the decline sets precedent, record it where your heir tracks framework-level decisions

Recording the Verdict

For self-audits and routine re-audits: the verdict lives in the commit message or the conversation. No separate file.

For external adoption (Mall unit, store skill) or any decline that sets precedent: write a verdict capturing gate results, rationale, required changes (if Revise), and the act-pass trail. This is the audit trail for adoption decisions, not a ceremonial form for every audit.

Anti-Patterns

Anti-patternCorrection
Accepting because the author is confidentConfidence ≠ quality. Run all five gates regardless of authorship
Declining without naming the gateAlways cite the specific gate; vague declines waste cycles
Accepting "trivial" candidates without auditTrivial-looking changes are where regressions hide
Skipping the act-pass trail on non-trivial auditsMedium-stakes audits (new artifact, external adoption) require the trimmed pass; routine re-audits of unchanged artifacts do not
Gates drifting from skill-creatorIf a gate here disagrees with a skill-creator phase, update skill-creator — this file is the source of truth

Falsifiability

This skill's five-gate model has failed if any of the following occur within 90 days:

  • An accepted candidate (all 5 gates passed) is reported broken by 2+ heirs
  • Declines cluster on one gate and are later reversed during re-audit (gate too strict or unclear)
  • Repeated audits of equivalent candidates produce contradictory gate outcomes
  • Gate 5 produces a false-positive rate >20% over a quarter (the semantic layer is too aggressive)
  • A defect that Gate 5 was supposed to catch ships through an accepted review

Track these as you would any falsified discipline (commit log, retraining notes, or curation ledger if your repo ships one) tagged [GATE-FAILURE].

Related

  • skill-creator — inverts these gates to author candidates that pass by construction
  • instruction-review — sibling for instructions
  • prompt-review — sibling for prompts
  • agent-review — sibling for agents
  • act-pass — required for medium-stakes audits
  • /review-skill prompt — slash-command entry point

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.