Skip to content

fix(startup): honor configured directory in resident launches - #492

Merged
Kuddev merged 2 commits into
mainfrom
fix/issue479-resident-startup-directory-20261006
Oct 5, 2026
Merged

Kuddev merged 2 commits into
mainfrom
fix/issue479-resident-startup-directory-20261006

Conversation

@Kuddev

@Kuddev Kuddev commented Oct 5, 2026

Copy link
Copy Markdown
Owner

用户结果

Refs #479. 补齐已有驻留实例时普通二次启动忽略“启动目录”的路径。冷启动部分已由 #485 修复,本 PR 不改会话恢复优先级。

修复逻辑

  • 二次启动目录顺序为:显式命令行目录、有效的启动目录设置、启动进程当前目录。
  • 设置目录的非空/存在/目录检查提取到 RuntimeSettings 的同一方法;现有 UI 新建和冷启动入口复用,持久化格式不变。
  • 显式目录仍保留上下文菜单修正、相对路径绝对化及原有无效路径行为。
  • 新窗口/新标签策略、指定 Shell、管理员隔离和已有交接协议均保持不变。
  • daemon 和带 -e 的命令启动提前退出交接检查,避免为这些路径额外读取设置。

范围和成本

只修改 3 个源码文件;无依赖、线程、轮询、配置项或协议字段增加。普通交接复用已有的一次设置读取;仅在没有显式目录时检查配置目录。没有修改实时终端、用户配置或发布资产。

已执行验证

  • Windows 产品测试目标编译通过,3 项新回归通过:中文/空格路径、普通二次启动、显式/相对目录优先级、失效/空白/文件设置回退,以及无效显式目录兼容。
  • 2 项既有冷启动回退测试、1 项既有 tab/window 可选 Shell 请求测试通过。
  • nebula-settings 全部 81 项测试通过。
  • rustfmt、差异检查和基于 main 的架构检查通过。

未执行真实 Windows 登录自启动或完整双进程 GUI 交接验收;远端当前提交 CI 待完成。#479 保持打开,避免把目录选择层验证当成所有入口的现场确认。

@github-actions github-actions Bot added the size/M 101–500 行 label Oct 5, 2026
@Kuddev

Kuddev commented Oct 5, 2026

Copy link
Copy Markdown
Owner Author

CI 的平台条件计数发现新增的两处 Windows 限定超过现有预算。已在 e52f0d7 移除纯标准库目录选择函数与测试的不必要 Windows 限定;未修改预算、检查器或 CI。函数/测试主体及 Windows 上执行的逻辑不变,Linux 现在也执行这 3 项测试。平台条件检查回到 468/468,架构和差异检查通过。等待新 head 的全部必需检查与所选原生测试。

@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.

维护者复核当前提交 e52f0d7:确认二次启动目录顺序为显式 CLI 目录、有效启动目录设置、进程 cwd。现有冷启动/UI 入口共用目录有效性检查;保留失效显式路径兼容、上下文菜单修正、Shell 选择、窗口策略、权限隔离与原交接协议。daemon/-e 提前返回,避免新增设置读取。仅 3 个源码文件,无依赖、线程、轮询、推广内容、配置项或门禁放宽。已核对 3 项目录回归、3 项既有兼容测试、81 项设置测试,当前 head 的五平台实际原生测试与 10 项必需检查均成功,main 为该 head 的祖先,无未解决评审线程。此为维护者本人复核,不是独立审批。完整登录/双进程 GUI 现场验收仍单独记录。

@Kuddev
Kuddev merged commit 843ae3d into main Oct 5, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/M 101–500 行

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant