ci: Tessl skill review for SKILL.md pull requests#74
Open
rohan-tessl wants to merge 1 commit intopbakaus:mainfrom
Open
ci: Tessl skill review for SKILL.md pull requests#74rohan-tessl wants to merge 1 commit intopbakaus:mainfrom
rohan-tessl wants to merge 1 commit intopbakaus:mainfrom
Conversation
Use tesslio/skill-review to run tessl skill review on changed skills and post results on the PR. No Tessl account required; only GITHUB_TOKEN. Made-with: Cursor
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 @pbakaus 👋
Impeccable stands out because you treat cross-harness skills as a real engineering problem — one source of truth, real transforms, and docs that match how people ship. I’m grateful you merged #55; the review was exactly the kind of sharp, fair maintainer signal that makes outside contributions land well (frontmatter detail, keeping PR-only files out of the tree).
Your note afterward stuck with me: you have your own pipeline to compile skills across providers, and it’s painful. This workflow is not trying to replace that. It doesn’t touch
bun run build, transformers, ordist/— it only adds an optional, lightweight step for reviewing skill content on PRs (the sametessl skill reviewidea you said Tessl looks “super needed” for), so contributors get concrete feedback without you having to run it by hand every time.What this adds
A single file:
.github/workflows/skill-review.yml.mainthat change any**/SKILL.md(coverssource/skills/**and other trees if those files appear in a diff).tesslio/skill-reviewinstalls the Tessl CLI, runstessl skill reviewon changed skills only, and posts one PR comment (updated on new pushes — no spam).GITHUB_TOKENis used to post that comment — standard for Actions that comment on PRs.fail-thresholdis off unless you add it). So it’s feedback-first; you can turn on a gate later if you like.Why a separate workflow (not stuffed into
ci.yml)Path filtering keeps this job from running on unrelated PRs. If you’d rather fold it into
ci.ymlor narrow paths (e.g. onlysource/skills/**/SKILL.md), I’m happy to adjust in a follow-up commit.Optional: quality gate later
If you want PRs to fail when a skill regresses:
Totally your call — leaving it out keeps the first merge low-risk.
Honest disclosure: I work at @tesslio on tooling around agent skills. This isn’t a land grab for your build — it’s a small automation that matches what you already found useful in conversation, and you can rip it out or fork the workflow anytime.
Thanks again for Impeccable — it’s a reference project in this space. 🙏