MarginFi — Solana lending and borrowing
56
47%
Does it follow best practices?
Impact
Pending
No eval scenarios have been run
Advisory
Suggest reviewing before use
Optimize this skill with Tessl
npx tessl skill review --optimize ./src/skills/bundled/marginfi/SKILL.mdQuality
Discovery
22%Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.
This description is too terse and lacks both concrete action verbs and explicit trigger guidance. While the mention of 'MarginFi' provides some distinctiveness, the description fails to communicate what specific operations the skill performs or when Claude should select it. It reads more like a category label than a functional skill description.
Suggestions
Add specific concrete actions the skill performs, e.g., 'Deposits collateral, borrows assets, manages lending positions, checks borrow/supply rates on MarginFi protocol.'
Add an explicit 'Use when...' clause, e.g., 'Use when the user asks about MarginFi, Solana lending, borrowing on Solana, DeFi lending rates, or managing MarginFi positions.'
Include common user-facing trigger terms and variations such as 'DeFi', 'supply', 'collateral', 'liquidation', 'borrow rate', and 'marginfi' to improve keyword coverage.
| Dimension | Reasoning | Score |
|---|---|---|
Specificity | The description only names the domain ('Solana lending and borrowing') without listing any concrete actions. It does not specify what operations the skill performs, such as depositing, withdrawing, managing positions, or querying rates. | 1 / 3 |
Completeness | The description weakly addresses 'what' (lending and borrowing) and completely omits 'when' — there is no 'Use when...' clause or any explicit trigger guidance for when Claude should select this skill. | 1 / 3 |
Trigger Term Quality | It includes some relevant keywords like 'MarginFi', 'Solana', 'lending', and 'borrowing' that users might naturally say, but misses common variations like 'DeFi', 'supply', 'borrow rates', 'collateral', 'liquidation', or 'marginfi protocol'. | 2 / 3 |
Distinctiveness Conflict Risk | 'MarginFi' is a specific protocol name which helps distinguish it, but 'Solana lending and borrowing' is broad enough to overlap with other Solana DeFi lending skills (e.g., Solend, Kamino). Without more specific triggers, there's moderate conflict risk. | 2 / 3 |
Total | 6 / 12 Passed |
Implementation
72%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This is a clean, well-structured command reference for MarginFi that is concise and easy to scan. Its main weakness is the lack of workflow guidance around risk management—given that liquidation is a real concern, the skill should include explicit steps for checking health factor before/after borrowing and guidance on what to do when health is low. The commands themselves are clear but could benefit from expected output examples.
Suggestions
Add an explicit workflow section showing the safe sequence: check health → borrow → verify health factor is still safe, with a warning threshold (e.g., 'If health < 1.5, consider repaying before borrowing more').
Include example output for key commands like `/marginfi health` and `/marginfi account` so Claude knows what to expect and can interpret results for users.
Add a brief safety note or validation step about checking health factor before and after borrow operations to prevent liquidation risk.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Very lean and efficient. The one-line description of MarginFi is appropriate context-setting without over-explaining. Every section earns its place with no unnecessary elaboration. | 3 / 3 |
Actionability | Commands are clearly listed with concrete syntax and examples, but these are slash commands rather than executable code. There's no guidance on what happens when things go wrong (e.g., insufficient collateral, health factor too low), and no details on expected outputs or return formats. | 2 / 3 |
Workflow Clarity | The examples section implicitly shows a workflow (deposit → borrow → check health → repay → withdraw), but there's no explicit sequencing, no validation checkpoints, and no guidance on checking health factor before borrowing or what to do if health factor drops dangerously low. For a protocol involving liquidation risk, missing feedback loops and safety checks is a notable gap. | 2 / 3 |
Progressive Disclosure | For a simple skill under 50 lines, the content is well-organized into logical sections (Commands, Examples, Configuration, See Also) with clear cross-references to related protocols. No unnecessary nesting or monolithic blocks. | 3 / 3 |
Total | 10 / 12 Passed |
Validation
90%Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.
Validation — 10 / 11 Passed
Validation for skill structure
| Criteria | Description | Result |
|---|---|---|
frontmatter_unknown_keys | Unknown frontmatter key(s) found; consider removing or moving to metadata | Warning |
Total | 10 / 11 Passed | |
e71a5f6
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.