fix(m196): GUI 产物 PE Subsystem → WINDOWS_GUI(2)，并用 AttachConsole 保住 CLI 输出
群友 2026-11-08 实测发布构建「运行时会带一个控制台窗口在底下」。根因：产物 PE 头 Subsystem 仍是 CUI(3)。同一个修复 2026-11-01 做过一次（37d3039），但只改了已被 workspace exclude 的 Slint 树（crates/app），从未到达 iced 产物。 - crates/app-iced/src/main.rs：加 #![cfg_attr(windows, windows_subsystem = "windows")]， 并写清为什么放 bin 层而不是 build.rs（PE 子系统是每个 bin 目标的属性）。 - crates/app-iced/src/platform/win32/console.rs（新）：AttachConsole(ATTACH_PARENT_PROCESS) 的落点 + 判据 should_attach（纯函数，可断言）。GUI 子系统进程不继承父控制台，实测 从 cmd 启动时三个 std 句柄全是 0；附着后 GetStdHandle 立刻给出控制台句柄（0x0→0x208）， 所以不改 std 句柄、不碰用户的重定向（ATTACH_PARENT_PROCESS 只会附到已存在的控制台， 永不新建窗口）。 - packaging/check_pe_subsystem.py（新）：读产物 PE 头 Subsystem 字段（GUI 必须 2、 CLI ull 必须 3），非 2 即红；--self-test 注入 5 条合成 PE 回归。 - 接进 release-build.yml（发布侧两槽位）与 ci.yml build job（PR 阶段）。 - guards.yml：unsafe 落点期望集合变成两处（console.rs + transparency.rs），集合相等判据其余不变； 平台层所有「全仓唯一 unsafe 落点」措辞同步改正。
This commit is contained in:
tower-worker committed
1 parent
30b61e2684
commit
14f9318e09
15 files changed
+693
-36
No files matched your search
@@ -207,6 +207,18 @@ jobs:
|
||||
# 同上：`--workspace` 不含 `unlucky-app`（M118）。被摘掉的正是 release 档
|
||||
# （lto=fat + codegen-units=1）最贵的那棵 slint + usvg 树。
|
||||
- run: cargo build --workspace --release
|
||||
# M196：PR 阶段就拦住「产物退回 CUI(3)」。判据是产物 PE 头的 `Subsystem` 字段，不是源码文本——
|
||||
# 这个 bug 的原始形态正是「源码写了、打在另一棵树上」，任何只看 `grep windows_subsystem` 的守卫
|
||||
# 都抓不住它。发布侧同一份脚本再跑一次（`release-build.yml`），两处判据同一个真源。
|
||||
- name: PE 子系统门禁（GUI 产物必须 WINDOWS_GUI/2，CLI 产物必须 CONSOLE/3）
|
||||
if: runner.os == 'Windows'
|
||||
shell: bash
|
||||
run: |
|
||||
set -euo pipefail
|
||||
PY=$(command -v python3 || command -v python)
|
||||
"$PY" packaging/check_pe_subsystem.py \
|
||||
--gui target/release/unlucky-launcher-iced.exe \
|
||||
--cli target/release/ull.exe
|
||||
|
||||
# 运行时冒烟（M62）：Linux 上**真渲染一帧**，补上「能编译 ≠ 能跑」的空白。
|
||||
# 上面四个 job 都只证明代码编得过、测得过，从不创建真实图形窗口；本 job 在 Xvfb 虚拟显示里
|
||||
|
||||
@@ -10,7 +10,7 @@ name: Guards
|
||||
# 3. design-zero-dep —— crates/design 零第三方依赖（M61 / M87）：依赖表（含 dev/build）必须为空 + 依赖树只允许出现自身
|
||||
# 4. file-size —— 全仓代码/文档单文件 ≤400 行（用户硬约束）
|
||||
# 5. licenses —— cargo-deny：许可证白名单 + GPL 依赖拦截 + 供应链来源校验
|
||||
# 6. unsafe-scope —— 全仓 `allow(unsafe_code)` 的文件集合必须恰好只有一个（D36 / M91）
|
||||
# 6. unsafe-scope —— 全仓 `allow(unsafe_code)` 的文件集合必须恰好等于期望集合（D36 / M91；M196 起两处）
|
||||
#
|
||||
# M118（D26）：根 Cargo.toml 已 `exclude = ["crates/app"]`（Slint 退役树）。逐条核对过本文件，
|
||||
# **没有**任何一条守卫以 `--workspace` 为前提，所以排除不会让它们空转：
|
||||
@@ -179,22 +179,22 @@ jobs:
|
||||
fi
|
||||
echo "OK: unlucky-design 依赖树只有自身，零第三方依赖"
|
||||
|
||||
# D36 定案：整窗 α 是全仓**唯一**需要 `unsafe` 的能力（Win32 的
|
||||
# `SetLayeredWindowAttributes` 没有安全替代），落点收在一个文件里。
|
||||
# 判据必须是**集合相等**而不是「黑名单命中」：
|
||||
# - 新增第二个显式放行 unsafe 的文件 → 红（黑名单写法在这里会假绿）；
|
||||
# D36 定案：整窗 α 需要 `unsafe`（`SetLayeredWindowAttributes` 没有安全替代）。
|
||||
# M196 加了第二处：控制台附着（`AttachConsole`）同样只有 Win32 一条路。两处各自成文件，
|
||||
# 判据是**集合相等**而不是「黑名单命中」：
|
||||
# - 任何第三个显式放行 unsafe 的文件 → 红（黑名单写法在这里会假绿）；
|
||||
# - 落点文件把放行删掉 / 改名 / 搬走 → 红（否则门禁守着一条已经不存在的路径）；
|
||||
# - 多行属性、同行合并属性、多行 `cfg_attr`、`expect(unsafe_code)` 都已覆盖
|
||||
# （r2 前的行级正则对前两种会假绿，当时只能靠 ci.yml 的 fmt job 兜底；r3 起由词法器直接抓）。
|
||||
unsafe-scope:
|
||||
name: unsafe 落点守门（集合必须恰好一个文件）
|
||||
name: unsafe 落点守门（集合必须恰好等于期望）
|
||||
runs-on: ubuntu-latest
|
||||
steps:
|
||||
- uses: actions/checkout@v4
|
||||
- name: 全仓显式放行 unsafe 的文件集合 == 唯一落点
|
||||
- name: 全仓显式放行 unsafe 的文件集合 == 期望集合
|
||||
run: |
|
||||
set -uo pipefail
|
||||
EXPECTED='crates/app-iced/src/platform/win32/transparency.rs'
|
||||
EXPECTED=$'crates/app-iced/src/platform/win32/console.rs\ncrates/app-iced/src/platform/win32/transparency.rs'
|
||||
# 扫描口径（M91 r3）：一个小词法器，而不是正则。
|
||||
#
|
||||
# 为什么必须用词法器：r2 的写法是「按全文方括号净深度合并续行」，但 Rust 里
|
||||
@@ -332,22 +332,23 @@ jobs:
|
||||
actual=$(git grep -l 'unsafe_code' -- '*.rs' 2>/dev/null | LC_ALL=C sort | while IFS= read -r f; do
|
||||
if scan "$f"; then printf '%s\n' "$f"; fi
|
||||
done | LC_ALL=C sort || true)
|
||||
echo "期望（恰好这一个）：$EXPECTED"
|
||||
echo "期望（恰好这两处，集合相等）：$EXPECTED"
|
||||
echo "实际扫到的集合："
|
||||
if [ -n "$actual" ]; then printf '%s\n' "$actual" | sed 's/^/ - /'; else echo " （空）"; fi
|
||||
if [ "$actual" != "$EXPECTED" ]; then
|
||||
extra=$(printf '%s\n' "$actual" | grep -vxF "$EXPECTED" || true)
|
||||
if [ -n "$extra" ]; then
|
||||
echo "::error::多出来的 unsafe 落点（D36 只允许一个）："
|
||||
echo "::error::多出来的 unsafe 落点（不在期望集合里）："
|
||||
printf '%s\n' "$extra" | sed 's/^/ - /'
|
||||
fi
|
||||
if ! printf '%s\n' "$actual" | grep -qxF "$EXPECTED"; then
|
||||
echo "::error file=$EXPECTED::这个文件里没有显式放行 unsafe（allow / expect(unsafe_code)）—— 落点被搬走/改名/删了放行？守门路径已失效。"
|
||||
if [ -n "$(printf '%s\n' "$EXPECTED" | grep -vxF "$actual" || true)" ]; then
|
||||
echo "::error::期望集合里的落点没被扫到（放行属性被删 / 文件改名搬走？）："
|
||||
printf '%s\n' "$EXPECTED" | grep -vxF "$actual" | sed 's/^/ - /' || true
|
||||
fi
|
||||
echo "::error::整窗 α 的 FFI 只允许住在 $EXPECTED（D36）。需要新的 unsafe 能力请先立 mission：改这条守门 + 更新 D36 的定案，不要顺手放行。"
|
||||
echo "::error::unsafe 的 FFI 只允许住在上面的期望集合里（D36 / M196）。新的 unsafe 能力要先立 mission：改守门 + 更新定案，不要顺手放行。"
|
||||
exit 1
|
||||
fi
|
||||
echo "OK: 全仓显式放行 unsafe 的文件恰好只有 $EXPECTED"
|
||||
echo "OK: 全仓显式放行 unsafe 的文件恰好等于期望集合"
|
||||
|
||||
file-size:
|
||||
name: 文件行数 ≤400 守门
|
||||
|
||||
@@ -259,6 +259,26 @@ jobs:
|
||||
echo "::error::静态 CRT 未生效：exe 仍动态依赖 VC++ 可再发行包"; exit 1
|
||||
fi
|
||||
|
||||
# M196：**读产物 PE 头的 `Subsystem`**，不读源码。同一个修复 2026-11-01 曾打在已退役的 Slint 树
|
||||
# （`37d3039` 只改了 `crates/app/src/main.rs`，而 `crates/app` 自 M118 起被 workspace `exclude`、
|
||||
# 不进发布矩阵），于是「代码里写过」与「产物里有没有」彻底脱钩——双击多一个控制台窗口这个 bug
|
||||
# 就是这么活到 2026-11-08 的。本步只认那 4 个字节：GUI 产物必须是 WINDOWS_GUI(2)，非 2 即红。
|
||||
# 反方向一并守：CLI 产物 `ull.exe` 必须仍是 CONSOLE(3)——它靠真控制台做 stdout/stderr，
|
||||
# 谁把属性抄到它头上，`ull --version | …` 会静默丢输出。守门本体在 packaging/check_pe_subsystem.py
|
||||
# （零第三方依赖，本地同一份代码可跑；`--self-test` 注入 5 条回归证明它会红）。
|
||||
- name: PE 子系统门禁（GUI 产物必须 WINDOWS_GUI/2，CLI 产物必须 CONSOLE/3）
|
||||
if: runner.os == 'Windows'
|
||||
shell: bash
|
||||
env:
|
||||
BUILD_DIR: ${{ inputs.build_dir }}
|
||||
run: |
|
||||
set -euo pipefail
|
||||
PY=$(command -v python3 || command -v python)
|
||||
# 读数（含文件字节偏移）直接进日志：红的时候要看的就是这个数，而不是「脚本说不对」。
|
||||
"$PY" packaging/check_pe_subsystem.py \
|
||||
--gui "$BUILD_DIR/unlucky-launcher-iced.exe" \
|
||||
--cli "$BUILD_DIR/ull.exe"
|
||||
|
||||
- name: 打包（产物 + SHA256SUMS）
|
||||
shell: bash
|
||||
env:
|
||||
|
||||
Reference in new issue
Block a user