Fix a lot of bugs in MetahookInstaller

Author: hzqst

.serena/memories/MetahookInstaller.md

@@ -101,10 +101,7 @@ svencoop 目标为 svencoop.exe]
 - 历史脚本链路：`scripts/install-helper-*.bat` 与该 GUI 工具职责重叠，说明安装器逻辑由批处理向 UI 工具演进。
 
 ## 注意事项
-- `HasImportedModule` 用 `BinaryReader.ReadString()` 读取 PE 导入模块名（PE 中通常是 C 风格零结尾字符串），存在解析不稳风险，可能导致 SDL 依赖判定为假阴性。
-- `PluginInfoComparer` 的 `Equals` 是大小写不敏感，但 `GetHashCode` 直接用默认 `string.GetHashCode()`，与比较语义不完全一致，可能造成 `Distinct` 去重边界异常。
 - 路径发现与配置文件读写大量依赖当前工作目录（如 `./Build`、`./lang`），若启动目录变化，可能出现“找不到 Build/语言配置”的行为偏差。
-- `GetLibraryFolders` / `GetInstallDirFromManifest` 采用行文本拆分解析 VDF/ACF，鲁棒性弱于正式 VDF 解析器；遇到格式变化可能失效。
 - 卸载逻辑基于固定白名单路径删除，覆盖面有限且不做备份；用户手工放入同路径下的内容存在被删除风险。
 - 桌面工程未显式请求提权；当目标游戏目录在受保护路径（如 `Program Files`）时，安装/卸载可能抛出权限异常（代码中已提示但不自动提权）。
 

toolsrc/MetahookInstaller/MetahookInstallerAvalonia/MetahookInstallerAvalonia.csproj

@@ -28,6 +28,7 @@
     <PackageReference Include="Irihi.Ursa.ReactiveUIExtension" Version="1.1.0" />
     <PackageReference Include="Irihi.Ursa.Themes.Semi" Version="1.13.0" />
     <PackageReference Include="ReactiveUI.Avalonia" Version="11.3.0" />
+    <PackageReference Include="Gameloop.Vdf" Version="0.6.2" />
     <PackageReference Include="securifybv.ShellLink" Version="0.1.0" />
     <PackageReference Include="Semi.Avalonia" Version="11.3.7" />
     <PackageReference Include="Xaml.Behaviors.Avalonia" Version="11.3.6.6" />

toolsrc/MetahookInstaller/MetahookInstallerAvalonia/ViewModels/MainViewModel.cs

@@ -4,6 +4,8 @@
 using MetahookInstallerAvalonia.Lang;
 using Microsoft.Win32;
 using ReactiveUI;
+using Gameloop.Vdf;
+using Gameloop.Vdf.Linq;
 using ShellLink;
 using System;
 using System.Collections.Generic;
@@ -105,6 +107,16 @@ public static bool IsLegitimatePE(string dllPath)
             return false;
         }
     }
+    private static string ReadNullTerminatedAscii(BinaryReader reader)
+    {
+        var bytes = new System.Collections.Generic.List<byte>();
+        byte b;
+        while ((b = reader.ReadByte()) != 0)
+        {
+            bytes.Add(b);
+        }
+        return System.Text.Encoding.ASCII.GetString(bytes.ToArray());
+    }
     private static long RvaToFileOffset(BinaryReader stream, int lfanew, uint rva)
     {
         try
@@ -204,7 +216,7 @@ public static bool HasImportedModule(string dllPath, string targetModule)
                 if (nameFileOffset == -1)
                     continue;
                 stream.Position = nameFileOffset;
-                string moduleName = reader.ReadString().ToLowerInvariant();
+                string moduleName = ReadNullTerminatedAscii(reader).ToLowerInvariant();
                 if (moduleName == targetModule)
                     return true;
             }
@@ -487,31 +499,38 @@ private string[] GetLibraryFolders()
         var configPath = Path.Combine(_steamPath, "steamapps", "libraryfolders.vdf");
         if (File.Exists(configPath))
         {
-            var config = File.ReadAllText(configPath);
-            var lines = config.Split('\n');
-            foreach (var line in lines)
+            try
             {
-                if (line.Contains("\"path\""))
+                var vdf = VdfConvert.Deserialize(File.ReadAllText(configPath));
+                foreach (var entry in vdf.Value.Children<VProperty>())
                 {
-                    var path = line.Split('"')[3].Replace("\\\\", "\\");
-                    libraryFolders.Add(path);
+                    var pathToken = entry.Value["path"];
+                    if (pathToken != null)
+                    {
+                        libraryFolders.Add(pathToken.ToString());
+                    }
                 }
             }
+            catch
+            {
+                // Fallback: ignore parse errors, return what we have
+            }
         }
 
         return [.. libraryFolders];
     }
     private static string? GetInstallDirFromManifest(string manifest)
     {
-        var lines = manifest.Split('\n');
-        foreach (var line in lines)
+        try
         {
-            if (line.Contains("\"installdir\""))
-            {
-                return line.Split('"')[3];
-            }
+            var vdf = VdfConvert.Deserialize(manifest);
+            var installDir = vdf.Value["installdir"];
+            return installDir?.ToString();
+        }
+        catch
+        {
+            return null;
         }
-        return null;
     }
     private static string? GetSteamPath()
     {
@@ -570,7 +589,7 @@ public bool Equals(PluginInfo? x, PluginInfo? y)
         }
         public int GetHashCode(PluginInfo obj)
         {
-            return obj?.Name?.GetHashCode() ?? 0;
+            return obj?.Name?.ToLowerInvariant().GetHashCode() ?? 0;
         }
     }
     private readonly ObservableCollection<PluginInfo> _plugins = [];