@ai-matrx/meet 0.7.7 → 0.7.9

package/CHANGELOG.md

@@ -1,8 +1,59 @@
 # Changelog — `@ai-matrx/meet`
 
+## 0.7.9
+### (next patch)
+
+- **Fix — a database refusal reads as a sentence for a person** (verifier, 2026-09-27, the
+  class behind raw toast text): `normalizeMeetError`'s `message` was
+  "meet.x: denied by the database (42501: …)" with a remedy about RLS and service-role
+  clients, and hosts render `message` + `remedy` in toasts. Now our RPC's own sentence
+  (`RAISE EXCEPTION 'Only the host …'`) is the message; Postgres wording, SQLSTATEs and class
+  names become a plain sentence ("You don't have access to do that in this meeting.", "This
+  meeting changed a moment ago…", "The server could not be reached."); the raw error stays on
+  `cause`. `SERVER_FAILED_SENTENCE` / `isRawServerText` now live in `core/errors.ts`
+  (still re-exported from the root).
+- **Consumer action:** none.
+
+## 0.7.8
+### (next patch)
+
+- **Fix — a refusal never reaches a person as a Python class name** (verifier, 2026-09-27): a
+  host's toast read "Internal server error: UnsupportedClientFeatureError". `readServerRefusal`
+  now skips raw server text (`Internal server error: …`, a bare `…Error` class, a snake_case
+  code) and falls through to the body's `user_message`, or to `SERVER_FAILED_SENTENCE`. The
+  three hand-copied refusal builders (`in-room.ts`, `management.ts`, `after.ts`) collapse into
+  ONE exported `refusalFromResponse(response, operation, fallbackRemedy)`, whose `message` is
+  the server's sentence — no `meet.x:` operation prefix — and whose `remedy` is the server's.
+- **Fix — a refused Remove / Make host keeps its pane open WITH the sentence** : the confirm
+  pane now shows the server's refusal inline (`role="alert"`) with its buttons live; before it
+  stayed up silently.
+- **Fix — every per-person menu offers "Message privately"**: a tile's menu (Room stage and a
+  host layout's strip, e.g. the Board) now opens the private chat through the store
+  (`snapshot.privateChatRequest`, answered by the one `ControlBar`); before only the People
+  panel could, so two menus for one person disagreed.
+- **Fix — attendance is per PERSON, never per join**: an export listed the same attendee twice
+  under "attended". `attendanceReport` merges a person's joins (account = one person across
+  devices and rejoins; an accountless guest's rejoin, which gets a fresh `guest:<random>`
+  identity, is matched by the typed name) into one row — first arrival, last departure, total
+  time with overlaps counted once, and a new `joins` count. `notesDocument`'s "Attended" line
+  and the record view's "Who attended" use the new exported `uniqueAttendees` / `attendeeKey`.
+- **Consumer action:** none (a consumer counting `bundle.attendees` itself should count
+  `uniqueAttendees(...)` instead).
+
 ## 0.7.7
 ### Meet wave 3: after the meeting
 
+- **Fix — the room header on a phone** (2026-09-27, 390px): the header wraps into rows and
+  each host control stays one line; before, the title read "Hollo…" and "End meeting for
+  everyone" stacked into four underlined words.
+- **Fix — a removed participant reads the server's sentence, not "invalid token: revoked"**
+  (2026-09-27, live room): a server-side disconnect now invalidates the cached room token, so
+  the next Join mints again and shows "A host removed you from this meeting."
+- **Fix — a host's per-person menu opens beside its button** (2026-09-27, live room): the
+  popover was hidden but IN FLOW on its first frame, so the anchor it measured was button +
+  menu and it opened a full menu-height lower — Make host, Report and Remove from meeting were
+  below the fold. It is now fixed-positioned from the first frame, re-placed when the layout
+  moves, and pinned + scrollable when neither side of the button holds it.
 - **Fix — the waiting room for people from another organization and for denied guests**
   (2026-09-27, verified in a real room): a signed-in link-follower now knocks as `user:<id>`, the
   identity the server's lobby row carries — before, the host saw two entries and admitting one

package/dist/index.cjs

