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

(dev-mode) devモードの改善 #12639

Merged
merged 5 commits into from
Dec 14, 2023

Conversation

samunohito
Copy link
Member

@samunohito samunohito commented Dec 13, 2023

What

#12632 の解決を図ります。

◎変更点

  • backend/src配下が変更された場合、再ビルド→プロセスの再起動が行われるようにした
  • misskey-js/src配下が変更された場合、再ビルドが行われるようにした
  • sw/src配下またはルートのpackage.jsonが変更された場合、再ビルドが行われるようにした
  • scripts/build-pre.jsのwatch対応により、ルートのpackage.jsonが変更された場合にbuilt/meta.jsonが再生成されるようにした
  • packages/backend/src/server/web/boot.jsで行われていた初期化処理のうち、以下に相当するものをdev-mode用の起動ファイルに実装
    • langがない時のフォールバック
    • localeがない時の読み込み処理(dev-mode時は常に行うようにした)
    • テーマの設定処理など

Why

fix #12632

Additional info (optional)

キャッシュやローカルストレージなどをすべて削除ののち、ルートのpnpm devを起動し以下を確認

  • フロント画面表示時に日本語になっていること
  • ログインに成功し、タイムラインに書き込めること
  • backend/src配下を変更時、再ビルド&再起動されること。再起動後にタイムラインへ書き込みできること
  • frontend/src配下を変更時、HMR(またはリロード)されること
  • misskey-js/src配下を変更時、再ビルドされること
  • sw/src配下を変更時、再ビルドされること(built配下を確認)
  • ルートのpackage.jsonに記載されているversionを変更時、以下が発生すること
    • sw.jsの再ビルド(built配下を確認)および埋め込まれているバージョンが更新されること
    • フロントのストレージ内のバージョンが変わること

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 packages/frontend Client side specific issue/PR packages/backend Server side specific issue/PR packages/misskey-js packages/sw labels Dec 13, 2023
Copy link

codecov bot commented Dec 13, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (a2a6a94) 79.66% compared to head (7f77da7) 79.67%.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop   #12639   +/-   ##
========================================
  Coverage    79.66%   79.67%           
========================================
  Files          958      959    +1     
  Lines       108402   108466   +64     
  Branches      8345     8348    +3     
========================================
+ Hits         86362    86420   +58     
- Misses       22040    22046    +6     

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

Copy link
Contributor

github-actions bot commented Dec 13, 2023

このPRによるapi.jsonの差分

差分はこちら

Get diff files from Workflow Page

@samunohito samunohito changed the title Fix/backend dev mode (dev-mode) devモードの改善 Dec 13, 2023
@samunohito samunohito marked this pull request as ready for review December 13, 2023 04:18
@syuilo
Copy link
Member

syuilo commented Dec 13, 2023

やっぱりローカリゼーションが読み込めてないのかこういうエラーが出ますね
image

@samunohito
Copy link
Member Author

sw.jsの時とはエラーが変わっていますね。navbar.tsなので、文言データの初期化に失敗していそうですが…
dev_boot.tsで文言データ(/locales/*.yml)を直接読んでストレージに入れるようにしているので、そこは特段問題ない認識でした。

当方でもpnpm clean-allで全消し後、pnpm install -> pnpm devで起動し、ブラウザのキャッシュをローカルストレージ含めすべてリフレッシュした状態で5173にアクセスして確認していますが、再現できていません…

@syuilo
Copy link
Member

syuilo commented Dec 13, 2023

シークレットモードでアクセスしても再現せずですか?

@samunohito
Copy link
Member Author

Chrome(シークレットモード含)、Firefox、Edge、Vivaldi、Android版Firefox、Android版Chromeでそれぞれ試しましたが、いずれも再現せずです…謎すぎる

@syuilo
Copy link
Member

syuilo commented Dec 13, 2023

@syuilo syuilo merged commit 386fced into misskey-dev:develop Dec 14, 2023
18 checks passed
@syuilo
Copy link
Member

syuilo commented Dec 14, 2023

🙏🏻 🙏🏻

camilla-ett pushed a commit to kaseiski/misskey that referenced this pull request Jan 2, 2024
* fix dev-mode

* fix dev-mode

* fix dev-mode

* fix dev-mode

---------

Co-authored-by: syuilo <Syuilotan@yahoo.co.jp>
@samunohito samunohito deleted the fix/backend-dev-mode branch January 11, 2024 00:10
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 packages/misskey-js packages/sw
Projects
None yet
Development

Successfully merging this pull request may close these issues.

pnpm devからbuildの処理が消えている?
2 participants