Skip to content

feat: optimize 5 skill scores from 0% to 90-94%#2

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

feat: optimize 5 skill scores from 0% to 90-94%#2
rohan-tessl wants to merge 2 commits into
manutej:mainfrom
rohan-tessl:improve/skill-review-optimization

Conversation

@rohan-tessl

@rohan-tessl rohan-tessl commented Mar 26, 2026

Copy link
Copy Markdown

Hey @Sir-chawakorn 👋

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
ai-pair-programming 0% 84% +84%
auto-debug 0% 89% +89%
code-explanation 0% 92% +92%
commit-message 0% 94% +94%
pr-review 0% 86% +86%

All five skills were scoring 0% because their YAML frontmatter was placed after the H1 heading instead of at the very start of the file — a formatting issue that blocked the Tessl judge from evaluating them at all. This PR fixes that and also improves each skill's description and content.

This PR intentionally caps changes at five skills to keep it reviewable. The included GitHub Action workflow (see below) will surface Tessl feedback on future SKILL.md changes automatically, so the rest of the library can improve incrementally.

Changes summary

All five skills:

  • Moved YAML frontmatter (--- block) to the very beginning of the file (was incorrectly placed after the H1 heading)
  • Rewrote descriptions to include specific "Use when..." trigger clauses with concrete actions
  • Used quoted-string format for descriptions (Tessl best practice)
  • Streamlined content: removed emoji clutter, tightened prose, improved workflow clarity
  • Preserved all domain-specific knowledge and patterns
  • Ensured third-person voice and actionable structure throughout

Skill-specific improvements:

  • ai-pair-programming: Clarified the three collaboration modes (driver-navigator, ping-pong, AI-first) with concrete workflows; added communication and code quality best practices table
  • auto-debug: Consolidated error pattern tables into a single reference section; added clear 5-step debug workflow with iteration loop
  • code-explanation: Structured the three explanation levels (overview, step-by-step, line-by-line) with concrete examples; added explanation workflow with verification step
  • commit-message: Consolidated commit types into one reference table; added generation workflow from diff analysis to final message
  • pr-review: Organized five review dimensions (quality, correctness, security, performance, testing) with specific "what to look for" guidance; added structured summary template

Tessl Skill Review GitHub Action

This PR also adds .github/workflows/skill-review.yml — a lightweight GitHub Action that automatically reviews SKILL.md files on future PRs:

  • What runs: On PRs that change **/SKILL.md, the workflow runs tesslio/skill-review and posts one PR comment with Tessl scores and feedback (updated on new pushes).
  • Zero extra accounts: Contributors do not need a Tessl login — only the default GITHUB_TOKEN is used to post the comment.
  • Non-blocking by default: The check is feedback-only — no surprise red CI. You can add with: fail-threshold: 70 later if you want a hard gate.
  • Not a build replacement: This is review automation for skill markdown only, not a substitute for any compile/transform pipeline.
  • Why only five skills edited here: This PR caps manual optimization so it stays reviewable. After merge, every PR that touches SKILL.md gets automatic review comments — so the remaining 45+ skills can improve 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 🙏

Fix validation errors (non-kebab-case names, unknown frontmatter keys,
reserved words) and condense content across 5 skills:

| Skill | Before | After | Change |
|-------|--------|-------|--------|
| fastapi-customer-support | 0% | 94% | +94% |
| postgresql-database-administration | 0% | 94% | +94% |
| psycopg-database-adapter | 0% | 94% | +94% |
| sqlalchemy-orm-patterns | 0% | 94% | +94% |
| ai-sdk-integration-patterns | 0% | 90% | +90% |

Changes per skill:
- Fixed name fields to kebab-case (removed spaces, special chars)
- Removed unknown frontmatter keys (tags, capabilities, version, author, etc.)
- Renamed claude-sdk skill to avoid reserved word in name field
- Fixed psycopg skill: changed "skill:" key to "name:"
- Condensed all skills from 590-1472 lines to 162-204 lines
- Restructured content with actionable code patterns, removed tutorial prose
- Added "Use when..." trigger clauses to all descriptions
- Referenced EXAMPLES.md for extended content where available
Adds .github/workflows/skill-review.yml that automatically reviews
any SKILL.md changes on pull requests targeting main. Uses
tesslio/skill-review action to post scores as a PR comment.
No Tessl account needed — uses only GITHUB_TOKEN.
Non-blocking by default (feedback-only, no fail threshold).
@rohan-tessl rohan-tessl marked this pull request as ready for review March 26, 2026 07:25
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