Skip to content

Conversation

@AyeshaI28
Copy link

@AyeshaI28 AyeshaI28 commented Oct 2, 2025

Related Issue #975

Changes

  • Implements auto-refresh for agents and MCP servers list
  • Updates relevant UI components and actions
  • Ensures data is revalidated after create, update, or delete operations

@AyeshaI28 AyeshaI28 requested a review from peterj as a code owner October 2, 2025 10:57
@AyeshaI28 AyeshaI28 changed the title #975 fix(ui): auto-update list of agents and MCP servers (#975) Oct 2, 2025
@AyeshaI28
Copy link
Author

WIP

@peterj peterj marked this pull request as draft October 2, 2025 22:54
@peterj
Copy link
Collaborator

peterj commented Oct 2, 2025

@AyeshaI28 , I converted this back to "Draft", but once it's ready for review, just click "Ready for review"

@AyeshaI28 AyeshaI28 force-pushed the fix/auto-update-agents-mcp-servers-975 branch from b8281cc to 63322a7 Compare October 3, 2025 17:21
@AyeshaI28 AyeshaI28 marked this pull request as ready for review October 3, 2025 17:22
@AyeshaI28 AyeshaI28 force-pushed the fix/auto-update-agents-mcp-servers-975 branch 2 times, most recently from 4173812 to 4ce0f8a Compare October 3, 2025 17:50
@AyeshaI28 AyeshaI28 force-pushed the fix/auto-update-agents-mcp-servers-975 branch from 4ce0f8a to cc47f9d Compare October 3, 2025 18:07
@AyeshaI28
Copy link
Author

@peterj

Copy link
Collaborator

@peterj peterj left a comment

Choose a reason for hiding this comment

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

I don't think this will work -- when the agents/servers are created it takes a little while for them to become available. calling revalidatePath only does a single "refresh'. An agent might take 30 seconds or so to come up and become available, so we'll need to use swr as in #987 to implement this.

@peterj
Copy link
Collaborator

peterj commented Oct 20, 2025

we're working on getting #987 merged. Closing.

@peterj peterj closed this Oct 20, 2025
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.

2 participants