Drop CLI wrapper and stabilize manager ids

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: mamoreau-devolutions

.github/workflows/cli-headless-e2e.yml

@@ -98,11 +98,6 @@ jobs:
           }
           dotnet @args
 
-      - name: Build CLI
-        working-directory: src
-        shell: pwsh
-        run: dotnet build UniGetUI.Cli/UniGetUI.Cli.csproj --no-restore --configuration ${{ env.CONFIGURATION }} --verbosity minimal
-
       - name: Upgrade pip tooling
         shell: pwsh
         run: python -m pip install --upgrade pip setuptools wheel

docs/CLI.md

@@ -10,8 +10,8 @@ This file documents the **public command-line surface** exposed by UniGetUI in t
 ```powershell
 unigetui status
 unigetui app status
-unigetui package search --manager ".NET Tool" --query dotnetsay
-unigetui package install --manager ".NET Tool" --id dotnetsay --version 2.1.4 --scope Global
+unigetui package search --manager dotnet-tool --query dotnetsay
+unigetui package install --manager dotnet-tool --id dotnetsay --version 2.1.4 --scope Global
 unigetui operation wait --id 123 --timeout 300
 ```
 
@@ -51,6 +51,7 @@ Related environment variables:
   - `--source` maps to `--package-source`
 - Boolean options use explicit values such as `--enabled true` or `--wait false`.
 - `--detach` is shorthand for asynchronous package operations (`--wait false`).
+- `--manager` uses stable manager ids, not GUI labels. Current ids: `apt`, `cargo`, `chocolatey`, `dnf`, `dotnet-tool`, `flatpak`, `homebrew`, `npm`, `pacman`, `pip`, `pwsh`, `scoop`, `snap`, `vcpkg`, `winget`, and `winps`.
 
 ## Command reference
 
@@ -67,7 +68,7 @@ Related environment variables:
 | --- | --- | --- | --- |
 | `app status` | None | None | Returns app/session state such as headless mode, page, and supported UI actions. |
 | `app show` | None | None | Shows and focuses the window when a GUI session exists. |
-| `app navigate` | `--page <page>` | `--manager <name>`, `--help-attachment <path>` | Valid pages include `discover`, `updates`, `installed`, `bundles`, `settings`, `managers`, `own-log`, `manager-log`, `operation-history`, `help`, `release-notes`, and `about`. |
+| `app navigate` | `--page <page>` | `--manager <id>`, `--help-attachment <path>` | Valid pages include `discover`, `updates`, `installed`, `bundles`, `settings`, `managers`, `own-log`, `manager-log`, `operation-history`, `help`, `release-notes`, and `about`. |
 | `app quit` | None | None | Gracefully shuts down the selected session, including headless daemons. |
 
 ### Operations
@@ -88,23 +89,23 @@ Related environment variables:
 | Command | Required options | Optional options | Notes |
 | --- | --- | --- | --- |
 | `manager list` | None | None | Lists managers and their automation-relevant capability flags. |
-| `manager maintenance` | `--manager <name>` | None | Returns maintenance metadata for one manager. |
-| `manager reload` | `--manager <name>` | None | Reloads one manager. |
-| `manager set-executable` | `--manager <name>`, `--path <path>` | None | Sets a custom executable override, then reloads the manager. |
-| `manager clear-executable` | `--manager <name>` | None | Clears the custom executable override, then reloads the manager. |
-| `manager action` | `--manager <name>`, `--action <action>` | `--confirm` | Runs a manager-specific maintenance action. |
-| `manager enable` | `--manager <name>` | None | Enables the manager. |
-| `manager disable` | `--manager <name>` | None | Disables the manager. |
-| `manager notifications enable` | `--manager <name>` | None | Enables update notifications for the manager. |
-| `manager notifications disable` | `--manager <name>` | None | Disables update notifications for the manager. |
+| `manager maintenance` | `--manager <id>` | None | Returns maintenance metadata for one manager. |
+| `manager reload` | `--manager <id>` | None | Reloads one manager. |
+| `manager set-executable` | `--manager <id>`, `--path <path>` | None | Sets a custom executable override, then reloads the manager. |
+| `manager clear-executable` | `--manager <id>` | None | Clears the custom executable override, then reloads the manager. |
+| `manager action` | `--manager <id>`, `--action <action>` | `--confirm` | Runs a manager-specific maintenance action. |
+| `manager enable` | `--manager <id>` | None | Enables the manager. |
+| `manager disable` | `--manager <id>` | None | Disables the manager. |
+| `manager notifications enable` | `--manager <id>` | None | Enables update notifications for the manager. |
+| `manager notifications disable` | `--manager <id>` | None | Disables update notifications for the manager. |
 
 ### Sources
 
 | Command | Required options | Optional options | Notes |
 | --- | --- | --- | --- |
