pum-agent 0.2.8-beta.1 → 0.2.9-beta.1

package/README.md

@@ -140,6 +140,7 @@ Set `PUM_DIR` to override PUM's complete configuration and data directory. Run `
 | `Ctrl+L` | Open the agent transcript selector |
 | `Shift+Tab` / `Ctrl+Shift+Tab` | Cycle through agent transcripts |
 | `Ctrl+H` | Open session history when the terminal reports the key distinctly |
+| `Ctrl+N` | Open recent answers (News) |
 | `Ctrl+End` | Scroll to the end of the selected transcript |
 | `Ctrl+P` | Open settings |
 | `Ctrl+T` | Open supervised external triggers |
@@ -147,7 +148,7 @@ Set `PUM_DIR` to override PUM's complete configuration and data directory. Run `
 | `Ctrl+C` | Clear the selected non-empty draft; on an empty draft, press twice to quit |
 | `?` | Show all controls when the prompt is empty |
 
-Useful commands include `/login`, `/history`, `/triggers`, `/check-path`, `/clear`, `/compress`, and `/worktree`.
+Useful commands include `/login`, `/history`, `/news`, `/triggers`, `/check-path`, `/clear`, `/compress`, and `/worktree`.
 
 ### Copy transcript text
 
@@ -171,6 +172,18 @@ Reload the `tmux` configuration after this change. Use the terminal's Shift-drag
 
 PUM limits remote OSC 52 payloads to 100,000 Base64 characters. This limit prevents large selections from corrupting terminal output.
 
+### Recent answers (News)
+
+Open the News popup with `Ctrl+N` or `/news`. It lists the final answers of user-initiated turns, newest first. Each entry shows the user prompt and any follow-up steers that produced the answer, above the answer itself.
+
+- `←` / `→` — move between answers
+- `Space` — toggle an answer between read and unread
+- `c` — copy the current answer to the clipboard
+- `Enter` — reply to the current answer with a quoted draft
+- `Esc` — close the popup
+
+PUM marks an answer read automatically only when a new user prompt follows it directly in the transcript. If anything else appears between the answer and the next prompt — a subagent message, a trigger event, a queued message, or an in-progress stream — the answer stays unread.
+
 ## Parallel subagents
 
 PUM runs up to 10 active subagents by default. Configure a limit from 1 through 25 in Settings. Only starting and running agents count toward the limit. Each subagent has these resources:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pum-agent",
-  "version": "0.2.8-beta.1",
+  "version": "0.2.9-beta.1",
   "description": "A compact terminal coding agent powered by pi and OpenTUI.",
   "license": "MIT",
   "type": "module",

package/src/app.tsx

@@ -9,7 +9,7 @@ import { randomUUID } from "node:crypto";
 import { useKeyboard, usePaste, useRenderer, useTerminalDimensions } from "@opentui/react";
 import { getSupportedThinkingLevels, type Model } from "@earendil-works/pi-ai";
 import type { AgentSession, ModelRuntime } from "@earendil-works/pi-coding-agent";
