@nanocollective/roster 0.1.0-alpha.12 → 0.1.0-alpha.13

package/package.json

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

package/templates/portal/js/views/checklist.js

@@ -10,6 +10,8 @@ import { el, toClipboard } from "../dom.js";
 const LEVEL_ORDER = { fail: 0, warn: 1, ok: 2 };
 
 /**
+ * @param opts.skip  finding ids the caller already shows as a step of its own, so they are not
+ *   reported twice. With `opts.hideWhenClean`, a list left empty draws nothing at all.
  * @param opts.shell  when Health draws this, the section it lives in: `{actions, say}`. The
  *   setup screen passes nothing and gets the layout it has always had. Its own screen is a
  *   wizard, where a chunky button and a paragraph under it are right; on Health it is one
@@ -32,9 +34,14 @@ export async function checklist(host, opts = {}) {
   const findings = (report.findings ?? []).slice().sort(
     (a, b) => (LEVEL_ORDER[a.level] ?? 3) - (LEVEL_ORDER[b.level] ?? 3),
   );
-  const bad = findings.filter((f) => f.level !== "ok");
+  const skip = new Set(opts.skip ?? []);
+  const bad = findings.filter((f) => f.level !== "ok" && !skip.has(f.id));
 
   host.replaceChildren();
+  if (!bad.length && opts.hideWhenClean) {
+    opts.onEmpty?.();
+    return;
+  }
   opts.shell?.say(bad.length ? String(bad.length) : "clean");
   host.append(
     el("p", {
@@ -83,7 +90,7 @@ export async function checklist(host, opts = {}) {
   const again = el("button", { className: btnClass, textContent: "Check again" });
   again.onclick = () => checklist(host, opts);
 
-  const feet = opts.shell?.actions ?? host;
+  const feet = opts.shell?.actions ?? host.appendChild(el("div", { className: "row" }));
   if (opts.shell) feet.replaceChildren();
   feet.append(copy, again, note);
   if (bad.length) {
@@ -92,8 +99,7 @@ export async function checklist(host, opts = {}) {
         className: noteClass,
         style: "flex-basis:100%",
         textContent:
-          "Paste it into Cursor, Claude Code or anything else that can edit files here, then " +
-          "check again. The ids above should be gone.",
+          "Paste them into your coding agent, then check again.",
       }),
     );
   }

package/templates/portal/js/views/setup.js

@@ -30,6 +30,12 @@ let S = null; // /api/setup/status, refreshed after anything that changes the wo
 /* Where the doctor checklist is drawn on this screen, so a save can re-run it. Left stale, it
    went on saying business.stub after business.md had been saved, until "Check again". */
 let healthHost = null;
+let healthStep = null;
+
+/* What the "What is left" cards already ask for. Doctor reporting them as well, right after a
+   successful create, read as the create having failed. */
+const COVERED = ["business", "business.stub", "priorities", "priorities.stub", "actions-access", "secrets", "agent"];
+const HEALTH = { skip: COVERED, hideWhenClean: true };
 
 /* Each "what is left" step that is a file, by id, so a save can mark it done in place rather
    than repainting the page and throwing away the diff somebody is still reading. */
@@ -75,11 +81,12 @@ export async function viewGettingStarted(main) {
 
   main.append(afterCreate(App.data?.staff?.length ? 1 : 2));
 
-  const health = step(App.data?.staff?.length ? 2 : 3, "What doctor still says", false);
+  const health = step(App.data?.staff?.length ? 2 : 3, "Other problems", false);
   healthHost = el("div", { style: "margin-top:12px" });
   health.append(healthHost);
   main.append(health);
-  checklist(healthHost, {});
+  healthStep = health;
+  checklist(healthHost, { ...HEALTH, onEmpty: () => health.remove() });
 }
 
 /** Whether the sidebar offers Getting started: while anything is left, or while it is open. */
@@ -93,7 +100,7 @@ export function paintSetupNav() {
    again rather than updated by hand. The sidebar's list comes from /api/org, which is the one
    the portal already reads. */
 async function recheck() {
-  if (healthHost) checklist(healthHost, {});
+  if (healthHost) checklist(healthHost, { ...HEALTH, onEmpty: () => healthStep?.remove() });
   try {
     S = await getSetup();
     for (const [id, mark] of fileSteps) mark(!(S.tenant.left ?? []).includes(id));
@@ -166,13 +173,9 @@ function stepOrg(main) {
       el("p", {
         className: "sub",
         textContent:
-          "Reload the page to open the portal. Anything below that is still undone stays under " +
-          "Getting started in the sidebar until it is.",
+          "Finish the steps below. They stay under Getting started in the sidebar until they're done.",
       }),
     );
-    healthHost = el("div", { style: "margin-top:12px" });
-    card.append(healthHost);
-    checklist(healthHost, {});
     return card;
   }