swe-workflow-skills 0.6.3 → 0.7.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/VERSION +1 -1
- package/catalog.json +2 -2
- package/package.json +1 -1
- package/skills/code-reviewing/SKILL.md +24 -6
- package/skills/code-reviewing/evals/evals.json +19 -3
- package/skills/code-slop-cleanup/SKILL.md +7 -0
- package/skills/code-slop-cleanup/evals/evals.json +13 -0
- package/skills/code-slop-cleanup/references/slop-patterns.md +4 -1
- package/skills/git-workflow/SKILL.md +119 -8
- package/skills/git-workflow/evals/evals.json +37 -5
- package/skills/git-workflow/templates/pull-request.md +10 -5
- package/skills/plan-execution/SKILL.md +38 -3
- package/skills/plan-execution/evals/evals.json +30 -0
- package/skills/project-documentation/SKILL.md +73 -6
- package/skills/project-documentation/evals/evals.json +17 -2
- package/skills/release-management/SKILL.md +1 -1
- package/skills/verification-before-completion/SKILL.md +109 -2
- package/skills/verification-before-completion/evals/evals.json +64 -3
- package/skills/verification-before-completion/references/pre-commit-gate.md +239 -0
- package/skills/writing-skills/SKILL.md +33 -0
|
@@ -34,6 +34,15 @@ tends to over-explain things Claude already knows and miss the real failure mode
|
|
|
34
34
|
3. **REFACTOR — re-test, capture *new* rationalizations, add counters.** Repeat
|
|
35
35
|
until the behavior holds under pressure. Stop when two consecutive runs pass.
|
|
36
36
|
|
|
37
|
+
**When it doesn't pass, stop by rule, not by patience.** Before each rewrite, name
|
|
38
|
+
the cause you believe is responsible and change *only* that. If the number doesn't
|
|
39
|
+
move, the hypothesis is dead — do not reword the same section a third time. Either
|
|
40
|
+
the assertion is unsatisfiable in this harness (see EVALS.md limitations 3 and 4:
|
|
41
|
+
reference-dependent and tool-dependent), or the behavior is a real cost of the
|
|
42
|
+
skill and belongs in the test's `expected_behavior` as an accepted trade with its
|
|
43
|
+
mechanism recorded. Both endings are results. Endless rewording is how a skill
|
|
44
|
+
grows to satisfy an instrument instead of a user.
|
|
45
|
+
|
|
37
46
|
### Match the form to the failure
|
|
38
47
|
|
|
39
48
|
| Baseline failure | Right form in the skill |
|
|
@@ -42,6 +51,30 @@ tends to over-explain things Claude already knows and miss the real failure mode
|
|
|
42
51
|
| Output has the wrong shape | A positive recipe / template stating what the output *is* |
|
|
43
52
|
| Omits a required element | A structural requirement (named field or slot) |
|
|
44
53
|
| Behavior should depend on a condition | A conditional rule ("if X, do Y"), not a blanket rule + exceptions |
|
|
54
|
+
| Skill defers instead of delivering | The grounding instruction needs a **delivery clause** (below) |
|
|
55
|
+
|
|
56
|
+
### Grounding instructions need a delivery clause
|
|
57
|
+
|
|
58
|
+
Any instruction of the form *"read/inventory/verify X before you answer"* degrades
|
|
59
|
+
into *"ask permission to read X, deliver later"* unless you say otherwise. This is
|
|
60
|
+
the most-repeated authoring failure in this library — five occurrences across two
|
|
61
|
+
cycles, in `architecture-design`/`data-modeling` (Step 0's first draft),
|
|
62
|
+
`project-documentation` (twice), `verification-before-completion`, and
|
|
63
|
+
`git-workflow` — and the base model, having no such instruction, often scores
|
|
64
|
+
*better* because it just answers.
|
|
65
|
+
|
|
66
|
+
Every grounding step must carry three things, and they are cheap:
|
|
67
|
+
|
|
68
|
+
1. **"This is not a gate on delivering."** Say it in those words.
|
|
69
|
+
2. **An exit condition** — the literal one-line sentence that closes the step, so
|
|
70
|
+
the model has something concrete to emit instead of a question.
|
|
71
|
+
3. **"…in the same response."** Plus the fallback: when the artifact can't be read,
|
|
72
|
+
state the assumption and deliver anyway. A draft the user corrects beats a
|
|
73
|
+
question they must answer.
|
|
74
|
+
|
|
75
|
+
`architecture-design` Step 0 is the reference implementation — copy its shape rather
|
|
76
|
+
than reinventing it. The tell in an eval is a GREEN reply that contains no artifact:
|
|
77
|
+
no message, no document, no recommendation, just a plan to produce one.
|
|
45
78
|
|
|
46
79
|
## Description discipline (the highest-leverage line)
|
|
47
80
|
|