Skip to content

test(completion): load native QA startup from an isolated file - #495

Merged
Kuddev merged 1 commit into
mainfrom
fix/completion-native-qa-startup-20261006
Oct 6, 2026
Merged

Kuddev merged 1 commit into
mainfrom
fix/completion-native-qa-startup-20261006

Conversation

@Kuddev

@Kuddev Kuddev commented Oct 6, 2026

Copy link
Copy Markdown
Owner

范围

Refs #358. 只修复原生补全验收的启动器,不修改发布构建中的补全、Shell 或窗口逻辑。

现有验收把完整初始化编码后传给 PowerShell。当前主线上一次真实独立桌面运行在输入编辑器启动前静默超时,ready/预测配置文件均未生成,Tab 场景尚未开始。不能把这个失败当成补全回归,也不能通过扩大超时处理。

修改

将完全相同的初始化内容写入本次 QA 输出目录的 UTF-8 BOM 文件,再用现有参数中的 -File 入口加载。BOM 保持 Windows PowerShell 5.1 对中文脚本内容的解释,文件参数保留 Windows 命令行引号。

仅 1 个源码文件的 cfg(test) 函数发生变化。未改初始化内容、用例断言、超时、忽略条件、CI 选择或发布代码。

已执行验证

  • 原启动器:真实 PowerShell 7 验收在原生编辑器启动前超时,原失败日志保留。
  • 文件入口:PowerShell 7.6.4 / PSReadLine 2.4.5 的 11 个场景通过,含三种补全模式、完整 Unicode 光标编辑、取消及真实 History 灰色预测隔离。
  • Windows PowerShell 5.1 / PSReadLine 2.0.0 的 8 个场景通过,QA 路径包含空格与中文。
  • 使用固定已校验的 ConPTY 文件对;均运行于不切换的独立桌面,没有全局输入注入。
  • 两项已有历史/混合模式回归在显式隔离配置下通过;格式、平台条件预算和架构检查通过。

验证边界

这使既有原生验收能进入实际补全场景,不宣称 #358 的所有配置都已修复。顶部/侧边完整工作区的窗口级按键分发正在单独核实,不包含在这一个文件的修复里。最终提交仍需远端必需检查。

@github-actions github-actions Bot added the size/S ≤100 行源码改动 label Oct 6, 2026

@Kuddev Kuddev left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

维护者复核 bad7341:仅 cfg(test) 的原生验收启动函数修改,完整初始化内容、断言、超时、忽略条件均保留;UTF-8 BOM 文件与带引号的 -File 参数覆盖 PS5.1 和 PS7。确认没有发布代码、依赖、线程、轮询、推广或门禁放宽。原启动失败记录保留;文件入口下 PS7 的 11 个场景与 PS5 的 8 个场景成功,使用隔离配置和不切换的独立桌面。当前提交五平台实际原生测试及 10 项必需检查全部成功,主线为其祖先,评审线程已核对。该 PR 修复验收入口,不等于宣布 #358 或顶部布局问题已全部修复。此为维护者本人复核记录,不是独立审批。

@Kuddev
Kuddev merged commit f34d9c6 into main Oct 6, 2026
16 checks passed
@Kuddev
Kuddev deleted the fix/completion-native-qa-startup-20261006 branch October 7, 2026 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/S ≤100 行源码改动

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant