feat(tasks): filter/sort controls and bulk operations in list view

2026-07-29 19:24:18 +00:00
parent 28d01f3839
commit 1cb932f0f6
2 changed files with 537 additions and 101 deletions
@@ -21,6 +21,10 @@ const bulkUpdateSchema = z.object({
  }),
 });

+const bulkDeleteSchema = z.object({
+  ids: z.array(z.string().uuid()).min(1).max(200),
+});
+
 type RouteContext = { params: Promise<{ domainId: string }> };

 // POST /api/domains/[domainId]/tasks/bulk — Bulk update tasks (order, status)
@@ -77,3 +81,51 @@ export const POST = withAuth<RouteContext>(async (request: NextRequest, user, co
    return createErrorResponse('INTERNAL_ERROR', 'Failed to bulk update tasks', 500);
  }
 });
+
+// DELETE /api/domains/[domainId]/tasks/bulk — Bulk soft-delete tasks
+export const DELETE = withAuth<RouteContext>(async (request: NextRequest, user, context) => {
+  const { domainId } = await context!.params;
+  await requireWorkspaceAccess(domainId);
+
+  try {
+    const body = await request.json();
+    const data = bulkDeleteSchema.parse(body);
+
+    // Verify all tasks belong to this domain
+    const existingTasks = await db.select({ id: tasks.id, title: tasks.title })
+      .from(tasks)
+      .where(and(inArray(tasks.id, data.ids), eq(tasks.domainId, domainId), isNull(tasks.deletedAt)));
+
+    if (existingTasks.length !== data.ids.length) {
+      return createErrorResponse('NOT_FOUND', 'One or more tasks not found', 404);
+    }
+
+    // Soft delete
+    await db.update(tasks)
+      .set({ deletedAt: new Date(), updatedAt: new Date() })
+      .where(inArray(tasks.id, data.ids));
+
+    // Record activity for each task
+    for (const task of existingTasks) {
+      await recordActivity({
+        actor: user.name,
+        action: 'bulk_deleted',
+        entityType: 'task',
+        entityId: task.id,
+        changes: { title: task.title },
+        workspaceId: domainId,
+      });
+    }
+
+    return NextResponse.json({ deleted: existingTasks.length });
+  } catch (error) {
+    if (error instanceof z.ZodError) {
+      return createErrorResponse('VALIDATION_ERROR', 'Invalid input', 400, error.issues);
+    }
+    if (error instanceof ApiError) {
+      return createErrorResponse(error.code, error.message, error.status);
+    }
+    console.error('[tasks bulk DELETE] error:', error);
+    return createErrorResponse('INTERNAL_ERROR', 'Failed to bulk delete tasks', 500);
+  }
+});
@@ -1,6 +1,7 @@
 'use client';

