agentleFS
vadimvitvickiyvadimvitvickiy/vv-agent-kit/skills/writing-skills/SKILL.md

writing-skills

Skill0 starsChanged 52 days agoMITView on GitHub

Use when authoring or editing a skill, agent, command or hook for this kit, or when deciding whether a piece of guidance belongs in a skill at all.

191 other projects use a version of this.

Adopted by

Most recently taken up by

Used with

Projects with it also use chinese-code-review, chinese-commit-conventions and 10 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.
  • Leodorareluctant259Skill

    chinese-code-review

    中文代码审查规范——在保持专业严谨的同时,用符合国内团队文化的方式给出有效反馈

    Also in 8 projects that use this

  • Leodorareluctant259Skill

    chinese-commit-conventions

    中文 Git 提交规范 — 适配国内团队的 commit message 规范和 changelog 自动化

    Also in 8 projects that use this

  • obraSkill

    diagnosing-superpowers

    Use when a superpowers session went wrong and your human partner wants to know why — repeated work, ignored plans, stumbles, poor results, a skill that didn't fire, "it took too long", "why is it so expensive", "what is it doing" — or wants to build a bug report for the superpowers maintainers, for the current session or a past one identified by id or path, on any harness.

    Also in 20 projects that use this

  • kntismSkill

    condition-based-waiting

    Use when tests have race conditions, timing dependencies, or inconsistent pass/fail behavior - replaces arbitrary timeouts with condition polling to wait for actual state changes, eliminating flaky tests from timing guesses

    Also in 3 projects that use this

  • aaddrickSkill

    writing-agents

    Use when creating new agents, editing existing agents, or defining specialized subagent roles for the Task tool

    Also in 3 projects that use this

  • frankxaiSkill

    defense-in-depth

    Use when invalid data causes failures deep in execution, requiring validation at multiple system layers - validates at every layer data passes through to make bugs structurally impossible

    Also in 3 projects that use this

  • +3 more from Leodorareluctant259

The file

We split this skill into the rules it gives an agent

5 rules. 0 are common, 4 are distinctive, and 1 is very specific to this project.

  • 0

    Common

    Most of the 180 edited versions we read keep these.

  • 4

    Distinctive

    This author's own choices. Worth a look.

  • 1

    Very specific

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

  • Common
  • Distinctive
  • Very specific
  1. Very specific:

    Only name and description are preloaded — in every session, in every repo.

    line 26
  2. Distinctive:

    After compaction only the first ~5,000 tokens of each recent skill are re-attached.

    line 33
  3. Distinctive:

    Keep every measured number and named failure — that is what changes behavior.

    line 44
  4. Distinctive:

    Use the words someone would actually use, including the *situation* — "a test fails", "behavior does not match expectation".

    line 67
  5. Distinctive:

    Every assertion has a measured number or a named failure behind it.

    line 83

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

Writing skills

A defect here is inherited silently by every project that installs the plugin. Nothing checks a skill against a compiler; the care taken writing it is the only guard.

Does it belong in a skill?

The guidance isIt belongs in
True in any repo, needed only for some tasksa skill
True in any repo, needed on literally every turnhooks/using-vvkit.md, injected at session start — a plugin's own CLAUDE.md is never loaded
Specific to one repotemplates/, or a stack plugin's templates/ and rules/, scaffolded in and owned by that repo
A rule that must hold whether or not the model cooperatesa hook
A task with its own context window and a narrow tool setan agent

Does this text stay true in another repo? A target or scheme name, a script path, a module layout, a ticket prefix, a branch rule, a CI job name — each means you are writing a template.

The cost model

  • Only name and description are preloaded — in every session, in every repo. The body is free until invoked.
  • The listing has a budget (~1% of context). On overflow, descriptions are dropped from the least invoked skills first, and a skill with no description stops triggering. A marginal skill taxes the ones that earn their place. Ship fewer, better.
  • The body enters once and stays for the session, unread again. Write standing instructions, not a one-shot procedure.
  • After compaction only the first ~5,000 tokens of each recent skill are re-attached. Load-bearing rule near the top.
  • Every rule competes with every other. In IFScale (Jaroslawicz et al., 2025, 20 models) the best reached 68% at 500 instructions, biased toward the first. Merge rules that say the same thing; when one goes in, look for one to take out.

