spx-gui: upload files when adding resource

[aofei]

• Added loading message type for global loading-feedback.
• Integrated file upload process into PreprocessModal's confirmation for files requiring preprocessing.
• Implemented loading message display for file uploads without preprocessing.
• Ensured resource addition only completes after successful file upload.
• Fixed filename change issue in PreprocessModal caused by RemoveBackground application.

Fixes #638

[nighca]

这里需要考虑离线的情况

[aofei]

考虑离线情况指的是什么？意思是离线时跳过这里的上传，并继续下面的步骤将资源添加到 project 中，也就是像以前那样将上传延后到 saveToCloud？

[nighca]

对的，离线情况跳过上传（saveFiles）就好

[aofei]

done https://github.com/goplus/builder/compare/7085514aac0d5239ad4dd626e18c3fa7b672bf90..32ddb87289ecdea7ad231c66a0ba5645681d540a

[nighca]

Nit: 这里跟 Costume（以及 Sound、Backdrop） export 的耦合得有点深，另外看字面逻辑容易误以为像 config 这样的信息（比如 Sound 的 config export 时是包含在 files 里的）也可能被上传，而这里的逻辑理应是针对图片、音频这样的“资源文件”的

我暂时也没想到太好的办法。把获得文件 & saveFiles 的逻辑也做到 model 上再由这里调用可能会稍好点，比如：

class Costume {
  async uploadFiles() {
    await saveFiles({ img: this.img })
  }
}

[aofei]

另外看字面逻辑容易误以为像 config 这样的信息（比如 Sound 的 config export 时是包含在 files 里的）也可能被上传，而这里的逻辑理应是针对图片、音频这样的“资源文件”的

我理解你的意思。不过如果认为它就是要去上传所有文件（包括 config），也不是不可以。换句话说，不考虑即将被上传的文件是 inlineable 的还是会被上传到 kodo 的 binary file。

其实现在就是为了把 saveToCloud 里涉及到的 saveFiles 提前到具体文件被添加时而已，这样 saveToCloud 就可以不再跟任何第三方服务打交道了（比如 kodo），只需跟 builder api 通信即可。

给 Costume 这些类引入 uploadFiles 方法的话，会不会有些奇怪呢？目前这种它们只负责提供（export）文件，由上面决定是否上传、上传哪些、以及上传到哪，感觉挺合理的？

[nighca]

给 Costume 这些类引入 uploadFiles 方法的话，会不会有些奇怪呢？

是的，这个会越过 Costume 等 model 目前的职责边界（如果对标 Project 的话这个不是什么大问题）

目前确实这俩做法没有哪个明显更好，可以先按你目前的实现来，我们之后有更多相关逻辑了再看

[nighca]

这里建议暴露个偏上层的 API 来满足这次的需求，比如

{
  withLoading<T>(promise: Promise<T>, content: string): Promise<T> {
    // 这里可以把 loading、destroy 的事儿干了
  }
}

一方面现有的 case 用起来会简单一点，另外一方面我们可以先不去暴露 duration、destroy 这样的基础能力；以 duration 为例，之前不暴露它是有意的：不同的场景 message 展示时长应当是一致的，因此不去放开由外部指定 duration 的能力

P.S. 再长远一点来看我们可能会通过类似 utils/exception.ts 的方式来暴露一个更上层的（可以把像 i18n 这样的项目逻辑做进来的）工具帮助简化 loading message 的使用；不过那个可以后边 case 多一点再说

[aofei]

好的，我来改下。

P.S. 再长远一点来看我们可能会通过类似 utils/exception.ts 的方式来暴露一个更上层的（可以把像 i18n 这样的项目逻辑做进来的）工具帮助简化 loading message 的使用；不过那个可以后边 case 多一点再说

比如给 useMessageHandle 加个 loadingMessage？一开始我是这么做的。

[aofei]

done https://github.com/goplus/builder/compare/aff2fc7d8ed87f1878afb42ce318322a58d5202f..7085514aac0d5239ad4dd626e18c3fa7b672bf90

[nighca]

比如给 useMessageHandle 加个 loadingMessage？一开始我是这么做的。

对的，类似；不过那样 useMessageHandle 干的事情太多了，就不一定好用 🤣

[qiniu-ci]

The PR environment is ready, please check the PR environment

[Attention]: This environment will be automatically cleaned up after a certain period of time., please make sure to test it in time. If you have any questions, please contact the builder team.