feat: optimize 5 lowest-scoring skills + add Tessl review workflow#15
Open
rohan-tessl wants to merge 2 commits intomoasq:mainfrom
Open
feat: optimize 5 lowest-scoring skills + add Tessl review workflow#15rohan-tessl wants to merge 2 commits intomoasq:mainfrom
rohan-tessl wants to merge 2 commits intomoasq:mainfrom
Conversation
Improved skill scores via tessl skill review: | Skill | Before | After | Change | |-------|--------|-------|--------| | safari-extension | 49% | 79% | +30% | | nanowave-pipeline-development | 63% | 89% | +26% | | share-extension | 66% | 79% | +13% | | app-clips | 70% | 86% | +16% | | authentication | 70% | 86% | +16% | Changes: - Removed duplicated content in safari-extension, share-extension, app-clips - Properly fenced code blocks with language hints - Expanded frontmatter descriptions with specific capabilities and "Use when" clauses - Added structured sections with clear headings - Improved authentication with decision table and code examples
Hey @moasq 👋 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 | |-------|--------|-------|--------| | safari-extension | 49% | 79% | +30% | | nanowave-pipeline-development | 63% | 89% | +26% | | share-extension | 66% | 79% | +13% | | app-clips | 70% | 86% | +16% | | authentication | 70% | 86% | +16% | This PR is intentionally capped at five skills to keep it reviewable. The included GitHub Action workflow will surface Tessl feedback on future SKILL.md changes, so the rest of the library can improve incrementally. Changes: - safari-extension, share-extension, app-clips: removed duplicated content, fenced code blocks, structured sections, expanded descriptions - nanowave-pipeline-development: expanded description with specific components and "Use when" clause - authentication: expanded description, added decision table and inline code examples Also adds .github/workflows/skill-review.yml (tesslio/skill-review) for automatic skill review comments on future PRs.
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
1 participant
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.
Hey @moasq 👋
I ran your skills through
tessl skill reviewat work and found some targeted improvements.Here's the full before/after:
This PR is intentionally capped at five skills to keep it reviewable. The included GitHub Action workflow (see below) will surface Tessl feedback on future
SKILL.mdchanges, so the rest of the library can improve incrementally.Changes summary
safari-extension, share-extension, app-clips — These three skills had their entire content duplicated below a
---separator, which was tanking scores. Removed the duplication, properly fenced all code blocks with Swift language hints, added structured markdown sections with clear headings, and expanded frontmatter descriptions with specific capabilities.nanowave-pipeline-development — Expanded the frontmatter description to include specific technical components (6-phase flow,
parseClaudeJSON[T],ComposeAgenticSystemPrompt) and added an explicit "Use when..." trigger clause. Content was already solid — description was the main improvement area.authentication — Expanded the frontmatter description to cover all auth modes, Supabase delegation, and session lifecycle. Added a decision table for auth method selection (replaces the plain list), added inline Swift code example for dependency injection pattern, and improved formatting consistency.
🤖 Included: Tessl Skill Review GitHub Action
This PR also adds
.github/workflows/skill-review.ymlso future PRs that touchSKILL.mdfiles get automatic feedback:**/SKILL.md, the workflow runstesslio/skill-reviewand posts one comment with Tessl scores and feedback (updated on new pushes).GITHUB_TOKENis used to post the comment.fail-thresholdis set.with: fail-threshold: 70(or any score) to the workflow step.SKILL.mdgets automatic review comments, so the rest of the library improves incrementally.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 🙏