MarginFi — Solana lending and borrowing
68
55%
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 minimal to effectively guide skill selection. While it identifies the protocol name and general domain, it lacks specific actions, explicit trigger conditions, and comprehensive keywords. The description would benefit significantly from listing concrete capabilities and adding a 'Use when...' clause.
Suggestions
Add specific actions like 'Deposit collateral, borrow assets, repay loans, check health factor, view positions, and manage liquidation risk'
Add a 'Use when...' clause such as 'Use when the user mentions MarginFi, wants to lend/borrow on Solana, or asks about DeFi lending positions'
Include additional trigger terms like 'DeFi', 'collateral', 'interest', 'APY', 'health factor', 'liquidation' to improve keyword coverage
| Dimension | Reasoning | Score |
|---|---|---|
Specificity | The description only names the domain ('Solana lending and borrowing') without listing any concrete actions. It doesn't specify what operations can be performed like depositing, withdrawing, liquidating, or checking positions. | 1 / 3 |
Completeness | The description weakly addresses 'what' (lending and borrowing) but completely lacks any 'when' guidance. There is no 'Use when...' clause or explicit trigger guidance. | 1 / 3 |
Trigger Term Quality | Includes some relevant keywords ('MarginFi', 'Solana', 'lending', 'borrowing') that users might naturally say, but missing common variations like 'DeFi', 'collateral', 'interest rates', 'borrow', 'lend', or 'loan'. | 2 / 3 |
Distinctiveness Conflict Risk | The 'MarginFi' brand name provides some distinctiveness, but 'Solana lending and borrowing' could overlap with other Solana DeFi lending protocols like Solend or Kamino. | 2 / 3 |
Total | 6 / 12 Passed |
Implementation
87%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This is a well-crafted, concise skill that efficiently documents MarginFi commands without unnecessary explanation. The command syntax and examples are immediately actionable. However, for a DeFi lending protocol where liquidation is a real risk, the skill would benefit from explicit workflow guidance around health factor monitoring before/after operations.
Suggestions
Add a brief workflow section showing safe borrowing pattern: check health -> borrow -> verify health (e.g., '/marginfi health' before and after '/marginfi borrow')
Include a warning or checkpoint about minimum health factor thresholds before borrowing operations
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Extremely lean and efficient. No unnecessary explanations of what lending/borrowing is or how Solana works. Every line serves a purpose - commands, examples, and configuration only. | 3 / 3 |
Actionability | Provides concrete, copy-paste ready commands with clear syntax. Examples show real usage patterns including 'all' keyword for full withdrawals/repayments. Configuration section has exact environment variable names. | 3 / 3 |
Workflow Clarity | While individual commands are clear, there's no explicit workflow for safe lending operations. Missing validation steps like checking health factor before borrowing more, or verifying positions after transactions. For DeFi operations involving liquidation risk, feedback loops would be valuable. | 2 / 3 |
Progressive Disclosure | Well-organized with clear sections (Commands, Examples, Configuration, See Also). For a simple command-reference skill under 50 lines, this structure is appropriate. Cross-references to related protocols are helpful without being nested. | 3 / 3 |
Total | 11 / 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 | |
2a8c94e
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.