chore(deps): refresh Windows packaging toolchain - #11
Conversation
|
🦞👀 Pull request received. I will update this pull request when review starts. |
|
Codex review: needs real behavior proof before merge. Reviewed August 31, 2026, 7:14 AM ET / 11:14 UTC. ClawSweeper reviewWhat this changesRefreshes the Windows MSIX packaging toolchain versions and adds an x64 packaged-launcher smoke check to the build workflow. Merge readiness⛔ Blocked until stronger real behavior proof is added - 3 items remain Keep open: this is a coherent, narrowly scoped packaging-toolchain refresh, but end-to-end Windows MSIX behavior with the refreshed runtime is not yet evidenced. Priority: P3 Review scores
Verification
How this fits togetherThe packaging workflow builds OpenClaw into an npm payload, combines it with a bundled Node.js runtime and NativeAOT launcher, then emits Windows MSIX packages. The refreshed toolchain affects both the CI build path and the runtime included in installed packages. flowchart TD
A[OpenClaw source revision] --> B[Build npm package]
C[Node and pnpm toolchain] --> B
B --> D[Windows payload]
E[.NET MSIX host] --> F[Unsigned MSIX package]
D --> F
F --> G[Gateway launcher]
Before merge
Agent review detailsSecurityNone. Review metrics
Merge-risk optionsMaintainer options:
Technical reviewBest possible solution: Merge after the Windows workflow completes successfully and an end-to-end packaged-launcher result is attached as redacted proof. Do we have a high-confidence way to reproduce the issue? Not applicable: this PR refreshes packaging dependencies rather than repairing a reported bug. Its supplied host build/test transcript is useful partial validation but does not reproduce the full Windows packaging path. Is this the best way to solve the issue? Yes, centralizing the version updates in the workflow, local packager, SDK selector, and package manifest is the maintainable approach; it needs completed package-level validation before landing. AGENTS.md: not found in the target repository. Codex review notes: model internal, reasoning high; reviewed against a9c383789e10. 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
|
Dependency refresh for the Windows packaging toolchain.
Also fixes an upstream pin that was overriding the workflow's pnpm version, and adds a packaged-launcher smoke check.
Local proof on this host:
Windows CI on this candidate is unverified until this PR runs it — that is the point of opening it. Signing remains manual-only; no scheduled monitoring workflows were touched. Codex autoreview: scoped-clean.