Skip to content

feat: improve skill scores and add Tessl review workflow#2

Open
rohan-tessl wants to merge 2 commits into
artmsilva:mainfrom
rohan-tessl:improve/skill-review-optimization
Open

feat: improve skill scores and add Tessl review workflow#2
rohan-tessl wants to merge 2 commits into
artmsilva:mainfrom
rohan-tessl:improve/skill-review-optimization

Conversation

@rohan-tessl

Copy link
Copy Markdown

Hey @artmsilva 👋

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
lit-best-practices 78% 93% +15%

This PR covers your 1 skill in the repo. I capped changes at five skills per PR to keep review manageable — the included GitHub Action covers ongoing review for any future skill edits.

What changed in lit-best-practices
  • Description rewritten with specific concrete actions (reactive property declaration, template rendering, CSS encapsulation, lifecycle callback ordering, custom event composition, ARIA accessibility, rendering performance optimization) and explicit "Use when..." clause
  • Added inline code examples for all 5 CRITICAL rules directly in SKILL.md — pure render(), static styles, composed events, super() call order, and ARIA attributes — so the skill is immediately actionable without loading external files
  • Added "How to Apply These Rules" workflow section with task-based guidance (reviewing, writing, optimizing, auditing)
  • Added verification checklist for post-writing/refactoring validation
  • Added 1-2 line actionable summaries to every rule in the index so agents get concrete guidance at a glance
  • Removed redundant "When to Use" section (duplicated the frontmatter description)
  • Removed external Resources section (Claude and other LLMs cannot browse URLs)

Tessl Skill Review GitHub Action ✅

I've also included a GitHub Action (.github/workflows/skill-review.yml) that automatically reviews any SKILL.md changed in future PRs and posts scores as a PR comment.

What this gives you:

  • 🔍 Automatic tessl skill review runs on every PR touching SKILL.md
  • 💬 One updated PR comment with scores and improvement feedback
  • 🔓 Zero extra accounts — contributors don't need a Tessl login; only GITHUB_TOKEN is used
  • Non-blocking by default — feedback-only, no surprise red CI (add fail-threshold: 70 later if you want a hard gate)
  • 📈 Covers future skills incrementally as contributors edit them

Want automatic AI optimization on every SKILL.md change? 🚀

The action I've added gives you review scores on PRs. We also have a more powerful variant — tesslio/skill-review-and-optimize — that can:

  • Run AI-powered optimization suggestions on every SKILL.md PR (requires adding TESSL_API_TOKEN as a repo secret)
  • Let contributors accept suggested improvements by commenting /apply-optimize
  • Still works in review-only mode with zero secrets

Interested? Tick the box and I'll raise a follow-up PR:

  • Yes please! Add the tesslio/skill-review-and-optimize action so every SKILL.md PR gets AI optimization suggestions + the /apply-optimize flow
  • No thanks — the review scores action is enough for now

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 🙏

| Skill | Before | After | Change |
|-------|--------|-------|--------|
| lit-best-practices | 78% | 93% | +15% |

Key changes:
- Description rewritten with specific concrete actions (reactive property
  declaration, template rendering, CSS encapsulation, lifecycle ordering,
  event composition, ARIA accessibility, performance optimization)
- Added inline code examples for all 5 CRITICAL rules (pure render,
  static styles, composed events, super() order, ARIA attributes)
- Added task-based workflow section ("How to Apply These Rules")
- Added verification checklist for post-writing/refactoring validation
- Removed redundant "When to Use" section (duplicated frontmatter)
- Removed external Resources section (Claude cannot browse URLs)
- Added 1-2 line actionable summaries to every rule in the index
Add .github/workflows/skill-review.yml that runs `tessl skill review`
on every PR touching SKILL.md and posts scores as a PR comment.
No Tessl login or API token needed — uses only GITHUB_TOKEN.
Non-blocking by default (fail-threshold: 0).
@rohan-tessl rohan-tessl marked this pull request as ready for review April 27, 2026 09:30
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