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

refactor(core): remove unnecessary | null from config types, unify timestamp naming #108

New Issue
Closed
opened 2026-04-25 01:46:35 +00:00 by xiaoju · 0 comments
xiaoju commented 2026-04-25 01:46:35 +00:00
Owner
Copy Link
Copy Source

Phase 1 — 最安全的机械替换

1. 去掉无意义的 | null

  • SenseReflexConfig.on: string[] | null → string[]（空数组 = 无条件）
  • SenseReflexConfig.interval: number | null → number（0 或默认值 = 不设间隔）
  • NerveConfig.workflows: Record<string, WorkflowConfig> | null → Record<string, WorkflowConfig>（空对象 = 无 workflow）
  • SenseConfig 里的 throttle/timeout/gracePeriod 的 | null 也审视一下，如果 null 和 0 语义相同就去掉

2. 统一时间戳命名

  • Signal.ts → Signal.timestamp（L5）
  • SenseInfo.lastSignalTs → SenseInfo.lastSignalTimestamp（L21）
  • 全局搜索 .ts 字段引用，确保所有消费方同步更新

约束

  • 所有消费这些类型的代码（daemon、cli、store、workflow-utils）都要同步改
  • pnpm run check && pnpm run build && pnpm -r run test 全过
  • 不改语义，只改类型表达
## Phase 1 — 最安全的机械替换 ### 1. 去掉无意义的 `| null` - `SenseReflexConfig.on: string[] | null` → `string[]`（空数组 = 无条件） - `SenseReflexConfig.interval: number | null` → `number`（0 或默认值 = 不设间隔） - `NerveConfig.workflows: Record<string, WorkflowConfig> | null` → `Record<string, WorkflowConfig>`（空对象 = 无 workflow） - `SenseConfig` 里的 `throttle`/`timeout`/`gracePeriod` 的 `| null` 也审视一下，如果 null 和 0 语义相同就去掉 ### 2. 统一时间戳命名 - `Signal.ts` → `Signal.timestamp`（L5） - `SenseInfo.lastSignalTs` → `SenseInfo.lastSignalTimestamp`（L21） - 全局搜索 `.ts` 字段引用，确保所有消费方同步更新 ### 约束 - 所有消费这些类型的代码（daemon、cli、store、workflow-utils）都要同步改 - `pnpm run check && pnpm run build && pnpm -r run test` 全过 - 不改语义，只改类型表达
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#108
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?