feat: publish Crabbox sandbox quickstart for skills.sh - #1911
Conversation
|
🦞👀 Pull request received. I will update this pull request when review starts. ClawSweeper review completeClawSweeper finished reviewing this revision. The review result is being finalized. |
|
Codex review: needs maintainer review before merge. Reviewed September 7, 2026, 1:11 AM ET / 05:11 UTC. ClawSweeper reviewWhat this changesAdds a beginner local-container skill and updates website publishing, discovery checks, installation guidance, and browser proof to support both Crabbox skills. Merge readiness✅ Ready for maintainer review This PR remains useful: current main and v0.51.0 publish only the existing crabbox skill. The earlier cleanup, layout, ordering, and catalog-validation concerns are resolved, and the refreshed proof supports the changed publishing and installation guide. Priority: P3 Review scores
Verification
How this fits togetherCrabbox’s documentation builder turns repository skill files into downloadable instructions and discovery catalogs. Skill installers consume those files to teach coding agents how to run Crabbox. flowchart LR
A[Canonical skill files] --> B[Validate metadata and copies]
B --> C[Documentation builder]
C --> D[Installation guide]
C --> E[Discovery catalog and downloads]
E --> F[Agent skill installers]
F --> G[Crabbox usage instructions]
Before mergeNone. Agent review detailsSecurityNone. Review metrics
Technical reviewBest possible solution: Publish the bounded quickstart through the existing discovery pipeline while preserving the full skill’s payload and directing advanced tasks back to it. Do we have a high-confidence way to reproduce the issue? Not applicable to the central skill-publication feature; the earlier layout defect is visibly resolved in the refreshed desktop and mobile screenshots. Is this the best way to solve the issue? Yes. Extending the existing publisher preserves one discovery implementation, while the short quickstart delegates advanced usage to the unchanged full skill. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning medium; reviewed against e59130d1999a. LabelsLabel justifications:
EvidenceWhat I checked:
Likely related people:
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
HistoryReview history (6 earlier review cycles)
|
a8bbed2 to
21d18e3
Compare
|
The quickstart rewrite is now on The replacement also dropped three mechanical fixes that are independent of the skill's subject:
Please retain or deliberately replace those fixes/tests while keeping the quickstart direction. The generated-test comment is also stale: importing the builder already generates the site before test registration. I am continuing the separate provider work rather than force-pushing over this revision. |
21d18e3 to
ca69f5c
Compare
ca69f5c to
952ed2b
Compare
952ed2b to
0b0daa7
Compare
Adds a short getting-started skill next to the existing `crabbox` reference, and generalizes skill publishing from one hardcoded skill to every `skills/` dir. - `skills/crabbox-quickstart/SKILL.md` (153 lines) plus its byte-identical `.agents/` projection: install, one zero-config run on a local container, the bare base image, warm reuse, sync, errors, and when to escalate. - Docs-site builder, drift checker, and Pages filter iterate every skill. - `skills/crabbox/SKILL.md` is unchanged; its digest and listing untouched. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
0b0daa7 to
fc51d7e
Compare
Make Crabbox's sandbox execution skills easy to install with the skills.sh CLI. The existing
crabboxskill covers sandbox execution, reusable remote environments and evidence; the newcrabbox-quickstartteaches a first credential-free Docker/Podman run and cleanup.The quickstart command becomes available from upstream after merge. Skills install agent instructions; the Crabbox CLI is installed separately. As skills.sh documents, leaderboard discovery comes from CLI installation telemetry. This PR does not claim the unmerged quickstart is already indexed.
Changes
warmupand--keepneed explicitstop.crabboxas the first discovery entry even with an alphabetically earlier skill; reject missing/invalid catalog names and string arrays.The existing
crabboxskill is unchanged. Changelog edits remain with maintainers. No configuration, credential or provider-runtime changes.Verification of 16ea048
git diff --checkpassed.check-docs.shcould not run its command-help checks locally because Go is unavailable. Earlier Docker walkthrough evidence is recorded in the PR history; it was not rerun for this documentation/publishing correction. Other CI jobs are still running.Proof report · Browser assertions