-| `source list` | None | `--manager <name>` | Lists sources, optionally filtered to one manager. |
-| `source add` | `--manager <name>`, `--name <source-name>` | `--url <source-url>` | Adds a source. |
-| `source remove` | `--manager <name>`, `--name <source-name>` | `--url <source-url>` | Removes a source. |
+| `source list` | None | `--manager <id>` | Lists sources, optionally filtered to one manager. |
+| `source add` | `--manager <id>`, `--name <source-name>` | `--url <source-url>` | Adds a source. |
+| `source remove` | `--manager <id>`, `--name <source-name>` | `--url <source-url>` | Removes a source. |
 
 ### Settings
 
@@ -139,7 +140,7 @@ Available keys live in:
 | --- | --- | --- | --- |
 | `log app` | None | `--level <n>` | Returns structured application log entries. |
 | `log operations` | None | None | Returns persisted operation history. |
-| `log manager` | None | `--manager <name>`, `--verbose` | Returns manager task logs. |
+| `log manager` | None | `--manager <id>`, `--verbose` | Returns manager task logs. |
 
 ### Backups
 
@@ -163,31 +164,31 @@ Available keys live in:
 | `bundle reset` | None | None | Clears the current in-memory bundle. |
 | `bundle import` | None | `--path <path>`, `--content <text>`, `--format <ubundle\|json\|yaml\|xml>`, `--append` | Imports bundle content from a file or raw content. |
 | `bundle export` | None | `--path <path>` | Exports the current bundle, optionally to disk. |
-| `bundle add` | `--id <package-id>` | `--manager <name>`, `--source <source>`, `--version <version>`, `--scope <scope>`, `--pre-release`, `--selection <search\|installed\|updates\|auto>` | Resolves a package and adds it to the bundle. |
-| `bundle remove` | `--id <package-id>` | `--manager <name>`, `--source <source>`, `--version <version>`, `--scope <scope>`, `--pre-release`, `--selection <mode>` | Removes matching package entries from the bundle. |
+| `bundle add` | `--id <package-id>` | `--manager <id>`, `--source <source>`, `--version <version>`, `--scope <scope>`, `--pre-release`, `--selection <search\|installed\|updates\|auto>` | Resolves a package and adds it to the bundle. |
+| `bundle remove` | `--id <package-id>` | `--manager <id>`, `--source <source>`, `--version <version>`, `--scope <scope>`, `--pre-release`, `--selection <mode>` | Removes matching package entries from the bundle. |
 | `bundle install` | None | `--include-installed true\|false`, `--elevated true\|false`, `--interactive true\|false`, `--skip-hash true\|false` | Installs the bundle through UniGetUI’s shared operation pipeline. |
 
 ### Packages
 
 | Command | Required options | Optional options | Notes |
 | --- | --- | --- | --- |
-| `package search` | `--query <text>` | `--manager <name>`, `--max-results <n>` | Searches packages. |
-| `package details` | `--id <package-id>` | `--manager <name>`, `--source <source>` | Returns the package details payload. |
-| `package versions` | `--id <package-id>` | `--manager <name>`, `--source <source>` | Returns installable versions when supported by the manager. |
-| `package installed` | None | `--manager <name>` | Lists installed packages. |
-| `package updates` | None | `--manager <name>` | Lists available updates. |
-| `package install` | `--id <package-id>` | `--manager <name>`, `--source <source>`, `--version <version>`, `--scope <scope>`, `--pre-release`, `--elevated true\|false`, `--interactive true\|false`, `--skip-hash true\|false`, `--architecture <value>`, `--location <path>`, `--wait true\|false`, `--detach` | Installs a package. Async mode returns an operation id immediately. |
-| `package download` | `--id <package-id>` | `--manager <name>`, `--source <source>`, `--version <version>`, `--scope <scope>`, `--wait true\|false`, `--detach`, `--output <path>` | Downloads a package artifact. |
+| `package search` | `--query <text>` | `--manager <id>`, `--max-results <n>` | Searches packages. |
+| `package details` | `--id <package-id>` | `--manager <id>`, `--source <source>` | Returns the package details payload. |
+| `package versions` | `--id <package-id>` | `--manager <id>`, `--source <source>` | Returns installable versions when supported by the manager. |
+| `package installed` | None | `--manager <id>` | Lists installed packages. |
+| `package updates` | None | `--manager <id>` | Lists available updates. |
+| `package install` | `--id <package-id>` | `--manager <id>`, `--source <source>`, `--version <version>`, `--scope <scope>`, `--pre-release`, `--elevated true\|false`, `--interactive true\|false`, `--skip-hash true\|false`, `--architecture <value>`, `--location <path>`, `--wait true\|false`, `--detach` | Installs a package. Async mode returns an operation id immediately. |
+| `package download` | `--id <package-id>` | `--manager <id>`, `--source <source>`, `--version <version>`, `--scope <scope>`, `--wait true\|false`, `--detach`, `--output <path>` | Downloads a package artifact. |
 | `package reinstall` | `--id <package-id>` | Same options as `package install` | Re-runs installation for an installed package. |
 | `package repair` | `--id <package-id>` | Same options as `package install`, plus `--remove-data true\|false` | Uninstalls then reinstalls the package. |
 | `package update` | `--id <package-id>` | Same options as `package install` | Updates one package. |
