koaやめる

[syuilo]

Summary

TypeScriptネイティブではないので、型定義上の問題が起きやすく不便

[syuilo]

良い代替のフレームワークなさそう

[tamaina]

@mei23 がまえにkoaやめる話をしてた気がする

[mei23]

前Fastify使おうとしていた
mei23#2285

[rinsuki]

これとかどうだろう https://github.com/talentlessguy/tinyhttp

[syuilo]

tinyhttpにしてみるか

[syuilo]

やたら「express互換だよ！express replacementだよ！」ってアピールされているのがちょっと気になる
別にexpressにこだわり無いから…

[syuilo]

tinyhttpこの間使ったけどなんかバグ踏んで散々な目にあった
tinyhttp/tinyhttp#279
@rinsuki

Fastifyバグっぽいのはあったことないけど、TypeScriptネイティブじゃないからTypeScriptでどう書くの～って悩む所はちょこちょこあったのだわ。
@mei23@misskey.m544.net

[mei23]

tinyhttp/tinyhttp#279

そういえばkoa-routerも同ルートで変なとこあった気がした
#6533

[sousuke0422]

tinyhttpはv2で完全なESMに移行したらしい
https://github.com/tinyhttp/tinyhttp/releases/tag/%40tinyhttp%2Fapp%402.0.0

[syuilo]

なんでHTTPフレームワークでデータベースの話が出てくるんだろう
fastify/fastify#3005

[syuilo]

fastifyの型が完全じゃないことがあるというのは明記されてた

As the team works on improving framework and type definition synergy, sometimes parts of the API will not be typed or may be typed incorrectly

[syuilo]

でもfastifyの方がメジャーではあるんだよな

[mei23]

tinyhttpのサイトって重かったり落ちてたりするからなんだか

[mei23]

express/koa/tinyhttpあたりのexpress系にあるnext()を呼ぶというのがわかりづらくて嫌いなので、その概念がないFastifyの方が好き。

[syuilo]

Fastifyでやってみるか

[syuilo]

型で困らないことを祈る

[rinsuki]

fastify なんかバリデーションとかまで入ってて壮大で、そうじゃないんだよな…感あった (いまさら)

[syuilo]

それは思った
ちょっと機能が多すぎな感じはある

[syuilo]

バックエンド全体の話になるけどNest.jsとか導入するのはどうだろうと思ってた
んだけど、試した感じ

• 事実上express使うことを余儀なくされる
• Nest.jsがそこら辺(express)を抽象化してくれる訳でもないため、express由来の型エラーに悩まされることも多かった
• REST APIを想定していて、Misskeyには合わなそう
• エラーが握りつぶされることがあり、デバッグに手間取った
• ドキュメントの情報量が多くて学習コストがかなり高い
• これはNest.jsというよりデザインパターンの話だけど、「ノートをお気に入り登録するのはNoteServiceに書くのか？FavoriteServiceに書くのか？」みたいな悩みが発生しがち

だから微妙な感じした
使いこなせれば今の状態より見通し良くなるかもしれないけど…

[syuilo]

fastifyに移行するのも結構大変かも

[mei23]

そう？

[syuilo]

コード量というよりは対応する機能を探すのが大変そうと思った

[mei23]

参考
https://github.com/mei23/misskey/pull/2285/files

fastifyStatic => koa_send みたいなの
pointOfView: view(pug) あたり
基本headerは先にsetする
next()で流れ落ちみたいな概念はないので、/users/:id みたいな APとWebでAccpetで出し分ける場所は1つのroute内で分岐になる？

[syuilo]

app.proxy が https://www.fastify.io/docs/latest/Server/#trustproxy か

[syuilo]

あと必ずルートで宣言しなければならないというのはFastifyの掲げるカプセル化の思想と逆行するわね(プラグイン内で定義で来てほしい)

👀 fastify/fastify#3615

ちょっと違う話っぽい(プラグイン全体に制約を「適用」したいという話で、プラグイン内に制約を「定義」したい話ではなさそう)

[acid-chicken]

fastify.addConstraintStrategy でいけそう

[syuilo]

fastify.addConstraintStrategy でいけそう

ちょうど見てた
thx

[syuilo]

でもやっぱり

Property 'validate' is missing in type '{ name: string; ...

エラーが出る

[syuilo]

validate なんてプロパティドキュメントのどこにもないんだけど

[syuilo]

げっ、しかも deriveConstraint の req って IncomingMessage じゃん
そうすると@fastify/acceptsとかが使えない...

[syuilo]

tasukete

[acid-chicken]

validate なんてプロパティドキュメントのどこにもないんだけど

これはなんか型の書き間違えっぽそう

げっ、しかも deriveConstraint の req って IncomingMessage じゃん そうすると@fastify/acceptsとかが使えない...

まあ @fastify/accepts は accepts のラッパっぽいし直接呼べばよさそう

[syuilo]

ドキュメントでは deriveConstraint ってなってたり deriveVersions ってなってたり型定義では validate ってなってたりどれが正解なんだ

[syuilo]

ヌァァァァァァァァァァァァァァァァァァァァァァァァァァァァァァァァァァンンンンオオオオンンオンオンオンオンンンンンンンンン゛ン゛！！！！！！！！！！！！！！！！

[syuilo]

nani

[syuilo]

どんなURLにしてもnot foundになるんだけど

[syuilo]

tasukete

[syuilo]

あとこれが出る

[syuilo]

preParsingフックなんて使ってないのになぜpreParsing処理を呼ぼうとしてるんだ

[syuilo]

これだけ単純化しても同じエラー出る

const fastify = Fastify({});

fastify.get('/hello', async (request, reply) => {
	return 'world!';
});

fastify.server.listen(this.config.port);

[syuilo]

fastify.server.listen(this.config.port);

を

fastify.listen({ port: this.config.port });

にしたら直った

[syuilo]

なんか Route Prefixing が機能してないような気がする

[syuilo]

🤯

[syuilo]

エラーは出なくなったけどなんかレスポンスが返ってこない

[syuilo]

直った

[syuilo]

なぜか request.body が undefined になる

[syuilo]

tasukete

[syuilo]

なんも分からん

[syuilo]

ヌァァァァァァァァァァァァァァァァァァァァァァァァァァァァァァァァァァンンンンオオオオンンオンオンオンオンンンンンンンンン゛ン゛！！！！！！！！！！！！！！！！

[syuilo]

なぜか request.body が undefined になる

解決した方に報奨金を進呈

[syuilo]

解決した

[tamaina]

結局何が原因だったの？