Skip to content

feat: improve skill review scores for 5 skills#17

Open
rohan-tessl wants to merge 1 commit intoBankrBot:mainfrom
rohan-tessl:improve/skill-review-optimization
Open

feat: improve skill review scores for 5 skills#17
rohan-tessl wants to merge 1 commit intoBankrBot:mainfrom
rohan-tessl:improve/skill-review-optimization

Conversation

@rohan-tessl
Copy link
Copy Markdown

Hey @0xdeployer 👋

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
bankr-client-patterns 16% 86% +70%
bankr-error-handling 17% 90% +73%
bankr-safety 17% 82% +65%
sdk-balance-queries 17% 95% +78%
bankr-token-trading 17% 89% +72%

This PR covers 5 of your 39 skills to keep the contribution focused and reviewable — more can be improved in follow-ups or via automated review on future PRs.

Changes summary

All 5 skills:

  • Fixed name frontmatter to use valid kebab-case format (was causing validation failures that blocked LLM judge scoring entirely)
  • Removed version from frontmatter (flagged as unknown key — use metadata.version if needed)
  • Rewrote description with explicit "Use when..." clause and concrete trigger terms for better skill selection
  • Added structured ## Workflow sections for clearer step-by-step guidance

bankr-client-patterns (16% → 86%):

  • Reduced from 509 lines to 189 lines — condensed verbose type definitions into a compact union type, tightened API function code, and compressed project scaffolding templates
  • Preserved all essential types, functions, and usage patterns

bankr-error-handling (17% → 90%):

  • Streamlined authentication resolution into a numbered list instead of separate code blocks per step
  • Added explicit "do NOT retry" guidance directly in the workflow

bankr-safety (17% → 82%):

  • Consolidated dedicated wallet section, merged recommended funding into a single line
  • Converted pre-deployment checklist from bullets to numbered steps for clearer sequence

sdk-balance-queries (17% → 95%):

  • Added explicit workflow steps (initialize client → call promptAndWait → parse response)
  • Cleaned up prompt patterns section for scannability

bankr-token-trading (17% → 89%):

  • Merged chain selection and slippage into concise paragraphs instead of bullet lists
  • Added safety note about failed slippage transactions (no funds lost)

Want your remaining skills optimised too? 🚀

This PR covers 5 of your 39 skills to keep the contribution focused and reviewable. We have tooling that can go further:

  • Optimize remaining 34 skills automatically (same AI-powered pass as above)
  • Add a GitHub Actiontesslio/skill-review-and-optimize — that automatically reviews (and optionally optimizes) any SKILL.md changed in future PRs. Review mode works with zero secrets; maintainers can add TESSL_API_TOKEN for AI suggestions and the /apply-optimize comment flow.

Interested? Just tick the box below and we'll raise a follow-up PR:

  • Yes please! Raise a follow-up PR: optimize the remaining 34 skills + add the Tessl skill-review-and-optimize GitHub Action
  • No thanks — happy with the 5 skills in this PR

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 @0xdeployer 👋

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 |
|-------|--------|-------|--------|
| bankr-client-patterns | 16% | 86% | +70% |
| bankr-error-handling | 17% | 90% | +73% |
| bankr-safety | 17% | 82% | +65% |
| sdk-balance-queries | 17% | 95% | +78% |
| bankr-token-trading | 17% | 89% | +72% |

This PR covers 5 of 39 skills to keep the contribution focused and reviewable.

Changes across all 5 skills:
- Fixed name frontmatter to valid kebab-case (was blocking LLM judge scoring)
- Removed unknown version key from frontmatter
- Rewrote descriptions with "Use when..." clauses and concrete trigger terms
- Added structured Workflow sections for clearer step-by-step guidance
- bankr-client-patterns: reduced from 509 to 189 lines
@rohan-tessl rohan-tessl marked this pull request as ready for review April 20, 2026 09:49
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