feat: add automated skill review GitHub Action#523
feat: add automated skill review GitHub Action#523rohan-tessl wants to merge 1 commit intoFritzAndFriends:mainfrom
Conversation
Hey @csharpfritz 👋 Following up on FritzAndFriends#501 — you mentioned wanting to do an inventory of all the skills in this project and keep them current. This adds a GitHub Action that automatically reviews any `SKILL.md` file changed in a PR, so you get quality feedback without having to remember to run it manually. ## What it does Any PR that touches a `SKILL.md` file will automatically get a review comment with scores and detailed feedback from `tessl skill review`. No Tessl account or API token needed — it runs the CLI locally and only uses the GitHub-provided `GITHUB_TOKEN` to post comments. ## Optional: quality gate If you want to enforce a minimum score, you can add `fail-threshold`: ```yaml - uses: tesslio/skill-review@main with: fail-threshold: 70 ``` PRs with any skill scoring below 70% would fail the check. The default is `0` (never fails — just informational). 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 - [@rohan-tessl](https://github.com/rohan-tessl) - if you hit any snags. Thanks in advance 🙏
|
That's very kind! I'll review and discuss this PR on my Thursday livestream. |
|
I have some concerns about you making changes to my skills and getting read and write access to the repository with a container that I'm not familiar with. |
Thanks for flagging that it's a fair concern and worth addressing clearly. Just to clear up the container point first: the action doesn't use Docker or a container at all. It's a standard GitHub Action that runs directly on the runner, downloads the Tessl CLI binary, runs On the write access question - the permissions:
pull-requests: write
contents: readNo signup or tokens required for the core functionality. Also, I'd be happy to join your Thursday livestream, walk through how it works, and chat about the broader idea of improving skills and steering agents - could make for a useful segment if that feels relevant. Just say the word! |
|
hey @csharpfritz 👋 i'm bap - colleague of @rohan-tessl's. caught your livestream review of #501, really appreciated how thoroughly you walked through the changes! on the security concern here - Rohan addressed it above but the tldr: no container, no write access to code. the action runs the CLI on the runner and posts a PR comment via the standard GITHUB_TOKEN. happy to clarify anything else. separately, we ran some scenario-based task evals on your
these are scenarios we put together, you can see the full evals results here. This means that if you want to create your own evaluation scenarios to pressure-test your skills against the tasks that matter most to you and your contributors, these are the kinds of tests you can run. no action needed - just thought you'd find it useful. |
Hey @csharpfritz 👋
Following up on #501 — you mentioned wanting to do an inventory of all the skills in this project and keep them current. This adds a GitHub Action that automatically reviews any
SKILL.mdfile changed in a PR, so you get quality feedback without having to remember to run it manually.What it does
Any PR that touches a
SKILL.mdfile will automatically get a review comment with scores and detailed feedback fromtessl skill review. No Tessl account or API token needed — it runs the CLI locally and only uses the GitHub-providedGITHUB_TOKENto post comments.optionally, you can enable optimize mode by adding a token as TESSL_API_TOKEN in your repo secrets. when enabled, the action suggests improvements you can accept by commenting /apply-optimize. this means that it gives you and your contributors an instant quality signal before you have to review yourself.
Example output
When a PR modifies a skill, you'll see a comment like:
Optional: quality gate
If you want to enforce a minimum score, you can add
fail-threshold:PRs with any skill scoring below 70% would fail the check. The default is
0(never fails — just informational).Changes
.github/workflows/skill-review.yml— a single lightweight workflow fileHonest 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 🙏