refactor(checker): 优化链接有效性检查中的 URL 检查部分

[DuckDuckStudio]

此次优化重构了 URL 检查结果处理逻辑。
在第一次检查一个 URL 时，将 URL 和检查结果一起存入 checkedUrls 中；
如果后续再次检查这个 URL，则使用先前检查的结果。

---

对比测试:

• 默认
  • main: https://github.com/DuckDuckStudio/winget-tools/actions/runs/18054019046 (14m 17s)
• 详细
  • main: https://github.com/DuckDuckStudio/winget-tools/actions/runs/18057018391 (> 1h 30m / 除 m 外 47m)
  • checked-urls: https://github.com/DuckDuckStudio/winget-tools/actions/runs/18058162222 (21m 14s)

---

• Resolves pref(checker): 不要重复检查相同的 URL #559

[DuckDuckStudio]

GitHub Action Run 18054481111 Crash Report.

Error:  winget-pkgs/manifests/2/2BrightSparks/SyncBackPro/11.2.33.0/2BrightSparks.SyncBackPro.installer.yaml 中的 https://www.2brightsparks.com/assets/software/SyncBack/SyncBackPro64_Setup.11.2.33.0.exe 发生错误: Operations that change non-concurrent collections must have exclusive access. A concurrent update was performed on this collection and corrupted its state. The collection's state is no longer correct.
Unhandled exception. System.InvalidOperationException: Collection was modified; enumeration operation may not execute.
   at System.Collections.Generic.HashSet`1.Enumerator.MoveNext()
   at System.Linq.Enumerable.Any[TSource](IEnumerable`1 source, Func`2 predicate)
   at checker.Program.CheckUrlAsync(HttpClient client, String filePath, String url, String failureLevel) in /home/runner/work/winget-tools/winget-tools/checker/Program.cs:line 259
   at checker.Program.<>c__DisplayClass3_0.<<CheckUrlsInYamlFilesParallel>b__0>d.MoveNext() in /home/runner/work/winget-tools/winget-tools/checker/Program.cs:line 129
--- End of stack trace from previous location ---
   at checker.Program.CheckUrlsInYamlFilesParallel(String folderPath, String failureLevel, Int32 maxConcurrency) in /home/runner/work/winget-tools/winget-tools/checker/Program.cs:line 141
   at checker.Program.Main(String[] args) in /home/runner/work/winget-tools/winget-tools/checker/Program.cs:line 75
   at checker.Program.<Main>(String[] args)
**********************************************************************************************************************
Error: Process completed with exit code 134.

翻译:
操作非并发集合时必须具有独占访问权限。该集合上执行了并发更新操作，导致其状态损坏。集合状态不再正确。
未处理的异常。System.InvalidOperationException: 集合已被修改；枚举操作可能无法执行。

[DuckDuckStudio]

依据实际结果，这个优化似乎对 默认 失败等级的运行时间没有明显优化，但对 详细 失败等级的运行时间有明显优化。
这也是预期中的。

[DuckDuckStudio]

除了重复检查，超时也是影响检查时间的一大因素。但我们似乎只能通过忽略来处理这些超时的 URL。

[DuckDuckStudio]

我可能会考虑延迟这个 PR 的合并，因为这个优化尚不能证明稳定，且详细模式很少用到。

[fjwxzde]

这里有一些建议。
另外，在处理错误结果时都是

errorMessage = $"";
checkedUrls.TryAdd(url, errorMessage);
WriteErrorMessage(errorMessage, filePath);

的格式，也许我们可以建一个方法来统一这些？

[fjwxzde]

由于现在没有去重，这里的收集也许可以简化？

[DuckDuckStudio]

我不知道该怎么简化。

[fjwxzde]

这个结果似乎也不应存储？

[DuckDuckStudio]

我不确定发布者多久能修复这个错误。
另外，有些发布者的网站将 404 Not Found 视为资源暂时不可用。

[fjwxzde]

另外，有些发布者的网站将 404 Not Found 视为资源暂时不可用。

这应该只是页面描述，实际应该还是响应 404。

[fjwxzde]

除了重复检查，超时也是影响检查时间的一大因素。但我们似乎只能通过忽略来处理这些超时的 URL。

也许这里存储超时结果能缓解超时 URL 带来的时间，但也许我们不该存储超时结果，因为它可能在后续缓解？
亦或者我们应该抛出 TimeoutException 让下面的 catch (TimeoutException e) 处理？

[DuckDuckStudio]

我觉得还是保持原样，在 catch 里抛出并不会被下面的 catch 捕获。

[DuckDuckStudio]

另外，在处理错误结果时都是

errorMessage = $"";
checkedUrls.TryAdd(url, errorMessage);
WriteErrorMessage(errorMessage, filePath);

的格式，也许我们可以建一个方法来统一这些？

我认为还是保留原样。有些 errorMessage 在失败模式 默认 和 详细 下有不同的 errorMessage。
在这种情况下，处理错误结果是这样的:

errorMessage = $"...";
if (failureLevel == "详细")
{
    errorMessage = string.Concat(errorMessage, $"[Hint] Sundry 命令: ...");
}
checkedUrls.TryAdd(url, errorMessage);
WriteErrorMessage(errorMessage, filePath);
if (failureLevel == "详细")
{
    return false;
}