Address race condition in rpc timeout logic

[1egoman]

In RpcClientManager's performRpc method, there was some code roughly like this:

setTimeout(() => {
  clearTimeout(id);
}, /* arbitrary delay a */);

await variableDurationAsyncFunction();

const id = setTimeout(() => { /* ... */ }, /* arbitrary delay b */);

If variableDurationAsyncFunction takes longer to run than arbitrary delay a, then it's possible that clearTimeout(id) could run before id gets assigned. Fix this so that id is explictly hoisted and set to null, and then only actually cleared if it is non-null.

[changeset-bot]

🦋 Changeset detected

Latest commit: 83209e7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
livekit-client	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[github-actions]

size-limit report 📦

Path	Size
dist/livekit-client.esm.mjs	100.24 KB (-0.04% 🔽)
dist/livekit-client.umd.js	109.32 KB (-0.07% 🔽)