Skip to content

Fix missing await in update_monitor_channels causing monitor channels to never update#134

Merged
psykzz merged 2 commits intomainfrom
claude/fix-missing-await-in-server-status
Mar 28, 2026
Merged

Fix missing await in update_monitor_channels causing monitor channels to never update#134
psykzz merged 2 commits intomainfrom
claude/fix-missing-await-in-server-status

Conversation

@Claude
Copy link
Copy Markdown
Contributor

@Claude Claude AI commented Mar 28, 2026

The update_monitor_channels method was calling the async update_guild_channel without await, causing monitor channels to never update and generating RuntimeWarning: coroutine was never awaited in Python 3.10+.

Changes

  • Added missing await keyword in update_monitor_channels loop (line 134)

Before

async def update_monitor_channels(self):
    for guild in self.bot.guilds:
        self.update_guild_channel(guild)  # ❌ Missing await

After

async def update_monitor_channels(self):
    for guild in self.bot.guilds:
        await self.update_guild_channel(guild)  # ✅ Fixed

This restores the monitor channel feature, which updates voice channel names with server status every 5 minutes via the background task loop.

Agent-Logs-Url: https://github.com/psykzz/cogs/sessions/f9f25cb1-4db4-41bc-886a-f72bf2a4133e

Co-authored-by: psykzz <1134201+psykzz@users.noreply.github.com>
@Claude Claude AI changed the title [WIP] Fix missing await in nw_server_status to update monitor channels Fix missing await in update_monitor_channels causing monitor channels to never update Mar 28, 2026
@Claude Claude AI requested a review from psykzz March 28, 2026 10:57
@psykzz psykzz marked this pull request as ready for review March 28, 2026 11:13
@psykzz psykzz merged commit a80cb29 into main Mar 28, 2026
2 checks passed
@psykzz psykzz deleted the claude/fix-missing-await-in-server-status branch March 28, 2026 11:13
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.

🐛 [CRITICAL] Missing await in nw_server_status causes monitor channels to never update

2 participants