Skip to content

refactor: use LazyLock instead of OnceLock#19

Open
justmangoou wants to merge 9 commits into
OpenActionPlugins:mainfrom
justmangoou:refactor-lazy-lock
Open

refactor: use LazyLock instead of OnceLock#19
justmangoou wants to merge 9 commits into
OpenActionPlugins:mainfrom
justmangoou:refactor-lazy-lock

Conversation

@justmangoou

@justmangoou justmangoou commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Summarize

  • Replaced OnceLock with LazyLock.
  • Remove RwLock for RECONNECTING_FLAG.
  • Centralized all states (VoiceSettings, VoiceMode, etc...) to client.rs.
  • Move audio_device_utils.rs to src/, actions package should not contain utilities.
  • Use Mutex instead of RwLock for DISCORD_CLIENT because it only writes.

@nekename

Copy link
Copy Markdown
Member

👍

this codebase is the work of Claude Opus 4.6 with extensive manual review, but no manual refactoring - if it worked it worked

@justmangoou justmangoou marked this pull request as ready for review June 20, 2026 16:06
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