@nanocollective/roster 0.1.0-alpha.43 → 0.1.0-alpha.44

package/templates/portal/index.html

@@ -13,6 +13,7 @@
 <link rel="stylesheet" href="/assets/css/markdown.css">
 <link rel="stylesheet" href="/assets/css/brain.css">
 <link rel="stylesheet" href="/assets/css/inbox.css">
+<link rel="stylesheet" href="/assets/css/home.css">
 <link rel="stylesheet" href="/assets/css/graph.css">
 <link rel="stylesheet" href="/assets/css/diff.css">
 <link rel="stylesheet" href="/assets/css/health.css">
@@ -30,18 +31,17 @@
     <!-- Only while something is left to set up; app.js decides. -->
     <button class="nav top" id="setupnav" data-view="setup" hidden>
       <span class="ic" data-icon="task-done"></span> Getting started
+      <span class="n" id="setupcount"></span>
     </button>
-    <button class="nav top" id="inboxnav" data-view="inbox">
-      <span class="ic" data-icon="inbox"></span> Inbox
+    <button class="nav top" id="homenav" data-view="home" title="What needs you, and who is working">
+      <span class="ic" data-icon="inbox"></span> Home
+      <span class="n" id="homecount"></span>
+    </button>
+    <button class="nav" id="inboxnav" data-view="inbox" title="Every open issue on each staff member's tracker">
+      <span class="ic" data-icon="issue-open"></span> Trackers
       <span class="n" id="inboxcount"></span>
     </button>
     <div class="staffviews" id="inboxsub" hidden></div>
-    <!-- "Pending work", not "Pull requests": what is behind it is finished work waiting on you,
-         and the word for how it is delivered is not the word for what it is. -->
-    <button class="nav" id="prsnav" data-view="prs" title="Finished work waiting on you to merge">
-      <span class="ic" data-icon="merged"></span> Pending work
-      <span class="n" id="prcount"></span>
-    </button>
     <button class="nav" id="runsnav" data-view="runs" title="What ran, how it ended, and what it cost">
       <span class="ic" data-icon="play"></span> Runs
     </button>

package/templates/portal/js/api.js

@@ -10,6 +10,9 @@ export const getSync = () => json("/api/sync").catch(() => null);
 /** Every staff member's recent runs and 30-day spend. Online only; offline it says so. */
 export const getRuns = (force) => json("/api/runs" + (force ? "?refresh=1" : ""));
 
+/** Who is running now, what started it, and what finished in the last few hours. Never cached. */
+export const getLive = () => json("/api/live");
+
 /** The repos org.yaml lists. Off disk, so the new-issue form does not wait on the inbox. */
 export const getRepos = () => json("/api/repos");
 

package/templates/portal/js/app.js

@@ -20,6 +20,7 @@ import { viewChanged } from "./views/changed.js";
 import { viewDocs } from "./views/docs.js";
 import { viewGraph } from "./views/graph.js";
 import { viewHealth } from "./views/health.js";
+import { viewHome } from "./views/home.js";
 import { viewInbox, viewPrs } from "./views/inbox.js";
 import { viewOrg } from "./views/org.js";
 import { viewPrompt } from "./views/prompt.js";
