@pi-unipi/workflow 0.1.5 → 0.1.7

package/commands.ts

@@ -6,7 +6,7 @@
  */
 
 import type { ExtensionAPI } from "@mariozechner/pi-coding-agent";
-import { readFileSync, readdirSync, existsSync } from "fs";
+import { readFileSync, readdirSync, existsSync, statSync } from "fs";
 import { join, basename } from "path";
 import { UNIPI_PREFIX, WORKFLOW_COMMANDS, getToolsForCommand, getSandboxLevel, type SandboxLevel } from "@pi-unipi/core";
 
@@ -41,13 +41,17 @@ function suggestSpecFiles(prefix: string): { value: string; label: string; descr
   if (!existsSync(specsDir)) return [];
 
   try {
-    const files = readdirSync(specsDir).filter((f) => f.endsWith(".md"));
+    const search = prefix?.trim().split(/\s+/).pop() ?? "";
+    const files = readdirSync(specsDir)
+      .filter((f) => f.endsWith(".md"))
+      .map((f) => ({ name: f, time: statSync(join(specsDir, f)).mtimeMs }))
+      .sort((a, b) => b.time - a.time);
     return files
-      .filter((f) => !prefix || f.includes(prefix))
+      .filter((f) => !search || f.name.includes(search))
       .map((f) => ({
-        value: `specs:${f}`,
-        label: basename(f, ".md"),
-        description: `Spec: ${f}`,
+        value: `specs:${f.name}`,
+        label: basename(f.name, ".md"),
+        description: `Spec: ${f.name}`,
       }));
   } catch {
     return [];
@@ -62,13 +66,17 @@ function suggestPlanFiles(prefix: string): { value: string; label: string; descr
   if (!existsSync(plansDir)) return [];
 
   try {
-    const files = readdirSync(plansDir).filter((f) => f.endsWith(".md"));
+    const search = prefix?.trim().split(/\s+/).pop() ?? "";
+    const files = readdirSync(plansDir)
+      .filter((f) => f.endsWith(".md"))
+      .map((f) => ({ name: f, time: statSync(join(plansDir, f)).mtimeMs }))
+      .sort((a, b) => b.time - a.time);
     return files
-      .filter((f) => !prefix || f.includes(prefix))
+      .filter((f) => !search || f.name.includes(search))
       .map((f) => ({
-        value: `plan:${f}`,
-        label: basename(f, ".md"),
-        description: `Plan: ${f}`,
+        value: `plan:${f.name}`,
+        label: basename(f.name, ".md"),
+        description: `Plan: ${f.name}`,
       }));
   } catch {
     return [];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-unipi/workflow",
-  "version": "0.1.5",
+  "version": "0.1.7",
   "description": "Structured development workflow commands for Pi coding agent",
   "type": "module",
   "license": "MIT",

package/skills/plan/SKILL.md

@@ -111,6 +111,20 @@ specs:
 - **Steps** are bite-sized — agent can follow without guessing
 - Order tasks by dependency (foundational work first)
 
+### Task Status Lifecycle
+
+Tasks use prefixes to track progress:
+
+| Status | Meaning |
+|--------|----------|
+| `unstarted:` | Not started |
+| `in-progress:` | Being worked on |
+| `completed:` | Done and verified |
+| `failed:` | Attempted but failed, needs investigation |
+| `awaiting_user:` | Needs user action (test, approve, provide input) |
+| `blocked:` | Waiting on dependency or external factor |
+| `skipped:` | Intentionally not doing (deferred, out of scope) |
+
 ---
 
 ## Phase 4: Update Brainstorm Checkboxes

package/skills/review-work/SKILL.md

@@ -46,9 +46,13 @@ For each task in plan:
 1. Read acceptance criteria
 2. Verify against actual implementation
 3. Determine status:
-   - **Done** — all criteria met
+   - **Done** — all criteria met → `completed:`
    - **Partially Done X/Y** — some steps complete, others not
-   - **Unstarted** — nothing done
+   - **Unstarted** — nothing done → `unstarted:`
+   - **Failed** — attempted but broken → `failed:`
+   - **Awaiting User** — needs user action → `awaiting_user:`
+   - **Blocked** — waiting on dependency → `blocked:`
+   - **Skipped** — intentionally not done → `skipped:`
 
 If `string(greedy)` scope provided, only check matching tasks.
 

package/skills/work/SKILL.md

@@ -59,7 +59,7 @@ If args not provided, ask user interactively:
 1. Read plan file(s)
 2. Review critically — identify questions or concerns
 3. If concerns: raise with user before starting
-4. Identify task status: `unstarted:` (pending), `in-progress:` (started), `completed:` (done)
+4. Identify task status: `unstarted:` (pending), `in-progress:` (started), `completed:` (done), `failed:` (needs investigation), `awaiting_user:` (needs user action), `blocked:` (waiting on dependency), `skipped:` (deferred)
 
 **Exit:** Plan reviewed, ready to execute.
 
@@ -67,14 +67,36 @@ If args not provided, ask user interactively:
 
 ## Phase 3: Execute Tasks
 
-For each `unstarted:` task in order:
+For each task in order, skip `completed:`, `skipped:`, and `awaiting_user:` tasks:
 
+### If `unstarted:`
 1. Change `unstarted:` to `in-progress:` in plan
 2. Follow each step exactly (plan has bite-sized steps)
 3. Run verifications as specified in acceptance criteria
 4. Change `in-progress:` to `completed:` when complete
 5. Update plan file with progress
 
+### If `in-progress:`
+1. Continue from where it left off
+2. Follow remaining steps
+3. Change to `completed:` when done
+
+### If `failed:`
+1. Read failure notes
+2. Investigate root cause
+3. Fix and re-verify
+4. Change to `completed:` when fixed, or keep `failed:` if still broken
+
+### If `awaiting_user:`
+1. Remind user what's needed
+2. Wait for user response
+3. Resume when user provides input
+
+### If `blocked:`
+1. Check if blocker is resolved
+2. If resolved → change to `unstarted:` and continue
+3. If still blocked → skip and move to next task
+
 ### When to Stop and Ask
 
 **STOP immediately when:**