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

update deps #15311

Merged
merged 29 commits into from
Feb 15, 2025
Merged

update deps #15311

merged 29 commits into from
Feb 15, 2025

Conversation

syuilo
Copy link
Member

@syuilo syuilo commented Jan 20, 2025

What

Why

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

@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 Jan 20, 2025
Copy link

codecov bot commented Jan 20, 2025

Codecov Report

Attention: Patch coverage is 39.00000% with 61 lines in your changes missing coverage. Please review.

Project coverage is 40.34%. Comparing base (1e88aa9) to head (530a804).
Report is 1 commits behind head on develop.

Files with missing lines Patch % Lines
packages/backend/src/core/WebAuthnService.ts 0.00% 9 Missing ⚠️
packages/frontend/src/components/MkTimeline.vue 0.00% 9 Missing ⚠️
packages/frontend/src/scripts/key-event.ts 0.00% 7 Missing ⚠️
packages/backend/src/queue/types.ts 0.00% 5 Missing ⚠️
...es/frontend/src/components/grid/cell-validators.ts 0.00% 4 Missing ⚠️
packages/frontend/src/components/grid/grid.ts 0.00% 4 Missing ⚠️
packages/frontend/src/components/grid/cell.ts 0.00% 3 Missing ⚠️
packages/frontend/src/components/grid/row.ts 0.00% 3 Missing ⚠️
packages/frontend/src/scripts/stream-mock.ts 0.00% 3 Missing ⚠️
packages/frontend/src/components/MkCaptcha.vue 0.00% 2 Missing ⚠️
... and 12 more
Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #15311       +/-   ##
============================================
+ Coverage    18.84%   40.34%   +21.49%     
============================================
  Files          768     1611      +843     
  Lines       111607   210727    +99120     
  Branches      1236     4052     +2816     
============================================
+ Hits         21037    85009    +63972     
- Misses       89979   125108    +35129     
- Partials       591      610       +19     

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

@syuilo
Copy link
Member Author

syuilo commented Jan 20, 2025

@kakkokari-gtyih kakkokari-gtyih added this to the v2025.1.0 milestone Jan 21, 2025
@kakkokari-gtyih
Copy link
Contributor

kakkokari-gtyih commented Jan 21, 2025

misskey-dev-eslint-pluginの新バージョンのリリースが必要(必要な更新はリポジトリ自体には入っているのでリリースするだけでOK)

@kakkokari-gtyih
Copy link
Contributor

misskey-dev-eslint-pluginの新バージョンのリリースが必要(必要な更新はリポジトリ自体には入っているのでリリースするだけでOK)

した

v7でOpenAPIのバリデーションが入るようになった関係でスコープ外での変更が避けられないため一時的に戻した
Copy link
Contributor

github-actions bot commented Jan 24, 2025

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

@kakkokari-gtyih
Copy link
Contributor

kakkokari-gtyih commented Jan 24, 2025

  • @simplewebauthn/server が v10 → v13 に更新されたが v12 まで上げた
    • v13では attestationTypeindirect を指定できなくなるので検討が必要かも
  • nsfwjs が v4.2.0 → v4.2.1 に更新されたが戻した
  • openapi-typescript が v6 → v7 に更新されたが戻した
    • OpenAPIスキーマがおかしい問題は別途対応するため
  • autogenのインデントの処理が中途半端に行われてしまっていた(コメント内部のインデントは直してくれず、インデントが混在するようになった)ので、autogen部分のインデントlintルールを無効化して、せめてインデントの方式自体を統一するように図った
  • frontend-shared/js/i18n.ts のlintでエラー(ルール違反によるエラーではなく解析時のエラー)が出たので、いったんファイルごと無視するようにした
  • happy-domのXMLSerializerが修正されてvalidなXMLを吐くようになったので、root要素につくようになったnsを正規表現で飛ばすようにした

@anatawa12
Copy link
Member

nsfwjs は #15248 で言及し忘れたけど infinitered/nsfwjs#910 を踏んだので意図的に落としてた。どっかにバージョンを latest にしない理由をまとめる仕組みあったほうがいいかもな...

Comment on lines +103 to +104
// See https://github.com/misskey-dev/misskey/pull/15311
'js/i18n.ts',
Copy link
Member

Choose a reason for hiding this comment

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

Upstreamに報告して / 見つけてそっちへのリンクを張りたいとおもったけど eslint の更新してないのが原因だと考えられるから一旦これでも良さそう。
eslint v9 / typescript-eslint v8のissueもないから切ったほうが良さそうですね。

Co-authored-by: anatawa12 <anatawa12@icloud.com>
@samunohito
Copy link
Member

samunohito commented Jan 25, 2025

動くかどうかは分からない

下記理由から、 2025.1.0 ではなく次以降のバージョンに取り込むのが良いと考えますが、どうでしょうか

  • WebAuthnService.ts や MfmService.ts に対して実際に確認が必要そうな変更が入っている
  • 各依存パッケージの更新による副作用を検証する期間が短い
  • 2025.1.0 には重要なbugfixが多くあり、出来る限り他のバグを埋め込むリスクを下げたい(特にこれ)

@kakkokari-gtyih
Copy link
Contributor

dependencyにも結構重要な脆弱性修正が入っている(Criticalあり)のでやったほうがいい気もするんですよね

@samunohito
Copy link
Member

それなら、該当パッケージをリストアップして最低限にとどめた方がリスクが低いように思えますが…
まあ、お任せします

@syuilo
Copy link
Member Author

syuilo commented Feb 4, 2025

このバージョンのnodemonに不具合あるかも(プロセスが終了しない)

@kakkokari-gtyih
Copy link
Contributor

このバージョンのnodemonに不具合あるかも(プロセスが終了しない)

