← ClaudeAtlas

mellions-harness-rulelisted

Turn a lesson into a check the repository enforces itself. Use when a defect has just been fixed and the same mistake could be made again — especially when it exists or existed in sibling repositories, or when the reason it was wrong is invisible from the call site.
LetA-Tech/mellions-coxen · ★ 1 · AI & Automation · score 67
Install: claude install-skill LetA-Tech/mellions-coxen
<!-- Mellions Engineer | LetA Tech Ltd. | leta@letatech.ca --> # Turning a lesson into a harness rule A fix repairs one occurrence. A harness rule stops the next one. The difference matters most for defects that are invisible locally. `AcquireDuration()` returns a `time.Duration`; comparing it to a threshold reads correctly at the call site and is wrong only because the value is cumulative. No amount of care at that line prevents it. A check that fails the build does. ## When this is worth doing Ask, after the fix is proven: - Could a competent engineer make this same mistake again next month? - Did the same defect exist in a sibling repository? (It usually did.) - Is the reason it is wrong invisible from the code that does it? Two yeses is enough. One is usually not. **When it is not worth doing:** a one-off in code nobody else touches, a defect whose wrongness is obvious on the line, or anything you would have to over-fit a matcher to catch. A rule that fires on correct code is worse than no rule — it gets suppressed, and then it protects nothing. ## Where the rule goes In the repository that has the defect, not in the engineer. A rule in a skill binds sessions that load it. A rule in the repository binds every future engineer, including the humans, including the ones who never heard of this. Prefer, in order: 1. **A test in the package itself** — no new tooling, runs in the existing suite, and the failure lands next to the code. 2. **A `go vet`-style analysi