@nanocollective/roster 0.1.0-alpha.21 → 0.1.0-alpha.22

package/dist/cli.js

@@ -6439,7 +6439,7 @@ async function portalCommand(argv) {
   const handleSetup = async (url, req, res) => {
     const route = url.pathname.slice("/api/setup/".length);
     if (route === "status") {
-      json(res, { ...await setupStatus(startedIn), startedIn });
+      json(res, { ...await setupStatus(startedIn), startedIn }, true);
       return;
     }
     if (route === "app") {
@@ -6820,7 +6820,7 @@ async function portalCommand(argv) {
         return;
       }
       if (url.pathname === "/api/doctor") {
-        collect({ ops: ws?.opsDir, offline: url.searchParams.get("offline") === "1" }).then((report3) => json(res, report3 ?? { findings: [], online: false, empty: true })).catch((err) => {
+        collect({ ops: ws?.opsDir, offline: url.searchParams.get("offline") === "1" }).then((report3) => json(res, report3 ?? { findings: [], online: false, empty: true }, true)).catch((err) => {
           res.writeHead(200, { "content-type": "application/json" });
           res.end(JSON.stringify({ findings: [], error: String(err?.message ?? err) }));
         });
@@ -6901,7 +6901,7 @@ async function portalCommand(argv) {
               throw new Error(`"${handle}" is not in org.yaml`);
             }
             const plan3 = buildRetirePlan(w, org, handle, parseYaml);
-            json(res, { action, plan: plan3 });
+            json(res, { action, plan: plan3 }, true);
             return;
           }
           if (!/^[a-z][a-z0-9-]{1,20}$/.test(handle)) {
@@ -6913,10 +6913,11 @@ async function portalCommand(argv) {
           const flags = hireFlags(url.searchParams);
           const plan2 = buildPlan(w, org, handle, flags, parseYaml);
           void readOrgSecret(org.org, plan2.staff.agentSecret).then(
-            (orgSecret) => json(res, {
-              action,
-              plan: { ...plan2, orgSecret, files: [...plan2.files.keys()].sort() }
-            })
+            (orgSecret) => json(
+              res,
+              { action, plan: { ...plan2, orgSecret, files: [...plan2.files.keys()].sort() } },
+              true
+            )
           );
         } catch (err) {
           res.writeHead(400, { "content-type": "application/json" });
@@ -6990,7 +6991,7 @@ async function portalCommand(argv) {
         return;
       }
       if (url.pathname === "/api/fix") {
-        gather(w, url.searchParams.get("offline") === "1").then((items) => json(res, { items, text: fixBrief(w, items) })).catch((err) => {
+        gather(w, url.searchParams.get("offline") === "1").then((items) => json(res, { items, text: fixBrief(w, items) }, true)).catch((err) => {
           res.writeHead(200, { "content-type": "application/json" });
           res.end(JSON.stringify({ items: [], error: String(err?.message ?? err) }));
         });
@@ -7515,12 +7516,13 @@ function refuseWrite(res) {
   res.writeHead(403, { "content-type": "application/json" });
   res.end(JSON.stringify({ error: "write actions need a local POST from the portal" }));
 }
-function json(res, data) {
+function json(res, data, advice = false) {
   res.writeHead(200, {
     "content-type": "application/json; charset=utf-8",
     "cache-control": "no-store"
   });
-  res.end(withBin(JSON.stringify(data)));
+  const text = JSON.stringify(data);
+  res.end(advice ? withBin(text) : text);
 }
 function hireFlags(params) {
   const num = (k) => params.get(k) ? Number(params.get(k)) : void 0;

package/package.json

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

package/templates/portal/css/setup.css

@@ -148,3 +148,13 @@
 .mode:hover{color:var(--ink)}
 .mode.on{background:var(--bg-elev);color:var(--ink);box-shadow:var(--shadow-sm)}
 .modes + div > .paste{border-top:0;margin-top:0;padding-top:0}
+
+/* ---------- a checked paste ---------- */
+.verdict{margin:0 0 10px;font:600 13.5px var(--sans)}
+.verdict.ok{color:var(--accent)}
+.verdict.bad{color:var(--err)}
+.pastepreview{border:1px solid var(--line);border-radius:var(--r);padding:14px 16px;margin-bottom:12px;background:var(--bg)}
+.pastenotes{margin:0 0 12px}
+.pastenotes summary{cursor:pointer;font:600 12.5px var(--sans);color:var(--ink-dim);margin-bottom:6px}
+.pastenotes .md{border-left:3px solid var(--line);padding-left:12px}
+.pastefile .diff{white-space:pre-wrap;overflow-wrap:anywhere}

package/templates/portal/js/views/orgedit.js

@@ -17,6 +17,10 @@ const BUSINESS_QUESTIONS = [
 
 const val = (input) => String(input.value ?? "").trim();
 
+/* A file still in the shape it shipped in holds the questions, not answers, and pre-filling the
+   fields with "Not a segment. The person…" reads as if somebody had already answered them. */
+const isStub = (text) => /This file is a stub|Or answer these by hand/.test(text);
+
 /**
  * org/business.md as five questions. With `text`, the answers already in the file fill the
  * fields, so changing one answer does not mean retyping the other four.
@@ -25,7 +29,7 @@ const val = (input) => String(input.value ?? "").trim();
  */
 export function businessForm(o) {
   const form = el("div");
-  const known = sections(o.text ?? "");
+  const known = isStub(o.text ?? "") ? new Map() : sections(o.text ?? "");
   const fields = BUSINESS_QUESTIONS.map(([id, label, heading, required]) => {
     const input = id === "line" ? el("input", { type: "text" }) : el("textarea", { className: "pastebox", rows: 3 });
     const had = known.get(heading.toLowerCase());
@@ -64,7 +68,7 @@ export function businessForm(o) {
 /** org/priorities.md as three lines and an out-of-scope list, filled from the file if it has them. */
 export function prioritiesForm(o) {
   const form = el("div");
-  const { items, outs } = parsePriorities(o.text ?? "");
+  const { items, outs } = isStub(o.text ?? "") ? { items: [], outs: [] } : parsePriorities(o.text ?? "");
   const ranks = [1, 2, 3].map((i) => {
     const input = el("input", { type: "text", placeholder: i === 1 ? "e.g. Tasks can be ticked off and removed" : "" });
     if (items[i - 1]) input.value = items[i - 1];

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

@@ -7,6 +7,7 @@
  * Nothing here writes. The panel parses, shows a diff, and waits for a second click. */
 
 import { getBrief, parsePaste, saveFile } from "../api.js";
+import { mdlite } from "../md.js";
 import { el, toClipboard } from "../dom.js";
 import { unifiedDiff, diffStat } from "../textdiff.js";
 
@@ -123,23 +124,47 @@ export function paste(opts) {
       result.append(note);
     }
 
+    // What the AI said around the file: its caveats and "check these" are worth reading first.
+    const notes = String(answer.value ?? "")
+      .replace(/<<<ROSTER FILE [^>]+>>>[\s\S]*?<<<ROSTER END>>>/g, "")
+      .trim();
+
     for (const file of data.files ?? []) {
-      if (file.unchanged) continue;
       const card = el("div", { className: "pastefile" });
-      const stat = diffStat(file.before, file.text);
+      const name = file.path.split("/").slice(-2).join("/");
+      if (file.unchanged) {
+        card.append(el("p", { className: "verdict", textContent: "✓ Same as what's already in " + name + ". Nothing to save." }));
+        result.append(card);
+        continue;
+      }
       card.append(
-        el("b", { textContent: file.path }),
-        el("span", {
-          className: "meta",
-          textContent: `+${stat.added} −${stat.removed}` + (file.writable ? "" : " · not writable"),
+        el("p", {
+          className: "verdict" + (file.writable ? " ok" : " bad"),
+          textContent: file.writable
+            ? "✓ Looks good. Saving replaces " + name + " with this:"
+            : "✗ " + file.path + " can't be written from here.",
         }),
       );
-      const diff = el("pre", { className: "diff" });
+      card.append(el("div", { className: "md doc pastepreview", innerHTML: mdlite(file.text) }));
+
+      if (notes) {
+        const said = el("details", { className: "pastenotes", open: true });
+        said.append(el("summary", { textContent: "Your AI also said" }), el("div", { className: "md doc", innerHTML: mdlite(notes) }));
+        card.append(said);
+      }
+
+      const diff = el("pre", { className: "diff", hidden: true });
       diff.textContent = unifiedDiff(file.before, file.text, file.path);
-      card.append(diff);
+      const stat = diffStat(file.before, file.text);
+      const toggle = el("button", { className: "ghbtn", textContent: `Show changes (+${stat.added} −${stat.removed})` });
+      toggle.onclick = () => {
+        diff.hidden = !diff.hidden;
+        toggle.textContent = (diff.hidden ? "Show changes" : "Hide changes") + ` (+${stat.added} −${stat.removed})`;
+      };
 
+      const row = el("div", { className: "row" });
       if (file.writable) {
-        const save = el("button", { className: "btn primary", textContent: "Save and commit" });
+        const save = el("button", { className: "btn primary", textContent: "Save" });
         save.onclick = async () => {
           save.disabled = true;
           save.textContent = "Saving…";
@@ -150,17 +175,19 @@ export function paste(opts) {
             opts.onSaved?.();
           } catch (err) {
             save.disabled = false;
-            save.textContent = "Save and commit";
+            save.textContent = "Save";
             card.append(el("p", { className: "err", textContent: String(err.message || err) }));
           }
         };
-        card.append(save);
+        row.append(save);
       }
+      row.append(toggle);
+      card.append(row, diff);
       result.append(card);
     }
 
     if (!(data.problems ?? []).length && !(data.files ?? []).length) {
-      result.append(el("p", { className: "sub", textContent: "Nothing to do." }));
+      result.append(el("p", { className: "verdict bad", textContent: "✗ No file found in that reply." }));
     }
   };