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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nanocollective/roster",
-  "version": "0.1.0-alpha.37",
+  "version": "0.1.0-alpha.39",
   "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

@@ -194,3 +194,6 @@ textarea.dropping{border-color:var(--accent);background:color-mix(in srgb,var(--
 /* 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)}
+
+/* A little room between rows, so unread cards and the selected row do not run together. */
+.irow{margin:3px 0}

package/templates/portal/css/layout.css

@@ -78,13 +78,15 @@
 .sidefoot .nav:hover{color:var(--ink-dim)}
 
 /* ---------- main ---------- */
-main{padding:36px 44px 96px;max-width:1220px;overflow-x:hidden}
+/* clip, not hidden: hidden makes main a scroll container, and nothing inside it can be sticky. */
+main{padding:36px 44px 96px;max-width:1220px;overflow-x:clip}
 
 /* ---------- tree beside viewer ---------- */
 .split{display:grid;grid-template-columns:314px 1fr;gap:16px;align-items:start}
 @media (max-width:900px){.split{grid-template-columns:1fr}}
+/* The list beside a viewer stays in view while a long file or thread scrolls. */
 .tree{background:var(--panel);border:1px solid var(--line);border-radius:var(--r-lg);padding:6px;
-      max-height:74vh;overflow:auto;box-shadow:var(--shadow-sm)}
+      max-height:calc(100vh - 40px);overflow:auto;box-shadow:var(--shadow-sm);position:sticky;top:20px}
 .tdir{font:600 11px var(--mono);color:var(--ink-faint);padding:9px 8px 4px;letter-spacing:.02em}
 .tdir.first{padding-top:3px}
 .tfile{display:flex;align-items:baseline;gap:10px;width:100%;text-align:left;background:none;

package/templates/portal/js/app.js

@@ -326,7 +326,6 @@ function paintInboxLinks() {
   // 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"],
   ];

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

@@ -686,7 +686,20 @@ function inboxScreen(m, opts) {
           body: body || undefined,
         });
         DRAFTS.delete(key);
-        await refreshAll(false);
+        /* What changed is known, so it is changed here rather than re-reading every repo,
+           which redrew the whole inbox for one issue. The next refresh confirms it. */
+        const me = humansOf()[0]?.github ?? "you";
+        const now = new Date().toISOString();
+        item.events = [
+          ...(item.events ?? []),
+          ...(body ? [{ type: "comment", actor: me, createdAt: now, body }] : []),
+          { type: closing ? "closed" : "reopened", actor: me, createdAt: now },
+        ];
+        item.state = closing ? "CLOSED" : "OPEN";
+        item.updatedAt = now;
+        stampCounts();
+        paint();
+        openThread({ repo: item.repo, number: item.number, kind: item.kind });
       } catch (e) { failed(e); }
     };
     buttons.push(closeBtn);
@@ -714,7 +727,12 @@ function inboxScreen(m, opts) {
         busy(true, "merging…");
         try {
           await post({ action: "merge", repo: item.repo, number: item.number });
-          await refreshAll(false);
+          // As with closing: the change is known, so it is made here rather than re-reading every repo.
+          item.state = "MERGED";
+          item.updatedAt = new Date().toISOString();
+          stampCounts();
+          paint();
+          openThread({ repo: item.repo, number: item.number, kind: item.kind });
         } catch (e) { failed(e); }
       };
       buttons.push(merge);
@@ -780,6 +798,19 @@ function inboxScreen(m, opts) {
  * you are still typing and the button has to stay as it was; once it closes there is a request
  * in flight against GitHub that can take seconds, and that is what needs saying.
  */
+/**
+ * Who a reply is for, by default: the staff member whose tracker the issue is on, or whose App
+ * opened it. They are only woken by an @mention, so a reply without one reached nobody.
+ */
+export function addressee(item) {
+  const staff = S.data?.staff ?? [];
+  const s =
+    staff.find((x) => x.brain === item.repo) ??
+    staff.find((x) => (x.soloBots ?? []).includes(item.author));
+  if (!s) return "";
+  return (s.mention || "@" + s.handle) + " ";
+}
+
 async function openReply(item, anchor, onSend) {
   const key = item.repo + "#" + item.number + (anchor ? " · " + anchor.path : "");
   let deaf = null;
@@ -788,7 +819,7 @@ async function openReply(item, anchor, onSend) {
     title: "Reply to " + item.repo + " #" + item.number,
     hint: anchor ? anchor.path : item.title,
     confirm: "Comment",
-    value: DRAFTS.get(key) ?? "",
+    value: DRAFTS.get(key) ?? addressee(item),
     /* It goes out through whoever's `gh` is signed in here, which is only knowably one person
        when the org has one. With two it would be a guess, and a wrong name on a reply box is
        worse than no name. */
@@ -799,6 +830,10 @@ async function openReply(item, anchor, onSend) {
         else DRAFTS.delete(key);
       };
       deaf = deafNote(ta, item, anchor);
+      // Typing starts after the @mention, not in front of it.
+      setTimeout(() => {
+        ta.selectionStart = ta.selectionEnd = ta.value.length;
+      }, 0);
       return el("div", {}, [attachBox(ta, () => item.repo).node, deaf.node]);
     },
   });