From 97
Designs and reviews public APIs, exported function signatures, module boundaries, types/interfaces, and code contracts to make correct usage easy and incorrect usage hard.
npx claudepluginhub oribarilan/97 --plugin 97This skill uses the workspace's default tool permissions.
The headline rule, from Scott Meyers (97/55), governs everything else: **make interfaces easy to use correctly and hard to use incorrectly.** Every other decision below is a tactic for that rule — encapsulate behavior so callers can't reach past the contract, lean on the type system so wrong calls fail at compile time.
Guides stable API and interface design for REST/GraphQL endpoints, module boundaries, component props, and public contracts between modules or teams.
Generates 3-4 radically different module or API contract designs in parallel under divergent constraints, presents sequentially with examples, and compares on depth, simplicity, ease-of-correct-use. For new interface shaping or 'design it twice'.
Designs stable, hard-to-misuse interfaces for REST endpoints, MCP tool schemas, module boundaries, and type contracts. Use when defining new or evolving public surfaces.
Share bugs, ideas, or general feedback.
The headline rule, from Scott Meyers (97/55), governs everything else: make interfaces easy to use correctly and hard to use incorrectly. Every other decision below is a tactic for that rule — encapsulate behavior so callers can't reach past the contract, lean on the type system so wrong calls fail at compile time.
This is a rigid skill. Run the decisions in order. If you can't satisfy one, stop and tell the user what's blocking you.
Invoke when you're about to:
public, internal, final, sealed, or privateIf you're not sure whether a change is "public," ask: will any code outside this file depend on the shape of what I'm about to write? If yes, invoke.
Run every decision in order. Decision 1 is the headline; the rest are how you satisfy it.
Url instead of String, take a non-empty list type instead of List plus a runtime check. Where free input is unavoidable, parse leniently and report specifically. Defaults should reflect the common case. Most caller mistakes are systematic — the API drew them in — not user incompetence.Order.addItem needs a credit check, the credit limit and the check belong on Customer, and Order asks Customer. Anti-pattern: an OrderManager / OrderService that holds all the logic while Order, Customer, and Item are records. When state and the behavior that depends on it live together, callers can't get the sequence wrong. And encapsulate one coherent behavior — a class with fourteen methods of which any caller uses two has the dual problem: every caller depends on a surface they don't all need. The exported surface should have one reason for callers to depend on it. (Martin, 97/76 — SRP at the boundary.)ship(weight: Kilograms) and ship(thrust: Newtons) cannot be confused at a call site; ship(weight: double) and ship(thrust: double) can — and the Mars Climate Orbiter is the canonical example of how that ends. In statically typed languages this becomes a compile-time guarantee; in dynamic ones, a small wrapper class plus a unit test gives you the same readability and the same encapsulation point for domain rules.Order can be in InProgress, Paid, or Shipped, then addItem is only legal in one state and ship is only legal in another. Pretending the state doesn't exist (one flat class with a pile of booleans) leads to nonsense like "shipped before paid" being representable. Either split into state types, or check the current state at the start of every operation that depends on it. Method signatures should reflect what's actually callable.parser.processNodes(text, false) is meaningless at the call site — the reader must consult docs to learn what false means. A boolean or enum flag whose value flips the meaning of the operation is two operations wearing one name. Split it: give callers two well-named methods, or a small composable vocabulary they can combine in ways you didn't anticipate. The "convenience" of one method with a switch is convenience for the implementer, not the caller.final / sealed / singleton / static may protect your future implementation choices, but it makes callers' code untestable — and your library will be replaced. Treat testability as a design constraint.if/switch on type tags. (Pepperdine, 97/59.) When the caller has to choose behavior by inspecting an enum or type code (if (item.kind == DOWNLOADABLE) shipByEmail(...) else shipBySurface(...)), the API has handed responsibility for a closed set of cases back to every caller. A polymorphic interface (item.ship(shipper)) puts the choice inside the type that already knows the answer. Count if/switch statements that branch on type — that's roughly your count of missed polymorphism opportunities. Sometimes a conditional is genuinely simpler; default to polymorphism and justify the conditional.These thoughts mean STOP — restart the decisions:
| Thought | Reality |
|---|---|
| "I'll document the right way to call it." | If the docs have to warn callers, the interface is wrong. Change the signature so the wrong call won't compile or won't typecheck. (97/55) |
"I'll add a bool strict parameter — easier than two methods." | A flag that flips the meaning of an operation is two operations wearing one name. Split it; give the caller real vocabulary. (97/19) |
| "I'll expose the field with a getter and setter — callers know what to do." | You've pushed the business rule into every caller. Encapsulate the behavior on the type that owns the state. (97/32) |
| "These two call sites do the same four lines — extract a shared helper." | Same shape today, different pressures tomorrow. Localize until a real shared concept emerges and earns a name. (97/7) |
"It takes a string — callers can pass whatever." | Strings and floats are an invitation to pass the wrong thing. A named type closes the door on the Mars-Orbiter class of bugs. (97/65) |
| "The state is implicit — callers will know the order to call methods." | Implicit state means callers can call ship before pay. Make the state a type or guard every operation that depends on it. (97/84) |
"I'll mark everything final / sealed to keep my options open." | Locked-down APIs are untestable for the code that uses them. Write a test of a caller before you decide what to seal. (97/35) |
"Callers can if on the type tag — it's only three cases." | Three cases become thirty, scattered across every caller. Move the choice inside the type with polymorphism. (97/59) |
| "If they pass bad input, they'll see a clear error message." | Errors are a sign of broken communication, not a feature. Eliminate the error condition or accept the common formats. (97/66) |
| "It's an internal API — the rules don't apply." | Internal today is exposed tomorrow, and the wrong-use bugs accumulate either way. The rules apply. (97/55) |
| "This class is the right home for it — it's already imported here." | Wedging a second concern onto a class that's already imported gives every caller a surface they don't all need. The exported surface should have one reason for callers to depend on it; split it. (97/76) |
| "I'll add another thin method that just forwards to an internal." | Shallow modules pay no abstraction tax. Make the module deep — fewer, more-powerful methods that hide implementation, not more thin pass-throughs. (Ousterhout/DeepModules) |
"I'll throw NotFound if the file doesn't exist on delete." | Define the error out of existence: idempotent delete, clamping substring, Option<T> lookup. Fewer error paths the caller has to remember. (Ousterhout/DefineErrorsOutOfExistence) |
| "I'll subclass and override the method to throw — callers shouldn't use it." | Subtype must be substitutable. If the override breaks caller assumptions, the hierarchy is wrong. Prefer composition. (Liskov/LSP) |
"I'll validate the input and pass the raw string downstream." | Parse, don't validate. Return a parsed domain type from the boundary; downstream code receives the proven shape, not the raw primitive. (King/ParseDontValidate) |
| "Callers will only use the documented behavior — internals can change freely." | Hyrum's Law: any observable behavior will be depended on by someone. Reason about the new API as if its current observable behavior were private. (Hyrum/Law) |
You are done when all of the following are true:
If any box is unchecked, you are not done. Either finish, or revert and re-plan.
| # | Principle | Author |
|---|---|---|
| 97/7 | Beware the Share | Udi Dahan |
| 97/19 | Convenience Is not an -ility | Gregor Hohpe |
| 97/32 | Encapsulate Behavior, Not Just State | Einar Landre |
| 97/35 | The Golden Rule of API Design | Michael Feathers |
| 97/55 | Make Interfaces Easy to Use Correctly and Hard to Use Incorrectly | Scott Meyers |
| 97/59 | Missing Opportunities for Polymorphism | Kirk Pepperdine |
| 97/65 | Prefer Domain-Specific Types to Primitive Types | Einar Landre |
| 97/66 | Prevent Errors | Giles Colborne |
| 97/76 | The Single Responsibility Principle (at the boundary) | Robert C. Martin |
| 97/84 | Thinking in States | Niclas Nilsson |
Ousterhout/DeepModules | Deep Modules | John Ousterhout |
Ousterhout/DefineErrorsOutOfExistence | Define Errors Out of Existence | John Ousterhout |
Liskov/LSP | Liskov Substitution Principle | Barbara Liskov |
King/ParseDontValidate | Parse, Don't Validate | Alexis King |
See principles.md for the long-form distillations, citations, and source links.