fix: Implement filterType parameter parsing in workspace devlogs API

Author: tikazyq

.devlog/entries/272-fix-filtertype-parameter-not-working-in-workspace-.json

@@ -0,0 +1,62 @@
+{
+  "id": 272,
+  "key": "fix-filtertype-parameter-not-working-in-workspace-",
+  "title": "Fix filterType parameter not working in workspace devlogs API",
+  "type": "bugfix",
+  "description": "The filterType=open query parameter in the /api/workspaces/default/devlogs API endpoint is not being parsed or applied, causing the filter to be ignored. Users cannot filter devlogs by 'open', 'closed', or other FilterType values.",
+  "status": "done",
+  "priority": "high",
+  "createdAt": "2025-07-24T12:48:37.552Z",
+  "updatedAt": "2025-07-24T12:51:10.830Z",
+  "notes": [
+    {
+      "id": "ff9c006a-9eae-4eef-955d-24461dcdc6db",
+      "timestamp": "2025-07-24T12:49:27.720Z",
+      "category": "solution",
+      "content": "✅ **ROOT CAUSE IDENTIFIED**: The API route was not parsing the `filterType` query parameter.\n\n**SOLUTION IMPLEMENTED**: \n1. Added import for `filterTypeToStatusFilter` and `FilterType` from `@devlog/core`\n2. Added parsing logic for `filterType` parameter that converts it to appropriate status array using the utility function\n3. Logic properly handles precedence: `filterType` sets initial status array, but individual `status` parameter can override it\n\n**KEY CHANGES**:\n- `filterType=open` → converts to `status: ['new', 'in-progress', 'blocked', 'in-review', 'testing']`\n- `filterType=closed` → converts to `status: ['done', 'cancelled']` \n- `filterType=total` → no status filtering applied\n- Individual status values like `filterType=new` → converts to `status: ['new']`\n\nThe fix maintains backward compatibility and follows the existing filtering architecture.",
+      "files": [
+        "packages/web/app/api/workspaces/[id]/devlogs/route.ts"
+      ]
+    },
+    {
+      "id": "4fac432b-ccf9-4f54-9fcf-c1340395fcef",
+      "timestamp": "2025-07-24T12:50:42.293Z",
+      "category": "progress",
+      "content": "✅ **COMPREHENSIVE TESTING COMPLETED** - All acceptance criteria validated!\n\n**Test Results:**\n\n1. **✅ filterType=open works correctly**:\n   - Returns devlogs with statuses: `new`, `in-progress`, `blocked`, `in-review`, `testing`\n   - Test confirmed returned entries had `\"status\":\"in-progress\"` and `\"status\":\"new\"`\n\n2. **✅ filterType=closed works correctly**:\n   - Returns devlogs with statuses: `done`, `cancelled`\n   - Test confirmed all returned entries had `\"status\":\"done\"`\n\n3. **✅ filterType=total works correctly**:\n   - Returns all devlogs regardless of status\n   - Test confirmed mixed statuses: `\"status\":\"done\"` and `\"status\":\"in-progress\"`\n\n4. **✅ Individual status filterType works**:\n   - `filterType=new` correctly returned only `\"status\":\"new\"` entries\n\n5. **✅ Backward compatibility maintained**:\n   - Original `status=done` parameter still works correctly\n\n6. **✅ Precedence rules work correctly**:\n   - `filterType=open&status=done` correctly prioritized `status` parameter\n   - Returned `\"status\":\"done\"` entries (not open statuses)\n\n7. **✅ Combined filtering works**:\n   - `filterType=open&type=bugfix` correctly filtered by both criteria\n   - Returned entries with open status AND bugfix type\n\n**API Endpoint Verified**: `http://localhost:3200/api/workspaces/default/devlogs?filterType=open`"
+    },
+    {
+      "id": "9c6e6248-788a-4162-b3c7-4bda49f72517",
+      "timestamp": "2025-07-24T12:51:10.829Z",
+      "category": "progress",
+      "content": "Completed: ✅ **SUCCESSFULLY FIXED** filterType parameter not working in workspace devlogs API.\n\n**Problem**: The `/api/workspaces/default/devlogs` endpoint was ignoring the `filterType=open` query parameter because the API route wasn't parsing it.\n\n**Solution**: \n- Added import for `filterTypeToStatusFilter` and `FilterType` from `@devlog/core`\n- Added parsing logic that converts `filterType` to appropriate status array\n- Implemented proper precedence: filterType sets initial filter, individual status parameter can override\n\n**Validation**: All acceptance criteria tested and confirmed working:\n- ✅ filterType=open returns only open statuses\n- ✅ filterType=closed returns only closed statuses  \n- ✅ filterType=total returns all devlogs\n- ✅ Individual status values work correctly\n- ✅ Backward compatibility maintained\n- ✅ Precedence rules work correctly\n- ✅ Combined with other filters works\n\nThe fix follows existing filtering architecture patterns and maintains full backward compatibility."
+    }
+  ],
+  "files": [],
+  "relatedDevlogs": [],
+  "context": {
+    "businessContext": "Users need to be able to filter devlog lists by categories like 'open' (showing active work) and 'closed' (showing completed work) for better task management and visibility into work status.",
+    "technicalContext": "The DevlogFilter interface includes a filterType property that accepts FilterType values ('open', 'closed', 'total', or individual statuses), but the API route is not parsing the filterType query parameter from the request and adding it to the filter object passed to listDevlogsFromWorkspace.",
+    "dependencies": [],
+    "decisions": [],
+    "acceptanceCriteria": [
+      "API parses filterType query parameter correctly",
+      "filterType=open returns only devlogs with open statuses (new, in-progress, blocked, in-review, testing)",
+      "filterType=closed returns only devlogs with closed statuses (done, cancelled)",
+      "filterType=total returns all devlogs regardless of status",
+      "Individual status filterType values work correctly",
+      "Existing filtering (status, type, priority, archived) continues to work",
+      "API maintains backward compatibility"
+    ],
+    "risks": []
+  },
+  "aiContext": {
+    "currentSummary": "",
+    "keyInsights": [],
+    "openQuestions": [],
+    "relatedPatterns": [],
+    "suggestedNextSteps": [],
+    "lastAIUpdate": "2025-07-24T12:48:37.552Z",
+    "contextVersion": 1
+  },
+  "closedAt": "2025-07-24T12:51:10.830Z"
+}

