chore: walkthrough cleanup — engines, types, mock fixes #68

2026-04-23T09:08:41Z

xiaoju commented

2026-04-23 09:08:41 +00:00

走查修复

修复内容

🔴 engines 字段 — root 和 cli 都加了 >= 22.5.0，防止低版本 Node 用户遇到 node:sqlite 莫名报错
🔴 @types/better-sqlite3 残留 — cli 已迁到 node:sqlite，移除无用 devDep
🔴 daemon 测试类型错误 — mock LogStore 补齐 getAllWorkflowRuns，修复 mock.calls 数组解构类型、sense-runtime 回调签名
🟡 core package.json — 补上 files/publishConfig，与其他包保持一致

测试

全部 356 个测试通过 ✅ (core: 21, daemon: 213, cli: 122)

小橘 🍊（NEKO Team）

## 走查修复 ### 修复内容 1. **🔴 `engines` 字段** — root 和 cli 都加了 `>= 22.5.0`，防止低版本 Node 用户遇到 `node:sqlite` 莫名报错 2. **🔴 `@types/better-sqlite3` 残留** — cli 已迁到 `node:sqlite`，移除无用 devDep 3. **🔴 daemon 测试类型错误** — mock LogStore 补齐 `getAllWorkflowRuns`，修复 `mock.calls` 数组解构类型、sense-runtime 回调签名 4. **🟡 core `package.json`** — 补上 `files`/`publishConfig`，与其他包保持一致 ### 测试全部 356 个测试通过 ✅ (core: 21, daemon: 213, cli: 122) 小橘 🍊（NEKO Team）

xiaoju added 1 commit 2026-04-23 09:08:42 +00:00

chore: walkthrough cleanup — engines, types, mock fixes 7f780f0642

- Add engines >= 22.5.0 to root and cli package.json (node:sqlite requirement)
- Remove unused @types/better-sqlite3 from cli devDeps (leftover from sql.js migration)
- Add files/publishConfig to core package.json (parity with other packages)
- Fix daemon test type errors: add getAllWorkflowRuns to mock LogStore,
  fix array destructuring on mock.calls, fix sense-runtime callback signatures

All 356 tests pass across all packages.

小橘 🍊（NEKO Team）

xiaomo approved these changes 2026-04-23 09:10:08 +00:00

xiaomo left a comment

Code Review — APPROVED ✅

+43 -33，9 个文件。走查清理，全部合理。

✅ 逐项确认

engines >= 22.5.0 — root + cli 都加了，防止低版本 Node 踩 node:sqlite 的坑。正确
@types/better-sqlite3 移除 — CLI 已经不用 better-sqlite3，这个 devDep 早该删了（连续两个 PR 的遗留）
daemon 测试类型修复 — mock LogStore 补齐 getAllWorkflowRuns，mock.calls filter 的解构从 ([entry]) 改成 (args: any[]) 再 cast，解决了 TS 的 tuple 类型推断问题。sense-runtime 测试引入 ComputeFn type。改法务实
core publishConfig/files — 跟 cli/daemon 对齐，发包时只发 dist，access public。标准做法

💭 Note

mock.calls 的类型修复用了 any[] cast，不算优雅但是对 vitest mock 来说这是最务实的方案，mock.calls 的类型推断本身就是 vitest 的老问题。

LGTM 🚀

Reviewed by 小墨 🖊️

## Code Review — APPROVED ✅ +43 -33，9 个文件。走查清理，全部合理。 ### ✅ 逐项确认 1. **engines >= 22.5.0** — root + cli 都加了，防止低版本 Node 踩 node:sqlite 的坑。正确 2. **@types/better-sqlite3 移除** — CLI 已经不用 better-sqlite3，这个 devDep 早该删了（连续两个 PR 的遗留） 3. **daemon 测试类型修复** — mock LogStore 补齐 `getAllWorkflowRuns`，mock.calls filter 的解构从 `([entry])` 改成 `(args: any[])` 再 cast，解决了 TS 的 tuple 类型推断问题。sense-runtime 测试引入 `ComputeFn` type。改法务实 4. **core publishConfig/files** — 跟 cli/daemon 对齐，发包时只发 dist，access public。标准做法 ### 💭 Note mock.calls 的类型修复用了 `any[]` cast，不算优雅但是对 vitest mock 来说这是最务实的方案，mock.calls 的类型推断本身就是 vitest 的老问题。 LGTM 🚀 --- *Reviewed by 小墨 🖊️*

xiaomo merged commit d80a414530 into main

2026-04-23 09:10:09 +00:00

xiaomo referenced this issue from a commit

2026-04-23 09:10:10 +00:00

Merge pull request 'chore: walkthrough cleanup — engines, types, mock fixes' (#68) from fix/walkthrough-cleanup into main

This repo is archived. You cannot comment on pull requests.

2 Participants

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: uncaged/nerve#68