low-level-code-review

reviewing a git diff for small localized coding mistakes that can be fixed without high-level understanding

1.26x

Quality

55%

Does it follow best practices?

Impact

86%

1.26x

Average score across 3 eval scenarios

Securityby

Risky

Do not use without reviewing

Optimize this skill with Tessl

npx tessl skill review --optimize ./.claude/skills/low-level-code-review/SKILL.md

Evaluation results

100%

43%

Code Review: File Storage Module

Output format + ACID disk violations

Criteria

Without context

With context

Issues grouped by file

100%

Line numbers cited

100%

Original code quoted

100%

Concrete fix provided

100%

One-sentence explanation

100%

Missing fsync before rename

100%

Missing directory fsync

100%

Raw pointer flagged

20%

100%

West const flagged

100%

Runtime errors prioritized

80%

100%

No compiler-caught issues

80%

100%

93%

Code Review: Account Cache Module

ACID memory violations + RAII guards

Criteria

Without context

With context

updateBalance missing lock

100%

batchUpdate not atomic

100%

batchUpdate missing lock

100%

87%

evict dirty-count bug

100%

Manual lock/unlock flagged

100%

flushAll exception safety

100%

getDirtyAccounts missing lock

100%

87%

Raw mutex pointer unnecessary

50%

37%

Line numbers cited

100%

Concrete fixes provided

100%

Focuses on added lines only

100%

65%

Code Review: Message Router Module

Large diff deduplication + null-deref

Criteria

Without context

With context

Null dereference in sendMessage

100%

Null dereference in sendMessageWithRetry

100%

Runtime errors before style

100%

West const deduplicated

West const flagged at least once

Large diff split suggestion

Line numbers cited

100%

Original code quoted

100%

Concrete fix provided

100%

No compiler-caught issues

33%

100%

Repository: stellar/stellar-core
Commit: 1b0eccd

Evaluated: 6 days ago
Agent: Claude Code
Model: Claude Sonnet 4.6

Table of Contents

Code Review: File Storage Module Code Review: Account Cache Module Code Review: Message Router Module

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.