Agent skill
architecture
Guides the design of safely disposable code through contracts (traits/interfaces) and dependency inversion. Use when designing new modules, refactoring existing code, or making architectural decisions about component boundaries.
Install this agent skill to your Project
npx add-skill https://github.com/ktnyt/cclsp/tree/main/.claude/skills/architecture
SKILL.md
Safely Disposable Code via Contracts
Every implementation should be disposable. The system's correctness is defined by its contracts, not by any particular implementation behind them.
Core Principle
Define what a component does (contract), not how it does it (implementation). Any implementation that satisfies the contract is interchangeable.
A component is "safely disposable" when:
- Its behavior is fully described by a contract (trait, interface, protocol)
- No consumer depends on implementation details
- It can be deleted and rewritten from the contract alone without breaking the system
Workflow
Step 1: Define the Contract
Start with the contract. Write the trait/interface before any implementation.
Rust:
pub trait UserRepository {
fn find_by_id(&self, id: UserId) -> Result<Option<User>, RepoError>;
fn save(&self, user: &User) -> Result<(), RepoError>;
}
Go:
type UserRepository interface {
FindByID(ctx context.Context, id UserID) (*User, error)
Save(ctx context.Context, user *User) error
}
TypeScript:
interface UserRepository {
findById(id: UserId): Promise<User | null>;
save(user: User): Promise<void>;
}
Rules:
- Keep contracts small (1-5 methods)
- Name contracts after the role, not the implementation (e.g.,
UserRepository, notPostgresUserStore) - Define contracts where they are consumed, not where they are implemented
- Use domain types in signatures, not infrastructure types
Step 2: Define Error Contracts
Errors are part of the contract. Define domain-level error types that hide infrastructure details.
#[derive(Debug, thiserror::Error)]
pub enum RepoError {
#[error("entity not found: {0}")]
NotFound(String),
#[error("conflict: {0}")]
Conflict(String),
#[error("internal error")]
Internal(#[source] Box<dyn std::error::Error + Send + Sync>),
}
The Internal variant wraps infrastructure errors without leaking them into the contract.
Step 3: Implement Against the Contract
Each implementation is a disposable artifact. Write it knowing it can be thrown away.
pub struct PgUserRepository {
pool: PgPool,
}
impl UserRepository for PgUserRepository {
fn find_by_id(&self, id: UserId) -> Result<Option<User>, RepoError> {
// Postgres-specific code here.
// This entire struct is disposable.
}
fn save(&self, user: &User) -> Result<(), RepoError> {
// ...
}
}
Step 4: Depend on Contracts, Not Implementations
Consumers accept the contract, never the concrete type.
pub struct UserService<R: UserRepository> {
repo: R,
}
impl<R: UserRepository> UserService<R> {
pub fn new(repo: R) -> Self {
Self { repo }
}
pub fn get_user(&self, id: UserId) -> Result<Option<User>, RepoError> {
self.repo.find_by_id(id)
}
}
In Go, this is implicit — accept the interface:
func NewUserService(repo UserRepository) *UserService {
return &UserService{repo: repo}
}
Step 5: Test Against the Contract
Write tests that verify the contract, not the implementation. These tests can be reused across implementations.
// A test suite that works for ANY UserRepository implementation.
fn test_repository_contract(repo: &impl UserRepository) {
let user = User::new("test@example.com");
repo.save(&user).unwrap();
let found = repo.find_by_id(user.id()).unwrap();
assert_eq!(found, Some(user));
}
#[test]
fn pg_repo_satisfies_contract() {
let repo = PgUserRepository::new(test_pool());
test_repository_contract(&repo);
}
#[test]
fn in_memory_repo_satisfies_contract() {
let repo = InMemoryUserRepository::new();
test_repository_contract(&repo);
}
Layered Architecture
Organize code so that dependencies always point inward toward the domain:
src/
├── domain/ # Contracts + domain types (zero external deps)
│ ├── model.rs # Domain entities and value objects
│ ├── repo.rs # Repository contracts (traits)
│ └── service.rs # Domain services using contracts
├── infra/ # Disposable implementations
│ ├── pg_repo.rs # Postgres implementation
│ └── http.rs # HTTP handlers
└── main.rs # Wiring (connects contracts to implementations)
domain/defines contracts and types. It imports nothing frominfra/.infra/implements contracts. It imports fromdomain/.main.rswires implementations to contracts.
Disposability Checklist
Before considering a component done, verify:
- A contract (trait/interface) exists and lives in the domain layer
- The contract is named after the role, not the technology
- Errors are domain-level, not infrastructure-level
- Consumers depend on the contract, not the implementation
- The implementation can be deleted without changing any consumer code
- Contract-level tests exist and can run against any implementation
- No implementation details leak through the contract (e.g., SQL types, HTTP types)
Anti-Patterns
| Anti-Pattern | Problem | Fix |
|---|---|---|
| Fat contract | 10+ methods, hard to implement | Split into focused contracts |
| Leaky contract | Infrastructure types in signatures | Use domain types only |
| Concrete dependency | Consumer imports the struct directly | Accept the trait/interface |
| God module | One module does everything | Extract contracts and split |
| Premature abstraction | Contract with only one possible implementation forever | Wait until there's a reason to abstract |
When NOT to Abstract
Not everything needs a contract. Skip abstraction when:
- The component is a pure function with no side effects
- There will genuinely never be an alternative implementation
- The "contract" would be a trivial 1:1 mirror of a standard library type
- You're early in exploration and the boundary isn't clear yet
Start concrete, extract a contract when the second use case appears or when you need testability.
Recommended Agent Skills
Expand your agent's capabilities with these related and highly-rated skills.
hands-on-test
Performs manual hands-on testing of a web application using playwright-cli. Spawns the dev server if needed, navigates to pages, performs browser actions, captures screenshots, checks outcomes, and produces a structured test report. Use when the user wants to visually verify a web feature, perform exploratory testing, or validate UI behavior.
security-review
Request a security expert assessment for code changes that touch child process spawning, file system access, configuration loading, or environment variable handling. Use when the Reviewer identifies security-sensitive changes in the MCP-LSP bridge.
playwright-cli
Automates browser interactions for web testing, form filling, screenshots, and data extraction. Use when the user needs to navigate websites, interact with web pages, fill forms, take screenshots, test web applications, or extract information from web pages.
verl-rl-training
Provides guidance for training LLMs with reinforcement learning using verl (Volcano Engine RL). Use when implementing RLHF, GRPO, PPO, or other RL algorithms for LLM post-training at scale with flexible infrastructure backends.
openrlhf-training
High-performance RLHF framework with Ray+vLLM acceleration. Use for PPO, GRPO, RLOO, DPO training of large models (7B-70B+). Built on Ray, vLLM, ZeRO-3. 2× faster than DeepSpeedChat with distributed architecture and GPU resource sharing.
gguf-quantization
GGUF format and llama.cpp quantization for efficient CPU/GPU inference. Use when deploying models on consumer hardware, Apple Silicon, or when needing flexible quantization from 2-8 bit without GPU requirements.
Didn't find tool you were looking for?