fix: normalize WebSocket URI path to prevent /task/service duplication

String concatenation in _get_websocket_uri produced malformed URLs in
two cases:
- If the URI already contained /task/service, it was appended again
- A trailing slash produced a double slash before /task/service

Fix: strip trailing slash, check for existing /task/service suffix,
then build the final URL with urlunparse instead of string formatting.
Added two parametrized test cases covering both failure modes.

Resolves #755

Author: koushik1359

packages/client-python/src/rocketride/mixins/connection.py

@@ -314,8 +314,12 @@ def _get_websocket_uri(uri: str) -> str:
         parsed = urllib.parse.urlparse(normalized)
 
         ws_scheme = 'wss' if parsed.scheme in ('https', 'wss') else 'ws'
-        ws_uri = parsed._replace(scheme=ws_scheme)
-        return f'{ws_uri.geturl()}/task/service'
+
+        path = parsed.path.rstrip('/')
+        if not path.endswith('/task/service'):
+            path = path + '/task/service'
+
+        return urllib.parse.urlunparse((ws_scheme, parsed.netloc, path, '', '', ''))
 
     def _set_uri(self, uri: str) -> None:
         """Update the server URI (internal)."""

packages/client-python/tests/RocketRideClient_test.py

@@ -2370,6 +2370,10 @@ async def _check():
         ('https://cloud.rocketride.ai', 'wss://cloud.rocketride.ai/task/service'),
         ('ws://localhost:5565', 'ws://localhost:5565/task/service'),
         ('http://localhost:5565', 'ws://localhost:5565/task/service'),
+        # Path already contains /task/service - must not duplicate
+        ('wss://cloud.rocketride.ai/task/service', 'wss://cloud.rocketride.ai/task/service'),
+        # Trailing slash must not produce double slash
+        ('http://localhost:5565/', 'ws://localhost:5565/task/service'),
     ],
 )
 def test_get_websocket_uri_normalization(input_uri: str, expected_uri: str) -> None: