Skip to content

Extract VotingPlugin background task - #1568

Merged
BenCodez merged 1 commit into
refactor/main-lifecycle-servicesfrom
refactor/background-task
Aug 26, 2026
Merged

Extract VotingPlugin background task#1568
BenCodez merged 1 commit into
refactor/main-lifecycle-servicesfrom
refactor/background-task

Conversation

@BenCodez

Copy link
Copy Markdown
Owner

Stacks on #1566.

Moves the periodic VotingPlugin refresh loop out of VotingPluginMain into backgroundtask/VotingPluginBackgroundTask.

The component owns:

  • update requested/running state
  • user streaming and temporary cache lifecycle
  • top-voter aggregation
  • voted-today calculation
  • per-user/global placeholder refreshes
  • server-data/sign refreshes
  • DiscordSRV top-voter refresh
  • elapsed run timing

VotingPluginMain#update(), setUpdate, isUpdate, isUpdateStarted, and getLastBackgroundTaskTimeTaken remain compatibility delegates. Includes a focused state regression test.

@BenCodez
BenCodez marked this pull request as ready for review August 25, 2026 00:19
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for security reviews. Please try again later.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 656e9b050c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@BenCodez
BenCodez force-pushed the refactor/background-task branch from 7130991 to f3dd2f2 Compare August 26, 2026 00:08
@BenCodez
BenCodez force-pushed the refactor/background-task branch from f3dd2f2 to 6f4a3bd Compare August 26, 2026 00:12
@BenCodez
BenCodez merged commit 192c114 into master Aug 26, 2026
4 checks passed
@BenCodez
BenCodez deleted the refactor/background-task branch August 26, 2026 01:34
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.

1 participant