agentleFS
maroffomaroffo/claude-forge/skills/score/SKILL.md

score

Skill17 starsChanged 50 days agoMITView on GitHub

Run the local quality gates (`make check` + `make test-e2e`) and report commit/PR/excellence readiness per `rules/quality-gates.md`. Use when user says /score, is this ready to commit, is this ready for a PR, or score my changes.

15 other projects use a version of this.

Adopted by

Most recently taken up by

Used with

Projects with it also use audit.

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.
  • OpenAISkill

    audit

    Audit or critique a product flow, journey, workflow, funnel, onboarding path, checkout path, settings path, screen, or multi-step product experience by capturing screenshots first, then reporting UX, design, and accessibility findings inline from that evidence. Use Figma only when the user explicitly asks for a board. Use when the user asks to audit, review, critique, inspect, assess, analyze, evaluate, or give feedback on a product experience.

    Also in 3 projects that use this

The file

We split this skill into the rules it gives an agent

7 rules. 0 are common, 5 are distinctive, and 2 are very specific to this project.

  • 0

    Common

    Most of the 13 edited versions we read keep these.

  • 5

    Distinctive

    This author's own choices. Worth a look.

  • 2

    Very specific

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

  • Common
  • Distinctive
  • Very specific
  1. Distinctive:

    Fail = 0: if either target fails, score is 0 (auto-fail per quality-gates: tests/build broken is CRITICAL).

    line 17
  2. Very specific:

    Pass = 100 baseline: if both pass, the static gate is green.

    line 18
  3. Distinctive:

    After the report, append the run and show where it sits against the previous ones:

    line 46
  4. Very specific:

    Missing gate = not zero: If make check or make test-e2e targets don't exist in the project, do NOT claim score 100.

    line 65
  5. Distinctive:

    Review integration is optional: If the user didn't ask for a full review, report the static gate result only and note that a rubric-based …

    line 66
  6. Distinctive:

    Before git commit: quick readiness check.

    line 70
  7. Distinctive:

    Before opening a PR: confirm ≥90 threshold.

    line 71

“Common” is measured: kept by at least 6 in 10 of the 13 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

ABOUTME: Quality gate scoring: runs make check + make test-e2e and grades against quality-gates

ABOUTME: Reports commit-ready (>=80), PR-ready (>=90), excellence (>=95) with honest auto-fails

/score

Compute a quick quality readiness signal for the current working tree.

Process

  1. Run make check && make test-e2e.
  2. Fail = 0: if either target fails, score is 0 (auto-fail per quality-gates: tests/build broken is CRITICAL).
  3. Pass = 100 baseline: if both pass, the static gate is green. For a full rubric-based score (Major/Minor deductions), invoke /gemini-review or /advanced-review and subtract:
    • 10 per Major finding
    • 3 per Minor finding
  4. Report against the three thresholds from rules/quality-gates.md:
    • ≥80 commit-ready
    • ≥90 PR-ready
    • ≥95 excellence
  5. Log the run, then render the trend (see History below). Never hand-write the row or the delta: the script computes both.

Output format

Score: <n>/100
Gate:  <commit-ready | PR-ready | excellence | BLOCKED>

Breakdown:
  make check:     <PASS|FAIL>
  make test-e2e:  <PASS|FAIL>
  Major findings: <n> (-<10n>)
  Minor findings: <n> (-<3n>)

Ready to commit:  <yes|no>
Ready to open PR: <yes|no>
Excellence:       <yes|no>

History

After the report, append the run and show where it sits against the previous ones:

SCORE_LOG="${CLAUDE_FORGE_ROOT:-$HOME/Development/private/claude-forge}/scripts/score-log.sh"
"$SCORE_LOG" --score <n> --threshold <t> --gate <commit|pr|excellence> --check <pass|fail> --e2e <pass|fail> --major <n> --minor <n>
"$SCORE_LOG" --trend

Paste the --trend output verbatim under the report. Values come from the breakdown just computed; --threshold and --gate are the pair the canonical trace line prints, SCORE: <n>/100 (threshold: <t>, gate: commit|pr|excellence) (rules/orchestrator-protocol.md): the action this run was aiming at and the number it is judged against, not the highest bar the score happened to clear. Log the same pair you print, or the row cannot be reconciled with its SCORE event.

  • The script does the append and the arithmetic. The model supplies the seven measured values and nothing else: a hand-written row or a hand-computed delta is exactly the failure this indirection removes.
  • The history file (quality_reports/score-history.jsonl at the target repo's git root) is a denormalized view of harness-trace SCORE events; change one, change the other.
  • The script gitignores the file in the target repo on first write. It stays local: a branch name next to a low score corroborates gitignored review findings.
  • Script missing or the run fails: report the score anyway and say the history was not written. A logging failure never blocks the gate.

Rules

  • Ambiguous output is inconclusive. Report it as such and say why; the score reflects only what the gates showed.
  • CRITICAL trumps all. A failing make check/test-e2e yields score 0 regardless of other factors.
  • Missing gate = not zero. If make check or make test-e2e targets don't exist in the project, do NOT claim score 100. Report "gate missing, run /project-checks to scaffold" and refuse to score.
  • Review integration is optional. If the user didn't ask for a full review, report the static gate result only and note that a rubric-based score requires a review agent pass.

When to use

  • Before git commit: quick readiness check.
  • Before opening a PR: confirm ≥90 threshold.
  • Mid-session checkpoint: answer "are we done?" with a number instead of vibes.

When NOT to use

  • Pre-commit already running as a hook: the hook gives a go/no-go; /score gives you a number and a breakdown. Different use.
  • Large refactors where full review is warranted: go straight to /advanced-review, not /score.

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.