pi-better-goal 0.1.21 → 0.2.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-better-goal",
-  "version": "0.1.21",
+  "version": "0.2.0",
   "description": "Pi extension for goal tracking with background-aware continuation.",
   "license": "MIT",
   "type": "module",

package/src/goal-clock.ts

@@ -62,7 +62,7 @@ export function formatGoalClock(
 ): string {
   const timing = goalTiming(goal, now);
   return (
-    `Goal [${goal.status}]: ${goal.objective}` +
+    `Goal [${goal.status}]: ${flattenObjectiveForRail(goal.objective)}` +
     ` | active ${formatClockDuration(timing.activeSeconds)}` +
     ` | elapsed ${formatClockDuration(timing.elapsedSeconds)}`
   );
@@ -85,10 +85,26 @@ export function renderGoalClockLine(
   if (fixedWidth >= width) {
     return truncateToWidth(`${heading} ${active}`, width);
   }
-  const objective = truncateToWidth(goal.objective, width - fixedWidth);
+  const objective = truncateToWidth(
+    flattenObjectiveForRail(goal.objective),
+    width - fixedWidth,
+  );
   return styleGoalRailLine(heading, `${active} ${objective}`, fg);
 }
 
+/**
+ * Goal rail / clock lines must stay one terminal row. Raw objectives can keep
+ * newlines for prompts and `/goal` inspection, but embedding them here makes
+ * the widget taller than the dock budget and desyncs main-screen differential
+ * paints (Working... / Elapsed stacking into scrollback).
+ */
+export function flattenObjectiveForRail(objective: string): string {
+  return objective
+    .replace(/[\r\n\t]+/g, " ")
+    .replace(/ {2,}/g, " ")
+    .trim();
+}
+
 function styleGoalRailLine(
   heading: string,
   tail: string,

package/src/index.ts

@@ -1,4 +1,5 @@
 import type { ExtensionAPI, ExtensionContext } from "@earendil-works/pi-coding-agent";
+import type { AutocompleteItem } from "@earendil-works/pi-tui";
 import { Type } from "typebox";
 
 import {
@@ -24,6 +25,7 @@ import {
 import { continuationEvidence, type ContinuationEvidence } from "./continuation.js";
 import { observeGoalStall } from "./stall.js";
 import {
+  flattenObjectiveForRail,
   goalClockRefreshDelayMs,
   goalTiming,
   isGoalClockVisible,
@@ -58,6 +60,19 @@ const MAX_NO_PROGRESS_RETRIES = parseRetryLimit(
   DEFAULT_MAX_NO_PROGRESS_RETRIES,
 );
 
+const GOAL_ACTIONS: readonly AutocompleteItem[] = [
+  { value: "pause", label: "pause", description: "Pause the active goal" },
+  { value: "resume", label: "resume", description: "Resume the paused goal" },
+  { value: "clear", label: "clear", description: "Remove the current goal" },
+  { value: "complete", label: "complete", description: "Mark the current goal complete" },
+];
+
+export function goalArgumentCompletions(argumentPrefix: string): AutocompleteItem[] | null {
+  const prefix = argumentPrefix.trimStart().toLowerCase();
+  const matches = GOAL_ACTIONS.filter((action) => action.value.startsWith(prefix));
+  return matches.length > 0 ? [...matches] : null;
+}
+
 function parseDurationEnv(raw: string | undefined, fallback: number): number {
   if (raw === undefined || raw.trim() === "") {
     return fallback;
@@ -485,6 +500,7 @@ export default function (pi: ExtensionAPI): void {
 
   pi.registerCommand("goal", {
     description: "Create, inspect, pause, resume, clear, or complete the active goal",
+    getArgumentCompletions: goalArgumentCompletions,
     handler: async (args, ctx) => {
       currentCtx = ctx;
       const trimmed = args.trim();
@@ -541,7 +557,7 @@ export default function (pi: ExtensionAPI): void {
       if (current && current.status !== "complete" && ctx.hasUI && !isForegroundBusy(ctx)) {
         const replace = await ctx.ui.confirm(
           "Replace active goal?",
-          `Current goal: ${current.objective}`,
+          `Current goal: ${flattenObjectiveForRail(current.objective)}`,
         );
         if (!replace) {
           return;
@@ -550,7 +566,7 @@ export default function (pi: ExtensionAPI): void {
 
       try {
         const goal = startOrReplaceGoal(trimmed, null, ctx, "command");
-        notifyGoal(ctx, `Goal set: ${goal.objective}`);
+        notifyGoal(ctx, `Goal set: ${flattenObjectiveForRail(goal.objective)}`);
       } catch (error) {
         notifyGoal(ctx, error instanceof Error ? error.message : String(error), "error");
       }