Skip to content

feat(ui)!: publish paired WASM and OpenUI assets - #128

Merged
MarcStdt merged 1 commit into
masterfrom
codex/local-openui-rc
Oct 27, 2026
Merged

MarcStdt merged 1 commit into
masterfrom
codex/local-openui-rc

Conversation

@MarcStdt

@MarcStdt MarcStdt commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Presign and upload app.openui beside app.wasm under the same immutable request prefix in both runtimes.
  • Validate both files before uploading; upload the document first and update the active package only after success.
  • Preserve the previous package on publication failure and keep worker environment expansion safe.
  • Add Node 24 and Rust/wasm32 tooling to the developer image.

Boundaries

No new public HTTP endpoints, CLI flags or protobuf fields. RuntimeUIPackageUpload changes are internal.

Verification

  • go test ./... passed (one initial existing supervisor cleanup-race failure; complete rerun passed).
  • Restore/retention and paired-publication regressions include missing document, upload failure and worker environment expansion.
  • Targeted supervisor, Docker and Kubernetes package tests passed again before commit.
  • Built/imported the local developer image and published a real WASM/document pair in Kubernetes.

Readiness

Draft: this integration was verified against the local build of highcard-dev/druid-ui#19 (9443dcb), published only to local Verdaccio as 4.0.0-rc.1. GitHub still reports SDK PR #19 as OPEN, not merged.

Before merge, publish the SDK RC, regenerate consumer lockfiles against the actual public tarballs (the current RC integrity hashes come from local packing), and rerun clean-install CI. Local registry URLs have been removed from committed lockfiles. Coordinate the host, runtime, Scrolls and developer image rollout; this is a breaking UI-package format change. No production deployment or public npm release was performed.

Coordinated PRs

BREAKING CHANGE: UI publication requires a nonempty .openui document beside the WASM component. Upload both under one immutable prefix before updating the active package.
@MarcStdt
MarcStdt marked this pull request as ready for review October 27, 2026 23:15
@MarcStdt
MarcStdt merged commit 4e6158e into master Oct 27, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant