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

misskey-js: Record<string, never>になっている箇所に対して適切な型定義を与える #14169

Open
KisaragiEffective opened this issue Jul 10, 2024 · 0 comments
Labels
✨Feature This adds/improves/enhances a feature 🔥high priority packages/misskey-js

Comments

@KisaragiEffective
Copy link
Collaborator

Purpose

現在のコードベースではpackages/misskey-js配下において、いくつかのプロパティがRecord<string, never>として型付けされてしまっている部分がある。これはopenapi-typescriptの意図した動作であるが、反面存在するはずのプロパティを与えることができず、as anyなど本来であれば不必要な型アサーションを招く。

Solution

そこで、そのような部分に対しては #14147#14167 のような手法を用いて適切な型定義をあてることを提唱する。こうすることでmisskey-jsがより実際のレスポンスに即してイディオマティックな形で使えるようになる。

@KisaragiEffective KisaragiEffective added 💚Refactor Rewriting code without changing behavior packages/misskey-js labels Jul 10, 2024
@kakkokari-gtyih kakkokari-gtyih added ✨Feature This adds/improves/enhances a feature 🔥high priority and removed 💚Refactor Rewriting code without changing behavior labels Jul 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨Feature This adds/improves/enhances a feature 🔥high priority packages/misskey-js
Projects
Development

No branches or pull requests

2 participants