feat: optimize 3 skills — solidity, battlechain, battlechain-tutorial#10
Open
rohan-tessl wants to merge 1 commit intoCyfrin:mainfrom
Open
feat: optimize 3 skills — solidity, battlechain, battlechain-tutorial#10rohan-tessl wants to merge 1 commit intoCyfrin:mainfrom
rohan-tessl wants to merge 1 commit intoCyfrin:mainfrom
Conversation
Hullo @PatrickAlphaC 👋 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 | |-------|--------|-------|--------| | solidity | 68% | 93% | +25% | | battlechain | 90% | 93% | +3% | | battlechain-tutorial | 64% | 93% | +29% | <details> <summary>Changes summary</summary> ### solidity (68% → 93%) - **Description**: Expanded with specific concrete actions (Foundry workflows, security patterns, gas optimization, testing strategies) and natural trigger terms (Solidity, EVM, ERC-20, ERC-721, DeFi, .sol files, Ethereum mainnet, Hardhat) - **Development Workflow**: Added an end-to-end workflow section (Write → Lint → Static Analysis → Test → Audit → Deploy) with explicit validation checkpoints between each step - **Progressive disclosure**: Extracted the branching tree technique into `BRANCHING-TREE.md` and governance patterns into `GOVERNANCE.md` — keeps SKILL.md focused while preserving all content - **Gas and Style Checklist**: Consolidated 8 short one-liner tips into a compact checklist format - **Trimmed redundancy**: Removed explanatory phrasing Claude already knows, tightened blog post references ### battlechain (90% → 93%) - **Description**: Made lifecycle action more concrete ("deploy → attack mode → promotion to mainnet"), added "contract verification" as trigger term, removed behavioral parenthetical - **Deployment Lifecycle**: Added a 7-step end-to-end lifecycle section with `cast call` verification checkpoints between critical steps (deploy, create agreement, request attack mode) - **Removed placeholder**: Removed empty "Hardhat — Coming soon" section - **Tightened intro**: Condensed the "What is BattleChain" heading into a single-line description ### battlechain-tutorial (64% → 93%) - **Description**: Expanded from vague "prepare projects for BattleChain deployment" to specific actions (scans Foundry contracts/scripts, gathers deployment parameters, generates deployment scripts, Safe Harbor agreements, attack mode requests) with domain trigger terms - **Progressive disclosure**: Extracted the 15-question flow into `QUESTIONS.md` and Solidity script templates/constants into `TEMPLATES.md` — reduced SKILL.md from 368 to 165 lines while preserving all content - **Question summary table**: Replaced inline question details with a concise reference table of all 15 questions - **Condensed AskUserQuestion pattern**: Moved repeated tool usage instructions to a shared section in QUESTIONS.md </details> 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](https://docs.tessl.io/evaluate/optimize-a-skill-using-best-practices) and ask it to optimize your skill. Ping me - [@popey](https://github.com/popey) - if you hit any snags. Thanks in advance 🙏
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hullo @PatrickAlphaC 👋
I ran your skills through
tessl skill reviewat work and found some targeted improvements. Here's the full before/after:Changes summary
solidity (68% → 93%)
BRANCHING-TREE.mdand governance patterns intoGOVERNANCE.md— keeps SKILL.md focused while preserving all contentbattlechain (90% → 93%)
cast callverification checkpoints between critical steps (deploy, create agreement, request attack mode)battlechain-tutorial (64% → 93%)
QUESTIONS.mdand Solidity script templates/constants intoTEMPLATES.md— reduced SKILL.md from 368 to 165 lines while preserving all contentHonest 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 - @popey - if you hit any snags.
Thanks in advance 🙏