Profiles (#1178)

Dalamud/Plugin/Internal/PluginManager.cs

@@ -23,6 +23,7 @@ using Dalamud.Interface.Internal;
 using Dalamud.IoC.Internal;
 using Dalamud.Logging.Internal;
 using Dalamud.Plugin.Internal.Exceptions;
+using Dalamud.Plugin.Internal.Profiles;
 using Dalamud.Plugin.Internal.Types;
 using Dalamud.Utility;
 using Dalamud.Utility.Timing;
@@ -77,6 +78,9 @@ Thanks and have fun!";
     [ServiceManager.ServiceDependency]
     private readonly DalamudStartInfo startInfo = Service<DalamudStartInfo>.Get();
 
+    [ServiceManager.ServiceDependency]
+    private readonly ProfileManager profileManager = Service<ProfileManager>.Get();
+
     [ServiceManager.ServiceConstructor]
     private PluginManager()
     {
@@ -421,7 +425,7 @@ Thanks and have fun!";
 
             try
             {
-                pluginDefs.Add(versionsDefs.OrderByDescending(x => x.Manifest!.EffectiveVersion).First());
+                pluginDefs.Add(versionsDefs.MaxBy(x => x.Manifest!.EffectiveVersion));
             }
             catch (Exception ex)
             {
@@ -839,7 +843,7 @@ Thanks and have fun!";
     /// <param name="isBoot">If this plugin is being loaded at boot.</param>
     /// <param name="doNotLoad">Don't load the plugin, just don't do it.</param>
     /// <returns>The loaded plugin.</returns>
-    public async Task<LocalPlugin> LoadPluginAsync(FileInfo dllFile, LocalPluginManifest? manifest, PluginLoadReason reason, bool isDev = false, bool isBoot = false, bool doNotLoad = false)
+    private async Task<LocalPlugin> LoadPluginAsync(FileInfo dllFile, LocalPluginManifest? manifest, PluginLoadReason reason, bool isDev = false, bool isBoot = false, bool doNotLoad = false)
     {
         var name = manifest?.Name ?? dllFile.Name;
         var loadPlugin = !doNotLoad;
@@ -859,8 +863,9 @@ Thanks and have fun!";
             loadPlugin &= !isBoot || devPlugin.StartOnBoot;
 
             // If we're not loading it, make sure it's disabled
-            if (!loadPlugin && !devPlugin.IsDisabled)
-                devPlugin.Disable();
+            // NOTE: Should be taken care of below by the profile code
+            // if (!loadPlugin && !devPlugin.IsDisabled)
+            //     devPlugin.Disable();
 
             plugin = devPlugin;
         }
@@ -870,17 +875,24 @@ Thanks and have fun!";
             plugin = new LocalPlugin(dllFile, manifest);
         }
 
+#pragma warning disable CS0618
+        var defaultState = manifest?.Disabled != true && loadPlugin;
+#pragma warning restore CS0618
+
+        // Need to do this here, so plugins that don't load are still added to the default profile
+        var wantToLoad = this.profileManager.GetWantState(plugin.Manifest.InternalName, defaultState);
+
         if (loadPlugin)
         {
             try
             {
-                if (!plugin.IsDisabled && !plugin.IsOrphaned)
+                if (wantToLoad && !plugin.IsOrphaned)
                 {
                     await plugin.LoadAsync(reason);
                 }
                 else
                 {
-                    Log.Verbose($"{name} not loaded, disabled:{plugin.IsDisabled} orphaned:{plugin.IsOrphaned}");
+                    Log.Verbose($"{name} not loaded, wantToLoad:{wantToLoad} orphaned:{plugin.IsOrphaned}");
                 }
             }
             catch (InvalidPluginException)
@@ -1073,7 +1085,7 @@ Thanks and have fun!";
             if (plugin.InstalledPlugin.IsDev)
                 continue;
 
-            if (plugin.InstalledPlugin.Manifest.Disabled && ignoreDisabled)
+            if (!plugin.InstalledPlugin.IsWantedByAnyProfile && ignoreDisabled)
                 continue;
 
             if (plugin.InstalledPlugin.Manifest.ScheduledForDeletion)
@@ -1132,6 +1144,7 @@ Thanks and have fun!";
 
             if (plugin.IsDev)
             {
+                // TODO: Does this ever work? Why? We should never update devplugins
                 try
                 {
                     plugin.DllFile.Delete();
@@ -1151,8 +1164,11 @@ Thanks and have fun!";
             {
                 try
                 {
+                    // TODO: Why were we ever doing this? We should never be loading the old version in the first place
+                    /*
                     if (!plugin.IsDisabled)
                         plugin.Disable();
+                        */
 
                     lock (this.pluginListLock)
                     {

Dalamud/Plugin/Internal/Profiles/Profile.cs

@@ -0,0 +1,214 @@
+using System;
+using System.Collections.Generic;
+using System.Diagnostics;
+using System.Linq;
+
+using Dalamud.Configuration.Internal;
+using Dalamud.Logging.Internal;
+using Dalamud.Utility;
+
+namespace Dalamud.Plugin.Internal.Profiles;
+
+/// <summary>
+/// Class representing a single runtime profile.
+/// </summary>
+internal class Profile
+{
+    private static readonly ModuleLog Log = new("PROFILE");
+
+    private readonly ProfileManager manager;
+    private readonly ProfileModelV1 modelV1;
+
+    /// <summary>
+    /// Initializes a new instance of the <see cref="Profile"/> class.
+    /// </summary>
+    /// <param name="manager">The manager this profile belongs to.</param>
+    /// <param name="model">The model this profile is tied to.</param>
+    /// <param name="isDefaultProfile">Whether or not this profile is the default profile.</param>
+    /// <param name="isBoot">Whether or not this profile was initialized during bootup.</param>
+    public Profile(ProfileManager manager, ProfileModel model, bool isDefaultProfile, bool isBoot)
+    {
+        this.manager = manager;
+        this.IsDefaultProfile = isDefaultProfile;
+        this.modelV1 = model as ProfileModelV1 ??
+                       throw new ArgumentException("Model was null or unhandled version");
+
+        // We don't actually enable plugins here, PM will do it on bootup
+        if (isDefaultProfile)
+        {
+            // Default profile cannot be disabled
+            this.IsEnabled = this.modelV1.IsEnabled = true;
+            this.Name = this.modelV1.Name = "DEFAULT";
+        }
+        else if (this.modelV1.AlwaysEnableOnBoot && isBoot)
+        {
+            this.IsEnabled = true;
+            Log.Verbose("{Guid} set enabled because bootup", this.modelV1.Guid);
+        }
+        else if (this.modelV1.IsEnabled)
+        {
+            this.IsEnabled = true;
+            Log.Verbose("{Guid} set enabled because remember", this.modelV1.Guid);
+        }
+        else
+        {
+            Log.Verbose("{Guid} not enabled", this.modelV1.Guid);
+        }
+    }
+
+    /// <summary>
+    /// Gets or sets this profile's name.
+    /// </summary>
+    public string Name
+    {
+        get => this.modelV1.Name;
+        set
+        {
+            this.modelV1.Name = value;
+            Service<DalamudConfiguration>.Get().QueueSave();
+        }
+    }
+
+    /// <summary>
+    /// Gets or sets a value indicating whether this profile shall always be enabled at boot.
+    /// </summary>
+    public bool AlwaysEnableAtBoot
+    {
+        get => this.modelV1.AlwaysEnableOnBoot;
+        set
+        {
+            this.modelV1.AlwaysEnableOnBoot = value;
+            Service<DalamudConfiguration>.Get().QueueSave();
+        }
+    }
+
+    /// <summary>
+    /// Gets this profile's guid.
+    /// </summary>
+    public Guid Guid => this.modelV1.Guid;
+
+    /// <summary>
+    /// Gets a value indicating whether or not this profile is currently enabled.
+    /// </summary>
+    public bool IsEnabled { get; private set; }
+
+    /// <summary>
+    /// Gets a value indicating whether or not this profile is the default profile.
+    /// </summary>
+    public bool IsDefaultProfile { get; }
+
+    /// <summary>
+    /// Gets all plugins declared in this profile.
+    /// </summary>
+    public IEnumerable<ProfilePluginEntry> Plugins =>
+        this.modelV1.Plugins.Select(x => new ProfilePluginEntry(x.InternalName, x.IsEnabled));
+
+    /// <summary>
+    /// Gets this profile's underlying model.
+    /// </summary>
+    public ProfileModel Model => this.modelV1;
+
+    /// <summary>
+    /// Set this profile's state. This cannot be called for the default profile.
+    /// This will block until all states have been applied.
+    /// </summary>
+    /// <param name="enabled">Whether or not the profile is enabled.</param>
+    /// <param name="apply">Whether or not the current state should immediately be applied.</param>
+    /// <exception cref="InvalidOperationException">Thrown when an untoggleable profile is toggled.</exception>
+    public void SetState(bool enabled, bool apply = true)
+    {
+        if (this.IsDefaultProfile)
+            throw new InvalidOperationException("Cannot set state of default profile");
+
+        Debug.Assert(this.IsEnabled != enabled, "Trying to set state of a profile to the same state");
+        this.IsEnabled = this.modelV1.IsEnabled = enabled;
+        Log.Verbose("Set state {State} for {Guid}", enabled, this.modelV1.Guid);
+
+        Service<DalamudConfiguration>.Get().QueueSave();
+
+        if (apply)
+            this.manager.ApplyAllWantStates();
+    }
+
+    /// <summary>
+    /// Check if this profile contains a specific plugin, and if it is enabled.
+    /// </summary>
+    /// <param name="internalName">The internal name of the plugin.</param>
+    /// <returns>Null if this profile does not declare the plugin, true if the profile declares the plugin and wants it enabled, false if the profile declares the plugin and does not want it enabled.</returns>
+    public bool? WantsPlugin(string internalName)
+    {
+        var entry = this.modelV1.Plugins.FirstOrDefault(x => x.InternalName == internalName);
+        return entry?.IsEnabled;
+    }
+
+    /// <summary>
+    /// Add a plugin to this profile with the desired state, or change the state of a plugin in this profile.
+    /// This will block until all states have been applied.
+    /// </summary>
+    /// <param name="internalName">The internal name of the plugin.</param>
+    /// <param name="state">Whether or not the plugin should be enabled.</param>
+    /// <param name="apply">Whether or not the current state should immediately be applied.</param>
+    public void AddOrUpdate(string internalName, bool state, bool apply = true)
+    {
+        Debug.Assert(!internalName.IsNullOrEmpty(), "!internalName.IsNullOrEmpty()");
+
+        var existing = this.modelV1.Plugins.FirstOrDefault(x => x.InternalName == internalName);
+        if (existing != null)
+        {
+            existing.IsEnabled = state;
+        }
+        else
+        {
+            this.modelV1.Plugins.Add(new ProfileModelV1.ProfileModelV1Plugin
+            {
+                InternalName = internalName,
+                IsEnabled = state,
+            });
+        }
+
+        // We need to remove this plugin from the default profile, if it declares it.
+        if (!this.IsDefaultProfile && this.manager.DefaultProfile.WantsPlugin(internalName) != null)
+        {
+            this.manager.DefaultProfile.Remove(internalName, false);
+        }
+
+        Service<DalamudConfiguration>.Get().QueueSave();
+
+        if (apply)
+            this.manager.ApplyAllWantStates();
+    }
+
+    /// <summary>
+    /// Remove a plugin from this profile.
+    /// This will block until all states have been applied.
+    /// </summary>
+    /// <param name="internalName">The internal name of the plugin.</param>
+    /// <param name="apply">Whether or not the current state should immediately be applied.</param>
+    public void Remove(string internalName, bool apply = true)
+    {
+        var entry = this.modelV1.Plugins.FirstOrDefault(x => x.InternalName == internalName);
+        if (entry == null)
+            throw new ArgumentException($"No plugin \"{internalName}\" in profile \"{this.Guid}\"");
+
+        if (!this.modelV1.Plugins.Remove(entry))
+            throw new Exception("Couldn't remove plugin from model collection");
+
+        // We need to add this plugin back to the default profile, if we were the last profile to have it.
+        if (!this.manager.IsInAnyProfile(internalName))
+        {
+            if (!this.IsDefaultProfile)
+            {
+                this.manager.DefaultProfile.AddOrUpdate(internalName, entry.IsEnabled, false);
+            }
+            else
+            {
+                throw new Exception("Removed plugin from default profile, but wasn't in any other profile");
+            }
+        }
+
+        Service<DalamudConfiguration>.Get().QueueSave();
+
+        if (apply)
+            this.manager.ApplyAllWantStates();
+    }
+}

Dalamud/Plugin/Internal/Profiles/ProfileCommandHandler.cs

@@ -0,0 +1,164 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Threading.Tasks;
+
+using CheapLoc;
+using Dalamud.Game;
+using Dalamud.Game.Command;
+using Dalamud.Game.Gui;
+using Dalamud.Utility;
+using Serilog;
+
+namespace Dalamud.Plugin.Internal.Profiles;
+
+[ServiceManager.EarlyLoadedService]
+internal class ProfileCommandHandler : IServiceType, IDisposable
+{
+    private readonly CommandManager cmd;
+    private readonly ProfileManager profileManager;
+    private readonly ChatGui chat;
+    private readonly Framework framework;
+
+    private Queue<(string, ProfileOp)> queue = new();
+
+    /// <summary>
+    /// Initializes a new instance of the <see cref="ProfileCommandHandler"/> class.
+    /// </summary>
+    /// <param name="cmd"></param>
+    [ServiceManager.ServiceConstructor]
+    public ProfileCommandHandler(CommandManager cmd, ProfileManager profileManager, ChatGui chat, Framework framework)
+    {
+        this.cmd = cmd;
+        this.profileManager = profileManager;
+        this.chat = chat;
+        this.framework = framework;
+
+        this.cmd.AddHandler("/xlenableprofile", new CommandInfo(this.OnEnableProfile)
+        {
+            HelpMessage = "",
+            ShowInHelp = true,
+        });
+
+        this.cmd.AddHandler("/xldisableprofile", new CommandInfo(this.OnDisableProfile)
+        {
+            HelpMessage = "",
+            ShowInHelp = true,
+        });
+
+        this.cmd.AddHandler("/xltoggleprofile", new CommandInfo(this.OnToggleProfile)
+        {
+            HelpMessage = "",
+            ShowInHelp = true,
+        });
+
+        this.framework.Update += this.FrameworkOnUpdate;
+    }
+
+    private void FrameworkOnUpdate(Framework framework1)
+    {
+        if (this.profileManager.IsBusy)
+            return;
+
+        if (this.queue.TryDequeue(out var op))
+        {
+            var profile = this.profileManager.Profiles.FirstOrDefault(x => x.Name == op.Item1);
+            if (profile == null || profile.IsDefaultProfile)
+                return;
+
+            switch (op.Item2)
+            {
+                case ProfileOp.Enable:
+                    if (!profile.IsEnabled)
+                        profile.SetState(true, false);
+                    break;
+                case ProfileOp.Disable:
+                    if (profile.IsEnabled)
+                        profile.SetState(false, false);
+                    break;
+                case ProfileOp.Toggle:
+                    profile.SetState(!profile.IsEnabled, false);
+                    break;
+                default:
+                    throw new ArgumentOutOfRangeException();
+            }
+
+            if (profile.IsEnabled)
+            {
+                this.chat.Print(Loc.Localize("ProfileCommandsEnabling", "Enabling profile \"{0}\"...").Format(profile.Name));
+            }
+            else
+            {
+                this.chat.Print(Loc.Localize("ProfileCommandsDisabling", "Disabling profile \"{0}\"...").Format(profile.Name));
+            }
+
+            Task.Run(() => this.profileManager.ApplyAllWantStates()).ContinueWith(t =>
+            {
+                if (!t.IsCompletedSuccessfully && t.Exception != null)
+                {
+                    Log.Error(t.Exception, "Could not apply profiles through commands");
+                    this.chat.PrintError(Loc.Localize("ProfileCommandsApplyFailed", "Failed to apply all of your profiles. Please check the console for errors."));
+                }
+                else
+                {
+                    this.chat.Print(Loc.Localize("ProfileCommandsApplySuccess", "Profiles applied."));
+                }
+            });
+        }
+    }
+
+    public void Dispose()
+    {
+        this.cmd.RemoveHandler("/xlenableprofile");
+        this.cmd.RemoveHandler("/xldisableprofile");
+        this.cmd.RemoveHandler("/xltoggleprofile");
+
+        this.framework.Update += this.FrameworkOnUpdate;
+    }
+
+    private void OnEnableProfile(string command, string arguments)
+    {
+        var name = this.ValidateName(arguments);
+        if (name == null)
+            return;
+
+        this.queue.Enqueue((name, ProfileOp.Enable));
+    }
+
+    private void OnDisableProfile(string command, string arguments)
+    {
+        var name = this.ValidateName(arguments);
+        if (name == null)
+            return;
+
+        this.queue.Enqueue((name, ProfileOp.Disable));
+    }
+
+    private void OnToggleProfile(string command, string arguments)
+    {
+        var name = this.ValidateName(arguments);
+        if (name == null)
+            return;
+
+        this.queue.Enqueue((name, ProfileOp.Toggle));
+    }
+
+    private string? ValidateName(string arguments)
+    {
+        var name = arguments.Replace("\"", string.Empty);
+        if (this.profileManager.Profiles.All(x => x.Name != name))
+        {
+            this.chat.PrintError($"No profile like \"{name}\".");
+            return null;
+        }
+
+        return name;
+    }
+
+    private enum ProfileOp
+    {
+        Enable,
+        Disable,
+        Toggle,
+    }
+}

Dalamud/Plugin/Internal/Profiles/ProfileManager.cs

@@ -0,0 +1,356 @@
+using System;
+using System.Collections.Generic;
+using System.Diagnostics;
+using System.Globalization;
+using System.Linq;
+using System.Text.RegularExpressions;
+using System.Threading.Tasks;
+
+using CheapLoc;
+using Dalamud.Configuration.Internal;
+using Dalamud.Logging.Internal;
+using Dalamud.Utility;
+
+namespace Dalamud.Plugin.Internal.Profiles;
+
+/// <summary>
+/// Class responsible for managing plugin profiles.
+/// </summary>
+[ServiceManager.BlockingEarlyLoadedService]
+internal class ProfileManager : IServiceType
+{
+    private static readonly ModuleLog Log = new("PROFMAN");
+    private readonly DalamudConfiguration config;
+
+    private readonly List<Profile> profiles = new();
+
+    private volatile bool isBusy = false;
+
+    /// <summary>
+    /// Initializes a new instance of the <see cref="ProfileManager"/> class.
+    /// </summary>
+    /// <param name="config">Dalamud config.</param>
+    [ServiceManager.ServiceConstructor]
+    public ProfileManager(DalamudConfiguration config)
+    {
+        this.config = config;
+
+        this.LoadProfilesFromConfigInitially();
+    }
+
+    /// <summary>
+    /// Gets the default profile.
+    /// </summary>
+    public Profile DefaultProfile => this.profiles.First(x => x.IsDefaultProfile);
+
+    /// <summary>
+    /// Gets all profiles, including the default profile.
+    /// </summary>
+    public IEnumerable<Profile> Profiles => this.profiles;
+
+    /// <summary>
+    /// Gets a value indicating whether or not the profile manager is busy enabling/disabling plugins.
+    /// </summary>
+    public bool IsBusy => this.isBusy;
+
+    /// <summary>
+    /// Check if any enabled profile wants a specific plugin enabled.
+    /// </summary>
+    /// <param name="internalName">The internal name of the plugin.</param>
+    /// <param name="defaultState">The state the plugin shall be in, if it needs to be added.</param>
+    /// <param name="addIfNotDeclared">Whether or not the plugin should be added to the default preset, if it's not present in any preset.</param>
+    /// <returns>Whether or not the plugin shall be enabled.</returns>
+    public bool GetWantState(string internalName, bool defaultState, bool addIfNotDeclared = true)
+    {
+        var want = false;
+        var wasInAnyProfile = false;
+
+        foreach (var profile in this.profiles)
+        {
+            var state = profile.WantsPlugin(internalName);
+            Log.Verbose("Checking {Name} in {Profile}: {State}", internalName, profile.Guid, state == null ? "null" : state);
+
+            if (state.HasValue)
+            {
+                want = want || (profile.IsEnabled && state.Value);
+                wasInAnyProfile = true;
+            }
+        }
+
+        if (!wasInAnyProfile && addIfNotDeclared)
+        {
+            Log.Warning("{Name} was not in any profile, adding to default with {Default}", internalName, defaultState);
+            this.DefaultProfile.AddOrUpdate(internalName, defaultState, false);
+            return defaultState;
+        }
+
+        return want;
+    }
+
+    /// <summary>
+    /// Check whether a plugin is declared in any profile.
+    /// </summary>
+    /// <param name="internalName">The internal name of the plugin.</param>
+    /// <returns>Whether or not the plugin is in any profile.</returns>
+    public bool IsInAnyProfile(string internalName)
+        => this.profiles.Any(x => x.WantsPlugin(internalName) != null);
+
+    /// <summary>
+    /// Check whether a plugin is only in the default profile.
+    /// A plugin can never be in the default profile if it is in any other profile.
+    /// </summary>
+    /// <param name="internalName">The internal name of the plugin.</param>
+    /// <returns>Whether or not the plugin is in the default profile.</returns>
+    public bool IsInDefaultProfile(string internalName)
+        => this.DefaultProfile.WantsPlugin(internalName) != null;
+
+    /// <summary>
+    /// Add a new profile.
+    /// </summary>
+    /// <returns>The added profile.</returns>
+    public Profile AddNewProfile()
+    {
+        var model = new ProfileModelV1
+        {
+            Guid = Guid.NewGuid(),
+            Name = this.GenerateUniqueProfileName(Loc.Localize("PluginProfilesNewProfile", "New Profile")),
+            IsEnabled = false,
+        };
+
+        this.config.SavedProfiles!.Add(model);
+        this.config.QueueSave();
+
+        var profile = new Profile(this, model, false, false);
+        this.profiles.Add(profile);
+
+        return profile;
+    }
+
+    /// <summary>
+    /// Clone a specified profile.
+    /// </summary>
+    /// <param name="toClone">The profile to clone.</param>
+    /// <returns>The newly cloned profile.</returns>
+    public Profile CloneProfile(Profile toClone)
+    {
+        var newProfile = this.ImportProfile(toClone.Model.Serialize());
+        if (newProfile == null)
+            throw new Exception("New profile was null while cloning");
+
+        return newProfile;
+    }
+
+    /// <summary>
+    /// Import a profile with a sharing string.
+    /// </summary>
+    /// <param name="data">The sharing string to import.</param>
+    /// <returns>The imported profile, or null, if the string was invalid.</returns>
+    public Profile? ImportProfile(string data)
+    {
+        var newModel = ProfileModel.Deserialize(data);
+        if (newModel == null)
+            return null;
+
+        newModel.Guid = Guid.NewGuid();
+        newModel.Name = this.GenerateUniqueProfileName(newModel.Name.IsNullOrEmpty() ? "Unknown Profile" : newModel.Name);
+        if (newModel is ProfileModelV1 modelV1)
+            modelV1.IsEnabled = false;
+
+        this.config.SavedProfiles!.Add(newModel);
+        this.config.QueueSave();
+
+        var profile = new Profile(this, newModel, false, false);
+        this.profiles.Add(profile);
+
+        return profile;
+    }
+
+    /// <summary>
+    /// Go through all profiles and plugins, and enable/disable plugins they want active.
+    /// This will block until all plugins have been loaded/reloaded!
+    /// </summary>
+    public void ApplyAllWantStates()
+    {
+        this.isBusy = true;
+        Log.Information("Getting want states...");
+
+        var wantActive = this.profiles
+                             .Where(x => x.IsEnabled)
+                             .SelectMany(profile => profile.Plugins.Where(plugin => plugin.IsEnabled)
+                                                           .Select(plugin => plugin.InternalName))
+                             .Distinct().ToList();
+
+        foreach (var internalName in wantActive)
+        {
+            Log.Information("\t=> Want {Name}", internalName);
+        }
+
+        Log.Information("Applying want states...");
+
+        var pm = Service<PluginManager>.Get();
+        var tasks = new List<Task>();
+
+        foreach (var installedPlugin in pm.InstalledPlugins)
+        {
+            if (installedPlugin.IsDev)
+                continue;
+
+            var wantThis = wantActive.Contains(installedPlugin.Manifest.InternalName);
+            switch (wantThis)
+            {
+                case true when !installedPlugin.IsLoaded:
+                    if (installedPlugin.ApplicableForLoad)
+                    {
+                        Log.Information("\t=> Enabling {Name}", installedPlugin.Manifest.InternalName);
+                        tasks.Add(installedPlugin.LoadAsync(PluginLoadReason.Installer));
+                    }
+                    else
+                    {
+                        Log.Warning("\t=> {Name} wanted active, but not applicable", installedPlugin.Manifest.InternalName);
+                    }
+
+                    break;
+                case false when installedPlugin.IsLoaded:
+                    Log.Information("\t=> Disabling {Name}", installedPlugin.Manifest.InternalName);
+                    tasks.Add(installedPlugin.UnloadAsync());
+                    break;
+            }
+        }
+
+        // This is probably not ideal... Might need to rethink the error handling strategy for this.
+        try
+        {
+            Task.WaitAll(tasks.ToArray());
+        }
+        catch (Exception e)
+        {
+            Log.Error(e, "Couldn't apply state for one or more plugins");
+        }
+
+        Log.Information("Applied!");
+        this.isBusy = false;
+    }
+
+    /// <summary>
+    /// Delete a profile and re-apply all profiles.
+    /// </summary>
+    /// <param name="profile">The profile to delete.</param>
+    public void DeleteProfile(Profile profile)
+    {
+        Debug.Assert(this.config.SavedProfiles!.Remove(profile.Model), "this.config.SavedProfiles!.Remove(profile.Model)");
+        Debug.Assert(this.profiles.Remove(profile), "this.profiles.Remove(profile)");
+
+        this.ApplyAllWantStates();
+        this.config.QueueSave();
+    }
+
+    private string GenerateUniqueProfileName(string startingWith)
+    {
+        if (this.profiles.All(x => x.Name != startingWith))
+            return startingWith;
+
+        startingWith = Regex.Replace(startingWith, @" \(.* Mix\)", string.Empty);
+
+        while (true)
+        {
+            var newName = $"{startingWith} ({CultureInfo.InvariantCulture.TextInfo.ToTitleCase(Util.GetRandomName())} Mix)";
+
+            if (this.profiles.All(x => x.Name != newName))
+                return newName;
+        }
+    }
+
+    private void LoadProfilesFromConfigInitially()
+    {
+        var needMigration = false;
+        if (this.config.DefaultProfile == null)
+        {
+            this.config.DefaultProfile = new ProfileModelV1();
+            needMigration = true;
+        }
+
+        this.profiles.Add(new Profile(this, this.config.DefaultProfile, true, true));
+
+        if (needMigration)
+        {
+            // Don't think we need this here with the migration logic in GetWantState
+            //this.MigratePluginsIntoDefaultProfile();
+        }
+
+        this.config.SavedProfiles ??= new List<ProfileModel>();
+        foreach (var profileModel in this.config.SavedProfiles)
+        {
+            this.profiles.Add(new Profile(this, profileModel, false, true));
+        }
+
+        this.config.QueueSave();
+    }
+
+    // This duplicates some of the original handling in PM; don't care though
+    /*
+    private void MigratePluginsIntoDefaultProfile()
+    {
+        var pluginDirectory = new DirectoryInfo(Service<DalamudStartInfo>.Get().PluginDirectory!);
+        var pluginDefs = new List<PluginDef>();
+
+        Log.Information($"Now migrating plugins at {pluginDirectory} into profiles");
+
+        // Nothing to migrate
+        if (!pluginDirectory.Exists)
+        {
+            Log.Information("\t=> Plugin directory didn't exist, nothing to migrate");
+            return;
+        }
+
+        // Add installed plugins. These are expected to be in a specific format so we can look for exactly that.
+        foreach (var pluginDir in pluginDirectory.GetDirectories())
+        {
+            var versionsDefs = new List<PluginDef>();
+            foreach (var versionDir in pluginDir.GetDirectories())
+            {
+                try
+                {
+                    var dllFile = new FileInfo(Path.Combine(versionDir.FullName, $"{pluginDir.Name}.dll"));
+                    var manifestFile = LocalPluginManifest.GetManifestFile(dllFile);
+
+                    if (!manifestFile.Exists)
+                        continue;
+
+                    var manifest = LocalPluginManifest.Load(manifestFile);
+                    versionsDefs.Add(new PluginDef(dllFile, manifest, false));
+                }
+                catch (Exception ex)
+                {
+                    Log.Error(ex, "Could not load manifest for installed at {Directory}", versionDir.FullName);
+                }
+            }
+
+            try
+            {
+                pluginDefs.Add(versionsDefs.MaxBy(x => x.Manifest!.EffectiveVersion));
+            }
+            catch (Exception ex)
+            {
+                Log.Error(ex, "Couldn't choose best version for plugin: {Name}", pluginDir.Name);
+            }
+        }
+
+        var defaultProfile = this.DefaultProfile;
+        foreach (var pluginDef in pluginDefs)
+        {
+            if (pluginDef.Manifest == null)
+            {
+                Log.Information($"\t=> Skipping DLL at {pluginDef.DllFile.FullName}, no valid manifest");
+                continue;
+            }
+
+            // OK for migration code
+#pragma warning disable CS0618
+            defaultProfile.AddOrUpdate(pluginDef.Manifest.InternalName, !pluginDef.Manifest.Disabled, false);
+            Log.Information(
+                $"\t=> Added {pluginDef.Manifest.InternalName} to default profile with {!pluginDef.Manifest.Disabled}");
+#pragma warning restore CS0618
+        }
+    }
+    */
+}

Dalamud/Plugin/Internal/Profiles/ProfileModel.cs

@@ -0,0 +1,40 @@
+using System;
+
+using Dalamud.Utility;
+using Newtonsoft.Json;
+
+namespace Dalamud.Plugin.Internal.Profiles;
+
+public abstract class ProfileModel
+{
+    [JsonProperty("id")]
+    public Guid Guid { get; set; } = Guid.Empty;
+
+    [JsonProperty("n")]
+    public string Name { get; set; } = "New Profile";
+
+    public static ProfileModel? Deserialize(string model)
+    {
+        var json = Util.DecompressString(Convert.FromBase64String(model.Substring(3)));
+
+        if (model.StartsWith(ProfileModelV1.SerializedPrefix))
+            return JsonConvert.DeserializeObject<ProfileModelV1>(json);
+
+        throw new ArgumentException("Was not a compressed style model.");
+    }
+
+    public string Serialize()
+    {
+        string prefix;
+        switch (this)
+        {
+            case ProfileModelV1:
+                prefix = ProfileModelV1.SerializedPrefix;
+                break;
+            default:
+                throw new ArgumentOutOfRangeException();
+        }
+
+        return prefix + Convert.ToBase64String(Util.CompressString(JsonConvert.SerializeObject(this)));
+    }
+}

Dalamud/Plugin/Internal/Profiles/ProfileModelV1.cs

@@ -0,0 +1,27 @@
+using System.Collections.Generic;
+using Newtonsoft.Json;
+
+namespace Dalamud.Plugin.Internal.Profiles;
+
+public class ProfileModelV1 : ProfileModel
+{
+    public static string SerializedPrefix => "DP1";
+
+    [JsonProperty("b")]
+    public bool AlwaysEnableOnBoot { get; set; } = false;
+
+    [JsonProperty("e")]
+    public bool IsEnabled { get; set; } = false;
+
+    [JsonProperty("c")]
+    public uint Color { get; set; }
+
+    public List<ProfileModelV1Plugin> Plugins { get; set; } = new();
+
+    public class ProfileModelV1Plugin
+    {
+        public string InternalName { get; set; }
+
+        public bool IsEnabled { get; set; }
+    }
+}

Dalamud/Plugin/Internal/Profiles/ProfilePluginEntry.cs

@@ -0,0 +1,14 @@
+namespace Dalamud.Plugin.Internal.Profiles;
+
+internal class ProfilePluginEntry
+{
+    public ProfilePluginEntry(string internalName, bool state)
+    {
+        this.InternalName = internalName;
+        this.IsEnabled = state;
+    }
+
+    public string InternalName { get; }
+
+    public bool IsEnabled { get; }
+}

Dalamud/Plugin/Internal/Types/LocalPlugin.cs

@@ -14,6 +14,7 @@ using Dalamud.IoC.Internal;
 using Dalamud.Logging.Internal;
 using Dalamud.Plugin.Internal.Exceptions;
 using Dalamud.Plugin.Internal.Loader;
+using Dalamud.Plugin.Internal.Profiles;
 using Dalamud.Utility;
 
 namespace Dalamud.Plugin.Internal.Types;
@@ -199,10 +200,20 @@ internal class LocalPlugin : IDisposable
     /// </summary>
     public bool IsLoaded => this.State == PluginState.Loaded;
 
+    /*
     /// <summary>
     /// Gets a value indicating whether the plugin is disabled.
     /// </summary>
+    [Obsolete("This is no longer accurate, use the profile manager instead.", true)]
     public bool IsDisabled => this.Manifest.Disabled;
+    */
+
+    /// <summary>
+    /// Gets a value indicating whether this plugin is wanted active by any profile.
+    /// INCLUDES the default profile.
+    /// </summary>
+    public bool IsWantedByAnyProfile =>
+        Service<ProfileManager>.Get().GetWantState(this.Manifest.InternalName, false, false);
 
     /// <summary>
     /// Gets a value indicating whether this plugin's API level is out of date.
@@ -238,6 +249,12 @@ internal class LocalPlugin : IDisposable
     /// </summary>
     public bool IsDev => this is LocalDevPlugin;
 
+    /// <summary>
+    /// Gets a value indicating whether this plugin should be allowed to load.
+    /// </summary>
+    public bool ApplicableForLoad => !this.IsBanned && !this.IsDecommissioned && !this.IsOrphaned && !this.IsOutdated
+                                     && !(!this.IsDev && this.State == PluginState.UnloadError) && this.CheckPolicy();
+
     /// <inheritdoc/>
     public void Dispose()
     {
@@ -275,7 +292,6 @@ internal class LocalPlugin : IDisposable
     /// <returns>A task.</returns>
     public async Task LoadAsync(PluginLoadReason reason, bool reloading = false)
     {
-        var configuration = await Service<DalamudConfiguration>.GetAsync();
         var framework = await Service<Framework>.GetAsync();
         var ioc = await Service<ServiceContainer>.GetAsync();
         var pluginManager = await Service<PluginManager>.GetAsync();
@@ -297,6 +313,10 @@ internal class LocalPlugin : IDisposable
                 this.ReloadManifest();
             }
 
+            // If we reload a plugin we don't want to delete it. Makes sense, right?
+            this.Manifest.ScheduledForDeletion = false;
+            this.SaveManifest();
+
             switch (this.State)
             {
                 case PluginState.Loaded:
@@ -329,8 +349,9 @@ internal class LocalPlugin : IDisposable
             if (this.Manifest.DalamudApiLevel < PluginManager.DalamudApiLevel && !pluginManager.LoadAllApiLevels)
                 throw new InvalidPluginOperationException($"Unable to load {this.Name}, incompatible API level");
 
-            if (this.Manifest.Disabled)
-                throw new InvalidPluginOperationException($"Unable to load {this.Name}, disabled");
+            // TODO: should we throw here?
+            if (!this.IsWantedByAnyProfile)
+                Log.Warning("{Name} is loading, but isn't wanted by any profile", this.Name);
 
             if (this.IsOrphaned)
                 throw new InvalidPluginOperationException($"Plugin {this.Name} had no associated repo.");
@@ -547,9 +568,11 @@ internal class LocalPlugin : IDisposable
         await this.LoadAsync(PluginLoadReason.Reload, true);
     }
 
+    /*
     /// <summary>
     /// Revert a disable. Must be unloaded first, does not load.
     /// </summary>
+    [Obsolete("Profile API", true)]
     public void Enable()
     {
         // Allowed: Unloaded, UnloadError
@@ -580,6 +603,7 @@ internal class LocalPlugin : IDisposable
         this.Manifest.ScheduledForDeletion = false;
         this.SaveManifest();
     }
+    */
 
     /// <summary>
     /// Check if anything forbids this plugin from loading.
@@ -602,9 +626,11 @@ internal class LocalPlugin : IDisposable
         return true;
     }
 
+    /*
     /// <summary>
     /// Disable this plugin, must be unloaded first.
     /// </summary>
+    [Obsolete("Profile API", true)]
     public void Disable()
     {
         // Allowed: Unloaded, UnloadError
@@ -631,6 +657,7 @@ internal class LocalPlugin : IDisposable
         this.Manifest.Disabled = true;
         this.SaveManifest();
     }
+    */
 
     /// <summary>
     /// Schedule the deletion of this plugin on next cleanup.
@@ -650,9 +677,9 @@ internal class LocalPlugin : IDisposable
         var manifest = LocalPluginManifest.GetManifestFile(this.DllFile);
         if (manifest.Exists)
         {
-            var isDisabled = this.IsDisabled; // saving the internal state because it could have been deleted
+            //var isDisabled = this.IsDisabled; // saving the internal state because it could have been deleted
             this.Manifest = LocalPluginManifest.Load(manifest);
-            this.Manifest.Disabled = isDisabled;
+            //this.Manifest.Disabled = isDisabled;
 
             this.SaveManifest();
         }

Dalamud/Plugin/Internal/Types/LocalPluginManifest.cs

@@ -28,6 +28,7 @@ internal record LocalPluginManifest : PluginManifest
     /// Gets or sets a value indicating whether the plugin is disabled and should not be loaded.
     /// This value supersedes the ".disabled" file functionality and should not be included in the plugin master.
     /// </summary>
+    [Obsolete("This is merely used for migrations now. Use the profile manager to check if a plugin shall be enabled.")]
     public bool Disabled { get; set; }
 
     /// <summary>

Dalamud/Plugin/Internal/Types/PluginManifest.cs

@@ -162,6 +162,12 @@ internal record PluginManifest
     [JsonProperty]
     public bool CanUnloadAsync { get; init; }
 
+    /// <summary>
+    /// Gets a value indicating whether the plugin supports profiles.
+    /// </summary>
+    [JsonProperty]
+    public bool SupportsProfiles { get; init; } = true;
+
     /// <summary>
     /// Gets a list of screenshot image URLs to show in the plugin installer.
     /// </summary>