[🐞]Maybe more friendly UX when useTask throws an error

[JerryWu1234]

Which component is affected?

Qwik Runtime

Describe the bug

useTask$(({ cleanup }) => {
    const handleKeyPress = (e) => {
      if (e.key === "`" && e.metaKey) {
        state.isOpen.value = !state.isOpen.value;
      }
      if (e.key === "Escape" && state.isOpen.value) {
        state.isOpen.value = false;
      }
    };
    const handleClickOutside = (e) => {
      if (state.isOpen.value && panelRef.value && !panelRef.value.contains(e.target)) {
        state.isOpen.value = false;
      }
    };
    if (!isBrowser) return;
    window.addEventListener("keydown", handleKeyPress);
    window.addEventListener("mousedown", handleClickOutside);
    cleanup(() => {
      window.removeEventListener("keydown", handleKeyPress);
      window.removeEventListener("mousedown", handleClickOutside);
    });
  });

if this useTask executes in server end, but which includes some client codes, and then the terminals will throws an error.
I'd better we could give a tips for user.
something likes "Please wrap the code with isBrowser to ensure it only runs on the client side."

Reproduction

https://qwik.new/

Steps to reproduce

No response

System Info

latest

Additional Information

No response

[wmertens]

Not a bad idea, maybe we can read the error and see if it mentions window or document and then show that message.

PR welcome!

[JerryWu1234]

Not a bad idea, maybe we can read the error and see if it mentions window or document and then show that message.

PR welcome!

@wmertens There is another situation: server-side code runs on the client side.

[wmertens]

@wmertens There is another situation: server-side code runs on the client side.

You mean like "process.x" errors? Yes, useful too but needs to have an isDev guard, we don't want to increase client build size

[JerryWu1234]

list I have to do.

1. I check server-side and client-side two scenarios.
2. add an isDev guard when checking "process.x" errors

[JerryWu1234]

#7571

done
related to this PR