Body style

  • H1, then the load-bearing rule in bold, one or two sentences.
  • Sections of short bullets. Each bullet is the rule; the reason follows in a clause only where it is not obvious. No narrative paragraphs, no recap at the end.
  • Keep every measured number and named failure — that is what changes behavior. Cut the prose around it.
  • Tables for decisions (situation → action); checklists for multi-step gates; code blocks only for commands and shapes the agent copies.

The description contract

Third person. Starts with Use when. Triggering conditions only.

  • A description that summarizes the procedure gets substituted for the body: the agent believes it has the rule and never opens the file. The superpowers plugin reports a description saying "code review between tasks" producing one review where the body required two.
  • Anthropic's examples pair what with when; this kit keeps triggers only, and its skills still fired first in every run measured.
# No — narrates the workflow, so it becomes the workflow
description: Reviews the diff, dispatches a reviewer per language, then reports findings by severity.

# Yes — states only when to reach for it
description: Use when reading a diff or pull request for defects, or when acting as the reviewer on a change you did not write.
  • Use the words someone would actually use, including the situation — "a test fails", "behavior does not match expectation". Undertriggering is the common failure.
  • Primary case first: entries are capped, and the tail is cut.

Naming

  • Neutral skills: verb-first gerunds — writing-tests. Stack skills: stack prefix — swift-testing, in plugins/<stack>/skills/, invoked as vvkit-swift:swift-testing. Plugins install per project; a single plugin skill cannot be hidden.
  • name equals the directory. A mismatch makes a cross-reference that looks right and resolves to nothing.
  • A stack skill opens with **REQUIRED BACKGROUND:** vvkit:<neutral-skill> and does not restate it. Restated rules drift until two contradict in front of a user.

Earning a claim

  • Every assertion has a measured number or a named failure behind it. "Prefer small functions" is already believed; "200 owners produced 209 initialisations and 201 distinct objects" changes the next action.
  • Can't say how you know it: it is filler, and filler makes agents skim. Say where a number comes from, and whether measured or reported.
  • Plain emphasis. Anthropic advises dialing back "CRITICAL" and "MUST"; current models overreact.

Splitting into references/

  • Move out material over ~100 lines, or needed by a minority of invocations — a field table, an API surface, a long example.
  • Link by plain relative path, never @, which loads it whenever the skill loads.
  • A reference over 100 lines opens with a table of contents; references stay one level deep. Agents preview with head. The validator checks the contents rule.
  • Name the interpreter for bundled scripts — bash scripts/tool.sh. Packagers can strip the executable bit.
  • Frontmatter fields and the validator's rules: references/frontmatter.md.

Test the behavior, not only the structure

The validator proves well-formed, not effective. A skill that changes nothing is pure listing cost.

  • claude -p "<task>" --plugin-dir <repo> --output-format stream-json --verbose loads an unpushed tree and logs every Skill call. claude plugin eval runs a checked-in suite with a no-plugin baseline.
  • Before writing: give a fresh subagent the tempting task without the skill; record what it does and its reasons, verbatim. Already behaves: stop. Otherwise target those reasons, re-run with the skill, repeat until it complies.
  • Five runs per variant, minimum. Wide disagreement means the wording does not bind yet.
  • Pressure-test discipline skills with three or more of: time, sunk cost, authority, exhaustion, "be pragmatic". Concrete paths, a forced choice, "what do you do?", no "I would ask" exit. Test techniques on a new case, references by whether the right entry is found.
  • Close each rationalization where it appears: a negation in the rule, an excuse/reality table row, a red-flag thought. Re-run; one closed loophole often opens the next.
  • Match form to failure. Broken under pressure: a prohibition plus the rationalizations it answers. Wrong output shape: a recipe — parts in order; prohibitions on shape get negotiated (superpowers wording tests, reported). A three-line report recipe in writing-tests took naming the caught bug from 0 of 3 runs to 3 of 3. Multi-step workflow: a checklist the agent copies and checks off.

Before shipping

  • ./scripts/validate.sh exits 0
  • ./scripts/validate.sh tests/fixtures still exits 1 — a validator that passes everything is worse than none
  • claude plugin validate . exits 0
  • The version in plugin.json is bumped. Installs are cached per version; an unbumped push reports success and changes nothing, exactly like a skill that failed to load
  • claude plugin marketplace update vv-agent-kit && claude plugin details vvkit shows the new component
  • Every claim in the body is one you could source if asked
  • A fresh subagent showed the failure without the skill, and not with it

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.