@@ -84,6 +84,7 @@ __export(src_exports, {
   attendanceCsv: () => attendanceCsv,
   attendanceReport: () => attendanceReport,
   attendanceTotals: () => attendanceTotals,
+  attendeeKey: () => attendeeKey,
   callInviteAction: () => callInviteAction,
   callInviteText: () => callInviteText,
   callStateLabel: () => callStateLabel,
@@ -187,6 +188,7 @@ __export(src_exports, {
   summarizeMeetingRecordRestrictions: () => summarizeMeetingRecordRestrictions,
   transcriptDocument: () => transcriptDocument,
   unavailableFrom: () => unavailableFrom,
+  uniqueAttendees: () => uniqueAttendees,
   userTopic: () => userTopic,
   wasRefused: () => wasRefused,
   wrapUpStatus: () => wrapUpStatus
@@ -218,6 +220,20 @@ var MeetError = class extends Error {
     return this.code === "permission-denied" || this.code === "device-not-found" || this.code === "device-in-use" || this.code === "insecure-context";
   }
 };
+var SERVER_FAILED_SENTENCE = "The server hit an unexpected problem and nothing was changed.";
+function isRawServerText(value) {
+  const text3 = value.trim();
+  return /^internal server error\b/i.test(text3) || /^[A-Z][A-Za-z0-9]*(Error|Exception)$/.test(text3) || /^[a-z][a-z0-9]*(_[a-z0-9]+)+$/.test(text3);
+}
+function humanDatabaseSentence(message) {
+  const text3 = message.trim();
+  if (text3.length === 0 || isRawServerText(text3)) return null;
+  if (/permission denied for|violates|relation "|does not exist|syntax error|column "|jwt|pgrst|duplicate key|null value in/i.test(
+    text3
+  ))
+    return null;
+  return text3;
+}
 var FORBIDDEN_CODES = /* @__PURE__ */ new Set(["42501", "PGRST301", "PGRST302"]);
 var NOT_FOUND_CODES = /* @__PURE__ */ new Set(["PGRST116", "PGRST205"]);
 var CONFLICT_CODES = /* @__PURE__ */ new Set(["23505", "40001", "55000"]);
@@ -245,8 +261,8 @@ function normalizeMeetError(error, operation) {
   if (code !== void 0 && FORBIDDEN_CODES.has(code)) {
     return new MeetError(
       "forbidden",
-      `${operation}: denied by the database (${code}: ${message})`,
-      "Authorization is RLS plus auth-checked RPCs (R5). Fix the policy or the caller's membership \u2014 never route around it with a service-role client in a browser.",
+      humanDatabaseSentence(message) ?? "You don't have access to do that in this meeting.",
+      "Ask the meeting's host to do it, or to give you access.",
       error
     );
   }
@@ -261,15 +277,16 @@ function normalizeMeetError(error, operation) {
   if (code !== void 0 && CONFLICT_CODES.has(code)) {
     return new MeetError(
       "conflict",
-      `${operation}: unique violation (${code}: ${message})`,
-      "A concurrent writer won. Re-read the meeting rather than retrying the insert \u2014 for a call invite this is the idempotency key doing its job.",
+      humanDatabaseSentence(message) ?? "This meeting changed a moment ago, so nothing was saved.",
+      "Reload to see the latest, then try again.",
       error
     );
   }
+  const hint = error?.hint;
   return new MeetError(
     "transport",
-    `${operation}: ${message}`,
-    "Transient transport failure. Reads retry once; a join failure surfaces to the room phase as `failed` with this message rather than hanging on a spinner.",
+    /failed to fetch|networkerror|load failed|network request failed/i.test(message) ? "The server could not be reached." : humanDatabaseSentence(message) ?? SERVER_FAILED_SENTENCE,
+    typeof hint === "string" && humanDatabaseSentence(hint) !== null ? hint.trim() : "Nothing was changed \u2014 retry in a moment; if it keeps happening, reload the page.",
     error
   );
 }
@@ -677,6 +694,24 @@ function readText(data) {
   }
   return "";
 }
+async function refusalFromResponse(response, operation, fallbackRemedy) {
+  let body = null;
+  try {
+    body = await response.json();
+  } catch {
+    body = null;
+  }
+  const said = readServerRefusal(body);
+  const code = response.status === 403 || response.status === 401 ? "forbidden" : response.status === 404 ? "not-found" : response.status === 409 ? "conflict" : "transport";
+  const error = new MeetError(
+    code,
+    said?.reason ?? SERVER_FAILED_SENTENCE,
+    said !== null && said.remedy.length > 0 ? said.remedy : fallbackRemedy
+  );
+  error.operation = operation;
+  error.status = response.status;
+  return error;
+}
 function readServerRefusal(body) {
   if (typeof body === "string") {
     return body.trim().length === 0 ? null : { reason: body.trim(), remedy: "" };
@@ -695,7 +730,8 @@ function readServerRefusal(body) {
     return messages.length === 0 ? null : { reason: messages.join("; "), remedy: "" };
   }
   if (typeof detail === "object" && detail !== null) return readServerRefusal(detail);
-  const reason = typeof row["reason"] === "string" ? row["reason"] : typeof row["message"] === "string" ? row["message"] : typeof detail === "string" ? detail : typeof row["error"] === "string" ? row["error"] : null;
+  const humanOrNull = (value) => typeof value === "string" && value.trim().length > 0 && !isRawServerText(value) ? value.trim() : null;
+  const reason = humanOrNull(row["reason"]) ?? humanOrNull(row["message"]) ?? humanOrNull(row["user_message"]) ?? (typeof detail === "string" ? humanOrNull(detail) : null) ?? (typeof row["message"] === "string" || typeof row["error"] === "string" || typeof detail === "string" ? SERVER_FAILED_SENTENCE : null);
   if (reason === null) return null;
   const remedy = typeof row["remedy"] === "string" ? row["remedy"] : "";
   if (remedy.length > 0) return { reason, remedy };
@@ -3378,6 +3414,7 @@ var EMPTY = {
   spotlight: null,
   moderationNotice: null,
   actionNotice: null,
+  privateChatRequest: null,
   effects: DEFAULT_EFFECTS,
   transcript: [],
   notes: [],
@@ -4068,9 +4105,55 @@ function ms(iso) {
   const at = new Date(iso).getTime();
   return Number.isNaN(at) ? null : at;
 }
+function attendeeKey(person, names = {}) {
+  if (person.userId !== null) return `user:${person.userId}`;
+  if (person.isAgent) return `agent:${person.identity}`;
+  const name = displayNameFor(names, person.identity, person.displayName).trim().toLowerCase();
+  return name.length > 0 ? `guest-name:${name}` : `identity:${person.identity}`;
+}
+function uniqueAttendees(attendees, names = {}) {
+  const byKey = /* @__PURE__ */ new Map();
+  for (const person of attendees) {
+    const key = attendeeKey(person, names);
+    const kept = byKey.get(key);
+    if (kept === void 0) {
+      byKey.set(key, person);
+      continue;
+    }
+    const keptJoin = ms(kept.joinedAt);
+    const join = ms(person.joinedAt);
+    if (join !== null && (keptJoin === null || join < keptJoin)) byKey.set(key, person);
+  }
+  return [...byKey.values()];
+}
+var KIND_RANK = {
+  host: 0,
+  cohost: 1,
+  member: 2,
+  "signed-in guest": 3,
+  guest: 4,
+  assistant: 5
+};
+function coveredMs(intervals) {
+  const sorted = [...intervals].sort((a, b) => a[0] - b[0]);
+  let total = 0;
+  let from = -Infinity;
+  let to = -Infinity;
+  for (const [start, end] of sorted) {
+    if (start > to) {
+      if (to > from) total += to - from;
+      from = start;
+      to = end;
+    } else if (end > to) {
+      to = end;
+    }
+  }
+  if (to > from) total += to - from;
+  return total;
+}
 function attendanceReport(meeting, attendees, names = {}) {
   const end = ms(meeting.endedAt);
-  const rows = attendees.map((person) => {
+  const perJoin = attendees.map((person) => {
     const joined = ms(person.joinedAt);
     const left = ms(person.leftAt);
     const until = left ?? end;
@@ -4085,17 +4168,45 @@ function attendanceReport(meeting, attendees, names = {}) {
       leftAt: person.leftAt,
       durationMs: duration,
       durationToEnd: duration !== null && left === null,
-      admission: person.admissionState
+      admission: person.admissionState,
+      joins: 1
     };
   });
-  const rank = {
-    host: 0,
-    cohost: 1,
-    member: 2,
-    "signed-in guest": 3,
-    guest: 4,
-    assistant: 5
-  };
+  const groups = /* @__PURE__ */ new Map();
+  attendees.forEach((person, index) => {
+    const key = attendeeKey(person, names);
+    const group = groups.get(key) ?? { rows: [], people: [] };
+    group.rows.push(perJoin[index]);
+    group.people.push(person);
+    groups.set(key, group);
+  });
+  const rows = [...groups.values()].map(({ rows: joins }) => {
+    if (joins.length === 1) return joins[0];
+    const came = joins.filter((row) => row.joinedAt !== null);
+    const first = [...came].sort((a, b) => (ms(a.joinedAt) ?? 0) - (ms(b.joinedAt) ?? 0))[0] ?? joins[0];
+    const kind = joins.reduce(
+      (best, row) => KIND_RANK[row.kind] < KIND_RANK[best] ? row.kind : best,
+      first.kind
+    );
+    const open = came.some((row) => row.leftAt === null);
+    const lastLeave = came.reduce((latest, row) => {
+      const left = ms(row.leftAt);
+      return left !== null && (latest === null || left > latest) ? left : latest;
+    }, null);
+    const intervals = came.flatMap((row) => {
+      const joined = ms(row.joinedAt);
+      return joined !== null && row.durationMs !== null ? [[joined, joined + row.durationMs]] : [];
+    });
+    return {
+      ...first,
+      kind,
+      leftAt: open ? null : lastLeave === null ? null : new Date(lastLeave).toISOString(),
+      durationMs: intervals.length === 0 ? null : coveredMs(intervals),
+      durationToEnd: came.some((row) => row.durationToEnd),
+      joins: came.length === 0 ? joins.length : came.length
+    };
+  });
+  const rank = KIND_RANK;
   return rows.sort((a, b) => {
     if (rank[a.kind] !== rank[b.kind]) return rank[a.kind] - rank[b.kind];
     return (ms(a.joinedAt) ?? Number.MAX_SAFE_INTEGER) - (ms(b.joinedAt) ?? Number.MAX_SAFE_INTEGER);
@@ -4253,7 +4364,10 @@ function bullet(note) {
 function notesDocument(bundle, options = {}) {
   const out = [`# ${bundle.meeting.title}`, ""];
   if (options.when) out.push(`_${options.when}_`, "");
-  const people = bundle.attendees.filter((person) => !person.isAgent && person.joinedAt !== null).map((person) => displayNameFor(bundle.names, person.identity, person.displayName));
+  const people = uniqueAttendees(
+    bundle.attendees.filter((person) => !person.isAgent && person.joinedAt !== null),
+    bundle.names
+  ).map((person) => displayNameFor(bundle.names, person.identity, person.displayName));
   if (people.length > 0) out.push(`**Attended:** ${people.join(", ")}`, "");
   out.push("## Summary", "", bundle.summary?.text.trim() || "No summary was written.", "");
   out.push("## Decisions", "");
@@ -4317,19 +4431,8 @@ async function loadMeetingChat(api, meetingId, roomToken = null) {
     refusal: null
   };
 }
-async function refusalOf(response, operation) {
-  let body = null;
-  try {
-    body = await response.json();
-  } catch {
-    body = null;
-  }
-  const said = readServerRefusal(body);
-  return new MeetError(
-    response.status === 403 || response.status === 401 ? "forbidden" : response.status === 404 ? "not-found" : response.status === 409 ? "conflict" : "transport",
-    `${operation}: ${said?.reason ?? `the server answered ${response.status}`}`,
-    said?.remedy || "Nothing was sent. Try again."
-  );
+function refusalOf(response, operation) {
+  return refusalFromResponse(response, operation, "Nothing was sent. Try again.");
 }
 function asRecipient(row) {
   const sources = Array.isArray(row["sources"]) ? row["sources"] : [];
@@ -4390,18 +4493,11 @@ async function sendFollowUp(api, meetingId, input) {
 }
 
 // src/core/management.ts
-async function refusal(response, operation) {
-  let body = null;
-  try {
-    body = await response.json();
-  } catch {
-    body = null;
-  }
-  const said = readServerRefusal(body);
-  return new MeetError(
-    response.status === 403 || response.status === 401 ? "forbidden" : response.status === 404 ? "not-found" : response.status === 409 ? "conflict" : "transport",
-    `${operation}: ${said?.reason ?? `the server answered ${response.status}`}`,
-    said?.remedy || "The meeting itself is saved \u2014 only the telling failed. Try again; the call is idempotent."
+function refusal(response, operation) {
+  return refusalFromResponse(
+    response,
+    operation,
+    "The meeting itself is saved \u2014 only the telling failed. Try again; the call is idempotent."
   );
 }
 function count(body, key) {
@@ -4727,6 +4823,10 @@ function createRoomEngine(options) {
       }
     },
     disconnected(reason) {
+      if (current !== null) {
+        tokens.invalidate(current.roomName);
+        current = null;
+      }
       store.patch({ phase: "left" });
       options.onLeft?.(reason);
     }
@@ -5029,19 +5129,8 @@ function createInRoomClient(options) {
     const token = options.resolveRoomToken();
     return token === null ? void 0 : { [ROOM_TOKEN_HEADER]: token };
   }
-  async function refusal2(response, operation, remedy) {
-    let body = null;
-    try {
-      body = await response.json();
-    } catch {
-      body = null;
-    }
-    const said = readServerRefusal(body);
-    return new MeetError(
-      response.status === 403 || response.status === 401 ? "forbidden" : response.status === 404 ? "not-found" : response.status === 409 ? "conflict" : "transport",
-      said?.reason ?? `${operation}: the server answered ${response.status}`,
-      said?.remedy !== void 0 && said.remedy.length > 0 ? said.remedy : remedy
-    );
+  function refusal2(response, operation, remedy) {
+    return refusalFromResponse(response, operation, remedy);
   }
   return {
     async moderate(request) {