Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Plugin reinstallation breaks plugin order #679

Open
4 tasks done
milroneth opened this issue Aug 10, 2024 · 3 comments
Open
4 tasks done

[BUG] Plugin reinstallation breaks plugin order #679

milroneth opened this issue Aug 10, 2024 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@milroneth
Copy link

milroneth commented Aug 10, 2024

Please confirm

  • I have searched existing issues
  • This issue is not a duplicate of an existing one
  • I have checked the common issues section in the readme file
  • I have attached logs to this bug report (failure to include logs will mean your issue may not be responded to).

Bug Report Description

When reinstalling plugins (either through the store page or the Decky settings "Plugins" tab), the uninstallation step involved removes the plugin from settings/loader.json:pluginOrder.

This leaves the plugin in limbo where the current Decky instance still knows its position in the list, but the next load will float the plugin to the edge. This can be avoided by the user immediately re-ordering the list by moving said plugin one place up/down and back again, so that it gets readded to pluginOrder in its previous place.

Expected Behaviour

A plugin being reinstalled should not lose its position in the plugin list unexpectedly on the next restart.

It should not be removed from pluginOrder when it's uninstalled as part of a reinstallation, or if that is impractical, it should be immediately re-added to the list in its current position in memory after re-installation is complete.

SteamOS version

SteamOS 3.5.19 Stable

Selected Update Channel

Prerelease

Have you modified the read-only filesystem at any point?

No

Backend Logs

deckylog.txt

Frontend Logs

cef_log.txt

@milroneth milroneth added the bug Something isn't working label Aug 10, 2024
@github-project-automation github-project-automation bot moved this to 🆕 New in SDH Tracker Aug 10, 2024
@TrainDoctor
Copy link
Member

I have also found that reloading plugins with permissions changed to the plugins directory and it's contents is causing plugins to not re-appear after being reloaded. Since these two issues are likely related, I'm adding this onto this bug report.

@TrainDoctor
Copy link
Member

@milroneth could you see if you can recreate this issue? I haven't had any reports of it since you made us aware.

@milroneth
Copy link
Author

milroneth commented Sep 13, 2024

Still valid in v3.0.0 stable. I forgot to pretend-reorder yesterday after updating Game Theme Music and today I found it on top of the list.

Just tested specifically for this issue, to make sure I wasn't wrong, I reinstalled a random plugin. It was removed from the json, jumped on top of the plugin list on reboot.

EDIT: Probably related to this, when a plugin is fresh installed, trying to reorder it before a loader restart is wonky at best, you start moving one plugin and end up moving another. I think that every plugin installation, no matter the source of that action, should trigger a re-write of loader.json to disk (without reading it first at that moment), before any other actions mess things up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: 🆕 New
Development

No branches or pull requests

5 participants