@@ -30,6 +31,7 @@ import { viewStaff } from "./views/staff.js";
 
 const SCREEN = {
   setup: viewGettingStarted,
+  home: viewHome,
   inbox: viewInbox,
   prs: viewPrs,
   runs: viewRuns,
@@ -57,8 +59,8 @@ const NEEDS = ["org", "name", "opsName", "staff"];
 /** The org-wide screens in the sidebar, by button. */
 const NAV = [
   ["#setupnav", "setup"],
+  ["#homenav", "home"],
   ["#inboxnav", "inbox"],
-  ["#prsnav", "prs"],
   ["#runsnav", "runs"],
   ["#staffnav", "staff"],
   ["#orgnav", "org"],
@@ -100,7 +102,7 @@ export async function boot() {
     /* Every one of these reads a tenant, so during setup they answer 409 and the sidebar is a
        row of four dead ends beside an empty Staff heading. There is one thing to do on this
        screen; the navigation comes back with the org. */
-    for (const el of document.querySelectorAll("#inboxnav, #prsnav, #runsnav, #orgnav, #staffnav, #docsnav, #refreshall, .sect, #stafflist")) {
+    for (const el of document.querySelectorAll("#homenav, #inboxnav, #runsnav, #orgnav, #staffnav, #docsnav, #refreshall, .sect, #stafflist")) {
       el.hidden = true;
     }
     for (const slot of document.querySelectorAll("[data-icon]")) {
@@ -274,10 +276,10 @@ function paintSidebar() {
       el("span", { className: "hh", textContent: s.handle }),
     );
     whatsLeft(s).then((left) => {
+      paintSetupNav();
       if (!left.length) return;
       dot.hidden = false;
       head.title = "Not ready to run: " + sentence(left);
-      paintSetupNav();
     });
 
     const views = el("div", { className: "staffviews" });

package/templates/portal/js/homesort.js

@@ -0,0 +1,154 @@
+/* Where every issue and pull request goes on Home.
+ *
+ * Every open item lands in exactly one place, so nothing is missed and nothing shows twice:
+ *
+ *   needs      an ask on a person, or a pull request ready to merge
+ *   requests   something a person asked a staff member for
+ *   staff      a staff member's own work: their status issue, a peer's ask, their own issues
+ *   elsewhere  draft pull requests, and product or ops issues nobody has put on a person
+ *
+ * Closed items only matter when a staff member closed them today: that is "Closed today", with
+ * Reopen beside each, because staff now close things without asking. */
+
+/** The kinds an ask on a person carries. Older trackers used other words for a chore. */
+const KINDS = ["decision", "review", "chore"];
+const CHORE_WORDS = ["setup", "submit", "data", "blocked"];
+
+const lower = (s) => String(s ?? "").toLowerCase();
+
+/** How old a run report can be and still be this morning's. */
+const REPORT_HOURS = 20;
+
+/** A run report as lines, without the @mention it opens with. */
+function reportLines(body) {
+  return String(body ?? "")
+    .split("\n")
+    .map((l) => l.replace(/^\s*(@[\w-]+\s*)+/, "").replace(/^[-*]\s+/, "").trim())
+    .filter(Boolean)
+    .slice(0, 5);
+}
+
+/**
+ * @param items   the inbox list: open and recently closed issues and pull requests
+ * @param staff   S.data.staff: handle, name, brain, statusIssue, mention
+ * @param humans  [{ github, marker }]
+ * @param live    /api/live's staff list, or [] before it has answered
+ * @param now     for "today", in the viewer's own day
+ */
+export function sortHome(items, staff, humans, live = [], now = new Date()) {
+  const people = new Set(humans.map((h) => lower(h.github)).filter(Boolean));
+  const markers = new Set(humans.map((h) => lower(h.marker)).filter(Boolean));
+  const byBrain = new Map(staff.filter((s) => s.brain).map((s) => [s.brain, s]));
+  const working = new Set();
+  for (const l of live) {
+    const s = staff.find((x) => x.handle === l.handle);
+    for (const r of l.running ?? []) if (s && r.issue) working.add(`${s.brain}#${r.issue}`);
+  }
+
+  const out = { reports: [], needs: [], requests: [], staff: [], elsewhere: [], closedToday: [] };
+  const today = dayOf(now);
+
+  for (const item of items) {
+    const owner = byBrain.get(item.repo) ?? null;
+
+    if (item.state !== "OPEN") {
+      if (
+        owner &&
+        item.kind === "issue" &&
+        item.closedAt &&
+        dayOf(new Date(item.closedAt)) === today &&
+        !people.has(lower(item.closedBy))
+      ) {
+        out.closedToday.push({ item, staff: owner, reason: reasonOf(item, people) });
+      }
+      continue;
+    }
+
+    // The run report: the last word on a status issue, from the staff member, in the last day.
+    const c = item.lastComment;
+    if (
+      owner &&
+      item.number === owner.statusIssue &&
+      c &&
+      !people.has(lower(c.author)) &&
+      now - new Date(c.createdAt) < REPORT_HOURS * 3_600_000
+    ) {
+      out.reports.push({ staff: owner, item, at: c.createdAt, lines: reportLines(c.body) });
+    }
+
+    const place = placeOf(item, owner, people, markers);
+    const entry = { item, staff: owner, ...place };
+    if (place.place === "requests") {
+      entry.status = working.has(`${item.repo}#${item.number}`)
+        ? "working"
+        : item.lastComment && !people.has(lower(item.lastComment.author))
+          ? "answered"
+          : "waiting";
+    }
+    out[place.place].push(entry);
+  }
+
+  const oldest = (a, b) => a.item.createdAt.localeCompare(b.item.createdAt);
+  out.needs.sort(oldest);
+  out.requests.sort((a, b) => b.item.updatedAt.localeCompare(a.item.updatedAt));
+  out.closedToday.sort((a, b) => b.item.closedAt.localeCompare(a.item.closedAt));
+  return out;
+}
+
+/** Which place an open item belongs in, and for an ask, which kind it is. */
+export function placeOf(item, owner, people, markers) {
+  if (item.kind === "pr") {
+    return item.draft ? { place: "elsewhere", why: "draft" } : { place: "needs", kind: "merge" };
+  }
+
+  const labels = (item.labels ?? []).map(lower);
+  const onPerson =
+    (item.assignees ?? []).some((a) => people.has(lower(a))) ||
+    labels.some((l) => markers.has(l));
+  const byPerson = people.has(lower(item.author));
+
+  if (!owner) {
+    return onPerson && !byPerson
+      ? { place: "needs", kind: kindOf(labels) }
+      : { place: "elsewhere", why: item.role === "product" ? "contributor" : "other" };
+  }
+  if (item.number === owner.statusIssue || labels.includes("keep-open")) {
+    return { place: "staff", why: "status" };
+  }
+  if (byPerson) return { place: "requests" };
+  if (onPerson) return { place: "needs", kind: kindOf(labels) };
+  return {
+    place: "staff",
+    why: labels.some((l) => l.startsWith("from-")) ? "peer" : "own",
+  };
+}
+
+function kindOf(labels) {
+  const found = KINDS.find((k) => labels.includes(k));
+  if (found) return found;
+  return labels.some((l) => CHORE_WORDS.includes(l)) ? "chore" : "ask";
+}
+
+/** The one line a staff member closed it with: their last comment's first line. */
+function reasonOf(item, people) {
+  const c = item.lastComment;
+  if (!c || people.has(lower(c.author))) return "";
+  const line = String(c.body ?? "")
+    .split("\n")
+    .map((l) => l.trim())
+    .find(Boolean);
+  return line ? line.replace(/^[#>*\s-]+/, "").slice(0, 160) : "";
+}
+
+/** A date as the viewer's own calendar day. */
+export function dayOf(d) {
+  return `${d.getFullYear()}-${d.getMonth() + 1}-${d.getDate()}`;
+}
+
+/** Whole days from today to a YYYY-MM-DD date: 0 today, negative once it has passed. */
+export function daysUntil(date, now = new Date()) {
+  const [y, m, d] = String(date).split("-").map(Number);
+  const due = new Date(y, m - 1, d);
+  const start = new Date(now.getFullYear(), now.getMonth(), now.getDate());
+  return Math.round((due - start) / 86_400_000);
+}

package/templates/portal/js/readiness.js

@@ -33,3 +33,38 @@ export function sentence(left) {
   if (left.length < 2) return left.join("");
   return left.slice(0, -1).join(", ") + " and " + left.at(-1);
 }
+
+/** Per staff member: hire, the GitHub App, the charter, the agent credential, a first run. */
+const STAFF_STEPS = 5;
+
+/**
+ * Setup as a count, from the same answers as everything above: the org's three steps (create
+ * it, business.md, priorities.md), then five per staff member. With nobody hired, the first
+ * hire's five are counted as still to do.
+ *
+ * Null until every staff member's readiness has answered. A count that starts high and drops
+ * when GitHub replies reads as steps coming undone.
+ */
+export function progress(
+  unfinished = S.data?.unfinished ?? [],
+  staff = S.data?.staff ?? [],
+  readiness = S.readiness ?? {},
+) {
+  const orgSteps = 3;
+  let done = orgSteps - unfinished.filter((id) => id !== "hire").length;
+  let total = orgSteps;
+  if (!staff.length) return { done, total: total + STAFF_STEPS };
+  for (const s of staff) {
+    const left = readiness[s.handle];
+    if (!left) return null;
+    total += STAFF_STEPS;
+    done += STAFF_STEPS - left.length;
+  }
+  return { done, total };
+}
+
+/** One staff member's own count, or null before their readiness is known. */
+export function staffProgress(handle, readiness = S.readiness ?? {}) {
+  const left = readiness[handle];
+  return left ? { done: STAFF_STEPS - left.length, total: STAFF_STEPS } : null;
+}

package/templates/portal/js/refresh.js

@@ -10,7 +10,8 @@
 import { getInbox, getOrg, getSync } from "./api.js";
 import { $, ago, el, esc } from "./dom.js";
 import { render } from "./router.js";
-import { openCount, openPrCount, S, unreadCount } from "./state.js";
+import { sortHome } from "./homesort.js";
+import { humansOf, openCount, S, unreadCount } from "./state.js";
 
 /**
  * The inbox, fetched at most once at a time.
@@ -53,8 +54,8 @@ export function ensureInbox(force) {
 export function stampCounts() {
   const waiting = !S.inbox;
   for (const [sel, n] of [
+    ["#homecount", needsCount],
     ["#inboxcount", openCount],
-    ["#prcount", openPrCount],
     ["#unreadcount", unreadCount],
   ]) {
     const slot = document.querySelector(sel);
@@ -71,6 +72,12 @@ export function stampCounts() {
  * which in practice meant "after you visit the Inbox". A sidebar that says nothing until you
  * look at it is not a sidebar.
  */
+/** What Home's "Needs you" holds, for the sidebar. */
+function needsCount() {
+  if (!S.inbox || !S.data) return 0;
+  return sortHome(S.inbox.items ?? [], S.data.staff, humansOf(), S.live?.staff ?? []).needs.length;
+}
+
 export function countInBackground() {
   stampCounts();
   return ensureInbox(false)

package/templates/portal/js/state.js

@@ -22,7 +22,7 @@ export const VIEWS = [
 export const VIEW_ALIAS = { memory: "brain" };
 
 /** Screens that belong to the org rather than to one staff member. */
-export const ORG_WIDE = new Set(["setup", "inbox", "prs", "runs", "org", "staff", "docs"]);
+export const ORG_WIDE = new Set(["setup", "home", "inbox", "prs", "runs", "org", "staff", "docs"]);
 
 /** The two screens that read the inbox, and so share its filters and its `?t=` thread. */
 const INBOXY = new Set(["inbox", "prs"]);
@@ -44,7 +44,7 @@ export const S = {
   staffHandle: null,
   /* What is waiting on you, not whose brain you read last. The org-wide screens are where a
      session starts, which is also why the Staff list below them opens folded. */
-  view: "inbox",
+  view: "home",
   /** The inbox filter box. Brain and "what changed" keep their own, below. */
   query: "",
 
@@ -90,10 +90,6 @@ export const openCount = () =>
 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;
-
 export const staff = () => S.data.staff.find((s) => s.handle === S.staffHandle) ?? S.data.staff[0];
 
 /**

package/templates/portal/js/views/brain.js

@@ -150,7 +150,7 @@ export function viewBrain(m) {
     const fileHits = plain.filter((f) => !q || f.path.toLowerCase().includes(q));
 
     /* ---- memory ---- */
-    const mem = group("Memory", s.facts.length + " facts");
+    const mem = group("What they know", s.facts.length + " facts");
     if (q) {
       for (const f of hits.slice(0, 200)) mem.append(row("fact:" + f.slug, f.slug, "", "tmem"));
       if (!hits.length) {

package/templates/portal/js/views/hire.js

@@ -353,6 +353,16 @@ function hireStep(host, role, onHired) {
     app = field("app", org + "-" + role.handle, "This staff member's GitHub App. App names are unique across GitHub.");
     const pub = field("public app", org + "-robot", "The App all staff share for public product repos.");
     fields.push(["app", app], ["publicApp", pub]);
+    /* Only shown when a product repo is public, which is the only time the shared App is
+       created. Hidden, it still sends the same default, so nothing changes underneath. */
+    pub.row.hidden = true;
+    getRepos()
+      .then((r) => {
+        pub.row.hidden = !(r.repos ?? []).some(
+          (x) => x.role === "product" && String(x.visibility ?? "").toLowerCase() !== "private",
+        );
+      })
+      .catch(() => {});
   }
 
   const advanced = el("div", { className: "hireadvanced", hidden: true }, [