fix(client): Add Query Parameter Support to WebSocket Client in hono/client

[naporin0624]

This pull request adds support for query parameters to the WebSocket client in hono/client. The enhancement allows the WebSocket client to include query parameters in the URL, enabling more dynamic and flexible WebSocket connections.

Key Changes:

• Query Parameter Handling: The WebSocket client can now accept and append query parameters to the WebSocket URL, allowing for more versatile and customized WebSocket connections.

Updated Code:

if (method === 'ws') {
  const webSocketUrl = replaceUrlProtocol(
    opts.args[0] && opts.args[0].param ? replaceUrlParam(url, opts.args[0].param) : url,
    'ws'
  );
  const targetUrl = new URL(webSocketUrl);
  for (const key in opts.args[0].query) {
    targetUrl.searchParams.set(key, opts.args[0].query[key]);
  }
  return new WebSocket(targetUrl.toString());
}

Motivation:

This change was motivated by the need to allow the WebSocket client to handle query parameters, providing greater flexibility for developers to customize WebSocket connections based on specific requirements.

Impact:

The added functionality has been tested to ensure that query parameters are correctly appended to the WebSocket URL, enhancing the WebSocket client's capabilities in hono/client.

These updates will improve the usability of the WebSocket client in hono/client, making it more adaptable to various networking needs by supporting query parameters in WebSocket URLs.

The author should do the following, if applicable

• Add tests
• Run tests
• bun run format:fix && bun run lint:fix to format the code
• Add TSDoc/JSDoc to document the code

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.87%. Comparing base (3327baf) to head (3734af8).
Report is 20 commits behind head on main.

❗ Current head 3734af8 differs from pull request most recent head 9225a14

Please upload reports for the commit 9225a14 to get more accurate results.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3066      +/-   ##
==========================================
+ Coverage   94.75%   95.87%   +1.12%     
==========================================
  Files         136      137       +1     
  Lines       13339    13464     +125     
  Branches     2241     2271      +30     
==========================================
+ Hits        12639    12909     +270     
+ Misses        700      555     -145     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[yusukebe]

Can you remove this console.log?

[naporin0624]

Thank you for your comment. It has been fixed!!

[yusukebe]

Hi @naporin0624

Thank you for the PR. I've left a comment. If it is okay, I'll merge this into the main and will release it as a patch release.

[yusukebe]

LGTM!

[yusukebe]

@naporin0624

Cool! Merging now. Thanks.