mirror of
https://github.com/wwiinnddyy/LanMountainDesktop.git
synced 2026-06-30 23:24:25 +08:00
Compare commits
7 Commits
setting
...
0361b83ea2
| Author | SHA1 | Date | |
|---|---|---|---|
|
0361b83ea2 | ||
|
|
cc85638a37 | ||
|
|
791e38d55e | ||
|
|
75aed3f6ad | ||
|
|
01cf32a610 | ||
|
|
69bcf2c6eb | ||
|
|
7a70476ce8 |
@@ -130,6 +130,7 @@ internal sealed class AirAppProcessStarter : IAirAppProcessStarter
|
||||
return startInfo;
|
||||
}
|
||||
|
||||
|
||||
private static void AddArgument(ProcessStartInfo startInfo, string name, string value)
|
||||
{
|
||||
startInfo.ArgumentList.Add(name);
|
||||
|
||||
@@ -107,4 +107,18 @@ public sealed class ComponentCategoryIconResolverTests
|
||||
var result = ComponentCategoryIconResolver.ResolveCategoryIcon("File", components);
|
||||
Assert.Equal(Icon.Folder, result);
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void ResolveCategoryIcon_Date_ResolvesCorrectly()
|
||||
{
|
||||
var result = ComponentCategoryIconResolver.ResolveCategoryIcon("Date", []);
|
||||
Assert.Equal(Icon.Calendar, result);
|
||||
}
|
||||
|
||||
[Fact]
|
||||
public void ResolveCategoryIcon_Study_ResolvesCorrectly()
|
||||
{
|
||||
var result = ComponentCategoryIconResolver.ResolveCategoryIcon("Study", []);
|
||||
Assert.Equal(Icon.Book, result);
|
||||
}
|
||||
}
|
||||
|
||||
@@ -14,15 +14,34 @@ public static class ComponentCategoryIconResolver
|
||||
return Icon.Apps;
|
||||
}
|
||||
|
||||
var icon = categoryId.ToLowerInvariant() switch
|
||||
{
|
||||
"clock" => Icon.Clock,
|
||||
"date" => Icon.Calendar,
|
||||
"weather" => Icon.WeatherSunny,
|
||||
"board" => Icon.Edit,
|
||||
"media" => Icon.Play,
|
||||
"info" => Icon.News,
|
||||
"calculator" => Icon.Calculator,
|
||||
"study" => Icon.Book,
|
||||
"file" => Icon.Folder,
|
||||
_ => (Icon?)null
|
||||
};
|
||||
|
||||
if (icon.HasValue)
|
||||
{
|
||||
return icon.Value;
|
||||
}
|
||||
|
||||
var firstComponent = categoryComponents.FirstOrDefault();
|
||||
if (firstComponent is null || string.IsNullOrWhiteSpace(firstComponent.IconKey))
|
||||
{
|
||||
return Icon.Apps;
|
||||
}
|
||||
|
||||
if (Enum.TryParse<Icon>(firstComponent.IconKey, ignoreCase: true, out var icon))
|
||||
if (Enum.TryParse<Icon>(firstComponent.IconKey, ignoreCase: true, out var resolvedIcon))
|
||||
{
|
||||
return icon;
|
||||
return resolvedIcon;
|
||||
}
|
||||
|
||||
return Icon.Apps;
|
||||
|
||||
@@ -54,7 +54,7 @@
|
||||
Background="{DynamicResource AdaptiveNavItemBackgroundBrush}">
|
||||
<Grid ColumnDefinitions="Auto,*"
|
||||
ColumnSpacing="8">
|
||||
<fi:SymbolIcon Symbol="{Binding Icon}"
|
||||
<fi:FluentIcon Icon="{Binding Icon}"
|
||||
IconVariant="Regular"
|
||||
FontSize="16" />
|
||||
<TextBlock Grid.Column="1"
|
||||
|
||||
@@ -108,7 +108,7 @@
|
||||
Click="OnFindMoreComponentsClick">
|
||||
<StackPanel Orientation="Horizontal" Spacing="6">
|
||||
<fi:FluentIcon Icon="Globe" IconVariant="Regular" FontSize="14"/>
|
||||
<TextBlock Text="查找更多组件" FontSize="12"/>
|
||||
<TextBlock Text="查找更多小组件" FontSize="12"/>
|
||||
</StackPanel>
|
||||
</Button>
|
||||
</StackPanel>
|
||||
@@ -132,6 +132,7 @@
|
||||
FontWeight="SemiBold"
|
||||
Foreground="{DynamicResource AdaptiveTextPrimaryBrush}"
|
||||
Text="{Binding SelectedComponent.DisplayName}"
|
||||
HorizontalAlignment="Center"
|
||||
TextTrimming="CharacterEllipsis"/>
|
||||
|
||||
<TextBlock Grid.Row="1"
|
||||
@@ -141,6 +142,7 @@
|
||||
Foreground="{DynamicResource AdaptiveTextSecondaryBrush}"
|
||||
Opacity="0.82"
|
||||
Text="{Binding SelectedComponent.Description}"
|
||||
HorizontalAlignment="Center"
|
||||
TextWrapping="Wrap"
|
||||
TextTrimming="CharacterEllipsis"/>
|
||||
|
||||
@@ -176,7 +178,7 @@
|
||||
Click="OnAddComponentClick">
|
||||
<StackPanel Orientation="Horizontal" Spacing="8">
|
||||
<fi:FluentIcon Icon="Add" IconVariant="Regular" FontSize="16"/>
|
||||
<TextBlock Text="添加" FontWeight="SemiBold"/>
|
||||
<TextBlock Text="添加小组件" FontWeight="SemiBold"/>
|
||||
</StackPanel>
|
||||
</Button>
|
||||
</Grid>
|
||||
|
||||
@@ -1,6 +1,7 @@
|
||||
<Window xmlns="https://github.com/avaloniaui"
|
||||
xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
|
||||
xmlns:controls="using:LanMountainDesktop.Views"
|
||||
xmlns:fi="using:FluentIcons.Avalonia"
|
||||
x:Class="LanMountainDesktop.Views.FusedDesktopComponentLibraryWindow"
|
||||
Width="740"
|
||||
Height="500"
|
||||
@@ -23,39 +24,37 @@
|
||||
HorizontalAlignment="Center"
|
||||
VerticalAlignment="Center"
|
||||
Padding="0"
|
||||
CornerRadius="{DynamicResource DesignCornerRadiusLg}"
|
||||
ClipToBounds="True">
|
||||
<Grid RowDefinitions="Auto,*,Auto">
|
||||
<Grid RowDefinitions="Auto,*">
|
||||
<Border Height="64"
|
||||
Padding="24,0,24,0"
|
||||
Background="Transparent"
|
||||
PointerPressed="OnWindowTitleBarPointerPressed">
|
||||
<TextBlock VerticalAlignment="Center"
|
||||
FontSize="22"
|
||||
FontWeight="SemiBold"
|
||||
Foreground="{DynamicResource AdaptiveTextPrimaryBrush}"
|
||||
Text="添加小组件" />
|
||||
<Grid ColumnDefinitions="*,Auto">
|
||||
<TextBlock VerticalAlignment="Center"
|
||||
FontSize="22"
|
||||
FontWeight="SemiBold"
|
||||
Foreground="{DynamicResource AdaptiveTextPrimaryBrush}"
|
||||
Text="添加小组件" />
|
||||
<Button Grid.Column="1"
|
||||
Width="32"
|
||||
Height="32"
|
||||
Padding="0"
|
||||
Background="Transparent"
|
||||
BorderThickness="0"
|
||||
Click="OnCloseClick"
|
||||
VerticalAlignment="Center">
|
||||
<fi:FluentIcon Icon="Dismiss"
|
||||
IconVariant="Regular"
|
||||
FontSize="16" />
|
||||
</Button>
|
||||
</Grid>
|
||||
</Border>
|
||||
|
||||
<controls:FusedDesktopComponentLibraryControl x:Name="LibraryControl"
|
||||
Grid.Row="1"
|
||||
Margin="22,0,22,8" />
|
||||
|
||||
<Border Grid.Row="2"
|
||||
Padding="24,16,24,22"
|
||||
BorderBrush="{DynamicResource AdaptiveGlassPanelBorderBrush}"
|
||||
BorderThickness="0,1,0,0">
|
||||
<Button x:Name="CloseWindowButton"
|
||||
HorizontalAlignment="Stretch"
|
||||
MinHeight="32"
|
||||
Padding="16,7"
|
||||
Background="{DynamicResource AdaptiveButtonBackgroundBrush}"
|
||||
BorderThickness="0"
|
||||
Click="OnCloseClick">
|
||||
<TextBlock HorizontalAlignment="Center"
|
||||
FontSize="14"
|
||||
Text="关闭" />
|
||||
</Button>
|
||||
</Border>
|
||||
Margin="22,0,22,22" />
|
||||
</Grid>
|
||||
</Border>
|
||||
</Grid>
|
||||
|
||||
@@ -4,7 +4,9 @@ using Avalonia.Controls;
|
||||
using Avalonia.Controls.ApplicationLifetimes;
|
||||
using Avalonia.Input;
|
||||
using Avalonia.Interactivity;
|
||||
using LanMountainDesktop.Appearance;
|
||||
using LanMountainDesktop.Services;
|
||||
using LanMountainDesktop.Settings.Core;
|
||||
|
||||
namespace LanMountainDesktop.Views;
|
||||
|
||||
@@ -15,6 +17,7 @@ public partial class FusedDesktopComponentLibraryWindow : Window
|
||||
public FusedDesktopComponentLibraryWindow()
|
||||
{
|
||||
InitializeComponent();
|
||||
ApplyFluentCornerRadius();
|
||||
|
||||
LibraryControl.AddComponentRequested += OnAddComponentRequested;
|
||||
KeyDown += OnWindowKeyDown;
|
||||
@@ -23,6 +26,25 @@ public partial class FusedDesktopComponentLibraryWindow : Window
|
||||
mainWindow?.RegisterFusedLibraryWindow(this);
|
||||
}
|
||||
|
||||
private void ApplyFluentCornerRadius()
|
||||
{
|
||||
if (RootGrid is null)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
var tokens = AppearanceCornerRadiusTokenFactory.Create(
|
||||
GlobalAppearanceSettings.CornerRadiusStyleFluent);
|
||||
RootGrid.Resources["DesignCornerRadiusMicro"] = tokens.Micro;
|
||||
RootGrid.Resources["DesignCornerRadiusXs"] = tokens.Xs;
|
||||
RootGrid.Resources["DesignCornerRadiusSm"] = tokens.Sm;
|
||||
RootGrid.Resources["DesignCornerRadiusMd"] = tokens.Md;
|
||||
RootGrid.Resources["DesignCornerRadiusLg"] = tokens.Lg;
|
||||
RootGrid.Resources["DesignCornerRadiusXl"] = tokens.Xl;
|
||||
RootGrid.Resources["DesignCornerRadiusIsland"] = tokens.Island;
|
||||
RootGrid.Resources["DesignCornerRadiusComponent"] = tokens.Component;
|
||||
}
|
||||
|
||||
public bool PreserveEditModeOnClose { get; private set; }
|
||||
|
||||
public void SetOverlayWindow(TransparentOverlayWindow overlayWindow)
|
||||
|
||||
@@ -557,7 +557,7 @@ begin
|
||||
if '{#MyAppArch}' = 'x64' then
|
||||
begin
|
||||
Result := ExpandConstant('{commonpf64}\dotnet\shared\Microsoft.WindowsDesktop.App');
|
||||
end;
|
||||
end
|
||||
else
|
||||
begin
|
||||
Result := ExpandConstant('{commonpf}\dotnet\shared\Microsoft.WindowsDesktop.App');
|
||||
@@ -574,7 +574,7 @@ begin
|
||||
if '{#MyAppArch}' = 'x64' then
|
||||
begin
|
||||
Result := DotNetRuntimeDownloadUrlX64;
|
||||
end;
|
||||
end
|
||||
else
|
||||
begin
|
||||
Result := DotNetRuntimeDownloadUrlX86;
|
||||
|
||||
@@ -183,6 +183,7 @@ function Assert-WindowsPayloadClean {
|
||||
$violations.Add((Get-RelativePathCompat -Root $Root -Path $_.FullName))
|
||||
}
|
||||
|
||||
|
||||
Get-ChildItem -LiteralPath $Root -Recurse -Directory -Filter "runtimes" -ErrorAction SilentlyContinue |
|
||||
ForEach-Object {
|
||||
Get-ChildItem -LiteralPath $_.FullName -Directory -ErrorAction SilentlyContinue |
|
||||
|
||||
@@ -1,4 +1,4 @@
|
||||
[CmdletBinding()]
|
||||
[CmdletBinding()]
|
||||
param(
|
||||
[string]$Project = "LanMountainDesktop.csproj",
|
||||
[string]$Configuration = "Release",
|
||||
@@ -318,6 +318,7 @@ function Publish-MainAppFrameworkDependentPayload {
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
$scriptRoot = Split-Path -Parent $MyInvocation.MyCommand.Path
|
||||
$repoRoot = Resolve-ExistingPath -PathValue (Join-Path $scriptRoot "..")
|
||||
|
||||
|
||||
@@ -4,7 +4,7 @@
|
||||
|
||||
为了确保桌面组件在不同尺寸、缩放比例下都能保持视觉一致性和美感，阑山桌面采用了 **固定圆角风格预设 (Fixed Corner Radius Styles)**，全面参考小米澎湃OS (Xiaomi HyperOS) 的设计语言。
|
||||
|
||||
此外，阑山桌面引入了 **Fluent** 预设，遵循 Microsoft Fluent Design System 规范。设置窗口始终使用 Fluent 圆角，独立于用户选择的全局圆角风格。
|
||||
此外，在系统管理与控制面板等特定区域，阑山桌面引入了 **Fluent** 预设，完全遵循 Microsoft Fluent Design System 规范，以便与宿主操作系统的应用视觉保持一致。
|
||||
|
||||
所有的组件和容器必须使用统一的资源键，禁止在 XAML 或代码中使用硬编码的像素值。
|
||||
|
||||
@@ -35,28 +35,52 @@
|
||||
| **Island** | 28px | 36px | 40px | 44px | 16px | 任务栏、全局大悬浮容器 |
|
||||
| **Component** | **20px** | **24px** | **28px** | **32px** | **8px** | **所有桌面组件 (Widget) 的主边框** |
|
||||
|
||||
## Fluent Design System 参考 (Fluent Reference)
|
||||
|
||||
Fluent 预设的核心值来源于 Microsoft 官方规范：
|
||||
|
||||
- **ControlCornerRadius = 4px**：用于标准持久 UI 元素（按钮、复选框、输入框等）
|
||||
- **OverlayCornerRadius = 8px**：用于临时覆盖 UI 元素（对话框、浮出菜单等）
|
||||
## 系统设计特例约束 (System Design Exceptions)
|
||||
|
||||
> [!IMPORTANT]
|
||||
> **设置窗口强制约束**：
|
||||
> 设置窗口 (`SettingsWindow`) 始终使用 Fluent 圆角 Token，不受用户全局圆角设置影响。这确保设置 UI 作为标准 Windows 应用窗口与 Fluent Design 一致。
|
||||
> **局部作用域隔离原则 (Scope Isolation)**
|
||||
> 为了确保系统级配置面板、向导及管理界面的设计规范性，部分特例区域必须**始终使用 Microsoft Fluent Design System 预设**，不受用户在“外观设置 -> 全局圆角”中所选风格的影响：
|
||||
>
|
||||
> 1. **设置窗口 (`SettingsWindow`)**：作为主配置中心，强制应用 Fluent 圆角，使其展现标准 Windows 应用的高级感与一致性。
|
||||
> 2. **融合桌面组件库 (`FusedDesktopComponentLibraryWindow` / `FusedDesktopComponentLibraryControl`)**：小组件库的管理添加窗口本身属于系统级向导，强制采用 Fluent 圆角设计（如外壳圆角为 `DesignCornerRadiusLg`，内部按钮为 `DesignCornerRadiusSm`），保证交互的高级感与系统级管理界面对齐。
|
||||
|
||||
### 实现机制 (Implementation Mechanism)
|
||||
|
||||
在上述特例窗口的初始化过程中，通过在其根网格/容器元素（如 `RootGrid`）下调用 `ApplyFluentCornerRadius()`，在局部作用域内覆盖所有的 `DesignCornerRadiusXxx` 资源键为 Fluent 阶梯对应的值：
|
||||
|
||||
```csharp
|
||||
private void ApplyFluentCornerRadius()
|
||||
{
|
||||
if (RootGrid is null) return;
|
||||
|
||||
var tokens = AppearanceCornerRadiusTokenFactory.Create(
|
||||
GlobalAppearanceSettings.CornerRadiusStyleFluent);
|
||||
|
||||
RootGrid.Resources["DesignCornerRadiusMicro"] = tokens.Micro;
|
||||
RootGrid.Resources["DesignCornerRadiusXs"] = tokens.Xs;
|
||||
RootGrid.Resources["DesignCornerRadiusSm"] = tokens.Sm;
|
||||
RootGrid.Resources["DesignCornerRadiusMd"] = tokens.Md;
|
||||
RootGrid.Resources["DesignCornerRadiusLg"] = tokens.Lg;
|
||||
RootGrid.Resources["DesignCornerRadiusXl"] = tokens.Xl;
|
||||
RootGrid.Resources["DesignCornerRadiusIsland"] = tokens.Island;
|
||||
RootGrid.Resources["DesignCornerRadiusComponent"] = tokens.Component;
|
||||
}
|
||||
```
|
||||
|
||||
这样使得所有内部子控件使用 `DynamicResource` 引用这些圆角资源时，解析到的都是隔离后且固定的 Fluent 设计弧度，实现不受全局用户偏好影响的精准渲染。
|
||||
|
||||
## 开发准则 (Implementation Rules)
|
||||
|
||||
> [!IMPORTANT]
|
||||
> **1. 桌面组件强制约束**：
|
||||
> 所有桌面组件（Widget / Desktop Component）的根容器边框必须使用 `{DynamicResource DesignCornerRadiusComponent}`。严禁对其进行任何比例运算或系数乘积（如 `* scale`），必须保持固定。
|
||||
> 所有桌面普通组件（Widget / Desktop Component）的根容器边框在设计时，必须统一且仅使用 `{DynamicResource DesignCornerRadiusComponent}`。严禁对其进行任何比例运算或系数乘积（如 `* scale`），以确保用户的全局圆角缩放设置能被正确、成比例地应用。
|
||||
|
||||
> [!TIP]
|
||||
> **2. 圆角嵌套规则**：
|
||||
> 当一个容器包裹另一个元素时，外层圆角应比内层圆角大一个阶梯。例如：
|
||||
> - 外部使用 `DesignCornerRadiusLg`
|
||||
> - 内部紧贴边缘的内容应使用 `DesignCornerRadiusMd`
|
||||
> - 外部大容器使用 `DesignCornerRadiusLg`
|
||||
> - 内部小卡片使用 `DesignCornerRadiusMd`
|
||||
> - 内部紧贴边缘的小图标或按钮使用 `DesignCornerRadiusSm`
|
||||
> 这样可以保证两条圆弧的圆心趋于重合，视觉重心更稳固。
|
||||
|
||||
> [!CAUTION]
|
||||
@@ -65,7 +89,7 @@ Fluent 预设的核心值来源于 Microsoft 官方规范：
|
||||
|
||||
## 常用资源键 (Common Resource Keys)
|
||||
|
||||
- `DesignCornerRadiusComponent` (最常用)
|
||||
- `DesignCornerRadiusComponent` (桌面组件主框专用)
|
||||
- `DesignCornerRadiusMicro`
|
||||
- `DesignCornerRadiusSm`
|
||||
- `DesignCornerRadiusMd`
|
||||
|
||||
257
docs/auto_commit_md/20250525_01cf32a.md
Normal file
257
docs/auto_commit_md/20250525_01cf32a.md
Normal file
@@ -0,0 +1,257 @@
|
||||
# Git 提交分析报告
|
||||
|
||||
**提交哈希**: 01cf32a610b8ba1b5d6eaca7666a9c93f86310bf
|
||||
**提交时间**: 2026-05-25 09:32:58 +0800
|
||||
**作者**: lincube \<lincube3@hotmail.com\>
|
||||
**提交信息**: changed.调整融合桌面组库的相关圆角
|
||||
|
||||
---
|
||||
|
||||
## 变更统计
|
||||
|
||||
- **修改文件数**: 5
|
||||
- **新增行数**: 59
|
||||
- **删除行数**: 3
|
||||
- **净变更行数**: +56
|
||||
|
||||
### 变更文件
|
||||
|
||||
| 文件 | 变更类型 | 变更行数 |
|
||||
|------|---------|---------|
|
||||
| LanMountainDesktop.Tests/ComponentCategoryIconResolverTests.cs | 新增测试 | +14 |
|
||||
| LanMountainDesktop/ComponentSystem/ComponentCategoryIconResolver.cs | 重构 | +23 / -2 |
|
||||
| LanMountainDesktop/Views/ComponentLibraryWindow.axaml | 修复 | +1 / -1 |
|
||||
| LanMountainDesktop/Views/FusedDesktopComponentLibraryWindow.axaml | 添加圆角 | +1 |
|
||||
| LanMountainDesktop/Views/FusedDesktopComponentLibraryWindow.axaml.cs | 新增逻辑 | +22 |
|
||||
|
||||
---
|
||||
|
||||
## 详细变更分析
|
||||
|
||||
### 1. LanMountainDesktop/ComponentSystem/ComponentCategoryIconResolver.cs
|
||||
|
||||
**核心逻辑重构**:
|
||||
|
||||
#### 变更 1: 添加预定义图标映射
|
||||
```diff
|
||||
+ var icon = categoryId.ToLowerInvariant() switch
|
||||
+ {
|
||||
+ "clock" => Icon.Clock,
|
||||
+ "date" => Icon.Calendar,
|
||||
+ "weather" => Icon.WeatherSunny,
|
||||
+ "board" => Icon.Edit,
|
||||
+ "media" => Icon.Play,
|
||||
+ "info" => Icon.News,
|
||||
+ "calculator" => Icon.Calculator,
|
||||
+ "study" => Icon.Book,
|
||||
+ "file" => Icon.Folder,
|
||||
+ _ => (Icon?)null
|
||||
+ };
|
||||
+
|
||||
+ if (icon.HasValue)
|
||||
+ {
|
||||
+ return icon.Value;
|
||||
+ }
|
||||
```
|
||||
|
||||
**变更说明**:
|
||||
- 添加了 10 种常用分类的预定义图标映射
|
||||
- 使用 switch 表达式，代码更简洁
|
||||
- 优先匹配预定义映射，提升性能
|
||||
|
||||
#### 变更 2: 变量重命名
|
||||
```diff
|
||||
- if (Enum.TryParse<Icon>(firstComponent.IconKey, ignoreCase: true, out var icon))
|
||||
+ if (Enum.TryParse<Icon>(firstComponent.IconKey, ignoreCase: true, out var resolvedIcon))
|
||||
```
|
||||
- 避免与新添加的 `icon` 变量冲突
|
||||
|
||||
---
|
||||
|
||||
### 2. LanMountainDesktop.Tests/ComponentCategoryIconResolverTests.cs
|
||||
|
||||
**新增单元测试**:
|
||||
|
||||
#### 测试 1: Date 分类图标解析
|
||||
```csharp
|
||||
[Fact]
|
||||
public void ResolveCategoryIcon_Date_ResolvesCorrectly()
|
||||
{
|
||||
var result = ComponentCategoryIconResolver.ResolveCategoryIcon("Date", []);
|
||||
Assert.Equal(Icon.Calendar, result);
|
||||
}
|
||||
```
|
||||
|
||||
#### 测试 2: Study 分类图标解析
|
||||
```csharp
|
||||
[Fact]
|
||||
public void ResolveCategoryIcon_Study_ResolvesCorrectly()
|
||||
{
|
||||
var result = ComponentCategoryIconResolver.ResolveCategoryIcon("Study", []);
|
||||
Assert.Equal(Icon.Book, result);
|
||||
}
|
||||
```
|
||||
|
||||
**测试覆盖**:
|
||||
- ✅ Date 分类 → Calendar 图标
|
||||
- ✅ Study 分类 → Book 图标
|
||||
- ⚠️ 建议：添加其他预定义映射的测试用例
|
||||
|
||||
---
|
||||
|
||||
### 3. LanMountainDesktop/Views/ComponentLibraryWindow.axaml
|
||||
|
||||
**修复**: 控件类型名称更新
|
||||
```diff
|
||||
- <fi:SymbolIcon Symbol="{Binding Icon}"
|
||||
+ <fi:FluentIcon Icon="{Binding Icon}"
|
||||
```
|
||||
- **变更**: `SymbolIcon` → `FluentIcon`
|
||||
- **原因**: 保持 API 一致性，使用最新的 FluentIcons.Avalonia API
|
||||
|
||||
---
|
||||
|
||||
### 4. LanMountainDesktop/Views/FusedDesktopComponentLibraryWindow.axaml
|
||||
|
||||
**添加圆角属性**:
|
||||
```diff
|
||||
+ CornerRadius="{DynamicResource DesignCornerRadiusLg}"
|
||||
```
|
||||
应用到主 Border 容器，确保与其他 UI 组件的圆角风格一致。
|
||||
|
||||
---
|
||||
|
||||
### 5. LanMountainDesktop/Views/FusedDesktopComponentLibraryWindow.axaml.cs
|
||||
|
||||
**新增圆角应用逻辑**:
|
||||
|
||||
#### 变更 1: 添加依赖
|
||||
```csharp
|
||||
using LanMountainDesktop.Appearance;
|
||||
using LanMountainDesktop.Settings.Core;
|
||||
```
|
||||
|
||||
#### 变更 2: 构造函数中调用圆角应用
|
||||
```csharp
|
||||
public FusedDesktopComponentLibraryWindow()
|
||||
{
|
||||
InitializeComponent();
|
||||
+ ApplyFluentCornerRadius();
|
||||
// ...
|
||||
}
|
||||
```
|
||||
|
||||
#### 变更 3: 新增 ApplyFluentCornerRadius 方法
|
||||
```csharp
|
||||
private void ApplyFluentCornerRadius()
|
||||
{
|
||||
if (RootGrid is null)
|
||||
{
|
||||
return;
|
||||
}
|
||||
|
||||
var tokens = AppearanceCornerRadiusTokenFactory.Create(
|
||||
GlobalAppearanceSettings.CornerRadiusStyleFluent);
|
||||
RootGrid.Resources["DesignCornerRadiusMicro"] = tokens.Micro;
|
||||
RootGrid.Resources["DesignCornerRadiusXs"] = tokens.Xs;
|
||||
RootGrid.Resources["DesignCornerRadiusSm"] = tokens.Sm;
|
||||
RootGrid.Resources["DesignCornerRadiusMd"] = tokens.Md;
|
||||
RootGrid.Resources["DesignCornerRadiusLg"] = tokens.Lg;
|
||||
RootGrid.Resources["DesignCornerRadiusXl"] = tokens.Xl;
|
||||
RootGrid.Resources["DesignCornerRadiusIsland"] = tokens.Island;
|
||||
RootGrid.Resources["DesignCornerRadiusComponent"] = tokens.Component;
|
||||
}
|
||||
```
|
||||
|
||||
**变更说明**:
|
||||
- 动态应用全局圆角设置到窗口的资源字典
|
||||
- 支持 Fluent 风格的圆角配置
|
||||
- 使用工厂模式创建圆角 Token
|
||||
|
||||
---
|
||||
|
||||
## 代码架构分析
|
||||
|
||||
### 新增依赖关系
|
||||
|
||||
```
|
||||
FusedDesktopComponentLibraryWindow.axaml.cs
|
||||
├── LanMountainDesktop.Appearance
|
||||
│ └── AppearanceCornerRadiusTokenFactory
|
||||
└── LanMountainDesktop.Settings.Core
|
||||
└── GlobalAppearanceSettings
|
||||
```
|
||||
|
||||
### 圆角系统架构
|
||||
|
||||
| 组件 | 职责 |
|
||||
|------|------|
|
||||
| `AppearanceCornerRadiusTokenFactory` | 工厂类，创建圆角配置 |
|
||||
| `GlobalAppearanceSettings` | 全局外观设置，包含圆角风格 |
|
||||
| `DesignCornerRadius*` | 动态资源键，存储具体圆角值 |
|
||||
|
||||
---
|
||||
|
||||
## 代码审查要点
|
||||
|
||||
### 潜在问题
|
||||
|
||||
1. **测试覆盖**:
|
||||
- ⚠️ 中等风险：只添加了 2 个新测试用例
|
||||
- 建议添加其他 8 个预定义映射的测试
|
||||
|
||||
2. **API 一致性**:
|
||||
- ⚠️ 低风险：`SymbolIcon` → `FluentIcon` 的变更需要确认是否影响其他位置
|
||||
- 建议：搜索项目中所有 `SymbolIcon` 的使用
|
||||
|
||||
3. **空值处理**:
|
||||
- ✅ 良好：`switch` 表达式正确处理了未知分类
|
||||
- ✅ 良好：fallback 到原有逻辑
|
||||
|
||||
4. **性能考虑**:
|
||||
- ✅ 优化：预定义映射避免了遍历组件列表
|
||||
- ✅ 优化：直接使用 `ToLowerInvariant()` 而非忽略大小写比较
|
||||
|
||||
### 建议
|
||||
|
||||
- ✅ **代码质量**: 重构清晰，逻辑简化
|
||||
- ✅ **测试**: 添加了单元测试，但可以更全面
|
||||
- ⚠️ **文档**: 考虑更新 ComponentCategoryIconResolver 的文档
|
||||
- 📝 **代码规范**: 遵循项目现有的代码风格
|
||||
|
||||
---
|
||||
|
||||
## 影响范围
|
||||
|
||||
- **图标系统**: 增强了分类图标解析功能
|
||||
- **UI/UX**: 统一了圆角风格
|
||||
- **测试覆盖**: 新增 2 个单元测试
|
||||
- **依赖关系**: 新增对 Appearance 和 Settings 模块的依赖
|
||||
|
||||
---
|
||||
|
||||
## 功能评估
|
||||
|
||||
### 新增功能
|
||||
|
||||
1. ✅ **预定义图标映射**: 10 种常用分类现在有明确的图标
|
||||
2. ✅ **动态圆角应用**: 支持 Fluent 风格的圆角配置
|
||||
3. ✅ **API 更新**: 使用最新的 FluentIcons.Avalonia API
|
||||
|
||||
### 改进点
|
||||
|
||||
1. ✅ **性能优化**: 预定义映射提升解析速度
|
||||
2. ✅ **代码可维护性**: 使用 switch 表达式更易读
|
||||
3. ✅ **一致性**: 统一使用 FluentIcon 控件
|
||||
|
||||
---
|
||||
|
||||
## 总结
|
||||
|
||||
这是一次全面的功能增强提交，主要改进包括：
|
||||
1. 重构图标解析逻辑，添加预定义映射
|
||||
2. 统一圆角风格，支持动态配置
|
||||
3. 更新 API 使用 FluentIcon
|
||||
4. 添加单元测试
|
||||
|
||||
**建议**: ✅ 可以合并，建议后续补充其他预定义映射的测试用例。
|
||||
407
docs/auto_commit_md/20250525_12f0caa.md
Normal file
407
docs/auto_commit_md/20250525_12f0caa.md
Normal file
@@ -0,0 +1,407 @@
|
||||
# Git 提交分析报告
|
||||
|
||||
**提交哈希**: 12f0caafc735aae8dc9c8d19f2c0829288106280
|
||||
**提交时间**: 2026-05-25 01:24:18 +0800
|
||||
**作者**: lincube \<lincube3@hotmail.com\>
|
||||
**提交信息**: fix.继续修复 .NET运行时问题
|
||||
|
||||
---
|
||||
|
||||
## 变更统计
|
||||
|
||||
- **修改文件数**: 3
|
||||
- **新增行数**: 181
|
||||
- **删除行数**: 16
|
||||
- **净变更行数**: +165
|
||||
|
||||
### 变更文件
|
||||
|
||||
| 文件 | 变更类型 | 变更行数 |
|
||||
|------|---------|---------|
|
||||
| LanMountainDesktop.Launcher/Services/DotNetRuntimeProbe.cs | 核心修复 | +80 / -15 |
|
||||
| LanMountainDesktop.Tests/DotNetRuntimeProbeTests.cs | 新增测试 | +109 |
|
||||
| LanMountainDesktop/installer/LanMountainDesktop.iss | 增强检测 | +8 |
|
||||
|
||||
---
|
||||
|
||||
## 详细变更分析
|
||||
|
||||
### 1. LanMountainDesktop.Launcher/Services/DotNetRuntimeProbe.cs
|
||||
|
||||
**核心问题修复**: 支持按用户安装的 .NET 运行时检测
|
||||
|
||||
#### 变更 1: 扩展选项配置
|
||||
```csharp
|
||||
public record DotNetRuntimeProbeOptions
|
||||
{
|
||||
// ... existing properties ...
|
||||
|
||||
+ public string? LocalAppDataPath { get; init; }
|
||||
}
|
||||
```
|
||||
- **新增**: `LocalAppDataPath` 配置项
|
||||
- **用途**: 支持检测 `%LOCALAPPDATA%\dotnet` 目录下的运行时
|
||||
|
||||
#### 变更 2: 定义必需的共享框架
|
||||
```csharp
|
||||
public const string RequiredSharedFrameworkName = "Microsoft.NETCore.App";
|
||||
+ public const string WindowsDesktopSharedFrameworkName = "Microsoft.WindowsDesktop.App";
|
||||
+
|
||||
+ private static readonly string[] RequiredSharedFrameworkNames =
|
||||
+ [
|
||||
+ RequiredSharedFrameworkName,
|
||||
+ WindowsDesktopSharedFrameworkName
|
||||
+ ];
|
||||
```
|
||||
- **新增**: Windows Desktop 运行时框架名称常量
|
||||
- **变更**: 将单一框架改为框架列表，支持多框架检测
|
||||
|
||||
#### 变更 3: 核心检测逻辑重构
|
||||
```csharp
|
||||
public static DotNetRuntimeProbeResult Probe(DotNetRuntimeProbeOptions? options = null)
|
||||
{
|
||||
// ... 初始化代码 ...
|
||||
|
||||
+ var localAppDataRoot = GetLocalAppDataPath(options);
|
||||
+ var perUserDotnetRoot = !string.IsNullOrWhiteSpace(localAppDataRoot)
|
||||
+ ? Path.Combine(localAppDataRoot, "dotnet")
|
||||
+ : null;
|
||||
+
|
||||
+ foreach (var frameworkName in RequiredSharedFrameworkNames)
|
||||
+ {
|
||||
+ foreach (var basePath in EnumerateDotNetInstallRoots(options))
|
||||
+ {
|
||||
+ var sharedFrameworkDirectory = Path.Combine(basePath, "shared", frameworkName);
|
||||
+ searchedPaths.Add(sharedFrameworkDirectory);
|
||||
+ var isPerUser = perUserDotnetRoot is not null &&
|
||||
+ string.Equals(basePath, perUserDotnetRoot, StringComparison.OrdinalIgnoreCase);
|
||||
+ AddDirectoryRuntimes(sharedFrameworkDirectory, frameworkName,
|
||||
+ isPerUser ? "shared-framework-directory-per-user" : "shared-framework-directory",
|
||||
+ detected);
|
||||
+ }
|
||||
+ }
|
||||
}
|
||||
```
|
||||
- **核心改进**:
|
||||
- 支持扫描多个 .NET 安装位置
|
||||
- 区分系统安装和按用户安装
|
||||
- 检测多个必需的共享框架
|
||||
|
||||
#### 变更 4: 新增安装根目录枚举
|
||||
```csharp
|
||||
+ private static IEnumerable<string> EnumerateDotNetInstallRoots(DotNetRuntimeProbeOptions options)
|
||||
+ {
|
||||
+ var programFilesRoot = options.Architecture == DotNetRuntimeArchitecture.X86
|
||||
+ ? GetProgramFilesX86Path(options)
|
||||
+ : GetProgramFilesPath(options);
|
||||
+
|
||||
+ yield return Path.Combine(programFilesRoot, "dotnet");
|
||||
+
|
||||
+ var localAppData = GetLocalAppDataPath(options);
|
||||
+ if (!string.IsNullOrWhiteSpace(localAppData))
|
||||
+ {
|
||||
+ var perUserDotnet = Path.Combine(localAppData, "dotnet");
|
||||
+ if (!string.Equals(perUserDotnet, Path.Combine(programFilesRoot, "dotnet"), StringComparison.OrdinalIgnoreCase))
|
||||
+ {
|
||||
+ yield return perUserDotnet;
|
||||
+ }
|
||||
+ }
|
||||
+ }
|
||||
```
|
||||
- **功能**: 枚举所有可能的 .NET 安装根目录
|
||||
- **包括**:
|
||||
- Program Files 下的系统安装
|
||||
- LocalAppData 下的按用户安装
|
||||
|
||||
#### 变更 5: 增强 dotnet host 路径搜索
|
||||
```csharp
|
||||
+ var localAppData = GetLocalAppDataPath(options);
|
||||
+ if (!string.IsNullOrWhiteSpace(localAppData))
|
||||
+ {
|
||||
+ var perUserHost = Path.Combine(localAppData, "dotnet", OperatingSystem.IsWindows() ? "dotnet.exe" : "dotnet");
|
||||
+ if (!string.Equals(perUserHost, Path.Combine(programFilesRoot, "dotnet", OperatingSystem.IsWindows() ? "dotnet.exe" : "dotnet"), StringComparison.OrdinalIgnoreCase))
|
||||
+ {
|
||||
+ yield return perUserHost;
|
||||
+ }
|
||||
+ }
|
||||
```
|
||||
- **改进**: 在按用户路径中搜索 dotnet host 可执行文件
|
||||
|
||||
#### 变更 6: 添加 LocalAppData 路径获取
|
||||
```csharp
|
||||
+ private static string GetLocalAppDataPath(DotNetRuntimeProbeOptions options)
|
||||
+ {
|
||||
+ if (!string.IsNullOrWhiteSpace(options.LocalAppDataPath))
|
||||
+ {
|
||||
+ return Path.GetFullPath(options.LocalAppDataPath);
|
||||
+ }
|
||||
+
|
||||
+ return Environment.GetFolderPath(Environment.SpecialFolder.LocalApplicationData);
|
||||
+ }
|
||||
```
|
||||
- **功能**: 获取 LocalAppData 路径，支持自定义配置
|
||||
|
||||
#### 变更 7: 增强 dotnet CLI 检测
|
||||
```csharp
|
||||
- private static void AddDotNetCliRuntimes(
|
||||
- string? dotNetHostPath,
|
||||
- string sharedFrameworkName,
|
||||
- List<DotNetRuntimeInfo> detected)
|
||||
+ private static void AddDotNetCliRuntimes(
|
||||
+ string? dotNetHostPath,
|
||||
+ List<DotNetRuntimeInfo> detected)
|
||||
```
|
||||
- **变更**: 移除 `sharedFrameworkName` 参数
|
||||
- **改进**: 使用 `RequiredSharedFrameworkNames` 列表，支持多框架检测
|
||||
|
||||
---
|
||||
|
||||
### 2. LanMountainDesktop.Tests/DotNetRuntimeProbeTests.cs
|
||||
|
||||
**新增单元测试**: 6 个全面的测试用例
|
||||
|
||||
#### 测试 1: 按用户运行时检测
|
||||
```csharp
|
||||
[Fact]
|
||||
public void Probe_DetectsPerUserRuntime()
|
||||
{
|
||||
CreateRuntime(_localAppData, "10.0.5",
|
||||
DotNetRuntimeProbe.RequiredSharedFrameworkName);
|
||||
|
||||
var result = DotNetRuntimeProbe.Probe(
|
||||
CreateOptions(DotNetRuntimeArchitecture.X64));
|
||||
|
||||
Assert.True(result.IsAvailable);
|
||||
Assert.Contains(result.DetectedRuntimes, runtime =>
|
||||
runtime.Version == "10.0.5" &&
|
||||
runtime.Source == "shared-framework-directory-per-user");
|
||||
}
|
||||
```
|
||||
|
||||
#### 测试 2: Windows Desktop 运行时检测
|
||||
```csharp
|
||||
[Fact]
|
||||
public void Probe_DetectsWindowsDesktopRuntime()
|
||||
{
|
||||
CreateRuntime(_programFiles, "10.0.5",
|
||||
DotNetRuntimeProbe.WindowsDesktopSharedFrameworkName);
|
||||
|
||||
var result = DotNetRuntimeProbe.Probe(
|
||||
CreateOptions(DotNetRuntimeArchitecture.X64));
|
||||
|
||||
Assert.False(result.IsAvailable);
|
||||
Assert.Contains(result.DetectedRuntimes, runtime =>
|
||||
runtime.Name == DotNetRuntimeProbe.WindowsDesktopSharedFrameworkName &&
|
||||
runtime.Version == "10.0.5");
|
||||
}
|
||||
```
|
||||
|
||||
#### 测试 3: 按用户 Windows Desktop 运行时检测
|
||||
```csharp
|
||||
[Fact]
|
||||
public void Probe_DetectsPerUserWindowsDesktopRuntime()
|
||||
{
|
||||
CreateRuntime(_localAppData, "10.0.5",
|
||||
DotNetRuntimeProbe.WindowsDesktopSharedFrameworkName);
|
||||
|
||||
var result = DotNetRuntimeProbe.Probe(
|
||||
CreateOptions(DotNetRuntimeArchitecture.X64));
|
||||
|
||||
Assert.Contains(result.DetectedRuntimes, runtime =>
|
||||
runtime.Name == DotNetRuntimeProbe.WindowsDesktopSharedFrameworkName &&
|
||||
runtime.Version == "10.0.5" &&
|
||||
runtime.Source == "shared-framework-directory-per-user");
|
||||
}
|
||||
```
|
||||
|
||||
#### 测试 4: 在按用户路径中查找 dotnet host
|
||||
```csharp
|
||||
[Fact]
|
||||
public void Probe_FindsDotNetHost_InPerUserPath()
|
||||
{
|
||||
var dotnetDir = Path.Combine(_localAppData, "dotnet");
|
||||
Directory.CreateDirectory(dotnetDir);
|
||||
File.WriteAllText(Path.Combine(dotnetDir, "dotnet.exe"), string.Empty);
|
||||
|
||||
var result = DotNetRuntimeProbe.Probe(new DotNetRuntimeProbeOptions
|
||||
{
|
||||
// ... 配置 ...
|
||||
LocalAppDataPath = _localAppData,
|
||||
IncludeRegistry = false,
|
||||
IncludeDotNetCli = false
|
||||
});
|
||||
|
||||
Assert.NotNull(result.DotNetHostPath);
|
||||
Assert.Contains("LocalAppData", result.DotNetHostPath);
|
||||
}
|
||||
```
|
||||
|
||||
#### 测试 5: 优先使用 Program Files host
|
||||
```csharp
|
||||
[Fact]
|
||||
public void Probe_PrefersProgramFilesHost_OverPerUserHost()
|
||||
{
|
||||
// 创建两个 dotnet.exe：一个在 Program Files，一个在 LocalAppData
|
||||
|
||||
var result = DotNetRuntimeProbe.Probe(/* ... */);
|
||||
|
||||
Assert.NotNull(result.DotNetHostPath);
|
||||
Assert.Contains("ProgramFiles", result.DotNetHostPath);
|
||||
}
|
||||
```
|
||||
|
||||
#### 测试 6: 合并系统和按用户运行时
|
||||
```csharp
|
||||
[Fact]
|
||||
public void Probe_CombinesSystemAndPerUserRuntimes()
|
||||
{
|
||||
CreateRuntime(_programFiles, "10.0.5");
|
||||
CreateRuntime(_localAppData, "10.0.3");
|
||||
|
||||
var result = DotNetRuntimeProbe.Probe(CreateOptions(
|
||||
DotNetRuntimeArchitecture.X64));
|
||||
|
||||
Assert.True(result.IsAvailable);
|
||||
Assert.Contains(result.DetectedRuntimes, runtime =>
|
||||
runtime.Version == "10.0.5");
|
||||
Assert.Contains(result.DetectedRuntimes, runtime =>
|
||||
runtime.Version == "10.0.3");
|
||||
}
|
||||
```
|
||||
|
||||
**测试覆盖评估**:
|
||||
- ✅ 按时用户安装检测
|
||||
- ✅ Windows Desktop 运行时检测
|
||||
- ✅ 混合安装场景
|
||||
- ✅ host 路径优先级
|
||||
- ⚠️ 建议：添加跨架构检测测试
|
||||
|
||||
---
|
||||
|
||||
### 3. LanMountainDesktop/installer/LanMountainDesktop.iss
|
||||
|
||||
**增强安装程序检测逻辑**:
|
||||
|
||||
#### 新增函数
|
||||
```pascal
|
||||
function GetPerUserDotNetDesktopRuntimePath(): String;
|
||||
begin
|
||||
Result := ExpandConstant('{localappdata}\dotnet\shared\Microsoft.WindowsDesktop.App');
|
||||
end;
|
||||
```
|
||||
|
||||
#### 增强检测函数
|
||||
```pascal
|
||||
function IsDotNetDesktopRuntimeInstalled(): Boolean;
|
||||
begin
|
||||
- Result := IsDotNet10RuntimePresent(GetTargetDotNetDesktopRuntimePath());
|
||||
+ Result := IsDotNet10RuntimePresent(GetTargetDotNetDesktopRuntimePath()) or
|
||||
+ IsDotNet10RuntimePresent(GetPerUserDotNetDesktopRuntimePath());
|
||||
end;
|
||||
```
|
||||
|
||||
**变更说明**:
|
||||
- 支持检测按用户安装的 .NET Desktop 运行时
|
||||
- 允许在缺少系统安装但有按用户安装时继续安装
|
||||
|
||||
---
|
||||
|
||||
## 问题分析与解决方案
|
||||
|
||||
### 原始问题
|
||||
|
||||
❌ **问题**: 只能检测系统级别的 .NET 运行时安装
|
||||
❌ **问题**: 无法识别用户通过 Visual Studio 或 winget 安装的 .NET 运行时
|
||||
❌ **问题**: 可能导致安装程序要求用户重新安装 .NET 运行时，即使已存在按用户安装
|
||||
|
||||
### 解决方案
|
||||
|
||||
✅ **扩展搜索路径**:
|
||||
- Program Files (系统级别)
|
||||
- %LOCALAPPDATA% (用户级别)
|
||||
|
||||
✅ **支持多框架检测**:
|
||||
- Microsoft.NETCore.App
|
||||
- Microsoft.WindowsDesktop.App
|
||||
|
||||
✅ **智能合并**:
|
||||
- 合并系统和用户安装的运行时
|
||||
- 优先使用系统级别的 dotnet host
|
||||
|
||||
---
|
||||
|
||||
## 代码审查要点
|
||||
|
||||
### 潜在问题
|
||||
|
||||
1. **性能考虑**:
|
||||
- ✅ 良好：使用 `yield return` 延迟枚举，避免不必要的文件系统访问
|
||||
- ⚠️ 低风险：路径比较使用 `OrdinalIgnoreCase`，性能影响可接受
|
||||
|
||||
2. **路径安全性**:
|
||||
- ✅ 良好：使用 `Path.GetFullPath()` 规范化路径
|
||||
- ✅ 良好：避免路径注入攻击
|
||||
|
||||
3. **错误处理**:
|
||||
- ✅ 良好：`string.IsNullOrWhiteSpace()` 检查空值
|
||||
- ✅ 良好：可选的配置参数，提供默认值
|
||||
|
||||
4. **测试覆盖**:
|
||||
- ✅ 优秀：6 个新的测试用例覆盖主要场景
|
||||
- ⚠️ 建议：添加边界情况测试（如路径包含特殊字符）
|
||||
|
||||
### 代码质量评估
|
||||
|
||||
| 方面 | 评分 | 说明 |
|
||||
|------|------|------|
|
||||
| 架构设计 | ⭐⭐⭐⭐⭐ | 清晰的分层和职责分离 |
|
||||
| 代码可读性 | ⭐⭐⭐⭐⭐ | 良好的命名和注释 |
|
||||
| 测试覆盖 | ⭐⭐⭐⭐⭐ | 全面的测试用例 |
|
||||
| 错误处理 | ⭐⭐⭐⭐ | 考虑周全，可进一步增强 |
|
||||
| 性能 | ⭐⭐⭐⭐⭐ | 延迟枚举优化性能 |
|
||||
|
||||
---
|
||||
|
||||
## 影响范围
|
||||
|
||||
### 功能影响
|
||||
|
||||
- ✅ **运行时检测**: 支持按用户安装的 .NET 运行时
|
||||
- ✅ **安装程序**: 更智能的 .NET 运行时检测
|
||||
- ✅ **用户体验**: 减少不必要的 .NET 运行时重新安装
|
||||
|
||||
### 技术影响
|
||||
|
||||
- ✅ **跨用户支持**: 支持同一机器上的多个用户配置
|
||||
- ✅ **混合安装**: 支持系统和按用户安装混合场景
|
||||
- ✅ **向后兼容**: 保持对现有系统安装的检测能力
|
||||
|
||||
---
|
||||
|
||||
## 安全考虑
|
||||
|
||||
1. **路径验证**: ✅ 使用 `Path.GetFullPath()` 防止路径注入
|
||||
2. **权限检查**: ✅ 区分系统和用户目录的访问权限
|
||||
3. **文件存在性**: ✅ 在访问前检查文件和目录存在性
|
||||
|
||||
---
|
||||
|
||||
## 总结
|
||||
|
||||
这是一次高质量的功能修复提交，主要解决了 .NET 运行时检测的关键问题：
|
||||
|
||||
### 核心改进
|
||||
|
||||
1. ✅ **扩展检测范围**: 支持按用户安装的 .NET 运行时
|
||||
2. ✅ **多框架支持**: 同时检测 Core 和 Desktop 运行时
|
||||
3. ✅ **智能合并**: 正确处理系统和用户安装的混合场景
|
||||
4. ✅ **全面测试**: 6 个新的单元测试确保可靠性
|
||||
5. ✅ **安装程序增强**: Inno Setup 脚本同步更新
|
||||
|
||||
### 代码质量
|
||||
|
||||
- 🏆 **优秀**: 架构清晰，代码规范
|
||||
- 🏆 **优秀**: 测试覆盖全面
|
||||
- 🏆 **优秀**: 错误处理周全
|
||||
|
||||
**建议**: ✅ 可以合并，建议后续添加更多边界情况测试。
|
||||
201
docs/auto_commit_md/20250525_75aed3f.md
Normal file
201
docs/auto_commit_md/20250525_75aed3f.md
Normal file
@@ -0,0 +1,201 @@
|
||||
# Git 提交分析报告
|
||||
|
||||
**提交哈希**: 75aed3f6ade7243a116163050014c2387d838ecb
|
||||
**提交时间**: 2026-05-25 10:16:00 +0800
|
||||
**作者**: lincube \<lincube3@hotmail.com\>
|
||||
**提交信息**: changed.调整了桌面组件库的UI
|
||||
|
||||
---
|
||||
|
||||
## 变更统计
|
||||
|
||||
- **修改文件数**: 2
|
||||
- **新增行数**: 26
|
||||
- **删除行数**: 26
|
||||
- **净变更行数**: 0
|
||||
|
||||
### 变更文件
|
||||
|
||||
| 文件 | 变更类型 | 变更行数 |
|
||||
|------|---------|---------|
|
||||
| LanMountainDesktop/Views/FusedDesktopComponentLibraryControl.axaml | 修改 | +6 / -6 |
|
||||
| LanMountainDesktop/Views/FusedDesktopComponentLibraryWindow.axaml | 修改 | +20 / -20 |
|
||||
|
||||
---
|
||||
|
||||
## 详细变更分析
|
||||
|
||||
### 1. LanMountainDesktop/Views/FusedDesktopComponentLibraryControl.axaml
|
||||
|
||||
**UI 文本优化**:
|
||||
|
||||
#### 变更 1: 按钮文本修改
|
||||
```diff
|
||||
- <TextBlock Text="查找更多组件" FontSize="12"/>
|
||||
+ <TextBlock Text="查找更多小组件" FontSize="12"/>
|
||||
```
|
||||
- **优化**: 使用更口语化的表述"小组件"
|
||||
|
||||
#### 变更 2: 添加水平居中对齐
|
||||
```diff
|
||||
+ HorizontalAlignment="Center"
|
||||
```
|
||||
应用到：
|
||||
- 组件显示名称（DisplayName）
|
||||
- 组件描述（Description）
|
||||
|
||||
**变更位置**:
|
||||
- 第 132 行：DisplayName 水平居中
|
||||
- 第 142 行：Description 水平居中
|
||||
|
||||
#### 变更 3: 添加按钮文本优化
|
||||
```diff
|
||||
- <TextBlock Text="添加" FontWeight="SemiBold"/>
|
||||
+ <TextBlock Text="添加小组件" FontWeight="SemiBold"/>
|
||||
```
|
||||
- **优化**: 明确操作目的，提高可读性
|
||||
|
||||
---
|
||||
|
||||
### 2. LanMountainDesktop/Views/FusedDesktopComponentLibraryWindow.axaml
|
||||
|
||||
**窗口布局重构**:
|
||||
|
||||
#### 变更 1: 导入 FluentIcons 命名空间
|
||||
```diff
|
||||
+ xmlns:fi="using:FluentIcons.Avalonia"
|
||||
```
|
||||
|
||||
#### 变更 2: 简化 Grid 行定义
|
||||
```diff
|
||||
- <Grid RowDefinitions="Auto,*,Auto">
|
||||
+ <Grid RowDefinitions="Auto,*">
|
||||
```
|
||||
- **移除**: 底部的"关闭"按钮区域
|
||||
|
||||
#### 变更 3: 添加自定义关闭按钮
|
||||
```diff
|
||||
+ <Button Grid.Column="1"
|
||||
+ Width="32"
|
||||
+ Height="32"
|
||||
+ Padding="0"
|
||||
+ Background="Transparent"
|
||||
+ BorderThickness="0"
|
||||
+ Click="OnCloseClick"
|
||||
+ VerticalAlignment="Center">
|
||||
+ <fi:FluentIcon Icon="Dismiss" IconVariant="Regular" FontSize="16" />
|
||||
+ </Button>
|
||||
```
|
||||
|
||||
**变更说明**:
|
||||
- 将窗口标题栏改为 Grid 布局（两列）
|
||||
- 左侧：窗口标题"添加小组件"
|
||||
- 右侧：自定义关闭按钮（使用 FluentIcon）
|
||||
- 移除了底部的"关闭"按钮，改用标题栏的关闭按钮
|
||||
|
||||
#### 变更 4: 调整内边距
|
||||
```diff
|
||||
- Margin="22,0,22,8"
|
||||
+ Margin="22,0,22,22"
|
||||
```
|
||||
|
||||
#### 变更 5: 移除底部边框和关闭按钮
|
||||
```diff
|
||||
- <Border Grid.Row="2"
|
||||
- Padding="24,16,24,22"
|
||||
- BorderBrush="{DynamicResource AdaptiveGlassPanelBorderBrush}"
|
||||
- BorderThickness="0,1,0,0">
|
||||
- <Button x:Name="CloseWindowButton"
|
||||
- HorizontalAlignment="Stretch"
|
||||
- MinHeight="32"
|
||||
- Padding="16,7"
|
||||
- Background="{DynamicResource AdaptiveButtonBackgroundBrush}"
|
||||
- BorderThickness="0"
|
||||
- Click="OnCloseClick">
|
||||
- <TextBlock HorizontalAlignment="Center"
|
||||
- FontSize="14"
|
||||
- Text="关闭" />
|
||||
- </Button>
|
||||
- </Border>
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## UI 变化对比
|
||||
|
||||
### 布局变化
|
||||
|
||||
| 方面 | 修改前 | 修改后 |
|
||||
|------|--------|--------|
|
||||
| 关闭按钮位置 | 底部栏 | 标题栏右侧 |
|
||||
| 窗口标题栏 | 仅文本 | 文本 + 关闭按钮 |
|
||||
| Grid 行数 | 3 行 | 2 行 |
|
||||
| 按钮样式 | 传统按钮 | FluentIcon |
|
||||
|
||||
### 文本变化
|
||||
|
||||
| 位置 | 修改前 | 修改后 |
|
||||
|------|--------|--------|
|
||||
| 查找按钮 | "查找更多组件" | "查找更多小组件" |
|
||||
| 添加按钮 | "添加" | "添加小组件" |
|
||||
|
||||
---
|
||||
|
||||
## 代码审查要点
|
||||
|
||||
### 潜在问题
|
||||
|
||||
1. **用户交互变化**:
|
||||
- ⚠️ 中风险：移除了底部"关闭"按钮，用户需要使用标题栏的关闭按钮
|
||||
- 确认用户是否习惯使用标题栏关闭按钮
|
||||
|
||||
2. **移动端适配**:
|
||||
- 自定义关闭按钮尺寸较小（32x32），在触摸设备上可能需要增大
|
||||
|
||||
3. **可访问性**:
|
||||
- 需要确保关闭按钮有适当的键盘快捷键支持（通常是 Escape 键）
|
||||
- 确认焦点顺序是否合理
|
||||
|
||||
### 建议
|
||||
|
||||
- ✅ **设计一致性**: 使用 FluentIcon 符合现代 UI 设计趋势
|
||||
- ✅ **空间优化**: 移除底部栏使界面更简洁
|
||||
- ⚠️ **测试建议**: 在不同屏幕尺寸下测试窗口布局
|
||||
- 📝 **文档建议**: 如果这是用户体验的重大变化，考虑更新相关文档
|
||||
|
||||
---
|
||||
|
||||
## 影响范围
|
||||
|
||||
- **UI/UX**: 显著影响用户界面和交互方式
|
||||
- **用户体验**: 界面更简洁，但关闭操作位置变化
|
||||
- **代码维护**: 两个 XAML 文件，变更清晰
|
||||
|
||||
---
|
||||
|
||||
## 设计评估
|
||||
|
||||
### 优点
|
||||
|
||||
1. ✅ **现代设计**: 使用 FluentIcon，符合 Fluent Design System
|
||||
2. ✅ **简化布局**: 移除多余的底部栏，界面更清爽
|
||||
3. ✅ **文本优化**: "添加小组件"比"添加"更明确
|
||||
4. ✅ **视觉一致性**: 水平居中对齐提升文本可读性
|
||||
|
||||
### 需要注意
|
||||
|
||||
1. ⚠️ **交互一致性**: 确保用户知道如何使用新的关闭按钮
|
||||
2. ⚠️ **键盘支持**: 验证 Escape 键等快捷键仍然有效
|
||||
3. ⚠️ **触摸友好**: 检查按钮尺寸是否适合触摸操作
|
||||
|
||||
---
|
||||
|
||||
## 总结
|
||||
|
||||
这是一次成功的 UI 优化提交，通过以下改进提升了用户体验：
|
||||
1. 使用 FluentIcon 替换传统按钮，更现代
|
||||
2. 移除底部关闭栏，简化布局
|
||||
3. 优化按钮文本，提高清晰度
|
||||
4. 统一文本对齐方式
|
||||
|
||||
**建议**: ✅ 可以合并，但建议在合并后进行 UI 测试以验证用户体验。
|
||||
111
docs/auto_commit_md/20250525_791e38d.md
Normal file
111
docs/auto_commit_md/20250525_791e38d.md
Normal file
@@ -0,0 +1,111 @@
|
||||
# Git 提交分析报告
|
||||
|
||||
**提交哈希**: 791e38d55ebef9c6cb568c72964ccac274141d1e
|
||||
**提交时间**: 2026-05-25 11:12:15 +0800
|
||||
**作者**: lincube \<lincube3@hotmail.com\>
|
||||
**提交信息**: fix.修复了错误的AirAppHost打包流程
|
||||
|
||||
---
|
||||
|
||||
## 变更统计
|
||||
|
||||
- **修改文件数**: 1
|
||||
- **新增行数**: 0
|
||||
- **删除行数**: 42
|
||||
- **净变更行数**: -42
|
||||
|
||||
### 变更文件
|
||||
|
||||
| 文件 | 变更类型 | 变更行数 |
|
||||
|------|---------|---------|
|
||||
| .github/workflows/release.yml | 删除 | -42 |
|
||||
|
||||
---
|
||||
|
||||
## 详细变更分析
|
||||
|
||||
### 1. .github/workflows/release.yml
|
||||
|
||||
**变更类型**: 大规模删除操作
|
||||
|
||||
**删除内容**:
|
||||
移除了整个 `Publish AirAppHost` GitHub Actions 步骤，包含了：
|
||||
- 条件化构建逻辑（self-contained vs lite 版本）
|
||||
- x64 架构的发布配置
|
||||
- 多行 PowerShell 命令调用
|
||||
|
||||
**具体删除代码**:
|
||||
```yaml
|
||||
- name: Publish AirAppHost
|
||||
run: |
|
||||
$arch = "${{ matrix.arch }}"
|
||||
$selfContained = "${{ matrix.self_contained }}" -eq "true"
|
||||
$publishDir = if ($selfContained) { "publish/windows-$arch" } else { "publish/windows-$arch-lite" }
|
||||
|
||||
if ($selfContained) {
|
||||
dotnet publish LanMountainDesktop.AirAppHost/LanMountainDesktop.AirAppHost.csproj `
|
||||
-c Release `
|
||||
-o ./$publishDir `
|
||||
--self-contained:false `
|
||||
# ... 其他参数
|
||||
} else {
|
||||
# ... else 分支的发布配置
|
||||
}
|
||||
shell: pwsh
|
||||
```
|
||||
|
||||
**变更说明**:
|
||||
- 移除了错误的 AirAppHost 打包流程
|
||||
- 这是一个修复性提交，旨在纠正之前的错误配置
|
||||
|
||||
---
|
||||
|
||||
## 代码审查要点
|
||||
|
||||
### 潜在问题
|
||||
|
||||
1. **修复范围**: 需要确认这个删除操作是完整的，之前的 AirAppHost 发布流程中是否还有其他相关的配置需要清理。
|
||||
|
||||
2. **版本兼容**: 移除 AirAppHost 发布步骤后，需要确认：
|
||||
- 其他工作流步骤是否依赖此步骤的输出
|
||||
- 发布流程的其他部分是否需要相应调整
|
||||
|
||||
3. **回归风险**:
|
||||
- ⚠️ 高风险：这是一个破坏性变更，需要在 CI/CD 环境中验证
|
||||
- 需要检查是否有其他工作流依赖于这个步骤的产物
|
||||
|
||||
### 建议
|
||||
|
||||
- ✅ **必要性**: 这个修复是必要的，移除了错误的打包流程
|
||||
- ⚠️ **验证要求**: 必须运行完整的 CI/CD 流程以验证没有破坏其他功能
|
||||
- 📝 **文档建议**: 考虑添加注释说明为什么移除了这个步骤，或添加相关的 issue/PR 链接
|
||||
- 🔍 **审查建议**: 确认是否需要在其他位置重新实现正确的 AirAppHost 打包流程
|
||||
|
||||
---
|
||||
|
||||
## 影响范围
|
||||
|
||||
- **CI/CD**: 显著影响发布工作流
|
||||
- **构建系统**: AirAppHost 的打包流程被禁用
|
||||
- **部署**: 可能影响最终发布包的内容
|
||||
- **功能影响**: 可能有功能影响，取决于 AirAppHost 的用途
|
||||
|
||||
---
|
||||
|
||||
## 相关上下文
|
||||
|
||||
根据提交信息 "修复了错误的 AirAppHost 打包流程"，这表明：
|
||||
1. 之前的 AirAppHost 发布流程配置有误
|
||||
2. 此提交是纠正错误的第一步或唯一步骤
|
||||
3. 可能需要进一步的后续提交来实现正确的打包流程
|
||||
|
||||
---
|
||||
|
||||
## 总结
|
||||
|
||||
这是一个重要的 CI/CD 修复提交，移除了错误的 AirAppHost 打包流程。虽然涉及大量代码删除，但这是修复性的，有助于恢复正确的构建流程。
|
||||
|
||||
**建议**: ✅ 可以合并，但需要：
|
||||
1. 在 CI 环境中完整测试发布流程
|
||||
2. 确认是否需要添加正确的 AirAppHost 打包配置
|
||||
3. 检查是否有其他工作流依赖于此步骤
|
||||
226
docs/auto_commit_md/20250525_SUMMARY.md
Normal file
226
docs/auto_commit_md/20250525_SUMMARY.md
Normal file
@@ -0,0 +1,226 @@
|
||||
# 2026-05-25 Git 提交汇总报告
|
||||
|
||||
**生成时间**: 2026-05-25 12:00:00
|
||||
**总提交数**: 5
|
||||
|
||||
---
|
||||
|
||||
## 提交概览
|
||||
|
||||
| # | 时间 | 提交哈希 | 作者 | 提交信息 | 风险等级 |
|
||||
|---|------|---------|------|---------|---------|
|
||||
| 1 | 11:54:04 | [cc85638](20250525_cc85638.md) | lincube | Update LanMountainDesktop.iss | 🟢 低 |
|
||||
| 2 | 11:12:15 | [791e38d](20250525_791e38d.md) | lincube | fix.修复了错误的AirAppHost打包流程 | 🔴 高 |
|
||||
| 3 | 10:16:00 | [75aed3f](20250525_75aed3f.md) | lincube | changed.调整了桌面组件库的UI | 🟡 中 |
|
||||
| 4 | 09:32:58 | [01cf32a](20250525_01cf32a.md) | lincube | changed.调整融合桌面组库的相关圆角 | 🟢 低 |
|
||||
| 5 | 01:24:18 | [12f0caa](20250525_12f0caa.md) | lincube | fix.继续修复 .NET运行时问题 | 🟡 中 |
|
||||
|
||||
---
|
||||
|
||||
## 变更统计总览
|
||||
|
||||
### 文件变更统计
|
||||
|
||||
| 指标 | 数量 |
|
||||
|------|------|
|
||||
| 修改文件总数 | 12 |
|
||||
| 新增代码行数 | 188 |
|
||||
| 删除代码行数 | 61 |
|
||||
| 净增加行数 | +127 |
|
||||
|
||||
### 按文件类型分布
|
||||
|
||||
| 文件类型 | 数量 | 说明 |
|
||||
|---------|------|------|
|
||||
| C# (.cs) | 4 | 核心逻辑和测试 |
|
||||
| XAML (.axaml) | 3 | UI 定义 |
|
||||
| YAML (.yml) | 1 | CI/CD 配置 |
|
||||
| Pascal Script (.iss) | 2 | 安装程序脚本 |
|
||||
| C# 代码后端 (.axaml.cs) | 2 | UI 逻辑 |
|
||||
|
||||
---
|
||||
|
||||
## 重点提交分析
|
||||
|
||||
### 🔴 高风险提交
|
||||
|
||||
#### 2. [791e38d](20250525_791e38d.md) - 修复 AirAppHost 打包流程
|
||||
|
||||
**影响**:
|
||||
- 移除 42 行 CI/CD 配置代码
|
||||
- 可能影响发布工作流
|
||||
|
||||
**建议**:
|
||||
- ✅ 必须在 CI 环境中完整测试
|
||||
- ⚠️ 确认是否需要重新实现正确的打包流程
|
||||
- ⚠️ 检查其他工作流依赖
|
||||
|
||||
---
|
||||
|
||||
### 🟡 中等风险提交
|
||||
|
||||
#### 3. [75aed3f](20250525_75aed3f.md) - UI 调整
|
||||
|
||||
**影响**:
|
||||
- 窗口布局重构
|
||||
- 关闭按钮位置变更
|
||||
|
||||
**建议**:
|
||||
- ✅ 进行 UI 兼容性测试
|
||||
- ⚠️ 验证键盘快捷键（Escape）仍然有效
|
||||
- ⚠️ 检查触摸设备上的交互体验
|
||||
|
||||
#### 5. [12f0caa](20250525_12f0caa.md) - .NET 运行时检测修复
|
||||
|
||||
**影响**:
|
||||
- 核心功能增强
|
||||
- 新增 6 个单元测试
|
||||
|
||||
**建议**:
|
||||
- ✅ 代码质量优秀，可以合并
|
||||
- ⚠️ 建议在多个环境中验证运行时检测
|
||||
- ⚠️ 测试按用户安装场景
|
||||
|
||||
---
|
||||
|
||||
### 🟢 低风险提交
|
||||
|
||||
#### 1. [cc85638](20250525_cc85638.md) - ISS 脚本优化
|
||||
|
||||
- 简单的代码风格调整
|
||||
- 无功能变更
|
||||
|
||||
#### 4. [01cf32a](20250525_01cf32a.md) - 圆角调整
|
||||
|
||||
- 增强图标解析功能
|
||||
- 添加动态圆角支持
|
||||
- 代码重构清晰
|
||||
|
||||
---
|
||||
|
||||
## 功能领域分布
|
||||
|
||||
### 按领域分类
|
||||
|
||||
| 功能领域 | 提交数 | 占比 |
|
||||
|---------|--------|------|
|
||||
| UI/UX | 2 | 40% |
|
||||
| CI/CD | 1 | 20% |
|
||||
| 核心功能 | 1 | 20% |
|
||||
| 安装程序 | 2 | 40% |
|
||||
|
||||
*注：一个提交可能涉及多个功能领域*
|
||||
|
||||
### 主要功能模块
|
||||
|
||||
1. **UI 组件系统**: 2 个提交
|
||||
- FusedDesktopComponentLibraryControl
|
||||
- ComponentCategoryIconResolver
|
||||
|
||||
2. **CI/CD 管道**: 1 个提交
|
||||
- GitHub Actions workflow
|
||||
|
||||
3. **.NET 运行时检测**: 1 个提交
|
||||
- DotNetRuntimeProbe
|
||||
|
||||
4. **安装程序**: 2 个提交
|
||||
- Inno Setup 脚本
|
||||
|
||||
---
|
||||
|
||||
## 代码质量评估
|
||||
|
||||
### 整体评分
|
||||
|
||||
| 指标 | 评分 | 说明 |
|
||||
|------|------|------|
|
||||
| 代码规范 | ⭐⭐⭐⭐⭐ | 遵循项目代码风格 |
|
||||
| 测试覆盖 | ⭐⭐⭐⭐ | 新增 8 个测试用例 |
|
||||
| 错误处理 | ⭐⭐⭐⭐ | 周全的错误和边界检查 |
|
||||
| 文档 | ⭐⭐⭐ | 变更说明清晰 |
|
||||
| 安全性 | ⭐⭐⭐⭐⭐ | 路径验证完善 |
|
||||
|
||||
### 代码审查统计
|
||||
|
||||
| 类型 | 数量 |
|
||||
|------|------|
|
||||
| 优点 | 12 |
|
||||
| 建议 | 8 |
|
||||
| 注意事项 | 5 |
|
||||
|
||||
---
|
||||
|
||||
## 风险与建议
|
||||
|
||||
### 需要关注的风险
|
||||
|
||||
1. **CI/CD 变更风险** 🔴
|
||||
- AirAppHost 打包流程移除
|
||||
- 需要完整验证发布流程
|
||||
|
||||
2. **UI 交互变更** 🟡
|
||||
- 关闭按钮位置变化
|
||||
- 需要用户接受度测试
|
||||
|
||||
3. **运行时检测** 🟢
|
||||
- 功能增强
|
||||
- 需要多环境验证
|
||||
|
||||
### 建议的测试计划
|
||||
|
||||
#### 必须测试
|
||||
|
||||
- [ ] 完整的 CI/CD 发布流程
|
||||
- [ ] 多种 .NET 运行时安装场景
|
||||
- [ ] UI 组件在不同屏幕尺寸下的显示
|
||||
|
||||
#### 建议测试
|
||||
|
||||
- [ ] 触摸设备上的 UI 交互
|
||||
- [ ] 按用户 vs 系统级别的运行时检测
|
||||
- [ ] 键盘快捷键功能
|
||||
|
||||
---
|
||||
|
||||
## 合并建议
|
||||
|
||||
### 总体建议
|
||||
|
||||
✅ **可以合并**: 所有提交的代码质量和意图都很好
|
||||
|
||||
### 合并顺序建议
|
||||
|
||||
1. **第一步**: 合并低风险提交（1, 4）
|
||||
- 风险最低，不会影响主要功能
|
||||
|
||||
2. **第二步**: 合并中等风险提交（3, 5）
|
||||
- 需要进行测试验证
|
||||
|
||||
3. **第三步**: 合并高风险提交（2）
|
||||
- 需要完整 CI/CD 测试
|
||||
- 可能需要额外的后续工作
|
||||
|
||||
---
|
||||
|
||||
## 相关资源
|
||||
|
||||
### 详细分析报告
|
||||
|
||||
- [cc85638 - ISS 脚本优化](20250525_cc85638.md)
|
||||
- [791e38d - AirAppHost 打包流程修复](20250525_791e38d.md)
|
||||
- [75aed3f - UI 调整](20250525_75aed3f.md)
|
||||
- [01cf32a - 圆角和图标解析](20250525_01cf32a.md)
|
||||
- [12f0caa - .NET 运行时检测](20250525_12f0caa.md)
|
||||
|
||||
---
|
||||
|
||||
## 总结
|
||||
|
||||
今天的提交整体质量很高，主要集中在：
|
||||
|
||||
1. **UI/UX 改进**: 40% 的提交涉及用户界面优化
|
||||
2. **核心功能增强**: .NET 运行时检测是重要的功能改进
|
||||
3. **CI/CD 优化**: 修复了错误的打包流程
|
||||
4. **代码质量**: 遵循项目规范，测试覆盖良好
|
||||
|
||||
**建议**: ✅ 可以按计划合并所有提交，建议在高风险提交合并前进行充分的 CI/CD 测试。
|
||||
87
docs/auto_commit_md/20250525_cc85638.md
Normal file
87
docs/auto_commit_md/20250525_cc85638.md
Normal file
@@ -0,0 +1,87 @@
|
||||
# Git 提交分析报告
|
||||
|
||||
**提交哈希**: cc85638a374b061018c9a3a691e55f6aa770f767
|
||||
**提交时间**: 2026-05-25 11:54:04 +0800
|
||||
**作者**: lincube \<lincube3@hotmail.com\>
|
||||
**提交信息**: Update LanMountainDesktop.iss
|
||||
|
||||
---
|
||||
|
||||
## 变更统计
|
||||
|
||||
- **修改文件数**: 1
|
||||
- **新增行数**: 0
|
||||
- **删除行数**: 0
|
||||
- **变更行数**: 2
|
||||
|
||||
### 变更文件
|
||||
|
||||
| 文件 | 变更类型 | 变更行数 |
|
||||
|------|---------|---------|
|
||||
| LanMountainDesktop/installer/LanMountainDesktop.iss | 修改 | +2 / -2 |
|
||||
|
||||
---
|
||||
|
||||
## 详细变更分析
|
||||
|
||||
### 1. LanMountainDesktop/installer/LanMountainDesktop.iss
|
||||
|
||||
**变更位置**:
|
||||
- 第 560 行附近：`GetTargetDotNetDesktopRuntimePath` 函数
|
||||
- 第 577 行附近：`GetDotNetRuntimeDownloadUrlX64` 函数
|
||||
|
||||
**具体变更**:
|
||||
```diff
|
||||
@@ -557,7 +557,7 @@ begin
|
||||
if '{#MyAppArch}' = 'x64' then
|
||||
begin
|
||||
Result := ExpandConstant('{commonpf64}\dotnet\shared\Microsoft.WindowsDesktop.App');
|
||||
- end;
|
||||
+ end
|
||||
else
|
||||
begin
|
||||
Result := ExpandConstant('{commonpf}\dotnet\shared\Microsoft.WindowsDesktop.App');
|
||||
@@ -574,7 +574,7 @@ begin
|
||||
if '{#MyAppArch}' = 'x64' then
|
||||
begin
|
||||
Result := DotNetRuntimeDownloadUrlX64;
|
||||
- end;
|
||||
+ end
|
||||
else
|
||||
begin
|
||||
Result := DotNetRuntimeDownloadUrlX86;
|
||||
```
|
||||
|
||||
**变更说明**:
|
||||
- 移除了两处 `if-else` 语句后的多余分号（`;`）
|
||||
- 这是代码风格的一致性调整
|
||||
|
||||
---
|
||||
|
||||
## 代码审查要点
|
||||
|
||||
### 潜在问题
|
||||
|
||||
1. **分号语法问题**: 此次修改移除了 Pascal Script 中 `if-else` 语句后的多余分号。虽然在某些 Pascal 方言中这可能不会导致编译错误，但删除分号是正确的做法，因为 `else` 关键字不应该与分号一起使用。
|
||||
|
||||
### 建议
|
||||
|
||||
- ✅ **良好实践**: 移除多余分号，保持代码风格一致
|
||||
- ⚠️ **注意**: 确保其他类似的 `if-else` 语句也遵循相同的风格
|
||||
- 📝 **建议**: 考虑在整个 ISS 脚本中进行一次全局的代码风格检查
|
||||
|
||||
---
|
||||
|
||||
## 影响范围
|
||||
|
||||
- **安装程序**: 影响 Windows 安装包的打包流程
|
||||
- **用户体验**: 无直接影响
|
||||
- **功能影响**: 无功能变更，仅代码风格调整
|
||||
|
||||
---
|
||||
|
||||
## 总结
|
||||
|
||||
本次提交是一个简单的代码风格优化，移除了 Inno Setup 脚本中的多余分号。虽然变更很小，但有助于提高代码质量和一致性。
|
||||
|
||||
**建议**: ✅ 可以合并
|
||||
Reference in New Issue
Block a user