Skip to content

[DRAFT] Plugin Game Settings Subsystem - #900

Draft
Cryotechnic wants to merge 6 commits into
mainfrom
plugin
Draft

[DRAFT] Plugin Game Settings Subsystem#900
Cryotechnic wants to merge 6 commits into
mainfrom
plugin

Conversation

@Cryotechnic

Copy link
Copy Markdown
Member

IN DRAFT PLEASE DO NOT MERGE

Main Goal

Add declarative plugin-defined game settings support to Plugin.Core and render those settings in Collapse.

Also harden plugin integration by:

  • Keeping the game-log callback alive throughout asynchronous game sessions.
  • Hydrating background notifications from an already-running activity’s current progress and status.
  • Correcting Wuthering Waves preload progress totals.

PR Status :

  • Overall Status : In Progress
  • Commits : In Progress
  • Synced to base (Collapse:main) : Not yet
  • Build status : Build Errors
  • Crashing : No
  • Bug found caused by PR : 0

Templates

Changelog Prefixes
  **[New]**
  **[Imp]**
  **[Fix]**
  **[Loc]**
  **[Doc]**

Expose `Progress` and `Status` on `IBackgroundActivity` and use them when wiring notification handlers in `BackgroundActivityManager`. This immediately initializes notification text/progress for already-running activities instead of waiting for the next event, preventing placeholder content from lingering.
Add a reusable speed-calculator reset helper in `ProgressBase` and use it in plugin install progress updates to avoid bogus speed deltas after state transitions. The wrapper now resets its download baseline when install state changes, computes per-tick bytes only from monotonic progress, and clears speed/time-left values appropriately. It also ensures completed installs report zero time left and clamp per-file progress to 100%.
@socket-security

socket-security Bot commented Aug 19, 2026

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedsentry@​6.8.0 ⏵ 6.9.08910090100100

View full report

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