Skip to content

fix(release): package embeddable chat module#9

Merged
DonPrus merged 2 commits into
nullclaw:mainfrom
vernonstinebaker:fix/release-embed-module
May 19, 2026
Merged

fix(release): package embeddable chat module#9
DonPrus merged 2 commits into
nullclaw:mainfrom
vernonstinebaker:fix/release-embed-module

Conversation

@vernonstinebaker
Copy link
Copy Markdown
Contributor

Summary

  • include the dist/ embeddable module artifacts in release archives when present
  • verify that release packages contain module.js in addition to the runnable static app bundle

Why

NullHub embeds nullclaw-chat-ui by dynamically importing /ui/<module>@<version>/module.js. The existing release packaging only shipped the static build/ app and CLI launcher, so NullHub could install the package but not load the embeddable module entrypoint. That left the chat tab failing with Failed to load module: Importing a module script failed.

Validation

  • npm run build
  • npm run build:module
  • bash ./scripts/package-release.sh vtest-pr release
  • bash ./scripts/verify-release-package.sh vtest-pr release
  • local NullHub smoke after installing the rebuilt package:
    • GET /ui/nullclaw-chat-ui@latest/module.js -> 200 OK
    • confirmed chat tabs load for default and willy-nilly

Notes

  • this is the release-packaging half of the chat fix
  • the corresponding NullHub installer compatibility fix is submitted separately

NullHub loads nullclaw-chat-ui through module.js, so release archives need to ship the module build alongside the static app bundle.
@DonPrus DonPrus merged commit 6ae9786 into nullclaw:main May 19, 2026
1 check passed
@vernonstinebaker vernonstinebaker deleted the fix/release-embed-module branch May 19, 2026 04:03
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.

2 participants