feat(server$): config argument, optional GET, ServerError

[PatrickJS]

Overview

fixes #5656
fixes #6288

[cloudflare-pages]

Deploying qwik-docs with    Cloudflare Pages

Latest commit:	fac3aaa
Status:	✅  Deploy successful!
Preview URL:	https://4077d306.qwik-8nx.pages.dev
Branch Preview URL:	https://feat-server--options.qwik-8nx.pages.dev

View logs

[PatrickJS]

in another pr I can add 3 levels of config. I also want to do this for better AbortSignal

[DustinJSilk]

This is awesome! 🤩

Would it make sense to have ServerError take a generic argument so any serializable data can be returned?

export class ServerError<T = Record<any, any>> extends Error {
  constructor(
    public status: number,
    public data: T
  ) {
    super();
  }
}

[wmertens]

Looking forward to this!

[wmertens]

Misko is ok with this API extension

done

[PatrickJS]

import { server$ } from "@builder.io/qwik-city";
import { ServerError } from "@builder.io/qwik-city/middleware/request-handler";

type ResponseTuple = [null | string, string];

const serverFunctionA = server$(async function a(): Promise<ResponseTuple> {
  throw new ServerError<[string]>(401, ["my error"]);
});

const serverFunctionB = server$(async function b(): Promise<ResponseTuple> {
  return [null, this.method || ""];
});

export const MultipleServerFunctionsInvokedInTask = component$(() => {
  const methodA = useSignal("");
  const methodB = useSignal("");

  useVisibleTask$(async () => {
    const [error /*, data */] = await serverFunctionA();
    if (error) {
      methodA.value = error;
    }
    await delay(1);
    //     err, method
    const [, method] = await serverFunctionB();
    methodB.value = method;
  });

  return (
    <div id="server-error">
      {methodA.value}
      {methodB.value}
    </div>
  );
});