core icon indicating copy to clipboard operation
core copied to clipboard

[BUG] 如何屏蔽node:log为.sumi的目录?

Open shaojiankui opened this issue 2 years ago • 1 comments

运行起来项目以后控制台一直打FileChangeEvent log。很乱,很影响性能。 我理解.sumi/的变化是应该排除在外的,他的变化不应该被监听。

[node:debug] FileChangeEvent: [{"uri":"file:///Users/jakey/.sumi/logs/20230810/app.1.log","type":1},{"uri":"file:///Users/jakey/.sumi/logs/20230810/app.2.log","type":1},{"uri":"file:///Users/jakey/.sumi/logs/20230810/app.3.log","type":1},{"uri":"file:///Users/jakey/.sumi/logs/20230810/app.4.log","type":1},{"uri":"file:///Users/jakey/.sumi/logs/20230810/app.5.log","type":1},{"uri":"file:///Users/jakey/.sumi/logs/20230810/app.6.log","type":1},{"uri":"file:///Users/jakey/.sumi/logs/2023081...[node][node:debug] FileChangeEvent: [{"uri":"file:///Users/jakey/.sumi/datas/explorer.json","type":1}]

shaojiankui avatar Aug 10 '23 08:08 shaojiankui

@shaojiankui logs 目录下的日志确实应该被屏蔽,但其余部分的日志确实有一定用途,如 .sumi/data 内存储的部分框架运行信息需要实时更新。

这里有一段逻辑处理的不太合理, 应该默认将 **/logs/** 规则应用至 .sumi 的文件监听内,否则就会有这个问题https://github.com/opensumi/core/blob/2fb02ec62456f5632006528bd7e99171f1e270a0/packages/preferences/src/browser/userstorage/user-storage.service.ts#L22

监听 .sumi 目录的逻辑在这里,猜测是这里的 excludes 处理不恰当 https://github.com/opensumi/core/blob/2fb02ec62456f5632006528bd7e99171f1e270a0/packages/preferences/src/browser/userstorage/user-storage.service.ts#L114

欢迎提个 PR 修复一下该问题 ~

erha19 avatar Aug 10 '23 09:08 erha19