uncaged/nerve
Archived
10
0
Fork 0
Code Issues 5 Pull Requests Actions Packages Projects Releases Wiki Activity

fix(daemon): 16 skipped kernel tests — flaky timing + routeResult undefined #213

New Issue
Closed
opened 2026-04-28 04:32:26 +00:00 by xiaoju · 0 comments
xiaoju commented 2026-04-28 04:32:26 +00:00
Owner
Copy Link
Copy Source

What

16 daemon tests currently skipped (it.skip) because they fail intermittently or consistently.

8 flaky tests (worker spawn timing)

  • phase6-integration: restartGroup, error isolation, auto-respawn (3)
  • kernel-integration: worker signals, compute round-trip, crash recovery (3)
  • kernel: routes signal, persists signals (2)

These pass sometimes, fail others — worker process spawn/exit timing is non-deterministic.

8 broken tests (routeResult undefined)

  • kernel-workflow-integration: all 8 tests

Error: Cannot read properties of undefined (reading 'message') at kernel.ts:163 — routeResult.error.message when routeResult is undefined.

Fix approach

  • Flaky: add proper waitFor/retry logic or mock worker lifecycle
  • Broken: fix handleSenseWorkerSignal to handle undefined routeResult

Ref

Skipped in chore/skip-flaky-daemon-tests branch

## What 16 daemon tests currently skipped (it.skip) because they fail intermittently or consistently. ### 8 flaky tests (worker spawn timing) - phase6-integration: restartGroup, error isolation, auto-respawn (3) - kernel-integration: worker signals, compute round-trip, crash recovery (3) - kernel: routes signal, persists signals (2) These pass sometimes, fail others — worker process spawn/exit timing is non-deterministic. ### 8 broken tests (routeResult undefined) - kernel-workflow-integration: all 8 tests Error: `Cannot read properties of undefined (reading 'message')` at kernel.ts:163 — `routeResult.error.message` when routeResult is undefined. ## Fix approach - Flaky: add proper waitFor/retry logic or mock worker lifecycle - Broken: fix handleSenseWorkerSignal to handle undefined routeResult ## Ref Skipped in chore/skip-flaky-daemon-tests branch
This repo is archived. You cannot comment on issues.
No Branch/Tag Specified
Branches Tags
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
aobing jiashuang (Jia Shuang) jiayi (Jiayi) luming scottwei tuanzi xiaoju xiaomo xiaonuo xingyue
No Assignees
1 Participants
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: uncaged/nerve#213
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?