modelcontextprotocol/example-remote-client
Watch
1
0
Fork
You've already forked example-remote-client
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix authentication state consistency across components

Browse source 
- Fix ChatInterface to use context's reactive isAuthenticated value
- Fix useAgentLoop to use context's reactive isAuthenticated value
- Ensure consistent authentication state checking across the application
- Add isAuthenticated to useCallback dependencies for proper reactivity

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
Jerome 2025-06-11 18:26:06 +01:00
commit aed7a25ea7
No known key found for this signature in database
2 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

2
src/components/ChatInterface.tsx
View file

@ -16,7 +16,7 @@ export function ChatInterface() {
getAgentLoopState, getAgentLoopState,
} = useConversation(); } = useConversation();
const { currentProvider } = useInference(); const { provider: currentProvider, isAuthenticated } = useInference();
const [isLoading, setIsLoading] = useState(false); const [isLoading, setIsLoading] = useState(false);
const messagesEndRef = useRef<HTMLDivElement>(null); const messagesEndRef = useRef<HTMLDivElement>(null);

6
src/hooks/useAgentLoop.ts
View file

@ -126,7 +126,7 @@ const DEFAULT_CONFIG: AgentLoopConfig = {
export function useAgentLoop(config: Partial<AgentLoopConfig> = {}): UseAgentLoopReturn { export function useAgentLoop(config: Partial<AgentLoopConfig> = {}): UseAgentLoopReturn {
const finalConfig = { ...DEFAULT_CONFIG, ...config }; const finalConfig = { ...DEFAULT_CONFIG, ...config };
const { currentProvider } = useInference(); const { provider: currentProvider, isAuthenticated } = useInference();
const { getAllTools, callTool: callMCPTool, connections } = useMCP(); const { getAllTools, callTool: callMCPTool, connections } = useMCP();
// Track running loops // Track running loops
@ -255,7 +255,7 @@ export function useAgentLoop(config: Partial<AgentLoopConfig> = {}): UseAgentLoo
conversation: Conversation, conversation: Conversation,
onUpdate: (conversation: Conversation) => void onUpdate: (conversation: Conversation) => void
) => { ) => {
if (!currentProvider?.isAuthenticated) { if (!isAuthenticated || !currentProvider) {
throw new Error('No authenticated inference provider available'); throw new Error('No authenticated inference provider available');
} }
@ -397,7 +397,7 @@ export function useAgentLoop(config: Partial<AgentLoopConfig> = {}): UseAgentLoo
} finally { } finally {
loopStates.current.delete(conversationId); loopStates.current.delete(conversationId);
} }
}, [currentProvider, finalConfig, toInferenceMessages, fromInferenceResponse, getAllTools, executeTool]); }, [isAuthenticated, currentProvider, finalConfig, toInferenceMessages, fromInferenceResponse, getAllTools, executeTool]);
const stopLoop = useCallback((conversationId: string) => { const stopLoop = useCallback((conversationId: string) => {
const loopState = loopStates.current.get(conversationId); const loopState = loopStates.current.get(conversationId);