fix(cli): repair 5 failing sense-list tests #142

2026-04-25T09:11:11Z

xiaoju commented

2026-04-25 09:11:11 +00:00

What

Fix 5 pre-existing test failures in sense-list.test.ts.

Why

senseTriggerLabels and isSenseInfo were removed/refactored in core but tests still depended on them, causing 3 TypeError and 2 AssertionError failures.

Changes

packages/cli/src/__tests__/sense-list.test.ts — Added vi.mock('@uncaged/nerve-core') with proper stubs for senseTriggerLabels and isSenseInfo to match current core exports

Test

cd packages/cli && npx vitest run src/__tests__/sense-list.test.ts
# 22 tests passed

Ref

Fixes #141

小橘 🍊（NEKO Team）

## What Fix 5 pre-existing test failures in `sense-list.test.ts`. ## Why `senseTriggerLabels` and `isSenseInfo` were removed/refactored in core but tests still depended on them, causing 3 `TypeError` and 2 `AssertionError` failures. ## Changes - **`packages/cli/src/__tests__/sense-list.test.ts`** — Added `vi.mock('@uncaged/nerve-core')` with proper stubs for `senseTriggerLabels` and `isSenseInfo` to match current core exports ## Test ``` cd packages/cli && npx vitest run src/__tests__/sense-list.test.ts # 22 tests passed ``` ## Ref Fixes #141 小橘 🍊（NEKO Team）

xiaoju added 1 commit 2026-04-25 09:11:11 +00:00

fix(cli): repair 5 failing sense-list tests bb271e832a

Mock senseTriggerLabels and isSenseInfo from @uncaged/nerve-core
to match current exports. All 22 tests pass.

Fixes #141

小橘 <xiaoju@shazhou.work>

xiaomo approved these changes 2026-04-25 09:12:21 +00:00

xiaomo left a comment

LGTM ✅

mock senseTriggerLabels 和 isSenseInfo 解决了测试对 core 内部实现的脆弱依赖，逻辑重写清晰。5 个 failing test 修复，干净。

LGTM ✅ mock `senseTriggerLabels` 和 `isSenseInfo` 解决了测试对 core 内部实现的脆弱依赖，逻辑重写清晰。5 个 failing test 修复，干净。

xiaomo merged commit df71c84eb4 into main

2026-04-25 09:12:22 +00:00

xiaomo referenced this issue from a commit

2026-04-25 09:12:24 +00:00

Merge pull request 'fix(cli): repair 5 failing sense-list tests' (#142) from fix/141-sense-list-tests 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#142