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

revert(dev): フロントエンド・バックエンドを分離する開発モードを廃止 #15284

Merged
merged 15 commits into from
Jan 25, 2025

Conversation

kakkokari-gtyih
Copy link
Contributor

@kakkokari-gtyih kakkokari-gtyih commented Jan 15, 2025

This reverts commit b0039f0.

What

書いてあるとおり
(とりあえず動きはしたけど何かおかしくなっているかもしれないのでよく確認してからマージすること)

Why

#15255 によりHMRが効かない問題が解消されたため

Fix #15186
Fix #12641

(その他、開発モード導入に伴って提出されたissueがすべて解決される)

Additional info (optional)

  • 分離した開発モードを使用している間にアップロードしたドライブファイルなどが読み込めなくなる可能性がある
  • アクセスするポート番号が変わるためconfigファイルを変更する必要がある場合がある

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Jan 15, 2025
Copy link

codecov bot commented Jan 15, 2025

Codecov Report

Attention: Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.

Project coverage is 38.78%. Comparing base (0a41e36) to head (9e5407c).
Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
...ages/backend/src/server/web/ClientServerService.ts 0.00% 5 Missing ⚠️
packages/frontend/src/pages/welcome.entrance.a.vue 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #15284      +/-   ##
===========================================
+ Coverage    38.77%   38.78%   +0.01%     
===========================================
  Files         1601     1600       -1     
  Lines       203902   203816      -86     
  Branches      4011     4007       -4     
===========================================
- Hits         79065    79058       -7     
+ Misses      124201   124155      -46     
+ Partials       636      603      -33     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@syuilo
Copy link
Member

syuilo commented Jan 19, 2025

フロントにアクセスするとなんか勝手に無限にリロードが行われるわね

@anatawa12
Copy link
Member

devcontainer向けの設定のforwardPortsに追加する必要があるかも

@kakkokari-gtyih
Copy link
Contributor Author

フロントにアクセスするとなんか勝手に無限にリロードが行われるわね

WSLでは起こらなかった(たしか開発環境Windowsでしたよね)

@syuilo
Copy link
Member

syuilo commented Jan 19, 2025

おま環なのか

@syuilo
Copy link
Member

syuilo commented Jan 20, 2025

なんか5173に接続しに行こうとしてる

@syuilo
Copy link
Member

syuilo commented Jan 20, 2025

TASUKETE

@kakkokari-gtyih
Copy link
Contributor Author

kakkokari-gtyih commented Jan 20, 2025

なんか5173に接続しに行こうとしてる

Viteのアセットは3000を経由するけどViteサーバー自体は5173/5174で生きているのでViteのHMR用WebSocketがそっちにつながるのは正常なはず
(もしくは、.config/default.ymlのhostを:5173から:3000に戻してないとか)

@syuilo
Copy link
Member

syuilo commented Jan 20, 2025

configは3000になってるわね

@syuilo
Copy link
Member

syuilo commented Jan 20, 2025

WebSocket connection to 'ws://127.0.0.1:5173/vite/' failed
が出てる

@anatawa12
Copy link
Member

devcontainer向けの設定のforwardPortsに追加する必要があるかも

devcontainer使ってるならこれではないでしょうか

@syuilo
Copy link
Member

syuilo commented Jan 20, 2025

devcontainerとやらは使ってないと思う

@anatawa12
Copy link
Member

ではどんな環境で実行してる感じですか?特に misskey backend がどんな環境にいるかが知りたいです

@anatawa12
Copy link
Member

anatawa12 commented Jan 20, 2025

あと念の為ログの上ではどのポートにvite起動してるか確認してください。strictPortがないので場合によってはviteのポートが別ポートになってるかもしれないです。

@syuilo
Copy link
Member

syuilo commented Jan 20, 2025

backendは単にpnpm devしてるだけですね
viteはこういうログになっています
image

@anatawa12
Copy link
Member

backendは単にpnpm devしてるだけですね

どのような環境(ターミナル)からですか?

viteのログは問題なさそう。

@syuilo
Copy link
Member

syuilo commented Jan 20, 2025

vscodeの下に出てくるところからpnpm devをやってます
image

@anatawa12
Copy link
Member

なるほどです。 PowerShell っぽいので Windows ネイティブですね。手元の環境で試してみます。

@anatawa12
Copy link
Member

試してみましたが、無限リロード再現しませんでした。

@syuilo
Copy link
Member

syuilo commented Jan 20, 2025

むむむ

@anatawa12
Copy link
Member

普段の開発環境じゃないので起動したあとほぼ何もやってないので、なにかトリガがある可能性はあるかもしれないです。

トップページをログインしたデフォルトUIで見るだけで再現するはずですか?(手元ではこれくらいしか試していないので別の操作であれば合わせたいです)

@syuilo
Copy link
Member

syuilo commented Jan 20, 2025

net::ERR_CONNECTION_REFUSED 以外の情報はなさそう

@anatawa12
Copy link
Member

ERR_CONNECTION_REFUSED、ポートが空いてないとか繋がらないのたぐいなのでファイヤーウォール系を私は疑いますね

@kakkokari-gtyih
Copy link
Contributor Author

これまだ動いてない感じかしら

@syuilo
Copy link
Member

syuilo commented Jan 25, 2025

そうね

@kakkokari-gtyih
Copy link
Contributor Author

kakkokari-gtyih commented Jan 25, 2025

localhostと127.0.0.1って別のアドレスとして取り扱われたっけ
別物ならviteの起動アドレスを127.0.0.1:5173/5174にあわせるなりするといけるかも

@kakkokari-gtyih
Copy link
Contributor Author

kakkokari-gtyih commented Jan 25, 2025

@syuilo 修正したのでもう一度試していただけるかしら

@tai-cha
Copy link
Contributor

tai-cha commented Jan 25, 2025

localhostと127.0.0.1って別のアドレスとして取り扱われたっけ

yes

@syuilo
Copy link
Member

syuilo commented Jan 25, 2025

image
うーむ

@kakkokari-gtyih
Copy link
Contributor Author

うーむ

試したけど再現しなかった(WSL2)

@kakkokari-gtyih
Copy link
Contributor Author

あーわかった

@syuilo
Copy link
Member

syuilo commented Jan 25, 2025

あーなんか直った

@github-actions github-actions bot added the packages/backend Server side specific issue/PR label Jan 25, 2025
@kakkokari-gtyih
Copy link
Contributor Author

多分 9e5407c でちゃんとなおるはず

Copy link
Contributor

このPRによるapi.jsonの差分
差分はありません。
Get diff files from Workflow Page

@syuilo syuilo merged commit 35104d8 into misskey-dev:develop Jan 25, 2025
32 checks passed
@syuilo
Copy link
Member

syuilo commented Jan 25, 2025

🙏🏻

@kakkokari-gtyih kakkokari-gtyih deleted the revert-separated-dev-mode branch January 25, 2025 11:58
@syuilo
Copy link
Member

syuilo commented Feb 4, 2025

Ctrl+C しても一見終了するけど、nodemonが生きててファイルを保存したりするとMisskeyプロセスが再始動する問題があるわね

@kakkokari-gtyih
Copy link
Contributor Author

kakkokari-gtyih commented Feb 4, 2025

Ctrl+C しても一見終了するけど、nodemonが生きててファイルを保存したりするとMisskeyプロセスが再始動する問題があるわね

再現したことがないのと、おそらくこれとは関係がない(フロントエンドでnodemonは使ってないので)

@syuilo
Copy link
Member

syuilo commented Feb 4, 2025

再始動するのはバックエンドだわね

@kakkokari-gtyih
Copy link
Contributor Author

再始動するのはバックエンドだわね

なおさらこのPRと関係無さそう

@syuilo
Copy link
Member

syuilo commented Feb 4, 2025

ほむん

@syuilo
Copy link
Member

syuilo commented Feb 4, 2025

これの影響だったかもしれない
#15311

@kakkokari-gtyih kakkokari-gtyih mentioned this pull request Feb 4, 2025
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/backend Server side specific issue/PR packages/frontend Client side specific issue/PR
Projects
5 participants