moleculerjs / moleculer / 31783699895

94%

master: 94%

Pull #1366

github

fix(runner): create the ESM runner's `require` lazily

`src/runner-esm.mjs` called `createRequire(import.meta.url)` at module scope.
Because `index.mjs` re-exports this module (`export const Runner = RunnerESM`),
every ESM import of `moleculer` eagerly evaluated it.

In hosts where `import.meta.url` is undefined -- notably bundled edge runtimes
such as Cloudflare Workers -- that throws before any user code runs:

    Uncaught TypeError: The argument 'path' must be a file URL object,
    a file URL string, or an absolute path string. Received 'undefined'
      at createRequire (node:module:34:15)

`require` is only used by `tryConfigPath()`, so creating it lazily keeps the
behaviour identical on Node while letting the ESM entry point be imported and
bundled anywhere.

Verified with `npm run test:esm` (config file resolution goes through the
lazy path) and by booting a ServiceBroker inside workerd.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

5354 of 5913 branches covered (90.55%)

Branch coverage included in aggregate %.

7245 of 7481 relevant lines covered (96.85%)

517.79 hits per line