agent-comms 9.0.8 → 9.0.9

package/src/bridges/user/web/dist/sw.js

@@ -1 +1 @@
-(function(){var e=`agent-comms-v1`,t=[{"revision":"41c6737126547228482ecc2b359801e6","url":"mesh-worker.js"},{"revision":"f1ee3afb8c9e8f0d7315fd93d6457316","url":"index.html"},{"revision":"3c68c960e2ae7cf0918c97b921c287da","url":"bundle.js"}].map(e=>e.url);self.addEventListener(`install`,n=>{n.waitUntil(self.caches.open(e).then(async e=>e.addAll(t)).then(async()=>self.skipWaiting()))}),self.addEventListener(`activate`,t=>{t.waitUntil(self.caches.keys().then(async t=>Promise.all(t.filter(t=>t!==e).map(async e=>self.caches.delete(e)))))});var n=200,r=300;function i(e,t){return e.method===`GET`&&t.status>=n&&t.status<r}self.addEventListener(`fetch`,t=>{if(new URL(t.request.url).pathname.startsWith(`/api/`)){t.respondWith(fetch(t.request).then(n=>{if(i(t.request,n)){let r=n.clone();self.caches.open(e).then(async e=>e.put(t.request,r))}return n}).catch(async()=>self.caches.open(e).then(async e=>e.match(t.request)).then(e=>e??new Response(`Offline`,{status:503}))));return}t.respondWith(self.caches.open(e).then(async e=>e.match(t.request)).then(async n=>n||fetch(t.request).then(n=>{if(i(t.request,n)){let r=n.clone();self.caches.open(e).then(async e=>e.put(t.request,r))}return n})))})})();
+(function(){var e=`agent-comms-v1`,t=[{"revision":"41c6737126547228482ecc2b359801e6","url":"mesh-worker.js"},{"revision":"f1ee3afb8c9e8f0d7315fd93d6457316","url":"index.html"},{"revision":"0bc3d9d0beacf516bee36c2e9be3835f","url":"bundle.js"}].map(e=>e.url);self.addEventListener(`install`,n=>{n.waitUntil(self.caches.open(e).then(async e=>e.addAll(t)).then(async()=>self.skipWaiting()))}),self.addEventListener(`activate`,t=>{t.waitUntil(self.caches.keys().then(async t=>Promise.all(t.filter(t=>t!==e).map(async e=>self.caches.delete(e)))))});var n=200,r=300;function i(e,t){return e.method===`GET`&&t.status>=n&&t.status<r}self.addEventListener(`fetch`,t=>{if(new URL(t.request.url).pathname.startsWith(`/api/`)){t.respondWith(fetch(t.request).then(n=>{if(i(t.request,n)){let r=n.clone();self.caches.open(e).then(async e=>e.put(t.request,r))}return n}).catch(async()=>self.caches.open(e).then(async e=>e.match(t.request)).then(e=>e??new Response(`Offline`,{status:503}))));return}t.respondWith(self.caches.open(e).then(async e=>e.match(t.request)).then(async n=>n||fetch(t.request).then(n=>{if(i(t.request,n)){let r=n.clone();self.caches.open(e).then(async e=>e.put(t.request,r))}return n})))})})();

package/src/core/machine-list-groups.ts

@@ -4,6 +4,7 @@
 
 import type { AgentIdentity } from "./types.js";
 import type { Namer } from "./naming.js";
+import { compareCodeUnits } from "./ordering.js";
 
 /** One run of listed agents and the heading above it; no heading when nothing is grouped at all. */
 export interface AgentGroup {
@@ -40,7 +41,7 @@ export function groupAgentsByMachine(
     .sort(([a], [b]) => {
       if (a === ownMachine) return -1;
       if (b === ownMachine) return 1;
-      return a.localeCompare(b);
+      return compareCodeUnits(a, b);
     })
     .map(([machine, members]) => ({
       heading: `Machine ${namer(machine)}${machine === ownMachine ? " (this machine)" : ""}:`,

package/src/core/ordering.ts

@@ -0,0 +1,27 @@
+/**
+ * Orderings that depend on neither a locale nor a runtime's collation data, so every implementation sorts the same names the same way. Strings compare by UTF-16 code unit, the order of JavaScript's `<`.
+ */
+
+/**
+ * Compares two strings by UTF-16 code unit.
+ * @returns A negative number when `a` sorts first, a positive one when `b` does, 0 when they are equal.
+ */
+export function compareCodeUnits(a: string, b: string): number {
+  if (a < b) return -1;
+  if (a > b) return 1;
+  return 0;
+}
+
+/** Folds the ASCII letters A to Z to lower case and leaves every other character as it is, so no Unicode case table is involved. */
+function foldAsciiCase(name: string): string {
+  return name.replace(/[A-Z]/gu, (letter) => letter.toLowerCase());
+}
+
+/**
+ * Compares two names a person reads: ASCII letters are folded to lower case before comparing by UTF-16 code unit, and names that fold equal are ordered by the names as written, so the order is total and stable.
+ * @returns A negative number when `a` sorts first, a positive one when `b` does, 0 when they are equal.
+ */
+export function compareDisplayNames(a: string, b: string): number {
+  const folded = compareCodeUnits(foldAsciiCase(a), foldAsciiCase(b));
+  return folded === 0 ? compareCodeUnits(a, b) : folded;
+}

package/src/core/relay-selection.ts

@@ -7,6 +7,7 @@
  */
 
 import { BlockList, isIPv4, isIPv6 } from "node:net";
+import { compareCodeUnits } from "./ordering.js";
 
 /** A relay offer this store has heard, with what it knows about the device making it. */
 export interface RelayOffer {
@@ -104,7 +105,7 @@ export function selectRelay(
 ): SelectedRelay {
   const eligible = input.offers
     .filter((offer) => offer.deviceHex !== input.ownDeviceHex)
-    .toSorted((a, b) => a.deviceHex.localeCompare(b.deviceHex));
+    .toSorted((a, b) => compareCodeUnits(a.deviceHex, b.deviceHex));
   for (const tier of OFFER_TIERS) {
     for (const offer of eligible) {
       const admitted =

package/src/core/store.ts

@@ -23,6 +23,7 @@ import {
   RoomMessageSchema,
   RoomSchema,
 } from "./types.js";
+import { compareCodeUnits } from "./ordering.js";
 import type {
   AgentIdentity,
   DeliveryEvent,
@@ -584,7 +585,8 @@ export class FileStore implements CommsStore {
           messages.push(msg);
         }
       }
-      messages.sort((a, b) => a.timestamp.localeCompare(b.timestamp));
+      // Every timestamp this store writes is `new Date().toISOString()`: fixed width, UTC, millisecond precision, so UTF-16 order is chronological order (held by a test at the producer).
+      messages.sort((a, b) => compareCodeUnits(a.timestamp, b.timestamp));
       return messages;
     } catch {
       return [];
@@ -653,6 +655,7 @@ export class FileStore implements CommsStore {
         events.push(event);
         await fs.unlink(path.join(dir, file));
       }
+      // Same fixed timestamp form as readRoomMessages sorts on.
       events.sort((a, b) => {
         const ta =
           a.type === "room_message" || a.type === "dm"
@@ -662,7 +665,7 @@ export class FileStore implements CommsStore {
           b.type === "room_message" || b.type === "dm"
             ? b.message.timestamp
             : "";
-        return ta.localeCompare(tb);
+        return compareCodeUnits(ta, tb);
       });
       return events;
     } catch {