diff --git a/.github/workflows/tessl-skill-review.yml b/.github/workflows/tessl-skill-review.yml index aac8eb7b..0c919f7e 100644 --- a/.github/workflows/tessl-skill-review.yml +++ b/.github/workflows/tessl-skill-review.yml @@ -1,17 +1,25 @@ name: Tessl Skill Review # Reviews changed SKILL.md files on PRs using the official tesslio/skill-review -# action, which runs `tessl skill review` and posts the quality scores as a PR -# comment. The action scans the PR diff for changed SKILL.md files (as +# action, which runs `tessl review run quality` and posts the quality scores as a +# PR comment. The action scans the PR diff for changed SKILL.md files (as # repo-relative paths) under the `path` root, so `path` is the repo root (`.`) # and every plugin's skills are covered — not just one. (Passing a sub-directory # as `path` double-prefixes the repo-relative changed paths and fails with # "SKILL.md not found at plugins//plugins//...".) # -# Auth: the skill-review composite runs tesslio/setup-tessl WITHOUT a token, and -# the tessl CLI authenticates from the TESSL_TOKEN env var — so it is provided at -# job level here. When the secret is absent the job no-ops with a warning instead -# of failing the PR. +# Auth: reviews are authenticated — the unauthenticated path upstream is closed. +# The token goes in via the action's `tessl-token` input (the composite forwards +# it to tesslio/setup-tessl and to the CLI env); a job-level TESSL_TOKEN env var +# is no longer picked up, because the composite sets its own step-level +# TESSL_TOKEN from the input. TESSL_TOKEN is an org secret shared with the Tessl +# publish workflow. When it is absent the job no-ops with a warning instead of +# failing the PR. +# +# Workspace: reviews run against one Tessl workspace, which the CLI resolves from +# the token when the token can see exactly one. If the token can see several the +# check fails asking for a workspace — add `workspace: ` (see +# `tessl workspace list`) when that happens. # # Pinning: skill-review has no tagged release yet, so it is pinned to a main # commit SHA. Bump the SHA to adopt upstream changes. @@ -29,11 +37,11 @@ jobs: review: name: Review changed skills runs-on: ubuntu-latest - env: - TESSL_TOKEN: ${{ secrets.TESSL_TOKEN }} steps: - name: Check TESSL_TOKEN is configured id: guard + env: + TESSL_TOKEN: ${{ secrets.TESSL_TOKEN }} run: | if [ -z "$TESSL_TOKEN" ]; then echo "::warning::TESSL_TOKEN not configured — skipping Tessl skill review" @@ -48,8 +56,9 @@ jobs: - name: Review changed skills if: steps.guard.outputs.skip != 'true' - uses: tesslio/skill-review@22e928dd837202b2b1d1397e0114c92e0fae5ead # main (no tagged release) + uses: tesslio/skill-review@7e7ea574722a6d72752aa5a316bcc813d06b7cd2 # main (no tagged release) with: + tessl-token: ${{ secrets.TESSL_TOKEN }} path: '.' comment: 'true' fail-threshold: '0'