@alphacoder-v0/pi-loops 0.18.0 → 0.19.3

package/src/web.mjs

@@ -452,8 +452,7 @@ function nextRuns() {
 	// This machine's file. Leadership is per host, and so is the clock a cron expression is matched
 	// against, so on a shared `$HOME` each machine keeps its own answers rather than overwriting the
 	// other's with times computed in a different timezone.
-	const safe = HOST.replace(/[^A-Za-z0-9._-]/g, "_");
-	const doc = readJson(path.join(LOOPS_DIR, `next-runs.${safe}.json`), { next: {} });
+	const doc = readJson(path.join(LOOPS_DIR, "next-runs.json"), { next: {} });
 	return doc?.next && typeof doc.next === "object" ? doc.next : {};
 }
 
@@ -467,16 +466,13 @@ function automation(cwd) {
 	// The directory is the evidence, not any one file in it: a project with rules and no cron jobs
 	// has no jobs.json at all, and reporting "pi-loops not found" there would be a lie.
 	if (!fs.existsSync(LOOPS_DIR)) return { installed: false, dir: LOOPS_DIR };
+	// A plain job runs only in the session that created it; the id in this session's header says
+	// whether that is this one, and the panel says "resume it" rather than promising a time.
+	const currentSession = sessionFile ? readHead(sessionFile)?.id : undefined;
 	const jobsFile = readJson(path.join(LOOPS_DIR, "jobs.json"), { jobs: [] });
 	/**
-	 * This project's, whatever machine they were made on.
-	 *
-	 * The host used to be part of this test, so a job stamped with another hostname vanished from
-	 * the panel — and a hostname changes on its own: a rebuilt container, a machine renamed by
-	 * DHCP, a restored backup. What you got was a job sitting enabled in `jobs.json`, never running,
-	 * invisible in the one place you would look. The terminal always listed it and said whose it
-	 * was; so does this, and the count of everything in other projects goes with it, because a list
-	 * that silently drops things is worse than a longer list.
+	 * This project's jobs, and the count of everything in other projects goes with it, because a
+	 * list that silently drops things is worse than a longer list.
 	 */
 	const inThisProject = (j) => sameProject(j.cwd, cwd);
 	const allJobs = jobsFile.jobs ?? [];
@@ -492,7 +488,6 @@ function automation(cwd) {
 		lastError: j.lastError,
 		running: !!j.running,
 		// Set only when it is not this machine's: the panel says so, and nothing else has to guess.
-		otherHost: j.host && j.host !== HOST ? j.host : undefined,
 		// What to type to fix it. The terminal's version of this line says `/cron set <n>`, where n
 		// is the position in a numbered list — which this panel does not have, so telling someone to
 		// use one was telling them to go and find a terminal. A name or a full id resolves anywhere.
@@ -502,7 +497,9 @@ function automation(cwd) {
 		// follows. A time that has already passed is a stale answer rather than a next run (the job
 		// fired and no tick has rewritten the file yet), and saying nothing beats saying something
 		// visibly wrong.
-		next: j.enabled && !(j.host && j.host !== HOST) ? futureOnly(nexts[j.id]) : undefined,
+		next: j.enabled && (j.stateful || (currentSession && j.sessionId === currentSession)) ? futureOnly(nexts[j.id]) : undefined,
+		// Set for a plain job of a session that is not this one: it is asleep, and this is what wakes it.
+		asleep: !j.stateful && !(currentSession && j.sessionId === currentSession) ? "session " + String(j.sessionId || "?").slice(0, 8) + " — resume it to run" : undefined,
 	}));
 	const allRules = readJson(path.join(LOOPS_DIR, "triggers.json"), { rules: [] }).rules ?? [];
 	const rules = allRules
@@ -547,6 +544,8 @@ const PAGE_COMMANDS = [
 	{ name: "clear", description: "start a new session — this one stays on disk, /resume brings it back" },
 	{ name: "new", description: "start a new session — this one stays on disk, /resume brings it back" },
 	{ name: "resume", description: "go back to an earlier session in this project" },
+	{ name: "sessions", description: "list this project's sessions: id, when, what was first said" },
+	{ name: "session", description: "export [path] | import <path> — the .pisession archive commands" },
 	{ name: "compact", description: "compact the context; anything after it steers the summary" },
 ];
 
