Skip to content

fix: Voiceチャンネルに入った状態でそのチャンネルのTextチャットを使うと!vrcが動かないバグの修正#3

Merged
Nanahuse merged 1 commit intomainfrom
fix/lock_error
Nov 28, 2025
Merged

fix: Voiceチャンネルに入った状態でそのチャンネルのTextチャットを使うと!vrcが動かないバグの修正#3
Nanahuse merged 1 commit intomainfrom
fix/lock_error

Conversation

@Nanahuse
Copy link
Owner

VoiceチャンネルのIDとVoiceチャンネルについているTextチャンネルのIDが同じであり、間違ってロックが発生していた

Copilot AI review requested due to automatic review settings November 28, 2025 12:16
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a bug where the !vrc command would fail when invoked from a text channel attached to a voice channel. The issue occurred because Discord uses the same ID for both the voice channel and its attached text channel, causing lock conflicts with the single channel_semaphore.

Key Changes:

  • Split channel_semaphore into separate text_channel_semaphore and voice_channel_semaphore instances
  • Updated both rc and vrc commands to use the appropriate semaphore(s)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

@Nanahuse Nanahuse merged commit 9d38314 into main Nov 28, 2025
7 checks passed
@Nanahuse Nanahuse deleted the fix/lock_error branch November 28, 2025 12:20
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