@nanocollective/roster 0.1.0-alpha.35 → 0.1.0-alpha.37

package/dist/cli.js

@@ -5816,6 +5816,32 @@ function short4(e) {
   return line.length > 200 ? line.slice(0, 199) + "\u2026" : line;
 }
 
+// src/lib/notifications.ts
+async function unreadFor(owner, pages = 5) {
+  const out = /* @__PURE__ */ new Map();
+  for (let page2 = 1; page2 <= pages; page2++) {
+    const res = await ghJson(["api", `notifications?per_page=100&page=${page2}`]);
+    if (!res.ok || !res.data?.length) break;
+    for (const n of res.data) {
+      if (!n.unread || n.repository?.owner?.login?.toLowerCase() !== owner.toLowerCase()) continue;
+      const number = /\/(?:issues|pulls)\/(\d+)$/.exec(n.subject?.url ?? "")?.[1];
+      if (!number || !n.repository?.full_name) continue;
+      out.set(`${n.repository.full_name}#${number}`, {
+        thread: n.id,
+        reason: n.reason,
+        updatedAt: n.updated_at
+      });
+    }
+    if (res.data.length < 100) break;
+  }
+  return out;
+}
+async function markRead(thread) {
+  if (!/^\d+$/.test(thread)) return false;
+  const res = await api(`notifications/threads/${thread}`, ["-X", "PATCH"]);
+  return res.ok;
+}
+
 // src/lib/paste.ts
 var BEGIN = (path) => `<<<ROSTER FILE ${path}>>>`;
 var END = "<<<ROSTER END>>>";
@@ -7349,6 +7375,24 @@ async function portalCommand(argv) {
         }).catch((err) => json(res, { labels: [], error: String(err?.message ?? err) }));
         return;
       }
