Skip to content

fix: remove kaku.lua on reset#116

Closed
Monster12138 wants to merge 1 commit intotw93:mainfrom
Monster12138:always-first-run-fix
Closed

fix: remove kaku.lua on reset#116
Monster12138 wants to merge 1 commit intotw93:mainfrom
Monster12138:always-first-run-fix

Conversation

@Monster12138
Copy link
Copy Markdown

@Monster12138 Monster12138 commented Feb 25, 2026

#115
执行 kaku reset 时删除用户配置文件 kaku.lua,确保在版本更新后可以正确恢复到当前版本的默认配置。

@tw93
Copy link
Copy Markdown
Owner

tw93 commented Feb 26, 2026

@Monster12138 感谢你提交 PR 和排查。

我确认了 #115 的根因是升级兼容。旧版用户配置仍读取 .kaku_config_version,新版流程只写 state.json,所以会反复触发引导。

这个 PR 只在执行 kaku reset 时删除 kaku.lua,不能覆盖正常升级路径。用户不执行 reset 仍会复现。
另外当前实现对 kaku.lua 调用了目录删除逻辑,kaku.lua 是文件,这里会失败。

我已经在 main 合入根因修复,提交是 96cdb9d。修复内容是同时写入 state.json.kaku_config_version,并补充 reset 对 legacy 标记清理。
所以这个 PR 我先不合并了。谢谢你的贡献,欢迎继续提改进。

@tw93 tw93 closed this Feb 26, 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