PR #62048: fix: use dist/extensions for OPENCLAW_BUNDLED_PLUGINS_DIR to fix Node 24 crash

• Repository: openclaw/openclaw
• Author: jcgau
• State: open | merged: False
• Link: https://github.com/openclaw/openclaw/pull/62048

Description (problem / solution / changelog)

Problem

On Node 24, process.features.typescript is true, meaning Node has native TypeScript support enabled. When OPENCLAW_BUNDLED_PLUGINS_DIR points to /app/extensions (the TypeScript source tree), Node 24 intercepts .ts file loading natively — but its native TS loader does not remap .js imports to .ts for static imports inside those files.

This causes a crash loop on startup:

Cannot find module '/app/extensions/matrix/src/account-selection.js'
imported from /app/extensions/matrix/src/credentials-read.ts

The call chain: JITI loads credentials-read.ts → inside that file, account-selection.js is imported statically → Node 24 native TS loader resolves credentials-read.ts but then tries to find account-selection.js literally (no .js→.ts remapping) → module not found → crash.

Note: the issue is not with JITI itself. JITI handles .js→.ts remapping correctly in isolation. The problem is Node 24 intercepting before JITI in this code path.

Fix

Point OPENCLAW_BUNDLED_PLUGINS_DIR to /app/dist/${OPENCLAW_BUNDLED_PLUGIN_DIR} (compiled JS) instead of /app/${OPENCLAW_BUNDLED_PLUGIN_DIR} (TypeScript source). The dist/ tree is already present in the runtime image via COPY --from=runtime-assets ... /app/dist ./dist (line 153).

Using compiled JS avoids the Node 24 native TS loader entirely for bundled plugins.

Testing

Verified on:

• Node 24 (Ubuntu 24.04, Docker)
• OpenClaw 2026.4.5
• Matrix channel: no crash loop, MatrixRTCSession syncing correctly after fix

Related

Closes #62044

Changed files

• Dockerfile (modified, +10/-3)

PR #62316: fix(docker): use built bundled plugins in runtime images

• Repository: openclaw/openclaw
• Author: gumadeiras
• State: closed | merged: True
• Link: https://github.com/openclaw/openclaw/pull/62316

Description (problem / solution / changelog)

Summary

• Problem: the Docker runtime image forced bundled plugin discovery to /app/extensions, which made packaged installs load raw source plugin entries instead of compiled runtime artifacts.
• Why it matters: on Node 24 this can boot through source-only plugin paths and crash fresh containers, with Matrix surfacing first in issue #62044.
• What changed: removed the Docker OPENCLAW_BUNDLED_PLUGINS_DIR override so packaged installs fall back to the normal resolver and use dist/extensions; updated the Dockerfile guard test; added a changelog fix entry.
• What did NOT change (scope boundary): no plugin loader redesign, no source-runtime compatibility work, no broader plugin import refactor.

Change Type (select all)

• Bug fix
• Feature
• Refactor required for the fix
• Docs
• Security hardening
• Chore/infra

Scope (select all touched areas)

• Gateway / orchestration
• Skills / tool execution
• Auth / tokens
• Memory / storage
• Integrations
• API / contracts
• UI / DX
• CI/CD / infra

Linked Issue/PR

• Closes #62044
• Related #50058
• This PR fixes a bug or regression

Root Cause (if applicable)

• Root cause: commit a2a9a553e1e03fcd6a3ec01196f3cd7b58710b7e added a Docker runtime override that pinned OPENCLAW_BUNDLED_PLUGINS_DIR to /app/extensions, bypassing the packaged-install resolver that already prefers dist/extensions.
• Missing detection / guardrail: the Dockerfile test locked in the source-tree override instead of asserting packaged-runtime default behavior.
• Contributing context (if known): the issue became user-visible after the Node 24 runtime image path started executing against source plugin graphs that are not the intended packaged runtime surface.

Regression Test Plan (if applicable)

• Coverage level that should have caught this:
  • Unit test
  • Seam / integration test
  • End-to-end test
  • Existing coverage already sufficient
• Target test or file: src/dockerfile.test.ts
• Scenario the test should lock in: runtime images must not override bundled plugin discovery away from packaged defaults.
• Why this is the smallest reliable guardrail: the regression was introduced directly in the Dockerfile and guarded by a Dockerfile-specific test.
• Existing test that already covers this (if any): src/plugins/bundled-dir.test.ts already covers packaged-install preference for dist/extensions.
• If no new test is added, why not: existing Dockerfile guard test was updated instead of adding a duplicate.

User-visible / Behavior Changes

Docker runtime images now use compiled bundled plugin artifacts again instead of forcing source plugin discovery.

Diagram (if applicable)

Before:
[docker runtime] -> [/app/extensions override] -> [source plugin entrypoints] -> [Node 24 source-path crash]

After:
[docker runtime] -> [default packaged resolver] -> [dist/extensions] -> [compiled plugin artifacts]

Security Impact (required)

• New permissions/capabilities? (Yes/No) No
• Secrets/tokens handling changed? (Yes/No) No
• New/changed network calls? (Yes/No) No
• Command/tool execution surface changed? (Yes/No) No
• Data access scope changed? (Yes/No) No
• If any Yes, explain risk + mitigation:

Repro + Verification

Environment

• OS: macOS
• Runtime/container: local repo runtime; Docker packaging path
• Model/provider: N/A
• Integration/channel (if any): bundled Matrix plugin as first visible reproducer
• Relevant config (redacted): default packaged bundled-plugin resolution

Steps

1. Inspect Docker runtime image config and bundled plugin resolver behavior.
2. Verify source bundled plugin path is forced by the Dockerfile override.
3. Remove the override so packaged installs fall back to dist/extensions.
4. Run targeted tests and full build.

Expected

• Packaged Docker/runtime installs use compiled bundled plugin artifacts.

Actual

• Before fix, Docker runtime images forced source plugin discovery.

Evidence

Attach at least one:

• Failing test/log before + passing after
• Trace/log snippets
• Screenshot/recording
• Perf numbers (if relevant)

Human Verification (required)

What you personally verified (not just CI), and how:

• Verified scenarios: pnpm test src/dockerfile.test.ts src/plugins/bundled-dir.test.ts; pnpm build
• Edge cases checked: packaged resolver still prefers built bundled artifacts; Dockerfile no longer pins the wrong runtime env var.
• What you did not verify: full rebuilt Docker image boot smoke.

Review Conversations

• I replied to or resolved every bot review conversation I addressed in this PR.
• I left unresolved only the conversations that still need reviewer or maintainer judgment.

Compatibility / Migration

• Backward compatible? (Yes/No) Yes
• Config/env changes? (Yes/No) No
• Migration needed? (Yes/No) No
• If yes, exact upgrade steps:

Risks and Mitigations

• Risk: a Docker-specific workflow may have been relying on source plugin discovery.
  • Mitigation: packaged installs already prefer dist/extensions, and the built plugin manifests are the intended runtime surface.

Changed files

• CHANGELOG.md (modified, +1/-0)
• Dockerfile (modified, +0/-4)
• src/dockerfile.test.ts (modified, +7/-5)