Circular dependency between src/util/minimal.js and src/util/longbits.js

[ericghpham]

I'm running a project locally using vite to bundle, and it's unable to successfully build the project. The following error is returned:

Circular dependency detected:
../.yalc/<project>/dist/external/_pnpm/protobufjs@7.4.0/external/protobufjs/src/util/longbits.js -> ../.yalc/<project>/dist/external/_pnpm/protobufjs@7.4.0/external/protobufjs/src/util/minimal.js -> ../.yalc/<project>/dist/external/_pnpm/protobufjs@7.4.0/external/protobufjs/src/util/longbits.js

I think a circular dependency also exists between:

• src/util.js <> src/type.js
• src/util.js <> src/enum.js

[dcodeIO]

I can reproduce the circular module graph, but I cannot reproduce this as a default Vite build failure on current master. A minimal Vite project importing protobufjs builds successfully.

Some cycles in protobuf.js are intentional and currently harmless at runtime because the respective imports are only used at call time, notably around reflection support. Treating Rollup circular-dependency warnings as fatal is therefore not supported right now.

Hence closing this issue as not actionable without a small reproduction where a default build fails.