pi-quests 0.2.0 → 0.3.0

package/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 ## [Unreleased]
 
+## [0.3.0] - 2026-04-11
+
+- feat: add reorder action to move quests by position (dataplane, tool, /quests command, renderer, revert support)
+- feat: extend clear action with optional all flag to remove all quests regardless of done state
+- feat: detect appended completion markers (e.g. "- DONE") and inject correction nudge via context hook
+- feat: update list rendering to show 1-based positions instead of raw IDs
+- fix: inline type imports across test files to comply with no-inline-imports rule
+- chore: extract shared prompt strings into prompts.ts to eliminate duplication
+- chore: fix release workflow output mapping in create-release job
+
 ## [0.2.0] - 2026-04-11
 
 - feat: add dynamic quest usage nudges via context hook

package/README.md

@@ -1,6 +1,6 @@
 # pi-quests
 
-[![version 0.2.0](https://img.shields.io/badge/version-0.2.0-blue)](CHANGELOG.md)
+[![version 0.3.0](https://img.shields.io/badge/version-0.3.0-blue)](CHANGELOG.md)
 [![MIT license](https://img.shields.io/badge/license-MIT-green)](LICENSE.md)
 [![pi extension](https://img.shields.io/badge/pi-extension-purple)](https://github.com/badlogic/pi-mono/tree/main/packages/coding-agent)
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-quests",
-  "version": "0.2.0",
+  "version": "0.3.0",
   "description": "A quest-log for your pi. Keep your agent on track, one quest at a time.",
   "type": "module",
   "keywords": [

package/src/commands/handler.ts

@@ -17,6 +17,7 @@ type MutatingCommand = Extract<
       | typeof QUEST_ACTIONS.update
       | typeof QUEST_ACTIONS.delete
       | typeof QUEST_ACTIONS.clear
+      | typeof QUEST_ACTIONS.reorder
       | typeof QUEST_ACTIONS.revert;
   }
 >["action"];
@@ -32,7 +33,12 @@ const commandActionBuilders: {
     description: p.description,
   }),
   [QUEST_ACTIONS.delete]: (p) => ({ type: QUEST_ACTIONS.delete, id: p.id }),
-  [QUEST_ACTIONS.clear]: () => ({ type: QUEST_ACTIONS.clear }),
+  [QUEST_ACTIONS.clear]: (p) => ({ type: QUEST_ACTIONS.clear, all: p.all }),
+  [QUEST_ACTIONS.reorder]: (p) => ({
+    type: QUEST_ACTIONS.reorder,
+    id: p.id,
+    targetIndex: p.targetIndex,
+  }),
   [QUEST_ACTIONS.revert]: () => ({ type: QUEST_ACTIONS.revert }),
 };
 
@@ -101,6 +107,7 @@ export function createQuestsHandler(pi: ExtensionAPI, questLog: QuestLog) {
       case QUEST_ACTIONS.update:
       case QUEST_ACTIONS.delete:
       case QUEST_ACTIONS.clear:
+      case QUEST_ACTIONS.reorder:
       case QUEST_ACTIONS.revert: {
         const builder = commandActionBuilders[parsed.action];
         const action = builder(parsed as never);
@@ -119,8 +126,9 @@ export function createQuestsHandler(pi: ExtensionAPI, questLog: QuestLog) {
         lines.push("  toggle <id>        - Toggle quest completion");
         lines.push("  delete <id>        - Delete a quest");
         lines.push("  update <id> <desc> - Update a quest description");
+        lines.push("  reorder <id> <idx> - Reorder a quest to index");
         lines.push("  revert             - Revert the last quest change");
-        lines.push("  clear              - Clear all quests");
+        lines.push("  clear [all]        - Clear completed quests, or optionally all quests");
         lines.push("  version            - Show version");
         lines.push("  changelog          - Show changelog");
         lines.push("  h, help            - Show this help message");

package/src/commands/parse-args.ts

@@ -7,7 +7,8 @@ export type ParsedArgs =
   | { action: typeof QUEST_ACTIONS.toggle; id: number }
   | { action: typeof QUEST_ACTIONS.update; id: number; description: string }
   | { action: typeof QUEST_ACTIONS.delete; id: number }
-  | { action: typeof QUEST_ACTIONS.clear }
+  | { action: typeof QUEST_ACTIONS.clear; all?: boolean }
+  | { action: typeof QUEST_ACTIONS.reorder; id: number; targetIndex: number }
   | { action: typeof QUEST_ACTIONS.revert }
   | { action: "help" }
   | { action: "version" }
@@ -36,7 +37,7 @@ export function parseQuestArgs(args: string): ParsedArgs {
 
   // Quest actions without arguments
   if (command === QUEST_ACTIONS.list) return { action: QUEST_ACTIONS.list };
-  if (command === QUEST_ACTIONS.clear) return { action: QUEST_ACTIONS.clear };
+  if (command === QUEST_ACTIONS.clear) return parseClearArgs(rest);
   if (command === QUEST_ACTIONS.revert) return { action: QUEST_ACTIONS.revert };
 
   // Quest actions with arguments
@@ -44,6 +45,7 @@ export function parseQuestArgs(args: string): ParsedArgs {
   if (command === QUEST_ACTIONS.toggle) return parseIdAction(QUEST_ACTIONS.toggle, rest);
   if (command === QUEST_ACTIONS.delete) return parseIdAction(QUEST_ACTIONS.delete, rest);
   if (command === QUEST_ACTIONS.update) return parseUpdateArgs(rest);
+  if (command === QUEST_ACTIONS.reorder) return parseReorderArgs(rest);
 
   return { error: `Unknown subcommand: ${command}. Use /quests help to see available commands.` };
 }
@@ -76,3 +78,17 @@ function parseUpdateArgs(tokens: string[]): ParsedArgs {
 
   return { action: QUEST_ACTIONS.update, id, description };
 }
+
+function parseClearArgs(tokens: string[]): ParsedArgs {
+  const all = tokens[0] === "all";
+  if (tokens.length > 0 && !all) return { error: "Usage: /quests clear [all]" };
+  return { action: QUEST_ACTIONS.clear, all };
+}
+
+function parseReorderArgs(tokens: string[]): ParsedArgs {
+  const id = Number(tokens[0]);
+  const targetIndex = Number(tokens[1]);
+  if (Number.isNaN(id) || Number.isNaN(targetIndex))
+    return { error: "Usage: /quests reorder <id> <index>" };
+  return { action: QUEST_ACTIONS.reorder, id, targetIndex };
+}

package/src/index.ts

@@ -1,5 +1,7 @@
+import type { UserMessage } from "@mariozechner/pi-ai";
 import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
 import { createQuestsHandler } from "./commands/handler.js";
+import { QUEST_PROMPT_GATE, QUEST_PROMPT_REMINDER } from "./prompts.js";
 import { QuestLog } from "./quest/dataplane.js";
 import { QuestUsageTracker } from "./quest/tracker.js";
 import { questChangelogRenderer } from "./renderers/changelog.js";
@@ -36,11 +38,20 @@ export default function (pi: ExtensionAPI): void {
 
     const activeQuestCount = questLog.getAll().filter((q) => !q.done).length;
     const nudge = tracker.getNudge(activeQuestCount, latestPrompt);
-    if (!nudge) return undefined;
 
-    const reminder: import("@mariozechner/pi-ai").UserMessage = {
+    const fakeDoneRegex =
+      /\s[-–—]\s*(DONE|COMPLETED|FINISHED)$|\s[([](DONE|COMPLETED|FINISHED)[)\]]$/i;
+    const fakeDone = questLog.getAll().find((q) => !q.done && fakeDoneRegex.test(q.description));
+    if (!nudge && !fakeDone) return undefined;
+
+    let content = nudge ?? "";
+    if (fakeDone) {
+      content += `\nQUEST REMINDER: Quest #${fakeDone.id} has a completion marker appended to its description but is not toggled done. Use the toggle action to mark it done. NEVER append completion markers to descriptions via the update action.`;
+    }
+
+    const reminder: UserMessage = {
       role: "user",
-      content: nudge,
+      content: content.trim(),
       timestamp: Date.now(),
     };
     return { messages: [...event.messages, reminder] };
@@ -48,11 +59,7 @@ export default function (pi: ExtensionAPI): void {
 
   pi.on("before_agent_start", async (event) => {
     const quests = questLog.getAll();
-    const gate =
-      "# Quest Management\nBefore reading files, running commands, or making edits, ALWAYS ensure the current work is tracked as specific, actionable quests. ALWAYS break broad requests into concrete steps.\n\n";
-
-    let reminder =
-      "## Quest Management\nUse the quest tool VERY frequently to track tasks, plans, and progress throughout the conversation. It is critical that you toggle quests to done as soon as you complete them. NEVER batch up multiple tasks before marking them completed.\n\nNEVER create a single vague quest for broad requests. Analyze the user's intent and break it into specific, independent, actionable quests that each represent a concrete step.";
+    let reminder = QUEST_PROMPT_REMINDER.join("\n");
     if (quests.length > 0) {
       const remaining = quests.filter((q) => !q.done).length;
       const list = quests
@@ -62,7 +69,9 @@ export default function (pi: ExtensionAPI): void {
       reminder += `\n\nActive quests (${remaining}/${quests.length}):\n${list}`;
     }
 
-    return { systemPrompt: `${gate}${event.systemPrompt}\n\n${reminder}` };
+    return {
+      systemPrompt: `# Quest Management\n${QUEST_PROMPT_GATE}${event.systemPrompt}\n\n## Quest Management\n${reminder}`,
+    };
   });
 
   registerQuestTool(pi, questLog);

package/src/prompts.ts

@@ -0,0 +1,16 @@
+export const QUEST_PROMPT_REMINDER = [
+  "Use the quest tool VERY frequently to track tasks, plans, and progress throughout the conversation.",
+  "Before reading files, running commands, or making edits, ensure the current work is tracked as specific, actionable quests.",
+  "Do not create a single vague quest for broad requests. Break them into concrete, independent steps.",
+  "When the user gives a plan or a list of tasks, add them as quests immediately.",
+  "When reading a skill file, implementation plan, or protocol document that contains numbered steps or checklists, add those steps as quests immediately so they are tracked, and reorder them as needed in the quest log.",
+  "It is critical that you toggle quests to done as soon as you complete them. Do NOT batch completions.",
+  "ALWAYS use the toggle action to mark a quest done. NEVER use the update action to append 'DONE', '- DONE', or any completion marker to a quest description.",
+  "As work evolves, use the reorder action to reflect changes in priority or sequencing.",
+  "Before delegating to a minion, add a quest for the delegated task.",
+  "For reorder, targetIndex is 0-based (array index), not a 1-based position. If the user says 'move quest to position 5', use targetIndex 4.",
+  "If you are unsure what to do next, use the list action to check active quests.",
+] as const;
+
+export const QUEST_PROMPT_GATE =
+  "Before reading files, running commands, or making edits, ALWAYS ensure the current work is tracked as specific, actionable quests. ALWAYS break broad requests into concrete steps.";

package/src/quest/dataplane.ts

@@ -2,7 +2,6 @@ import type { AgentToolResult, ExtensionContext } from "@mariozechner/pi-coding-
 import { logger } from "../logger.js";
 import {
   formatBatchAddResult,
-  formatClearResult,
   formatDeleteResult,
   formatNotFound,
   formatQuestList,
@@ -16,7 +15,14 @@ export type HistoryEntry =
   | { type: typeof QUEST_ACTIONS.toggle; id: number }
   | { type: typeof QUEST_ACTIONS.update; id: number; previousDescription: string }
   | { type: typeof QUEST_ACTIONS.delete; quest: Quest; index: number }
-  | { type: typeof QUEST_ACTIONS.clear; quests: Quest[]; nextId: number };
+  | {
+      type: typeof QUEST_ACTIONS.clear;
+      previousQuests: Quest[];
+      previousNextId: number;
+      all: false;
+    }
+  | { type: typeof QUEST_ACTIONS.clear; quests: Quest[]; nextId: number; all: true }
+  | { type: typeof QUEST_ACTIONS.reorder; quest: Quest; oldIndex: number; previousIds: number[] };
 
 export type QuestAction =
   | { type: typeof QUEST_ACTIONS.add; descriptions?: string[] }
@@ -24,7 +30,8 @@ export type QuestAction =
   | { type: typeof QUEST_ACTIONS.toggle; id?: number }
   | { type: typeof QUEST_ACTIONS.update; id?: number; description?: string }
   | { type: typeof QUEST_ACTIONS.delete; id?: number }
-  | { type: typeof QUEST_ACTIONS.clear }
+  | { type: typeof QUEST_ACTIONS.clear; all?: boolean }
+  | { type: typeof QUEST_ACTIONS.reorder; id?: number; targetIndex?: number }
   | { type: typeof QUEST_ACTIONS.revert };
 
 export type QuestOperationResult = { success: boolean; message: string; quest?: Quest };
@@ -88,12 +95,35 @@ export class QuestLog {
       return { success: true, message: `Reverted delete for quest #${entry.quest.id}` };
     },
     [QUEST_ACTIONS.clear]: (entry) => {
+      if ("previousQuests" in entry) {
+        const restoredCount = entry.previousQuests.length - this.quests.length;
+        this.quests = [...entry.previousQuests];
+        this.nextId = entry.previousNextId;
+
+        return {
+          success: true,
+          message: `Reverted clear (${restoredCount} quests restored)`,
+        };
+      }
+
       this.quests = [...entry.quests];
       this.nextId = entry.nextId;
 
-      logger.debug("quests:state", "revert-clear", { count: entry.quests.length });
       return { success: true, message: `Reverted clear (${entry.quests.length} quests restored)` };
     },
+    [QUEST_ACTIONS.reorder]: (entry) => {
+      const currentIndex = this.quests.indexOf(entry.quest);
+      if (currentIndex === -1) return { success: false, message: "Reordered quest not found" };
+
+      this.quests.splice(currentIndex, 1);
+      this.quests.splice(entry.oldIndex, 0, entry.quest);
+      for (let i = 0; i < this.quests.length; i++) {
+        this.quests[i].id = entry.previousIds[i];
+      }
+
+      this.nextId = Math.max(...entry.previousIds, 0) + 1;
+      return { success: true, message: `Reverted reorder for quest #${entry.quest.id}` };
+    },
   };
 
   getAll(): Quest[] {
@@ -174,16 +204,63 @@ export class QuestLog {
     return quest;
   }
 
-  clear(): number {
+  clear(all = false): number {
+    if (!all) {
+      const done = this.quests.filter((q) => q.done);
+      const previousQuests = this.quests.map((q) => ({ ...q }));
+      const previousNextId = this.nextId;
+      this.quests = this.quests.filter((q) => !q.done);
+      for (let i = 0; i < this.quests.length; i++) {
+        this.quests[i].id = i + 1;
+      }
+      this.nextId = this.quests.length + 1;
+      this.history.push({
+        type: QUEST_ACTIONS.clear,
+        previousQuests,
+        previousNextId,
+        all: false,
+      });
+      logger.debug("quests:state", QUEST_ACTIONS.clear, { count: done.length, all });
+      return done.length;
+    }
     const count = this.quests.length;
-    this.history.push({ type: QUEST_ACTIONS.clear, quests: [...this.quests], nextId: this.nextId });
+    this.history.push({
+      type: QUEST_ACTIONS.clear,
+      quests: [...this.quests],
+      nextId: this.nextId,
+      all: true,
+    });
     this.quests = [];
     this.nextId = 1;
-
-    logger.debug("quests:state", QUEST_ACTIONS.clear, { count });
+    logger.debug("quests:state", QUEST_ACTIONS.clear, { count, all });
     return count;
   }
 
+  reorder(id: number, targetIndex: number): Quest | undefined {
+    const idx = this.quests.findIndex((q) => q.id === id);
+    if (idx === -1) {
+      logger.debug("quests:state", "reorder-not-found", { id });
+      return undefined;
+    }
+
+    const previousIds = this.quests.map((q) => q.id);
+    const [quest] = this.quests.splice(idx, 1);
+    this.quests.splice(targetIndex, 0, quest);
+    for (let i = 0; i < this.quests.length; i++) {
+      this.quests[i].id = i + 1;
+    }
+
+    this.nextId = this.quests.length + 1;
+    this.history.push({ type: QUEST_ACTIONS.reorder, quest, oldIndex: idx, previousIds });
+    logger.debug("quests:state", QUEST_ACTIONS.reorder, {
+      id: quest.id,
+      targetIndex,
+      total: this.quests.length,
+    });
+
+    return quest;
+  }
+
   revert(): QuestOperationResult {
     const entry = this.history.pop();
     if (!entry) {
@@ -274,8 +351,24 @@ export class QuestLog {
         return { success: true, message: formatDeleteResult(q), quest: q };
       }
       case QUEST_ACTIONS.clear: {
-        const count = this.clear();
-        return { success: true, message: formatClearResult(count) };
+        const count = this.clear(action.all);
+        const message = action.all
+          ? `Cleared ${count} quests`
+          : `Cleared ${count} completed quests`;
+
+        return { success: true, message };
+      }
+      case QUEST_ACTIONS.reorder: {
+        if (action.id === undefined)
+          return { success: false, message: "Error: id is required for reorder action" };
+
+        if (action.targetIndex === undefined)
+          return { success: false, message: "Error: targetIndex is required for reorder action" };
+
+        const q = this.reorder(action.id, action.targetIndex);
+        if (!q) return { success: false, message: formatNotFound(action.id) };
+
+        return { success: true, message: `Reordered quest #${q.id}: ${q.description}`, quest: q };
       }
       case QUEST_ACTIONS.revert: {
         return this.revert();

package/src/quest/formatters.ts

@@ -2,7 +2,8 @@ export function formatQuestList(
   quests: { id: number; description: string; done: boolean }[],
 ): string {
   if (quests.length === 0) return "No quests.";
-  return quests.map((q) => `#${q.id} [${q.done ? "x" : " "}] ${q.description}`).join("\n");
+
+  return quests.map((q, i) => `#${i + 1} [${q.done ? "x" : " "}] ${q.description}`).join("\n");
 }
 
 export function formatAddResult(q: { id: number; description: string }): string {

package/src/quest/types.ts

@@ -9,6 +9,7 @@ export const QUEST_ACTIONS = {
   update: "update",
   delete: "delete",
   clear: "clear",
+  reorder: "reorder",
   revert: "revert",
 } as const;
 
@@ -19,6 +20,7 @@ export const QUEST_ACTION_VALUES = [
   QUEST_ACTIONS.update,
   QUEST_ACTIONS.delete,
   QUEST_ACTIONS.clear,
+  QUEST_ACTIONS.reorder,
   QUEST_ACTIONS.revert,
 ] as const;
 

package/src/renderers/commands.ts

@@ -110,9 +110,11 @@ export class QuestListWidget {
 
       const start = this.page * this.pageSize;
       const pageQuests = this.quests.slice(start, start + this.pageSize);
-      for (const q of pageQuests) {
+      for (let i = 0; i < pageQuests.length; i++) {
+        const q = pageQuests[i];
+        const pos = start + i + 1;
         const marker = q.done ? th.fg("success", "  ✓ ") : th.fg("muted", "  ○ ");
-        const idStr = th.fg(q.done ? "dim" : "accent", `#${q.id}`);
+        const idStr = th.fg(q.done ? "dim" : "accent", `#${pos}`);
         const desc = q.done
           ? th.fg("dim", th.strikethrough(q.description))
           : th.fg("text", q.description);

package/src/renderers/tools.ts

@@ -46,15 +46,20 @@ export function renderQuestResult(
     isPartial: options.isPartial,
   });
   const details = result.details as Record<string, unknown> | undefined;
-  const questsToRender = details?.displayQuests ?? details?.quests;
+  const allQuests =
+    (details?.quests as Array<{ id: number; description: string; done: boolean }> | undefined) ??
+    [];
+  const questsToRender = (details?.displayQuests ?? details?.quests) as
+    | Array<{ id: number; description: string; done: boolean }>
+    | undefined;
 
   if (Array.isArray(questsToRender) && questsToRender.length > 0) {
-    const lines = (questsToRender as Array<{ id: number; description: string; done: boolean }>).map(
-      (q) => {
-        const marker = q.done ? theme.fg("success", "✓") : theme.fg("dim", "○");
-        return `${marker} ${theme.fg("text", `#${q.id}`)} ${theme.fg("muted", q.description)}`;
-      },
-    );
+    const lines = questsToRender.map((q) => {
+      const pos = allQuests.findIndex((x) => x.id === q.id) + 1 || 1;
+      const marker = q.done ? theme.fg("success", "✓") : theme.fg("dim", "○");
+
+      return `${marker} ${theme.fg("text", `#${pos}`)} ${theme.fg("muted", q.description)}`;
+    });
 
     return new Text(lines.join("\n"), 0, 0);
   }

package/src/tools/handler.ts

@@ -1,6 +1,7 @@
 import type { AgentToolResult, ExtensionAPI } from "@mariozechner/pi-coding-agent";
 import type { Static } from "@sinclair/typebox";
 import { logger } from "../logger.js";
+import { QUEST_PROMPT_GATE, QUEST_PROMPT_REMINDER } from "../prompts.js";
 import { makeToolResult, type QuestAction, type QuestLog } from "../quest/dataplane.js";
 import { QUEST_ACTIONS } from "../quest/types.js";
 import { renderQuestCall, renderQuestResult } from "../renderers/tools.js";
@@ -37,8 +38,15 @@ const toolHandlers: {
   [QUEST_ACTIONS.delete](questLog, params, toolCallId) {
     return runTool(questLog, toolCallId, { type: QUEST_ACTIONS.delete, id: params.id });
   },
-  [QUEST_ACTIONS.clear](questLog, _params, toolCallId) {
-    return runTool(questLog, toolCallId, { type: QUEST_ACTIONS.clear });
+  [QUEST_ACTIONS.clear](questLog, params, toolCallId) {
+    return runTool(questLog, toolCallId, { type: QUEST_ACTIONS.clear, all: params.all });
+  },
+  [QUEST_ACTIONS.reorder](questLog, params, toolCallId) {
+    return runTool(questLog, toolCallId, {
+      type: QUEST_ACTIONS.reorder,
+      id: params.id,
+      targetIndex: params.targetIndex,
+    });
   },
   [QUEST_ACTIONS.revert](questLog, _params, toolCallId) {
     return runTool(questLog, toolCallId, { type: QUEST_ACTIONS.revert });
@@ -83,14 +91,7 @@ export function registerQuestTool(pi: ExtensionAPI, questLog: QuestLog): void {
     description:
       "Manage the session quest log. Use this VERY frequently to track tasks, plans, and progress throughout the conversation.",
     promptSnippet: "Add, list, toggle, update, delete, clear, or revert quest items",
-    promptGuidelines: [
-      "Before reading files, running commands, or making edits, ensure the current work is tracked as specific, actionable quests.",
-      "Do not create a single vague quest for broad requests. Break them into concrete, independent steps.",
-      "When the user gives a plan or a list of tasks, add them as quests immediately.",
-      "It is critical that you toggle quests to done as soon as you complete them. Do NOT batch completions.",
-      "Before delegating to a minion, add a quest for the delegated task.",
-      "If you are unsure what to do next, use the list action to check active quests.",
-    ],
+    promptGuidelines: [...QUEST_PROMPT_GATE, ...QUEST_PROMPT_REMINDER],
     parameters: QuestParams,
     execute: (toolCallId, params, _signal, _onUpdate, _ctx) =>
       questToolExecute(questLog, toolCallId, params),

package/src/tools/params.ts

@@ -18,7 +18,18 @@ export const QuestParams = Type.Object({
   ),
   id: Type.Optional(
     Type.Number({
-      description: "Quest ID (required for toggle, update, delete actions)",
+      description: "Quest ID (required for toggle, update, delete, reorder actions)",
+    }),
+  ),
+  targetIndex: Type.Optional(
+    Type.Number({
+      description:
+        "Target index for reorder action (0-based array index). If moving to the 3rd slot in the list, use 2.",
+    }),
+  ),
+  all: Type.Optional(
+    Type.Boolean({
+      description: "Clear all quests when true (defaults to clearing only completed quests)",
     }),
   ),
 });