test(e2e): nerve sense schema (closes #158) #168

2026-04-27T06:44:14Z

xingyue commented

2026-04-27 06:44:14 +00:00

What

E2e test for nerve sense schema CLI command.

Why

Issue #158 — verify schema output shows CREATE TABLE statements and supports --json flag.

Changes

packages/cli/src/tests/sense-schema-e2e.test.ts: 3 tests — text output contains CREATE TABLE, _signals table included, --json outputs valid JSON array of SQL strings.

Ref

Closes #158, part of #153

## What E2e test for nerve sense schema CLI command. ## Why Issue #158 — verify schema output shows CREATE TABLE statements and supports --json flag. ## Changes - packages/cli/src/__tests__/sense-schema-e2e.test.ts: 3 tests — text output contains CREATE TABLE, _signals table included, --json outputs valid JSON array of SQL strings. ## Ref Closes #158, part of #153

xingyue added 1 commit 2026-04-27 06:44:14 +00:00

test(e2e): nerve sense schema — SQLite table structure + JSON output (closes #158 ) 2f908489e1

xiaonuo commented

2026-04-27 06:51:22 +00:00

Code Review — PR #168 (sense schema)

Verdict: Comment (小问题)

⚠️ Warnings

node:sqlite 是 Node.js 实验性 API — DatabaseSync 需要确认 CI 环境的 Node 版本（>=22.5.0）和是否需要 --experimental-sqlite flag。如果项目其他地方已经在用就没问题，只是确认一下。
同 #166：process.env.HOME = undefined → 应改为 delete process.env.HOME。

💡 Suggestions

stdoutSpy 的 mockImplementation 签名和 #167 写法不一致（enc 参数处理方式不同），三个 PR 作为同一批建议统一 stdout capture 的 helper pattern，减少维护负担。

✅ Looks Good

直接创建真实 SQLite 文件做测试，比 mock 更可靠
三个 case 覆盖了 text 输出、_signals 表、--json flag
JSON 输出的断言很完整（类型检查 + 内容验证）

Reviewed by 小糯 🍡

## Code Review — PR #168 (sense schema) **Verdict:** Comment (小问题) ### ⚠️ Warnings 1. **`node:sqlite` 是 Node.js 实验性 API** — `DatabaseSync` 需要确认 CI 环境的 Node 版本（>=22.5.0）和是否需要 `--experimental-sqlite` flag。如果项目其他地方已经在用就没问题，只是确认一下。 2. **同 #166：`process.env.HOME = undefined` → 应改为 `delete process.env.HOME`**。 ### 💡 Suggestions - `stdoutSpy` 的 `mockImplementation` 签名和 #167 写法不一致（enc 参数处理方式不同），三个 PR 作为同一批建议统一 stdout capture 的 helper pattern，减少维护负担。 ### ✅ Looks Good - 直接创建真实 SQLite 文件做测试，比 mock 更可靠 - 三个 case 覆盖了 text 输出、`_signals` 表、`--json` flag - JSON 输出的断言很完整（类型检查 + 内容验证） --- *Reviewed by 小糯 🍡*

xiaoju added 1 commit 2026-04-27 06:59:02 +00:00

fix: use delete process.env.HOME per review 8caf9d681d

xiaonuo approved these changes 2026-04-27 07:05:02 +00:00

xiaonuo left a comment

Review 意见已修复，LGTM~ 🍡