diff --git a/.github/workflows/pr-check.yml b/.github/workflows/pr-check.yml index 99fbc52..04aef81 100644 --- a/.github/workflows/pr-check.yml +++ b/.github/workflows/pr-check.yml @@ -15,18 +15,14 @@ jobs: const body = context.payload.pull_request.body || ''; const hasClawhubLink = /https:\/\/clawhub\.ai\/[\w-]+\/[\w-]+/.test(body); - const hasGithubLink = /https:\/\/github\.com\/openclaw\/skills\/tree\/main\/skills\/[\w-]+\/[\w-]+/.test(body); const errors = []; if (!hasClawhubLink) { errors.push('Missing ClawHub link (e.g. https://clawhub.ai/steipete/slack)'); } - if (!hasGithubLink) { - errors.push('Missing GitHub link (e.g. https://github.com/openclaw/skills/tree/main/skills/steipete/slack)'); - } if (errors.length > 0) { - const message = `## ❌ PR Description Check Failed\n\nYour PR description must include both links:\n\n${errors.map(e => '- ' + e).join('\n')}\n\nSee [CONTRIBUTING.md](../blob/main/CONTRIBUTING.md#pr-description) for details.`; + const message = `## ❌ PR Description Check Failed\n\nYour PR description must include the required link:\n\n${errors.map(e => '- ' + e).join('\n')}\n\nThe skill must also be published on ClawHub with passing tests and a clean (non-suspicious) security status.\n\nSee [CONTRIBUTING.md](../blob/main/CONTRIBUTING.md#pr-description) for details.`; await github.rest.issues.createComment({ owner: context.repo.owner, @@ -35,5 +31,5 @@ jobs: body: message }); - core.setFailed('PR description is missing required links.'); + core.setFailed('PR description is missing the required ClawHub link.'); } diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index d185c58..77fb3e2 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -24,6 +24,7 @@ If an author has multiple skills in the same area, please don't add them one by ### Requirements - **Skill must already be published on [ClawHub](https://clawhub.ai), OpenClaw's public skills registry.** We do not accept skills hosted elsewhere — no personal repos, no gists, no external links. If it's not on ClawHub, it doesn't belong here. +- **The skill's tests on ClawHub must be passing**, and its security status must be clean (not flagged as suspicious). - Has documentation (SKILL.md) - Description must be concise — 10 words or fewer - Skill must have real community usage. We focus on community-adopted, proven skills published by development teams and proven in real-world usage. Brand new skills are not accepted — give your skill time to mature and gain users before submitting