Skip to content

fix(sandbox): 清理旧布局 CLI 身份文件 - #1596

Merged
deepcoldy merged 1 commit into
deepcoldy:masterfrom
kunshanyep-beep:fix/legacy-cli-identity-cleanup
Sep 29, 2026
Merged

deepcoldy merged 1 commit into
deepcoldy:masterfrom
kunshanyep-beep:fix/legacy-cli-identity-cleanup

Conversation

@kunshanyep-beep

Copy link
Copy Markdown
Contributor

问题

#1543 将 trigger-user 身份文件迁移到每个 session 的 .bin/.data/
目录下,但升级前生成的 <session>.<tool>.env 和 <session>.turn
不会再被覆盖或清除,其中的用户令牌可能在 session 关闭后长期残留。

修改

  • clearSessionIdentity 同时删除当前 session/tool 对应的旧布局 env 文件
  • session teardown 时删除旧布局 turn 标记
  • 清理范围限定为当前 session 和已知工具名,不扫描或删除其他文件
  • 添加回归测试,覆盖其他工具、其他 session 和未知文件不受影响

验证

  • npx -y bun@1.4.2 run test -- test/cli-identity.test.ts test/turn-cli-identity.test.ts
    • 104 tests passed
  • npx -y bun@1.4.2 run build
  • git diff --check

Closes #1556

@kunshanyep-beep
kunshanyep-beep force-pushed the fix/legacy-cli-identity-cleanup branch from ceac92f to 76b6e55 Compare September 28, 2026 11:07
@deepcoldy

Copy link
Copy Markdown
Owner

你好,你的 PR #1596 评审群已自动创建,维护者和多个 reviewer 会在群内进行评审讨论。请点击以下飞书群链接申请加入评审群(链接一年有效):

https://applink.larkoffice.com/client/chat/chatter/add_by_link?link_token=f15gfdd9-e621-4e89-a78b-f884adefac68

另外,当前自动拉群名单暂时没能关联上你的飞书账号(自动邀请不可见,所以没能直接拉你入群)。方便的话可以把 GitHub 账号与飞书信息补录到名单文档:https://bytedance.larkoffice.com/wiki/WJ1nwWbtxi89erkNGNbcgkt9nUe ,补录后后续评审会自动把你拉进群。

本条为自动评审流程发出的消息,最终结论以维护者审阅为准,谢谢!

@deepcoldy
deepcoldy merged commit 8ea747b into deepcoldy:master Sep 29, 2026
9 checks passed
@github-actions

Copy link
Copy Markdown

🚀 Released in v3.33.0

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.

fix(sandbox): 清理升级遗留的旧布局 cli-identity 身份文件

2 participants