-import { Fragment, useEffect, useMemo, useRef, useState } from "react";
+import { Fragment, useEffect, useLayoutEffect, useMemo, useRef, useState } from "react";
 import { AnimationProvider, supportsTrueColor, useWorkingRule, type WorkingRuleRole } from "./animation";
 import {
   filterModels,
@@ -135,6 +135,7 @@ import {
 } from "./triggers/popup";
 import type { TerminalTitleController } from "./terminal-title";
 import { readClipboardText } from "./text-paste";
+import { copyTextToClipboard } from "./clipboard";
 import { NewsPopup } from "./news-popup";
 import {
   NEWS_CAPACITY,
@@ -142,6 +143,7 @@ import {
   saveNewsItems,
   tagNewsLines,
   type NewsItem,
+  type NewsPrompt,
 } from "./news";
 
 type Stream = { kind: "assistant" | "thinking"; text: string } | null;
@@ -387,6 +389,7 @@ export function App({
   captureImage = captureClipboardImage,
   readPastedText = readClipboardText,
   stagePastedText = stagePastedTextDefault,
+  copyNewsAnswerText = copyTextToClipboard,
   onExit = () => process.exit(0),
   checkApprovalCoordinator,
   checkApprovalStore,
@@ -415,6 +418,8 @@ export function App({
   readPastedText?: typeof readClipboardText;
   /** Store oversized pasted text in a temp file and show a marker in its place. */
   stagePastedText?: typeof stagePastedTextDefault;
+  /** Copies the selected news answer for the popup. */
+  copyNewsAnswerText?: typeof copyTextToClipboard;
   onExit?: () => void | Promise<void>;
   checkApprovalCoordinator?: CheckApprovalCoordinator;
   checkApprovalStore?: CheckApprovalStore;
@@ -428,19 +433,36 @@ export function App({
 }) {
   const cwd = process.cwd();
   const [session, setSession] = useState(initialSession);
-  const [tx, setTx] = useState<Transcript>(() => ({
+  // Load the news companion file exactly once on mount. The transcript
+  // initializer reads these items from the ref instead of re-reading the file.
+  const newsRef = useRef<NewsItem[]>([]);
+  const [news, setNews] = useState<NewsItem[]>(() => {
+    const items = loadNewsItems(initialSession.sessionFile);
+    newsRef.current = items;
+    return items;
+  });
+  const [tx, setTx] = useState<Transcript>(() => {
     // A resumed session already holds messages; show them instead of a blank pane.
-    lines: [
-      ...replayEntries(
-        initialSession.sessionManager.buildContextEntries(),
-        cwd,
-        initial.showThinking,
-      ),
-      ...startupWarnings.map((text): Line => ({ kind: "text", role: "system", text })),
-    ],
-    stream: null,
-    pending: [],
-  }));
+    const replayedLines = replayEntries(
+      initialSession.sessionManager.buildContextEntries(),
+      cwd,
+      initial.showThinking,
+    );
+    return {
+      lines: [
+        ...tagNewsLines(replayedLines, newsRef.current),
+        ...startupWarnings.map((text): Line => ({ kind: "text", role: "system", text })),
+      ],
+      stream: null,
+      pending: [],
+    };
+  });
+  const txRef = useRef<Transcript>({ lines: [], stream: null, pending: [] });
+  // Layout effect: commit may hand control to a user handler before paint, so
+  // the mirror must update synchronously to avoid a stale transcript read.
+  useLayoutEffect(() => {
+    txRef.current = tx;
+  }, [tx]);
   const [busy, setBusy] = useState(false);
   const [quitArmed, setQuitArmed] = useState(false);
   const [cancelArmed, setCancelArmed] = useState(false);
@@ -494,12 +516,6 @@ export function App({
   const [triggerCursor, setTriggerCursor] = useState(0);
   const [, setTriggerRevision] = useState(0);
 
-  const newsRef = useRef<NewsItem[]>([]);
-  const [news, setNews] = useState<NewsItem[]>(() => {
-    const items = loadNewsItems(initialSession.sessionFile);
-    newsRef.current = items;
-    return items;
-  });
   const [newsOpen, setNewsOpen] = useState(false);
   const newsOpenRef = useRef(false);
   const [newsCursor, setNewsCursor] = useState(0);
@@ -604,6 +620,8 @@ export function App({
   const userTurnActiveRef = useRef(false);
   /** Text of the final assistant message; reset at each assistant message start. */
   const answerBufRef = useRef("");
+  /** User prompt and steer texts of the running main turn, oldest first. */
+  const turnPromptsRef = useRef<NewsPrompt[]>([]);
   const resetQuitArm = () => {
     lastQuitPress.current = 0;
     clearTimeout(quitTimer.current);
@@ -1172,6 +1190,7 @@ export function App({
               at: Date.now(),
               read: false,
               answered: false,
+              prompts: turnPromptsRef.current,
             };
             const nextNews = [item, ...newsRef.current].slice(0, NEWS_CAPACITY);
             newsRef.current = nextNews;
@@ -1193,6 +1212,7 @@ export function App({
           }
           answerBufRef.current = "";
           userTurnActiveRef.current = false;
+          turnPromptsRef.current = [];
           releasePostTurnPastedTexts("main");
           setWorking(false);
           break;
@@ -1481,8 +1501,24 @@ export function App({
   };
 
   const markNewestNewsAnswered = () => {
+    // When a resumed answer's text no longer matches a replayed line (for
+    // example after compaction or regeneration), the line cannot be tagged
+    // with its news id, so a direct prompt will not mark it read. This is
+    // intentional, not a bug.
     const first = newsRef.current[0];
     if (!first || (first.read && first.answered)) return;
+    const current = txRef.current;
+    const last = current.lines[current.lines.length - 1];
+    // Mark read only when the new user prompt lands directly after the newest
+    // answer. Any interleaved line (agent message, trigger event, queued
+    // message, or an in-progress stream) means the prompt is not a direct reply.
+    const directReply =
+      current.stream === null &&
+      current.pending.length === 0 &&
+      last?.kind === "text" &&
+      last.role === "assistant" &&
+      last.newsId === first.id;
+    if (!directReply) return;
     commitNews(
       newsRef.current.map((entry, index) =>
         index === 0 ? { ...entry, read: true, answered: true } : entry,
@@ -1513,6 +1549,16 @@ export function App({
     });
   };
 
+  const copyNewsAnswer = () => {
+    const item = newsRef.current[newsCursorRef.current];
+    if (!item) return;
+    copyNewsAnswerText(item.text, {
+      osc52: (value) => renderer.copyToClipboardOSC52(value),
+    }).catch((error) =>
+      append({ kind: "text", role: "error", text: `copy failed: ${String(error)}` }),
+    );
+  };
+
   const selectHistorySession = (path: string) => {
     setHistoryOpen(false);
     if (path === session.sessionFile) {
@@ -1718,6 +1764,7 @@ export function App({
       text: displayText.trim(),
     };
 
+    turnPromptsRef.current.push({ text: displayText.trim(), steer: busyRef.current });
     // Working already: keep the steering message pending at the transcript
     // bottom until pi emits message_start for its actual insertion.
     if (busyRef.current) {
@@ -2320,6 +2367,7 @@ export function App({
       else if (key.name === "left") moveNewsCursor(1);
       else if (key.name === "right") moveNewsCursor(-1);
       else if (key.name === "space" || key.sequence === " ") toggleCurrentNewsRead();
+      else if (key.name === "c" || key.sequence === "c") void copyNewsAnswer();
       else if (key.name === "return" || key.name === "enter" || key.name === "kpenter")
         replyToCurrentNews();
       return;

package/src/clipboard.ts

@@ -82,7 +82,23 @@ export async function copySelectionText(
   text: string,
   options: ClipboardCopyOptions = {},
 ): Promise<ClipboardRoute> {
-  if (!text) throw new Error("Cannot copy an empty selection");
+  return copyTextCore(text, options, "Cannot copy an empty selection");
+}
+
+/** Copy arbitrary text (a news answer, for example) through the same routes. */
+export async function copyTextToClipboard(
+  text: string,
+  options: ClipboardCopyOptions = {},
+): Promise<ClipboardRoute> {
+  return copyTextCore(text, options, "Cannot copy empty text");
+}
+
+async function copyTextCore(
+  text: string,
+  options: ClipboardCopyOptions,
+  emptyMessage: string,
+): Promise<ClipboardRoute> {
+  if (!text) throw new Error(emptyMessage);
 
   const platform = options.platform ?? process.platform;
   const env = options.env ?? process.env;

package/src/commands.ts

@@ -25,6 +25,10 @@ export const COMMANDS: Command[] = [
     name: "/login",
     description: "Add or update a provider login",
   },
+  {
+    name: "/news",
+    description: "Open recent answers (News)",
+  },
   {
     name: "/check-path",
     description: "Manage additional Check mode directory roots",

package/src/help-popup.tsx

@@ -26,6 +26,7 @@ export const HELP_GROUPS: HelpGroup[] = [
       ["Alt+Enter", "Cache without sending"],
       ["Alt+V", "Attach a clipboard image"],
       ["Ctrl+Backspace", "Delete the previous word"],
+      ["Ctrl+W", "Delete the previous word"],
       ["Questionnaire", "↑↓/←→ move · Enter select · Esc cancel"],
     ],
   },

package/src/news-popup.tsx

@@ -3,11 +3,12 @@ import type { ScrollBoxRenderable } from "@opentui/core";
 import { buildSyntaxStyle } from "./syntax";
 import { PopupFrame } from "./popup-frame";
 import { formatAge, type NewsItem } from "./news";
+import { TextLine } from "./transcript";
 import type { Theme } from "./theme";
 
 /**
  * Presentational popup for recent final answers. It owns no keyboard logic;
- * `app.tsx` routes arrows, Space, Enter, and Esc to the shared handlers.
+ * `app.tsx` routes arrows, Space, Enter, Esc, and C to the shared handlers.
  */
 export function NewsPopup({
   theme,
@@ -28,9 +29,9 @@ export function NewsPopup({
   const marginY = terminalHeight >= 16 ? Math.max(1, Math.floor(terminalHeight * 0.07)) : 0;
   const width = Math.max(1, terminalWidth - marginX * 2);
   const height = Math.max(1, terminalHeight - marginY * 2);
-  // PopupFrame adds a 1-c wide border and 1-c padding per side, and the gutter
-  // takes 2 columns, so the markdown body gets a concrete numeric width.
-  const bodyWidth = Math.max(1, width - 6);
+  // PopupFrame adds a 1-c wide border and 1-c padding per side, so the
+  // markdown body gets a concrete numeric width.
+  const bodyWidth = Math.max(1, width - 4);
   const count = items.length;
   const current = items[cursor];
   const seen = current ? current.read : false;
@@ -61,38 +62,45 @@ export function NewsPopup({
             <text content={formatAge(current.at)} fg={theme.dim} bg={theme.popupBg} wrapMode="none" />
           </box>
           <box style={{ height: 1, flexShrink: 0 }} />
-          <box
-            style={{
-              flexGrow: 1,
-              flexShrink: 1,
-              minHeight: 0,
-              flexDirection: "row",
-            }}
+          <scrollbox
+            ref={markdownScrollRef}
+            verticalScrollbarOptions={{ visible: true }}
+            style={{ width: bodyWidth, flexGrow: 1, flexShrink: 0, minWidth: 0 }}
           >
-            <box style={{ width: 2, flexShrink: 0 }}>
-              <text
-                content={seen ? "✓" : "◦"}
-                fg={seen ? theme.success : theme.dim}
-                bg={theme.popupBg}
-                wrapMode="none"
-              />
-            </box>
-            <scrollbox
-              ref={markdownScrollRef}
-              verticalScrollbarOptions={{ visible: true }}
-              style={{ width: bodyWidth, flexGrow: 1, flexShrink: 0, minWidth: 0 }}
-            >
-              <box style={{ width: "100%", paddingRight: 1 }}>
-                <markdown
-                  content={current.text}
-                  streaming={false}
+            <box style={{ width: "100%", paddingRight: 1 }}>
+              {current.prompts?.map((prompt, index) => (
+                <TextLine
+                  key={`${index}:${prompt.text}:${prompt.steer}`}
+                  theme={theme}
                   syntaxStyle={syntaxStyle}
-                  fg={seen ? theme.dim : theme.assistant}
-                  style={{ flexGrow: 1, flexShrink: 1, minWidth: 0, width: "100%" }}
+                  role="user"
+                  text={prompt.text}
                 />
+              ))}
+              {current.prompts && current.prompts.length > 0 ? (
+                <box style={{ height: 1, flexShrink: 0 }} />
+              ) : null}
+              <box style={{ flexDirection: "row", width: "100%" }}>
+                <box style={{ width: 2, flexShrink: 0 }}>
+                  <text
+                    content={seen ? "✓ " : "◦ "}
+                    fg={seen ? theme.success : theme.dim}
+                    bg={theme.popupBg}
+                    wrapMode="none"
+                  />
+                </box>
+                <box style={{ flexDirection: "row", flexGrow: 1, flexShrink: 1, minWidth: 0 }}>
+                  <markdown
+                    content={current.text}
+                    streaming={false}
+                    syntaxStyle={syntaxStyle}
+                    fg={seen ? theme.dim : theme.assistant}
+                    style={{ flexGrow: 1, flexShrink: 1, minWidth: 0, width: "100%" }}
+                  />
+                </box>
               </box>
-            </scrollbox>
-          </box>
+            </box>
+          </scrollbox>
         </>
       ) : (
         <box style={{ flexGrow: 1, flexDirection: "column" }}>
@@ -103,7 +111,7 @@ export function NewsPopup({
       <text
         content={count === 0
           ? "esc close"
-          : "← → navigate · space read/unread · enter reply · esc close"}
+          : "← → navigate · space read/unread · c copy · enter reply · esc close"}
         fg={theme.dim}
         bg={theme.popupBg}
         wrapMode="none"

package/src/news.ts

@@ -1,6 +1,13 @@
 import { basename, dirname, join } from "node:path";
 import { existsSync, readFileSync, renameSync, writeFileSync } from "node:fs";
 
+export type NewsPrompt = {
+  /** The user prompt or steer text that produced the answer. */
+  text: string;
+  /** True when the text steered an already-running turn. */
+  steer: boolean;
+};
+
 export type NewsItem = {
   /** Stable identifier used to tag the matching transcript line. */
   id: string;
@@ -12,6 +19,8 @@ export type NewsItem = {
   read: boolean;
   /** True when the user replied to it. */
   answered: boolean;
+  /** User prompt and steers that produced this answer, oldest first. */
+  prompts?: NewsPrompt[];
 };
 
 /** The list never holds more than this many answers. */
@@ -31,7 +40,15 @@ function isNewsItem(value: unknown): value is NewsItem {
     typeof item.text === "string" &&
     typeof item.at === "number" &&
     typeof item.read === "boolean" &&
-    typeof item.answered === "boolean"
+    typeof item.answered === "boolean" &&
+    (item.prompts === undefined ||
+      (Array.isArray(item.prompts) &&
+        item.prompts.every(
+          (prompt) =>
+            Boolean(prompt) &&
+            typeof (prompt as Record<string, unknown>).text === "string" &&
+            typeof (prompt as Record<string, unknown>).steer === "boolean",
+        )))
   );
 }
 

package/src/settings.ts

@@ -1,4 +1,4 @@
-import { readFileSync, writeFileSync } from "node:fs";
+import { readFileSync, renameSync, writeFileSync } from "node:fs";
 import { join } from "node:path";
 import { AGENT_DIR } from "./config";
 import { DEFAULT_CHECK_MODEL } from "./check-mode";
@@ -152,5 +152,10 @@ export function loadSettings(): PumSettings {
 }
 
 export function saveSettings(settings: PumSettings): void {
-  writeFileSync(SETTINGS_PATH, JSON.stringify(settings, null, 2));
+  // Write to a temporary sibling file, then rename into place, so a crash
+  // during the write cannot corrupt pum.json. loadSettings keeps its tolerance
+  // for a corrupt or missing file, so a leftover temp file never affects it.
+  const temporary = `${SETTINGS_PATH}.${process.pid}.${Date.now()}.tmp`;
+  writeFileSync(temporary, JSON.stringify(settings, null, 2));
+  renameSync(temporary, SETTINGS_PATH);
 }

package/src/status-bar.tsx

@@ -44,6 +44,9 @@ const REMOVAL_ORDER: readonly (StatusMetadataItem["key"] | "title")[] = [
   "title",
 ];
 
+/** The WorkingPulse spinner occupies exactly one terminal column. */
+const PULSE_GLYPH_WIDTH = 1;
+
 type WorkingMode = "full" | "compact" | "pulse" | null;
 type LeftPart = "model" | "thinking" | "agents" | "activeAgent";
 
@@ -105,8 +108,14 @@ function agentTextWidth(input: StatusBarLayoutInput, layout: StatusBarLayout): n
   let width = 0;
   if (layout.showIdleAgents && idle > 0) width += statusTextWidth(`◇ ${idle}`);
   if (layout.showRunningAgents && input.runningAgentCount > 0) {
+    // One space separates the idle block, not the pulse glyph.
     if (width) width += 1;
-    width += 1 + statusTextWidth(` ${input.runningAgentCount}/${input.maxActiveAgentCount}`);
+    // The running block renders as the pulse glyph (PULSE_GLYPH_WIDTH columns)
+    // followed by the " N/M" counter. That leading space is inside the text
+    // that statusTextWidth measures.
+    width += PULSE_GLYPH_WIDTH + statusTextWidth(
+      ` ${input.runningAgentCount}/${input.maxActiveAgentCount}`,
+    );
   }
   return width;
 }
@@ -122,7 +131,7 @@ function leftParts(input: StatusBarLayoutInput, layout: StatusBarLayout): LeftPa
 
 function workingWidth(input: StatusBarLayoutInput, mode: WorkingMode): number {
   if (!mode) return 0;
-  if (mode === "pulse") return 1;
+  if (mode === "pulse") return PULSE_GLYPH_WIDTH;
   const elapsedWidth = statusTextWidth(fmtElapsed(input.elapsedSec));
   return mode === "full" ? 12 + elapsedWidth : 4 + elapsedWidth;
 }