refactor: createLoggerを少しリファクタリング

[Hiroshiba]

内容

タイトルの通り、createLoggerを少しリファクタリングしました。

その他

本当はelectron環境問わずloggerを作れる関数を作りたかったんですが、フロントエンドでelectron-logをimportしようとするとエラーになる問題が解決できずに諦めました。。。
なにか解決策ないのかな。。

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

[voicevox-preview-pages]

🚀 プレビュー用ページを作成しました 🚀

• 📝 エディタ
• 📖 Storybook

更新時点でのコミットハッシュ：35f4cbf

[sevenc-nanashi]

フロントエンドでelectron-logをimportしようとするとエラーになる問題

require()で動的に読み込めば良さそう？
とは言っても一時凌ぎですが...（Electron側をESModuleにしようとした時に詰む）

[sevenc-nanashi]

ESModuleにしても問題ないようにするなら、

https://github.com/KeJunMao/unplugin-preprocessor-directives

こういうのでビルド段階で処理するしかなさそう？

[Hiroshiba]

助言助かります！！
やっぱりESModuleでなんか頑張る方法は簡単なのは無さそうなんですね〜
多少複雑になっても良いから、いつでもどこからでも作れるloggerあると便利そうではある･･･。

[sevenc-nanashi]

初回だけ遅れるのを許容すればdynamic importでいい感じにできると思います。

[Hiroshiba]

たしかに！！
と思ったけど、asyncになっちゃうからトップレベルawaitできずちょっと不便かも。。。。。

[sevenc-nanashi]

asyncになっちゃうからトップレベルawaitできずちょっと不便かも。。。。。

let electronLogOrNull;

const logInternal = (args) => {
  electronLogOrNull.log(args);
}

export const log = (args) => {
  if (electronLogOrNull) {
    logInternal(args);
  } else {
    void import("electron-log").then((l) => {
      electronLogOrNull = l;
      logInternal(args);
    });
  }
}

こういうイメージです

[Hiroshiba]

なるほどです、なんかできそうな気がしてきますね！！
ログが前後しうるのはちょい危ないかも？

とりあえずこのPRマージしちゃえると良さそうかも。
その後気が向いたら実装挑戦してみます！

[sevenc-nanashi]

ログが前後しうる

electron内では前後しないはず？electronLogOrNullに値が入ってからawait入れなければ他のスレッドに移らないはずなので

[sevenc-nanashi]

良さそう。