Skip to content

fix(release): restore frozen validation startup#110210

Merged
RomneyDa merged 2 commits into
extended-stable/2026.6.33from
dev/extended-stable-2026.6.33-v6.11-repair3
Jul 17, 2026
Merged

fix(release): restore frozen validation startup#110210
RomneyDa merged 2 commits into
extended-stable/2026.6.33from
dev/extended-stable-2026.6.33-v6.11-repair3

Conversation

@RomneyDa

@RomneyDa RomneyDa commented Jul 17, 2026

Copy link
Copy Markdown
Member

What Problem This Solves

Full Release Validation run https://github.com/openclaw/openclaw/actions/runs/29617653209 correctly executed from the canonical extended-stable/2026.6.33 branch, but GitHub rejected two child workflows before any jobs ran:

  • Plugin Prerelease 29617672669: nested push_docker_e2e_images and push_live_test_image requested packages: write through a read-only caller.
  • Release Checks 29617674873: nested Install Smoke push_root_dockerfile_image requested packages: write through a read-only caller.

After resolving the startup errors, focused PR-head probing also showed the frozen Release Checks trust guard omitted the canonical extended-stable/YYYY.M.33 workflow branch.

This restores the complete canonical-branch startup contract without granting publication permissions.

Why This Change Was Made

Upstream PR #103834 / b597a8d364f2849c704b2d0087a9c6c087def2f7 fixed the permission contract by separating image publication from no-write validation. The v6.11 candidate carried the preceding no-write work but missed that follow-up. Upstream PR #99352 / c7810fc6972c0152cb5b5c473011bb8d8aefee0b added the canonical extended-stable Release Checks allowance; the candidate carried the surrounding publisher work but omitted those guard lines.

This is the narrow 6.x adaptation:

  • remove the three validation-owned image publisher jobs;
  • default shared/root images to immutable artifact transport;
  • retain read-only reuse of existing registry images;
  • remove validation-time packages: write permissions;
  • authorize only the canonical extended-stable/YYYY.M.33 Release Checks ref shape;
  • update focused workflow contract tests.

The broad upstream installer refactor and separate Docker publisher workflows are intentionally omitted. Extended-stable 2026.6.33 is npm-only and must not publish Docker images.

User Impact

No OpenClaw runtime or package contents change. This only allows the frozen candidate's own release workflows to start with least-privilege permissions.

Release scope remains v2026.6.33, npm extended-stable, core openclaw plus the canonical all-publishable official plugin inventory at exact version 2026.6.33. GitHub Releases, native apps, Docker images, ClawHub, mobile, and website publication remain excluded. No tag or publication has occurred.

Security impact is positive: validation no longer contains nested jobs capable of obtaining packages: write; the added branch allowance is exact and month-bounded. Rollback is a revert, which would restore the deterministic startup/trust failures.

Evidence

After merge, npm preflight and Full Release Validation will be rerun from the canonical branch at its new exact head. Historical runs will not be reused.

Adapt the no-write validation portion of b597a8d (#103834) for the 6.x extended-stable workflow set. Remove validation-owned image publisher jobs instead of granting write permissions; Docker publication remains out of scope.
@openclaw-barnacle openclaw-barnacle Bot added size: M maintainer Maintainer-authored PR labels Jul 17, 2026
Restore the openclaw-release-checks.yml portion of c7810fc (#99352) that authorizes the canonical extended-stable/YYYY.M.33 workflow branch.
@clawsweeper clawsweeper Bot added rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR. labels Jul 17, 2026
@RomneyDa

Copy link
Copy Markdown
Member Author

Land-ready proof for exact head e3eb01a3af7898cea2b6f36b8ade7f9bb60290b1:

No runtime/package/dependency/version change. No tag or publication. After merge, fresh npm preflight and Full Release Validation will bind to the new canonical head.

@RomneyDa
RomneyDa merged commit 3341ade into extended-stable/2026.6.33 Jul 17, 2026
99 of 101 checks passed
@RomneyDa
RomneyDa deleted the dev/extended-stable-2026.6.33-v6.11-repair3 branch July 17, 2026 22:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer Maintainer-authored PR rating: 🐚 platinum hermit Good normal PR readiness with ordinary maintainer review expected. size: M status: 👀 ready for maintainer look ClawSweeper has no concrete contributor-facing blocker left for this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant