Runtime agnostic error handler

[cctidal]

Describe the feature

Currently you can specify an error handler for deno and bun by using runtime specific options (as described in https://srvx.unjs.io/guide/options). The nodejs runtime doesn't have a similar option.

// bun
serve({
  bun: {
    error(error) {
      return new Response(`<pre>${error}\n${error.stack}</pre>`, {
        headers: { "Content-Type": "text/html" },
      });
    },
  },
  fetch: () => new Response("👋 Hello there!"),
});

// deno
serve({
  deno: {
    onError(error) {
      return new Response(`<pre>${error}\n${error.stack}</pre>`, {
        headers: { "Content-Type": "text/html" },
      });
    },
  },
  fetch: () => new Response("👋 Hello there!"),
});

// nodejs
const server = serve({
  fetch(_request) {
    try {
      return new Response('👋 Hello there!');
    } catch (error) {
      return new Response(`<pre>${error}\n${error.stack}</pre>`, {
        headers: { 'Content-Type': 'text/html' },
      });
    }
  },
});

On nodejs we have to manually catch the error and return a response, but it's not as concise as the other runtime.

• Proposal 1: Add an onError or similar option to the serve api. This onError would be runtime agnostic and would work for deno, bun and nodejs.

serve({
  onError(error) {
    return new Response(`<pre>${error}\n${error.stack}</pre>`, {
      headers: { "Content-Type": "text/html" },
    });
  },
  fetch: () => new Response("👋 Hello there!"),
});

• Proposal 2: Add an error hook as a plugin

export const errorHandler: ServerPlugin = (server) => {
  return {
    name: 'error-handler',
    error(error) {
      return new Response(`<pre>${error}\n${error.stack}</pre>`, {
        headers: { 'Content-Type': 'text/html' },
      });
    },
  };
};

serve({
  plugins: [errorHandler],
  fetch: () => new Response("👋 Hello there!"),
});

Additional information

• Would you be willing to help implement this feature?

[pi0]

PR more than welcome! We can start from onError on top level serve

[danielpza]

Created a PR #48