@allpepper/task-orchestrator-tui 1.2.1 → 2.0.0

package/src/ui/hooks/use-feature-kanban.ts

@@ -1,23 +1,19 @@
 import { useState, useEffect, useCallback, useMemo } from 'react';
 import { useAdapter } from '../context/adapter-context';
-import type { Feature, FeatureStatus } from '@allpepper/task-orchestrator';
+import type { Feature, Task } from '@allpepper/task-orchestrator';
 import type { BoardFeature, FeatureBoardColumn } from '../lib/types';
+import { isCompletedStatus } from '../lib/colors';
 
 /**
- * Feature-status columns for the feature-based Kanban board
+ * v2 Feature-status columns for the feature-based Kanban board
+ * Features: NEW → ACTIVE → READY_TO_PROD → CLOSED (+ WILL_NOT_IMPLEMENT)
  */
 export const FEATURE_KANBAN_STATUSES = [
-  { id: 'draft', title: 'Draft', status: 'DRAFT' },
-  { id: 'planning', title: 'Planning', status: 'PLANNING' },
-  { id: 'in-development', title: 'In Development', status: 'IN_DEVELOPMENT' },
-  { id: 'testing', title: 'Testing', status: 'TESTING' },
-  { id: 'validating', title: 'Validating', status: 'VALIDATING' },
-  { id: 'pending-review', title: 'Pending Review', status: 'PENDING_REVIEW' },
-  { id: 'blocked', title: 'Blocked', status: 'BLOCKED' },
-  { id: 'on-hold', title: 'On Hold', status: 'ON_HOLD' },
-  { id: 'deployed', title: 'Deployed', status: 'DEPLOYED' },
-  { id: 'completed', title: 'Completed', status: 'COMPLETED' },
-  { id: 'archived', title: 'Archived', status: 'ARCHIVED' },
+  { id: 'new', title: 'New', status: 'NEW' },
+  { id: 'active', title: 'Active', status: 'ACTIVE' },
+  { id: 'ready-to-prod', title: 'Ready to Prod', status: 'READY_TO_PROD' },
+  { id: 'closed', title: 'Closed', status: 'CLOSED' },
+  { id: 'will-not-implement', title: 'Will Not Implement', status: 'WILL_NOT_IMPLEMENT' },
 ] as const;
 
 interface UseFeatureKanbanReturn {
@@ -30,14 +26,12 @@ interface UseFeatureKanbanReturn {
 
 /**
  * Hook for managing the feature-based Kanban board.
- *
- * Fetches features and tasks for a project, groups features by their status
- * into columns, and nests tasks within each feature card.
+ * In v2, features move through the pipeline via advance/revert.
  */
 export function useFeatureKanban(projectId: string): UseFeatureKanbanReturn {
   const { adapter } = useAdapter();
   const [features, setFeatures] = useState<Feature[]>([]);
-  const [tasksByFeature, setTasksByFeature] = useState<Map<string, import('@allpepper/task-orchestrator/src/domain/types').Task[]>>(new Map());
+  const [tasksByFeature, setTasksByFeature] = useState<Map<string, Task[]>>(new Map());
   const [loading, setLoading] = useState(true);
   const [error, setError] = useState<string | null>(null);
   const [refreshTrigger, setRefreshTrigger] = useState(0);
@@ -69,8 +63,7 @@ export function useFeatureKanban(projectId: string): UseFeatureKanbanReturn {
 
     setFeatures(featuresResult.data);
 
-    // Group tasks by featureId
-    const grouped = new Map<string, import('@allpepper/task-orchestrator/src/domain/types').Task[]>();
+    const grouped = new Map<string, Task[]>();
     for (const task of tasksResult.data) {
       if (task.featureId) {
         const list = grouped.get(task.featureId) || [];
@@ -97,7 +90,7 @@ export function useFeatureKanban(projectId: string): UseFeatureKanbanReturn {
             tasks,
             taskCounts: {
               total: tasks.length,
-              completed: tasks.filter((t) => t.status === 'COMPLETED' || t.status === 'DEPLOYED').length,
+              completed: tasks.filter((t) => isCompletedStatus(t.status)).length,
             },
           };
         });
@@ -113,7 +106,6 @@ export function useFeatureKanban(projectId: string): UseFeatureKanbanReturn {
 
   const moveFeature = useCallback(
     async (featureId: string, newStatus: string): Promise<boolean> => {
-      // Find feature in current data
       const feature = features.find((f) => f.id === featureId);
       if (!feature) {
         refresh();
@@ -121,20 +113,43 @@ export function useFeatureKanban(projectId: string): UseFeatureKanbanReturn {
       }
 
       try {
-        const result = await adapter.setFeatureStatus(
-          featureId,
-          newStatus as FeatureStatus,
-          feature.version
-        );
+        // Determine direction based on pipeline position
+        const currentIdx = FEATURE_KANBAN_STATUSES.findIndex(s => s.status === feature.status);
+        const targetIdx = FEATURE_KANBAN_STATUSES.findIndex(s => s.status === newStatus);
 
-        if (result.success) {
-          refresh();
-          return true;
-        } else {
+        if (currentIdx === -1 || targetIdx === -1) {
           refresh();
           return false;
         }
+
+        // Handle terminate (WILL_NOT_IMPLEMENT)
+        if (newStatus === 'WILL_NOT_IMPLEMENT') {
+          const result = await adapter.terminate('feature', featureId, feature.version);
+          refresh();
+          return result.success;
+        }
+
+        // Step through the pipeline
+        let currentVersion = feature.version;
+        const direction = targetIdx > currentIdx ? 'advance' : 'revert';
+        const steps = Math.abs(targetIdx - currentIdx);
+
+        for (let i = 0; i < steps; i++) {
+          const result = direction === 'advance'
+            ? await adapter.advance('feature', featureId, currentVersion)
+            : await adapter.revert('feature', featureId, currentVersion);
+
+          if (!result.success) {
+            refresh();
+            return false;
+          }
+          currentVersion = result.data.entity.version;
+        }
+
+        refresh();
+        return true;
       } catch (_err) {
+        refresh();
         return false;
       }
     },

package/src/ui/hooks/use-kanban.ts

@@ -5,14 +5,14 @@ import type { BoardColumn } from '../lib/types';
 import { useBoardData } from './use-data';
 
 /**
- * Standard Kanban column definitions
+ * v2 Kanban column definitions (pipeline states)
  */
 const KANBAN_STATUSES = [
-  { id: 'pending', title: 'Pending', status: 'PENDING' },
-  { id: 'in-progress', title: 'In Progress', status: 'IN_PROGRESS' },
-  { id: 'in-review', title: 'In Review', status: 'IN_REVIEW' },
-  { id: 'blocked', title: 'Blocked', status: 'BLOCKED' },
-  { id: 'completed', title: 'Completed', status: 'COMPLETED' },
+  { id: 'new', title: 'New', status: 'NEW' },
+  { id: 'active', title: 'Active', status: 'ACTIVE' },
+  { id: 'to-be-tested', title: 'To Be Tested', status: 'TO_BE_TESTED' },
+  { id: 'ready-to-prod', title: 'Ready to Prod', status: 'READY_TO_PROD' },
+  { id: 'closed', title: 'Closed', status: 'CLOSED' },
 ] as const;
 
 interface UseKanbanReturn {
@@ -24,13 +24,9 @@ interface UseKanbanReturn {
 }
 
 /**
- * Hook for managing Kanban board state
- *
- * Fetches tasks for a project and organizes them into Kanban columns by status.
- * Provides functionality to move tasks between columns with optimistic updates.
- *
- * @param projectId - The project ID to fetch tasks for
- * @returns Kanban board state and operations
+ * Hook for managing Kanban board state.
+ * In v2, tasks move through the pipeline via advance/revert.
+ * The moveTask function uses advance for forward moves and revert for backward moves.
  */
 export function useKanban(projectId: string): UseKanbanReturn {
   const { adapter } = useAdapter();
@@ -46,15 +42,10 @@ export function useKanban(projectId: string): UseKanbanReturn {
   ), [columnsByStatus]);
 
   /**
-   * Move a task to a new status
-   *
-   * @param taskId - ID of the task to move
-   * @param newStatus - New status to assign
-   * @returns true if successful, false otherwise
+   * Move a task to a new status via advance/revert pipeline operations
    */
   const moveTask = useCallback(
     async (taskId: string, newStatus: string): Promise<boolean> => {
-      // Find the task in current columns
       let task: Task | undefined;
       for (const column of columns) {
         task = column.tasks.find((t) => t.id === taskId);
@@ -62,24 +53,41 @@ export function useKanban(projectId: string): UseKanbanReturn {
       }
 
       if (!task) {
-        // Board may be stale; ask the caller to refresh and retry.
         refresh();
         return false;
       }
 
       try {
-        // Call adapter to update task status
-        const result = await adapter.setTaskStatus(taskId, newStatus as any, task.version);
+        // Determine direction based on pipeline position
+        const currentIdx = KANBAN_STATUSES.findIndex(s => s.status === task!.status);
+        const targetIdx = KANBAN_STATUSES.findIndex(s => s.status === newStatus);
 
-        if (result.success) {
-          // Refresh the board on success
-          refresh();
-          return true;
-        } else {
+        if (currentIdx === -1 || targetIdx === -1) {
           refresh();
           return false;
         }
+
+        // Step through the pipeline one step at a time
+        let currentVersion = task.version;
+        const direction = targetIdx > currentIdx ? 'advance' : 'revert';
+        const steps = Math.abs(targetIdx - currentIdx);
+
+        for (let i = 0; i < steps; i++) {
+          const result = direction === 'advance'
+            ? await adapter.advance('task', taskId, currentVersion)
+            : await adapter.revert('task', taskId, currentVersion);
+
+          if (!result.success) {
+            refresh();
+            return false;
+          }
+          currentVersion = result.data.entity.version;
+        }
+
+        refresh();
+        return true;
       } catch (_err) {
+        refresh();
         return false;
       }
     },

package/src/ui/index.ts

@@ -8,6 +8,7 @@
 export { darkTheme } from './themes/dark';
 export { lightTheme } from './themes/light';
 export type { Theme, TaskCounts, StatusKey } from './themes/types';
+export type { WorkflowState, TransitionResult } from './adapters/types';
 
 // Lib - Types
 export {

package/src/ui/lib/colors.ts

@@ -1,9 +1,16 @@
+/**
+ * Color and Status Utilities
+ *
+ * Theme-aware color helpers for status badges, priority indicators, etc.
+ * Updated for v2 pipeline states.
+ */
+
 import type { Theme, StatusKey } from '../themes/types';
 import type { Priority } from '@allpepper/task-orchestrator';
 
 /**
- * Get the color for a status value
- * Falls back to muted color if status not found
+ * Get the color for a status value.
+ * Falls back to muted color if status not found.
  */
 export function getStatusColor(status: string, theme: Theme): string {
   const color = theme.colors.status[status as StatusKey];
@@ -19,9 +26,6 @@ export function getPriorityColor(priority: Priority, theme: Theme): string {
 
 /**
  * Get priority dots visual indicator
- * HIGH: ●●● (3 filled)
- * MEDIUM: ●●○ (2 filled)
- * LOW: ●○○ (1 filled)
  */
 export function getPriorityDots(priority: Priority): string {
   switch (priority) {
@@ -47,33 +51,32 @@ export function getSemanticColor(
 }
 
 /**
- * Determine if a status represents an "active" state
+ * Check if a status represents an active/in-progress state
  */
 export function isActiveStatus(status: string): boolean {
-  const activeStatuses = [
-    'IN_PROGRESS',
-    'IN_DEVELOPMENT',
-    'IN_REVIEW',
-    'TESTING',
-    'VALIDATING',
-    'INVESTIGATING',
-    'READY_FOR_QA',
-  ];
-  return activeStatuses.includes(status);
+  return status === 'ACTIVE' || status === 'TO_BE_TESTED' || status === 'READY_TO_PROD';
 }
 
 /**
- * Determine if a status represents a "blocked" state
+ * Check if a status represents a terminal/completed state
  */
-export function isBlockedStatus(status: string): boolean {
-  const blockedStatuses = ['BLOCKED', 'ON_HOLD', 'CHANGES_REQUESTED'];
-  return blockedStatuses.includes(status);
+export function isCompletedStatus(status: string): boolean {
+  return status === 'CLOSED' || status === 'WILL_NOT_IMPLEMENT';
 }
 
 /**
- * Determine if a status represents a "completed" state
+ * Check if a status is the initial state
  */
-export function isCompletedStatus(status: string): boolean {
-  const completedStatuses = ['COMPLETED', 'DEPLOYED', 'ARCHIVED', 'CANCELLED'];
-  return completedStatuses.includes(status);
+export function isNewStatus(status: string): boolean {
+  return status === 'NEW';
+}
+
+/**
+ * In v2, blocking is field-based (blockedBy array), not a status.
+ * This function always returns false since no status is "blocked".
+ * Use the blockedBy field on the entity instead.
+ * @deprecated Use entity.blockedBy.length > 0 instead
+ */
+export function isBlockedStatus(_status: string): boolean {
+  return false;
 }

package/src/ui/lib/markdown.tsx

@@ -64,7 +64,7 @@ function MarkdownLine({ line }: MarkdownLineProps) {
 
   // Header (## Header)
   const headerMatch = line.match(/^(#{1,6})\s+(.*)$/);
-  if (headerMatch) {
+  if (headerMatch && headerMatch[1] && headerMatch[2] !== undefined) {
     const level = headerMatch[1].length;
     const text = headerMatch[2];
     return (
@@ -78,7 +78,7 @@ function MarkdownLine({ line }: MarkdownLineProps) {
 
   // Bullet point (- item or * item)
   const bulletMatch = line.match(/^(\s*)([-*])\s+(.*)$/);
-  if (bulletMatch) {
+  if (bulletMatch && bulletMatch[1] !== undefined && bulletMatch[3] !== undefined) {
     const indent = bulletMatch[1].length;
     const content = bulletMatch[3];
     return (
@@ -114,7 +114,7 @@ function InlineMarkdown({ text }: InlineMarkdownProps) {
   while (remaining.length > 0) {
     // Check for inline code `code`
     const codeMatch = remaining.match(/^(.*?)`([^`]+)`(.*)$/);
-    if (codeMatch) {
+    if (codeMatch && codeMatch[2] !== undefined && codeMatch[3] !== undefined) {
       if (codeMatch[1]) {
         parts.push(<BoldMarkdown key={key++} text={codeMatch[1]} />);
       }
@@ -129,7 +129,7 @@ function InlineMarkdown({ text }: InlineMarkdownProps) {
 
     // Check for bold **text**
     const boldMatch = remaining.match(/^(.*?)\*\*([^*]+)\*\*(.*)$/);
-    if (boldMatch) {
+    if (boldMatch && boldMatch[2] !== undefined && boldMatch[3] !== undefined) {
       if (boldMatch[1]) {
         parts.push(<Text key={key++}>{boldMatch[1]}</Text>);
       }
@@ -160,7 +160,7 @@ function BoldMarkdown({ text }: { text: string }) {
 
   while (remaining.length > 0) {
     const boldMatch = remaining.match(/^(.*?)\*\*([^*]+)\*\*(.*)$/);
-    if (boldMatch) {
+    if (boldMatch && boldMatch[2] !== undefined && boldMatch[3] !== undefined) {
       if (boldMatch[1]) {
         parts.push(<Text key={key++}>{boldMatch[1]}</Text>);
       }

package/src/ui/lib/types.ts

@@ -113,7 +113,6 @@ export interface ProjectOverview {
     id: string;
     name: string;
     summary: string;
-    status: string;
   };
   taskCounts: {
     total: number;

package/src/ui/themes/dark.ts

@@ -10,37 +10,19 @@ export const darkTheme: Theme = {
     muted: '#6b6b8d',
     border: '#3d3d5c',
 
-    // Status colors (unique keys only)
+    // Status colors (v2 pipeline states)
     status: {
-      // Project/Feature/Task shared
-      PLANNING: '#a0a0c0',
-      IN_DEVELOPMENT: '#4da6ff',
-      ON_HOLD: '#cc9966',
-      CANCELLED: '#808080',
-      COMPLETED: '#4ade80',
-      ARCHIVED: '#555555',
-
-      // Feature/Task shared
-      TESTING: '#66cccc',
-      BLOCKED: '#ff6666',
-      DEPLOYED: '#34d399',
-
-      // Feature only
-      DRAFT: '#6b6b8d',
-      VALIDATING: '#66cc99',
-      PENDING_REVIEW: '#b366ff',
-
-      // Task only
-      BACKLOG: '#6b6b8d',
-      PENDING: '#a0a0c0',
-      IN_PROGRESS: '#4da6ff',
-      IN_REVIEW: '#b366ff',
-      CHANGES_REQUESTED: '#ff9966',
-      READY_FOR_QA: '#66cc99',
-      INVESTIGATING: '#ffcc66',
-      DEFERRED: '#999999',
+      NEW: '#a0a0c0',
+      ACTIVE: '#4da6ff',
+      TO_BE_TESTED: '#66cccc',
+      READY_TO_PROD: '#66cc99',
+      CLOSED: '#4ade80',
+      WILL_NOT_IMPLEMENT: '#808080',
     },
 
+    // Blocked overlay color
+    blocked: '#ff6666',
+
     // Priority colors
     priority: {
       [Priority.HIGH]: '#ff6b6b',

package/src/ui/themes/light.ts

@@ -7,57 +7,39 @@ export const lightTheme: Theme = {
     // Base colors
     background: '#ffffff',
     foreground: '#1a1a2e',
-    muted: '#6b7280',
-    border: '#e5e7eb',
+    muted: '#9090a0',
+    border: '#d0d0e0',
 
-    // Status colors (unique keys only)
+    // Status colors (v2 pipeline states)
     status: {
-      // Project/Feature/Task shared
-      PLANNING: '#6b7280',
-      IN_DEVELOPMENT: '#3b82f6',
-      ON_HOLD: '#d97706',
-      CANCELLED: '#6b7280',
-      COMPLETED: '#16a34a',
-      ARCHIVED: '#9ca3af',
-
-      // Feature/Task shared
-      TESTING: '#06b6d4',
-      BLOCKED: '#ef4444',
-      DEPLOYED: '#22c55e',
-
-      // Feature only
-      DRAFT: '#9ca3af',
-      VALIDATING: '#14b8a6',
-      PENDING_REVIEW: '#8b5cf6',
-
-      // Task only
-      BACKLOG: '#9ca3af',
-      PENDING: '#6b7280',
-      IN_PROGRESS: '#3b82f6',
-      IN_REVIEW: '#8b5cf6',
-      CHANGES_REQUESTED: '#f97316',
-      READY_FOR_QA: '#14b8a6',
-      INVESTIGATING: '#eab308',
-      DEFERRED: '#9ca3af',
+      NEW: '#7070a0',
+      ACTIVE: '#2563eb',
+      TO_BE_TESTED: '#0891b2',
+      READY_TO_PROD: '#059669',
+      CLOSED: '#16a34a',
+      WILL_NOT_IMPLEMENT: '#6b7280',
     },
 
+    // Blocked overlay color
+    blocked: '#dc2626',
+
     // Priority colors
     priority: {
       [Priority.HIGH]: '#dc2626',
-      [Priority.MEDIUM]: '#ca8a04',
+      [Priority.MEDIUM]: '#d97706',
       [Priority.LOW]: '#16a34a',
     },
 
     // Semantic colors
     accent: '#2563eb',
     success: '#16a34a',
-    warning: '#ca8a04',
+    warning: '#d97706',
     error: '#dc2626',
     danger: '#dc2626',
     info: '#0891b2',
 
     // Interactive colors
-    selection: '#dbeafe',
-    highlight: '#bfdbfe',
+    selection: '#e0e0f0',
+    highlight: '#c0c0e0',
   },
 };

package/src/ui/themes/types.ts

@@ -1,33 +1,18 @@
 import type { Priority } from '@allpepper/task-orchestrator';
 
 /**
- * All possible status values across Project, Feature, and Task
- * Using string literal union to avoid duplicate key issues
+ * v2 pipeline status keys
+ * Task: NEW, ACTIVE, TO_BE_TESTED, READY_TO_PROD, CLOSED, WILL_NOT_IMPLEMENT
+ * Feature: NEW, ACTIVE, READY_TO_PROD, CLOSED, WILL_NOT_IMPLEMENT
+ * Projects: stateless (no status)
  */
 export type StatusKey =
-  // Project statuses
-  | 'PLANNING'
-  | 'IN_DEVELOPMENT'
-  | 'ON_HOLD'
-  | 'CANCELLED'
-  | 'COMPLETED'
-  | 'ARCHIVED'
-  // Feature-only statuses
-  | 'DRAFT'
-  | 'TESTING'
-  | 'VALIDATING'
-  | 'PENDING_REVIEW'
-  | 'BLOCKED'
-  | 'DEPLOYED'
-  // Task-only statuses
-  | 'BACKLOG'
-  | 'PENDING'
-  | 'IN_PROGRESS'
-  | 'IN_REVIEW'
-  | 'CHANGES_REQUESTED'
-  | 'READY_FOR_QA'
-  | 'INVESTIGATING'
-  | 'DEFERRED';
+  | 'NEW'
+  | 'ACTIVE'
+  | 'TO_BE_TESTED'
+  | 'READY_TO_PROD'
+  | 'CLOSED'
+  | 'WILL_NOT_IMPLEMENT';
 
 /**
  * Theme interface for the Task Orchestrator UI
@@ -42,9 +27,12 @@ export interface Theme {
     muted: string;
     border: string;
 
-    // Status colors - maps all unique status values
+    // Status colors - maps v2 pipeline states
     status: Record<StatusKey, string>;
 
+    // Blocked overlay color (for field-based blocking indicator)
+    blocked: string;
+
     // Priority colors
     priority: Record<Priority, string>;