From 958ed75a6533fa571a5201542f0afc525ed810fa Mon Sep 17 00:00:00 2001
From: "coderabbitai[bot]"
 <136622811+coderabbitai[bot]@users.noreply.github.com>
Date: Mon, 16 Mar 2026 20:29:47 +0000
Subject: [PATCH] =?UTF-8?q?=F0=9F=93=9D=20CodeRabbit=20Chat:=20Implement?=
 =?UTF-8?q?=20requested=20code=20changes?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../Models/Workspace/ContentTypePriority.cs   | 23 ++++++++++--
 GenHub/GenHub/Common/Views/MainView.axaml     | 35 +++++++++++++++----
 .../Content/Services/ContentStorageService.cs | 26 ++++++++++++--
 3 files changed, 72 insertions(+), 12 deletions(-)

diff --git a/GenHub/GenHub.Core/Models/Workspace/ContentTypePriority.cs b/GenHub/GenHub.Core/Models/Workspace/ContentTypePriority.cs
index 88dd778ad..79570a50e 100644
--- a/GenHub/GenHub.Core/Models/Workspace/ContentTypePriority.cs
+++ b/GenHub/GenHub.Core/Models/Workspace/ContentTypePriority.cs
@@ -1,3 +1,4 @@
+using System;
 using GenHub.Core.Models.Enums;
 
 namespace GenHub.Core.Models.Workspace;
@@ -22,8 +23,26 @@ public static int GetPriority(ContentType contentType)
             ContentType.Patch => 90,               // Patches override base content
             ContentType.GameClient => 50,          // Community executables override official
             ContentType.Addon => 40,               // Addons (maps, etc.)
-            ContentType.GameInstallation => 10,    // Lowest: Base game files
-            _ => 0,                                // Unknown/undefined types
+            ContentType.MapPack => 40,             // Map collections (same tier as Addon)
+            ContentType.Mission => 40,             // Story missions (same tier as Addon)
+            ContentType.Map => 40,                 // Individual maps (same tier as Addon)
+            ContentType.LanguagePack => 30,        // Localization packs
+            ContentType.Skin => 30,                // UI/visual customizations
+            ContentType.Video => 20,               // Video content
+            ContentType.Replay => 20,              // Replay files
+            ContentType.Screensaver => 20,         // Screensaver files
+            ContentType.Executable => 20,          // Standalone executables
+            ContentType.ModdingTool => 20,         // Modding/mapping tools
+            ContentType.ContentBundle => 0,        // Meta: collection of other content
+            ContentType.PublisherReferral => 0,    // Meta: link to publisher content
+            ContentType.ContentReferral => 0,      // Meta: link to specific content
+            ContentType.UnknownContentType => 0,   // Unknown: lowest priority
+            ContentType.GameInstallation => 10,    // Lowest physical: Base game files
+            _ => throw new ArgumentOutOfRangeException(
+                     nameof(contentType),
+                     contentType,
+                     $"ContentType '{contentType}' is not mapped in {nameof(ContentTypePriority)}. " +
+                     "Add an explicit priority entry for this type.")
         };
     }
 
diff --git a/GenHub/GenHub/Common/Views/MainView.axaml b/GenHub/GenHub/Common/Views/MainView.axaml
index 7866c0e14..53521f23c 100644
--- a/GenHub/GenHub/Common/Views/MainView.axaml
+++ b/GenHub/GenHub/Common/Views/MainView.axaml
@@ -93,6 +93,22 @@
 
 
 
+        <!-- Version label button: transparent, no hover/pressed overlay -->
+        <Style Selector="Button.version-label">
+            <Setter Property="Background" Value="Transparent" />
+            <Setter Property="BorderThickness" Value="0" />
+            <Setter Property="Padding" Value="0" />
+            <Setter Property="Template">
+                <ControlTemplate>
+                    <ContentPresenter Content="{TemplateBinding Content}"
+                                      ContentTemplate="{TemplateBinding ContentTemplate}"
+                                      Padding="{TemplateBinding Padding}"
+                                      HorizontalContentAlignment="{TemplateBinding HorizontalContentAlignment}"
+                                      VerticalContentAlignment="{TemplateBinding VerticalContentAlignment}" />
+                </ControlTemplate>
+            </Setter>
+        </Style>
+
         <!-- Header gradient background -->
         <Style Selector="Border.header-background">
             <Setter Property="Background">
@@ -212,12 +228,17 @@
         <Grid Grid.Row="1">
             <ContentControl Content="{Binding CurrentTabViewModel}" />
             <!-- Version label in bottom right corner -->
-            <TextBlock Text="{x:Static constants:AppConstants.DisplayVersion}"
-                       FontSize="9"
-                       Foreground="#808080"
-                       HorizontalAlignment="Right"
-                       VerticalAlignment="Bottom"
-                       Margin="0,0,10,5" />
+            <Button Classes="version-label"
+                    Command="{Binding CopyVersionToClipboardCommand}"
+                    HorizontalAlignment="Right"
+                    VerticalAlignment="Bottom"
+                    Margin="0,0,10,5"
+                    Cursor="Hand"
+                    ToolTip.Tip="Click to copy full version (including git hash) to clipboard">
+                <TextBlock Text="{x:Static constants:AppConstants.FullDisplayVersion}"
+                           FontSize="9"
+                           Foreground="#808080" />
+            </Button>
         </Grid>
     </Grid>
-</UserControl>
+</UserControl>
\ No newline at end of file
diff --git a/GenHub/GenHub/Features/Content/Services/ContentStorageService.cs b/GenHub/GenHub/Features/Content/Services/ContentStorageService.cs
index 8476bb6a4..01327bfef 100644
--- a/GenHub/GenHub/Features/Content/Services/ContentStorageService.cs
+++ b/GenHub/GenHub/Features/Content/Services/ContentStorageService.cs
@@ -408,7 +408,7 @@ public async Task<OperationResult<bool>> RemoveContentAsync(ManifestId manifestI
 
             // Remove source.path mapping file if it exists
             var contentDir = Path.Combine(_storageRoot, DirectoryNames.Data, manifestId.Value);
-            var sourcePathFile = Path.Combine(contentDir, "source.path");
+            var sourcePathFile = Path.Combine(contentDir, FileTypes.SourcePathFileName);
             FileOperationsService.DeleteFileIfExists(sourcePathFile);
 
             // Clean up the data directory if empty
@@ -495,13 +495,18 @@ private async Task<OperationResult<ContentManifest>> StoreManifestOnlyAsync(
 
             // Create source.path mapping if we have a valid source directory
             // This allows GetContentDirectoryAsync to resolve the content location
+            bool contentDirCreatedByThisCall = false;
+            bool sourcePathWrittenByThisCall = false;
             if (!string.IsNullOrWhiteSpace(sourceDirectory) && Directory.Exists(sourceDirectory))
             {
                 var contentDir = Path.Combine(_storageRoot, DirectoryNames.Data, manifest.Id.Value);
+                bool dirAlreadyExisted = Directory.Exists(contentDir);
                 Directory.CreateDirectory(contentDir);
+                contentDirCreatedByThisCall = !dirAlreadyExisted;
 
-                var sourcePathFile = Path.Combine(contentDir, "source.path");
+                var sourcePathFile = Path.Combine(contentDir, FileTypes.SourcePathFileName);
                 await File.WriteAllTextAsync(sourcePathFile, sourceDirectory, cancellationToken);
+                sourcePathWrittenByThisCall = true;
 
                 _logger.LogInformation(
                     "Created source path mapping for {ManifestId}: {SourcePath}",
@@ -534,6 +539,21 @@ private async Task<OperationResult<ContentManifest>> StoreManifestOnlyAsync(
             {
                 FileOperationsService.DeleteFileIfExists(manifestPath);
 
+                // Clean up the content dir created for source.path mapping
+                var contentDir = Path.Combine(_storageRoot, DirectoryNames.Data, manifest.Id.Value);
+                if (contentDirCreatedByThisCall)
+                {
+                    // We created this directory in the current call — safe to remove entirely.
+                    if (Directory.Exists(contentDir))
+                        Directory.Delete(contentDir, recursive: true);
+                }
+                else if (sourcePathWrittenByThisCall && Directory.Exists(contentDir))
+                {
+                    // Directory pre-existed but we wrote source.path this call — only remove that file.
+                    var sourcePathFile = Path.Combine(contentDir, FileTypes.SourcePathFileName);
+                    FileOperationsService.DeleteFileIfExists(sourcePathFile);
+                }
+
                 // Untrack manifest if we failed to save its metadata but had already tracked/refreshed references.
                 await _referenceTracker.UntrackManifestAsync(manifest.Id, CancellationToken.None);
             }
@@ -793,4 +813,4 @@ private async Task<ContentManifest> StoreContentFilesAsync(
 
         return manifest;
     }
-}
+}
\ No newline at end of file