pi-quests 0.6.1 → 0.7.0

package/CHANGELOG.md

@@ -2,6 +2,17 @@
 
 ## [Unreleased]
 
+## [0.7.0] - 2026-04-26
+
+- feat: add batch toggle and batch delete support for quests (#2) — @pascal-de-ladurantaye
+- feat: add redo action and rename revert to undo
+- chore: migrate from taskfile to npm scripts and update dependencies
+
+## [0.6.2] - 2026-04-17
+
+- fix: system prompt was too weak
+- chore: add more roadmap items
+
 ## [0.6.1] - 2026-04-17
 
 - fix: make quest prompt injection static and cache-friendly with `nudges.enable` and `display.showStatus` toggles

package/README.md

@@ -1,6 +1,6 @@
 # pi-quests
 
-[![version 0.6.1](https://img.shields.io/badge/version-0.6.1-blue)](CHANGELOG.md)
+[![version 0.7.0](https://img.shields.io/badge/version-0.7.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.6.1",
+  "version": "0.7.0",
   "description": "A quest-log for your pi. Keep your agent on track, one quest at a time.",
   "type": "module",
   "keywords": [
@@ -35,22 +35,27 @@
     "@mariozechner/pi-ai": "*",
     "@mariozechner/pi-coding-agent": "*",
     "@mariozechner/pi-tui": "*",
-    "@sinclair/typebox": "*"
+    "typebox": "*"
   },
   "devDependencies": {
-    "@biomejs/biome": "^2.4.10",
-    "@mariozechner/pi-coding-agent": "^0.63.2",
-    "@mariozechner/pi-tui": "^0.63.0",
-    "typescript": "^5.7.0",
-    "vitest": "^3.0.0"
+    "@biomejs/biome": "^2.4.13",
+    "@mariozechner/pi-coding-agent": "^0.70.2",
+    "@mariozechner/pi-tui": "^0.70.2",
+    "typebox": "^1.1.33",
+    "typescript": "^6.0.3",
+    "vitest": "^4.1.5"
   },
   "scripts": {
     "test": "vitest run",
     "test:watch": "vitest",
     "typecheck": "tsc --noEmit",
-    "style": "biome check ./src ./test",
-    "style:fix": "biome check --write --unsafe --no-errors-on-unmatched ./src ./test",
+    "style": "biome check --error-on-warnings ./src ./test",
+    "style:fix": "biome check --write --unsafe --no-errors-on-unmatched --error-on-warnings ./src ./test",
     "format": "biome format --write --no-errors-on-unmatched ./src ./test",
-    "prepublishOnly": "npx tsc --noEmit && npx vitest run"
+    "prepublishOnly": "npx tsc --noEmit && npx vitest run",
+    "prepare": "bash scripts/setup.sh",
+    "cleanup": "bash scripts/cleanup.sh",
+    "dev": "bash scripts/dev.sh",
+    "logs": "bash scripts/logs.sh"
   }
 }

package/src/commands/handler.ts

@@ -25,7 +25,8 @@ type MutatingCommand = Extract<
       | typeof QUEST_ACTIONS.clear
       | typeof QUEST_ACTIONS.reorder
       | typeof QUEST_ACTIONS.reparent
-      | typeof QUEST_ACTIONS.revert;
+      | typeof QUEST_ACTIONS.undo
+      | typeof QUEST_ACTIONS.redo;
   }
 >["action"];
 
@@ -41,13 +42,13 @@ const commandActionBuilders: {
     id: p.id,
     descriptions: p.descriptions,
   }),
-  [QUEST_ACTIONS.toggle]: (p) => ({ type: QUEST_ACTIONS.toggle, id: p.id }),
+  [QUEST_ACTIONS.toggle]: (p) => ({ type: QUEST_ACTIONS.toggle, ids: p.ids }),
   [QUEST_ACTIONS.update]: (p) => ({
     type: QUEST_ACTIONS.update,
     id: p.id,
     description: p.description,
   }),
-  [QUEST_ACTIONS.delete]: (p) => ({ type: QUEST_ACTIONS.delete, id: p.id }),
+  [QUEST_ACTIONS.delete]: (p) => ({ type: QUEST_ACTIONS.delete, ids: p.ids }),
   [QUEST_ACTIONS.clear]: (p) => ({ type: QUEST_ACTIONS.clear, all: p.all }),
   [QUEST_ACTIONS.reorder]: (p) => ({
     type: QUEST_ACTIONS.reorder,
@@ -59,7 +60,8 @@ const commandActionBuilders: {
     id: p.id,
     parentId: p.parentId,
   }),
-  [QUEST_ACTIONS.revert]: () => ({ type: QUEST_ACTIONS.revert }),
+  [QUEST_ACTIONS.undo]: () => ({ type: QUEST_ACTIONS.undo }),
+  [QUEST_ACTIONS.redo]: () => ({ type: QUEST_ACTIONS.redo }),
 };
 
 export function openQuestList(
@@ -143,7 +145,8 @@ export function createQuestsHandler(
       case QUEST_ACTIONS.clear:
       case QUEST_ACTIONS.reorder:
       case QUEST_ACTIONS.reparent:
-      case QUEST_ACTIONS.revert: {
+      case QUEST_ACTIONS.undo:
+      case QUEST_ACTIONS.redo: {
         const builder = commandActionBuilders[parsed.action];
         const action = builder(parsed as never);
         const result = questLog.execute(action);
@@ -161,12 +164,13 @@ export function createQuestsHandler(
         lines.push("  add <description>           - Add a new top-level quest");
         lines.push("  add-step <id> <description> - Split a quest into a step");
         lines.push("  list               - List all quests");
-        lines.push("  toggle <id>        - Toggle quest completion");
-        lines.push("  delete <id>        - Delete a quest");
+        lines.push("  toggle <id> [moreIds...] - Toggle one or more quests");
+        lines.push("  delete <id> [moreIds...] - Delete one or more quests");
         lines.push("  update <id> <desc> - Update a quest description");
         lines.push("  reparent <id> [parentId] - Reparent a quest/step (omit parentId to promote)");
         lines.push("  reorder <id> <targetId> - Reorder a quest before the target quest");
-        lines.push("  revert             - Revert the last quest change");
+        lines.push("  undo               - Undo the last quest change");
+        lines.push("  redo               - Redo the last undone action");
         lines.push("  clear [all]        - Clear completed quests, or optionally all quests");
         lines.push("  version            - Show version");
         lines.push("  changelog          - Show changelog");

package/src/commands/parse-args.ts

@@ -5,12 +5,13 @@ export type ParsedArgs =
   | { action: typeof QUEST_ACTIONS.add; descriptions: string[] }
   | { action: typeof QUEST_ACTIONS.split; id: string; descriptions: string[] }
   | { action: typeof QUEST_ACTIONS.list }
-  | { action: typeof QUEST_ACTIONS.toggle; id: string }
+  | { action: typeof QUEST_ACTIONS.toggle; ids: string[] }
   | { action: typeof QUEST_ACTIONS.update; id: string; description: string }
-  | { action: typeof QUEST_ACTIONS.delete; id: string }
+  | { action: typeof QUEST_ACTIONS.delete; ids: string[] }
   | { action: typeof QUEST_ACTIONS.clear; all?: boolean }
   | { action: typeof QUEST_ACTIONS.reorder; id: string; targetId: string }
-  | { action: typeof QUEST_ACTIONS.revert }
+  | { action: typeof QUEST_ACTIONS.undo }
+  | { action: typeof QUEST_ACTIONS.redo }
   | { action: typeof QUEST_ACTIONS.reparent; id: string; parentId?: string }
   | { action: "help" }
   | { action: "version" }
@@ -40,13 +41,14 @@ export function parseQuestArgs(args: string, idLength = 2): ParsedArgs {
   // Quest actions without arguments
   if (command === QUEST_ACTIONS.list) return { action: QUEST_ACTIONS.list };
   if (command === QUEST_ACTIONS.clear) return parseClearArgs(rest);
-  if (command === QUEST_ACTIONS.revert) return { action: QUEST_ACTIONS.revert };
+  if (command === QUEST_ACTIONS.undo) return { action: QUEST_ACTIONS.undo };
+  if (command === QUEST_ACTIONS.redo) return { action: QUEST_ACTIONS.redo };
 
   // Quest actions with arguments
   if (command === QUEST_ACTIONS.add) return parseAddArgs(rest);
   if (command === "add-step") return parseAddStepArgs(rest, idLength);
-  if (command === QUEST_ACTIONS.toggle) return parseIdAction(QUEST_ACTIONS.toggle, rest, idLength);
-  if (command === QUEST_ACTIONS.delete) return parseIdAction(QUEST_ACTIONS.delete, rest, idLength);
+  if (command === QUEST_ACTIONS.toggle) return parseToggleArgs(rest, idLength);
+  if (command === QUEST_ACTIONS.delete) return parseDeleteArgs(rest, idLength);
   if (command === QUEST_ACTIONS.update) return parseUpdateArgs(rest, idLength);
   if (command === QUEST_ACTIONS.reorder) return parseReorderArgs(rest, idLength);
   if (command === QUEST_ACTIONS.reparent) return parseReparentArgs(rest, idLength);
@@ -70,16 +72,26 @@ function parseAddStepArgs(tokens: string[], idLength: number): ParsedArgs {
   return { action: QUEST_ACTIONS.split, id, descriptions: [description] };
 }
 
-function parseIdAction(
-  action: typeof QUEST_ACTIONS.toggle | typeof QUEST_ACTIONS.delete,
-  tokens: string[],
-  idLength: number,
-): ParsedArgs {
-  const id = tokens[0] ? tokens[0].toLowerCase() : "";
+function parseToggleArgs(tokens: string[], idLength: number): ParsedArgs {
   const pattern = new RegExp(`^[0-9a-f]{${idLength}}$`);
-  if (!pattern.test(id)) return { error: `Usage: /quests ${action} <id>` };
+  const ids = tokens.map((token) => token.toLowerCase());
+
+  if (ids.length === 0 || ids.some((id) => !pattern.test(id))) {
+    return { error: "Usage: /quests toggle <id> [moreIds...]" };
+  }
+
+  return { action: QUEST_ACTIONS.toggle, ids };
+}
+
+function parseDeleteArgs(tokens: string[], idLength: number): ParsedArgs {
+  const pattern = new RegExp(`^[0-9a-f]{${idLength}}$`);
+  const ids = tokens.map((token) => token.toLowerCase());
+
+  if (ids.length === 0 || ids.some((id) => !pattern.test(id))) {
+    return { error: "Usage: /quests delete <id> [moreIds...]" };
+  }
 
-  return { action, id };
+  return { action: QUEST_ACTIONS.delete, ids };
 }
 
 function parseUpdateArgs(tokens: string[], idLength: number): ParsedArgs {

package/src/index.ts

@@ -22,7 +22,7 @@ import { registerQuestTool } from "./tools/handler.js";
  * - Provide rollback support to restore a previous snapshot.
  */
 const NO_QUESTS_REMINDER =
-  "No active quests. Use the quest tool to track your work. Use action: 'skill' for usage documentation.";
+  "No active quests. Use the learn_quests tool to understand how to use quests and use quests to track this task. DO THIS NOW and acknowledge it.";
 
 const FAKE_DONE_REMINDER =
   "QUEST REMINDER: A quest has a completion marker appended to its description but is not toggled done. Use the list action to find it, then toggle it done. NEVER append completion markers via update.";
@@ -121,7 +121,11 @@ export default function (pi: ExtensionAPI): void {
   });
 
   pi.on("before_agent_start", async (event) => {
-    if (tracker.hasUsedQuestTool) return undefined;
+    if (questLog.getAll().length === 0)
+      return {
+        systemPrompt: `${event.systemPrompt}\n\nUse quests to track this task. DO THIS now and acknowledge it.`,
+      };
+
     return {
       systemPrompt: `# Quest Management\n${QUEST_PROMPT_GATE}${event.systemPrompt}\n\n## Quest Management\n${NO_QUESTS_REMINDER}`,
     };

package/src/prompts/skill.md

@@ -5,56 +5,55 @@ description: Quest management best practices for the pi-quests extension. Use wh
 
 # Quest Management
 
-## When to use quests
-
-Use the quest tool at the start of any non-trivial task. If the request involves multiple steps, files, tool calls, or minion delegation, track it with quests.
-
-## Capabilities
-
-| Action | Purpose |
-|--------|---------|
-| `add` | Create top-level quests |
-| `split` / `add_step` | Break a quest into steps under a parent |
-| `reparent` | Promote, demote, or move a quest/step via optional `parentId` |
-| `toggle` | Mark a quest or step done/undone |
-| `update` | Change a quest description |
-| `delete` | Remove a quest or step |
-| `clear` | Remove completed quests (or all with `all: true`) |
-| `reorder` | Change the priority order of top-level quests |
-| `revert` | Undo the most recent mutating action |
-| `list` | View all quests and steps |
-
-## Common patterns
-
-### Multi-step task workflow
-1. Add many top-level quests for the overall goal
-2. Split them into steps for each distinct deliverable
-3. Execute steps sequentially, toggling each done as you finish
-4. Toggle the parent quest done only after all steps are complete
-
-### Delegation workflow
-1. Add a quest for the delegated task
-2. Spawn the minion and assign the work
-3. Toggle the quest done when the minion returns successfully
+This skill provides guidelines and best practices for using the quest management system in the pi-quests extension. It covers the core actions available for managing quests, recommended workflows for common use cases, and important rules to follow for effective task tracking and progress management.
+
+## Action Reference
+
+| Action | Required Params | Behavior |
+|--------|----------------|----------|
+| `add` | `descriptions[]` | Create top-level quests. Batch multiple descriptions in one call. |
+| `split` | `id`, `descriptions[]` | Break a quest into steps under it. Alias: `add_step`. |
+| `list` | — | View all quests and steps. |
+| `toggle` | `id` | Flip done status. Blocked if parent has incomplete steps. |
+| `update` | `id`, `description` | Change description. |
+| `delete` | `id` | Remove a quest. Blocked if parent has incomplete steps. Cascade-deletes done steps of a done parent. |
+| `clear` | — | Remove completed quests. `all: true` removes everything. |
+| `reorder` | `id`, `targetId` | Move a top-level quest before `targetId`. Steps cannot be reordered. |
+| `reparent` | `id`, `parentId?` | Demote to step under `parentId`, or promote to top-level if `parentId` omitted. |
+| `undo` | — | Undo the most recent mutating action. One level only. |
+| `redo` | — | Redo the most recently undone action. |
+| `rules` / `skill` | — | Return this document. |
+
+## Workflows
+
+### Multi-step task
+1. `add` top-level quests for the overall goal
+2. `split` them into steps for each deliverable
+3. Execute steps sequentially, `toggle` each done as you finish
+4. `add` other top-level quests for other related work of this goal
+4. `toggle` the parent done only after all steps are complete
+
+### Delegation
+1. `add` a quest for the delegated task
+2. `spawn` the minion and assign the work
+3. `toggle` the quest done when the minion returns successfully
 
 ### Reparenting
 - Promote a step: `reparent <step-id>` (omit `parentId`)
 - Demote a quest: `reparent <quest-id> <parent-id>`
 - Move a step: `reparent <step-id> <new-parent-id>`
 
-## Rules
-
-- Steps cannot have nested steps. Only top-level quests can be parents.
-- A parent with incomplete steps cannot be toggled done or deleted.
-- Deleting a done parent cascade-deletes its done steps.
-- Revert only undoes the most recent mutating action.
-- Quest IDs are random hex strings shown in square brackets (e.g., `[01]`, `[a3f1]`).
-- Always use the hex ID for actions, never the positional number.
+### Cleanup
+- `clear` completed quests before adding unrelated work. Keeps the log focused.
 
-## Gotchas
+## Rules
 
-- Use toggle for done states. NEVER use `update` to append "DONE", "- DONE", or any completion marker to a quest description.
-- IDs are hex, not positional. The list shows `#1 [01] ...`. Use `01` in tool calls, not `1`.
-- Steps cannot be reordered independently. Use `reorder` on top-level quests only.
-- Clear completed quests before adding unrelated work. This keeps the log focused and reduces context noise.
-- Parent blocked?: If you cannot toggle a parent done, check that all its steps are toggled done first.
+- ALWAYS use quests for any work that involves multiple turns.
+- NEVER try to nest steps. Only top-level quests can be parents.
+- NEVER mark a parent as done if they have incomplete steps.
+- NEVER delete a quest with incomplete steps.
+- Use undo if you make a mistake to undo an action.
+- Use redo to replay an action you just undid. Redo is cleared when a new mutation happens.
+- ALWAYS use the hex ID, never the positional number when referring to quests in actions.
+- ALWAYS Use `toggle` for done states. NEVER use `update` to append "DONE" or any completion marker to a description.
+- If a parent toggle is blocked, check that all its steps are done first.