@stevecastaneda/paseo-progress 1.9.0 → 1.11.0

package/README.md

@@ -24,7 +24,7 @@ The panel lives in Explorer, beside the agent chat, so you can watch both. Open
 
 <img src="images/4-pill.png" alt="The pill above the message box, showing two questions waiting" width="600">
 
-On phones, where Paseo has no Explorer pane, Progress opens as a tab of its own, and the pill stays above the message box while a run is open so there's always a way in. SVG files and videos preview on desktop; on phones they open on the daemon host.
+On phones with Paseo 0.13 or later, Progress opens in Explorer too. Older phone apps have no Explorer, so there it opens as a tab of its own. On any phone, the pill stays above the message box while a run is open, so there's always a way in. SVG files and videos preview on desktop; on phones they open on the daemon host.
 
 The first time a run starts in a worktree whose Progress panel has never been opened, the pill shows a spinner and reads **Progress** instead; pressing it opens the panel. Once the panel has been opened there, the plugin writes an empty `progress-panel-opened` file next to the progress file and the nudge never comes back, even for later runs. It doesn't open the panel by itself because Paseo switches to a workspace to open its panel, which would pull you away from whatever you're looking at.
 
@@ -145,6 +145,14 @@ The progress file, `progress.jsonl`, holds one JSON event per line, each with a
 
 ## Changelog
 
+### 1.11.0
+
+- [Improved] **Agents can change their recommendation.** When you ask an agent about an open question and explaining it changes the agent's mind, it can now change its recommendation, and the question card shows the new one. In Keep building mode, the agent also switches the work it already built to the new recommendation.
+
+### 1.10.0
+
+- [Improved] **Progress opens in Explorer on phones.** On Paseo 0.13 or later, the phone app opens Progress in Explorer, as the desktop app does, and lists it in Explorer's New tab menu. If a Progress tab from an earlier version is still open on your phone, you can close it.
+
 ### 1.9.0
 
 - [Improved] **The pill's popover shows one question at a time.** You used to scroll past every open question to reach the one you wanted. Now **Question 2 of 3** sits at the top with back and next arrows. Copy an answer and the popover opens on the next question the next time you press the pill.

package/client/pill-popover.tsx

@@ -4,7 +4,6 @@ import React, { useRef, useState } from "react";
 import { Text, View } from "react-native";
 import type { Dashboard } from "../shared/dashboard";
 import { useDashboard } from "./dashboard-query";
-import { progressPanel } from "./pills";
 import { PressScale } from "./motion";
 import { type Attachment, isPreviewable, useAttachmentOpener } from "./attachments";
 import { StackedDialog } from "./dialog-stack";
