feat(ci): publish signed Gateway MSIX releases - #33
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: blocked before merge. Reviewed September 12, 2026, 3:36 AM ET / 07:36 UTC. ClawSweeper reviewWhat this changesThe PR bundles x64 and ARM64 Windows packages, validates and signs them, and publishes permanent GitHub Release downloads using explicit release-policy versions. Merge readiness⛔ Blocked before merge - 4 items remain Keep open: permanent signed releases remain absent from main, and the prerequisite signing setup has merged. This member-authored successor contains useful, distinct work. Priority: P2 Review scores
Verification
How this fits togetherThis repository packages OpenClaw Gateway for Windows. Its workflow turns an approved upstream revision into installation packages, validates their contents, and passes authorized artifacts through signing to downloadable releases. flowchart TD
A[Approved Gateway revision] --> B[x64 and ARM64 packages]
B --> C[Combined installation bundle]
B --> D[Release policy and content validation]
C --> D
D --> E[Protected Azure signing]
E --> F[GitHub Release downloads]
F --> G[Windows installation]
Decision needed
Why: The existing discussion schedules verification after the first official run, while this workflow publishes automatically; choosing the acceptable exposure and test-install transition requires release-owner intent. Before merge
Agent review detailsSecurityNone. Review metrics
Root-cause clusterRelationship: Members:
Proposal only: this assessment does not dispatch repair, suppress jobs, mutate sibling items, close, or merge anything. Merge-risk optionsMaintainer options:
Technical reviewBest possible solution: Stage the initial signed assets as a draft, then promote verified Windows downloads with an explicit transition policy for existing test installations. Do we have a high-confidence way to reproduce the issue? Not applicable: this adds release distribution, and source review established no discrete introduced runtime defect; Windows installation was not executed. Is this the best way to solve the issue? Yes, extending the existing packaging pipeline is appropriate; separating initial signing from public promotion would better support the documented first-run validation plan. AGENTS.md: not found in the target repository. Codex review notes: model internal, reasoning medium; reviewed against cc1d83318281. LabelsLabel changes:
Label justifications:
EvidenceWhat I checked:
Likely related people:
Rank-up movesOptional improvements that raise the rating; they are not merge blockers.
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
|
|
Owner decision: publish the initial v0.0.0.0 signing proof immediately after merge. This is intentionally a plumbing/signing validation release, not an upgrade-compatible production version. Existing higher-version unsigned or test-signed installations may require uninstall before installing it; there is no prior official release to upgrade from. Exact-head CI passed standalone x64/ARM64 MakeAppx composition and bundle composition. @clawsweeper re-review |
Summary
Validation
Supersedes closed stacked PR #32 after #24 merged.