@nanocollective/roster 0.1.0-alpha.34 → 0.1.0-alpha.36

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.34",
+  "version": "0.1.0-alpha.36",
   "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

@@ -57,7 +57,7 @@
 /* A bottom margin here would collapse against the row's own 16px and change nothing, so the
    extra room is put on the row. General sibling, not adjacent: labels can sit between them. */
 .tconflict ~ .tacts{margin-top:24px}
-.tlabels{margin-top:10px}
+.tmetalabels{display:inline-flex;gap:6px;margin-left:10px;vertical-align:middle;font-family:var(--sans)}
 /* A comment is a card with a header strip, so each message has a clear start and end. */
 .cmt{border:1px solid var(--line);border-radius:var(--r-lg);margin:14px 0;background:var(--panel);overflow:hidden}
 .cmt.human{border-color:color-mix(in srgb,var(--accent) 40%,var(--line))}
@@ -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(--accent)}
+.irow.unread .ititle{font-weight:700;color:var(--ink)}

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(
@@ -301,7 +304,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,22 +363,33 @@ 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 =
       '<div class="meta">' + esc(item.repo) + " · " + (item.kind === "pr" ? "PR " : "") + "#" + item.number +
         ' · <span class="' + (item.state === "OPEN" ? "ok" : "") + '">' + esc(item.state) + "</span>" +
-        " · " + esc(item.author) + " · " + ago(item.updatedAt) + "</div>" +
+        " · " + esc(item.author) + " · " + ago(item.updatedAt) +
+        // The labels sit on this line, after the time, rather than taking a line of their own.
+        (item.labels.length
+          ? '<span class="tmetalabels">' +
+            item.labels.map((l) => '<span class="chip ' + tone(l) + '">' + esc(l) + "</span>").join("") +
+            "</span>"
+          : "") +
+        "</div>" +
       "<h3>" + esc(item.title) + "</h3>" +
       (item.mergeable === "CONFLICTING"
         ? '<p class="tconflict">This branch conflicts with its base and cannot be merged ' +
           "until someone rebases or merges the base into it.</p>"
         : "") +
-      (item.labels.length
-        ? '<div class="row tlabels">' +
-          item.labels.map((l) => '<span class="chip ' + tone(l) + '">' + esc(l) + "</span>").join("") +
-          "</div>"
-        : "");
+      "";
     head.append(threadActions(item));
     delete viewer.dataset.wait;
     viewer.replaceChildren(head);