CtrlK
BlogDocsLog inGet started
Tessl Logo

1k-defi-module-integration

Navigate and implement OneKey App Earn, Borrow, Staking, and DeFi Portfolio flows. Use for protocol data, portfolio actions, transactions, pending/history, refresh, and Earn routes.

64

Quality

76%

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 ./.skillshare/skills/1k-defi-module-integration/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

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

An exemplary lean overview with best-in-class progressive disclosure and a clear verified workflow. Its one weakness is actionability: the core guidance is stated as abstract invariants rather than executable steps, so a reader must infer how to run checks or where code lives.

Suggestions

Add one concrete executable anchor per workflow step, e.g. the actual test command or repository-check invocation referenced in 'Run nearby tests and repository-required checks'.

Turn at least one abstract invariant (e.g. 'reconcile chain allowance when an approval-sensitive dialog mounts') into a short worked example showing where the check lives and what to compare.

Name the entry points or directory areas for 'Reproduce the real entry and affected platform' so the first Quick Start step is executable rather than inferential.

DimensionReasoningScore

Conciseness

The ~60-line body is lean and dense with project-specific invariants ("A seeded approval/allowance or cached position is a display/input hint, not proof of current chain state") and contains no padding or explanation of concepts Claude already knows — every token earns its place.

5 / 5

Actionability

Concrete artifacts exist — the trace chain 'entry -> data/position -> action -> transaction -> status -> refresh', failure-class reference routing, and the Finish checklist — but most guidance is abstract invariant statements ("Keep the full operation scope...", "Close a selector/modal at the user action boundary") with no commands, paths, or named tests, leaving key execution details to inference.

3 / 5

Workflow Clarity

The Quick Start is a clear four-step sequence (reproduce -> trace -> find first wrong owner -> reuse-and-verify) with explicit checkpoints ('verify the changed path and a relevant sibling', 'Run nearby tests and repository-required checks, then prove the affected route reaches the intended terminal and refresh state'), though no fix-and-retry error-recovery loop is spelled out.

4 / 5

Progressive Disclosure

The body is a concise overview routing to four references by failure class (architecture, operation flow, portfolio actions, validation), each clearly signaled with its scope; all four linked files exist in the bundle and contain no nested references, so the structure is one level deep and easy to navigate.

5 / 5

Total

17

/

20

Passed

Description

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

A distinct, well-triggered description with explicit 'Use for...' guidance, held back only by generic action verbs. Naming one or two concrete capabilities (e.g. tracing operation status, reconciling approvals) alongside the domain enumeration would lift it into the top band.

Suggestions

Replace the generic 'Navigate and implement' with 2-3 concrete capabilities, e.g. 'Trace Earn/Borrow operations from entry through transaction status and refresh, reconcile approvals, and implement portfolio actions'.

Add natural synonyms users might say for the covered flows (e.g. 'supply/lending', 'position history', 'transaction pending list') to broaden trigger coverage.

Make the 'what' side as explicit as the 'when' side by stating the skill's actual deliverable (ownership boundaries and reference routing for DeFi module changes).

DimensionReasoningScore

Specificity

The description names the domain precisely ("OneKey App Earn, Borrow, Staking, and DeFi Portfolio flows") and enumerates covered areas ("protocol data, portfolio actions, transactions, pending/history, refresh"), but the only verbs are the generic "Navigate and implement" — matching the anchor for 1-2 concrete actions without comprehensive coverage.

3 / 5

Completeness

Both parts are present — the "Use for protocol data, portfolio actions, transactions..." clause gives explicit trigger guidance, but the 'what' side ("Navigate and implement ... flows") stays abstract about what the skill concretely does, keeping it below the fully-explicit anchor.

4 / 5

Trigger Term Quality

Natural domain terms like "Earn", "Borrow", "Staking", "portfolio actions", "pending/history", and "refresh" give good keyword coverage a user in this codebase would actually say, though common synonyms and variations are missing (e.g. 'supply', 'lending', 'position history').

4 / 5

Distinctiveness Conflict Risk

The OneKey App Earn/Borrow/Staking/DeFi Portfolio niche is product-specific with distinct triggers, so it is clearly distinguishable from and unlikely to fire for unrelated skills.

5 / 5

Total

16

/

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
OneKeyHQ/app-monorepo
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.