#15284 (comment) ってこのブランチで試してたの?

@syuilo
Copy link
Member Author

syuilo commented Feb 4, 2025

ブランチ切り替えた時にpnpm iし忘れていた可能性がある(ほかのブランチでpnpm iしたら再現しなくなった)

@kakkokari-gtyih
Copy link
Contributor

kakkokari-gtyih commented Feb 5, 2025

このバージョンのnodemonに不具合あるかも(プロセスが終了しない)

いま試したけどちゃんと止まったわね(WSL)

@syuilo
Copy link
Member Author

syuilo commented Feb 6, 2025

二回連続でpnpm devすると発生することが判った(nodemonのバージョンは無関係だった)

@kakkokari-gtyih
Copy link
Contributor

じゃあ一旦これはマージできる状態?

@syuilo
Copy link
Member Author

syuilo commented Feb 6, 2025

かも

@samunohito samunohito merged commit ffd8cf0 into develop Feb 15, 2025
34 of 36 checks passed
@samunohito samunohito deleted the update-deps-20250120 branch February 15, 2025 01:24
Ruruke pushed a commit to Ruruke/rukey that referenced this pull request Feb 17, 2025
* wip

* bump misskey-dev/eslint-plugin

* lint fixes (backend)

* lint fixes (frontend)

* lint fixes (frontend-embed)

* rollback nsfwjs to 4.2.0

ref: infinitered/nsfwjs#904

* rollback openapi-typescript to v6

v7でOpenAPIのバリデーションが入るようになった関係でスコープ外での変更が避けられないため一時的に戻した

* lint fixes (misskey-js)

* temporarily disable errored lint rule (frontend-shared)

* fix lint

* temporarily ignore errored file for lint (frontend-shared)

* rollback simplewebauthn/server to 12.0.0

v13 contains breaking changes that require some decision making

* lint fixes (frontend-shared)

* build misskey-js with types

* fix(backend): migrate simplewebauthn/server to v12

* fix(misskey-js/autogen): ignore indent rules to generate consistent output

* attempt to fix test

changes due to capricorn86/happy-dom#1617 (XMLSerializer now produces valid XML)

* attempt to fix test

changes due to capricorn86/happy-dom#1617 (XMLSerializer now produces valid XML)

* fix test

* fix test

* fix test

* Apply suggestions from code review

Co-authored-by: anatawa12 <anatawa12@icloud.com>

* bump summaly to v5.2.0

* update tabler-icons to v3.30.0-based

---------

Co-authored-by: かっこかり <67428053+kakkokari-gtyih@users.noreply.github.com>
Co-authored-by: anatawa12 <anatawa12@icloud.com>
Ruruke pushed a commit to Ruruke/rukey that referenced this pull request Feb 17, 2025
* wip

* bump misskey-dev/eslint-plugin

* lint fixes (backend)

* lint fixes (frontend)

* lint fixes (frontend-embed)

* rollback nsfwjs to 4.2.0

ref: infinitered/nsfwjs#904

* rollback openapi-typescript to v6

v7でOpenAPIのバリデーションが入るようになった関係でスコープ外での変更が避けられないため一時的に戻した

* lint fixes (misskey-js)

* temporarily disable errored lint rule (frontend-shared)

* fix lint

* temporarily ignore errored file for lint (frontend-shared)

* rollback simplewebauthn/server to 12.0.0

v13 contains breaking changes that require some decision making

* lint fixes (frontend-shared)

* build misskey-js with types

* fix(backend): migrate simplewebauthn/server to v12

* fix(misskey-js/autogen): ignore indent rules to generate consistent output

* attempt to fix test

changes due to capricorn86/happy-dom#1617 (XMLSerializer now produces valid XML)

* attempt to fix test

changes due to capricorn86/happy-dom#1617 (XMLSerializer now produces valid XML)

* fix test

* fix test

* fix test

* Apply suggestions from code review

Co-authored-by: anatawa12 <anatawa12@icloud.com>

* bump summaly to v5.2.0

* update tabler-icons to v3.30.0-based

---------

Co-authored-by: かっこかり <67428053+kakkokari-gtyih@users.noreply.github.com>
Co-authored-by: anatawa12 <anatawa12@icloud.com>
harumaki2000 pushed a commit to harumaki2000/misskey-layp that referenced this pull request Feb 21, 2025
* wip

* bump misskey-dev/eslint-plugin

* lint fixes (backend)

* lint fixes (frontend)

* lint fixes (frontend-embed)

* rollback nsfwjs to 4.2.0

ref: infinitered/nsfwjs#904

* rollback openapi-typescript to v6

v7でOpenAPIのバリデーションが入るようになった関係でスコープ外での変更が避けられないため一時的に戻した

* lint fixes (misskey-js)

* temporarily disable errored lint rule (frontend-shared)

* fix lint

* temporarily ignore errored file for lint (frontend-shared)

* rollback simplewebauthn/server to 12.0.0

v13 contains breaking changes that require some decision making

* lint fixes (frontend-shared)

* build misskey-js with types

* fix(backend): migrate simplewebauthn/server to v12

* fix(misskey-js/autogen): ignore indent rules to generate consistent output

* attempt to fix test

changes due to capricorn86/happy-dom#1617 (XMLSerializer now produces valid XML)

* attempt to fix test

changes due to capricorn86/happy-dom#1617 (XMLSerializer now produces valid XML)

* fix test

* fix test

* fix test

* Apply suggestions from code review

Co-authored-by: anatawa12 <anatawa12@icloud.com>

* bump summaly to v5.2.0

* update tabler-icons to v3.30.0-based

---------

Co-authored-by: かっこかり <67428053+kakkokari-gtyih@users.noreply.github.com>
Co-authored-by: anatawa12 <anatawa12@icloud.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

4 participants