fix: handle windows session workdir separators - #486
Conversation
|
commit: |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a9d53c31d7
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
a9d53c3 to
9427739
Compare
|
刚完成了修改,现在按照session的地址存入函数来比对,应该没啥问题了 |
|
代码没问题,但应该补一个单元测试 @qkunio |
|
@liruifengv 我补了两个单元测试。 这个pr修改的思路是:恢复 session 时先用 pathe.resolve 统一当前目录和 session 记录目录的路径格式,避免 Windows 下 \ 与 / 不一致被误判为不同目录。 所以模拟了Windows下查(C:\Users\kimi\project)和存(C:/Users/kimi/project)的一致性判断,最终判断成功。 测试已通过,麻烦帮忙审阅一下,万分感谢🥹🥹有任何需要修改的,我马上改 |
|
@qkunio 忘了加 changeset 了,可以再提一个 PR 来补一下,运行 gen-changesets skill |
|
@liruifengv 实在不好意思,刚刚没看到消息,下次我尝试做贡献的时候会注意测试和changeset!感谢补全 |
Related Issue
Resolve #477
Problem
Windows 下恢复 session 时,使用
kimi -r误报:原因是Windows下对于路径的判断逻辑有点问题,保存的时候是normalized,统一两种路径分隔符(比如“C:/a/b”),比对的时候取当前目录却用原始路径(比如"C:\a\b"),同一个目录在比较的时候可能同时出现 “/” 和 ”\“ 两种路径分隔符,直接字符串比较会被判断为不同目录。统一这个下划线格式就好了。
What changed
恢复 session 前比较工作目录时,先统一路径分隔符。
影响位置
(两个判断,一个是在kimi -r session_1exxxxxxxxx,一个是在kimi -p "hello"-r session_1exxxxxxxxxxxx):
apps/kimi-code/src/cli/run-prompt.tsapps/kimi-code/src/tui/kimi-tui.ts修复地很简单直接把原始路径里的'\'都换成'/'就可以了,目前测试下来没有问题,当然欢迎更好的方法。
Checklist
gen-changesetsskill, or this PR needs no changeset.gen-docsskill, or this PR needs no doc update.