Rename and modify channels store

I tried to have a custom class for Channel objects, but Svelte's
automatic proxy logic works only on bare objects, as far as I could
tell. So that broke everything. I resorted to a function that would
build the bare objects, but we still lack methods that I think would
make life easier ("touch last read" etc).

3 files changed, 63 insertions, 37 deletions

diff --git a/ui/lib/store.js b/ui/lib/store.js
index 3b20e05..47ebbc2 100644
--- a/ui/lib/store.js
+++ b/ui/lib/store.js
@@ -1,5 +1,5 @@
 import { writable } from 'svelte/store';
-import { Channels } from '$lib/store/channels';
+import { Channels } from '$lib/store/channels.svelte.js';
 import { Messages } from '$lib/store/messages.svelte.js';
 import { Logins } from '$lib/store/logins';
 
diff --git a/ui/lib/store/channels.js b/ui/lib/store/channels.js
deleted file mode 100644
index 37dc673..0000000
--- a/ui/lib/store/channels.js
+++ /dev/null
@@ -1,36 +0,0 @@
-export class Channels {
-  constructor() {
-    this.channels = [];
-  }
-
-  setChannels(channels) {
-    this.channels = [...channels];
-    this.sort();
-    return this;
-  }
-
-  addChannel(id, name) {
-    this.channels = [...this.channels, { id, name }];
-    this.sort();
-    return this;
-  }
-
-  deleteChannel(id) {
-    const channelIndex = this.channels.map((e) => e.id).indexOf(id);
-    if (channelIndex !== -1) {
-      this.channels.splice(channelIndex, 1);
-    }
-    return this;
-  }
-
-  sort() {
-    this.channels.sort((a, b) => {
-      if (a.name < b.name) {
-        return -1;
-      } else if (a.name > b.name) {
-        return 1;
-      }
-      return 0;
-    });
-  }
-}
diff --git a/ui/lib/store/channels.svelte.js b/ui/lib/store/channels.svelte.js
new file mode 100644
index 0000000..8919be0
--- /dev/null
+++ b/ui/lib/store/channels.svelte.js
@@ -0,0 +1,62 @@
+import { DateTime } from 'luxon';
+const EPOCH_STRING = "1970-01-01T00:00:00Z";
+
+// For reasons unclear to me, a straight up class definition with a constructor
+// doesn't seem to work, reactively. So we resort to this.
+// Owen suggests that this sentence in the Svelte docs should make the reason
+// clear:
+// > If $state is used with an array or a simple object, the result is a deeply
+// > reactive state proxy.
+// Emphasis on "simple object".
+// --Kit
+function makeChannelObject({ id, name, draft = '', lastReadAt = null, scrollPosition = null }) {
+  return {
+    id,
+    name,
+    lastReadAt: lastReadAt || DateTime.fromISO(EPOCH_STRING),
+    draft,
+    scrollPosition,
+  };
+}
+
+export class Channels {
+  channels = $state([]);
+
+  getChannel(channelId) {
+    return this.channels.filter((ch) => ch.id === channelId)[0] || null;
+  }
+
+  setChannels(channels) {
+    this.channels = channels.map(makeChannelObject);
+    this.sort();
+    return this;
+  }
+
+  addChannel(id, name) {
+    this.channels = [
+      ...this.channels,
+      makeChannelObject({ id, name }),
+    ];
+    this.sort();
+    return this;
+  }
+
+  deleteChannel(id) {
+    const channelIndex = this.channels.map((e) => e.id).indexOf(id);
+    if (channelIndex !== -1) {
+      this.channels.splice(channelIndex, 1);
+    }
+    return this;
+  }
+
+  sort() {
+    this.channels.sort((a, b) => {
+      if (a.name < b.name) {
+        return -1;
+      } else if (a.name > b.name) {
+        return 1;
+      }
+      return 0;
+    });
+  }
+}