Skip to content

Conversation

@lgh-solace
Copy link
Collaborator

@lgh-solace lgh-solace commented Aug 20, 2025

This PR pushes agent fetching to the chat context in order to:

  1. Reduce the number of times agents are fetched
  2. Facilitate testing

It also adds a refresh agents button to the Agents page.
image

@github-actions
Copy link

github-actions bot commented Aug 20, 2025

WhiteSource Policy Violation Summary

✅︎ No Blocking Whitesource Policy Violations found in solaceai/solace-agent-mesh-ui-pr-247!

@github-actions
Copy link

github-actions bot commented Aug 20, 2025

WhiteSource Policy Violation Summary

✅︎ No Blocking Whitesource Policy Violations found in solaceai/solace-agent-mesh-pr-247!

@sonarqube-solacecloud
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

import reactHooks from 'eslint-plugin-react-hooks'
import reactRefresh from 'eslint-plugin-react-refresh'
import tseslint from 'typescript-eslint'
import js from "@eslint/js";
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Only "prettified".

@@ -1,11 +1,11 @@
{
"name": "solace-agent-mesh-ui",
"name": "@SolaceLabs/solace-agent-mesh-ui",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Correcting name.

import { useChatContext } from "@/lib/hooks";

interface MorePopoverProps {
interface ArtifactMorePopoverProps {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Renaming component only.

<Header
title="Agents"
buttons={[
<Button variant="ghost" title="Refresh Agents" onClick={() => agentsRefetch()}>
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Adding refresh button to header.

const latestStatusText = useRef<string | null>(null);
const sseEventSequenceRef = useRef<number>(0);

// Agents State
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Pushing agent fetch to chat context.

@lgh-solace lgh-solace marked this pull request as ready for review August 20, 2025 16:05
@lgh-solace lgh-solace merged commit 330f6a1 into main Aug 20, 2025
15 checks passed
@lgh-solace lgh-solace deleted the lgh-storybook branch August 20, 2025 17:57
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