Address work product review feedback

Co-Authored-By: Paperclip <noreply@paperclip.ing>

server/src/services/work-product-adoption.ts

@@ -111,7 +111,6 @@ function githubPullRequestIdentity(url: string): { externalId: string; title: st
   } catch {
     return null;
   }
-  return null;
 }
 
 function shortSha(value: string) {

server/src/services/work-products.ts

@@ -5,6 +5,8 @@ import type { IssueWorkProduct } from "@paperclipai/shared";
 
 type IssueWorkProductRow = typeof issueWorkProducts.$inferSelect;
 
+const EXECUTION_WORKSPACE_WORK_PRODUCTS_LIMIT = 200;
+
 function toIssueWorkProduct(row: IssueWorkProductRow): IssueWorkProduct {
   return {
     id: row.id,
@@ -52,7 +54,8 @@ export function workProductService(db: Db) {
             eq(issueWorkProducts.executionWorkspaceId, executionWorkspaceId),
           ),
         )
-        .orderBy(desc(issueWorkProducts.isPrimary), desc(issueWorkProducts.updatedAt));
+        .orderBy(desc(issueWorkProducts.isPrimary), desc(issueWorkProducts.updatedAt))
+        .limit(EXECUTION_WORKSPACE_WORK_PRODUCTS_LIMIT);
       return rows.map(toIssueWorkProduct);
     },
 

ui/src/App.tsx

@@ -177,6 +177,7 @@ function boardRoutes() {
       <Route path="routines/:routineId/:section" element={<RoutineDetail />} />
       <Route path="execution-workspaces/:workspaceId" element={<ExecutionWorkspaceDetail />} />
       <Route path="execution-workspaces/:workspaceId/services" element={<ExecutionWorkspaceDetail />} />
+      <Route path="execution-workspaces/:workspaceId/work-products" element={<ExecutionWorkspaceDetail />} />
       <Route path="execution-workspaces/:workspaceId/configuration" element={<ExecutionWorkspaceDetail />} />
       <Route path="execution-workspaces/:workspaceId/runtime-logs" element={<ExecutionWorkspaceDetail />} />
       <Route path="execution-workspaces/:workspaceId/issues" element={<ExecutionWorkspaceDetail />} />
@@ -440,6 +441,7 @@ export function App() {
           <Route path="workspaces" element={<UnprefixedBoardRedirect />} />
           <Route path="execution-workspaces/:workspaceId" element={<UnprefixedBoardRedirect />} />
           <Route path="execution-workspaces/:workspaceId/services" element={<UnprefixedBoardRedirect />} />
+          <Route path="execution-workspaces/:workspaceId/work-products" element={<UnprefixedBoardRedirect />} />
           <Route path="execution-workspaces/:workspaceId/configuration" element={<UnprefixedBoardRedirect />} />
           <Route path="execution-workspaces/:workspaceId/runtime-logs" element={<UnprefixedBoardRedirect />} />
           <Route path="execution-workspaces/:workspaceId/issues" element={<UnprefixedBoardRedirect />} />