Skip to content

feat: improve skill review scores (+21% avg across 5 skills)#98

Open
rohan-tessl wants to merge 1 commit into
gate:developfrom
rohan-tessl:improve/skill-review-optimization
Open

feat: improve skill review scores (+21% avg across 5 skills)#98
rohan-tessl wants to merge 1 commit into
gate:developfrom
rohan-tessl:improve/skill-review-optimization

Conversation

@rohan-tessl
Copy link
Copy Markdown

@rohan-tessl rohan-tessl commented Apr 2, 2026

Hey @gate 👋

I ran your skills through tessl skill review at work and found some targeted improvements.

image

Here's the full before/after:

Skill Before After Change
gate-exchange-affiliate 68% 93% +25%
gate-exchange-collateralloan 69% 93% +24%
gate-exchange-referral 69% 93% +24%
gate-mcp-cursor-installer 69% 87% +18%
gate-exchange-marketanalysis 69% 84% +15%

This PR is intentionally scoped to 5 skills to keep the review manageable — more skills can be improved in follow-ups or via automated review on future PRs.

Changes summary

gate-exchange-affiliate (68% → 93%)

  • Expanded frontmatter description with specific concrete actions and trigger terms
  • Eliminated repeated user_id warnings (stated 5-6 times → once in Critical Rules)
  • Made aggregation rules concrete: group by asset type, deduplicate across paginated responses
  • Moved bulk API reference, report templates, and validation examples to existing references/ files
  • Consolidated error handling into clean table format
  • Reduced from 641 lines to 182 while preserving all routing logic

gate-exchange-collateralloan (69% → 93%)

  • Expanded description with specific loan actions (create, repay, add/redeem collateral, check LTV)
  • Removed duplicate MCP tool listing (already documented in Available MCP Tools table)
  • Added concrete JSON examples for all three write operations
  • Added post-execution verification step (Step 6)
  • Eliminated redundant Judgment Logic Summary and Domain Knowledge sub-flows

gate-exchange-referral (69% → 93%)

  • Expanded description with specific actions and added "commission rebate" trigger
  • Consolidated Case Routing Map, Judgment Logic, and Workflow into unified 3-step workflow
  • Moved detailed product definitions and 14-case routing to references/ via progressive disclosure
  • Reduced URL repetition (defined once instead of 7+ times)
  • Reduced from 256 lines to 161

gate-mcp-cursor-installer (69% → 87%)

  • Expanded description with specific configuration actions
  • Consolidated four redundant MCP server listings into single authoritative Resources table
  • Removed cross-IDE references (Codex, Claude, OpenClaw) — skill is Cursor-specific
  • Added complete mcp.json example showing expected output
  • Added verification step with JSON validity checks

gate-exchange-marketanalysis (69% → 84%)

  • Rewrote description with action verbs and added trigger terms
  • Consolidated triple-redundant sections (Sub-Modules, Routing Rules, Domain Knowledge) into single Scenario Routing table
  • Added inline MCP call example for spot liquidity analysis
  • Added error handling and degradation steps
  • Reduced from 127 lines to 100

Honest disclosure — I work at @tesslio where we build tooling around skills like these. Not a pitch - just saw room for improvement and wanted to contribute.

Want to self-improve your skills? Just point your agent (Claude Code, Codex, etc.) at this Tessl guide and ask it to optimize your skill. Ping me - @rohan-tessl - if you hit any snags.

Thanks in advance 🙏

Hey @gate 👋

I ran your skills through `tessl skill review` at work and found some targeted improvements. Here's the full before/after:

| Skill | Before | After | Change |
|-------|--------|-------|--------|
| gate-exchange-affiliate | 68% | 93% | +25% |
| gate-exchange-collateralloan | 69% | 93% | +24% |
| gate-exchange-referral | 69% | 93% | +24% |
| gate-mcp-cursor-installer | 69% | 87% | +18% |
| gate-exchange-marketanalysis | 69% | 84% | +15% |

This PR is intentionally scoped to 5 skills to keep the review manageable — more skills can be improved in follow-ups or via automated review on future PRs.

Changes:
- Expanded frontmatter descriptions with specific actions and trigger terms
- Consolidated redundant sections (duplicate tool listings, repeated warnings, triple-redundant routing tables)
- Added concrete examples (JSON payloads, MCP call examples, mcp.json config)
- Added verification/error handling steps
- Moved bulk content to references/ for progressive disclosure
- Net reduction of ~580 lines while adding more actionable content
@rohan-tessl rohan-tessl marked this pull request as ready for review April 2, 2026 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant