ci: 移除发布流程中的 smoke check

HanFengRuYue · HanFengRuYue · commit 8185a3fca096 · 2026-04-11T16:20:06.000+08:00
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -437,73 +437,6 @@ jobs:
             @{ Name='lite';     SC='false'; Suffix='-lite';     Edition='lite' }
           )
 
-          function Test-FrontendSmoke {
-            param(
-              [Parameter(Mandatory = $true)]
-              [string]$ExePath,
-              [Parameter(Mandatory = $true)]
-              [string]$Label
-            )
-
-            function Get-StatusCode {
-              param([string]$Url)
-              try {
-                return [int](Invoke-WebRequest -Uri $Url -UseBasicParsing -TimeoutSec 2).StatusCode
-              } catch {
-                if ($_.Exception.Response) {
-                  return [int]$_.Exception.Response.StatusCode
-                }
-                return $null
-              }
-            }
-
-            $probeRoot = Join-Path $env:RUNNER_TEMP ("xunitytoolkit-smoke-" + [guid]::NewGuid().ToString('N'))
-            $launcherDir = Join-Path $env:RUNNER_TEMP ("xunitytoolkit-launcher-" + [guid]::NewGuid().ToString('N'))
-            $port = Get-Random -Minimum 42000 -Maximum 52000
-            $proc = $null
-
-            try {
-              New-Item -ItemType Directory -Path $probeRoot -Force | Out-Null
-              New-Item -ItemType Directory -Path $launcherDir -Force | Out-Null
-
-              $settings = @{ aiTranslation = @{ port = $port } } | ConvertTo-Json -Depth 3
-              Set-Content -Path (Join-Path $probeRoot 'settings.json') -Value $settings -Encoding UTF8
-
-              $env:AppData__Root = $probeRoot
-              $proc = Start-Process -FilePath $ExePath -WorkingDirectory $launcherDir -PassThru
-
-              $rootStatus = $null
-              $apiStatus = $null
-              for ($i = 0; $i -lt 40; $i++) {
-                Start-Sleep -Milliseconds 500
-                $rootStatus = Get-StatusCode "http://127.0.0.1:$port/"
-                $apiStatus = Get-StatusCode "http://127.0.0.1:$port/api/settings/version"
-                if ($null -ne $rootStatus -and $null -ne $apiStatus) {
-                  break
-                }
-              }
-
-              if ($rootStatus -ne 200 -or $apiStatus -ne 200) {
-                $rootText = if ($null -eq $rootStatus) { 'n/a' } else { [string]$rootStatus }
-                $apiText = if ($null -eq $apiStatus) { 'n/a' } else { [string]$apiStatus }
-                throw ("Smoke check failed for {0}: root={1} api={2}" -f $Label, $rootText, $apiText)
-              }
-
-              Write-Host ("Smoke check passed for {0} (root={1}, api={2})" -f $Label, $rootStatus, $apiStatus)
-            } finally {
-              if ($proc -and -not $proc.HasExited) {
-                Stop-Process -Id $proc.Id -Force -ErrorAction SilentlyContinue
-              }
-              Remove-Item Env:AppData__Root -ErrorAction SilentlyContinue
-              if (Test-Path $probeRoot) {
-                Remove-Item $probeRoot -Recurse -Force -ErrorAction SilentlyContinue
-              }
-              if (Test-Path $launcherDir) {
-                Remove-Item $launcherDir -Recurse -Force -ErrorAction SilentlyContinue
-              }
-            }
-          }
-
           foreach ($ed in $editions) {
             $outDir = "Release\win-x64$($ed.Suffix)"
             Write-Host "`n--- Publishing edition: $($ed.Name) ---"
@@ -550,8 +483,6 @@ jobs:
               Copy-Item $updaterExe "$outDir/" -Force
             }
 
-            Test-FrontendSmoke -ExePath (Join-Path $outDir "XUnityToolkit-WebUI.exe") -Label $ed.Name
-
             $exeFile = Get-Item (Join-Path $outDir "XUnityToolkit-WebUI.exe")
             $sizeMB = [math]::Round($exeFile.Length / 1MB, 1)
             Write-Host "$($ed.Name) done (exe: $sizeMB MB)"
diff --git a/AGENTS.md b/AGENTS.md
@@ -127,7 +127,7 @@ dotnet build TranslatorEndpoint/TranslatorEndpoint.csproj -c Release
 - `XUnityToolkit-WebUI.csproj` 默认会在构建前自动执行前端 `npm install` + `npm run build`。
 - 前端开发代理到 `http://127.0.0.1:51821`，不要改成 `localhost`。
 - 完整 UI 预览优先看后端端口 `51821`，因为它同时承载静态前端和 API。
-- 发布产物首页可用性现在由 `build.ps1` 与 `.github/workflows/build.yml` 中的 `Test-FrontendSmoke` 双重守卫；它们会从错误工作目录启动 EXE，并要求 `/` 与 `/api/settings/version` 同时返回 `200`。
+- 发布流程当前不会在构建脚本或 GitHub Actions 中自动启动 EXE 做首页 smoke check；若需要运行态验收，请单独执行。
 
 ## 6. 运行时架构
 
@@ -456,7 +456,7 @@ CI：
 - CI 逻辑与 `build.ps1` 是两份并行维护的实现，改构建流程时必须双改
 - CI 不直接调用 `build.ps1`
 - 更新器是增量更新的关键组件，含备份、替换、删除、回滚逻辑
-- 首页静态资源可用性 smoke check 现在也是双维护实现；若调整启动端口解析、静态资源目录、启动方式或首页健康检查，必须同时更新 `build.ps1` 与 `.github/workflows/build.yml` 中的 `Test-FrontendSmoke`
+- 若调整启动端口解析、静态资源目录或启动方式，注意分别评估本地构建脚本与 GitHub Actions 的发布行为，但当前没有内置 smoke check 守卫
 - MSI 由 WiX 生成，且不同 edition 构建时必须注意清理 `Installer/obj/...`
 
 ## 14. 关键约束与不变量
@@ -656,7 +656,7 @@ CI：
 - `RecordError`、`NormalizeForCache`、`ApplicationStopping` 回调、日志级别过滤、SignalR 事件名与阶段名，都属于“改一处必须全链路核对”的同步点
 - 翻译解析契约、运行时占位符保护与 `Persistable` 过滤属于新的高频同步点；凡是新增翻译调用方或缓存写入点，都要核对是否错误接收了非结构化回退结果
 - `build.ps1` 与 `.github/workflows/build.yml` 不是同一实现的不同入口，而是两份并行维护脚本；流程、版本号、资源来源、构建 edition 发生变化时必须双改
-- 若变更首页可用性、静态资源目录、启动端口或启动方式，还要同步更新两处 `Test-FrontendSmoke`，确保“错误工作目录启动时 `/` 与 `/api/settings/version` 都返回 `200`”这个回归守卫不失效
+- 若变更首页可用性、静态资源目录、启动端口或启动方式，需要分别核对 `build.ps1` 与 `.github/workflows/build.yml` 的发布流程，但当前不再维护 `Test-FrontendSmoke` 回归守卫
 - Git 提交标题规范、`.github/workflows/build.yml` 中 `### Changelog` 的生成逻辑，以及 `XUnityToolkit-Vue/src/views/SettingsView.vue` 的 `typeLabels` / 正则解析属于联动点；若调整提交格式、更新内容展示样式或 changelog 生成方式，必须同时核对这三处，且注意 `--no-merges` 会让 merge commit 不进入工具箱更新列表
 - `llama.cpp` 版本更新需要同时同步 `build.ps1`、`build.yml`、`LocalLlmService.LlamaVersion`、下载资源命名模式、README/本手册说明
 
diff --git a/build.ps1 b/build.ps1
@@ -47,73 +47,6 @@ function Invoke-WithRetry {
     }
 }
 
-function Test-FrontendSmoke {
-    param(
-        [Parameter(Mandatory = $true)]
-        [string]$ExePath,
-        [Parameter(Mandatory = $true)]
-        [string]$Label
-    )
-
-    function Get-StatusCode {
-        param([string]$Url)
-        try {
-            return [int](Invoke-WebRequest -Uri $Url -UseBasicParsing -TimeoutSec 2).StatusCode
-        } catch {
-            if ($_.Exception.Response) {
-                return [int]$_.Exception.Response.StatusCode
-            }
-            return $null
-        }
-    }
-
-    $probeRoot = Join-Path $env:TEMP ("xunitytoolkit-smoke-" + [guid]::NewGuid().ToString('N'))
-    $launcherDir = Join-Path $env:TEMP ("xunitytoolkit-launcher-" + [guid]::NewGuid().ToString('N'))
-    $port = Get-Random -Minimum 42000 -Maximum 52000
-    $proc = $null
-
-    try {
-        New-Item -ItemType Directory -Path $probeRoot -Force | Out-Null
-        New-Item -ItemType Directory -Path $launcherDir -Force | Out-Null
-
-        $settings = @{ aiTranslation = @{ port = $port } } | ConvertTo-Json -Depth 3
-        Set-Content -Path (Join-Path $probeRoot 'settings.json') -Value $settings -Encoding UTF8
-
-        $env:AppData__Root = $probeRoot
-        $proc = Start-Process -FilePath $ExePath -WorkingDirectory $launcherDir -PassThru
-
-        $rootStatus = $null
-        $apiStatus = $null
-        for ($i = 0; $i -lt 40; $i++) {
-            Start-Sleep -Milliseconds 500
-            $rootStatus = Get-StatusCode "http://127.0.0.1:$port/"
-            $apiStatus = Get-StatusCode "http://127.0.0.1:$port/api/settings/version"
-            if ($null -ne $rootStatus -and $null -ne $apiStatus) {
-                break
-            }
-        }
-
-        if ($rootStatus -ne 200 -or $apiStatus -ne 200) {
-            $rootText = if ($null -eq $rootStatus) { 'n/a' } else { [string]$rootStatus }
-            $apiText = if ($null -eq $apiStatus) { 'n/a' } else { [string]$apiStatus }
-            throw ("Smoke check failed for {0}: root={1} api={2}" -f $Label, $rootText, $apiText)
-        }
-
-        Write-Host ("  [smoke] {0} passed (root={1}, api={2})" -f $Label, $rootStatus, $apiStatus) -ForegroundColor Green
-    } finally {
-        if ($proc -and -not $proc.HasExited) {
-            Stop-Process -Id $proc.Id -Force -ErrorAction SilentlyContinue
-        }
-        Remove-Item Env:AppData__Root -ErrorAction SilentlyContinue
-        if (Test-Path $probeRoot) {
-            Remove-Item $probeRoot -Recurse -Force -ErrorAction SilentlyContinue
-        }
-        if (Test-Path $launcherDir) {
-            Remove-Item $launcherDir -Recurse -Force -ErrorAction SilentlyContinue
-        }
-    }
-}
-
 try {
 
 # Ensure TLS 1.2+ for all HTTPS requests (PowerShell 5.1 defaults to TLS 1.0)
@@ -542,8 +475,6 @@ if (Test-Path $bundledSrc) {
     Write-Host "  Copied bundled assets." -ForegroundColor DarkGray
 }
 
-Test-FrontendSmoke -ExePath (Join-Path $OutputDir 'XUnityToolkit-WebUI.exe') -Label "$rid ($Edition)"
-
 $exeFile = Get-Item (Join-Path $OutputDir 'XUnityToolkit-WebUI.exe')
 $exeSize = [math]::Round($exeFile.Length / 1MB, 1)
 Write-Host "  $rid done (exe: $exeSize MB)" -ForegroundColor Green
