Skip to content
This repository was archived by the owner on Aug 6, 2026. It is now read-only.

feat: use task summaries endpoint for sidebar loads - #1945

Merged
jonathanlab merged 5 commits into
mainfrom
fix/faster-sidebar-loads
May 6, 2026
Merged

jonathanlab merged 5 commits into
mainfrom
fix/faster-sidebar-loads

Conversation

@jonathanlab

@jonathanlab jonathanlab commented Apr 29, 2026 •

Copy link
Copy Markdown
Contributor

Generated-By: PostHog Code
Task-Id: a801cf87-213b-451b-9cdc-5150e50bc059
@greptile-apps

greptile-apps Bot commented Apr 29, 2026

Copy link
Copy Markdown
Contributor
Prompt To Fix All With AI
This is a comment left during a code review.
Path: apps/code/src/renderer/features/sidebar/hooks/useSidebarData.ts
Line: 144-148

Comment:
**`useTasks` polled unconditionally on the optimized path**

When `showAllUsers` is `false`, `useTaskSummaries` is used for sidebar data and `fullTasks` is explicitly discarded (`rawTasks = summaryTasks`). Yet `useTasks` is called without an `enabled` guard, so the heavy `/tasks/` list endpoint is still polled at `TASK_LIST_POLL_INTERVAL_MS` in the background on every sidebar mount — directly undercutting the optimisation this PR introduces. Adding `enabled: showAllUsers` (or an equivalent guard) to the hook options would restrict the full-list poll to only the code path that needs it.

How can I resolve this? If you propose a fix, please make it concise.

Reviews (1): Last reviewed commit: "feat: use task summaries endpoint for si..." | Re-trigger Greptile

Comment thread apps/code/src/renderer/features/sidebar/hooks/useSidebarData.ts Outdated
# Conflicts:
#	apps/code/src/renderer/features/sidebar/hooks/useSidebarData.ts
@jonathanlab jonathanlab added the Create Release This will trigger a new release label May 5, 2026
@jonathanlab
jonathanlab merged commit fa973aa into main May 6, 2026
15 checks passed
@jonathanlab
jonathanlab deleted the fix/faster-sidebar-loads branch May 6, 2026 09:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Create Release This will trigger a new release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants