@stevecastaneda/paseo-progress 1.10.0 → 1.11.0

package/README.md

@@ -145,6 +145,10 @@ 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.

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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stevecastaneda/paseo-progress",
-  "version": "1.10.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