-| `package uninstall` | `--id <package-id>` | `--manager <name>`, `--source <source>`, `--scope <scope>`, `--remove-data true\|false`, `--elevated true\|false`, `--interactive true\|false`, `--wait true\|false`, `--detach` | Uninstalls a package. |
+| `package uninstall` | `--id <package-id>` | `--manager <id>`, `--source <source>`, `--scope <scope>`, `--remove-data true\|false`, `--elevated true\|false`, `--interactive true\|false`, `--wait true\|false`, `--detach` | Uninstalls a package. |
 | `package show` | `--id <package-id>`, `--source <source>` | None | Opens the package details UI flow. |
 | `package ignored list` | None | None | Lists ignored-update rules tracked by UniGetUI. |
-| `package ignored add` | `--id <package-id>` | `--manager <name>`, `--version <version>`, `--source <source>` | Adds an ignored-update rule. |
-| `package ignored remove` | `--id <package-id>` | `--manager <name>`, `--version <version>`, `--source <source>` | Removes an ignored-update rule. |
+| `package ignored add` | `--id <package-id>` | `--manager <id>`, `--version <version>`, `--source <source>` | Adds an ignored-update rule. |
+| `package ignored remove` | `--id <package-id>` | `--manager <id>`, `--version <version>`, `--source <source>` | Removes an ignored-update rule. |
 | `package update-all` | None | None | Queues updates for all currently upgradable packages. |
-| `package update-manager` | `--manager <name>` | None | Queues updates for all upgradable packages handled by one manager. |
+| `package update-manager` | `--manager <id>` | None | Queues updates for all upgradable packages handled by one manager. |
 
 ## Headless behavior
 

docs/IPC.md

@@ -147,7 +147,7 @@ All request bodies use **camelCase** JSON.
 
 | Field | Type | Meaning |
 | --- | --- | --- |
-| `managerName` | string | Required manager name |
+| `managerName` | string | Required stable manager id |
 | `action` | string | Manager action name for `/action` |
 | `path` | string | Custom executable path for `/executable/set` |
 | `confirm` | boolean | Confirmation flag for destructive actions |
@@ -185,7 +185,7 @@ All request bodies use **camelCase** JSON.
 | Field | Type | Meaning |
 | --- | --- | --- |
 | `packageId` | string | Package identifier |
-| `managerName` | string | Manager display name |
+| `managerName` | string | Stable manager id |
 | `packageSource` | string | Source/feed name |
 | `version` | string | Requested version |
 | `scope` | string | Requested scope |
@@ -210,7 +210,7 @@ These keys are used by package-related endpoints such as install, update, uninst
 | Query key | Meaning |
 | --- | --- |
 | `packageId` | Package identifier |
-| `manager` | Manager display name |
+| `manager` | Stable manager id |
 | `packageSource` | Source/feed name |
 | `version` | Requested version |
 | `scope` | Install scope |

src/UniGetUI.Avalonia.slnx

@@ -225,10 +225,6 @@
       <Platform Solution="*|x64" Project="x64" />
     </Project>
   </Folder>
-  <Project Path="UniGetUI.Cli/UniGetUI.Cli.csproj">
-    <Platform Solution="*|arm64" Project="arm64" />
-    <Platform Solution="*|x64" Project="x64" />
-  </Project>
   <Folder Name="/UniGetUI.PackageEngine.Tests/">
     <Project Path="UniGetUI.PackageEngine.Tests/UniGetUI.PackageEngine.Tests.csproj">
       <Platform Solution="*|arm64" Project="arm64" />

src/UniGetUI.Avalonia/Infrastructure/AvaloniaBootstrapper.cs

