From 9c964573fa07454789c9bee541cab31768f126d4 Mon Sep 17 00:00:00 2001 From: m-shaka Date: Tue, 17 Sep 2024 10:46:29 +0900 Subject: [PATCH] Tips for rpc perf (#490) * docs(rpc): add a tip: compiling before using app * docs(rpc): add a tip: specifying type arguments manually * docs(rpc): add a tip: splitting your app and client * docs(rpc): add descriptions about compilation process * docs(rpc): suggest to compile the client * format and add a short description for `tsserver` --- docs/guides/rpc.md | 99 ++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 99 insertions(+) diff --git a/docs/guides/rpc.md b/docs/guides/rpc.md index 2b3e6c07..e7a2bb1f 100644 --- a/docs/guides/rpc.md +++ b/docs/guides/rpc.md @@ -395,3 +395,102 @@ export type AppType = typeof routes ``` You can now create a new client using the registered AppType and use it as you would normally. + +## Known issues + +### IDE performance + +When using RPC, the more routes you have, the slower your IDE will become. One of the main reasons for this is that massive amounts of type instantiations are executed to infer the type of your app. + +For example, suppose your app has a route like this: + +```ts +// app.ts +export const app = new Hono().get('foo/:id', (c) => + c.json({ ok: true }, 200) +) +``` + +Hono will infer the type as follows: + +```ts +export const app = Hono().get< + 'foo/:id', + 'foo/:id', + JSONRespondReturn<{ ok: boolean }, 200>, + BlankInput, + BlankEnv +>('foo/:id', (c) => c.json({ ok: true }, 200)) +``` + +This is a type instantiation for a single route. While the user doesn't need to write these type arguments manually, which is a good thing, it's known that type instantiation takes much time. `tsserver` used in your IDE does this time consuming task every time you use the app. If you have a lot of routes, this can slow down your IDE significantly. + +However, we have some tips to mitigate this issue. + +#### Compile your code before using it (recommended) + +`tsc` can do heavy tasks like type instantiation at compile time! Then, `tsserver` doesn't need to instantiate all the type arguments every time you use it. It will make your IDE a lot faster! + +Compiling your client including the server app gives you the best performance. Put the following code in your project: + +```ts +import { app } from './app' +import { hc } from 'hono/client' + +// this is a trick to calculate the type when compiling +const client = hc('') +export type Client = typeof client + +export const hcWithType = (...args: Parameters): Client => + hc(...args) +``` + +After compiling, you can use `hcWithType` instead of `hc` to get the client with the type already calculated. + +```ts +const client = hcWithType('http://localhost:8787/') +const res = await client.posts.$post({ + form: { + title: 'Hello', + body: 'Hono is a cool project', + }, +}) +``` + +If your project is a monorepo, this solution does fit well. Using a tool like [`turborepo`](https://turbo.build/repo/docs), you can easily separate the server project and the client project and get better integration managing dependencies between them. Here is [a working example](https://github.com/m-shaka/hono-rpc-perf-tips-example). + +If your client and server are in a single project, [project references](https://www.typescriptlang.org/docs/handbook/project-references.html) of `tsc` is a good option. + +You can also coordinate your build process manually with tools like `concurrently` or `npm-run-all`. + +#### Specify type arguments manually + +This is a bit cumbersome, but you can specify type arguments manually to avoid type instantiation. + +```ts +const app = new Hono().get<'foo/:id'>('foo/:id', (c) => + c.json({ ok: true }, 200) +) +``` + +Specifying just single type argument make a difference in performance, while it may take you a lot of time and effort if you have a lot of routes. + +#### Split your app and client into multiple files + +As described in [Using RPC with larger applications](#using-rpc-with-larger-applications), you can split your app into multiple apps. You can also create a client for each app: + +```ts +// authors-cli.ts +import { app as authorsApp } from './authors' +import { hc } from 'hono/client' + +const authorsClient = hc('/authors') + +// books-cli.ts +import { app as booksApp } from './books' +import { hc } from 'hono/client' + +const booksClient = hc('/books') +``` + +This way, `tsserver` doesn't need to instantiate types for all routes at once.