+      if (url.pathname === "/api/notifications/read" && req.method === "POST") {
+        if (!writeAllowed(req)) {
+          refuseWrite(res);
+          return;
+        }
+        body(req).then(async (raw) => {
+          const payload = JSON.parse(raw || "{}");
+          const ok = await markRead(String(payload.thread ?? ""));
+          if (ok && cache && payload.ref) {
+            const data = JSON.parse(cache.body);
+            for (const i of data.items ?? [])
+              if (`${i.repo}#${i.number}` === payload.ref) i.unread = null;
+            cache = { at: cache.at, body: JSON.stringify(data) };
+          }
+          json(res, { ok });
+        }).catch((err) => json(res, { ok: false, error: String(err?.message ?? err) }));
+        return;
+      }
       if (url.pathname === "/api/inbox") {
         const fresh = url.searchParams.get("refresh") === "1";
         if (!fresh && cache && Date.now() - cache.at < TTL) {
@@ -7359,9 +7403,15 @@ async function portalCommand(argv) {
           res.end(cache.body);
           return;
         }
-        fetchInbox(knownRepos()).then((data) => {
+        const owner = readOrg(w.opsDir, parseYaml).org;
+        Promise.all([fetchInbox(knownRepos()), unreadFor(owner).catch(() => /* @__PURE__ */ new Map())]).then(([data, unread]) => {
           const body2 = JSON.stringify({
             ...data,
+            // Marked from GitHub's notifications, so unread here is unread there.
+            items: data.items.map((i) => ({
+              ...i,
+              unread: unread.get(`${i.repo}#${i.number}`) ?? null
+            })),
             repos: knownRepos(),
             fetchedAt: (/* @__PURE__ */ new Date()).toISOString()
           });

package/docs/portal.md

@@ -83,13 +83,13 @@ goes away; the repo picker lives on under [Org](#org).
 Everything open across the org, from one GraphQL call per repo. Bodies and full timelines come
 down with the list, so opening a thread is a render rather than a request.
 
-- **Filter by staff member.** An item belongs to somebody if it is in their brain repo, their
-  own App wrote it, a peer addressed it to them with a `from-<handle>` label, or it is assigned
-  to them. The shared public identity cannot name one staff member, so an item it wrote counts
-  for anyone who works in that repo. Items authored by humans belong to nobody, which is
-  correct.
-- **Scope** to everything, what is assigned to you, decisions, or open PRs.
-- **Open, recently closed, or both.** Open by default: an inbox is what is waiting on
+- **The links under Inbox in the sidebar filter it.** All; Unread; each staff member, which
+  lists the issues on their own tracker whoever filed them; and Issues, the product repos.
+- **Unread comes from your GitHub notifications.** A thread with activity you have not read has
+  a bar on the left and a bold title, and Unread lists only those, with a count. Opening one
+  marks it read on GitHub too, and reading it on GitHub clears it here. GitHub only notifies you
+  about repos you watch and threads you are part of.
+- **Open or closed.** Open by default: an inbox is what is waiting on
   somebody, and months of finished work mixed into that answers a different question. Closed
   work reaches back 45 days, up to 30 issues and 30 pull requests per repository, and carries
   a shorter timeline than open work because it is there to be read rather than triaged. A

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nanocollective/roster",
-  "version": "0.1.0-alpha.35",
+  "version": "0.1.0-alpha.37",
   "description": "An agent-run org, powered by GitHub. Scaffold AI staff members whose brain is a repo.",
   "type": "module",
   "bin": {

package/templates/portal/css/inbox.css

@@ -190,3 +190,7 @@ textarea.dropping{border-color:var(--accent);background:color-mix(in srgb,var(--
 .tevmore[aria-expanded="true"] .caret{transform:none}
 .tevquiet{border-left:1px solid var(--line);margin-left:9px;padding-left:14px}
 .tevquiet .tev{padding:5px 0;font-size:12px}
+
+/* Unread, from GitHub's notifications: a bar on the left and a heavier title. Cleared on open. */
+.irow.unread{box-shadow:inset 3px 0 0 var(--blue);background:color-mix(in srgb,var(--blue) 6%,transparent)}
+.irow.unread .ititle{font-weight:700;color:var(--ink)}

package/templates/portal/css/layout.css

@@ -169,3 +169,5 @@ main{padding:36px 44px 96px;max-width:1220px;overflow-x:hidden}
 .nav.inboxlink{display:flex;align-items:center;gap:8px}
 .nav.inboxlink .ic,.staffrow .ic{--ic:14px;color:var(--ink-faint);flex:none}
 .nav.inboxlink[aria-current="true"] .ic{color:var(--accent)}
+/* The unread count matches the unread rows. Empty, it stays grey like every other badge. */
+#unreadcount:not(:empty):not(.wait){background:var(--blue);color:#fff}

package/templates/portal/js/app.js

@@ -325,6 +325,8 @@ function paintInboxLinks() {
   sub.replaceChildren();
   // Inbox itself is everything. Under it: each staff member, then the product repos' issues.
   const links = [
+    ["", "All", "inbox"],
+    ["@unread", "Unread", "dot"],
     ...S.data.staff.map((s) => [s.handle, s.name, staffIcon(s)]),
     ["@issues", "Issues", "issue-open"],
   ];
@@ -332,12 +334,12 @@ function paintInboxLinks() {
     const b = el("button", { className: "nav subnav inboxlink" }, [
       icon(glyph, "ic"),
       el("span", { className: "t", textContent: label }),
+      ...(who === "@unread" ? [el("span", { className: "n", id: "unreadcount" })] : []),
     ]);
     b.dataset.who = who;
     b.onclick = () => {
       S.view = "inbox";
-      // A second click on the one you are on goes back to everything.
-      S.inboxStaff = S.inboxStaff === who ? "" : who;
+      S.inboxStaff = who;
       S.inboxFilter = "";
       S.inboxOpen = null;
       render();

package/templates/portal/js/refresh.js

@@ -10,7 +10,7 @@
 import { getInbox, getOrg, getSync } from "./api.js";
 import { $, ago, el, esc } from "./dom.js";
 import { render } from "./router.js";
-import { openCount, openPrCount, S } from "./state.js";
+import { openCount, openPrCount, S, unreadCount } from "./state.js";
 
 /**
  * The inbox, fetched at most once at a time.
@@ -55,6 +55,7 @@ export function stampCounts() {
   for (const [sel, n] of [
     ["#inboxcount", openCount],
     ["#prcount", openPrCount],
+    ["#unreadcount", unreadCount],
   ]) {
     const slot = document.querySelector(sel);
     if (!slot) continue;

package/templates/portal/js/state.js

@@ -86,6 +86,10 @@ export const S = {
 export const openCount = () =>
   (S.inbox?.items ?? []).filter((i) => i.state === "OPEN" && i.kind !== "pr").length;
 
+/** Issues with activity you have not read, from GitHub's notifications. */
+export const unreadCount = () =>
+  (S.inbox?.items ?? []).filter((i) => i.unread && i.state === "OPEN" && i.kind !== "pr").length;
+
 /** The same, for the screen that is only pull requests. */
 export const openPrCount = () =>
   (S.inbox?.items ?? []).filter((i) => i.state === "OPEN" && i.kind === "pr").length;

package/templates/portal/js/views/inbox.js

@@ -235,6 +235,7 @@ function inboxScreen(m, opts) {
     // "Issues" is the product repos: every repo a staff member works in.
     const products = new Set(S.data.staff.flatMap((s) => s.worksIn ?? []));
     const issuesOnly = staffPick === "@issues";
+    const unreadOnly = staffPick === "@unread";
     const whoseStaff = staffPick && !issuesOnly ? S.data.staff.find((s) => s.handle === staffPick) : null;
     /* The two screens partition the org rather than overlap on it. Pending work is every pull
        request; the inbox is everything else. An inbox that also listed the PRs said the same
@@ -242,7 +243,9 @@ function inboxScreen(m, opts) {
     const mine = S.inbox.items
       /* A staff member's link is their tracker: what is in their repo, whoever filed it. Matching
          on who filed or was named put an issue the CMO filed for the CTO under both of them. */
-      .filter((i) => (issuesOnly ? products.has(i.repo) : !whoseStaff || i.repo === whoseStaff.brain))
+      .filter((i) =>
+        unreadOnly ? Boolean(i.unread) : issuesOnly ? products.has(i.repo) : !whoseStaff || i.repo === whoseStaff.brain,
+      )
       .filter((i) => (i.kind === "pr") === !!opts.prs);
     const isOpen = (i) => i.state === "OPEN";
     const scoped = mine.filter(
@@ -291,9 +294,15 @@ function inboxScreen(m, opts) {
       list.append(el("div", { className: "tdir err", textContent: e }));
     }
 
-    // Newest first, flat. Grouping by repo buried a fresh comment under whichever project
-    // happened to sort first; the project moves onto the row instead.
-    items = items.slice().sort((a, b) => String(b.updatedAt).localeCompare(String(a.updatedAt)));
+    // Unread first, then newest first, flat. Grouping by repo buried a fresh comment under
+    // whichever project happened to sort first; the project moves onto the row instead.
+    items = items
+      .slice()
+      .sort(
+        (a, b) =>
+          Number(Boolean(b.unread)) - Number(Boolean(a.unread)) ||
+          String(b.updatedAt).localeCompare(String(a.updatedAt)),
+      );
     if (!items.length) {
       list.append(el("p", { className: "empty", textContent: "Nothing matches." }));
       return;
@@ -301,7 +310,7 @@ function inboxScreen(m, opts) {
 
     for (const i of items) {
       const shutState = i.state === "MERGED" ? "merged" : i.state === "OPEN" ? "" : "closed";
-      const b = el("button", { className: "irow" + (shutState ? " shut" : "") });
+      const b = el("button", { className: "irow" + (shutState ? " shut" : "") + (i.unread ? " unread" : "") });
       const yours = onAHuman(i);
       const chips = i.labels
         .slice(0, 3)
@@ -360,6 +369,14 @@ function inboxScreen(m, opts) {
     }
     const row = list.querySelector('[data-ref="' + item.repo + "#" + item.number + '"]');
     if (row) markCurrent(list, row);
+    // Opening it is reading it: cleared here at once, and on GitHub in the background.
+    if (item.unread) {
+      const thread = item.unread.thread;
+      item.unread = null;
+      row?.classList.remove("unread");
+      stampCounts();
+      post({ thread, ref: item.repo + "#" + item.number }, "/api/notifications/read").catch(() => {});
+    }
 
     const head = el("div", { className: "thead" });
     head.innerHTML =