packages/web/app/api/workspaces/[id]/devlogs/route.ts

@@ -1,5 +1,6 @@
 import { NextRequest, NextResponse } from 'next/server';
 import { getSharedWorkspaceManager } from '@/lib/shared-workspace-manager';
+import { filterTypeToStatusFilter, type FilterType } from '@devlog/core';
 
 // Mark this route as dynamic to prevent static generation
 export const dynamic = 'force-dynamic';
@@ -12,7 +13,18 @@ export async function GET(request: NextRequest, { params }: { params: { id: stri
     const { searchParams } = new URL(request.url);
     const filter: any = {};
 
-    // Parse query parameters (same as main devlogs API)
+    // Parse filterType parameter first (has precedence over individual status filtering)
+    const filterType = searchParams.get('filterType') as FilterType;
+    if (filterType) {
+      const statusArray = filterTypeToStatusFilter(filterType);
+      if (statusArray) {
+        filter.status = statusArray;
+      }
+      // If filterType is 'total', statusArray will be undefined and no status filtering is applied
+    }
+
+    // Parse other query parameters (same as main devlogs API)
+    // Note: individual status parameter will override filterType if both are provided
     if (searchParams.get('status')) filter.status = searchParams.get('status')?.split(',');
     if (searchParams.get('type')) filter.type = searchParams.get('type');
     if (searchParams.get('priority')) filter.priority = searchParams.get('priority');