refactor&fix: RateLimiterService

[anatawa12]

What

#13994 (comment)

とりあえずの対応としてはこれでいいと思ってますが、new Promiseのコールバックが長いことが原因の一端になってると思うので、Limiter.prototype.getをpromise化してそれをasyncで呼ぶ形に近い将来したいと思ったのでちょっと準備してます

を実装しました

Why

What参照

Additional info (optional)

diffとしては大きいですが、小さな作業単位にcommitを分けたので、レビューするときはコミット単位にみてもらうとおそらく見やすいと思います。

#13994を先にマージする必要があるため、マージされるまでDraftにしますが Ready for Review です

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]

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

差分はこちら

Get diff files from Workflow Page

[codecov]

Codecov Report

Attention: Patch coverage is 13.33333% with 52 lines in your changes missing coverage. Please review.

Project coverage is 65.29%. Comparing base (9368eb3) to head (0265d91).
Report is 4 commits behind head on develop.

Files	Patch %	Lines
...kages/backend/src/server/api/RateLimiterService.ts	16.32%	41 Missing ⚠️
packages/backend/src/server/api/ApiCallService.ts	0.00%	9 Missing ⚠️
...ackages/backend/src/server/api/SigninApiService.ts	0.00%	2 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #13997      +/-   ##
===========================================
+ Coverage    65.28%   65.29%   +0.01%     
===========================================
  Files         1004     1004              
  Lines       115245   115228      -17     
  Branches      4617     5541     +924     
===========================================
+ Hits         75233    75241       +8     
+ Misses       39985    39960      -25     
  Partials        27       27              

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

[u1-liquid]

ちょっとoutofscopeかもしれないけどrefactorついでにpolicyでrateLimitFactorを調整すると
回数だけではなく時間も影響されて実際の影響が倍になる問題を直したほうがいいかもしれません
(durationとmax両方でfactorをかけ算／割り算するのではなくmaxにだけ割り算した方が良さそう）

[KisaragiEffective]

実際の影響が倍になる

正確には2乗に比例するはずです。

[KisaragiEffective]

mark this pull request as "ready for review": #13994 is merged

[anatawa12]

ちょっとoutofscopeかもしれないけどrefactorついでにpolicyでrateLimitFactorを調整すると 回数だけではなく時間も影響されて実際の影響が倍になる問題を直したほうがいいかもしれません (durationとmax両方でfactorをかけ算／割り算するのではなくmaxにだけ割り算した方が良さそう）

実装しました

[anatawa12]

IDEのeslintがコンフィグのロードに失敗してましたすみません

[KisaragiEffective]

あ、セマンティックコミットじゃない 🙇🏻