Maintainability and design review instructions for the expert agent
Install with Tessl CLI
npx tessl i github:athal7/dotfiles --skill review-maintainability77
Quality
46%
Does it follow best practices?
Impact
99%
1.45xAverage score across 6 eval scenarios
Optimize this skill with Tessl
npx tessl skill review --optimize ./dot_config/opencode/skill/review-maintainability/SKILL.mdPhase 1 exploration before findings
Exploration log present
100%
100%
Full file read logged
50%
100%
Caller search logged
53%
100%
AGENTS.md read logged
100%
100%
Test file read logged
100%
100%
JSON findings block
100%
100%
JSON escalations block
0%
100%
Debug print flagged
100%
100%
Duplicate helper flagged
100%
100%
Vague variable name flagged
100%
100%
Without context: $0.2322 · 1m · 10 turns · 13 in / 3,881 out tokens
With context: $0.5741 · 2m 21s · 26 turns · 31 in / 8,394 out tokens
Issue context scope creep and test coverage
Scope creep detected
100%
100%
Missing case-insensitive test
100%
100%
Exploration log present
100%
100%
Full file read logged
100%
100%
Test file read logged
100%
100%
JSON findings key present
100%
100%
JSON escalations key present
0%
100%
Scope creep finding references issue
100%
100%
Findings in scope only
0%
0%
Without context: $0.1847 · 1m 4s · 7 turns · 12 in / 3,519 out tokens
With context: $0.4889 · 2m 23s · 22 turns · 59 in / 7,234 out tokens
JSON output format escalations and finding tone
JSON object structure
100%
100%
subprocess not in findings
0%
100%
Correctness escalated
0%
100%
I statements or questions
0%
100%
Valid severity values
0%
100%
suggested_fix on every finding
0%
100%
Vague naming flagged
30%
100%
Duplicate logic flagged
100%
100%
Missing tests flagged
100%
100%
Without context: $0.2169 · 1m 26s · 10 turns · 15 in / 4,418 out tokens
With context: $0.5711 · 2m 32s · 23 turns · 61 in / 8,816 out tokens
Pre-existing issue detection via git blame
git blame step logged
38%
100%
Pre-existing severity used
22%
100%
Old issues tagged pre-existing
53%
100%
Debug print flagged as new
100%
100%
DRY violation flagged as new
100%
100%
Exploration log present
100%
100%
Missing export_csv test flagged
0%
100%
Valid JSON structure
28%
100%
Without context: $0.3201 · 1m 37s · 15 turns · 20 in / 6,123 out tokens
With context: $0.6919 · 2m 38s · 34 turns · 286 in / 8,489 out tokens
Style tolerance and avoiding false positives
CONVENTIONS.md read logged
100%
100%
handle_auth NOT flagged
100%
100%
temp_token NOT flagged
100%
100%
Missing test flagged
100%
100%
Test file read logged
100%
100%
Valid JSON output
0%
100%
I statements or questions
0%
100%
Without context: $0.3252 · 1m 36s · 16 turns · 21 in / 5,510 out tokens
With context: $0.5311 · 2m 38s · 19 turns · 22 in / 8,815 out tokens
Single responsibility, drift risk, and minimize diff
Single responsibility flagged
81%
100%
Whitespace diff flagged
100%
100%
Drift risk flagged
100%
100%
Exploration log present
100%
100%
InvoiceBuilder read logged
100%
100%
Security/correctness not in findings
0%
100%
Finding titles 2-5 words
0%
100%
I statements or questions
0%
100%
Without context: $0.2294 · 1m 23s · 9 turns · 14 in / 4,322 out tokens
With context: $0.5397 · 2m 37s · 19 turns · 22 in / 9,455 out tokens
Table of Contents
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.