Skip to content

deps: migrate unmaintained dependencies#17296

Draft
kakkokari-gtyih wants to merge 4 commits intomisskey-dev:developfrom
kakkokari-gtyih:deps-migrate-unmaintained
Draft

deps: migrate unmaintained dependencies#17296
kakkokari-gtyih wants to merge 4 commits intomisskey-dev:developfrom
kakkokari-gtyih:deps-migrate-unmaintained

Conversation

@kakkokari-gtyih
Copy link
Copy Markdown
Contributor

What

https://e18e.dev/docs/replacements/ に記載のある代替のある依存関係のうち、

  • js-yamlyaml
  • body-parsermilliparsec

に変更

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 labels Apr 9, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 9, 2026

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

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.89%. Comparing base (22577e2) to head (b7f97cb).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #17296      +/-   ##
===========================================
- Coverage    63.65%   61.89%   -1.77%     
===========================================
  Files         1162     1162              
  Lines       116471   116471              
  Branches      8450     8164     -286     
===========================================
- Hits         74143    72084    -2059     
- Misses       40124    42203    +2079     
+ Partials      2204     2184      -20     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 9, 2026

Backend memory usage comparison

Before GC

Metric base (MB) head (MB) Diff (MB) Diff (%)
VmRSS 316.68 MB 316.69 MB 0.01 MB 0%
VmHWM 316.68 MB 316.69 MB 0.01 MB 0%
VmSize 23110.60 MB 23109.02 MB -1.57 MB 0%
VmData 1381.38 MB 1379.62 MB -1.75 MB -0.12%

After GC

Metric base (MB) head (MB) Diff (MB) Diff (%)
VmRSS 316.68 MB 316.70 MB 0.01 MB 0%
VmHWM 316.68 MB 316.70 MB 0.01 MB 0%
VmSize 23110.60 MB 23109.02 MB -1.57 MB 0%
VmData 1381.38 MB 1379.62 MB -1.75 MB -0.12%

After Request

Metric base (MB) head (MB) Diff (MB) Diff (%)
VmRSS 317.00 MB 316.92 MB -0.08 MB -0.02%
VmHWM 317.00 MB 316.92 MB -0.08 MB -0.02%
VmSize 23110.60 MB 23109.10 MB -1.49 MB 0%
VmData 1381.38 MB 1379.71 MB -1.67 MB -0.12%

See workflow logs for details

@kakkokari-gtyih kakkokari-gtyih marked this pull request as ready for review April 9, 2026 11:22
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Apr 9, 2026
@syuilo
Copy link
Copy Markdown
Member

syuilo commented Apr 10, 2026

テストが成功しないわね

@kakkokari-gtyih
Copy link
Copy Markdown
Contributor Author

なんかミスってるかもしれない

@kakkokari-gtyih kakkokari-gtyih marked this pull request as draft April 10, 2026 02:02
@kakkokari-gtyih
Copy link
Copy Markdown
Contributor Author

body-parser代替がだめそう(今検証できる環境に居ないので後で)

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 size:M This PR changes 30-99 lines, ignoring generated files.

Projects

Development

Successfully merging this pull request may close these issues.

2 participants