@@ -1731,7 +1730,7 @@ function liveHostPid() {
  * their automation is dead when it is not.
  */
 function automationToKeepRunning() {
-	const ours = (x) => x.enabled && (!x.host || x.host === HOST);
+	const ours = (x) => x.enabled;
 	const jobs = (readJson(path.join(LOOPS_DIR, "jobs.json"), { jobs: [] }).jobs ?? []).filter((j) => ours(j) && j.stateful);
 	const rules = (readJson(path.join(LOOPS_DIR, "triggers.json"), { rules: [] }).rules ?? []).filter(ours);
 	return jobs.length + rules.length;
@@ -2443,6 +2442,10 @@ const api = (p, b) =>
 		.catch((e) => ({ success: false, error: String(e) }));
 const $ = (id) => document.getElementById(id);
 const feed = $("feed"), statusEl = $("status");
+/** How long a tab stays subscribed after it goes into the background. */
+const STREAM_PARK_MS = 15000;
+/** How long a send may take before the page says that it is taking long. */
+const SEND_WATCHDOG_MS = 6000;
 let state = {}, cwd = "", busy = false, atBottom = true, takesImages = true;
 /** Installed once the stream is up; until then a poll has nothing to compare against. */
 let checkStream;
@@ -3429,9 +3432,9 @@ function renderSidebar(s) {
         '<button data-run="' + safeText(j.id) + '">run</button></div>' +
         '<div class="m">' + safeText(str(j.prompt).slice(0, 90)) + "</div>" +
         '<div class="m">' + (j.running ? "running · " : "") + "runs " + num(j.runCount) + (j.next ? " · next " + safeText(whenNext(j.next)) : "") + "</div>" +
+        (j.asleep ? '<div class="m" style="color:#c93">' + safeText(j.asleep) + "</div>" : "") +
         // A job belonging to a hostname this machine no longer has: it is listed, because it exists,
         // and it says why nothing is happening rather than leaving you to find out from the silence.
-        (j.otherHost ? '<div class="m" style="color:#c93">other host: ' + safeText(j.otherHost) + " — run <b>/cron set " + safeText(j.ref) + " --host here</b></div>" : "") +
         (j.lastError ? '<div class="m" style="color:#c66">' + safeText(str(j.lastError).slice(0, 120)) + "</div>" : "") + "</div>";
     }
     for (const r of a.rules) {
@@ -3788,7 +3791,7 @@ $("input").addEventListener("paste", (e) => {
 $("composer").onsubmit = async (e) => {
   e.preventDefault();
   const input = $("input");
-  const text = input.value.trim();
+  let text = input.value.trim();
   if (!text && !images.length) return;
   input.value = ""; hidePop();
   if (text) { sentPrompts.push(text); if (sentPrompts.length > 200) sentPrompts.shift(); }
@@ -3797,6 +3800,9 @@ $("composer").onsubmit = async (e) => {
   // instead of being sent: anything attached stays in the tray, and a session about to be replaced
   // is not given a message it would carry nowhere.
   if (pageCommand(text)) return;
+  // '/session export|import' are the archive commands here. pi's own '/session' is a terminal
+  // command that does not exist over rpc, so the two-word spelling is free to mean the extension's.
+  text = text.replace(/^\/session\s+(export|import)\b/i, (_m, sub) => "/session-" + sub.toLowerCase());
   const payload = { text, images, mode: busy ? "follow_up" : undefined };
   const shown = text + (images.length ? "\n[" + images.length + " image(s)]" : "");
   if (!busy) {
@@ -3806,7 +3812,17 @@ $("composer").onsubmit = async (e) => {
     if (drewLocally.length > 20) drewLocally.shift();
   }
   images = []; drawThumbs();
-  const r = await api("/prompt", payload);
+  // A send that neither succeeds nor fails is a send the browser has queued behind its connection
+  // pool — every other tab of this address holding a stream is the usual reason (#14). The fetch is
+  // not given up on; the page says what is happening instead of looking alive.
+  const slow = setTimeout(() => { statusEl.textContent = "still sending — if it never lands, close the other tabs of this address"; }, SEND_WATCHDOG_MS);
+  let r;
+  try {
+    r = await api("/prompt", payload);
+  } finally {
+    clearTimeout(slow);
+    if (statusEl.textContent.startsWith("still sending")) statusEl.textContent = busy ? "working…" : "ready";
+  }
   if (!r.success) row("err", "error", r.error || JSON.stringify(r));
 };
 $("stop").onclick = () => api("/abort", {});
@@ -3918,14 +3934,31 @@ function pageCommand(text) {
   if (!m) return false;
   const name = m[1].toLowerCase();
   const rest = m[2].trim();
-  if (name !== "clear" && name !== "new" && name !== "resume" && name !== "compact") return false;
+  if (name === "session" && /^(export|import)\b/i.test(rest)) return false; // an extension command, sent as one
+  if (name !== "clear" && name !== "new" && name !== "resume" && name !== "compact" && name !== "sessions" && name !== "session") return false;
   row("user", "you", text.trim());
   if (name === "compact") compactNow(rest);
   else if (name === "resume") openSessions();
+  else if (name === "sessions") listSessionsHere();
+  else if (name === "session") row("system", "session", "usage: /session export [path] [--exclude-triggers] | /session import <path>\n/sessions lists this project's sessions; the header shows this one's model, cost and context.");
   else swap("/session/new", {}, "new session — the one you left is under resume");
   return true;
 }
 
+/* /sessions: one line per session, newest first — short id, when it started, what was first said. No backticks in here: this is inside the page template. */
+async function listSessionsHere() {
+  const r = await api("/sessions");
+  const list = r.sessions || [];
+  if (!list.length) return row("system", "sessions", "(no sessions for this project yet)");
+  const lines = list.map((s) => {
+    const when = String(s.startedAt || "").slice(0, 16);
+    const said = plain(s.name || s.first || "").replace(/\s+/g, " ");
+    const chars = [...said];
+    return String(s.id).slice(0, 16) + "  " + when + (s.current ? "  [this one]" : "") + "  " + (chars.length > 80 ? chars.slice(0, 80).join("") + "…" : said);
+  });
+  row("system", "sessions", lines.join("\n"));
+}
+
 $("compact").onclick = () => compactNow("");
 $("clear").onclick = () => swap("/session/new", {}, "new session — the one you left is under resume");
 
@@ -4479,15 +4512,46 @@ function applyEvent(ev) {
     }
   };
 
-  const es = new EventSource("/events?token=" + TOKEN);
-  es.onmessage = (e) => {
-    const ev = JSON.parse(e.data);
-    if (resyncing) return void waiting.push(ev); // applied once the transcript is back
-    applyEvent(ev);
+  /*
+   * One stream per tab, and only while the tab is looked at.
+   *
+   * A browser gives one address six connections, and every open tab used to hold one of them for
+   * its event stream — so the sixth tab, or a few forgotten behind the first, left nothing for the
+   * request that sends what you typed: the fetch queued, neither succeeding nor failing, the bubble
+   * drawn and the message never sent (#14). A tab in the background does not need to hear events as
+   * they happen: it closes its stream after a moment, and when it is looked at again it opens a new
+   * one and polls, and the seq/epoch check above reloads whatever it missed. The pi process behind
+   * this page is not involved either way; it runs, and its jobs run, whether anyone is listening.
+   */
+  let es;
+  const openStream = () => {
+    if (es) return;
+    es = new EventSource("/events?token=" + TOKEN);
+    es.onmessage = (e) => {
+      const ev = JSON.parse(e.data);
+      if (resyncing) return void waiting.push(ev); // applied once the transcript is back
+      applyEvent(ev);
+    };
+    es.onerror = () => { statusEl.textContent = "reconnecting…"; };
+    es.onopen = () => refresh();
   };
-
-  es.onerror = () => { statusEl.textContent = "reconnecting…"; };
-  es.onopen = () => refresh();
+  openStream();
+  let parking;
+  document.addEventListener("visibilitychange", () => {
+    clearTimeout(parking);
+    if (document.visibilityState === "hidden") {
+      // Not at once: switching tabs and back should not cost a reconnect and a reload.
+      parking = setTimeout(() => {
+        if (!es || document.visibilityState !== "hidden") return;
+        es.close();
+        es = undefined;
+        statusEl.textContent = "paused in background";
+      }, STREAM_PARK_MS);
+    } else if (!es) {
+      openStream();
+      refresh().catch(() => {});
+    }
+  });
   setInterval(refresh, 8000);
 })();
 </script>`;