Skip to content

Conversation

@WormW
Copy link
Contributor

@WormW WormW commented Sep 29, 2025

参照ZeroOmega的同步配置功能
结合ClaudeCode完成的同步到gist和下载配置功能
初衷是方便我这种多套配置多台电脑的用户使用

@WormW
Copy link
Contributor Author

WormW commented Sep 29, 2025

解决了和main分支之间的冲突,精简了部分代码

@farion1231
Copy link
Owner

farion1231 commented Sep 30, 2025

你好 WormW,非常感谢你为 CC Switch 提交这个 PR,非常非常感激你付出的时间和努力!
我仔细看了一下,这是一个非常实用的功能,不过,暂时可能无法合并这个 PR,主要原因是这个功能目前还在早期讨论中 #48 https://x.com/Jason_Young1231/status/1969972925005738356 可以看到相当多的用户非常在意安全性,因此希望保持 CC Switch 完全的本地运行,不要有任何网络相关功能,目前还在讨论一个合适的方案。
还有一点是这个 PR 的代码量非常大(接近 3000 行),在开源协作中,为了保证代码质量和审查效率,一般推荐将一个大的功能拆分成一系列小的、逻辑独立的、可逐步审查的 PR。一个理想的 PR 通常只专注做一件事,代码改动最好控制在几百行以内。这样不仅便于快速理解和审查(3000行的 PR 非常难以 review),也能更快地将你的贡献合并进来。
非常希望你能加入到 #48 的讨论中来,分享你对于这个功能的想法和见解,你的观点对至关重要!如果将来讨论决定做云同步,希望能邀请你来做这个功能,再次感谢你的贡献!

@WormW
Copy link
Contributor Author

WormW commented Sep 30, 2025

你好 WormW,非常感谢你为 CC Switch 提交这个 PR,非常非常感激你付出的时间和努力! 我仔细看了一下,这是一个非常实用的功能,不过,暂时可能无法合并这个 PR,主要原因是这个功能目前还在早期讨论中 #48 x.com/Jason_Young1231/status/1969972925005738356 可以看到相当多的用户非常在意安全性,因此希望保持 CC Switch 完全的本地运行,不要有任何网络相关功能,目前还在讨论一个合适的方案。 还有一点是这个 PR 的代码量非常大(接近 3000 行),在开源协作中,为了保证代码质量和审查效率,一般推荐将一个大的功能拆分成一系列小的、逻辑独立的、可逐步审查的 PR。一个理想的 PR 通常只专注做一件事,代码改动最好控制在几百行以内。这样不仅便于快速理解和审查(3000行的 PR 非常难以 review),也能更快地将你的贡献合并进来。 非常希望你能加入到 #48 的讨论中来,分享你对于这个功能的想法和见解,你的观点对至关重要!如果将来讨论决定做云同步,希望能邀请你来做这个功能,再次感谢你的贡献!

我对代码进行了基于现有风格的精简,确实,我觉得可以没有云同步功能,有导入导出功能给用户提供选择的自由也是个不错的方向

@WormW
Copy link
Contributor Author

WormW commented Sep 30, 2025

我感觉云同步和本地导入导出逻辑类似,就在一个分支上完成了这两个功能。

@farion1231
Copy link
Owner

本地导入导出非常棒!可以单独提交一个 PR 吗?因为云储存功能和相关实现方式还在讨论中

@ZyphrZero
Copy link
Contributor

ZyphrZero commented Oct 10, 2025

我认为云同步是一个可选项,不至于导致安全问题呀,如果怕泄露可以对数据做一些加密,客户端解密完全是可行的方案
例如像Cherry Studio,不需要该功能的用户完全可以选择不开启

@ZyphrZero
Copy link
Contributor

ZyphrZero commented Oct 10, 2025

本地导入导出非常棒!可以单独提交一个 PR 吗?因为云储存功能和相关实现方式还在讨论中

还有一个办法,就是可以让用户自定义配置文件的位置,这样用户就能直接使用OneDrive或者自己NAS云盘中的config配置了,这是最小的改动

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.

3 participants