This is a comment left during a code review.
Path: src/tasks/task-registry.ts
Line: 72-73

Comment:
**Live map reference passed to `saveSnapshot`**

`persistTaskRegistry()` passes the module-level `tasks` map directly to `saveSnapshot`. The parameter is typed as `ReadonlyMap`, which prevents mutations through the reference, but the underlying map continues to be mutated by the registry between calls. The current JSON store is safe because it iterates and serializes immediately, but any future store implementation that defers reading (e.g., async batching/flush) would observe a later, modified state rather than the snapshot captured at write time.

Since this PR explicitly positions `TaskRegistryStore` as an infrastructure seam for future backends, it's worth documenting the contract — or passing a snapshot at call time:

```ts
function persistTaskRegistry() {
  // Pass a shallow copy to give store implementations a stable snapshot.
  // The live map is mutable; any async store must copy immediately on save.
  getTaskRegistryStore().saveSnapshot(new Map(tasks));
}
```

Alternatively, document in the `TaskRegistryStore` type that `saveSnapshot` must read the map synchronously before returning.

How can I resolve this? If you propose a fix, please make it concise.

---

This is a comment left during a code review.
Path: src/tasks/task-registry.ts
Line: 283-297

Comment:
**`restored` hook event is silently skipped for empty snapshots**

When `loadSnapshot()` returns an empty map, the function returns early on line 284 without emitting any hook event. A consumer that relies on the `restored` event to know the registry has finished initializing (e.g., to start indexing or drain a queue) will never receive it on a fresh or empty store.

If the intention is "no event on empty restore", that's a reasonable choice, but it should be documented in the `TaskRegistryHooks` or `TaskRegistryHookEvent` types. Alternatively, emit the event unconditionally (with an empty `tasks` array) so the hook contract is predictable:

```ts
// always emit restored so consumers know initialization is done
emitTaskRegistryHookEvent(() => ({
  kind: "restored",
  tasks: snapshotTaskRecords(tasks),
}));
```

How can I resolve this? If you propose a fix, please make it concise.