-import { useEffect, useState, useCallback } from 'react';
+import { useEffect, useState, useCallback, useMemo } from 'react';
+import { useRouter, useSearchParams, usePathname } from 'next/navigation';
 import {
  Table,
  TableBody,
@@ -12,8 +13,19 @@ import {
 import { Badge } from '@/components/ui/badge';
 import { Button } from '@/components/ui/button';
 import { Checkbox } from '@/components/ui/checkbox';
+import { Input } from '@/components/ui/input';
 import { ScrollArea, ScrollBar } from '@/components/ui/scroll-area';
-import { Calendar, MoreHorizontal, Pencil, Trash2 } from 'lucide-react';
+import {
+  Calendar,
+  MoreHorizontal,
+  Pencil,
+  Trash2,
+  ArrowUpDown,
+  ArrowUp,
+  ArrowDown,
+  Search,
+  X,
+} from 'lucide-react';
 import { toast } from 'sonner';
 import { TaskDetailPanel } from './task-detail-panel';
 import {
@@ -32,6 +44,18 @@ import {
  AlertDialogHeader,
  AlertDialogTitle,
 } from '@/components/ui/alert-dialog';
+import {
+  Select,
+  SelectContent,
+  SelectItem,
+  SelectTrigger,
+  SelectValue,
+} from '@/components/ui/select';
+import {
+  Popover,
+  PopoverContent,
+  PopoverTrigger,
+} from '@/components/ui/popover';
 import { useRealtimeContext } from '@/components/realtime-provider';

 interface Task {
@@ -54,6 +78,89 @@ const priorityColors: Record<string, string> = {
  low: 'secondary',
 };

+const statusOptions = ['todo', 'in_progress', 'done', 'cancelled'] as const;
+const priorityOptions = ['low', 'medium', 'high', 'urgent'] as const;
+
+const sortableColumns = [
+  { key: 'title', label: 'Task' },
+  { key: 'status', label: 'Status' },
+  { key: 'priority', label: 'Priority' },
+  { key: 'due_date', label: 'Due Date' },
+] as const;
+
+function useDebounce<T>(value: T, delay: number): T {
+  const [debounced, setDebounced] = useState(value);
+  useEffect(() => {
+    const timer = setTimeout(() => setDebounced(value), delay);
+    return () => clearTimeout(timer);
+  }, [value, delay]);
+  return debounced;
+}
+
+function MultiSelectFilter({
+  label,
+  options,
+  selected,
+  onChange,
+}: {
+  label: string;
+  options: readonly string[];
+  selected: string[];
+  onChange: (values: string[]) => void;
+}) {
+  const [open, setOpen] = useState(false);
+
+  return (
+    <Popover open={open} onOpenChange={setOpen}>
+      <PopoverTrigger asChild>
+        <Button variant="outline" size="sm" className="h-9">
+          {label}
+          {selected.length > 0 && (
+            <span className="ml-1 rounded bg-primary/10 px-1.5 text-xs font-medium">
+              {selected.length}
+            </span>
+          )}
+        </Button>
+      </PopoverTrigger>
+      <PopoverContent className="w-48 p-2" align="start">
+        <div className="space-y-1">
+          {options.map((opt) => {
+            const checked = selected.includes(opt);
+            return (
+              <label
+                key={opt}
+                className="flex cursor-pointer items-center gap-2 rounded px-2 py-1.5 text-sm hover:bg-accent"
+              >
+                <Checkbox
+                  checked={checked}
+                  onCheckedChange={() => {
+                    onChange(
+                      checked
+                        ? selected.filter((s) => s !== opt)
+                        : [...selected, opt]
+                    );
+                  }}
+                />
+                <span className="capitalize">{opt.replace('_', ' ')}</span>
+              </label>
+            );
+          })}
+        </div>
+        {selected.length > 0 && (
+          <Button
+            variant="ghost"
+            size="sm"
+            className="mt-2 w-full text-xs"
+            onClick={() => onChange([])}
+          >
+            Clear
+          </Button>
+        )}
+      </PopoverContent>
+    </Popover>
+  );
+}
+
 export function TasksListView({
  domainId,
  onRefresh,
@@ -61,6 +168,10 @@ export function TasksListView({
  domainId: string;
  onRefresh?: () => void;
 }) {
+  const router = useRouter();
+  const pathname = usePathname();
+  const searchParams = useSearchParams();
+
  const [tasks, setTasks] = useState<Task[]>([]);
  const [loading, setLoading] = useState(true);
  const [selectedTask, setSelectedTask] = useState<Task | null>(null);
@@ -69,10 +180,59 @@ export function TasksListView({
  const [domainMap, setDomainMap] = useState<Map<string, string>>(new Map());
  const { subscribe } = useRealtimeContext();

+  // Filter/sort state from URL params
+  const statusFilter = searchParams.get('status')?.split(',').filter(Boolean) || [];
+  const priorityFilter = searchParams.get('priority')?.split(',').filter(Boolean) || [];
+  const domainFilter = searchParams.get('domain') || '';
+  const searchQuery = searchParams.get('search') || '';
+  const sort = searchParams.get('sort') || 'order';
+  const order = searchParams.get('order') || 'asc';
+
+  // Local search input (debounced)
+  const [searchInput, setSearchInput] = useState(searchQuery);
+  const debouncedSearch = useDebounce(searchInput, 300);
+
+  // Bulk selection
+  const [selectedIds, setSelectedIds] = useState<Set<string>>(new Set());
+  const [bulkStatusOpen, setBulkStatusOpen] = useState(false);
+  const [bulkPriorityOpen, setBulkPriorityOpen] = useState(false);
+  const [bulkDeleting, setBulkDeleting] = useState(false);
+
+  // Sync debounced search to URL
+  useEffect(() => {
+    if (debouncedSearch !== searchQuery) {
+      const params = new URLSearchParams(searchParams.toString());
+      if (debouncedSearch) {
+        params.set('search', debouncedSearch);
+      } else {
+        params.delete('search');
+      }
+      router.replace(`${pathname}?${params.toString()}`, { scroll: false });
+    }
+  }, [debouncedSearch, searchQuery, searchParams, pathname, router]);
+
+  const updateURLParam = useCallback(
+    (key: string, value: string | null) => {
+      const params = new URLSearchParams(searchParams.toString());
+      if (value) {
+        params.set(key, value);
+      } else {
+        params.delete(key);
+      }
+      router.replace(`${pathname}?${params.toString()}`, { scroll: false });
+    },
+    [searchParams, pathname, router]
+  );
+
  const fetchTasks = useCallback(async () => {
    if (!domainId) return;
    try {
-      const response = await fetch(`/api/domains/${domainId}/tasks?sort=order&limit=200`);
+      const params = new URLSearchParams({ sort, order, limit: '200' });
+      if (statusFilter.length > 0) params.set('status', statusFilter.join(','));
+      if (priorityFilter.length > 0) params.set('priority', priorityFilter.join(','));
+      if (searchQuery) params.set('search', searchQuery);
+
+      const response = await fetch(`/api/domains/${domainId}/tasks?${params.toString()}`);
      if (!response.ok) throw new Error('Unable to load tasks');
      const data = await response.json();
      setTasks(data.items || []);
@@ -82,7 +242,7 @@ export function TasksListView({
    } finally {
      setLoading(false);
    }
-  }, [domainId]);
+  }, [domainId, sort, order, statusFilter, priorityFilter, searchQuery]);

  const fetchDomains = useCallback(async () => {
    try {
@@ -115,6 +275,11 @@ export function TasksListView({
    return unsubscribe;
  }, [domainId, subscribe, fetchTasks, onRefresh]);

+  // Clear selection when tasks change
+  useEffect(() => {
+    setSelectedIds(new Set());
+  }, [tasks]);
+
  async function toggleTaskComplete(task: Task) {
    const newStatus = task.status === 'done' ? 'todo' : 'done';
    try {
@@ -134,31 +299,249 @@ export function TasksListView({
    }
  }

+  function toggleSort(column: string) {
+    if (sort === column) {
+      updateURLParam('order', order === 'asc' ? 'desc' : 'asc');
+    } else {
+      updateURLParam('sort', column);
+      updateURLParam('order', 'asc');
+    }
+  }
+
+  function getSortIcon(column: string) {
+    if (sort !== column) return <ArrowUpDown className="ml-1 h-3 w-3 opacity-40" />;
+    return order === 'asc' ? (
+      <ArrowUp className="ml-1 h-3 w-3" />
+    ) : (
+      <ArrowDown className="ml-1 h-3 w-3" />
+    );
+  }
+
+  // Bulk actions
+  const allSelected = tasks.length > 0 && selectedIds.size === tasks.length;
+
+  function toggleSelectAll() {
+    if (allSelected) {
+      setSelectedIds(new Set());
+    } else {
+      setSelectedIds(new Set(tasks.map((t) => t.id)));
+    }
+  }
+
+  function toggleSelect(id: string) {
+    setSelectedIds((prev) => {
+      const next = new Set(prev);
+      if (next.has(id)) next.delete(id);
+      else next.add(id);
+      return next;
+    });
+  }
+
+  async function handleBulkStatus(newStatus: string) {
+    if (selectedIds.size === 0) return;
+    setBulkStatusOpen(false);
+    try {
+      const res = await fetch(`/api/domains/${domainId}/tasks/bulk`, {
+        method: 'POST',
+        headers: { 'Content-Type': 'application/json' },
+        body: JSON.stringify({
+          ids: Array.from(selectedIds),
+          updates: { status: newStatus },
+        }),
+      });
+      if (!res.ok) throw new Error();
+      toast.success(`Marked ${selectedIds.size} tasks as ${newStatus.replace('_', ' ')}`);
+      setSelectedIds(new Set());
+      fetchTasks();
+    } catch {
+      toast.error('Unable to update tasks');
+    }
+  }
+
+  async function handleBulkPriority(newPriority: string) {
+    if (selectedIds.size === 0) return;
+    setBulkPriorityOpen(false);
+    try {
+      const res = await fetch(`/api/domains/${domainId}/tasks/bulk`, {
+        method: 'POST',
+        headers: { 'Content-Type': 'application/json' },
+        body: JSON.stringify({
+          ids: Array.from(selectedIds),
+          updates: { priority: newPriority },
+        }),
+      });
+      if (!res.ok) throw new Error();
+      toast.success(`Set priority for ${selectedIds.size} tasks`);
+      setSelectedIds(new Set());
+      fetchTasks();
+    } catch {
+      toast.error('Unable to update tasks');
+    }
+  }
+
+  async function handleBulkDelete() {
+    if (selectedIds.size === 0) return;
+    setBulkDeleting(true);
+    try {
+      const res = await fetch(`/api/domains/${domainId}/tasks/bulk`, {
+        method: 'DELETE',
+        headers: { 'Content-Type': 'application/json' },
+        body: JSON.stringify({ ids: Array.from(selectedIds) }),
+      });
+      if (!res.ok) throw new Error();
+      toast.success(`Deleted ${selectedIds.size} tasks`);
+      setSelectedIds(new Set());
+      fetchTasks();
+    } catch {
+      toast.error('Unable to delete tasks');
+    } finally {
+      setBulkDeleting(false);
+    }
+  }
+
  if (loading) {
    return <p className="text-muted-foreground">Loading tasks...</p>;
  }

-  if (tasks.length === 0) {
  return (
+    <>
+      {/* Filter bar */}
+      <div className="mb-4 flex flex-wrap items-center gap-2">
+        <MultiSelectFilter
+          label="Status"
+          options={statusOptions}
+          selected={statusFilter}
+          onChange={(values) =>
+            updateURLParam('status', values.length > 0 ? values.join(',') : null)
+          }
+        />
+        <MultiSelectFilter
+          label="Priority"
+          options={priorityOptions}
+          selected={priorityFilter}
+          onChange={(values) =>
+            updateURLParam('priority', values.length > 0 ? values.join(',') : null)
+          }
+        />
+        <Select
+          value={domainFilter}
+          onValueChange={(v) => updateURLParam('domain', v || null)}
+        >
+          <SelectTrigger className="h-9 w-40">
+            <SelectValue placeholder="All domains" />
+          </SelectTrigger>
+          <SelectContent>
+            <SelectItem value=" ">All domains</SelectItem>
+            {Array.from(domainMap.entries()).map(([id, name]) => (
+              <SelectItem key={id} value={id}>
+                {name}
+              </SelectItem>
+            ))}
+          </SelectContent>
+        </Select>
+        <div className="relative flex-1 max-w-xs">
+          <Search className="absolute left-2.5 top-1/2 h-4 w-4 -translate-y-1/2 text-muted-foreground" />
+          <Input
+            placeholder="Search tasks..."
+            value={searchInput}
+            onChange={(e) => setSearchInput(e.target.value)}
+            className="h-9 pl-8 pr-8"
+          />
+          {searchInput && (
+            <button
+              onClick={() => setSearchInput('')}
+              className="absolute right-2 top-1/2 -translate-y-1/2 text-muted-foreground hover:text-foreground"
+            >
+              <X className="h-4 w-4" />
+            </button>
+          )}
+        </div>
+      </div>
+
+      {/* Bulk action bar */}
+      {selectedIds.size > 0 && (
+        <div className="sticky top-0 z-10 mb-2 flex items-center gap-2 rounded-md border bg-background/95 p-2 backdrop-blur">
+          <span className="text-sm font-medium text-muted-foreground">
+            {selectedIds.size} selected
+          </span>
+          <div className="ml-2 flex items-center gap-1">
+            <Button
+              variant="secondary"
+              size="sm"
+              onClick={() => handleBulkStatus('done')}
+            >
+              Mark done
+            </Button>
+            <DropdownMenu open={bulkStatusOpen} onOpenChange={setBulkStatusOpen}>
+              <DropdownMenuTrigger asChild>
+                <Button variant="outline" size="sm">
+                  Set status
+                </Button>
+              </DropdownMenuTrigger>
+              <DropdownMenuContent align="start">
+                {statusOptions.map((s) => (
+                  <DropdownMenuItem key={s} onClick={() => handleBulkStatus(s)}>
+                    <span className="capitalize">{s.replace('_', ' ')}</span>
+                  </DropdownMenuItem>
+                ))}
+              </DropdownMenuContent>
+            </DropdownMenu>
+            <DropdownMenu open={bulkPriorityOpen} onOpenChange={setBulkPriorityOpen}>
+              <DropdownMenuTrigger asChild>
+                <Button variant="outline" size="sm">
+                  Set priority
+                </Button>
+              </DropdownMenuTrigger>
+              <DropdownMenuContent align="start">
+                {priorityOptions.map((p) => (
+                  <DropdownMenuItem key={p} onClick={() => handleBulkPriority(p)}>
+                    <span className="capitalize">{p}</span>
+                  </DropdownMenuItem>
+                ))}
+              </DropdownMenuContent>
+            </DropdownMenu>
+            <Button
+              variant="destructive"
+              size="sm"
+              onClick={handleBulkDelete}
+              disabled={bulkDeleting}
+            >
+              {bulkDeleting ? 'Deleting...' : 'Delete'}
+            </Button>
+          </div>
+        </div>
+      )}
+
+      {tasks.length === 0 ? (
        <div className="py-12 text-center">
          <p className="text-muted-foreground">No tasks yet. Create your first task to get started!</p>
        </div>
-    );
-  }
-
-  return (
-    <>
+      ) : (
        <ScrollArea className="w-full">
-        <div className="min-w-[700px]">
+          <div className="min-w-[800px]">
            <Table>
              <TableHeader>
                <TableRow>
-                <TableHead className="w-[50px]"></TableHead>
-                <TableHead>Task</TableHead>
-                <TableHead>Status</TableHead>
-                <TableHead>Priority</TableHead>
+                  <TableHead className="w-[40px]">
+                    <Checkbox
+                      checked={allSelected}
+                      onCheckedChange={toggleSelectAll}
+                      aria-label="Select all tasks"
+                    />
+                  </TableHead>
+                  {sortableColumns.map((col) => (
+                    <TableHead
+                      key={col.key}
+                      className="cursor-pointer select-none"
+                      onClick={() => toggleSort(col.key)}
+                    >
+                      <span className="inline-flex items-center">
+                        {col.label}
+                        {getSortIcon(col.key)}
+                      </span>
+                    </TableHead>
+                  ))}
                  <TableHead>Domain</TableHead>
-                <TableHead>Due Date</TableHead>
                  <TableHead className="w-[50px]"></TableHead>
                </TableRow>
              </TableHeader>
@@ -167,9 +550,9 @@ export function TasksListView({
                  <TableRow key={task.id}>
                    <TableCell>
                      <Checkbox
-                      checked={task.status === 'done'}
-                      onCheckedChange={() => toggleTaskComplete(task)}
-                      aria-label={`Mark "${task.title}" as ${task.status === 'done' ? 'incomplete' : 'complete'}`}
+                        checked={selectedIds.has(task.id)}
+                        onCheckedChange={() => toggleSelect(task.id)}
+                        aria-label={`Select "${task.title}"`}
                      />
                    </TableCell>
                    <TableCell>
@@ -196,9 +579,6 @@ export function TasksListView({
                        {task.priority}
                      </Badge>
                    </TableCell>
-                  <TableCell>
-                    <Badge variant="outline">{domainMap.get(task.domainId) || task.domainId.slice(0, 8)}</Badge>
-                  </TableCell>
                    <TableCell>
                      {task.dueDate && (
                        <span className="flex items-center gap-1 text-sm text-muted-foreground">
@@ -207,6 +587,9 @@ export function TasksListView({
                        </span>
                      )}
                    </TableCell>
+                    <TableCell>
+                      <Badge variant="outline">{domainMap.get(task.domainId) || task.domainId.slice(0, 8)}</Badge>
+                    </TableCell>
                    <TableCell>
                      <DropdownMenu>
                        <DropdownMenuTrigger asChild>
@@ -238,6 +621,7 @@ export function TasksListView({
          </div>
          <ScrollBar orientation="horizontal" />
        </ScrollArea>
+      )}

      <AlertDialog open={!!deleteId} onOpenChange={(o) => !o && setDeleteId(null)}>
        <AlertDialogContent>