Skip to content

tell the user which master a plugin is missing and why - #24206

Merged
IDCs merged 5 commits into
task/laz-1141from
task/laz-1030
Sep 17, 2026
Merged

IDCs merged 5 commits into
task/laz-1141from
task/laz-1030

Conversation

@IDCs

@IDCs IDCs commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

The Missing Masters health check now says why each master is unavailable, in LOOT's words: not-installed and not-enabled are the user's install state and stay warnings, while removed-externally, unscanned and not-deployed once nothing is left to rescan mean the plugin list disagrees with the game folder, and are errors. Those cases go to telemetry once per master per state, with the mod and collection on both sides of the dependency, so a broken collection revision is identifiable from one span. A sort that fails on an unloaded master names it the same way, and the check cannot be hidden.

Stacked on #24196.

part of LAZ-1030
part of LAZ-1039

A check that returns allowSuppress false is shown even when its notification id
carries a suppression, so a check that is not hideable cannot be silenced by an
entry the user set earlier.

part of LAZ-1030
The index reads the collection's slug and revision off the container mod, and
falls back to the membership rules' reference tags for mods whose attributes do
not carry them.

part of LAZ-1039
Every unavailable master is classified as an install state the user can fix or
as the plugin list contradicting the game folder, phrased the way LOOT does.
The contradictions go to telemetry once per master per state, and the health
check cannot be suppressed, so a stale suppression cannot hide an error-grade
run.

part of LAZ-1030
part of LAZ-1039
@IDCs
IDCs requested a review from a team as a code owner September 16, 2026 09:26
@IDCs IDCs self-assigned this Sep 16, 2026
@IDCs
IDCs added this pull request to stack #24207 September 16, 2026 09:26
@IDCs
IDCs merged commit d0cff76 into integration/gamebryo Sep 17, 2026
3 checks passed
@IDCs
IDCs deleted the task/laz-1030 branch September 17, 2026 11:11
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.

3 participants