Skip to content

packages/pinus/lib/modules/watchServer.ts 试图导入项目没有安装的模块heapdump #999

Open

Description

@eddywen

packages/pinus/lib/modules/watchServer.ts 523

heapdump = require('heapdump');

目前项目中的包似乎都没有安装过 heapdump,是否可以考虑改为使用Node.js 内置的 v8.getHeapSnapshot(Node.js 12+)?

Activity

  1. whtiehack commented on May 26, 2025

    @whtiehack
    Member

    这个可能是 admin monitor的相关功能。 你在自己项目里安装这个 heapdump就行了。

    另外你也可以自己实现一个 admin module或者其它方式来实现。都可以。

    你也可以提交个PR,修改成你说的内置的。(但是因为这个是pomelo遗传下来的功能。所以要在保证兼容性的前提下修改。)

  2. eddywen commented on May 26, 2025

    @eddywen
    Author
  3. whtiehack commented on May 26, 2025

    @whtiehack
    Member

    没懂。 没接触过你说的 ESM

  4. eddywen commented on May 26, 2025

    @eddywen
    Author

    ESM(ECMAScript Modules)。目前pinus的模块支持CommonJS,比如tsconfig.json里设置: "module": "commonjs", //指定生成哪个模块系统代码。如果是支持ESM,会设置成"module": "NodeNext"之类。支持ESM的package需要在package.json设置:"type": "module"。

  5. whtiehack commented on May 27, 2025

    @whtiehack
    Member

    对这个了解不多,另外好像没啥支持的必要,都是用的ts在写代码啊。。难道性能有区别?

    另外框架大量的依赖 require,估计这玩意是个大工程。 我是没动力研究的。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions