feat(tasks-panel): move task and automation filters server-side#3173
Open
feat(tasks-panel): move task and automation filters server-side#3173
Conversation
Task member/type filters now drive useTasks query params instead of filtering already-fetched data in the component. Automation name search uses an ILIKE query via the AUTOMATION_LIST tool instead of client-side array filter. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Contributor
🧪 BenchmarkShould we run the Virtual MCP strategy benchmark for this PR? React with 👍 to run the benchmark.
Benchmark will run on the next push after you react. |
Contributor
Release OptionsSuggested: Minor ( React with an emoji to override the release type:
Current version:
|
myTasks was hardcoded to owner:"me" so in "All members" mode manual tasks still only showed the current user's chats. Both queries now share taskOwner derived from memberFilter. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What is this contribution about?
Task member/type filters in the tasks panel were filtering already-fetched data client-side in
TasksSection. Automation name search in the automations list was also a client-side array filter. Both are now server-side.memberFilter("mine"/"all") now drives theownerparam of the automation tasks query;typeFilter("all"/"manual"/"automation") controls which server result-sets are included. Filter state lifted fromTasksSectiontoTasksPanelContent.AUTOMATION_LISTtool now accepts asearchparam, passed as an ILIKE condition inlistWithTriggerCounts.useAutomationshook threads the param through;automations-list.tsxusesuseTransitionfor a responsive input while the query refetches.Screenshots/Demonstration
No visual change — behavior is identical, filtering now happens at the DB/query level.
How to Test
searchargument rather than filtering the full list client-side.Review Checklist
Summary by cubic
Moved task and automation filters to the server to fetch only needed data and reduce client-side work. Also fixed member filtering so “All members” applies to both manual and automation queries; no UI changes.
Refactors
memberFilterdrivesownerinuseTasks("me"/"all");typeFilterchooses included result sets; filter state lifted toTasksPanelContentwithuseTransition.TasksSection; now acceptsfilterandmemberFilterprops and renders server results as-is.AUTOMATION_LISTacceptssearch; storagelistWithTriggerCountsappliesILIKEon names.useAutomationsacceptssearch; threads it through withKEYS.automationsincludingsearch;automations-list.tsxuses a transition-backed search input.Bug Fixes
ownerfrommemberFilter.Written for commit 8edba91. Summary will update on new commits.