Skip to content

fix(file-tree): 修复切换仓库后目录树展开状态丢失的问题#336

Closed
lwt-sadais wants to merge 1 commit intoJ3n5en:mainfrom
lwt-sadais:main
Closed

fix(file-tree): 修复切换仓库后目录树展开状态丢失的问题#336
lwt-sadais wants to merge 1 commit intoJ3n5en:mainfrom
lwt-sadais:main

Conversation

@lwt-sadais
Copy link
Contributor

问题描述

当用户在目录树中展开多层文件夹,然后收起父节点或某个祖先节点之后,切换到目录树的别的仓库,再次切换回来时,目录树的展开状态会丢失。

具体表现:

  • 用户收起父节点后,子节点的展开路径残留在 localStorage 中
  • 切换仓库后,这些孤立的路径在恢复时找不到对应的树节点
  • 用户重新打开父节点时,子节点显示"展开"箭头但内容为空

根因分析

折叠逻辑(collectCompactedPaths)仅处理"压缩链"(只有单个子目录的连续路径),当父节点有多个子目录时:

  • 仅删除父节点路径
  • 所有子节点路径残留在 expandedPaths Set 和 localStorage 中

切换仓库回来时(restoreChildren):

  • 尝试恢复孤立的子路径,但其父节点未展开
  • updateTreeWithChain 在树结构中找不到这些路径,静默失败
  • expandedPaths 中残留孤立路径,导致 UI 显示不一致

修复方案

简化折叠逻辑:直接清除目标路径及其所有展开的后代路径。

// 之前:仅沿 compact chain 收集路径
const pathsToCollapse = collectCompactedPaths(treeRef.current, path);

// 之后:清除所有后代路径
for (const p of [...newExpanded]) {
  if (p === path || p.startsWith(`${path}/`)) {
    newExpanded.delete(p);
  }
}

行为对比

场景 修复前 修复后
折叠有多个子目录的父节点 ❌ 子路径残留 ✅ 全部清除
折叠 compact chain ✅ 沿链逐个删除 ✅ 同样全部删除
切换仓库再切回 ❌ 节点箭头但无内容 ✅ 展开状态正确

测试步骤

  1. 打开项目并展开多层文件夹(如 src/renderer/components/files/)
  2. 收起最上层父节点(如 /components)
  3. 切换到另一个仓库
  4. 再次切换回当前仓库
  5. 重新展开该父节点,应能正确看到文件树内容(无空节点)

🤖 Generated with Claude Code

修复折叠逻辑,当用户折叠一个包含多个子目录的父节点时,现在会同时清除该目录及其所有展开的后代路径。这防止了孤立的子路径残留在 localStorage 中,导致用户在切换仓库回来后看到子节点显示为"展开"状态但内容为空的现象。

变更:
- 简化折叠逻辑:移除复杂的 collectCompactedPaths 递归函数
- 改为直接清除目标路径及所有以其开头的后代路径
- 此行为对压缩链(single-child directories)仍然有效,因为它们本质上就是后代关系

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2026

Claude encountered an error —— View job


I'll analyze this and get back to you.

@lwt-sadais lwt-sadais closed this Mar 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants