docs: group skills for skills.sh#12
Conversation
|
Codex review: needs maintainer review before merge. Reviewed June 13, 2026, 5:35 PM ET / 21:35 UTC. Summary Reproducibility: not applicable. This PR adds repository display metadata rather than fixing a reproducible bug. Source checks verify the manifest inventory, and the PR body now supplies real consumer proof for the synthetic merge. Review metrics: 2 noteworthy metrics.
Merge readiness Overall follows the weaker of proof and patch quality, so missing proof can cap an otherwise strong patch. Rank-up moves:
Risk before merge
Maintainer options:
Next step before merge
Security Review detailsBest possible solution: Land this root manifest after Peter approves or edits the six section names and memberships, preserving the current-inventory-only approach unless legacy aliases should be encoded. Do we have a high-confidence way to reproduce the issue? Not applicable; this PR adds repository display metadata rather than fixing a reproducible bug. Source checks verify the manifest inventory, and the PR body now supplies real consumer proof for the synthetic merge. Is this the best way to solve the issue? Yes, conditionally: a root AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against a3026aea93b9. Label changesLabel changes:
Label justifications:
Evidence reviewedWhat I checked:
Likely related people:
What the crustacean ranks mean
Shiny media proof means a screenshot, video, or linked artifact directly shows the changed behavior. Runtime, network, CSP, and security claims still need visible diagnostics. How this review workflow works
|
78c65bb to
0323c09
Compare
|
@clawsweeper re-review The PR body now contains exact 49-skill inventory results, current |
|
🦞🧹 I asked ClawSweeper to review this item again. Re-review progress:
|
Adds a root
skills.sh.jsonso the skills.sh repository page can render this catalog in six curated sections instead of one install-sorted list. ClawHub's GitHub-backed publisher catalog uses the same manifest shape when a repository is configured as a source.Scope:
skills/*/SKILL.mdslugsmaintainer-orchestratorskill underGitHub & OpenClawnotGroupedatbottomso future unlisted skills remain visibleskillsCLI does not discover themConsumer proof:
npx -y skills@1.5.11 add steipete/agent-scripts --listcloned the public repository and found 49 skills, includingmaintainer-orchestrator.origin/mainplus this manifest) passed the same CLI path with 49 skills.e86aa30a77aa130bfdc10e65885d0623cc1db4f2accepted the synthetic merge throughbuildGitHubSkillSourceSnapshotandbuildGitHubSkillCatalogDisplay: manifest statusok, 49 discovered, section counts6/9/9/6/9/10, 49 rendered, zeroOther skillsentries.Validation:
uvx check-jsonschema --schemafile https://skills.sh/schemas/skills.sh.schema.json skills.sh.jsonscripts/validate-skillshooks/pre-commit.github/workflows/ci.ymlgit diff --check origin/main...HEADOwner product choice:
1password,brave-search,codex-review,parallels-vm,sonos-debug). This manifest intentionally describes the canonical current 49-skill inventory; those historical records may remain underOther skills. Encoding stale telemetry aliases in repository metadata is not recommended.