CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

architecture-renovator

Resolves deep architectural debt in the Kotlin Android codebase through macro-level refactoring. Use this skill to split God classes (like massive ViewModels), extract UseCases, enforce strict layer mapping (e.g., DTOs to UI models), implement interface segregation, and replace brittle inheritance with class delegation.

Quality

88%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Passed

No findings from the security scan

SKILL.md
Quality
Evals
Security

Quality

Content

78%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

A well-structured, largely actionable refactoring workflow with explicit verification commands, quantified heuristics, and sensible guardrails (ask-first list, never-do list). Its weaknesses are mild: some philosophy restating knowledge Claude already has, no failure-handling loop in VERIFY, and referenced artifacts (journal format, mapping boundaries) described but never exemplified.

Suggestions

Trim or compress the Philosophy section into one line of team-specific principles — SRP and composition-over-inheritance are knowledge Claude already has and cost tokens without adding guidance.

Add a brief error-recovery loop to VERIFY, e.g., 'If tests or lint fail: fix, re-run, and only proceed when green', to earn a full feedback cycle.

Include one compact example each of a mapping boundary (e.g., a two-line fun UserDto.toUiModel(): UserUiModel) and a properly formatted journal entry, since both are mandated but currently only described.

DimensionReasoningScore

Conciseness

The body is efficient with concrete commands and thresholds, but the Philosophy section restates SRP and composition-over-inheritance that Claude already knows, and decorative emoji headers add minor padding — anchor 4, not 5.

4 / 5

Actionability

Gives executable gradle commands ('./gradlew ktfmtFormat', 'testDebugUnitTest', 'lintDebug', 'assembleDebug'), quantified thresholds (500-line ViewModels, 30+ method interfaces, ~300-line change cap) and a worked PR title. Minor gaps: no example of a mapping boundary (toDomain()/toUiModel()) or journal entry despite requiring both.

4 / 5

Workflow Clarity

Clear 5-step sequence (OBSERVE, SELECT & PROPOSE with an approval gate, RENOVATE, VERIFY, PRESENT) with an explicit verification step running four checks. However there is no error-recovery loop (what to do when tests or lint fail), so it fits anchor 4 rather than 5.

4 / 5

Progressive Disclosure

The ~49-line body has clean, well-labeled sections (Goal, Constraints, Instructions, Examples) with no need for external reference files, and none are bundled — per the rubric's simple-skill guidance this scores 5.

5 / 5

Total

17

/

20

Passed

Description

92%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

A strong description: concrete, comprehensive, third-person, with an explicit 'Use this skill to...' trigger clause. The only weakness is reliance on architectural jargon for triggers without natural-language synonyms or casual phrasings a user might use.

DimensionReasoningScore

Specificity

Lists five specific concrete actions ('split God classes', 'extract UseCases', 'enforce strict layer mapping (e.g., DTOs to UI models)', 'implement interface segregation', 'replace brittle inheritance with class delegation') with comprehensive coverage of the domain in third-person voice.

5 / 5

Completeness

Explicitly answers 'what' ('Resolves deep architectural debt in the Kotlin Android codebase through macro-level refactoring' plus the capability list) and 'when' ('Use this skill to split God classes...') with concrete trigger phrases.

5 / 5

Trigger Term Quality

Good natural domain keywords ('God classes', 'massive ViewModels', 'UseCases', 'DTOs', 'class delegation') but misses plainer synonyms a user would naturally say ('this ViewModel is too big', 'clean up this module'), so it falls between anchor 3 and 5 — closer to 4.

4 / 5

Distinctiveness Conflict Risk

Clear niche (Kotlin Android macro-refactoring) with distinct triggers ('ViewModels', 'UseCases', 'DTOs to UI models') that would not naturally fire competing skills; minimal conflict risk.

5 / 5

Total

19

/

20

Passed

Validation

100%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository

nekomangaorg/Neko

Path

.agents/skills/renovator/SKILL.md

Reviewed

Table of Contents

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.