@@ -33,8 +32,8 @@ export function AttentionPopover({ theme, host, layout, workspaceId, close, open
   const colors = theme.colors;
   // Paseo draws the desktop popover on surface1 and the phone sheet on surface0.
   const onSurface1 = !layout.compact;
-  // Phones open the panel as its own tab, and are pressed with a finger.
-  const touch = progressPanel().location === "workspace";
+  // The phone apps are pressed with a finger.
+  const touch = layout.platform !== "web";
   const directory = useWorkspace(workspaceId, (workspace) => workspace.directory);
   const query = useDashboard(host.id, workspaceId, directory);
   const dashboard = query.data?.configured ? query.data.dashboard : null;

package/client/pill-state.ts

@@ -5,8 +5,7 @@ import type { AttentionResult } from "../shared/rpc";
 // icon and what pressing it does can't disagree.
 // - "attention": questions waiting or something stuck; opens a popover.
 // - "started": a run whose panel has never been opened; a one-time nudge.
-// - "open": on a phone, the way into the panel while a run is open, since
-//   phones have no Explorer to open it from.
+// - "open": on a phone, a one-tap way into the panel while a run is open.
 export type Pill =
   | { mode: "attention"; label: string; icon: "stuck" | "questions" }
   | { mode: "started"; label: "Progress"; icon: "started" }

package/client/pills.tsx

@@ -18,13 +18,17 @@ export const PILL_POLL_MS = 5_000;
 // still trigger a check, at most once a poll interval.
 export const IDLE_POLL_EVERY = 6;
 
-// Paseo's desktop app shows the panel in Explorer, beside the chat. The phone
-// apps never draw Explorer's pane, so there the panel is a tab of its own.
-// It has its own id on phones: Paseo 0.10.1 reopens an existing tab where it
+// Phones on Paseo before 0.13 have no Explorer, so there the panel is a tab of
+// its own. addFilePanel arrived in 0.13, the release that gave phones Explorer.
+// The tab has its own id: Paseo 0.10.1 reopens an existing tab where it
 // already is, and earlier versions of this plugin left a "progress" tab in the
 // phone's hidden Explorer pane, which a plugin can't move or close.
-export function progressPanel(): { id: string; location: "explorer" | "workspace" } {
-  return Platform.OS === "web" ? { id: "progress", location: "explorer" } : { id: "progress-tab", location: "workspace" };
+type ProgressPanelPlacement = { id: "progress"; location: "explorer" } | { id: "progress-tab"; location: "workspace" };
+
+export function progressPanel(client: PluginClientContext): ProgressPanelPlacement {
+  return Platform.OS === "web" || "addFilePanel" in client
+    ? { id: "progress", location: "explorer" }
+    : { id: "progress-tab", location: "workspace" };
 }
 
 type Workspace = { id: string; workspaceDirectory?: string; projectRootPath: string };
@@ -44,7 +48,7 @@ export function contributePills(client: PluginClientContext) {
   // The last report per workspace; pillFor turns it into what the pill shows.
   const reports = new Map<string, AttentionResult>();
   let locale = resolveLocale("system");
-  const where = { phone: progressPanel().location === "workspace" };
+  const where = { phone: Platform.OS !== "web" };
   const pillOf = (workspaceId: string) => pillFor(reports.get(workspaceId), where, messagesFor(locale));
   // Bumped on each report so a poll that was already in flight can't bring the pill back.
   let openedReports = 0;
@@ -83,9 +87,9 @@ export function contributePills(client: PluginClientContext) {
     );
   }
 
+  const panel = progressPanel(client);
   const openPanel = (workspaceId: string) => {
-    const { id, location } = progressPanel();
-    client.openPanel(id, { workspaceId, location });
+    client.openPanel(panel.id, { workspaceId, location: panel.location });
   };
 
   const buttonFor = (workspaceId: string, mode: Pill["mode"], label: string) => mode === "attention"

package/dist/server/cli.js

@@ -19796,14 +19796,14 @@ var eventSchema = external_exports.discriminatedUnion("type", [
     // True when the agent had to undo work built on another option.
     changedCourse: external_exports.boolean().optional()
   }),
-  // Adds detail to an open question; its reference and default stay.
   external_exports.object({
     ...base,
     type: external_exports.literal("question.update"),
     id: text,
     question: text.optional(),
     background: text.optional(),
-    files: external_exports.array(fileLinkSchema).optional()
+    files: external_exports.array(fileLinkSchema).optional(),
+    recommend: text.optional()
   }),
   external_exports.object({ ...base, type: external_exports.literal("question.remove"), id: text }),
   // `path` is relative to the worktree root, or absolute when outside it.
@@ -19971,6 +19971,7 @@ function reduceProgress(parsed, now) {
         if (event.question !== void 0) question.question = event.question;
         if (event.background !== void 0) question.background = event.background;
         if (event.files !== void 0) question.files = [...question.files, ...event.files];
+        if (event.recommend !== void 0) question.recommended = event.recommend;
         break;
       }
       case "question.remove":
@@ -20702,6 +20703,13 @@ function existingQuestion(state, id) {
   }
   return question;
 }
+function recommendOption(recommend, letters) {
+  if (!letters.length) return recommend;
+  if (!/^[A-Za-z]$/.test(recommend) || !letters.includes(recommend.toUpperCase())) {
+    throw new UsageError(`--recommend must be one of the option letters: ${letters.join(", ")}.`);
+  }
+  return recommend.toUpperCase();
+}
 function existingTicket(state, id) {
   const wanted = required2(id, "ticket id").toUpperCase();
   const ticket = state.dashboard.tickets.find((candidate) => candidate.id.toUpperCase() === wanted);
@@ -20840,13 +20848,7 @@ ${modeLine(mode)}`
       const letters = options.map((option) => option.letter);
       const duplicate = letters.find((letter, index) => letters.indexOf(letter) !== index);
       if (duplicate) throw new UsageError(`Option ${duplicate} is given twice.`);
-      let recommend = required2(stringOption(values.recommend), "--recommend (the option you would pick)");
-      if (options.length) {
-        if (!/^[A-Za-z]$/.test(recommend) || !letters.includes(recommend.toUpperCase())) {
-          throw new UsageError(`--recommend must be one of the option letters: ${letters.join(", ")}.`);
-        }
-        recommend = recommend.toUpperCase();
-      }
+      const recommend = recommendOption(required2(stringOption(values.recommend), "--recommend (the option you would pick)"), letters);
       const id = state.nextIds.question;
       const big = values.big === void 0 ? void 0 : required2(stringOption(values.big), "--big reason (why the question is big)");
       const rules = QUESTION_MODE_RULES[mode];
@@ -20892,21 +20894,32 @@ ${modeLine(mode)}`
     }
   },
   "question update": {
-    usage: `question update <id> [--background <text>] [--question <text>] [--file <path or http(s) URL>[=<label>] ...]`,
-    summary: "Add detail to a question: replace its background or wording, or add files. Its reference and default stay.",
-    options: { background: { type: "string" }, question: { type: "string" }, file: { type: "string", multiple: true } },
-    run({ positionals, values, state, cwd, root }) {
+    usage: `question update <id> [--recommend <letter or word>] [--background <text>] [--question <text>] [--file <path or http(s) URL>[=<label>] ...]`,
+    summary: "Add detail to a question: change your recommendation while it's open, replace its background or wording, or add files. Its reference stays.",
+    options: { recommend: { type: "string" }, background: { type: "string" }, question: { type: "string" }, file: { type: "string", multiple: true } },
+    run({ positionals, values, state, cwd, root, mode }) {
       const question = existingQuestion(state, positionals[0]);
       const files = listOption(values.file).map((raw) => parseFile(raw, cwd, root));
+      let recommend;
+      if (values.recommend !== void 0) {
+        if (question.answer) throw new UsageError(`${question.id} was already answered, so its recommendation can't change.`);
+        const letters = question.options.map((option) => option.letter);
+        recommend = recommendOption(required2(stringOption(values.recommend), "--recommend (the option you would pick)"), letters);
+        if (recommend === question.recommended) throw new UsageError(`${question.id} already recommends ${recommend}.`);
+      }
       const event = {
         type: "question.update",
         id: question.id,
         question: stringOption(values.question),
         background: stringOption(values.background),
-        files: files.length ? files : void 0
+        files: files.length ? files : void 0,
+        recommend
       };
-      if (!event.question && !event.background && !event.files) throw new UsageError("Nothing to change. Pass --background, --question, or --file.");
-      return { event, message: `Updated ${question.id} (${question.title})` };
+      if (!event.question && !event.background && !event.files && !recommend) throw new UsageError("Nothing to change. Pass --recommend, --background, --question, or --file.");
+      if (!recommend) return { event, message: `Updated ${question.id} (${question.title})` };
+      const rules = QUESTION_MODE_RULES[mode];
+      const next = question.state === "building" ? `Undo the work built on ${question.recommended} and build ${recommend} instead.` : `Mode: ${rules.label}. ${rules.askRule({ recommend, big: Boolean(question.big) })}`;
+      return { event, message: `${question.id} now recommends ${recommend} (was ${question.recommended}). ${next}` };
     }
   },
   "question remove": {

package/index.client.tsx

@@ -9,7 +9,7 @@ const SETTINGS_ID = "settings";
 
 export default function contribute(client: PluginClientContext) {
   const stopPills = contributePills(client);
-  const panel = progressPanel();
+  const panel = progressPanel(client);
   const stopPanel = client.addWorkspacePanel({
     id: panel.id,
     title: "Progress",
@@ -24,7 +24,6 @@ export default function contribute(client: PluginClientContext) {
     icon: "ListChecks",
     keywords: ["dashboard", "tickets", "status", "questions"],
     context: "workspace",
-    // Beside the agent chat where Paseo can show both; a tab of its own on phones.
     onSelect({ openPanel }) {
       openPanel(panel.id, { location: panel.location });
     },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stevecastaneda/paseo-progress",
-  "version": "1.9.0",
+  "version": "1.11.0",
   "description": "A live progress dashboard per worktree for Paseo, drawn from a file agents write with the paseo-progress command.",
   "license": "MIT",
   "author": "Steve Castaneda",

package/server/cli.ts

@@ -121,6 +121,14 @@ function existingQuestion(state: State, id: string | undefined) {
   return question;
 }
 
+function recommendOption(recommend: string, letters: string[]): string {
+  if (!letters.length) return recommend;
+  if (!/^[A-Za-z]$/.test(recommend) || !letters.includes(recommend.toUpperCase())) {
+    throw new UsageError(`--recommend must be one of the option letters: ${letters.join(", ")}.`);
+  }
+  return recommend.toUpperCase();
+}
+
 function existingTicket(state: State, id: string | undefined) {
   const wanted = required(id, "ticket id").toUpperCase();
   const ticket = state.dashboard.tickets.find((candidate) => candidate.id.toUpperCase() === wanted);
@@ -269,13 +277,7 @@ const commands: Record<string, Command> = {
       const letters = options.map((option) => option.letter);
       const duplicate = letters.find((letter, index) => letters.indexOf(letter) !== index);
       if (duplicate) throw new UsageError(`Option ${duplicate} is given twice.`);
-      let recommend = required(stringOption(values.recommend), "--recommend (the option you would pick)");
-      if (options.length) {
-        if (!/^[A-Za-z]$/.test(recommend) || !letters.includes(recommend.toUpperCase())) {
-          throw new UsageError(`--recommend must be one of the option letters: ${letters.join(", ")}.`);
-        }
-        recommend = recommend.toUpperCase();
-      }
+      const recommend = recommendOption(required(stringOption(values.recommend), "--recommend (the option you would pick)"), letters);
       const id = state.nextIds.question;
       const big = values.big === undefined ? undefined : required(stringOption(values.big), "--big reason (why the question is big)");
       const rules = QUESTION_MODE_RULES[mode];
@@ -323,21 +325,34 @@ const commands: Record<string, Command> = {
     },
   },
   "question update": {
-    usage: `question update <id> [--background <text>] [--question <text>] [--file <path or http(s) URL>[=<label>] ...]`,
-    summary: "Add detail to a question: replace its background or wording, or add files. Its reference and default stay.",
-    options: { background: { type: "string" }, question: { type: "string" }, file: { type: "string", multiple: true } },
-    run({ positionals, values, state, cwd, root }) {
+    usage: `question update <id> [--recommend <letter or word>] [--background <text>] [--question <text>] [--file <path or http(s) URL>[=<label>] ...]`,
+    summary: "Add detail to a question: change your recommendation while it's open, replace its background or wording, or add files. Its reference stays.",
+    options: { recommend: { type: "string" }, background: { type: "string" }, question: { type: "string" }, file: { type: "string", multiple: true } },
+    run({ positionals, values, state, cwd, root, mode }) {
       const question = existingQuestion(state, positionals[0]);
       const files = listOption(values.file).map((raw) => parseFile(raw, cwd, root));
+      let recommend: string | undefined;
+      if (values.recommend !== undefined) {
+        if (question.answer) throw new UsageError(`${question.id} was already answered, so its recommendation can't change.`);
+        const letters = question.options.map((option) => option.letter);
+        recommend = recommendOption(required(stringOption(values.recommend), "--recommend (the option you would pick)"), letters);
+        if (recommend === question.recommended) throw new UsageError(`${question.id} already recommends ${recommend}.`);
+      }
       const event = {
         type: "question.update" as const,
         id: question.id,
         question: stringOption(values.question),
         background: stringOption(values.background),
         files: files.length ? files : undefined,
+        recommend,
       };
-      if (!event.question && !event.background && !event.files) throw new UsageError("Nothing to change. Pass --background, --question, or --file.");
-      return { event, message: `Updated ${question.id} (${question.title})` };
+      if (!event.question && !event.background && !event.files && !recommend) throw new UsageError("Nothing to change. Pass --recommend, --background, --question, or --file.");
+      if (!recommend) return { event, message: `Updated ${question.id} (${question.title})` };
+      const rules = QUESTION_MODE_RULES[mode];
+      const next = question.state === "building"
+        ? `Undo the work built on ${question.recommended} and build ${recommend} instead.`
+        : `Mode: ${rules.label}. ${rules.askRule({ recommend, big: Boolean(question.big) })}`;
+      return { event, message: `${question.id} now recommends ${recommend} (was ${question.recommended}). ${next}` };
     },
   },
   "question remove": {

package/shared/dashboard.ts

@@ -308,6 +308,7 @@ export function reduceProgress(parsed: { events: ParsedLine[]; issues: FileIssue
         if (event.question !== undefined) question.question = event.question;
         if (event.background !== undefined) question.background = event.background;
         if (event.files !== undefined) question.files = [...question.files, ...event.files];
+        if (event.recommend !== undefined) question.recommended = event.recommend;
         break;
       }
       case "question.remove":

package/shared/events.ts

@@ -112,7 +112,6 @@ export const eventSchema = z.discriminatedUnion("type", [
     // True when the agent had to undo work built on another option.
     changedCourse: z.boolean().optional(),
   }),
-  // Adds detail to an open question; its reference and default stay.
   z.object({
     ...base,
     type: z.literal("question.update"),
@@ -120,6 +119,7 @@ export const eventSchema = z.discriminatedUnion("type", [
     question: text.optional(),
     background: text.optional(),
     files: z.array(fileLinkSchema).optional(),
+    recommend: text.optional(),
   }),
   z.object({ ...base, type: z.literal("question.remove"), id: text }),
   // `path` is relative to the worktree root, or absolute when outside it.

package/skills/paseo-progress/SKILL.md

@@ -41,7 +41,7 @@ When a decision belongs to the user, ask it with `question ask`, lettered `--opt
 The user picks the question mode in Paseo's Settings, and `question ask` prints what that mode means for this question. Do what it prints. Record the user's reply ("Q7 A", or their own words) with `question answer Q7 A --words "<their words>"`, and do what that prints too.
 
 - When the question is about how something looks, attach the screenshot or link with `--file`.
-- When the user asks about a question, answer in chat and add the explanation with `question update Q7 --background`.
+- When the user asks about a question, answer in chat and add the explanation with `question update Q7 --background`. If explaining it changes your mind, also pass `--recommend` with the new option, say in `--background` what changed, and do what it prints.
 
 ## 4. Finish