Skip to content

Selecting task selects agent name and task id - #22

Merged
declan-scale merged 2 commits into
mainfrom
declan-scale/sidebar-agent-name
Oct 30, 2025
Merged

Selecting task selects agent name and task id#22
declan-scale merged 2 commits into
mainfrom
declan-scale/sidebar-agent-name

Conversation

@declan-scale

@declan-scale declan-scale commented Oct 29, 2025

Copy link
Copy Markdown
Collaborator

Also get rid of unused code in TaskProvider and use loading.tsx instead of suspense

Copilot AI review requested due to automatic review settings October 29, 2025 22:43

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request implements functionality to automatically select both the agent name and task ID when a task is selected from the task sidebar. The changes ensure that when a user clicks on a task, the first agent associated with that task is automatically selected, providing a more seamless user experience.

  • Removes fallback functionality from TaskProvider component
  • Updates task selection to automatically set both task ID and first agent name
  • Removes unused Suspense wrapper from the root component

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
task-provider.tsx Removes fallback prop and related conditional rendering logic
task-sidebar.tsx Updates task selection to include automatic agent name selection
agentex-ui-root.tsx Removes unused Suspense import and wrapper component

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread agentex-ui/components/agentex/task-sidebar.tsx Outdated
@declan-scale
declan-scale merged commit 5c9e69e into main Oct 30, 2025
3 checks passed
@declan-scale
declan-scale deleted the declan-scale/sidebar-agent-name branch October 30, 2025 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants