Full Linux support (#42)

* fix debug disposal

* Handle null `TextureData` during `GuiDialog` initialization.

* Engine support for finding files on linux systems the way the game would do it wiht abstractions layers like Wine or Valve Proton

* Add support for the `--useDefaultBaseline` option and improve baseline selection logic

* Remove obsolete filesystem abstraction utilities for directory and file globbing.

* Introduce `PetroglyphFileSystem` abstraction for cross-platform filesystem operations.

* Refactor `PetroglyphXmlFileParserBase`: remove redundant `IServiceProvider` field, optimize path handling with OS-specific logic.

* Add `PG.StarWarsGame.Engine.FileSystem` project to solution

* Update dependencies and add reference to `PG.StarWarsGame.Engine.FileSystem` project

* Refactor: Replace generic filesystem abstraction with `PetroglyphFileSystem` for consistent, cross-platform path handling across the engine.

* Refactor: Simplify `NormalizePath` logic and remove unused return value

* Refactor: Use `ValueStringBuilder` in `PathStartsWithDataDirectory` for performance and normalize input paths

* Refactor: Replace direct filename handling with `NormalizeFileName` for consistency and re-enable particle name mismatch validation

* Refactor path normalization to use PathNormalizer

Replaces custom path normalization logic in PetroglyphFileSystem with PathNormalizer from AnakinRaW.CommonUtilities. Moves LinuxDirectorySeparatorNormalizeOptions to a static readonly field. Cleans up unused usings and retains legacy code as comments for reference.

* Rename StringExtensions and add Enum.Parse<T> extension

Renamed StringExtensions to Extensions. Added a generic Enum.Parse<T> extension method for NETFRAMEWORK builds to simplify enum parsing from strings.

* Refactor AudioFileVerifier: move FriendlyName, remove Verify

Moved FriendlyName override to class body and removed the
Verify method implementation from AudioFileVerifier.

* Add comprehensive unit tests for `PG.StarWarsGame.Engine.FileSystem` and include test project in solution

* Remove unused test case for volume file names and update path handling in `FileExists` test

* Optimize case-insensitive file existence check in `PetroglyphFileSystem` and add additional test cases

* Update README with detailed mod verification examples for Windows and Linux

* Add Linux-specific build target to release workflow

* Fix mod paths in Linux instructions

* revert to old impl

* Add Linux-specific tests for case-insensitive file existence handling in `PetroglyphFileSystem`

* Update README: include `--engine` parameter in examples and add Linux baseline creation example

* update dependencies

* Refactor file path handling in GetFileInfoFromMasterMeg

Check filePath length before allocating ValueStringBuilder to improve efficiency. Update warning log to use filePath.ToString() for overlong paths. Reorganize normalization logic for clarity and ensure proper resource disposal.

* rename method to express intent

* formatting

* Update path normalization for Windows-like behavior on Linux

Replaced LinuxDirectorySeparatorNormalizeOptions with PGFileSystemDirectorySeparatorNormalizeOptions to ensure consistent Windows-like path handling on Linux. Updated class documentation to clarify behavior. Refactored IsDirectorySeparator to use defined separator constants for improved clarity.

* documentation

* Refactor path handling to use platform-specific separators

ModPaths and AdditionalFallbackPath are now single string properties using the platform-specific path separator, instead of IList<string>. SettingsBuilder splits and normalizes these paths accordingly. Help texts are updated, and unit tests are added to verify correct path parsing and normalization using a mock file system.

* fix reporting of missing texutres

* add new algorithm to evaluate

* Handle "none" GUI textures and unify repo lookups

Refactored texture existence checks by introducing IsNone and GuiSpecialTextureExists helpers. Special cases for "none" textures now avoid false warnings and are consistently cached. Centralized repository lookup logic for Scanlines, ButtonMiddle, and FrameBackground types. Improved error messages by clarifying origin labeling and adjusting error context.

* update sub

* update verifiers

* update sub

* update deps

* fix false positive for "none" textures

* exceptions stack traces from a verifystep now are now shown instead of the unusefull pipeline trace

* add an alternate wine-style file lookup algorithm

* more consistent path handling

* fix and test dot segements handling

* update and fix wine file lookup

* add linux fs path caching

* spacing

* search animations based on relative path instead of absolute path.

* fix potential memory leaks because of undisposed VSB

* implement file system strategies to find files in a case-insensitive manner

* add design spec for live virtual file-exists strategy

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* window stategy is default engine for virtual stategy on windows

* fix model corrupted findings are incorrectly reported if the the engine had crc collision and loaded a different file.

* add implementation plan for live virtual file-exists strategy

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* remove design spec and implementation plan for live virtual file-exists strategy

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* rename to baseDirectory

* conditional using

* add some test gaps

* update baseline

---------

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: AnakinRaW

.github/workflows/release.yml

@@ -47,6 +47,8 @@ jobs:
       - name: Create Net Core Release
         # use publish for .NET Core
         run:  dotnet publish ${{ env.TOOL_PROJ_PATH }} --configuration Release -f net10.0 --output ./releases/net10.0 /p:DebugType=None /p:DebugSymbols=false
+      - name: Create Linux Release
+        run:  dotnet publish ${{ env.TOOL_PROJ_PATH }} --configuration Release -f net10.0 --runtime linux-x64 --self-contained true --output ./releases/linux-x64 /p:DebugType=None /p:DebugSymbols=false
       - name: Upload a Build Artifact
         uses: actions/upload-artifact@v7
         with:
@@ -117,4 +119,5 @@ jobs:
           generate_release_notes: true
           files: | 
             ./releases/net481/ModVerify.exe
-            ./releases/ModVerify-Net10.zip
+            ./releases/ModVerify-Net10.zip
+            ./releases/linux-x64/ModVerify

Directory.Build.props

@@ -33,7 +33,7 @@
 
 	<ItemGroup>
 		<PackageReference Update="SauceControl.InheritDoc" Version="2.0.2" />
-		<PackageReference Include="Microsoft.SourceLink.GitHub" Version="10.0.201" PrivateAssets="All"/>
+		<PackageReference Include="Microsoft.SourceLink.GitHub" Version="10.0.203" PrivateAssets="All"/>
 		<PackageReference Include="Nerdbank.GitVersioning" Condition="!Exists('packages.config')">
 			<PrivateAssets>all</PrivateAssets>
 			<Version>3.9.50</Version>

ModVerify.slnx

@@ -17,6 +17,8 @@
     <Project Path="modules/ModdingToolBase/src/Updater/ExternalUpdater.Core/ExternalUpdater.Core.csproj" />
   </Folder>
   <Folder Name="/PetroglyphTools/">
+    <Project Path="src/PetroglyphTools/PG.StarWarsGame.Engine.FileSystem.Test/PG.StarWarsGame.Engine.FileSystem.Test.csproj" />
+    <Project Path="src/PetroglyphTools/PG.StarWarsGame.Engine.FileSystem/PG.StarWarsGame.Engine.FileSystem.csproj" />
     <Project Path="src/PetroglyphTools/PG.StarWarsGame.Engine/PG.StarWarsGame.Engine.csproj" />
     <Project Path="src/PetroglyphTools/PG.StarWarsGame.Files.ALO/PG.StarWarsGame.Files.ALO.csproj" />
     <Project Path="src/PetroglyphTools/PG.StarWarsGame.Files.ChunkFiles/PG.StarWarsGame.Files.ChunkFiles.csproj" />

README.md

@@ -80,13 +80,42 @@ In general ModVerify has two operation mods.
 1. `verify` Verifying a game or mod 
 2. `createBaseline` Creating a baseline for a game or mod, that can be used for further verifications in order to verify you did not add more errors to your mods.
 
-### Example
-This is an example run configuration that analyzes a specific mod, uses a the FoC basline and writes the output into a dedicated directory: 
+### Examples
 
-```bash
+#### Example 1: Auto-detection with a custom baseline
+Analyzes a specific mod, uses the FoC baseline and writes the output into a dedicated directory:
+
+**Windows:**
+```bat
 .\ModVerify.exe verify --path "C:\My Games\FoC\Mods\MyMod" --outDir "C:\My Games\FoC\Mods\MyMod\verifyResults" --baseline ./focBaseline.json
 ```
 
+**Linux:**
+```bash
+./ModVerify verify \
+  --path "/home/user/games/FoC/Mods/MyMod" \
+  --outDir "/home/user/games/FoC/Mods/MyMod/verifyResults" \
+  --baseline ./focBaseline.json
+```
+
+#### Example 2: Manual mod setup with sub-mods, EaW fallback and default baseline
+Uses manual mod setup, including sub-mods and the EaW fallback game, and uses the default embedded baseline:
+
+**Windows:**
+```bat
+.\ModVerify.exe verify --mods "C:\My Games\FoC\Mods\MySubMod;C:\My Games\FoC\Mods\MyMod" --game "C:\My Games\FoC" --fallbackGame "C:\My Games\EaW" --engine FOC --useDefaultBaseline
+```
+
+**Linux:**
+```bash
+./ModVerify verify \
+  --mods "/home/user/games/FoC/Mods/MySubMod:/home/user/games/FoC/Mods/MyMod" \
+  --game "/home/user/games/FoC" \
+  --fallbackGame "/home/user/games/EaW" \
+  --engine FOC \
+  --useDefaultBaseline
+```
+
 ---
 
 ## Available Checks
@@ -116,6 +145,14 @@ The following verifiers are currently implemented:
 If you want to create your own baseline use the `createBaseline` option. 
 
 ### Example
+
+**Windows**
 ```bash
 ModVerify.exe createBaseline --outFile myBaseline.json --path "C:\My Games\FoC\Mods\MyMod"
 ```
+**Linux**
+```bash
+./ModVerify createBaseline \
+  --outFile myBaseline.json \
+  --path "C:\My Games\FoC\Mods\MyMod"
+```

modules/ModdingToolBase

@@ -1,5 +1,6 @@
 using System;
 using System.IO.Abstractions;
+using System.Linq;
 using System.Threading.Tasks;
 using AET.ModVerify.App.GameFinder;
 using AET.ModVerify.App.Reporting;
@@ -9,6 +10,7 @@
 using AET.ModVerify.Reporting.Baseline;
 using AET.ModVerify.Reporting.Suppressions;
 using AnakinRaW.ApplicationBase;
+using AnakinRaW.CommonUtilities.SimplePipeline;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Logging;
 
@@ -74,9 +76,34 @@ public async Task<int> ExecuteAsync()
         var verificationResult = await VerifyTargetAsync(verificationTarget)
             .ConfigureAwait(false);
 
+        Console.WriteLine();
+
+        switch (verificationResult.Status)
+        {
+            case VerificationCompletionStatus.Cancelled:
+                return ModVerifyConstants.VerifyCancelled;
+            case VerificationCompletionStatus.Failed:
+                return ReportVerificationFailure(verificationResult.Exception!);
+        }
+
         return await ProcessResult(verificationResult);
     }
 
+    private int ReportVerificationFailure(Exception verificationException)
+    {
+        var exceptionToReport = verificationException switch
+        {
+            AggregateException aggregate => aggregate.InnerExceptions.FirstOrDefault() ?? aggregate,
+            StepFailureException stepFailure => stepFailure.FailedSteps.First().Error!,
+            _ => verificationException
+        };
+
+        ConsoleUtilities.WriteApplicationFatalError(_appEnvironment.ApplicationName, exceptionToReport);
+        Logger?.LogError(exceptionToReport, exceptionToReport.Message);
+
+        return exceptionToReport.HResult;
+    }
+
     protected abstract Task<int> ProcessResult(VerificationResult result);
 
     protected abstract VerificationBaseline GetBaseline(VerificationTarget verificationTarget);
@@ -112,6 +139,10 @@ private async Task<VerificationResult> VerifyTargetAsync(VerificationTarget veri
                 case VerificationCompletionStatus.Cancelled:
                     Logger?.LogWarning(ModVerifyConstants.ConsoleEventId, "Verification was cancelled.");
                     break;
+                case VerificationCompletionStatus.Failed:
+                    progressReporter.ReportError("Verification failed!", 
+                        $"An unexpected error occurred while verifying '{verificationTarget.Name}'.");
+                    break;
                 case VerificationCompletionStatus.Completed:
                 default:
                     Logger?.LogInformation(ModVerifyConstants.ConsoleEventId, "Verification completed successfully.");

src/ModVerify.CliApp/App/ModVerifyApplicationAction.cs

@@ -61,7 +61,8 @@ protected override VerificationBaseline GetBaseline(VerificationTarget verificat
         {
             Console.WriteLine();
             ModVerifyConsoleUtilities.WriteBaselineInfo(baseline, baselinePath);
-            Logger?.LogDebug("Using baseline {Baseline} from location '{Path}'", baseline.ToString(), baselinePath);
+            Logger?.LogDebug("Using baseline {Baseline} from location '{Path}'", 
+                baseline.ToString(), baselinePath ?? "Embedded");
             Console.WriteLine();
         }
         return baseline;

src/ModVerify.CliApp/App/VerifyAction.cs

@@ -37,11 +37,11 @@
 		<PackageReference Include="Serilog.Extensions.Logging" Version="10.0.0" />
 		<PackageReference Include="AlamoEngineTools.SteamAbstraction" Version="5.0.7" />
 		<PackageReference Include="CommandLineParser" Version="2.9.1" />
-		<PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="10.0.5" />
-		<PackageReference Include="Microsoft.Extensions.Logging" Version="10.0.5" />
-		<PackageReference Include="Microsoft.Extensions.Logging.Abstractions" Version="10.0.5" />
-		<PackageReference Include="Microsoft.Extensions.Logging.Console" Version="10.0.5" />
-		<PackageReference Include="Microsoft.Extensions.Logging.Debug" Version="10.0.5" />
+		<PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="10.0.7" />
+		<PackageReference Include="Microsoft.Extensions.Logging" Version="10.0.7" />
+		<PackageReference Include="Microsoft.Extensions.Logging.Abstractions" Version="10.0.7" />
+		<PackageReference Include="Microsoft.Extensions.Logging.Console" Version="10.0.7" />
+		<PackageReference Include="Microsoft.Extensions.Logging.Debug" Version="10.0.7" />
 		<PackageReference Include="Serilog.Sinks.Console" Version="6.1.1" />
 		<PackageReference Include="Serilog.Sinks.File" Version="7.0.0" />
 		<PackageReference Include="Serilog.Extensions.Logging.File" Version="3.0.0" />
@@ -51,7 +51,7 @@
 	</ItemGroup>
 
 	<ItemGroup>
-		<PackageReference Include="Costura.Fody" Version="6.0.0">
+		<PackageReference Include="Costura.Fody" Version="6.1.0">
 			<PrivateAssets>all</PrivateAssets>
 		</PackageReference>
 		<PackageReference Include="PolySharp" Version="1.15.0">
@@ -62,15 +62,15 @@
 
 	<!-- Exclude System.Index from all dependencies, excluding Microsoft.Bcl.Memory -->
 	<ItemGroup>
-		<PackageReference Include="Vanara.Core" Version="5.0.1" Condition="'$(TargetFramework)' == 'net481'">
+		<PackageReference Include="Vanara.Core" Version="5.0.4" Condition="'$(TargetFramework)' == 'net481'">
 			<ExcludeAssets>compile</ExcludeAssets>
 			<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
 		</PackageReference>
-		<PackageReference Include="IndexRange" Version="1.1.0" Condition="'$(TargetFramework)' == 'net481'">
+		<PackageReference Include="IndexRange" Version="1.1.1" Condition="'$(TargetFramework)' == 'net481'">
 			<ExcludeAssets>compile</ExcludeAssets>
 			<IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
 		</PackageReference>
-		<PackageReference Include="Microsoft.Bcl.Memory" Version="10.0.5" Condition="'$(TargetFramework)' == 'net481'" />
+		<PackageReference Include="Microsoft.Bcl.Memory" Version="10.0.7" Condition="'$(TargetFramework)' == 'net481'" />
 	</ItemGroup>
 
 	<ItemGroup>

src/ModVerify.CliApp/ModVerify.CliApp.csproj

@@ -12,6 +12,7 @@ internal static class ModVerifyConstants
     public const int Success = 0;
     public const int CompletedWithFindings = 1;
     public const int ErrorBadArguments = 0xA0;
+    public const int VerifyCancelled = -1;
 
     public static readonly EventId ConsoleEventId = new(ConsoleEventIdValue, "LogToConsole");
 }

src/ModVerify.CliApp/ModVerifyConstants.cs

@@ -42,7 +42,7 @@ public VerificationBaseline SelectBaseline(VerificationTarget verificationTarget
             }
         }
 
-        if (!settings.ReportSettings.SearchBaselineLocally)
+        if (settings.ReportSettings is { SearchBaselineLocally: false, UseDefaultBaseline: false })
         {
             _logger?.LogDebug(ModVerifyConstants.ConsoleEventId, 
                 "No baseline path specified and local search is not enabled. Using empty baseline.");
@@ -134,7 +134,7 @@ internal static VerificationBaseline LoadEmbeddedBaseline(GameEngineType engineT
     private VerificationBaseline FindBaselineNonInteractive(VerificationTarget target, out string? usedPath)
     {
         if (_baselineFactory.TryFindBaselineInDirectory(
-                target.Location.TargetPath, 
+                target.Location.TargetPath,
                 b => IsBaselineCompatible(b, target),
                 out var baseline,
                 out usedPath))
@@ -144,6 +144,20 @@ private VerificationBaseline FindBaselineNonInteractive(VerificationTarget targe
         }
         _logger?.LogTrace("No baseline file found in taget path '{TargetPath}'.", target.Location.TargetPath);
         usedPath = null;
+        if (settings.ReportSettings.UseDefaultBaseline)
+        {
+            try
+            {
+                var defaultBaseline = LoadEmbeddedBaseline(target.Engine);
+                _logger?.LogInformation(ModVerifyConstants.ConsoleEventId, "Automatically applying default embedded baseline for engine '{Engine}'.", target.Engine);
+                return defaultBaseline;
+            }
+            catch (InvalidBaselineException)
+            {
+                throw new InvalidOperationException(
+                    "Invalid baseline packed along ModVerify App. Please reach out to the creators. Thanks!");
+            }
+        }
         return VerificationBaseline.Empty;
     }