@nanocollective/roster 0.1.0-alpha.1 → 0.1.0-alpha.10

package/templates/portal/js/views/memory.js

@@ -13,6 +13,19 @@ import { inline } from "../md.js";
 import { S } from "../state.js";
 import { matchesFact } from "./brain.js";
 
+/**
+ * Which of the three kinds of provenance a tag is, as a class the stylesheet knows.
+ *
+ * A fact is measured, derived, or ruled on by a human — and the third one is written with that
+ * human's own marker, `[will]` or `[sam]` or whatever the org chose. The class used to be the
+ * marker itself against a stylesheet that knew one word, so every org but the one this was
+ * written in lost the "a person decided this" colour, which is the one that matters most.
+ */
+export function provenanceKind(provenance) {
+  const word = String(provenance).toLowerCase();
+  return word === "measured" || word === "derived" ? word : "ruled";
+}
+
 export function showMemory(viewer, s, key, pick) {
   const section = key.startsWith("mem:") ? key.slice(4) : null;
   const slug = key.startsWith("fact:") ? key.slice(5) : null;
@@ -61,7 +74,9 @@ export function showMemory(viewer, s, key, pick) {
     d.innerHTML =
       '<span class="slug" data-goto="' + esc(f.slug) + '" title="Show just this fact">' +
         esc(f.slug) + "</span>" +
-      (f.provenance ? '<span class="tag ' + esc(f.provenance) + '">' + esc(f.provenance) + "</span>" : "") +
+      (f.provenance
+        ? '<span class="tag ' + provenanceKind(f.provenance) + '">' + esc(f.provenance) + "</span>"
+        : "") +
       (f.note ? '<span class="notelink" data-note="' + esc(f.note) + '">' + esc(f.note) + "</span>" : "") +
       '<p class="stmt">' + inline(f.statement) + "</p>" +
       (f.consequence

package/templates/portal/js/views/org.js

@@ -1,27 +1,51 @@
 /* The org layer, in one place.
  *
- * These five files are what every staff member inherits. Editing `org/voice.md` once is the
- * whole extensibility story — the thing this arrangement exists to make possible — and until
- * now you reached them through a staff member's Prompt screen, which is a strange route to a
- * file that belongs to nobody in particular.
+ * These files are what every staff member inherits. Editing `org/voice.md` once is the whole
+ * extensibility story — the thing this arrangement exists to make possible — and until recently
+ * you reached them through a staff member's Prompt screen, which is a strange route to a file
+ * that belongs to nobody in particular.
+ *
+ * The list comes off disk rather than out of this file. It used to be five paths written here,
+ * which meant a tenant that added `org/pricing.md` could not open it at all, and one that had
+ * not written `org/business.md` yet got "not found" with nothing to do about it. Both are the
+ * same mistake: a list of what a tenant *should* have, standing in for what it has.
  *
  * `org.yaml` is here too, and is the one file that stops every prompt composing when it is
  * wrong. The server validates it before writing.
  */
 
-import { getFile, post } from "../api.js";
-import { el, esc, grow, kb } from "../dom.js";
+import { getFile, getOrgLayer, post } from "../api.js";
+import { askYes } from "../dialog.js";
+import { el, esc, grow, kb, skeleton } from "../dom.js";
+import { icon } from "../icons.js";
 import { mdlite } from "../md.js";
-import { S, writeHash } from "../state.js";
-
-/* What each file is for, in one line, because a list of five filenames tells you nothing
-   about which one to open. */
-const DOCS = [
-  ["org.yaml", "Who and what: the humans, the repos, the defaults, which agent runs them.", "yaml"],
-  ["org/business.md", "What the business actually is. Every prompt is composed on top of it, and it is the file that stops the agents writing generic slop.", "md"],
-  ["org/operating.md", "The autonomy contract: what they do without asking, what they escalate, how a run starts and ends.", "md"],
-  ["org/guardrails.md", "The non-negotiables. Every staff member is bound by these and cannot argue with them.", "md"],
-  ["org/voice.md", "How everything they write reads. Change it once, and every surface changes on the next run.", "md"],
+import { humansOf, S, writeHash } from "../state.js";
+import { yamlPre } from "../yaml.js";
+import { repoPicker } from "./repos.js";
+
+/* What the files roster itself ships are for, in one line each. A filename says nothing about
+   which one to open; anything a tenant has added of its own falls back to its first heading. */
+const KNOWN = {
+  "org.yaml": "Who and what: the humans, the repos, the defaults, which agent runs them.",
+  "org/business.md":
+    "What the business actually is. Every prompt is composed on top of it, and it is the file that stops the agents writing generic slop.",
+  "org/priorities.md":
+    "What matters this month, ranked, and what is out of scope. Every daily run reads it, and every PR says which priority it serves.",
+  "org/operating.md":
+    "The autonomy contract: what they do without asking, what they escalate, how a run starts and ends.",
+  "org/guardrails.md":
+    "The non-negotiables. Every staff member is bound by these and cannot argue with them.",
+  "org/voice.md":
+    "How everything they write reads. Change it once, and every surface changes on the next run.",
+  "prompts/daily.md": "The whole of a scheduled run, before the fragments are pulled into it.",
+  "prompts/mention.md": "What they are sent when you @-mention them.",
+  "prompts/_identity.md": "Who they post as, and what wakes you.",
+  "prompts/_paths.md": "Where everything is in the runner's checkout.",
+};
+
+const GROUPS = [
+  ["org", "The org layer", "Inherited by everybody, on their next run."],
+  ["prompts", "The prompts", "The runs themselves. Edit these last: they are the framework's."],
 ];
 
 export function viewOrg(m) {
@@ -37,44 +61,91 @@ export function viewOrg(m) {
   );
 
   const facts = el("div", { className: "grid", style: "margin-bottom:18px" });
-  const human = org.human ?? {};
+  const humans = humansOf();
   facts.append(
     kv("Organisation", esc(org.org), org.name),
-    kv("They answer to", esc(human.name ?? human.github ?? "nobody"), human.github ? "@" + human.github : "no github handle in org.yaml"),
+    /* Plural on purpose. An org can answer to more than one person, and a card that names one
+       of two is worse than one that names neither: it reads as the only one who counts. */
+    kv(
+      "They answer to",
+      humans.length
+        ? humans.map((h) => esc(h.name ?? h.github)).join(", ")
+        : "nobody",
+      humans.length
+        ? humans.map((h) => (h.github ? "@" + h.github : "no github handle")).join(" · ")
+        : "org.yaml names no human, so nothing can wake an agent",
+    ),
     kv("Staff", String(org.staff.length), org.staff.map((s) => s.handle).join(", ")),
   );
   m.append(facts);
 
+  /* Marking a product repo after setup. The picker only lived on the setup screen, so a repo
+     created later meant typing `role: product` into org.yaml by hand. Folded, because it asks
+     GitHub for every repo in the org and most visits here are to read a file. */
+  const repoPane = el("div");
+  const repos = el("button", { className: "ghbtn", textContent: "Add a product repo" });
+  repos.onclick = () => {
+    if (repoPane.children.length) {
+      repoPane.replaceChildren();
+      return;
+    }
+    repoPane.replaceChildren(repoPicker({ org: org.org, onAdded: () => open("org.yaml") }));
+  };
+  m.append(el("div", { className: "row", style: "margin-bottom:14px" }, [repos]), repoPane);
+
   const split = el("div", { className: "split" });
   const tree = el("div", { className: "tree" });
   const viewer = el("div", { className: "viewer" });
   split.append(tree, viewer);
   m.append(split);
 
-  const group = el("div", { className: "navgroup" });
-  group.append(el("div", { className: "ghead", textContent: "The org layer" }));
-  for (const [path, why] of DOCS) {
-    const b = el("button", { className: "tfile tlayer", title: why });
-    b.dataset.key = path;
-    const t = el("span", { className: "t" });
-    t.append(
-      el("span", { className: "lname", textContent: path.replace(/^org\//, "") }),
-      el("span", { className: "lrepo", textContent: why }),
-    );
-    b.append(t);
-    b.onclick = () => open(path);
-    group.append(b);
-  }
-  tree.append(group);
-
-  /* Declared before the first `open()`, not after: `const` is not hoisted, and putting it
-     below the call made the first paint fail with "cannot access 'full' before
-     initialization" rather than rendering anything. */
   const full = (path) => S.data.opsName + "/" + path;
 
-  open(S.orgOpen ?? DOCS[0][0]);
+  tree.replaceChildren(...skeleton("row", 6));
+  getOrgLayer()
+    .then(({ files }) => {
+      S.orgFiles = files ?? [];
+      paintTree();
+      const first = S.orgFiles[0]?.path;
+      open(S.orgFiles.some((f) => f.path === S.orgOpen) ? S.orgOpen : first);
+    })
+    .catch((e) => {
+      tree.replaceChildren(el("p", { className: "empty err", textContent: e.message }));
+    });
+
+  function why(file) {
+    return KNOWN[file.path] ?? file.title ?? "";
+  }
+
+  function paintTree() {
+    tree.replaceChildren();
+    if (!S.orgFiles.length) {
+      tree.append(el("p", { className: "empty", textContent: "No org files here." }));
+      return;
+    }
+    for (const [id, heading, note] of GROUPS) {
+      const mine = S.orgFiles.filter((f) => (f.group ?? "org") === id);
+      if (!mine.length) continue;
+      const group = el("div", { className: "navgroup" });
+      group.append(el("div", { className: "ghead", textContent: heading, title: note }));
+      for (const f of mine) {
+        const b = el("button", { className: "tfile tlayer", title: why(f) });
+        b.dataset.key = f.path;
+        const t = el("span", { className: "t" });
+        t.append(
+          el("span", { className: "lname", textContent: f.path.replace(/^(org|prompts)\//, "") }),
+          el("span", { className: "lrepo", textContent: why(f) }),
+        );
+        b.append(t);
+        b.onclick = () => open(f.path);
+        group.append(b);
+      }
+      tree.append(group);
+    }
+  }
 
   function open(path) {
+    if (!path) return;
     S.orgOpen = path;
     writeHash(false);
     for (const o of tree.querySelectorAll(".tfile")) {
@@ -84,7 +155,7 @@ export function viewOrg(m) {
   }
 
   async function show(path) {
-    viewer.replaceChildren(el("p", { className: "empty", textContent: "Loading…" }));
+    viewer.replaceChildren(...skeleton("head", 1), ...skeleton("line", 7));
     let text;
     try {
       text = await getFile(full(path));
@@ -97,55 +168,84 @@ export function viewOrg(m) {
 
   function render(path, text) {
     viewer.replaceChildren();
-    const why = DOCS.find(([p]) => p === path)?.[1] ?? "";
-    const head = el("div", { className: "row", style: "margin-bottom:4px" });
+    const head = el("div", { className: "row filehead" });
     head.append(el("span", { className: "meta", textContent: full(path) + " · " + kb(text.length) }));
-    head.append(el("button", { className: "ghbtn", textContent: "Edit", onclick: () => edit(path, text) }));
-    viewer.append(head, el("p", { className: "factsub", textContent: why }));
+    /* The edit control used to be a word in a row of grey text, which is why "there is no easy
+       way to edit the org docs" was a fair thing to say about a screen that could edit them. */
+    const edit = el("button", { className: "ghbtn primary editbtn", onclick: () => editor(path, text) });
+    edit.append(icon("edit", "ic"), el("span", { textContent: "Edit" }));
+    head.append(edit);
+    const file = S.orgFiles.find((f) => f.path === path) ?? { path };
+    viewer.append(head, el("p", { className: "factsub", textContent: why(file) }));
 
-    if (path.endsWith(".yaml")) {
-      viewer.append(el("pre", { className: "code", textContent: text }));
+    if (path.endsWith(".yaml") || path.endsWith(".yml")) {
+      viewer.append(yamlPre(text));
       return;
     }
     viewer.append(el("div", { className: "md doc", innerHTML: mdlite(text) }));
   }
 
-  function edit(path, text) {
+  function editor(path, text) {
     viewer.replaceChildren();
-    viewer.append(
-      el("div", { className: "row", style: "margin-bottom:10px" }, [
-        el("span", { className: "meta", textContent: full(path) }),
-      ]),
-    );
+    const isYaml = /\.ya?ml$/.test(path);
+
+    const status = el("span", { className: "meta" });
+    const save = el("button", { className: "ghbtn primary", textContent: "Save and commit" });
+    const cancel = el("button", { className: "ghbtn", textContent: "Cancel" });
+
+    const bar = el("div", { className: "row filehead" });
+    bar.append(el("span", { className: "meta", textContent: full(path) }), save, cancel, status);
+    viewer.append(bar);
 
     const ta = el("textarea", { value: text, className: "editor" });
     viewer.append(ta);
     grow(ta);
     ta.addEventListener("input", () => grow(ta));
+    // ⌘S is what a person's hands do in a text box. Without it, saving means finding a button
+    // above a screenful of textarea you have just scrolled past.
+    ta.addEventListener("keydown", (e) => {
+      if ((e.metaKey || e.ctrlKey) && e.key === "s") {
+        e.preventDefault();
+        commit();
+      }
+    });
+    ta.focus?.();
 
-    const status = el("span", { className: "meta" });
-    const save = el("button", { className: "ghbtn primary", textContent: "Save and commit" });
-    const cancel = el("button", { className: "ghbtn", textContent: "Cancel" });
     cancel.onclick = () => render(path, text);
-    viewer.append(el("div", { className: "row", style: "margin-top:10px" }, [save, cancel, status]));
+    save.onclick = commit;
+
     viewer.append(
       el("p", {
         className: "editnote",
         textContent:
-          "Commits to " + S.data.opsName + " and pushes. Picked up by every staff member on their " +
-          "next run." +
-          (path.endsWith(".yaml")
+          "⌘S saves. Commits to " + S.data.opsName + " and pushes, and every staff member picks " +
+          "it up on their next run." +
+          (isYaml
             ? " org.yaml is checked before it is written: bad YAML, or a missing org or name, is refused rather than committed."
             : ""),
       }),
     );
 
-    save.onclick = async () => {
+    async function commit() {
       if (ta.value === text) {
         status.textContent = "nothing changed";
+        status.className = "meta";
+        return;
+      }
+      /* Only the manifest asks. A prose edit is one commit to revert and the agents pick it up
+         on their next run; org.yaml is the file that can stop every prompt composing, and the
+         portal has just told you it validates it. Asking every time taught people to click
+         through the question, which is worse than not asking. */
+      if (
+        isYaml &&
+        !(await askYes({
+          title: "Commit " + full(path) + " and push?",
+          hint: "Every staff member picks this up on their next run.",
+          confirm: "Commit and push",
+        }))
+      ) {
         return;
       }
-      if (!confirm("Commit " + full(path) + " and push?\n\nEvery staff member picks this up on their next run.")) return;
       save.disabled = cancel.disabled = true;
       status.textContent = "committing…";
       status.className = "meta";
@@ -162,7 +262,7 @@ export function viewOrg(m) {
         status.className = "meta err";
         save.disabled = cancel.disabled = false;
       }
-    };
+    }
   }
 }
 

package/templates/portal/js/views/paste.js

@@ -20,6 +20,12 @@ export function paste(opts) {
   const status = el("p", { className: "sub", textContent: "Building the prompt…" });
   box.append(status);
 
+  /* A charter brief carries a worked example to model the shape on, matched to the role. The
+     choice is on the page because the match is a guess, and "none" is a fair answer. */
+  const model = el("select", { hidden: true });
+  const modelRow = el("label", { className: "sub", hidden: true }, ["Model it on: ", model]);
+  box.append(modelRow);
+
   const actions = el("div", { className: "row" });
   const copy = el("button", { className: "btn primary", textContent: "Copy the prompt" });
   const show = el("button", { className: "btn", textContent: "Show it" });
@@ -42,10 +48,33 @@ export function paste(opts) {
 
   let brief = null;
 
+  const NAMES = { cto: "the CTO example", cmo: "the CMO example", support: "the support example", none: "no example" };
+  model.onchange = () => {
+    copy.disabled = true;
+    getBrief(opts.kind, opts.staff, model.value)
+      .then((data) => {
+        if (data.error) throw new Error(data.error);
+        brief = data;
+        if (!preview.hidden) preview.textContent = brief.text;
+      })
+      .catch((err) => {
+        status.className = "err";
+        status.textContent = String(err.message || err);
+      })
+      .finally(() => {
+        copy.disabled = false;
+      });
+  };
+
   getBrief(opts.kind, opts.staff)
     .then((data) => {
       if (data.error) throw new Error(data.error);
       brief = data;
+      if (data.examples) {
+        model.replaceChildren(...data.examples.map((e) => el("option", { value: e, textContent: NAMES[e] ?? e })));
+        model.value = data.example ?? "none";
+        model.hidden = modelRow.hidden = false;
+      }
       status.textContent =
         "Paste this into Claude, ChatGPT or anything else. It carries every file it refers to, " +
         "so there is nothing to attach and nothing for it to ask you for. It will interview you " +

package/templates/portal/js/views/prompt.js

@@ -10,10 +10,11 @@
  */
 
 import { getFile, post } from "../api.js";
-import { askText } from "../dialog.js";
+import { askText, askYes } from "../dialog.js";
 import { el, esc, grow, kb, toClipboard } from "../dom.js";
 import { icon } from "../icons.js";
 import { mdlite } from "../md.js";
+import { render } from "../router.js";
 import { S, staff, writeHash } from "../state.js";
 import { diffStat, unifiedDiff } from "../textdiff.js";
 import { renderDiff } from "./changed.js";
@@ -21,9 +22,53 @@ import { renderDiff } from "./changed.js";
 const KINDS = [
   ["daily", "Daily", "the scheduled run"],
   ["mention", "Mention", "someone typed @them in an issue"],
-  ["pr-mention", "PR mention", "someone typed @them on a pull request"],
 ];
 
+/**
+ * Build the paste-ready brief on the server, where compose.mjs lives, and copy it.
+ *
+ * A finding pre-fills the box rather than skipping it: what it wrote is a starting point, and
+ * "and keep it in operating.md" is exactly the sort of thing you want to add.
+ *
+ * Exported because the findings themselves live on Health now, and the fix for one of them is
+ * the same brief whichever screen you asked from.
+ */
+export async function copyAmendBrief(handle, kind, want, btn, note) {
+  const label = btn.textContent;
+  const asked = await askText({
+    title: "What do you want changed?",
+    hint:
+      "This goes at the top of a brief carrying the whole prompt and every file it is " +
+      "made of. Say it the way you would say it to a person.",
+    value: want,
+    placeholder: "stop opening decision issues for anything reversible",
+    confirm: "Copy the brief",
+  });
+  if (!asked) return;
+  btn.disabled = true;
+  btn.textContent = "building…";
+  try {
+    const url =
+      "/api/amend?staff=" + encodeURIComponent(handle) +
+      "&kind=" + encodeURIComponent(kind) +
+      "&want=" + encodeURIComponent(asked);
+    const text = await (await fetch(url, { cache: "no-store" })).text();
+    btn.disabled = false;
+    toClipboard(text, btn, label);
+    if (note) {
+      note.textContent = "paste it into whatever agent you use";
+      note.className = "meta";
+    }
+  } catch (e) {
+    btn.disabled = false;
+    btn.textContent = label;
+    if (note) {
+      note.textContent = e.message;
+      note.className = "meta err";
+    }
+  }
+}
+
 export function viewPrompt(m) {
   const s = staff();
   if (!S.promptKind) S.promptKind = "daily";
@@ -54,40 +99,7 @@ export function viewPrompt(m) {
   const note = el("span", { className: "meta" });
   m.append(el("div", { className: "row", style: "margin-bottom:16px" }, [kind, help, copy, note]));
 
-  /* Build the paste-ready brief on the server, where compose.mjs lives, and copy it.
-     A finding pre-fills the box rather than skipping it: what it wrote is a starting point,
-     and "and keep it in operating.md" is exactly the sort of thing you want to add. */
-  async function copyAmend(want, btn) {
-    const target = btn ?? help;
-    const label = target.textContent;
-    const asked = await askText({
-      title: "What do you want changed?",
-      hint:
-        "This goes at the top of a brief carrying the whole prompt and every file it is " +
-        "made of. Say it the way you would say it to a person.",
-      value: want,
-      placeholder: "stop opening decision issues for anything reversible",
-      confirm: "Copy the brief",
-    });
-    if (!asked) return;
-    target.disabled = true;
-    target.textContent = "building…";
-    try {
-      const url =
-        "/api/amend?staff=" + encodeURIComponent(s.handle) +
-        "&kind=" + encodeURIComponent(S.promptKind) +
-        "&want=" + encodeURIComponent(asked);
-      const text = await (await fetch(url, { cache: "no-store" })).text();
-      target.disabled = false;
-      toClipboard(text, target, label);
-      note.textContent = "paste it into whatever agent you use";
-    } catch (e) {
-      target.disabled = false;
-      target.textContent = label;
-      note.textContent = e.message;
-      note.className = "meta err";
-    }
-  }
+  const copyAmend = (want, btn) => copyAmendBrief(s.handle, S.promptKind, want, btn ?? help, note);
 
   const split = el("div", { className: "split" });
   const tree = el("div", { className: "tree" });
@@ -140,12 +152,9 @@ export function viewPrompt(m) {
     for (const l of view.layers) inlined.append(layerRow(l));
     tree.append(inlined);
 
-    if (view.problems?.length) {
-      const bad = group("Problems", String(view.problems.length));
-      for (const p of view.problems) bad.append(problemRow(p));
-      tree.append(bad);
-    }
-
+    /* No Problems section. This screen answers "what is sent"; whether what is sent is any
+       good is a health question, and it is asked and answered on Health. Having both here
+       made the tree beside the prompt half prompt and half complaints. */
     const named = group("Named, not inlined");
     named.append(
       el("p", {
@@ -160,28 +169,6 @@ export function viewPrompt(m) {
     open(S.promptOpen ?? "composed");
   }
 
-  /* A finding is only half useful. The other half is the sentence that goes into the brief,
-     which is why every one of them carries a `want`. */
-  function problemRow(p) {
-    const b = el("div", { className: "prob " + p.level });
-    b.append(el("div", { className: "ptitle", textContent: p.title }));
-    b.append(el("div", { className: "pdetail", textContent: p.detail }));
-    const fix = el("button", { className: "ghbtn", textContent: "Copy a prompt to fix this" });
-    fix.onclick = () => copyAmend(p.want, fix);
-    const row = el("div", { className: "row", style: "margin-top:8px" }, [fix]);
-    if (p.path) {
-      row.append(
-        el("button", {
-          className: "ghbtn",
-          textContent: "Open the file",
-          onclick: () => open(p.path),
-        }),
-      );
-    }
-    b.append(row);
-    return b;
-  }
-
   function group(title, right) {
     const g = el("div", { className: "navgroup" });
     const h = el("div", { className: "ghead", textContent: title });
@@ -388,8 +375,12 @@ export function viewPrompt(m) {
         status.textContent = "nothing changed";
         return;
       }
-      if (!confirm("Commit " + layer.path + " to " + layer.repo + " and push?\n\n" +
-                   "Picked up by " + reach + ".")) return;
+      const yes = await askYes({
+        title: "Commit " + layer.path + " to " + layer.repo + " and push?",
+        hint: "Picked up by " + reach + ".",
+        confirm: "Commit and push",
+      });
+      if (!yes) return;
       save.disabled = cancel.disabled = true;
       status.textContent = "committing…";
       status.className = "meta";

package/templates/portal/js/views/repos.js

@@ -5,11 +5,12 @@
  * spelling a repo name exactly right in a YAML file, which is the kind of thing that fails
  * quietly and reads as the tool being broken. */
 
-import { addRepo, listRepos } from "../api.js";
+import { addRepo, getRepos, listRepos } from "../api.js";
 import { el } from "../dom.js";
 
 /**
- * @param {{org: string, known: string[], onAdded?: () => void}} opts
+ * @param {{org: string, known?: string[], onAdded?: () => void}} opts  without `known`, what
+ *   org.yaml already lists is read off disk, which is how the Org screen uses it long after setup.
  */
 export function repoPicker(opts) {
   const box = el("div", { className: "manual" });
@@ -18,7 +19,8 @@ export function repoPicker(opts) {
     el("p", {
       textContent:
         "Repos marked as products are the ones staff contribute to without owning. A new hire " +
-        "picks them up automatically.",
+        "picks them up automatically; somebody already hired keeps the works_in list in their " +
+        "own staff.yaml.",
     }),
   );
 
@@ -26,10 +28,13 @@ export function repoPicker(opts) {
   box.append(out);
   out.replaceChildren(el("p", { className: "sub", textContent: "Reading your repos…" }));
 
-  listRepos(opts.org)
-    .then((data) => {
+  const known = opts.known
+    ? Promise.resolve(opts.known)
+    : getRepos().then((d) => (d.repos ?? []).map((r) => r.name));
+  Promise.all([listRepos(opts.org), known])
+    .then(([data, listed]) => {
       if (data.error) throw new Error(data.error);
-      const already = new Set(opts.known ?? []);
+      const already = new Set(listed);
       const candidates = (data.repos ?? []).filter((r) => !already.has(r.name));
 
       if (!candidates.length) {
@@ -52,7 +57,7 @@ export function repoPicker(opts) {
           add.disabled = true;
           add.textContent = "Adding…";
           try {
-            await addRepo(repo.name, "product");
+            await addRepo(repo.name, "product", repo.visibility);
             add.textContent = "Added";
             row.classList.add("added");
             opts.onAdded?.();