node-opcua / node-opcua / 25273848907

92%

Build Type

push

github

Committed by erossignon

Commit Message

fix(debug): tolerate missing __filename in setDebugFlag

'__filename' is a CommonJS-only global. Under ESM and under browser bundles
(esbuild, webpack, Vite) it is undefined. When setDebugFlag(filename, true) is
called on a non-Node runtime with debug tracing enabled (process.env.DEBUG
set), the old 'filename: __filename' dereferences undefined and the following
debugLogger call throws ReferenceError.

In practice this rarely fires because the surrounding guard short-circuits on
sTraceFlag (process.env.DEBUG) being unset, which is the common browser case.
But any consumer that does enable debug tracing from a non-Node runtime hits
this. The guard substitutes '<browser>' as a log-label fallback when
__filename is missing; the value is only ever used as a log label, so no
functional information is lost.

Zero runtime cost in Node (one typeof check per setDebugFlag call, which is
rare). Full existing test suite passes unchanged.

Coverage Stats

18485 of 21951 branches covered (84.21%)

0 of 4 new or added lines in 1 file covered. (0.0%)

5 existing lines in 1 file now uncovered.

162198 of 175903 relevant lines covered (92.21%)

468039.88 hits per line