maotoumao
commented
2 months ago 谢谢你的工作~ 简单说下我的想法：
目前其实存在两种类型的插件：
- 一个是加载用户目录下的插件，在应用加载时以文本形式读入并加载；
- 另一种是内置插件，目前有且仅有一种，即用于播放本地文件的插件，代码位置在
src\main\core\plugin-manager\local-plugin.ts，它在代码中以模块的方式引入，并且会打包在应用中；本地插件的hash强制写死为了”本地“，所以不会和计算出的文件hash重复
按照之前的设计，我没有考虑到 【扩展内置插件】 这一行为，因此这部分的代码目前的实现其实不是很合理，在代码中应该有个专门的位置存放内置插件，并且它们的调用应该统一管理；
更具体些，在 src\main\core\plugin-manager\index.ts 路径下针对 localPluginHash 的判定应该修改成更加通用的写法:
/** 调用插件方法 */
function callPluginMethod({
hash,
platform,
method,
args,
}: ICallPluginMethodParams<keyof IPlugin.IPluginInstanceMethods>) {
let plugin: Plugin;
// TODO: 修改为更加通用的判定
if (hash === localPluginHash || platform === localPluginName) {
plugin = localPlugin;
} else if (hash) {
plugin = plugins.find((item) => item.hash === hash);
} else if (platform) {
plugin = plugins.find((item) => item.name === platform);
}
if (!plugin) {
return null;
}
const result = plugin.methods[method]?.apply?.({ plugin }, args);
console.log(plugin, method, args);
return result;
}
如果以上内容改好的话，应该可以实现一个更通用的【内置插件】。
我个人不太倾向于新增个【应用目录】的概念，因为这样子的话貌似和直接在启动时把内置的代码写入到用户目录下区别不大，以及这部分的插件作为软件内置的行为，不太想直接让用户接触到（防止修改什么导致软件崩掉）。
如果按这个设计思路的话，插件的更新规则、版本比较可以引入一个优先级的规则：
- 软件内置部分插件（如本地音乐），优先级最低。
- 内置插件可以更新，但无法卸载；更新后会把最新版本放在【用户目录】的插件
- 【用户目录】的插件支持正常的更新、卸载，规则和其他普通插件相同；其优先级高于【内置插件】。
似乎这样可以解决问题，有其他的想法也欢迎讨论 😄
我不太了解loongarch架构，如果可以的话可以帮忙科普一下，适配过程中如果有些改动也欢迎提PR 😆
最后，还是希望尽量不要内置一些有版权风险的插件，不然可能又要收函了… 😢
ziqi-cn
我目前正在给loongarch架构打包此软件，有用户希望可以一起打包插件，因此希望可以从应用目录下加载插件，能否支持？
目前软件是从用户目录下加载插件的，如果要把插件打包并用包管理器管理，就需要从应用安装目录下读取插件，我目前已经实现了该逻辑 https://github.com/ziqi-cn/MusicFreeDesktop/commit/33944c9a0b514f1b30fc9a4bb0866b26cc5f2cb1 ，可以同时从用户目录和应用目录下读取插件。
但目前还有几个问题，一是目前加载只比较插件的哈希，相同哈希的插件不加载，但如果两个目录下的插件版本不同如何处理比较合适？然后是不是应该应用目录下的插件在插件页面显示该插件由系统管理禁止用户卸载更新？
或许还有更多细节需要考虑，希望能在此充分讨论。