ファイル保存全般をアトミックにする

[tsunekazuomija]

ファイル保存全般をアトミックにする

@RikitoNoto さんが #2098 で作成してくださったファイル保存を
src/helpers/fileHelper.tsにヘルパー関数として切り出しました。

それに伴い、

• src/backend/electron/electronConfig.ts の ElectronConfigManager.save
• src/backend/electron/main.tsのWRITE_FILE部分
• src/backend/electron/manager/RuntimeInfoManager.ts のexportFile 関数

のファイル保存処理をヘルパー関数で置き換えました。

それぞれ、

• 設定/ツールバーのカスタマイズから設定を変更し、保存ボタンを押した時
• プロジェクトを変更し、保存した時
• 起動時のruntime-info.json生成時

にwriteFileSafelyのmoveFile呼び出しの前でelectronを停止したときにtmpファイルが存在しており、electronを再起動してファイルが正しく開ける/正しく起動できることを確認しました。

関連 Issue

ref #2103 #2098
close #2103

その他

src/backend/electron/main.tsのWRITE_FILE部分を非同期関数に変えてしまったのですが、こちらで問題ないでしょうか？

[tsunekazuomija]

ビルドに失敗してますね...
解決できそうか調べてみます！

[sabonerune]

fileHelper.tsはレンダラープロセスで使用されています。
そのためnodeAPI(fsやmove-file)をインポートするとバンドルに失敗してビルドが失敗しているのではないかと思います。

[tsunekazuomija]

webアプリ全般に詳しくなく、見当がついていない状況だったので、レビュー大変助かります！

fileHelper.tsはレンダラープロセスで使用されています。
そのためnodeAPI(fsやmove-file)をインポートするとバンドルに失敗してビルドが失敗しているのではないかと思います。

なるほどです。であれば、追加するヘルパー関数はsrc/backend/electron以下に別途ファイルを作るなどして、メインプロセスからのみ使用するファイルに記載する必要がありそうですね。

メインプロセス、レンダラープロセスについて理解していない部分があるので、調べつつ修正してみます！

[Hiroshiba]

あっ！！！　なるほど、すみませんでした！！！！

とりあえず一旦の解決策としては、electron用のディレクトリにヘルパー関数を置くのが良さそうだと思います！！
（と言おうとしたら既にそう提案されてますね！！！　非常に良いと思います！！！）

ちなみに取っ掛かりとしてelectronの仕組みをちょっとだけご説明すると。
electronはchromiumブラウザでGUIを表示するプロセス（レンダラー側）と、あとはそのGUIを起動したり、ブラウザではできないことをする用のメインプロセスに分かれていて、ファイル操作はメインプロセスじゃないとできないようになっている感じです！
なのでレンダラー側からファイル操作の処理をimportしたときに、そんなものがないのでエラーになる感じかなと･･･！！