@@ -306,7 +306,7 @@ private static IpcCommandResult QuitApp()
         }
 
         return PEInterface.Managers.FirstOrDefault(manager =>
-            manager.Name.Equals(managerName, StringComparison.OrdinalIgnoreCase))
+            manager.Id.Equals(managerName, StringComparison.OrdinalIgnoreCase))
             ?? throw new InvalidOperationException(
                 $"Unknown manager \"{managerName}\"."
             );

src/UniGetUI.Avalonia/Infrastructure/AvaloniaPackageOperationHelper.cs

@@ -42,7 +42,7 @@ public static async Task UpdateAllAsync()
     public static async Task UpdateAllForManagerAsync(string managerName)
     {
         foreach (var pkg in UpgradablePackagesLoader.Instance.Packages
-            .Where(p => p.Manager.Name == managerName || p.Manager.DisplayName == managerName)
+            .Where(p => p.Manager.Id == managerName)
             .ToList())
         {
             if (pkg.Tag is PackageTag.BeingProcessed or PackageTag.OnQueue) continue;

src/UniGetUI.Avalonia/Views/SoftwarePages/PackageBundlesPage.cs

@@ -466,7 +466,7 @@ public static IPackage DeserializePackage(SerializablePackage raw)
         IPackageManager? manager = null;
         foreach (var m in PEInterface.Managers)
         {
-            if (m.Name == raw.ManagerName || m.DisplayName == raw.ManagerName)
+            if (m.Id == raw.ManagerName || m.Name == raw.ManagerName || m.DisplayName == raw.ManagerName)
             { manager = m; break; }
         }
 

src/UniGetUI.Cli/Program.cs

@@ -354,7 +354,7 @@ private static async Task<IpcBundlePackageInfo> ToBundlePackageInfoAsync(IPackag
                     : serialized.InstallationOptions.InstallationScope,
                 PreRelease = serialized.InstallationOptions.PreRelease,
                 Source = imported.Source.AsString_DisplayName,
-                Manager = imported.Manager.Name,
+                Manager = IpcManagerSettingsApi.GetPublicManagerId(imported.Manager),
                 IsCompatible = true,
                 IsInstalled = imported.Tag == PackageTag.AlreadyInstalled,
                 IsUpgradable = imported.Tag == PackageTag.IsUpgradable || imported.IsUpgradable,
@@ -371,7 +371,7 @@ private static async Task<IpcBundlePackageInfo> ToBundlePackageInfoAsync(IPackag
                 Version = serialized.Version,
                 DisplayVersion = invalid.VersionString,
                 Source = invalid.SourceAsString,
-                Manager = invalid.Manager.Name,
+                Manager = IpcManagerSettingsApi.GetPublicManagerId(invalid.Manager),
                 IsCompatible = false,
                 IsInstalled = false,
                 IsUpgradable = false,
@@ -385,7 +385,7 @@ private static async Task<IpcBundlePackageInfo> ToBundlePackageInfoAsync(IPackag
             Version = package.VersionString,
             DisplayVersion = package.VersionString,
             Source = package.Source.AsString_DisplayName,
-            Manager = package.Manager.Name,
+            Manager = IpcManagerSettingsApi.GetPublicManagerId(package.Manager),
             IsCompatible = !package.Source.IsVirtualManager,
             IsInstalled = package.Tag == PackageTag.AlreadyInstalled,
             IsUpgradable = package.Tag == PackageTag.IsUpgradable || package.IsUpgradable,
@@ -430,11 +430,7 @@ IpcBundlePackageRequest request
             return false;
         }
 
-        if (
-            !string.IsNullOrWhiteSpace(request.ManagerName)
-            && !package.Manager.Name.Equals(request.ManagerName, StringComparison.OrdinalIgnoreCase)
-            && !package.Manager.DisplayName.Equals(request.ManagerName, StringComparison.OrdinalIgnoreCase)
-        )
+        if (!IpcManagerSettingsApi.MatchesManagerId(package.Manager, request.ManagerName))
         {
             return false;
         }
@@ -703,17 +699,7 @@ BundleFormatType format
 
     private static IPackage DeserializePackage(SerializablePackage raw)
     {
-        IPackageManager? manager = null;
-        foreach (var candidate in PEInterface.Managers)
-        {
-            if (
-                candidate.Name == raw.ManagerName || candidate.DisplayName == raw.ManagerName
-            )
-            {
-                manager = candidate;
-                break;
-            }
-        }
+        IPackageManager? manager = IpcManagerSettingsApi.ResolveImportedManager(raw.ManagerName);
 
         IManagerSource? source;
         if (manager?.Capabilities.SupportsCustomSources == true)