Try to fix CI build

This turned out to be a hassle

src/Artemis.Storage/Entities/Workshop/EntryEntity.cs

@@ -14,6 +14,7 @@ public class EntryEntity
     public int EntryType { get; set; }
 
     public string Author { get; set; } = string.Empty;
+    public bool IsOfficial { get; set; }
     public string Name { get; set; } = string.Empty;
     public string Summary { get; set; } = string.Empty;
     public long Downloads { get; set; }

src/Artemis.Storage/Migrations/20240722084220_AutoUpdating.Designer.cs

@@ -11,8 +11,8 @@ using Microsoft.EntityFrameworkCore.Storage.ValueConversion;
 namespace Artemis.Storage.Migrations
 {
     [DbContext(typeof(ArtemisDbContext))]
-    [Migration("20240706131336_ExpandInstalledEntry")]
+    [Migration("20240722084220_AutoUpdating")]
-    partial class ExpandInstalledEntry
+    partial class AutoUpdating
     {
         /// <inheritdoc />
         protected override void BuildTargetModel(ModelBuilder modelBuilder)
@@ -264,6 +264,9 @@ namespace Artemis.Storage.Migrations
                     b.Property<DateTimeOffset>("InstalledAt")
                         .HasColumnType("TEXT");
 
+                    b.Property<bool>("IsOfficial")
+                        .HasColumnType("INTEGER");
+
                     b.Property<long?>("LatestReleaseId")
                         .HasColumnType("INTEGER");
 

src/Artemis.Storage/Migrations/20240722084220_AutoUpdating.cs

@@ -6,7 +6,7 @@ using Microsoft.EntityFrameworkCore.Migrations;
 namespace Artemis.Storage.Migrations
 {
     /// <inheritdoc />
-    public partial class ExpandInstalledEntry : Migration
+    public partial class AutoUpdating : Migration
     {
         /// <inheritdoc />
         protected override void Up(MigrationBuilder migrationBuilder)
@@ -38,6 +38,13 @@ namespace Artemis.Storage.Migrations
                 nullable: false,
                 defaultValue: 0L);
 
+            migrationBuilder.AddColumn<bool>(
+                name: "IsOfficial",
+                table: "Entries",
+                type: "INTEGER",
+                nullable: false,
+                defaultValue: false);
+
             migrationBuilder.AddColumn<long>(
                 name: "LatestReleaseId",
                 table: "Entries",
@@ -87,6 +94,10 @@ namespace Artemis.Storage.Migrations
                 name: "Downloads",
                 table: "Entries");
 
+            migrationBuilder.DropColumn(
+                name: "IsOfficial",
+                table: "Entries");
+
             migrationBuilder.DropColumn(
                 name: "LatestReleaseId",
                 table: "Entries");

src/Artemis.Storage/Migrations/ArtemisDbContextModelSnapshot.cs

@@ -261,6 +261,9 @@ namespace Artemis.Storage.Migrations
                     b.Property<DateTimeOffset>("InstalledAt")
                         .HasColumnType("TEXT");
 
+                    b.Property<bool>("IsOfficial")
+                        .HasColumnType("INTEGER");
+
                     b.Property<long?>("LatestReleaseId")
                         .HasColumnType("INTEGER");
 

src/Artemis.UI/Screens/Workshop/Library/Tabs/SubmissionsTabItemView.axaml

@@ -39,7 +39,7 @@
                 <TextBlock Grid.Row="0" Margin="0 0 0 5" TextTrimming="CharacterEllipsis">
                     <Run Classes="h5" Text="{CompiledBinding Entry.Name, FallbackValue=Title}" />
                     <Run Classes="subtitle">by you</Run>
-                    <Run Classes="subtitle" Text="{CompiledBinding Emoji}" />
+                    <Run Text="{CompiledBinding Emoji}" />
                 </TextBlock>
                 <TextBlock Grid.Row="1"
                            Classes="subtitle"

src/Artemis.UI/Screens/Workshop/Library/Tabs/SubmissionsTabItemViewModel.cs

@@ -11,7 +11,7 @@ namespace Artemis.UI.Screens.Workshop.Library.Tabs;
 
 public class SubmissionsTabItemViewModel : ViewModelBase
 {
-    private static readonly string[] Emojis = ["❤️", "🧡", "💛", "💚", "💙", "💜", "🤍", "💔", "❣️", "💕", "💞", "💓", "💗", "💖", "💘", "💝", "😍", "🥰"];
+    private static readonly string[] Emojis = ["❤️", "🧡", "💛", "💚", "💙", "💜", "💔", "❣️", "💕", "💞", "💓", "💗", "💖", "💘", "💝", "😍", "🥰"];
     private readonly IRouter _router;
 
     public SubmissionsTabItemViewModel(IGetSubmittedEntries_SubmittedEntries entry, IRouter router)

src/Artemis.UI/Services/Updating/WorkshopUpdateService.cs

@@ -4,7 +4,6 @@ using System.Threading.Tasks;
 using Artemis.Core;
 using Artemis.Core.Services;
 using Artemis.UI.Services.Interfaces;
-using Artemis.UI.Shared.Services;
 using Artemis.UI.Shared.Utilities;
 using Artemis.WebClient.Workshop;
 using Artemis.WebClient.Workshop.Handlers.InstallationHandlers;
@@ -19,13 +18,11 @@ public class WorkshopUpdateService : IWorkshopUpdateService
 {
     private readonly ILogger _logger;
     private readonly IWorkshopClient _client;
-    private readonly INotificationService _notificationService;
     private readonly IWorkshopService _workshopService;
     private readonly Lazy<IUpdateNotificationProvider> _updateNotificationProvider;
     private readonly PluginSetting<bool> _showNotifications;
 
-    public WorkshopUpdateService(ILogger logger, IWorkshopClient client, IWorkshopService workshopService, ISettingsService settingsService,
+    public WorkshopUpdateService(ILogger logger, IWorkshopClient client, IWorkshopService workshopService, ISettingsService settingsService, Lazy<IUpdateNotificationProvider> updateNotificationProvider)
-        Lazy<IUpdateNotificationProvider> updateNotificationProvider)
     {
         _logger = logger;
         _client = client;
@@ -57,28 +54,42 @@ public class WorkshopUpdateService : IWorkshopUpdateService
             _updateNotificationProvider.Value.ShowWorkshopNotification(updatedEntries);
     }
 
-    public async Task<bool> AutoUpdateEntry(InstalledEntry entry)
+    public async Task<bool> AutoUpdateEntry(InstalledEntry installedEntry)
     {
         // Query the latest version
-        IOperationResult<IGetEntryLatestReleaseByIdResult> latestReleaseResult = await _client.GetEntryLatestReleaseById.ExecuteAsync(entry.Id);
+        IOperationResult<IGetEntryLatestReleaseByIdResult> latestReleaseResult = await _client.GetEntryLatestReleaseById.ExecuteAsync(installedEntry.Id);
-
+        IEntrySummary? entry = latestReleaseResult.Data?.Entry?.LatestRelease?.Entry;
+        if (entry == null)
+            return false;
         if (latestReleaseResult.Data?.Entry?.LatestRelease is not IRelease latestRelease)
             return false;
-        if (latestRelease.Id == entry.ReleaseId)
+        if (latestRelease.Id == installedEntry.ReleaseId)
             return false;
 
         _logger.Information("Auto-updating entry {Entry} to version {Version}", entry, latestRelease.Version);
 
+        try
+        {
             EntryInstallResult updateResult = await _workshopService.InstallEntry(entry, latestRelease, new Progress<StreamProgress>(), CancellationToken.None);
 
             // This happens during installation too but not on our reference of the entry
             if (updateResult.IsSuccess)
-            entry.ApplyRelease(latestRelease);
+                installedEntry.ApplyRelease(latestRelease);
 
-        _logger.Information("Auto-update result: {Result}", updateResult);
+            if (updateResult.IsSuccess)
+                _logger.Information("Auto-update successful for entry {Entry}", entry);
+            else
+                _logger.Warning("Auto-update failed for entry {Entry}: {Message}", entry, updateResult.Message);
 
             return updateResult.IsSuccess;
         }
+        catch (Exception e)
+        {
+            _logger.Warning(e, "Auto-update failed for entry {Entry}", entry);
+        }
+
+        return false;
+    }
 
     /// <inheritdoc />
     public void DisableNotifications()

src/Artemis.WebClient.Workshop/Models/InstalledEntry.cs

@@ -6,22 +6,22 @@ using Artemis.Storage.Entities.Workshop;
 
 namespace Artemis.WebClient.Workshop.Models;
 
-public class InstalledEntry : CorePropertyChanged, IEntrySummary
+public class InstalledEntry : CorePropertyChanged
 {
-    private Dictionary<string, JsonNode> _metadata = new();
-    private long _id;
     private string _author;
-    private bool _isOfficial;
+    private bool _autoUpdate;
-    private string _name;
+    private IReadOnlyList<InstalledEntryCategory> _categories;
-    private string _summary;
-    private EntryType _entryType;
-    private long _downloads;
     private DateTimeOffset _createdAt;
+    private long _downloads;
+    private EntryType _entryType;
+    private long _id;
+    private bool _isOfficial;
     private long? _latestReleaseId;
-    private IReadOnlyList<IGetDependantEntries_Entries_Items_Categories> _categories;
+    private Dictionary<string, JsonNode> _metadata = new();
+    private string _name;
     private long _releaseId;
     private string _releaseVersion = string.Empty;
-    private bool _autoUpdate;
+    private string _summary;
 
     internal InstalledEntry(EntryEntity entity)
     {
@@ -40,7 +40,6 @@ public class InstalledEntry : CorePropertyChanged, IEntrySummary
         AutoUpdate = true;
     }
 
-    internal EntryEntity Entity { get; }
     public DateTimeOffset InstalledAt { get; set; }
 
     public long ReleaseId
@@ -61,55 +60,82 @@ public class InstalledEntry : CorePropertyChanged, IEntrySummary
         set => SetAndNotify(ref _autoUpdate, value);
     }
 
-    internal void Load()
+    public long Id
     {
-        Id = Entity.EntryId;
+        get => _id;
-        Author = Entity.Author;
+        private set => SetAndNotify(ref _id, value);
-        Name = Entity.Name;
-        Summary = Entity.Summary;
-        EntryType = (EntryType) Entity.EntryType;
-        Downloads = Entity.Downloads;
-        CreatedAt = Entity.CreatedAt;
-        LatestReleaseId = Entity.LatestReleaseId;
-        Categories = Entity.Categories?.Select(c => new GetDependantEntries_Entries_Items_Categories_Category(c.Name, c.Icon)).ToList() ?? [];
-        
-        ReleaseId = Entity.ReleaseId;
-        ReleaseVersion = Entity.ReleaseVersion;
-        InstalledAt = Entity.InstalledAt;
-        AutoUpdate = Entity.AutoUpdate;
-
-        _metadata = Entity.Metadata != null ? new Dictionary<string, JsonNode>(Entity.Metadata) : [];
     }
 
-    internal void Save()
+    public string Author
     {
-        Entity.EntryId = Id;
+        get => _author;
-        Entity.EntryType = (int) EntryType;
+        private set => SetAndNotify(ref _author, value);
-
-        Entity.Author = Author;
-        Entity.Name = Name;
-        Entity.Summary = Summary;
-        Entity.Downloads = Downloads;
-        Entity.CreatedAt = CreatedAt;
-        Entity.LatestReleaseId = LatestReleaseId;
-        Entity.Categories = Categories.Select(c => new EntryCategoryEntity(c.Name, c.Icon)).ToList();
-
-        Entity.ReleaseId = ReleaseId;
-        Entity.ReleaseVersion = ReleaseVersion;
-        Entity.InstalledAt = InstalledAt;
-        Entity.AutoUpdate = AutoUpdate;
-        
-        Entity.Metadata = new Dictionary<string, JsonNode>(_metadata);
     }
 
+    public bool IsOfficial
+    {
+        get => _isOfficial;
+        private set => SetAndNotify(ref _isOfficial, value);
+    }
+
+    public string Name
+    {
+        get => _name;
+        private set => SetAndNotify(ref _name, value);
+    }
+
+    public string Summary
+    {
+        get => _summary;
+        private set => SetAndNotify(ref _summary, value);
+    }
+
+    public EntryType EntryType
+    {
+        get => _entryType;
+        private set => SetAndNotify(ref _entryType, value);
+    }
+
+    public long Downloads
+    {
+        get => _downloads;
+        private set => SetAndNotify(ref _downloads, value);
+    }
+
+    public DateTimeOffset CreatedAt
+    {
+        get => _createdAt;
+        private set => SetAndNotify(ref _createdAt, value);
+    }
+
+    public long? LatestReleaseId
+    {
+        get => _latestReleaseId;
+        private set => SetAndNotify(ref _latestReleaseId, value);
+    }
+
+    public IReadOnlyList<InstalledEntryCategory> Categories
+    {
+        get => _categories;
+        private set => SetAndNotify(ref _categories, value);
+    }
+
+    internal EntryEntity Entity { get; }
+
     /// <summary>
     ///     Gets the metadata value associated with the specified key.
     /// </summary>
     /// <param name="key">The key of the value to get.</param>
-    /// <param name="value">When this method returns, contains the value associated with the specified key, if the key is found and of type <typeparamref name="T"/>;
+    /// <param name="value">
-    /// otherwise, the default value for the type of the value parameter. This parameter is passed uninitialized.</param>
+    ///     When this method returns, contains the value associated with the specified key, if the key is found and of type
+    ///     <typeparamref name="T" />;
+    ///     otherwise, the default value for the type of the value parameter. This parameter is passed uninitialized.
+    /// </param>
     /// <typeparam name="T">The type of the value.</typeparam>
-    /// <returns><see langword="true"/> if the metadata contains an element with the specified key; otherwise, <see langword="false"/>.</returns>
+    /// <returns>
+    ///     <see langword="true" /> if the metadata contains an element with the specified key; otherwise,
+    ///     <see langword="false" />.
+    /// </returns>
     public bool TryGetMetadata<T>(string key, [NotNullWhen(true)] out T? value)
     {
         if (!_metadata.TryGetValue(key, out JsonNode? element))
@@ -183,86 +209,55 @@ public class InstalledEntry : CorePropertyChanged, IEntrySummary
         Downloads = entry.Downloads;
         CreatedAt = entry.CreatedAt;
         LatestReleaseId = entry.LatestReleaseId;
-        Categories = entry.Categories;
+        Categories = entry.Categories.Select(c => new InstalledEntryCategory(c.Name, c.Icon)).ToList();
     }
 
-    #region Implementation of IEntrySummary
 
-    /// <inheritdoc />
-    public long Id
-    {
-        get => _id;
-        private set => SetAndNotify(ref _id, value);
-    }
-
-    /// <inheritdoc />
-    public string Author
-    {
-        get => _author;
-        private set => SetAndNotify(ref _author, value);
-    }
-
-    /// <inheritdoc />
-    public bool IsOfficial
-    {
-        get => _isOfficial;
-        private set => SetAndNotify(ref _isOfficial, value);
-    }
-
-    /// <inheritdoc />
-    public string Name  
-    {
-        get => _name;
-        private set => SetAndNotify(ref _name, value);
-    }
-
-    /// <inheritdoc />
-    public string Summary
-    {
-        get => _summary;
-        private set => SetAndNotify(ref _summary, value);
-    }
-
-    /// <inheritdoc />
-    public EntryType EntryType  
-    {
-        get => _entryType;
-        private set => SetAndNotify(ref _entryType, value);
-    }
-
-    /// <inheritdoc />
-    public long Downloads
-    {
-        get => _downloads;
-        private set => SetAndNotify(ref _downloads, value);
-    }
-
-    /// <inheritdoc />
-    public DateTimeOffset CreatedAt
-    {
-        get => _createdAt;
-        private set => SetAndNotify(ref _createdAt, value);
-    }
-
-    /// <inheritdoc />
-    public long? LatestReleaseId
-    {
-        get => _latestReleaseId;
-        private set => SetAndNotify(ref _latestReleaseId, value);
-    }
-
-    /// <inheritdoc />
-    public IReadOnlyList<IGetDependantEntries_Entries_Items_Categories> Categories
-    {
-        get => _categories;
-        private set => SetAndNotify(ref _categories, value);
-    }
-
-    #endregion
-
-    /// <inheritdoc />
     public override string ToString()
     {
         return $"[{EntryType}] {Id} - {Name}";
     }
+
+    internal void Load()
+    {
+        Id = Entity.EntryId;
+        Author = Entity.Author;
+        IsOfficial = Entity.IsOfficial;
+        Name = Entity.Name;
+        Summary = Entity.Summary;
+        EntryType = (EntryType) Entity.EntryType;
+        Downloads = Entity.Downloads;
+        CreatedAt = Entity.CreatedAt;
+        LatestReleaseId = Entity.LatestReleaseId;
+        Categories = Entity.Categories?.Select(c => new InstalledEntryCategory(c.Name, c.Icon)).ToList() ?? [];
+
+        ReleaseId = Entity.ReleaseId;
+        ReleaseVersion = Entity.ReleaseVersion;
+        InstalledAt = Entity.InstalledAt;
+        AutoUpdate = Entity.AutoUpdate;
+
+        _metadata = Entity.Metadata != null ? new Dictionary<string, JsonNode>(Entity.Metadata) : [];
+    }
+
+    internal void Save()
+    {
+        Entity.EntryId = Id;
+        Entity.EntryType = (int) EntryType;
+
+        Entity.Author = Author;
+        Entity.IsOfficial = IsOfficial;
+        Entity.Name = Name;
+        Entity.Summary = Summary;
+        Entity.Downloads = Downloads;
+        Entity.CreatedAt = CreatedAt;
+        Entity.LatestReleaseId = LatestReleaseId;
+        Entity.Categories = Categories.Select(c => new EntryCategoryEntity(c.Name, c.Icon)).ToList();
+
+        Entity.ReleaseId = ReleaseId;
+        Entity.ReleaseVersion = ReleaseVersion;
+        Entity.InstalledAt = InstalledAt;
+        Entity.AutoUpdate = AutoUpdate;
+
+        Entity.Metadata = new Dictionary<string, JsonNode>(_metadata);
+    }
 }

src/Artemis.WebClient.Workshop/Models/InstalledEntryCategory.cs

@@ -0,0 +1,14 @@
+namespace Artemis.WebClient.Workshop.Models;
+
+public class InstalledEntryCategory
+{
+    public InstalledEntryCategory(string name, string icon)
+    {
+        Name = name;
+        Icon = icon;
+    }
+
+    public string Name { get; }
+
+    public string Icon { get; }
+}