refactor: ensure data as readonly when possible

[federicobozzini]

Changes

• make sure views always use readonly model data
• ts target updated to es2023. This is compatible with VS code 101

Screenshots

[awphi]

Looks sensible on its own but I wonder if we should make this even stricter and have the model return readonly values rather than claiming immutability for a handful of values halfway through the view layer.

---

Also think we should enforce this for objects too via Readonly (will need to write a recursive type to make it deep). Wondering if we could hoist all this up and just define loadables as, say

export type Loaded<T> = DeepReadonly<{ status: 'loaded'; data: T; loading: boolean }>;

Because you should never mutate any part of a loadable after its created? I think that's a safe assertion to make?

[federicobozzini]

@awphi I've introduced a fairly aggressive DeepReadOnly usage af79306

WDYT?

I'm afraid it would be better but it might be hard to be consistent with this over time.