Reviewing files that changed from the base of the PR and between 1e8819a and 90c73e5.

• Flow.Launcher.Core/packages.lock.json (2 hunks)
• Flow.Launcher.Infrastructure/Flow.Launcher.Infrastructure.csproj (1 hunks)
• Flow.Launcher.Infrastructure/Image/ThumbnailReader.cs (3 hunks)
• Flow.Launcher.Infrastructure/packages.lock.json (1 hunks)
• Flow.Launcher/packages.lock.json (2 hunks)

Flow.Launcher.Infrastructure/packages.lock.json (1)

32-37: Lock entry added — OK.

Direct dependency captured with correct version/hash. Keep this in sync with other lockfiles after any subsequent restores.

Flow.Launcher.Core/packages.lock.json (2)

86-90: Transitive addition acknowledged.

Core now records ini-parser as transitive. No action needed beyond ensuring CI restores in locked mode.

---

271-273: Infrastructure project deps updated — OK.

Dependency fan‑out correctly includes ini-parser.

Flow.Launcher/packages.lock.json (2)

207-211: Root lock transitive entry — OK.

Matches infra/core locks. Keep hashes aligned after any restore.

---

871-873: Project dependency list updated — OK.

ini-parser included with other infra deps; nothing else to do.

Flow.Launcher.Infrastructure/Flow.Launcher.Infrastructure.csproj (1)

63-63: Add ini-parser to third‑party notices and regenerate package locks

• Found in: Plugins/Flow.Launcher.Plugin.Program/Flow.Launcher.Plugin.Program.csproj and Flow.Launcher.Infrastructure/Flow.Launcher.Infrastructure.csproj; packages.lock.json in Flow.Launcher, Flow.Launcher.Core, Flow.Launcher.Infrastructure show resolved: 2.5.2.
• Action: Add ini-parser to third‑party notices per repo policy; run dotnet restore --locked-mode with the .NET SDK (CI or locally) to validate/regenerate packages.lock.json and commit any updated lockfiles.
• Verification: dotnet restore --locked-mode could not be executed in the verification sandbox (dotnet: command not found).

Flow.Launcher.Infrastructure/Image/ThumbnailReader.cs (1)

125-148: .url IconFile parsing must handle IconIndex, file.dll, env vars, quotes, and relative paths

File: Flow.Launcher.Infrastructure/Image/ThumbnailReader.cs (lines 125–148). Repo search for ExtractIconEx/SHDefExtractIcon/HICON/GetIconInfo returned no matches — no reusable icon-extraction helper found.

Minimum fix: normalize and resolve IconFile (trim quotes, ExpandEnvironmentVariables, resolve relative to the .url directory, split off optional ",idx" even if index is ignored). Apply the diff below.

-            try
-            {
-                var parser = new FileIniDataParser();
-                var data = parser.ReadFile(fileName);
-                var urlSection = data["InternetShortcut"];
-
-                var iconPath = urlSection?["IconFile"];
-                if (string.IsNullOrEmpty(iconPath))
-                {
-                    throw new FileNotFoundException();
-                }
-                hBitmap = GetHBitmap(Path.GetFullPath(iconPath), width, height, options);
-            }
+            try
+            {
+                var parser = new FileIniDataParser();
+                var data = parser.ReadFile(fileName);
+                var urlSection = data["InternetShortcut"];
+
+                var iconPathRaw = urlSection?["IconFile"]?.Trim().Trim('"');
+                if (string.IsNullOrWhiteSpace(iconPathRaw))
+                    throw new FileNotFoundException();
+
+                // Expand %ENV% and resolve relative to .url location
+                var expanded = Environment.ExpandEnvironmentVariables(iconPathRaw);
+                var baseDir = Path.GetDirectoryName(Path.GetFullPath(fileName))!;
+                var resolved = Path.IsPathRooted(expanded) ? expanded : Path.GetFullPath(Path.Combine(baseDir, expanded));
+
+                // Handle "file,idx" pattern (idx may be negative). We ignore idx for now (see TODO below).
+                var (iconFileOnly, _) = SplitIconLocation(resolved);
+
+                hBitmap = GetHBitmap(Path.GetFullPath(iconFileOnly), width, height, options);
+            }
             catch
             {
                 hBitmap = GetHBitmap(Path.GetFullPath(fileName), width, height, options);
             }

Add helper (outside the shown range):

private static (string file, int? index) SplitIconLocation(string iconLocation)
{
    // e.g., "C:\\Path\\imageres.dll,15" or "C:\\Path\\file.ico"
    // Find last comma not within quotes
    var s = iconLocation.Trim();
    var lastComma = s.LastIndexOf(',');
    if (lastComma > 0 && lastComma < s.Length - 1 && int.TryParse(s[(lastComma + 1)..].Trim(), out var idx))
        return (s[..lastComma].Trim().Trim('"'), idx);
    return (s.Trim().Trim('"'), null);
}

Follow‑ups (required): properly honor IconIndex / file,idx by extracting that specific icon (SHDefExtractIcon/ExtractIconEx → HICON → convert to HBITMAP); add unit tests for absolute .ico, file.dll,idx, env-var paths, quoted paths, and relative paths.

Reviewing files that changed from the base of the PR and between 130033c and 3bd6906.

• Flow.Launcher.Infrastructure/Image/ThumbnailReader.cs (3 hunks)

Flow.Launcher.Infrastructure/Image/ThumbnailReader.cs (1)

41-53: Solid .url detection and delegation.

The early branch to a dedicated handler keeps the main path unchanged and limits risk. Looks good.