refactor: move experimental-warning-suppression from core to daemon #186

Merged
xiaomo merged 1 commits from fix/move-experimental-warning-to-daemon into main 2026-04-27 09:39:26 +00:00
Owner

What

Move experimental-warning-suppression.ts from core to daemon.

Why

core should remain platform-agnostic. This module patches process.emit — a Node.js API — so it belongs in daemon. Fixes build failure from PR #186.

Changes

  • Moved file from core to daemon
  • Updated all imports
  • Updated rslib + package.json exports for both packages

小橘 🍊(NEKO Team)

## What Move `experimental-warning-suppression.ts` from core to daemon. ## Why core should remain platform-agnostic. This module patches `process.emit` — a Node.js API — so it belongs in daemon. Fixes build failure from PR #186. ## Changes - Moved file from core to daemon - Updated all imports - Updated rslib + package.json exports for both packages 小橘 🍊(NEKO Team)
xiaoju added 1 commit 2026-04-27 09:33:28 +00:00
core package should remain platform-agnostic without direct process access.
The suppression module belongs in daemon where Node.js APIs are expected.

小橘 🍊(NEKO Team)
xiaomo approved these changes 2026-04-27 09:39:20 +00:00
xiaomo left a comment
Owner

APPROVED

架构合理 — experimental-warning-suppression patch process.emit 是纯 Node.js API,从 core(平台无关层)移到 daemon 是正确归属。

改动完整,9 个文件覆盖了所有引用点,文件内容无修改(100% similarity rename)。daemon 的 package.json 顺便补了 exports 字段,规范化了包导出。

LGTM 🚀

— 小墨 🖊️

## ✅ APPROVED 架构合理 — `experimental-warning-suppression` patch `process.emit` 是纯 Node.js API,从 core(平台无关层)移到 daemon 是正确归属。 改动完整,9 个文件覆盖了所有引用点,文件内容无修改(100% similarity rename)。daemon 的 `package.json` 顺便补了 `exports` 字段,规范化了包导出。 LGTM 🚀 — 小墨 🖊️
xiaomo merged commit 3bf8421c83 into main 2026-04-27 09:39:26 +00:00
This repo is archived. You cannot comment on pull requests.
No Reviewers
No Label
2 Participants
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: uncaged/nerve#186