fix: added grace time to client timeout

[tegefaulkes]

Description

This PR fixes the following issues and addresses some general code clean up.

Issues Fixed

• Fixes #57
• Fixes #58

Tasks

• [x] 1. Address #57
• [x] 2. Address #58
• [x] 3. General clean up of code and tests
• [x] 4. Remove registerOnTimeoutCallback and onTimeoutCallback usage.

Final checklist

• [x] Domain specific tests
• [x] Full tests
• [x] Updated inline-comment documentation
• [x] Lint fixed
• [x] Squash and rebased
• [x] Sanity check the final build

[ghost]

👇 Click on the image for a new way to code review

#### [](https://app.codesee.io/r/reviews?pr=59&src=https%3A%2F%2Fgithub.com%2FMatrixAI%2Fjs-rpc) #### Legend

[tegefaulkes]

This should've never been merged into staging

    // Wait for server and client to timeout by checking the flag
    await new Promise<void>((resolve) => {
      const checkFlag = () => {
        if (serverTimedOut && clientTimedOut) resolve();
        else setTimeout(() => checkFlag(), 10);
      };
      checkFlag();
    });

I've removed it along with some other related code.

[tegefaulkes]

Ready for merge if anyone wants to have a look over it.

[tegefaulkes]

Most of the general clean up was just syntax and coding style fixes to keep things consistent. I added in some missing type annotation since it's better to be explicit about these things.