Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

src/type/ipc.tsのファイル名を変える? #2252

Open
3 tasks
Hiroshiba opened this issue Aug 26, 2024 · 1 comment
Open
3 tasks

src/type/ipc.tsのファイル名を変える? #2252

Hiroshiba opened this issue Aug 26, 2024 · 1 comment

Comments

@Hiroshiba
Copy link
Member

Hiroshiba commented Aug 26, 2024

内容

src/type/ipc.tsというファイル名は結構実態に合っていないかもしれません。

(electronの) IPC 通信に関係した物が入っている・・・というより、フロント(レンダラープロセス)からバックエンドの処理を実行したい時の関数群の型が入っている感じです。

とりあえずこのファイルの名称が適したものに変わればそれで良さそう。

Pros 良くなる点

分かりやすくなる。

実現方法

名前をどうしようか若干迷ってます。
レンダラープロセス側の名称をまだ決めてないんですよね・・・。
(レンダラープロセスというのはelectronのドメイン用語であって、ブラウザ版とかから見ると別にレンダラープロセスが立ってるわけではない)

VOICEVOXのバージョン

0.20.0

OSの種類/ディストリ/バージョン

  • Windows
  • macOS
  • Linux

その他

ちなみにsrcディレクトリ内を整理したい旨のissueもあります。
そこは別に IPC 通信とかの話をしてなさそうでした。

@Hiroshiba Hiroshiba changed the title src/type/ipc.tsのファイル名を変える src/type/ipc.tsのファイル名を変える? Aug 26, 2024
@Hiroshiba
Copy link
Member Author

とりあえず意見お待ちするフェーズだと思ってます。
なんとなーーーーーーーくですが

  • chromiumやブラウザ内で表示されるものは frontend と呼ぶことになりそう?
    • electronやbrowserをbackendと呼んでるので。
    • 他には editor とかが候補になりそう(vscodeはこれ?)
    • でもウェルカム画面とかも backendじゃない側に配置したく、ウェルカム画面はeditorではなさそうなので、frontendかなぁと
  • backendfrontendの通信用のディレクトリを作る・・・・・・・・・?
    • まだ二者のやり取りをするための関数群を置いておくファイルしかなさそうなので、さすがにそれほどではなさそう?
  • backendfronendのやりとりをするドメイン用語を考えると良さそう
    • ・・・・・・・・・・・・・何にすれば良いんだろう。
    • message・・・・?はさすがに汎用的すぎてダメそう
    • ・・・bridgeとか?
  • ドメイン用語を考えるの大変なので、一旦ファイル名にするのが良さそう
    • これなら話は簡単で、とりあえずsrc/type/frontAndBackBridge.tsとかで良さそう?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants
@Hiroshiba and others