CtrlK
BlogDocsLog inGet started
Tessl Logo

subsystem-summary-of-ledger

read this skill for a token-efficient summary of the ledger subsystem

44

Quality

43%

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

Fix and improve this skill with Tessl

tessl review fix ./.claude/skills/subsystem-summary-of-ledger/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

38%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.

The body is an accurate, well-organized, information-dense summary of the ledger subsystem that mostly respects Claude's intelligence and avoids padding. However, it is entirely descriptive with no actionable guidance, and it is a ~315-line monolith with zero progressive disclosure — detailed per-class material should live in referenced files with SKILL.md as an overview.

Suggestions

Split the per-class member documentation (LedgerManagerImpl, LedgerTxn/LedgerTxnRoot internals, SorobanNetworkConfig, InMemorySorobanState) into references/ files (e.g., classes.md, ledger-txn.md) and keep SKILL.md as an overview with clearly signaled links.

Add at least one actionable section telling Claude what to do with this knowledge (e.g., where to start when investigating a ledger-close bug, which files to open for a given change type).

Trim exhaustive inventories (genesis constants, every SorobanNetworkConfig setting, full method lists) to the high-signal subset, or move them to a reference file, to improve token efficiency.

DimensionReasoningScore

Conciseness

The body is dense, factual, and largely free of concepts Claude already knows — it assumes competence and covers non-public details (class members, phase transitions, genesis constants). But at ~315 lines with exhaustive per-class method/member lists (e.g., every SorobanNetworkConfig setting group, full genesis constants), much could be trimmed or split out, matching 'mostly efficient but could be tightened' rather than 4's minor instances.

3 / 5

Actionability

The content is a descriptive reference, not instruction: it names real files, classes, and methods (useful pointers), but contains no commands, code, or direction on what Claude should do with this knowledge — 'describes rather than instructs'. It sits above 1 because the file/class/method inventory is concrete and navigable, but below 3 because no executable or procedural guidance is given for any task.

2 / 5

Workflow Clarity

The 'Key Control Flow' section gives a well-sequenced, detailed walkthrough (ledger close steps a–n, parallel Soroban apply, commit flow), but this is the system's behavior, not a Claude-facing workflow — there are no steps, checkpoints, or validation guidance for using the skill. The destructive/batch cap does not apply (nothing instructs Claude to mutate anything), so this lands at 'sequence present but checkpoints missing'.

3 / 5

Progressive Disclosure

No bundle files (references/, scripts/, assets/) exist and the body contains no references to any external files — everything is inlined in SKILL.md. Per-class member documentation (LedgerManagerImpl, LedgerTxn internals, SorobanNetworkConfig settings) clearly belongs in separate reference files, matching 'content that clearly belongs in separate files is inlined'. It is above 1 only because section headers do provide navigable structure.

2 / 5

Total

10

/

20

Passed

Description

48%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.

The description is serviceable but thin: it names the domain and purpose in a natural third-person-adjacent imperative, yet provides no concrete capability list and no 'use when' trigger clause. Adding what the summary covers (e.g., ledger close flow, LedgerTxn, Soroban apply) and explicit trigger phrases would lift both completeness and trigger quality.

Suggestions

Add a 'Use when...' clause, e.g., 'Use when working on stellar-core's ledger code — LedgerManager, LedgerTxn, ledger close, or Soroban apply'.

Replace the meta phrasing 'read this skill for a token-efficient summary' with concrete capabilities: 'Summarizes the ledger subsystem: ledger close flow, LedgerTxn transactional layer, parallel Soroban apply, and Soroban network config'.

Include natural synonyms users would say (LedgerManager, LedgerTxn, ledger close, stellar-core) to improve trigger-term coverage.

DimensionReasoningScore

Specificity

The description names the domain ("ledger subsystem") and a vague quality claim ("token-efficient summary"), but names no concrete actions or capabilities — it is meta-language ("read this skill for a summary") equivalent to 'Processes PDF files', not the multi-action pattern of higher anchors. It is not a 1 because the domain term and the summary purpose are stated.

2 / 5

Completeness

The 'what' is reasonably clear (a summary of the ledger subsystem), but there is no 'Use when...' clause or equivalent trigger guidance, which caps completeness at 3 per the judging guidelines. It is above 2 because 'what' is stated clearly enough; below 4 because 'when' is entirely absent, not merely imprecise.

3 / 5

Trigger Term Quality

"ledger subsystem", "summary", and "token-efficient" are relevant keywords a contributor might say, but common variations and synonyms are missing (e.g., LedgerManager, LedgerTxn, stellar-core, ledger close, Soroban apply). Matches the 'some relevant keywords but missing common variations' anchor rather than 4, whose keyword coverage is both broader and more natural.

3 / 5

Distinctiveness Conflict Risk

"ledger subsystem" is a fairly distinct niche (stellar-core's ledger/LedgerTxn code) unlikely to trigger for unrelated skills, though "summary" is generic and it could overlap with other subsystem-summary skills in the same repo. Mostly distinct with minor overlap risk, matching the 4 anchor rather than 5's 'clear niche with distinct triggers'.

4 / 5

Total

12

/

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
stellar/stellar-core
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.