Enforces a clear naming convention for getters to prevent their misuse in domain logic.
Install
mkdir -p .claude/skills/breach-encapsulation-naming && curl -L -o skill.zip "https://agentskills.codes/api/skills/download/16375" && unzip -o skill.zip -d .claude/skills/breach-encapsulation-naming && rm skill.zipInstalls to .claude/skills/breach-encapsulation-naming
Activation
This is the description your AI agent reads to decide when to run this skill — the better it matches your request, the more reliably it fires.
getterの濫用を防ぐための命名規約スキル。ドメインモデルでgetterが必要な場合(永続化、JSON変換など)に `breachEncapsulationOf` プレフィックスを付与することで、カプセル化を破っていることを明示する。 これにより、Tell Don't Ask原則の違反を未然に防ぎ、getterの意図しない使用を抑制する。 コードレビュー、新規実装、リファクタリング時にgetter設計が必要な場合に使用。 対象言語: Java, Kotlin, Scala, TypeScript, Python, Go, Rust。 トリガー:「getterの命名規約」「カプセル化を破るgetter」「永続化用のgetter」 「breachEncapsulation」「getterを作りたいが濫用を防ぎたい」といったgetter命名関連リクエストで起動。Key capabilities
- →Add 'breachEncapsulationOf' prefix to getters for domain models
- →Prevent unintended use of getters in business logic
- →Identify misuse of getters in code reviews
- →Support getter design for persistence and JSON conversion
- →Apply naming conventions to Java, Kotlin, Scala, TypeScript, Python, Go, and Rust
How it works
The skill suggests adding a 'breachEncapsulationOf' prefix to getters in domain models to explicitly indicate a breach of encapsulation. This naming convention makes the exceptional use of getters visible and discourages their misuse in business logic.
Inputs & outputs
When to use breach-encapsulation-naming
- →Refactoring domain models
- →Enforcing strict code reviews
- →Standardizing getter naming
About this skill
Breach Encapsulation Naming
getterを作るなら「カプセル化を破っている」と名前で叫べ。
核心原則
ドメインモデルのgetterには breachEncapsulationOf プレフィックスを付与し、カプセル化を破っていることを明示する。
| アプローチ | 特徴 | 効果 |
|---|---|---|
通常のgetter (getName()) | 気軽に使える | 濫用されやすい |
明示的なgetter (breachEncapsulationOfName()) | 使用時に「破っている」と意識 | 濫用を抑制 |
なぜこの命名規約が必要か
ジレンマ
- Tell Don't Ask原則: getterを使わず、オブジェクトに命じるべき
- 現実の制約: 永続化やJSON変換ではgetterが必要
- 問題: getterがあると、ビジネスロジックでも使ってしまう
解決策
getterを「長くて目立つ名前」にすることで:
- 使うたびに「これは例外的な使用だ」と意識させる
- コードレビューで発見しやすくなる
- 静的解析ツールで検出可能になる
命名パターン
基本形式
breachEncapsulationOf<PropertyName>()
言語別の例
// Java
public String breachEncapsulationOfName() { return this.name; }
public Money breachEncapsulationOfPrice() { return this.price; }
// Kotlin
fun breachEncapsulationOfName(): String = name
fun breachEncapsulationOfPrice(): Money = price
// TypeScript
breachEncapsulationOfName(): string { return this.name; }
breachEncapsulationOfPrice(): Money { return this.price; }
# Python
def breach_encapsulation_of_name(self) -> str:
return self._name
// Go
func (u *User) BreachEncapsulationOfName() string { return u.name }
// Rust
pub fn breach_encapsulation_of_name(&self) -> &str { &self.name }
適用判断フロー
getterが必要か?
↓
├─ NO → getterを作らない(Tell Don't Ask)
│
└─ YES → 対象は?
│
├─ 値オブジェクト → 通常のアクセサでOK
│ (イミュータブルかつ振る舞いが限定的なため)
│ 例: Money.amount(), UserId.value()
│
└─ エンティティ → breachEncapsulationOf を使用
│
└─ なぜ必要?
├─ 永続化/シリアライズ → ✅ 許容
├─ 表示/UI → ✅ 許容
├─ テスト → ✅ 許容
└─ ビジネスロジック → ❌ Tell パターンに変換
値オブジェクト vs エンティティ
| 種類 | 特徴 | getter方針 |
|---|---|---|
| 値オブジェクト | イミュータブル、等価性で識別 | 通常のアクセサ可(amount(), value()) |
| エンティティ | ミュータブル、IDで識別 | breachEncapsulationOf を使用 |
理由: 値オブジェクトは内部状態が変わらないため、getterを公開しても「状態を取得→外部で判断→更新」というAskパターンが発生しにくい。
アンチパターン検出
以下のパターンを見つけたら警告:
// ❌ breachEncapsulationOf + if → Tell Don't Ask違反
if (user.breachEncapsulationOfAge() >= 18) {
// ロジック
}
// ❌ breachEncapsulationOf + 計算 → ロジックが外部に漏れている
total = item.breachEncapsulationOfPrice() * item.breachEncapsulationOfQuantity();
// ❌ 連鎖呼び出し → デメテルの法則違反
order.breachEncapsulationOfCustomer().breachEncapsulationOfAddress().getCity();
許容される使用例
1. 永続化層(リポジトリ実装)
// ✅ 永続化のためのアクセスは許容
public UserEntity toEntity(User user) {
return new UserEntity(
user.breachEncapsulationOfId(),
user.breachEncapsulationOfName(),
user.breachEncapsulationOfEmail()
);
}
2. JSON/XMLシリアライズ
// ✅ DTOへの変換は許容
toJson(): UserJson {
return {
id: this.breachEncapsulationOfId(),
name: this.breachEncapsulationOfName()
};
}
3. テストでのアサーション
// ✅ テストでの検証は許容
@Test
void shouldChangeName() {
user.rename("New Name");
assertEquals("New Name", user.breachEncapsulationOfName());
}
4. デバッグ/ログ出力
# ✅ デバッグ目的は許容
logger.debug(f"User: {user.breach_encapsulation_of_name()}")
実装ガイドライン
1. ドメインモデル側
public class User {
private final UserId id;
private String name;
private Email email;
// ❌ 通常のgetterは作らない
// public String getName() { return name; }
// ✅ カプセル化を破ることを明示
public String breachEncapsulationOfName() {
return name;
}
// ✅ ビジネスロジックは振る舞いとして提供
public void rename(String newName) {
validateName(newName);
this.name = newName;
}
public boolean hasName(String name) {
return this.name.equals(name);
}
}
2. インフラ層での使用
// リポジトリ実装
public class JpaUserRepository implements UserRepository {
@Override
public void save(User user) {
UserEntity entity = new UserEntity();
entity.setId(user.breachEncapsulationOfId().value());
entity.setName(user.breachEncapsulationOfName());
entity.setEmail(user.breachEncapsulationOfEmail().value());
jpa.save(entity);
}
}
コードレビュー観点
| チェック項目 | 対応 |
|---|---|
breachEncapsulationOf + if | Tellパターンへの変換を提案 |
breachEncapsulationOf + 計算 | 計算ロジックをオブジェクトに移動 |
| ドメイン層での使用 | 永続化/シリアライズ以外なら警告 |
| 連鎖呼び出し | 委譲メソッドの追加を提案 |
通常のgetter (getName()) | breachEncapsulationOf への変更を提案 |
静的解析との連携
カスタムリントルール例
# 検出ルール
1. breachEncapsulationOf の後に if/switch が続く → 警告
2. ドメイン層で breachEncapsulationOf を呼び出している → 警告
3. 通常の get プレフィックスがドメインモデルにある → 警告
関連スキル
| スキル | 関係 |
|---|---|
| tell-dont-ask | 本スキルの前提。getterを使わない設計を優先 |
| first-class-collection | コレクションのカプセル化にも同じ原則を適用 |
| domain-building-blocks | 値オブジェクト設計との整合性 |
参考文献
- かとじゅん「ドメインオブジェクトのためのGetter/Setter」(2018)
- https://blog.j5ik2o.me/entry/2018/08/14/134125
- 本スキルの原典。カプセル化の歴史的劣化とbreachEncapsulationOf命名規約の提案
詳細ガイドライン
言語別の詳細な実装パターンは references/patterns.md を参照。
関連スキル(併読推奨)
このスキルを使用する際は、以下のスキルも併せて参照すること:
tell-dont-ask: getterを避けるべき理由の基盤原則law-of-demeter: getter連鎖が違反する構造面の原則first-class-collection: コレクションのカプセル化パターン
When not to use it
- →When a getter is not needed, follow the Tell Don't Ask principle
- →For value objects, use regular accessors
- →When using getters for business logic, convert to Tell pattern
Limitations
- →This skill does not apply to value objects
- →This skill does not support using 'breachEncapsulationOf' getters for business logic
How it compares
This approach explicitly names getters that break encapsulation, making their use a conscious decision, unlike generic getters that can be easily misused.
Compared to similar skills
breach-encapsulation-naming side by side with the closest alternatives in the catalog.
| Skill | Installs | Updated | Safety | Difficulty |
|---|---|---|---|---|
| breach-encapsulation-naming (this skill) | 0 | 5mo | No flags | Beginner |
| ast-grep-find | 3 | 7mo | Review | Intermediate |
| naming-analyzer | 2 | 6mo | Review | Beginner |
| dead-code | 1 | 7mo | Review | Intermediate |
Try saying
Example prompts that trigger this skill in your AI assistant.
More by j5ik2o
View all by j5ik2o →You might also like
ast-grep-find
parcadei
AST-based code search and refactoring via ast-grep MCP
naming-analyzer
davila7
Suggest better variable, function, and class names based on context and conventions.
dead-code
parcadei
Find unused functions and dead code in the codebase
tldr-code
parcadei
Token-efficient code analysis via 5-layer stack (AST, Call Graph, CFG, DFG, PDG). 95% token savings.
lsp-extract-function
blackwell-systems
Extract a selected code block into a named function. Primary path uses the language server's extract-function code action; falls back to manual extraction when no code action is available. Validates captured variables, scope shadowing, and compilation after extraction.
clean-code
RepairYourTech
Applies battle-tested clean code principles to writing, reviewing, and refactoring code. Covers naming, functions, comments, error handling, class design, and the critical difference between clever code and clear code.