CtrlK
BlogDocsLog inGet started
Tessl Logo

fla-mr-readiness

Checklist and workflow for preparing an MR/PR in the FLA repo. Covers CONTRIBUTING.md compliance, test plan, benchmark evidence, and PR body structure.

65

Quality

82%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Passed

No findings from the security scan

SKILL.md
Quality
Evals
Security

Quality

Content

88%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

A lean, highly actionable checklist skill: exact commands, a copy-paste PR body template, explicit CI-enforcement rules, and a genuine flaky-test feedback loop, with essentially no filler. The only real gaps are minor: slight redundancy between the CONTRIBUTING.md steps, and loosely-signaled delegation to the sibling performance skill and repo template where a clearer pointer or a split-out reference file would help.

DimensionReasoningScore

Conciseness

The body is dense with repo-specific operational detail Claude would not know (CI-enforced checklist semantics, wrapper names, REST-API-vs-`gh pr edit`), with essentially no explanation of known concepts. Minor trimming opportunities exist — step 1 and step 7 both route to `CONTRIBUTING.md`, and step 6's 'never trim it' note overlaps the later `check-pr-title` enforcement section — matching 'Efficient; minor instances of over-explanation that could be trimmed' rather than the perfectly lean 5.

4 / 5

Actionability

Everything is executable: exact commands (`gh pr list --repo ... --search "<keywords>"`, `python scripts/find_dependent_tests.py <changed_file_or_dir>`), a copy-paste-ready PR body template with placeholders, named `fla.utils` wrappers, and the precise N/A/tick rules. This matches 'Fully executable; copy-paste ready code or commands; specific examples cover the common cases.'

5 / 5

Workflow Clarity

The seven-step pre-flight checklist is clearly sequenced with explicit validation ('Run those tests locally and ensure they pass') and a feedback loop for error recovery ('If a test is flaky, retry once; if it still fails, explain why in the PR'), plus a checklist template and CI enforcement as final checkpoints. This matches the 5 anchor's 'clear sequence with explicit validation steps; feedback loops for error recovery; checklists.'

5 / 5

Progressive Disclosure

No bundle files exist (no references/, scripts/, or assets/), and the skill is ~113 lines. Sections are well organized and heavy detail is correctly offloaded — kernel performance evidence is delegated to the 'fla-nvidia-performance' skill, and the template points to `.github/pull_request_template.md` as source of truth — but that delegation is only loosely signaled (no link/path for the sibling skill) and the full PR template is inlined rather than split out. This fits 'Good structure; most content is appropriately placed; references mostly clear; minor organization gaps' rather than the cleanly one-level-deep, well-signaled references of a 5.

4 / 5

Total

18

/

20

Passed

Description

70%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

A specific, well-scoped, third-person description that names the domain and four concrete coverage areas with good natural keywords. Its main weakness is the absence of an explicit 'Use when...' trigger clause, which leaves the timing of use implied rather than stated. Voice is correctly third person, so no specificity penalty applies.

Suggestions

Append an explicit trigger clause, e.g. 'Use when preparing to open a pull request or merge request in the FLA repo, or when the user mentions MR/PR readiness, PR review feedback, or benchmark requirements.'

Spell out 'pull request' at least once alongside 'MR/PR' so users who say either phrase match naturally.

Optionally mention the pre-flight checks that distinguish this from a generic PR guide (duplicate-work search, dependent-tests discovery) to sharpen distinctiveness and coverage.

DimensionReasoningScore

Specificity

The description lists several concrete coverage areas — 'CONTRIBUTING.md compliance, test plan, benchmark evidence, and PR body structure' — which are specific and actionable, though phrased as topic nouns rather than actions, with minor gaps (e.g. duplicate-work search, code style). This matches the anchor 'Lists several specific actions; minor gaps in coverage'; it is below 5 because the coverage list is not fully comprehensive and above 3 because it goes well beyond 1-2 actions.

4 / 5

Completeness

The 'what' is clear ('Checklist and workflow for preparing an MR/PR in the FLA repo' with enumerated coverage areas), but there is no 'Use when...' clause or equivalent explicit trigger guidance — timing is only weakly implied by 'preparing'. Per the judging guidelines, a missing 'Use when...' clause caps completeness at 3; it is not a 4 because 'when' is never stated explicitly.

3 / 5

Trigger Term Quality

Natural terms include 'MR/PR', 'FLA repo', 'test plan', 'benchmark evidence', and 'PR body' — phrases a contributor would plausibly say. It sits at 'Good keyword coverage; a few natural terms missing' (no spelled-out 'pull request', 'open a PR', or 'pre-submit review' variants) rather than the comprehensive synonym coverage of a 5, and well above the sparse keyword sets of a 3.

4 / 5

Distinctiveness Conflict Risk

'Preparing an MR/PR in the FLA repo' carves out a clear niche with distinct, repo-specific triggers (FLA, MR/PR, benchmark evidence), with minimal conflict risk against generic skills. It clearly matches 'Clear niche with distinct triggers; minimal conflict risk' — the neighbor anchor 4 would require noted overlap with closely related skills, which is not the case.

5 / 5

Total

16

/

20

Passed

Validation

87%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 14 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

relative_links

Relative link issues: 3 suspicious

Warning

referenced_paths_exist

Referenced path issues: 1 missing

Warning

Total

14

/

16

Passed

Repository
fla-org/flash-linear-attention
Reviewed

Table of Contents

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.