
galahad
by lambdamechanic
SKILL.md
name: galahad description: how to approach tests, types and coverage
Coding Agent Quality Rules (Galahad Principle)
Based on Jonathan Lange’s “The Galahad Principle”: https://jml.io/galahad-principle/
Core idea: getting to 100% yields disproportionate value—especially simplicity and trust. When checks are truly “all green”, any new failure is a strong, unambiguous signal; “absence of evidence becomes evidence of absence”.
Non-negotiables: never evade feedback
Treat type errors, test failures, pre-commit hooks, lint errors, and coverage warnings as helpful feedback. Fix root causes.
Absolutely forbidden (unless the user explicitly orders it)
- Type escapes / silencing
any, sketchyunknownlaundering, unchecked casts,as any,@ts-ignore,# type: ignore,noqa, disabling strict mode, weakening compiler flags, etc.
- Coverage gaming
- Ignoring/excluding lines/branches/files just to hit targets (
/* istanbul ignore */,# pragma: no cover, “generated” tricks, config exclusions, decorator/macro suppression).
- Ignoring/excluding lines/branches/files just to hit targets (
- Faking results
- Skipping CI steps and claiming success; “snapshotting” coverage; lowering thresholds; marking tests flaky to ignore them.
Priorities
Type safety is part of correctness and outranks tests.
When tradeoffs exist, prioritize in this order:
- Type safety / soundness
- Correctness + meaningful tests
- Clarity / maintainability
- Performance
- Backwards compatibility (lowest)
Breaking changes are acceptable when they improve verifiability and simplify the system.
Default workflow (when anything fails)
- Read the failure output carefully.
- Restate the real invariant being violated in plain English.
- Fix the root cause (not the symptom).
- Improve tests so the behavior is pinned and regressions get caught.
- Refactor production code if needed to make it easy to type-check and validate.
Run checks in this order
- Typecheck
- Unit tests
- Integration tests
- Lint / pre-commit
- Coverage
Goal: a repo where “all green” is normal, and any new red is a loud, trustworthy signal.
“Hard to test” means refactor
If something is hard to test or hard to type:
- Treat it as a design smell.
- Refactor towards:
- smaller pure functions
- explicit data flow, minimal global state
- clear boundaries between logic and side effects
- typed domain models over stringly-typed blobs
Mocks: don’t overuse them
Avoid injecting mocks via monkeypatching or replacing system utilities by default.
Preferred approach:
- Make the function under test able to operate in multiple environments by passing in the substitutable operations explicitly (usually as function parameters or small interfaces).
- Only do this for operations that genuinely need substitution in tests (time, randomness, network, filesystem, process execution, etc.).
- This makes the injection point explicit, documents what varies, and keeps tests honest without fragile mocking.
What “good” looks like
- Types encode invariants; no “trust me” casts.
- Tests assert observable behavior (not implementation trivia).
- Coverage comes from exercising real behavior, not exclusions.
- If a thing can’t be verified cleanly, refactor until it can.
Score
Total Score
Based on repository quality metrics
SKILL.mdファイルが含まれている
ライセンスが設定されている
100文字以上の説明がある
GitHub Stars 100以上
3ヶ月以内に更新がある
10回以上フォークされている
オープンIssueが50未満
プログラミング言語が設定されている
1つ以上のタグが設定されている
Reviews
Reviews coming soon