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

fix: たまに./src/_boot_.tsがトランスパイルされない問題を修正 #147

Merged
merged 3 commits into from
Aug 15, 2023

Conversation

u1-liquid
Copy link
Member

@u1-liquid u1-liquid commented Aug 15, 2023

What

@rollup/plugin-typescript を入れる

Why

何かの拍子でviteのrollupに設定してるinput { app: './src/_boot_.ts' }がトランスパイルされてないままboot.jsに入ってしまい、APP_IMPORTエラーでフロントエンドが全く動かないことになる現象がある
(実際DevToolsでネットワークタブをみると/vite/_boot_tsをリクエストして404をもらってる感じ)

viteとrollupのビルドパイプライン上typescriptを処理してるのはvue-tscだけみたいだったので、
vueじゃないtypescriptは誰が処理してくれてるのかというと・・・vue-tscぽいので
viteがvue-tscを呼び出してくれない状態(なんでかはわからない)だと.tsがトランスパイルされないみたい
普通にtypescriptモジュールをrollupのビルドに入れて上げれば直るでしょと思って入れたらうまくいったって感じです

Additional info (optional)

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

Copy link
Collaborator

@riku6460 riku6460 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pnpm のロックファイルがコミットできてなさそう

riku6460
riku6460 previously approved these changes Aug 15, 2023
@riku6460
Copy link
Collaborator

なぜこれで直るのかを PR に書いておいてほしい(これだけ見ても謎なので)

@u1-liquid
Copy link
Member Author

なぜこれで直るのかを PR に書いておいてほしい(これだけ見ても謎なので)

書いた

@riku6460 riku6460 merged commit 8d5b1a4 into MisskeyIO:io Aug 15, 2023
14 checks passed
@u1-liquid u1-liquid deleted the fix-boot-ts branch August 15, 2023 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants