agentleFS
vzhukovsvzhukovs/podman-desktop-kit/skills/quickfix/SKILL.md

quickfix

Skill0 starsChanged 11 days agoApache-2.0View on GitHub

Small fix without the planning ceremony.

The file

We split this skill into the rules it gives an agent

4 rules. About 0 are common, 2 are distinctive, and 2 are very specific to this project.

  • ≈0

    Common · estimate

    Many files like this have a rule of this kind.

  • ≈2

    Distinctive · estimate

    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. Very specific:

    Commit with a conventional subject and one Signed-off-by.

    line 30
  2. Very specific:

    If the fix outgrows them, stop and escalate — pdkit issue escalate <n>.

    line 40
  3. Distinctive:

    Every preflight check. None of them is waived by the route being small.

    line 53
  4. Distinctive:

    No e2e test. It inflates both review and CI time, which contradicts the one thing a small PR is for.

    line 55

≈ “Common” is an estimate: at least 1 in 5 of the 350,495 skill files we read have a rule of the same kind, not the same words. “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

Planning is skipped deliberately: if the diff fits in one sentence, a plan is overhead. Everything else about the discipline is unchanged.

Preconditions

pdkit state <issue> says triaged and the route is quickfix. If triage did not say that, this is the wrong command — go back to /pd:triage.

Steps

  1. pdkit branch create --issue <n> --slug <short-description>, then pdkit state <n> --to quickfix.

    The state move is not bookkeeping. preflight-green is reachable from quickfix and not from triaged, and the push gate is issued from preflight-green only — so an issue left in triaged reaches a green preflight and then cannot open the gate at all. Found on the first live run, after both passes were green.

  2. Make the fix. Add or extend a test for it; "too small to test" is how a reviewer's first comment gets written for them.
  3. Commit with a conventional subject and one Signed-off-by. Squash with git reset --soft <base> && git commit — never rebase -i, which the husky hook rejects for the duplicate sign-off it just created.
  4. pdkit preflight <n> → hand over to /pd:pr.

The thresholds are the point

From quickfix in the config: 20 changed lines, 3 files, and nothing under packages/extension-api/** or matching **/*.schema.*.

If the fix outgrows them, stop and escalate — pdkit issue escalate <n>. That returns the issue to triaged and clears the route, so planning runs properly and R-IDs are allocated from the issue's requirements.

The ordering matters and is not bureaucracy: R-IDs derived from a diff you have already written describe what you built, not what was required, and the whole trace becomes self-confirming. Escalating means re-deriving requirements from the issue, with the code you wrote as evidence rather than as the answer.

What does not change on this route

  • Steps to check in the PR body, at least three, each with an expected result. Size does not make a reviewer better at guessing.
  • Every preflight check. None of them is waived by the route being small.
  • The PR body carries Fixes #<n>, and tracing goes by issue number.
  • No e2e test. It inflates both review and CI time, which contradicts the one thing a small PR is for.

Next

/pd:pr <issue> — the same gate, the same body, the same preflight. What was skipped is planning, not publishing.

In a new session, small as this was. The session that made the fix has read the code around it and the test output, and /pd:pr needs neither: the branch, the commit and preflight.json are on disk. Publishing is where a person reads the push command, and that is better done at the start of a short session than at the end of a longer one. pdkit next <issue> prints the line.

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.