fix: exclude @remix-run/server-runtime from external deps to reduce bundle size

[OnurGvnc]

Hi!
zodix is a great work! thank you very much 🙏

with the current build configuration, the size of the dist/index.js file was 258kb because the @remix-run/server-runtime and its dependencies were included in the bundle file.

https://unpkg.com/browse/zodix@0.4.0/dist/

I have added the @remix-run/server-runtime package as an external package.

The file size has reduced to ~8kB.

❯ du -h dist/index.js 
8.0K    dist/index.js

[xmlking]

@remix-run/server-runtime also don't have to be in peer dependencies as it can be used with SvelteKit

[OnurGvnc]

@remix-run/server-runtime is used in two places: errors.ts and parser.ts.

It is used within errors.ts for the json helper method.

zodix/src/errors.ts

Lines 1 to 15 in f39de6c

	import { json } from '@remix-run/server-runtime';
	const DEFAULT_ERROR_MESSAGE = 'Bad Request';
	const DEFAULT_ERROR_STATUS = 400;
	export function createErrorResponse(
	options: {
	message?: string;
	status?: number;
	} = {}
	): Response {
	const statusText = options?.message || DEFAULT_ERROR_MESSAGE;
	const status = options?.status || DEFAULT_ERROR_STATUS;
	return json(statusText, { status, statusText });
	}

this dependency can be removed by making the above change.

---

In parser.ts it is used for type Params = LoaderArgs['params']

zodix/src/parsers.ts

Line 3 in f39de6c

import type { LoaderArgs } from '@remix-run/server-runtime';

zodix/src/parsers.ts

Line 12 in f39de6c

type Params = LoaderArgs['params'];

LoaderArgs['params'] -> Params comes from the package @remix-run/router. this dependency can also be removed.

// [@remix-run/router] react-router/packages/router/utils.ts#L234
export type Params<Key extends string = string> = {
  readonly [key in Key]: string | undefined;
};

• https://github.com/remix-run/remix/blob/1bdeebf04d72f41dda8952af0aa7f8a786d6aa1a/packages/remix-server-runtime/routeModules.ts#L17
• https://github.com/remix-run/remix/blob/1bdeebf04d72f41dda8952af0aa7f8a786d6aa1a/packages/remix-server-runtime/routeModules.ts#L2
• https://github.com/remix-run/react-router/blob/cc9a5c41c27c011cfcc4055a8571de285b5d939d/packages/router/utils.ts#L234-L236C3

[xmlking]

Just a thought , can we make error transformation optional ?
Since zod has builtin support for flatten and other customization? https://github.com/colinhacks/zod/blob/master/ERROR_HANDLING.md
this lib has potential to be used by multiple frameworks

[rileytomasek]

@OnurGvnc: thanks for this. It won't impact runtime bundle size, but it should be external because it's required by Remix.

I'm not willing to rewrite part of @remix-run/server-runtime to drop the dependency because it's required by Remix, and this library is for Remix.

[xHomu]

Oh wow, thanks for fix!