refactor(core): remove unnecessary | null, unify timestamp naming #112

2026-04-25T01:54:05Z

xiaoju commented

2026-04-25 01:54:05 +00:00

What

Remove unnecessary | null from config types and unify timestamp field naming.

Changes

SenseReflexConfig.on: string[] | null → string[]
NerveConfig.workflows: Record | null → Record
Signal.ts → Signal.timestamp, lastSignalTs → lastSignalTimestamp
All consumers updated, 378 tests pass

Fixes #108

## What Remove unnecessary `| null` from config types and unify timestamp field naming. ## Changes - `SenseReflexConfig.on`: `string[] | null` → `string[]` - `NerveConfig.workflows`: `Record | null` → `Record` - `Signal.ts` → `Signal.timestamp`, `lastSignalTs` → `lastSignalTimestamp` - All consumers updated, 378 tests pass Fixes #108

xiaoju added 1 commit 2026-04-25 01:54:05 +00:00

refactor(core): remove unnecessary | null, unify timestamp naming 020a1bfe85

- SenseReflexConfig.on: string[] | null → string[] (empty = no conditions)
- NerveConfig.workflows: Record | null → Record (empty = no workflows)
- Signal.ts → Signal.timestamp
- SenseInfo.lastSignalTs → SenseInfo.lastSignalTimestamp
- All consumers across daemon/cli/store updated
- parseNerveConfig: on defaults to [], workflows defaults to {}

Fixes #108

xiaoju merged commit 529cceba06 into main

2026-04-25 01:57:48 +00:00

xiaoju referenced this issue from a commit

2026-04-25 01:57:49 +00:00

Merge pull request 'refactor(core): remove unnecessary | null, unify timestamp naming' (#112) from refactor/108-remove-null-unify-ts into main

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

1 Participants

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: uncaged/nerve#112