fix(release): make ClawHub publish best effort#2440
Conversation
|
Status update: checks are green and the diff is limited to .github/workflows/release.yml. The �0.6.6-preview release is published for the critical channels, but the workflow is red only because ClawHub rejected slug �egis as already owned by /pleasechooseusername/aegis. This PR makes that optional channel best-effort so future releases do not fail on this external registry ownership issue. |
Do not fail the whole release when optional ClawHub publishing rejects the existing slug or has an external registry issue. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
987da60 to
c8cf1bb
Compare
There was a problem hiding this comment.
👁️ Argus Review — PR #2440
⚠️ BLOCKING: Targets `main`
Per project policy, all agent PRs target `develop`. This PR targets `main`.
Exception pathway: If this is an authorized hotfix, I need explicit confirmation from Ema (OneStepAt4time) before I can approve.
Diff Review (4 additions, 1 deletion)
What it does: Makes ClawHub login/publish best-effort so a slug ownership failure doesn't block the entire release workflow. Also renames slug `aegis` → `onestep-aegis`.
Findings:
- ✅ Correct approach — `continue-on-error: true` is the right pattern for optional publish channels
- ✅ Slug rename addresses the root cause of the v0.6.6-preview failure
- ✅ `set -euo pipefail` inside the publish step is good hygiene — ensures partial execution fails cleanly
⚠️ Commit hygiene: Commit co-author shows `test@example.com` (identity: "Aegis Test"). This should not appear in production commits targeting `main`. Please amend with correct git identity.
Gates Status
| Gate | Status |
|---|---|
| Targets `develop` | ❌ BLOCKING — targets `main` |
| CI green | ⏳ Pending |
| Review completed | ✅ Diff reviewed |
| Security clean | ✅ No secrets, workflow-only change |
| Tests | ⏳ CI pending |
Verdict
Changes requested — pending Ema's authorization for `main` target + commit hygiene fix.
Once Ema confirms this is an authorized hotfix and the commit author is cleaned up, I'll re-review and approve if CI is green.
|
Closing and replacing with a clean branch because the commit author identity is wrong (Aegis Test test@example.com). The replacement PR will keep the same scoped workflow change with proper commit identity. |
Summary
continue-on-errorv0.6.6-previewrelease failure where ClawHub rejected slugaegisas already owned by/pleasechooseusername/aegisValidation
npx --yes js-yaml .github/workflows/release.ymlactionlint .github/workflows/release.ymlnpm run hygiene-checknpm run security-checknpm run dashboard:tokens:gatenpm run dashboard:clickable:gatenpx tsc --noEmitnpm run buildnpx vitest run src/__tests__/server-core-coverage.test.ts src/__tests__/mcp-integration-smoke-1898.test.tsnpm testRelease context
v0.6.6-previewpublished all critical artifacts successfully, but the release workflow is red becausepublish-clawhubfailed with an external slug ownership error. This hotfix prevents that optional channel from making future release runs fail.