chore: add constrained Crabbox setup#264
Conversation
|
Codex review: needs changes before merge. Latest ClawSweeper review: 2026-05-22 21:45 UTC / May 22, 2026, 5:45 PM ET. Workflow note: Future ClawSweeper reviews update this same comment in place. How this review workflow works
Summary Reproducibility: not applicable. as a user bug. The review finding is source-reproducible: the new workflow uses mutable action refs at the changed lines while current main pins comparable actions to SHAs. PR rating Rank-up moves:
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. Real behavior proof Risk before merge
Maintainer options:
Copy recommended automerge instructionNext step before merge Security Review findings
Review detailsBest possible solution: Pin the new workflow actions to immutable SHAs with version comments, then validate the constrained Crabbox hydrate path on the intended runner labels before merge. Do we have a high-confidence way to reproduce the issue? Not applicable as a user bug. The review finding is source-reproducible: the new workflow uses mutable action refs at the changed lines while current main pins comparable actions to SHAs. Is this the best way to solve the issue? No. The constrained Crabbox setup is a plausible direction, but it should follow the existing pinned-action security pattern and receive operator smoke proof before merge. Label justifications:
Full review comments:
Overall correctness: patch is incorrect Security concerns:
Acceptance criteria:
What I checked:
Likely related people:
Codex review notes: model gpt-5.5, reasoning high; reviewed against 62cf3c3163aa. |
|
ClawSweeper PR egg 🔥 Warming up: real-behavior proof passed; findings, security review, or rank-up moves are still in progress. Hatch commandComment Hatchability rules:
What is this egg doing here?
|
Summary
openclaw/openclaw.package.json.This is the narrowed replacement shape for the earlier broad setup baseline. It intentionally does not add CodeQL, stale automation, licensing changes, Dependabot, package-manager files, or unrelated policy defaults.
Verification
git diff --check.crabbox.yaml,.github/actionlint.yaml, and.github/workflows/crabbox-hydrate.ymlactionlint -config-file .github/actionlint.yaml .github/workflows/crabbox-hydrate.ymlopenclaw/openclaw:ed512c0b0385fae7f6c5c14a7e9e6236ab68936506687a99ca976873492bdc43package.jsonexiststest -z "$(gofmt -l .)"for Go reposNotes
No live Crabbox lease was started for this setup-only patch.