Fix authentication event loop corruption by converting get_current_user to async

[abrookins]

The get_current_user() function was using asyncio.run() within FastAPI's async context, which creates a new event loop and causes "Event loop is closed" errors. This led to intermittent authentication failures where requests would alternate between success (200) and failure (500).

Changes:

• Convert get_current_user() from sync to async function
• Replace asyncio.run(verify_token()) with await verify_token()
• Update require_scope() and require_role() dependency functions to be async
• Fix all related test cases to use await when calling these functions

This resolves the issue where tool calls to the memory server would intermittently fail with 500 errors during token verification.

🤖 Generated with Claude Code

[Copilot]

Pull Request Overview

This PR fixes an authentication event loop corruption issue by converting get_current_user() from a synchronous to an asynchronous function. The core problem was that the function was using asyncio.run() within FastAPI's existing async context, causing "Event loop is closed" errors and intermittent authentication failures.

• Convert get_current_user() to async and replace asyncio.run() with await
• Update require_scope() and require_role() dependency functions to be async
• Fix all test cases to properly await the converted async functions

Reviewed Changes

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

File	Description
agent_memory_server/auth.py	Converts authentication functions to async and removes problematic asyncio.run() call
tests/test_token_auth.py	Updates test cases to use await with converted async functions and adds proper async decorators
tests/test_auth.py	Updates existing test cases to use await with the now-async authentication functions