Feature/c1 optimization

[priosshrsth]

Changes

Feature branch for task app optimization (esp detail page)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
tasks-app	Ready	Preview, Comment	May 25, 2026 4:06am

[greptile-apps]

Greptile Summary

This PR optimises the detail page and reduces server-side latency by calling the service layer directly (bypassing HTTP round-trips), parallelising DB queries in TasksService.getOneTask, and moving workspace/assignee fetching to lightweight client-side SWR components in the root layout.

• Detail page overhaul: authenticateWithToken + four parallel loaders replace six serial HTTP self-calls; WorkflowState is now included in the initial findFirst and parent-task traversal collapses N serial queries into one findMany.
• Global client-side fetchers: WorkspaceFetcher and AssigneesFetcher (with retry) are placed in the root layout, allowing portalUrl and assignee data to be read from Redux everywhere without prop-drilling through server components.
• State seeding refactor: SeedActiveTask / SeedActiveTemplate extract active-entity lifecycle from ClientSideStateUpdate, using a dual-effect pattern to handle React 18 concurrent-mode unmount races; React.cache() is added to getWorkspace and getInternalUser for per-request memoisation.

Confidence Score: 4/5

Safe to merge after fixing the VERCEL_BRANCH_URL priority in getAppUrl(); all other changes are clean optimisations.

The getAppUrl() refactor in src/config/index.ts now checks VERCEL_BRANCH_URL before VERCEL_URL. VERCEL_BRANCH_URL is a Vercel-managed branch alias that gets reassigned to newer deployments, so an in-flight SSR request on deployment A can route its internal API calls to deployment B after B goes live. The previous code used VERCEL_URL (unique per deployment) which avoids this cross-deployment version skew entirely. All other changes are well-structured and free of correctness issues.

src/config/index.ts — the new getAppUrl priority order.

Important Files Changed

Filename	Overview
src/config/index.ts	Refactored getAppUrl() now prioritises VERCEL_BRANCH_URL over VERCEL_URL, which can route internal API calls to a different deployment during rolling deployments and cause version skew.
src/app/detail/[task_id]/[user_type]/page.tsx	Detail page now calls service layer directly via authenticateWithToken plus parallel loaders - clean optimisation with no issues found.
src/app/detail/[task_id]/[user_type]/loaders.ts	New loader layer with typed error handling; NOT_FOUND is correctly caught and rethrown for other errors.
src/hoc/state-seeders.tsx	New SeedActiveTask and SeedActiveTemplate components extract state seeding from ClientSideStateUpdate; dual-effect pattern correctly handles React 18 concurrent-mode races.
src/utils/CopilotAPI.ts	getWorkspace and getInternalUser now use React.cache() for per-request memoisation; private methods called directly inside the cache to avoid recursion.
src/app/api/tasks/tasks.service.ts	DB queries consolidated: workflowState included in initial findFirst; body update, subtask count, and assignee fetch run in a single Promise.all; parent traversal optimised from N serial queries to one findMany.
src/app/_fetchers/AssigneesFetcher.tsx	Replaced SSR AssigneeFetcher with a client-side SWR component in the layout; errorRetryCount and errorRetryInterval address the previously flagged fetch-retry regression.
src/app/_fetchers/WorkspaceFetcher.tsx	New client-side SWR component for workspace data; skips fetch when already in Redux store and when no token is present.
src/app/layout.tsx	WorkspaceFetcher and AssigneesFetcher added globally in root layout inside Suspense boundaries; both guard on token presence.
src/hoc/ClientSideStateUpdate.tsx	Removed task, accesibleTaskIds, and template props in favour of dedicated SeedActive* components; cleanup effects now live in the seeders.

Sequence Diagram

sequenceDiagram
    participant Browser
    participant DetailPage as DetailPage (Server)
    participant Loaders as loaders.ts
    participant DB as Database
    participant WF as WorkspaceFetcher (Client)
    participant AF as AssigneesFetcher (Client)
    participant Redux as Redux Store

    Browser->>DetailPage: Navigate to detail page

    DetailPage->>Loaders: authenticateWithToken
    Loaders-->>DetailPage: user

    par Parallel DB calls
        DetailPage->>Loaders: loadTask
        Loaders->>DB: findFirst + workflowState + assignee + subtask count
        DB-->>Loaders: task data
    and
        DetailPage->>Loaders: loadSubtaskStatus
        Loaders->>DB: getSubtaskStatus
        DB-->>Loaders: status
    and
        DetailPage->>Loaders: loadTaskPath
        Loaders->>DB: findMany parents
        DB-->>Loaders: ancestors
    and
        DetailPage->>Loaders: loadViewSettings
        Loaders->>DB: getViewSettings
        DB-->>Loaders: settings
    end

    DetailPage-->>Browser: HTML with SSR task data

    Browser->>WF: Mount
    WF->>Redux: dispatch setWorkspace

    Browser->>AF: Mount
    AF->>Redux: dispatch setAssigneeList

Loading

_{Reviews (2): Last reviewed commit: "thrown error when getting subtask count ..." | Re-trigger Greptile}

[vercel]

Deployment failed with the following error:

Deploying Serverless Functions to multiple regions is restricted to the Pro and Enterprise plans.

Learn More: https://vercel.link/multiple-function-regions

[priosshrsth]

@greptileai review the pr again.