fix: add nostr-tools to root dependencies#48803
Conversation
…OUND in Docker setup
Greptile SummaryThis PR adds
Confidence Score: 5/5
Last reviewed commit: 5bea524 |
|
Closing this as implemented after Codex review. Current main already handles bundled channel runtime dependencies without mirroring What I checked:
So I’m closing this as already implemented rather than keeping a duplicate issue open. Review notes: reviewed against 01bc49c88c0c; fix evidence: release v2026.4.22, commit 00bd2cf7a376. |
…odule is missing When a bundled channel plugin (e.g. `nostr`, `whatsapp`, `msteams`) fails to load because its plugin-local runtime dependency is not yet staged, the loader emitted a bare `Cannot find module 'X'` warning with no remediation hint. The intended repair path is `openclaw doctor --fix` (per openclaw#48803), so users were left to discover this themselves. Detect `ERR_MODULE_NOT_FOUND` / `MODULE_NOT_FOUND` in the bundled channel load error path and append the canonical `run \`openclaw doctor --fix\` to install missing bundled runtime dependencies for channel <id>` guidance to the warning detail. Applied at all seven `[channels] failed to load bundled channel ...` warn sites in `bundled.ts`. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…odule is missing When a bundled channel plugin (e.g. `nostr`, `whatsapp`, `msteams`) fails to load because its plugin-local runtime dependency is not yet staged, the loader emitted a bare `Cannot find module 'X'` warning with no remediation hint. The intended repair path is `openclaw doctor --fix` (per openclaw#48803), so users were left to discover this themselves. Detect `ERR_MODULE_NOT_FOUND` / `MODULE_NOT_FOUND` in the bundled channel load error path and append the canonical `run \`openclaw doctor --fix\` to install missing bundled runtime dependencies for channel <id>` guidance to the warning detail. Applied at all seven `[channels] failed to load bundled channel ...` warn sites in `bundled.ts`. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> (cherry picked from commit 17e6794)
This PR adds
nostr-toolsto the rootpackage.jsonto ensure it is installed in Docker containers and available for the bundled application, resolving theERR_MODULE_NOT_FOUNDerror reported in #48797.